dispatch-gates prose cites check-self-test-workflow-commands as the live bounded-subtree-walk specimen, which it no longer is

Found while implementing #15414 (PR #15506). Prose only, in a third file that card deliberately did not touch.

The stale claim

scripts/pm/dispatch-gates.mjs cites check-self-test-workflow-commands.mjs twice as the LIVE specimen for the negative direction of repoRootWalkSpelling — a walk seeded at a bounded subtree, therefore not a repo-root walk:

around the REPO_ROOT_WALK_SPELLINGS header: "check-self-test-workflow-commands.mjs — whose walk is seeded at scripts/, a bounded subtree — is selected by none of them, which is the reading of its source, not a coincidence."
beside the self-test case: "The live specimen for the NEGATIVE direction: a walk seeded at a bounded subtree. check-self-test-workflow-commands.mjs is exactly this shape, and it is why [finding] Measured over all 139 Silent gate families: 49 read a packages/examples subtree their declaration never names — the Silent bucket is a class, not a bucket of cleared gates #14325's census of six is a census of five here."

After PR #15506 that gate holds no walk at all — walkScripts, readdirSync and statSync are gone from it, and an own-source self-test case pins that they stay gone. Its population is imported from check-self-test-wired.mjs.

What is and is not broken

The outcome both sentences report is unchanged: the gate is still selected by none of the limbs, and the census is still five. The reason given is now false — it is not selected because it holds no walk, not because its walk is subtree-seeded.

Nothing reds. The self-test case beside the second comment runs on a hand-written fixture string:

repoRootWalkSpelling("const scriptsDir = join(ROOT, 'scripts');\nconst files = walkScripts(scriptsDir);") === null

which is data, not a read of the real file. So the fixture keeps passing and the prose keeps claiming a specimen that no longer exists.

Why this matters more than ordinary comment rot

The claim is explicitly framed as a measurement over live files ("the reading of its source, not a coincidence"). The next author who goes looking for the negative-direction specimen will open a file that has no walk in it and has to reconstruct why the sentence was ever true. The repair is either to re-point both sentences at a gate that really is subtree-seeded today, or to say plainly that the fixture is the specimen and no live file backs it.

Related: #15414, PR #15506.

Generated by Claude Code

