fix(spec): hook condition row declares expression, not javascript (#20475)

Fixes #20439

Clause-②: no

## What

`packages/spec/src/data/hook.form.ts`'s `condition` row declared
`language: 'javascript'` over `HookSchema.condition`, which is
`EvaluatedExpressionInputSchema` — a CEL predicate, not a script
(`hook.zod.ts`: `'Predicate (CEL); hook runs only when TRUE …'`). Every
sibling predicate row (`field.form.ts` / `object.form.ts`'s
`visibleWhen` / `readonlyWhen` / `requiredWhen`, and the formula
`expression` row) already declares `language: 'expression'`. A consumer
keyed on the row's declared language (objectui#10963's `CodeWidget` fix)
could not tell this row apart from a real script row (`body.source`,
`action.source`, both genuinely `'javascript'`).

- `condition` row now declares `language: 'expression'`, `helpText: 'CEL
predicate — the hook runs only when TRUE'` (triage's wording, matching
the sibling rows' phrasing).
- Pinned in
`packages/spec/src/system/metadata-form-declared-rows.pin.test.ts` (no
existing row-language assertion was found, so the pin was added beside
that file's existing form-row pins): asserts `hook.condition` declares
`type: 'code'` / `language: 'expression'`, with a control against
`field.visibleWhen` (a sibling predicate row already correct) so the
assertion is shown to actually discriminate.
- `en.metadata-forms.generated.ts` regenerated (`node
scripts/check-i18n-bundles.mjs --write`) to pick up the new helpText.
`zh-CN` / `ja-JP` / `es-ES` kept their existing translated values under
the tool's merge behaviour (now stale relative to the new English source
— the tool does not auto-translate).
- Changeset: `@objectstack/spec` patch.

## Why this landing site

Matches the constraint's expected surface exactly: `hook.form.ts`, the
pin, the regenerated translation bundles, and the changeset. No other
producer or consumer needed a change.

## Tests

- `pnpm --filter @objectstack/spec exec vitest run --maxWorkers=2
src/system/metadata-form-declared-rows.pin.test.ts
src/system/metadata-form-zod-reconciliation.test.ts
src/data/hook.test.ts src/data/hook-body.test.ts` — 173 passed.
- `pnpm --filter @objectstack/platform-objects exec vitest run
--maxWorkers=2 src/apps/translations` — 399 passed (21 files), including
`metadata-forms-vocabulary.test.ts` and
`hook-execution-panel-echo-decisions.test.ts`.
- `pnpm --filter @objectstack/spec build && pnpm --filter
@objectstack/spec check:generated` — all 15 generated artifacts up to
date.
- `pnpm check:i18n` — all 9 packages in sync, no undeclared authoring
keys.
- `pnpm --filter @objectstack/spec typecheck` — clean.
- Gate derivation: `node scripts/pm/dispatch-gates.mjs --repo
objectstack-ai/objectstack --commands` → 85 derived families, all run
and reconciled (`--ran`): 84 green, 1 honestly NOT-MEASURED — `pnpm
check:dual-build-cjs-loads` exits its own `PREREQUISITE NOT MET` (3)
because this worktree never ran a full `pnpm build` across every package
in the monorepo (studio, client-react, several
connectors/plugins/services have no `dist/`); that whole-workspace build
is disproportionate to this one-row change and is CI's own "Build Core"
job to run. Nothing about this diff is implicated in that gate.

## Acceptance notes

None — no out-of-scope findings surfaced while making this change.

---
_Generated by [Claude
Code](https://claude.ai/code/session_014EJ1ED8X4MMrT18BhVx4tx)_

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: objectstack-fleet[bot]

.changeset/20439-hook-condition-expression-row.md

@@ -0,0 +1,11 @@
+---
+'@objectstack/spec': patch
+---
+
+`hook.form.ts`'s `condition` row now declares `language: 'expression'`, matching the CEL predicate `HookSchema.condition` actually is.
+
+Clause-②: no
+
+The row previously declared `language: 'javascript'` — the same declared language as a real script row (`body.source`) — over a field that is `EvaluatedExpressionInputSchema`, a CEL predicate. A consumer keyed on the row's declared language could not tell the predicate apart from a script. The `helpText` moves from "Optional formula — skip the hook when this evaluates to false" to "CEL predicate — the hook runs only when TRUE", matching the phrasing every sibling predicate row (`field.form.ts` / `object.form.ts`'s `visibleWhen` / `readonlyWhen` / `requiredWhen`, and the formula `expression` row) already uses.
+
+No key is added, removed, narrowed or widened, and no parse verdict changes — `type: 'code'` and `language` are already-declared form-DSL vocabulary. `metadata-form-declared-rows.pin.test.ts` pins the new value, with a control against a sibling predicate row.

packages/platform-objects/src/apps/translations/en.metadata-forms.generated.ts

@@ -903,7 +903,7 @@ export const enMetadataForms: NonNullable<TranslationData['metadataForms']> = {
       },
       condition: {
         label: "Condition",
-        helpText: "Optional formula — skip the hook when this evaluates to false"
+        helpText: "CEL predicate — the hook runs only when TRUE"
       },
       retryPolicy: {
         label: "Retry Policy",

packages/spec/src/data/hook.form.ts

@@ -74,7 +74,7 @@ export const hookForm = defineForm({
           { label: 'System (elevated)', value: 'system' },
           { label: 'User (triggering user)', value: 'user' },
         ] },
-        { field: 'condition', type: 'code', language: 'javascript', colSpan: 2, helpText: 'Optional formula — skip the hook when this evaluates to false' },
+        { field: 'condition', type: 'code', language: 'expression', colSpan: 2, helpText: 'CEL predicate — the hook runs only when TRUE' },
         {
           field: 'retryPolicy',
           type: 'composite',

packages/spec/src/system/metadata-form-declared-rows.pin.test.ts

@@ -48,6 +48,7 @@ type FormEntry = {
   field?: string;
   widget?: string;
   type?: string;
+  language?: string;
   helpText?: string;
   visibleWhen?: string | { dialect?: string; source?: string };
   fields?: FormEntry[];
@@ -168,3 +169,36 @@ describe('declared keys that now have a form row (#19085)', () => {
     expect(row.helpText).toContain('"name"');
   });
 });
+
+/**
+ * **A `type: 'code'` row's `language` must name the slot it actually edits**
+ * (#20439).
+ *
+ * `hook.form.ts`'s `condition` row declared `language: 'javascript'` over
+ * `HookSchema.condition`, which is `EvaluatedExpressionInputSchema` — a CEL
+ * predicate, not a script. Every sibling predicate row (`field.form.ts` /
+ * `object.form.ts`'s `visibleWhen` / `readonlyWhen` / `requiredWhen`, and the
+ * formula `expression` row) already declares `language: 'expression'`; a
+ * consumer keyed on the declared language (objectui#10963's `CodeWidget`
+ * fix) cannot otherwise tell this row apart from a real script row
+ * (`body.source`, `action.source`, both genuinely `'javascript'`).
+ */
+describe('hook.condition declares the CEL predicate language it edits (#20439)', () => {
+  it('hook.condition is a `code` row declaring language: expression, not javascript', () => {
+    const [row, ...extra] = rowFor('hook', 'condition');
+    expect(row, '`condition` is declared by HookSchema; the hook form must offer it').toBeDefined();
+    expect(extra, 'one row only').toEqual([]);
+    expect(row.type).toBe('code');
+    expect(row.language).toBe('expression');
+  });
+
+  it('CONTROL: a sibling predicate row (field.visibleWhen) also declares expression', () => {
+    // Proves the assertion above actually discriminates: run the identical
+    // probe against a row known to already be correct, so a helper that
+    // stopped reading `language` at all would fail here rather than passing
+    // hook.condition vacuously.
+    const [row] = rowFor('field', 'visibleWhen');
+    expect(row.type).toBe('code');
+    expect(row.language).toBe('expression');
+  });
+});