test(spec): automation test titles state each cited decision in words instead of a tracker number (stage 10) (#21713)

Part of #20749
Clause-②: no

Stage 10 of this card, and the first area of class (e): the test strings
shipped under `packages/spec/src`, as ruled in `5902360492` on #20513.
This stage takes the whole `automation/` directory. Its 105 test-title
and test-message literals carried 110 tracker ids citing 55 records.
Each id now either states what its record decided, in words (form D), or
is dropped where the title already says it. Text only: no assertion,
fixture value, test count or code comment changes.

## Census at the base (`7e0066af7a`, the claim's base)

Instrument: stage 9's `census.cjs` (md5
`6e42a45a926d375013c32d62f16a296e`, byte-identical), plus one added
classification pass. A literal counts as a test title when its folded
message is argument 0 of a `describe` / `it` / `test` call, `.each` /
`.skip` / `.only` chains included. Everything else is an "other" string.

Reference: the same instrument reads **1803 messages / 1919 ids in 425
files at `9b8c7f38d7`**, stage 9's reading exactly. Since then, #21699
added 1 / 1 (`ui/component-props-unknown-members.pin.test.ts:143`) and
#21700 moved 3 / 3 (the two reader literals it re-anchored), which gives
1801 / 1917 at the base.

| directory | files | titles msg / ids | other msg / ids | total msg /
ids | excluded files |
|:--|--:|--:|--:|--:|:--|
| `data/` | 95 | 445 / 475 | 23 / 26 | 468 / 501 | |
| `ui/` | 81 | 374 / 397 | 18 / 18 | 392 / 415 | `report.test.ts` 3 / 3
|
| `api/` | 40 | 181 / 193 | 8 / 8 | 189 / 201 | |
| `system/` | 34 | 128 / 138 | 26 / 27 | 154 / 165 | `job.test.ts` 4 / 4
|
| (files directly in `src/`) | 30 | 117 / 119 | 1 / 1 | 118 / 120 | |
| **`automation/`** (this PR) | 21 | 101 / 106 | 4 / 4 | **105 / 110** |
|
| `kernel/` | 36 | 92 / 96 | 10 / 10 | 102 / 106 | |
| `shared/` | 21 | 73 / 81 | 12 / 14 | 85 / 95 | |
| `contracts/` | 25 | 59 / 70 | 4 / 4 | 63 / 74 | |
| `conversions/` | 9 | 34 / 34 | 0 | 34 / 34 | |
| `security/` | 8 | 28 / 28 | 0 | 28 / 28 | |
| `ai/` | 9 | 13 / 15 | 5 / 5 | 18 / 20 | |
| `identity/` | 6 | 14 / 14 | 1 / 1 | 15 / 15 | |
| `integration/` | 4 | 13 / 13 | 1 / 1 | 14 / 14 | |
| `migrations/` | 2 | 9 / 12 | 0 | 9 / 12 | |
| `marketplace/`, `meta-spelling/`, `studio/` | 5 | 7 / 7 | 0 | 7 / 7 |
|
| **total** | **426** | **1688 / 1798** | **113 / 119** | **1801 /
1917** | 7 / 7 |

- **Excluded, in flight under this seat:** `system/job.test.ts` carries
#16292 (`:92`), #14478 (`:471`), #4667 (`:836`) and #19184 (`:881`), and
`ui/report.test.ts` carries #20161 (`:233`), #3916 (`:334`) and #5013
(`:426`). All seven sit in titles. They wait for a later stage, after
#21703 and #21702.
- **Controls.** Lit, single line: `automation/approval.test.ts:135`
reads one title with #3508. Lit, multi-line:
`api/discovery-environment-subset.pin.test.ts:63-66`, a `+` chain, reads
as ONE message with its id on `:65`. Dark: the `// #3508` comment at
`automation/approval.test.ts:131` reads 0. Planted in a scratch copy of
the head file: an id in a title reads 1 / 1, and an id in a comment
reads 0.
- **A wider pattern** (any `#` plus digits, so two-digit and six-digit
numbers too) reads the same 105 / 110 in `automation/` at the base, and
0 / 0 at the head.
- **At the head:** 1696 messages / 1807 ids in 405 files. `automation/`
reads 0 / 0. Nothing else moved.

## How the area was chosen

The directories are ranked by id count, and a stage takes whole
directories up to about 100 ids. The four busiest each exceed that bound
alone: `data/` (501), `ui/` (415), `api/` (201) and `system/` (165). The
files directly in `src/` (120) are 20% over. `automation/` (110) is the
busiest whole directory within about 10% of the bound, so it is this
stage. The rule picked it before any card was read.

Its 55 records (53 in this repository, 2 in objectui) were all readable
in one pass. 54 answer 200. #6362 answers 404, and its decision was read
from its landing commit `b5404f496`.

**Named for the next stages** (by directory, from the table): `data/`
(about five stages, by subdirectory or file group; `data/driver/` alone
is 52), `ui/` (about four), `api/` (two), `system/` (two), the files
directly in `src/` (one), `kernel/` (one), `shared/` (one), `contracts/`
with `conversions/` (one, 108), and `security/`, `ai/`, `identity/`,
`integration/`, `migrations/`, `marketplace/`, `meta-spelling/` and
`studio/` together (one, 96). The seven excluded ids join `system/` and
`ui/` once their owners land.

## What each id became

38 literals (40 ids) now state a decision in words. 67 literals (70 ids)
drop a citation the title already explains. Each record was read with
its comments through REST, and where a record has no comments, from what
landed.

| record | ids | result |
|:--|--:|:--|
| #3508 | 2 | `APPROVER_VALUE_BINDINGS`: "an approver value is picked
from the records the engine resolves". `APPROVER_VALUE_SOURCES` (the
follow-up): "where each picker finds its candidates, published on the
wire". |
| objectui#2955 | 1 | "for the decision dialog to render and enforce":
both decision entry points collect the typed outputs, and `required` is
enforced. |
| #3810 | 1 | "names the match-everything-write hazard": a filter
emptied by interpolation matches every row, and the node is refused
instead. The test pins the words `match-everything write`. |
| #4001 | 13 | "strict as of #4001 批 9" becomes "an unknown key is
refused, not stripped" (5 titles). Also "refused, not stripped", "an
unknown key is refused, per shape" and "the unknown-key gate". Dropped
from 5 titles that already read "unknown keys are rejected, not
stripped". |
| objectui#2670 | 1 | "so the flow designer renders it as a template":
the designer's loop and region rendering reads this marker. |
| #4396 | 2 | "a function that writes says so; pure is the default", and
"the authoring surface where a function declares its effect" (landed
`eb4204b`). |
| #4697 | 2 | `defaultValue`: "a declared variable is bound on every
path" (ruling A). Dropped once. |
| #3896 | 3 | "outputSchema retired: declared, never validated" (the
audit close-out's reason, as `flow.zod.ts`'s tombstone records it).
Dropped twice. |
| #4247 | 1 | "maxRetries — one default, and no zero-attempt “retry”"
(landed `a648e96`). |
| #16134, #15713 | 7 | "a region node reusing a top-level id is refused
— one node-id space now spans every region" (ruling, batch 61). Dropped
from 5 titles that state uniqueness. |
| #9205 | 5 | "template reference — notify content localized through an
email template" (the ruled emailTemplates route). Three titles say "the
template path" where they said "pre-#9205". Dropped once. |
| #7086 | 1 | "severity — the closed info \| warning \| critical
vocabulary" (the enum route). |
| #4415 | 3 | "FlowNodeSchema parses its own regions" (ruling
2026-08-07, direction 1). Dropped from 2 titles that already say it. |
| #4347 | 1 | "collectFlowGraphs — every region is walked, not only the
top level" (landed `31e0be9`). |
| #4401 | 2 | "FLOW_REGION_SLOTS, the one declaration of where regions
live" (landed `4bfd455`). Dropped once. |
| #4414 | 1 | "`condition` is pointed at the out-edges, NOT given the
one-edit rename" (one working routing model, landed `5293114`). |
| #15429 | 1 | "taking every true branch must be declared" (ruling item
2: explicit `inclusive`). |
| #14149 | 3 | "every entry older than the value role" (ruling A: a
value-role CEL slot on `assignment`). Dropped twice. |
| #19938 | 4 | "the CRUD `fields.*` value slots added exactly two rows".
Dropped three times. |
| #15572 | 3 | "predicateSlotRefusal — a predicate slot holds bare CEL
text", and "the other two doors refuse it". Dropped once. |
| #15662 | 1 | "structuralConditionRefusal — a structural condition is
CEL text or an expression". |
| #15792, #15807 | 4 | "REFUSES an `ast`-only envelope — admitted at
first, refused once an evaluated slot required a `source`". Dropped from
2 titles that state the rule. |
| #17493 | 3 | Table row "a blank string — blanks are refused" (ruling
A). Dropped twice. |
| #14945 | 4 | "EndConfigSchema — the `end` node contract: it may refuse
the run with a message" (ruling 2′). Dropped three times. |
| #15617 | 4 | "`failed` is the fold INCLUDING what a delegating node
rolled up from its child — it answers what the run caused" (ruling
option 1). Dropped three times. |
| dropped only | 36 | #3196, #3266, #4158, #4277, #4343 (2), #4389,
#4525, #4738, #4964 (2), #6758, #7085, #9106, #12278, #14964, #15430,
#15646 (3), #16752, #17306, #17852, #18102, #18112 (2), #18847, #19151,
#19961 (4), #20316 (2): each title already states the pinned decision.
For #4988 and #6414, the two expect messages already say what was
retired. |
| #6362 (404) | 1 | Dropped. The title "PRESERVES all seven envelope
keys — measured, not assumed" carries the decision recorded in
`b5404f496` (`webhook` was measured, and all seven keys survive). |

## Readers

- **Test-name filters:** none. A tracked-tree search for `-t` and
`--testNamePattern` finds only `packages/qa/dogfood/README.md:142` (`-t
"owner-scoped"`), which is unrelated.
- **Snapshots:** none. `automation/` has no `__snapshots__` and no
`toMatchSnapshot`.
- **Titles by substring:** every old title, plus a window around each id
(250 needles), was searched across the tracked tree outside its own
file. No gate, doc or script matches one. The hits are other files' own
titles with the same words: `identity/`, `security/` and `automation/`
siblings, a later stage's lot. There are also two code comments in
`flow.zod.ts` and `flow-function.zod.ts`, which belong to the comment
lane.
- **Twin tables, not readers:**
`packages/lint/src/validate-expressions.test.ts:4442/4445` and
`packages/services/service-automation/src/decision-branch-expression-absent.test.ts:61/64`
repeat the `flow-decision-branch-expression-absent` table's row names.
Nothing compares them mechanically: the "same table" parity is prose in
the headers, and it covers assertions, not names. They are outside this
stage's surface (see Acceptance notes).

## Text-only proof

A scratch tool (`textonly10.cjs`) compares base and head file by file on
three legs:
1. **Skeleton:** the full AST, with string pieces masked. It must be
identical.
2. **Comments:** every comment, byte-equal.
3. **Strings:** each string leaf that changed must sit in a test-call
title position, or on one of the 4 declared lines
(`flow-decision-branch-expression-absent.test.ts:63` and `:66`, table
`name` values that feed `$name` titles; `sync-retirement.test.ts:120`
and `:158`, expect messages). It must carry a tracker id before and no
`#` plus digits after.

- **Result:** 21 of 21 files SAME, 105 changed (101 title, 4 declared),
on all three legs.
- **Diff hunks:** exactly the 105 planned lines, with every file keeping
its line count.
- **Controls (10 of 10 as predicted, on scratch copies, each anchor hit
once):** identifier rename DIFF; numeric literal DIFF; comment edit
COMMENT DIFF; a non-title string with an id VIOLATION; a rewritten title
given a new id VIOLATION; a title that was id-free at base edited
VIOLATION; one title reverted to base SAME (104 changed); a declared
string keeping an id VIOLATION; an undeclared expect message changed
VIOLATION; a title re-split into a `+` chain DIFF.

**Test counts:** the 21 files run at the base (in a separate base
worktree) and at the head: 801 / 801 tests on both sides, with the same
count and status sequence per file in 21 of 21. 560 full test names
change, and each equals the base name with the planned replacements
applied.

## Changeset: `skip-changeset`

Measured, not assumed:
- `npm pack --dry-run` of `@objectstack/spec` lists 2068 files, under
`files[]` (`dist`, `src/**/*.zod.ts` and the rest). 0 of the 21 touched
files are in it, and 0 `*.test.ts` at all. The control
`src/automation/flow.zod.ts` is in it.
- In `dist/`, three new phrases and three old ones each read in 0 files.
The control `A predicate slot holds BARE CEL TEXT` reads in 2.

So this PR publishes nothing, and no changeset is added.

## Verification (at `f3dc3fab03`)

- `pnpm turbo run build` over all packages: 71 / 71.
- `@objectstack/spec`: `vitest run --project local`, 612 files and 18185
passed, 1 todo. `typecheck` exit 0, including `check:test-typecheck`,
whose program holds all 21 touched files.
- **Gates:** `dispatch-gates --commands` derived 79 families, and all 79
exit 0. `--ran` reconciles: 79 derived, 79 run, 0 NOT-MEASURED, 0 UNRUN.
- **ESLint, a proven narrowing:** `--no-inline-config` over the 21
files, 0 errors and 0 warnings. The population comes from ESLint's own
config: 21 configured, 0 ignored. No `parserOptions.project` or
`projectService`, so no untouched file's verdict can move.
- `check-governed-merges --test`: NOT governed, 210 changed lines.

## Acceptance notes

- **Twin row names in other packages:** the lint and service-automation
copies of the decision-branch table keep their `the #19961 shape` and
`the #17493 control` row names, and their twin `describe` titles keep
their ids. The lint copy is this card's own later share (the
`packages/lint` test strings). The service-automation copy belongs to
that package's lane.
- **Code comments still carry ids** in these 21 files (for example
`approval.test.ts:56`, `:69` and `:131`). They are the comment lane's,
untouched here.
- **`origin/main` moved** three commits past the base before this PR
opened (#21701, #21707, #21706). None touches `packages/spec`, so
nothing was merged.

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

---------

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

Author: objectstack-fleet[bot]

packages/spec/src/automation/approval.test.ts

@@ -132,7 +132,7 @@ describe('ORG_MEMBERSHIP_LEVELS derives from BUILTIN_MEMBERSHIP_ROLES', () => {
 // record-backed kinds MUST match the engine's resolution semantics
 // (`plugin-approvals` resolveApproverSpec / expand*Users) — these assertions
 // pin the object names and stored fields the engine actually queries.
-describe('APPROVER_VALUE_BINDINGS (#3508)', () => {
+describe('APPROVER_VALUE_BINDINGS — an approver value is picked from the records the engine resolves', () => {
   it('covers every ApproverType member', () => {
     for (const t of ApproverType.options) {
       expect(APPROVER_VALUE_BINDINGS[t]).toBeDefined();
@@ -173,7 +173,7 @@ describe('APPROVER_VALUE_BINDINGS (#3508)', () => {
 // #3508 follow-up: `xRef.map` names a picker KIND but never said where that
 // picker's candidates live — which is how the designer came to query the
 // metadata registry for data records. The data contract now ships on the wire.
-describe('APPROVER_VALUE_SOURCES (#3508 follow-up)', () => {
+describe('APPROVER_VALUE_SOURCES — where each picker finds its candidates, published on the wire', () => {
   it('covers every ApproverType member, exactly like the bindings it projects', () => {
     for (const t of ApproverType.options) {
       expect(APPROVER_VALUE_SOURCES[t], `no source published for '${t}'`).toBeDefined();
@@ -309,7 +309,7 @@ describe('ApprovalNodeConfigSchema', () => {
     expect(() => ApprovalNodeConfigSchema.parse({ ...minimal, behavior: 'weighted' })).toThrow();
   });
 
-  it('accepts quorum / per_group behaviors with minApprovals and grouped approvers (#3266)', () => {
+  it('accepts quorum / per_group behaviors with minApprovals and grouped approvers', () => {
     const quorum = ApprovalNodeConfigSchema.parse({ ...minimal, behavior: 'quorum', minApprovals: 2 });
     expect(quorum.behavior).toBe('quorum');
     expect(quorum.minApprovals).toBe(2);
@@ -486,7 +486,7 @@ describe('ApprovalEscalationSchema', () => {
   // escalation block exists at all; within a block carrying timeoutHours,
   // escalation is ON unless explicitly turned off. Declared in
   // DEFAULT_CHANGES_BY_MAJOR (17) — this pin is what keeps the fingerprint honest.
-  it('defaults enabled to true — a block carrying timeoutHours is live unless explicitly off (#12278)', () => {
+  it('defaults enabled to true — a block carrying timeoutHours is live unless explicitly off', () => {
     const omitted = ApprovalEscalationSchema.parse({ timeoutHours: 24 });
     expect(omitted.enabled).toBe(true);
     const explicit = ApprovalEscalationSchema.parse({ enabled: false, timeoutHours: 24 });
@@ -512,7 +512,7 @@ describe('normalizeDecisionOutputs', () => {
     ]);
   });
 
-  it('carries the widget hints and the required flag through (objectui#2955)', () => {
+  it('carries the widget hints and the required flag through, for the decision dialog to render and enforce', () => {
     expect(normalizeDecisionOutputs([
       { key: 'positions', label: 'Co-signers', type: 'position', multiple: true, required: true },
     ])).toEqual([
@@ -540,7 +540,7 @@ describe('normalizeDecisionOutputs', () => {
 // a v17-new surface, tightened before stored volume exists. The published JSON
 // schema now carries additionalProperties:false into the Studio form AND
 // registerFlow()'s per-node config validation (#4027/#4040) — asserted below.
-describe('unknown keys are rejected, not stripped (#4001)', () => {
+describe('unknown keys are rejected, not stripped', () => {
   const unknownKeyIssue = (schema: { safeParse: (v: unknown) => any }, value: unknown) => {
     const result = schema.safeParse(value);
     expect(result.success).toBe(false);
@@ -607,7 +607,7 @@ describe('unknown keys are rejected, not stripped (#4001)', () => {
      * cases fail loudly instead, which is what "declared = enforced" needs on
      * the declaring side.
      */
-    it('accepts `required` — the key the runtime enforces (#4525)', () => {
+    it('accepts `required` — the key the runtime enforces', () => {
       expect(DecisionOutputDefSchema.parse({ key: 'next_reviewers', required: true }))
         .toEqual({ key: 'next_reviewers', required: true });
       expect(DecisionOutputDefSchema.parse({ key: 'note', required: false }))

packages/spec/src/automation/builtin-node-config.test.ts

@@ -52,7 +52,7 @@ function unknownKeyMessage(schema: Parseable, value: unknown): string | undefine
   return result.error!.issues.find((i) => i.code === 'unrecognized_keys')?.message;
 }
 
-describe('CRUD config contracts — strict as of #4001 批 9', () => {
+describe('CRUD config contracts — an unknown key is refused, not stripped', () => {
   it('accepts every declared key on each of the four', () => {
     expect(GetRecordConfigSchema.parse({
       objectName: 'lead', filter: { status: 'new' }, fields: ['id'], limit: 5, outputVariable: 'leads',
@@ -85,7 +85,7 @@ describe('CRUD config contracts — strict as of #4001 批 9', () => {
     ['update_record', UpdateRecordConfigSchema, { objectName: 'lead' }],
     ['delete_record', DeleteRecordConfigSchema, { objectName: 'lead' }],
   ] as ReadonlyArray<[string, Parseable, Record<string, unknown>]>)(
-    '%s: prescribes `filter` for the retired `filters` spelling, and names the #3810 hazard',
+    '%s: prescribes `filter` for the retired `filters` spelling, and names the match-everything-write hazard',
     (_nodeType, schema, base) => {
       const message = unknownKeyMessage(schema, { ...base, filters: { status: 'stale' } })!;
       expect(message).toContain('flow-node-crud-filter-alias');
@@ -204,7 +204,7 @@ describe('CRUD config contracts — strict as of #4001 批 9', () => {
   });
 });
 
-describe('ScreenConfigSchema / ScreenFieldConfigSchema — strict as of #4001 批 9', () => {
+describe('ScreenConfigSchema / ScreenFieldConfigSchema — an unknown key is refused, not stripped', () => {
   it('accepts the flat and object-form shapes in full', () => {
     expect(ScreenConfigSchema.safeParse({
       title: 'Details', description: 'Fill this in', waitForInput: true,
@@ -249,7 +249,7 @@ describe('ScreenConfigSchema / ScreenFieldConfigSchema — strict as of #4001 
   });
 });
 
-describe('ScreenFieldConfigSchema — the bound pair, help text and lookup target (#17306)', () => {
+describe('ScreenFieldConfigSchema — the bound pair, help text and lookup target', () => {
   const BASE = { name: 'discount', type: 'number' };
 
   // ── Direction 1: the three intents are now expressible ──────────────────
@@ -397,7 +397,7 @@ describe('ScreenFieldConfigSchema — the bound pair, help text and lookup targe
   });
 });
 
-describe('MapConfigSchema — strict as of #4001 批 9', () => {
+describe('MapConfigSchema — an unknown key is refused, not stripped', () => {
   it('accepts every declared key', () => {
     expect(MapConfigSchema.parse({
       collection: '{tasks}', flowName: 'one_task_signoff', iteratorVariable: 'item',
@@ -430,7 +430,7 @@ describe('MapConfigSchema — strict as of #4001 批 9', () => {
 
 // ─── assignment (#14149) ─────────────────────────────────────────────
 
-describe('assignment value contract — a CEL envelope beside `{token}` interpolation (#14149)', () => {
+describe('assignment value contract — a CEL envelope beside `{token}` interpolation', () => {
   const DIGEST_SOURCE = 'joinNonEmpty(overdue_tasks.map(t, t.subject), "\\n")';
   const DIGEST_ENVELOPE = { dialect: 'cel', source: DIGEST_SOURCE };
 
@@ -568,7 +568,7 @@ describe('assignment value contract — a CEL envelope beside `{token}` interpol
  * registered, and faulted at run time (the `ast`-only one with the engine's
  * own prescription; the blank one with a parse error on `"   "`).
  */
-describe('assignment value envelope — an evaluated slot requires what the engine can evaluate (#15430)', () => {
+describe('assignment value envelope — an evaluated slot requires what the engine can evaluate', () => {
   const AST_ONLY = { dialect: 'cel', ast: { kind: 'const', value: 1 } };
   const BLANK_SOURCE = { dialect: 'cel', source: '   ' };
 
@@ -627,7 +627,7 @@ describe('assignment value envelope — an evaluated slot requires what the engi
  * same structural reason as the sibling slot: `z.record()`'s open-key branch
  * skips it before any key schema — including `.min(1)` — ever runs.
  */
-describe('AssignmentConfigSchema.assignments — __proto__ pre-parse guard, constructor/prototype UNCHANGED (#17852 / #18847)', () => {
+describe('AssignmentConfigSchema.assignments — __proto__ pre-parse guard, constructor/prototype UNCHANGED', () => {
   it('refuses `assignments` carrying a `__proto__` own key, named at `assignments.__proto__`', () => {
     // `JSON.parse` is what makes `__proto__` an OWN enumerable key — an
     // object literal's `{ __proto__: ... }` sets the actual prototype
@@ -688,7 +688,7 @@ describe('AssignmentConfigSchema.assignments — __proto__ pre-parse guard, cons
  * `__proto__` must be refused LOUDLY — told apart here from the two silent
  * outcomes it could otherwise have (silently dropped, silently kept).
  */
-describe('AssignmentConfigSchema — top-level __proto__ refused at the catchall (#19151)', () => {
+describe('AssignmentConfigSchema — top-level __proto__ refused at the catchall', () => {
   /**
    * The three outcomes an authored key can meet, discriminated by one
    * function so a test cannot accidentally assert the wrong one. A bare
@@ -807,7 +807,7 @@ describe('AssignmentConfigSchema — top-level __proto__ refused at the catchall
  * CEL value envelope), the edge #14149 accepted on `assignments.*`. Everything
  * else parses exactly as before.
  */
-describe('CRUD `fields` value contract — the CEL value envelope beside `{token}` templates (#19938)', () => {
+describe('CRUD `fields` value contract — the CEL value envelope beside `{token}` templates', () => {
   const PRICE_ENVELOPE = { dialect: 'cel', source: 'round(price * 100) / 100.0' };
   const configs = [
     ['create_record', CreateRecordConfigSchema, (fields: unknown) => ({ objectName: 'quote', fields })],

packages/spec/src/automation/control-flow.test.ts

@@ -61,7 +61,7 @@ describe('LoopConfigSchema', () => {
     ).toThrow();
   });
 
-  it('emits the xExpression:"template" marker on `collection` through z.toJSONSchema (objectui #2670)', () => {
+  it('emits the xExpression:"template" marker on `collection` through z.toJSONSchema, so the flow designer renders it as a template', () => {
     // The marker rides the same `.meta()` → JSON-Schema channel as
     // `xRef` / `xEnumDeprecated`, telling the flow designer `collection` is an
     // `interpolate()` `{var}` template (not bare CEL).
@@ -77,7 +77,7 @@ describe('LoopConfigSchema', () => {
     );
   });
 
-  it('accepts an inline array collection — the union map.collection declares (#4277)', () => {
+  it('accepts an inline array collection — the union map.collection declares', () => {
     // The executor has always resolved an already-an-array collection (shared
     // logic with `map`); the string-only declaration under-declared what it
     // reads, which the execute-time parse wiring surfaced.
@@ -272,7 +272,7 @@ describe('validateControlFlow', () => {
 
 // ─── [#4001 批 10] unknown keys are rejected, not stripped ──────────────────
 
-describe('[#4001] control-flow strictness — per shape', () => {
+describe('control-flow strictness — an unknown key is refused, per shape', () => {
   it('FlowRegion: `name` and `label` get wrong-layer prescriptions, not renames', () => {
     for (const [key, expected] of [
       ['name', 'A region is not named'],
@@ -438,7 +438,7 @@ describe('TryCatchErrorValueSchema', () => {
     expect(TryCatchErrorValueSchema.parse(both)).toEqual(both);
   });
 
-  it('`code` is an OPEN string, not a closed enum — a third-party or tenant-authored code parses (ADR-0112 D3/D4 + #9106)', () => {
+  it('`code` is an OPEN string, not a closed enum — a third-party or tenant-authored code parses (ADR-0112 D3/D4)', () => {
     // `NodeExecutor` is third-party-registrable and the code vocabulary is
     // `StandardErrorCode` ∪ registered ledger codes ∪ tenant-authored codes,
     // so a closed type would be false the moment anyone registers an executor.
@@ -491,7 +491,7 @@ describe('TryCatchErrorValueSchema', () => {
 // `MAX_REGION_DEPTH` (32) `analyzeRegion`'s own `duplicate node id` line is
 // the only refusal of a within-region duplicate. `flow.test.ts`'s
 // `the seam at MAX_REGION_DEPTH` case pins that hand-off.
-describe('[#4001] validateControlFlow and the key gate meet at the region-slot seam', () => {
+describe('validateControlFlow and the unknown-key gate meet at the region-slot seam', () => {
   const flowWith = (cfg: Record<string, unknown>, type = LOOP_NODE_TYPE) =>
     ({ nodes: [{ ...node('c1', type), config: cfg }] } as never);
 
@@ -540,7 +540,7 @@ describe('[#4001] validateControlFlow and the key gate meet at the region-slot s
     expect(() => validateControlFlow(flowWith({ collection: '{items}', iteratorVariable: 'x' }))).not.toThrow();
   });
 
-  it('nested regions are key-checked at depth, like the structural check (#4389)', () => {
+  it('nested regions are key-checked at depth, like the structural check', () => {
     let message = '';
     try {
       validateControlFlow(flowWith({

packages/spec/src/automation/end-node-outcome.test.ts

@@ -40,7 +40,7 @@ const flowEndingWith = (config: Record<string, unknown> | undefined): Flow => ({
 
 const endConfigOf = (flow: { nodes: Array<{ config?: unknown }> }) => flow.nodes[1].config;
 
-describe('EndConfigSchema — the `end` node contract (#14945)', () => {
+describe('EndConfigSchema — the `end` node contract: it may refuse the run with a message', () => {
   it('defaults `outcome` to `completed` on an empty config', () => {
     expect(EndConfigSchema.parse({})).toEqual({ outcome: 'completed' });
     expect(EndConfigSchema.parse({ outcome: 'completed' })).toEqual({ outcome: 'completed' });
@@ -127,7 +127,7 @@ describe('EndConfigSchema — the `end` node contract (#14945)', () => {
   });
 });
 
-describe('FlowSchema applies the `end` contract — the structural node\'s only door (#14945)', () => {
+describe('FlowSchema applies the `end` contract — the structural node\'s only door', () => {
   it('accepts the card-shape probe and writes the parsed config back', () => {
     const result = FlowSchema.safeParse(flowEndingWith({ outcome: 'refused', message: REFUSAL }));
     expect(result.success).toBe(true);
@@ -217,7 +217,7 @@ describe('FlowSchema applies the `end` contract — the structural node\'s only
     expect(issues?.map((i) => [i.code, i.path])).toEqual([['custom', ['nodes', 1, 'config', 'message']]]);
   });
 
-  it('a region-nested `end` is refused by the FLOW parse itself (#15646/#18112) — the region-door reading this test used to pin is unreachable, because the shape is gone', () => {
+  it('a region-nested `end` is refused by the FLOW parse itself — the region-door reading this test used to pin is unreachable, because the shape is gone', () => {
     // ⚠️ REPLACED, not re-spelled. This case used to assert that the flow parse
     // was GREEN here and that `validateControlFlow` was the door — a true
     // reading of `parseFlowNodeRegions` leaving a refused region raw (#4389).
@@ -278,7 +278,7 @@ describe('FlowSchema applies the `end` contract — the structural node\'s only
   });
 });
 
-describe('the run row carries the refusal (#14945)', () => {
+describe('the run row carries the refusal', () => {
   const run = {
     id: 'exec_refused_001',
     flowName: 'lead_conversion',

packages/spec/src/automation/execution.test.ts

@@ -24,7 +24,7 @@ describe('ExecutionStatus', () => {
     });
   });
 
-  it('names the refused terminal (#14945) — appended LAST, so every `.options` index reader keeps its positions', () => {
+  it('names the refused terminal — appended LAST, so every `.options` index reader keeps its positions', () => {
     expect(ExecutionStatus.options).toContain('refused');
     expect(ExecutionStatus.options.at(-1)).toBe('refused');
     expect(ExecutionStatus.options.slice(0, 8)).toEqual(
@@ -204,7 +204,7 @@ describe('ExecutionStepLogSchema', () => {
     expect(step.branch).toBe(1);
   });
 
-  it('a `success` step that delegated to a child carries the child\'s contained failures on `metrics.failures` (#15617)', () => {
+  it('a `success` step that delegated to a child carries the child\'s contained failures on `metrics.failures`', () => {
     // The card's shape: `loop { subflow(child) }`, one iteration whose child
     // COMPLETED while losing a row. The subflow step itself succeeded — the
     // failure is the child's, contained — so `status` stays `success` and the
@@ -415,7 +415,7 @@ describe('FlowRunSummarySchema', () => {
     expect(summary.failed).toBe(summary.nodes.reduce((sum, node) => sum + node.failures, 0));
   });
 
-  it('`failed` is the fold INCLUDING what a delegating node rolled up from its child — the card\'s measured shape, as ruled (#15617)', () => {
+  it('`failed` is the fold INCLUDING what a delegating node rolled up from its child — it answers what the run caused', () => {
     // Parent `loop { subflow(child) }` over five rows; the child COMPLETED on
     // every iteration and contained one failure on the last. The subflow node
     // succeeded five times — `status: success`, its own executions never
@@ -440,7 +440,7 @@ describe('FlowRunSummarySchema', () => {
     expect(call?.failures).toBe(1);
   });
 
-  it('the control keeps counting as before: a child that FAILED is the delegating step\'s own failure, counted once (#15617)', () => {
+  it('the control keeps counting as before: a child that FAILED is the delegating step\'s own failure, counted once', () => {
     // Same parent, but the child FAILED on the last row rather than containing
     // the failure. That is the subflow step's own `status: failure` — one
     // execution failed — and nothing rides up on top of it: the child's own
@@ -457,7 +457,7 @@ describe('FlowRunSummarySchema', () => {
     expect(summary.failed).toBe(summary.nodes.reduce((sum, node) => sum + node.failures, 0));
   });
 
-  it('declares the roll-up at the point of use — the field describes say so, not only the paragraph above the schema (#15617)', () => {
+  it('declares the roll-up at the point of use — the field describes say so, not only the paragraph above the schema', () => {
     // Triage's explicit failure mode for this card: a reconciliation that
     // leaves `failed`'s own `.describe()` saying the narrow thing. A consumer
     // reads the field's description, so the widened rule has to be there.

packages/spec/src/automation/flow-decision-branch-expression-absent.test.ts

@@ -60,14 +60,14 @@ const AT_BRANCH_0 = ['nodes', 1, 'config', 'conditions', 0, 'expression'];
  * refused row's message is the ONE judge's, byte for byte.
  */
 const TABLE: Array<{ name: string; branch: Node; refused: boolean; refusedWith?: unknown }> = [
-  { name: 'no `expression` key — the #19961 shape', branch: { label: 'y' }, refused: true, refusedWith: undefined },
+  { name: 'no `expression` key', branch: { label: 'y' }, refused: true, refusedWith: undefined },
   { name: '`expression: null`', branch: { label: 'y', expression: null }, refused: true, refusedWith: null },
   { name: 'the predicate under the edge\'s spelling `condition`', branch: { label: 'y', condition: 'true' }, refused: true, refusedWith: undefined },
-  { name: 'a blank string — the #17493 control', branch: { label: 'y', expression: '   ' }, refused: true, refusedWith: '   ' },
+  { name: 'a blank string — blanks are refused', branch: { label: 'y', expression: '   ' }, refused: true, refusedWith: '   ' },
   { name: 'a real predicate — the accept control', branch: { label: 'y', expression: 'true' }, refused: false },
 ];
 
-describe('FlowSchema.parse refuses a decision branch with no `expression` (#19961)', () => {
+describe('FlowSchema.parse refuses a decision branch with no `expression`', () => {
   it.each(TABLE)('$name', ({ branch, refused, refusedWith }) => {
     const issues = issuesOf(flowWith(decision(branch)));
     if (!refused) {