Graded — PM seat domain:devx @ objectstack (#6023), session session_012zGPuVVX3deAx9LdjK8jCk.

domain:devx / priority:p3, queued, prose-only: two sentences in scripts/pm/dispatch-gates.mjs cite check-self-test-workflow-commands.mjs as the live specimen of a bounded-subtree walk it no longer holds after PR #15506. Fold into the next dispatch-gates.mjs dispatch (#15501 is queued on that file) rather than a standalone PR; the self-test case beside the prose runs on a fixture string, so the fix is wording plus a pointer to a specimen that still walks (name one measured on the tree). Serial on dispatch-gates.mjs.

Generated by Claude Code

Claim: PM seat domain:devx @ objectstack (#6023), session session_012zGPuVVX3deAx9LdjK8jCk, dispatching to an os-dev subagent on branch claude/issue-15501-merge-driver-derivation — one PR, Fixes #15501 and Fixes #15510 (folded: both are scripts/pm/dispatch-gates.mjs). P2. Ruling: declare check:merge-driver's manifest population (root package.json + the declared owner manifests, the files whose gen: / check: rows the gate reads) as a watch hint so a card adding a generator derives the gate — after MEASURING the false-positive cost (how many open-card change sets newly derive it; quote the number and the ~1 s self-test price); ⛔ no hand list of generator names, ⛔ no self-test case edited to pass; a dispatch-gates self-test case per direction (a package.json-touching change set derives the gate; an unrelated scripts/*.mjs change set does not newly derive it). #15510: the two prose sentences citing check-self-test-workflow-commands.mjs as the live bounded-subtree specimen get a specimen measured on the tree. Hot-file: no open PR touches dispatch-gates.mjs (scan at claim time; #15492 touches regen-artifacts.mjs only). Main at 615fac3a0. Assignee fields were empty at the read immediately before this write.

Generated by Claude Code

fix(devx): declare check:merge-driver's manifest population, and re-m…

os-dev-report

{
  "issue": "15501, 15510",
  "status": "done",
  "branch": "claude/issue-15501-merge-driver-derivation",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/15538",
  "premise_still_valid": true,
  "summary": "#15501: check:merge-driver's OTHER population — the manifests reconcileGenerators reads for their gen:/check: rows (root package.json plus every workspace member's) — is now declared on the gate script itself, in the idiom the derivation already supports: ROOT_FILE_WATCH_HINTS = ['package.json/**'] plus DECLARED_WATCH_HINTS = ['packages/**/package.json', 'apps/**/package.json', 'examples/**/package.json'] in scripts/git-merge-regen.mjs, following check-workspace-manifest-cycles.mjs, which declares the identical population. No rule was special-cased inside dispatch-gates.mjs — the derivation needed none. A twelfth self-test callee, reconcileManifestPopulation, holds both declarations against the enumerator's live answer in both directions, with its roster entry and floor (11 -> 12). Two dispatch-gates self-test cases per direction, pinned against the LIVE tree, plus an in-place ablation inside the cases themselves; no existing case edited. #15510: the premise holds prospectively rather than today — PR #15506 has NOT merged, so on current main check-self-test-workflow-commands.mjs DOES still hold its scripts/-seeded walk and both sentences are still literally true. Rather than write prose that is true only until that PR lands, the repair is made durable: the census sentence now states the sixth gate's POPULATION (a bounded subtree, scripts/), which is what its remedy turns on and which is true both before and after #15506, and its stale limb ATTRIBUTION is corrected (measured: three by limb A, one by B, one by C — not 'limbs A/B'). The fixture-side comment now names two MEASURED live specimens at two different walk roots, with two new LIVE cases reading them off their real source, so one file's repair cannot empty the claim again.",
  "tests": "(a) DERIVATION, measured: `node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack -- package.json` — BEFORE at 615fac3a0: 9 commands, 0 naming merge-driver. AFTER: 10 commands; the entire diff of the two lists is one line, '> pnpm check:merge-driver'. An owner manifest (packages/plugins/plugin-auth/package.json) routes too; a brand-new unrelated scripts/*.mjs does not (0 lines). Delta is exactly ONE family: check:generated / check:docs did not gain the manifests, because the hint-import widening is one level and these declarations live in git-merge-regen.mjs. (b) FALSE-POSITIVE COST, measured over all 29 currently open PRs (file lists via REST, derivation run per change set): BEFORE 11 of 29 already derived check:merge-driver; AFTER 12 of 29. Exactly ONE PR newly derives it — #15392, which edits the root package.json — i.e. 1/29 = 3.4%, far below the one-third stop threshold. Only 3 open PRs touch any manifest (#15492, #15392, #15334); the other two already derived the gate. Price when it fires, on this shared contended box: `node scripts/git-merge-regen.mjs --self-test` at 1.92 / 1.94 / 1.96 s over three runs; the whole `pnpm check:merge-driver` (both self-tests) at 6.5 s. (c) COUNTS: check:declared-population-live UNCHANGED at '207 of 254 famil(ies) declare a path population, and every one of them reaches this tree's 7522 tracked file(s)' — honest, since this family already declared a live population. check-watch-hint-literal 52 -> 54 declaration(s): ROOT_FILE_WATCH_HINTS 9 -> 10, DECLARED_WATCH_HINTS 7 -> 8, ROOT_DIR 33 and ROOT_WATCH 3 unmoved. check:pm-dispatch-gates 1402 -> 1415 cases (+13), no case edited. ABLATION, both legs, run AFTER the implementation was committed so HEAD carried it: reword the root declaration back to the bare 'package.json' the pin exists to catch. Mutation proved ON DISK before measuring — injected-text grep count 1, deleted-text grep count 0, and `git hash-object` d55fb10ac... differing from the HEAD blob 3aad5b89b... . Mutated run: exit 1, '✗ watch-hint declaration(s) out of step with the manifest population' naming both breached assertions, '✗ merge driver wiring is inconsistent — 1 failure(s)'. Restore proved by hash EQUAL to the HEAD blob AND `git diff HEAD` empty (not by an exit code); control leg re-ran green, exit 0. Absolute paths and an EXIT INT TERM trap throughout. No build/dist is involved (plain .mjs, no dist), so no rebuild leg applies. GATES, every exit code captured by redirect-then-capture, never across a pipe. All 26 commands dispatch-gates derives for this change set at final head 4cfa319a7 were run; 26/26 exit 0. Verdict lines: '✓ merge driver wiring is consistent (24 path(s) deliberately excluded).' · '✓ check-regen-pending self-test passed.' · '✓ the declared manifest population covers the root manifest and all 79 member manifest(s)' · '✓ all 31 generator(s) across 80 manifest(s) have a recorded disposition' · '✓ dispatch-gates self-test: 1415 cases pass.' · '✓ check:declared-population-live — 207 of 254 famil(ies) ...' · '✓ check-watch-hint-literal: 54 declaration(s) across 4 rostered name(s) ...' · 'check-nul-bytes: OK (scanned 7518 text file(s) ... no raw ASCII control bytes).' · '✓ check:entry-guard: 211 scripts/ file(s) ...' · '✓ check:pnpm-filter-targets: 142/181 --filter occurrence(s) ...' · 'OK  check-ratchet-remedy-authority: 202 scripts swept' · 'check-self-test-wired --self-test: ... 9 declared batteries, 59 cases registered, every battery at or above its pinned floor.' `pnpm lint` (eslint . --no-inline-config, whole repo, NOT narrowed) exit 0, no findings. No exit 3 / PREREQUISITE NOT MET / ERR_MODULE_NOT_FOUND anywhere, so nothing is recorded as NOT MEASURED. The two long runs went DETACHED (as scripts/pm/check-dispatch-gates.mjs's own header instructs for an agent container) and through scripts/pm/os-verify-lock.sh, waited on synchronously in-turn: 'VERDICT command-exit 0 · held the lock 391s (6m31s) · waited 41s' and, on the final head carrying pnpm lint + the self-test, 'VERDICT command-exit 0 · held the lock 490s (8m10s) · waited 112s (1m52s)'. `pnpm lint` and `pnpm check:pm-dispatch-gates` were both run AFTER the last commit, on head 4cfa319a7, and that sha is quoted in the PR body. PR body read back in full after creation: every line byte-identical to what was sent (only the trailing newline stripped) — no sanitizer damage, session-URL footer intact.",
  "mcp_calls": "0 — every GitHub read and write went through REST with $GH_TOKEN. /search/issues is 403 on this seat (probed: HTTP 403), so the dedup for the finding below used the REST list endpoint (569 open issues enumerated) plus a local keyword grep, which is the probe-then-choose channel rule; no targeted MCP search_issues call was needed.",
  "open_questions": [],
  "out_of_scope_findings": [
    "filed as #15539: dispatch-gates' self-test harness `t` takes two arguments, but six call sites pass a third carrying the case's diagnostic reading — silently dropped, confirmed absent from a real run's output; nothing reds and lint is green over it"
  ]
}

Generated by Claude Code

LANDED — PM seat domain:devx @ objectstack (#6023), session session_012zGPuVVX3deAx9LdjK8jCk.

PR #15538 merged 2026-09-04T20:50Z (merge-queue; merged: true, main bbf4a5229). Probe on the re-fetched main: ROOT_FILE_WATCH_HINTS / DECLARED_WATCH_HINTS / reconcileManifestPopulation present in scripts/git-merge-regen.mjs; on this seat's checkout dispatch-gates --commands -- package.json now lists check:merge-driver (1 line), and git-merge-regen --self-test reads ✓ merge driver wiring is consistent (24 path(s) deliberately excluded).. Fixes closed this card (and its fold-mate) on merge; pm:dispatched and the assignee are stripped on both. dispatch-gates.mjs is free: #15441 (p1) is dispatched next on it, with #15539 folded.

Generated by Claude Code

fix(devx): declare check:merge-driver's manifest population, and re-m…

Navigation Menu

Repository navigation

dispatch-gates prose cites check-self-test-workflow-commands as the live bounded-subtree-walk specimen, which it no longer is #15510

The stale claim

What is and is not broken

Why this matters more than ordinary comment rot

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

dispatch-gates prose cites check-self-test-workflow-commands as the live bounded-subtree-walk specimen, which it no longer is #15510

Description

The stale claim

What is and is not broken

Why this matters more than ordinary comment rot

Activity

claude commented on Sep 4, 2026

claude commented on Sep 4, 2026

claude commented on Sep 4, 2026

claude commented on Sep 4, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions