fix(contracts): anchor the predicate double-cast annotation to the cast

`check:api-validation:strict` counted 9 unannotated double-casts against a
baseline of 8, failing CI. The predicate leaf schema was annotated, but the
annotation sat above the declaration while the checker anchors on the line
carrying the cast — five lines below, at the close of the object literal. The
scanner walks back at most three lines and stops at the first non-comment one,
so it hit `value: z.unknown().optional(),` and never saw the reason.

Splitting the object schema from the cast puts them adjacent, so the existing
reason binds. No behavior change — the cast, the schema, and the reasoning are
unchanged.

Also lowers the rawJsonReads ratchet 6 -> 5 to match the current count, which
had drifted down; leaving it high lets a removed raw read silently come back.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: TheodoreSpeaks

apps/sim/lib/api/contracts/tables.ts

@@ -477,14 +477,16 @@ function predicateTreeTooLarge(root: unknown): string | null {
  * before it ran. Strict on BOTH branches is required: strict on the group alone
  * would just fall through to the leaf branch, which is the more dangerous reading.
  */
-// double-cast-allowed: `z.unknown()` keeps the runtime permissive (a leaf value
-// is arbitrary JSON), but infers `unknown`, which is wider than
-// `Predicate['value']`. The narrowing is type-level only — nothing is coerced.
-const predicateLeafSchema = z.strictObject({
+const predicateLeafObjectSchema = z.strictObject({
   field: z.string().min(1, 'field is required').max(128),
   op: z.enum(FILTER_OPS),
   value: z.unknown().optional(),
-}) as unknown as z.ZodType<Predicate>
+})
+
+// double-cast-allowed: `z.unknown()` keeps the runtime permissive (a leaf value
+// is arbitrary JSON), but infers `unknown`, which is wider than
+// `Predicate['value']`. The narrowing is type-level only — nothing is coerced.
+const predicateLeafSchema = predicateLeafObjectSchema as unknown as z.ZodType<Predicate>
 
 const predicateNodeSchema: z.ZodType<PredicateNode> = z.lazy(() =>
   z.union([predicateGroupSchema, predicateLeafSchema])

scripts/check-api-validation-contracts.ts

@@ -25,7 +25,7 @@ const BOUNDARY_POLICY_BASELINE = {
   clientHookRawFetches: 0,
   clientSameOriginApiFetches: 0,
   doubleCasts: 8,
-  rawJsonReads: 6,
+  rawJsonReads: 5,
   untypedResponses: 0,
   annotationsMissingReason: 0,
 } as const