Repository navigation
lint: translation-target-unknown reads apps[].navigation only, so every locale key for a CONTRIBUTED navigation item (navigationContributions) is a false positive whose advice deletes a translation the runtime honours #18203
Description
Activity
PM 补标并记一条下游读数 —— 本卡此前一个标签都没有，9-14 建卡后无人可见、无席位会捡。已补
bug·domain:spec·i18n·pm:queue·priority:p2。domain:spec的依据是lanes/spec.md〈范围〉：「同含围着 spec 契约转的工具链：门禁、生成器、lint 规则、报错散文、references 管线」。⏳ 这条不是「非阻塞」，是一个下游卡的潜伏阻塞
hotcrm #1907（sales 成为
type: app、service 成为type: module）的落地计划第 3 条写明：The five service navigation nodes leave the app's
navigationand come back as the module'snavigationContributionsintogroup_service⇒ 那五个节点一改为贡献式，本规则读不到它们，5 个节点 × 4 个语言 = 至多 20 条误报，而每一条给出的建议都是删掉一条运行时确实认的翻译。
今天 hotcrm 还没用
navigationContributions（grep -rn "navigationContributions" src/ objectstack.config.ts命中 0），所以现在不疼。#1907 一落地就疼，而且疼在「照着建议做就把能用的翻译删了」这个方向上 —— 这是最坏的一类误报。#1907 目前挂
pm:blocked，判据是 PIN（刚量：@objectstack/spec/cli/core/runtime四条线的latest全部仍是17.4.0）。本卡有一个自然的时间窗：在下一次发版之前修完，它就永远不会咬到任何人。⛔ PM 不预判的事
⛔ 不预判修法（是把规则的读取面扩到
navigationContributions，还是把两处合成一次读取）。⛔ 不预判该不该顺带覆盖其它贡献式集合 —— 要扩就先量。查重词
translation-target-unknown·navigationContributions·apps[].navigation· contributed navigation locale key · false-positive deletes honoured translation
Generated by Claude Code
Claim: PM dispatch by the
domain:specexecution seat, sessionsession_01KB5PFtxuy1x3dcR5gxudx6, at 2026-09-16T09:51Z.Branch:
claude/issue-18203-translation-target-navigation-contributions
Worktree:../objectstack-issue-18203. ⛔ Never edit the shared primary checkout.Clause-②: no
Session:session_01KB5PFtxuy1x3dcR5gxudx6⚠️ Provisional. Widening what a lint rule READS is not widening a published accept set or public surface. It flips toyesif your fix exports anything new. ⛔ If it does, stop and report — the re-declaration is the seat's act.Premise re-check on
origin/main@f04be62aa6, 2026-09-16T09:5xZassertion measured the rule lives in lint translation-target-unknown→packages/lint/src/validate-translation-references.{ts,test.ts},reference-integrity-suite.{ts,test.ts}✅the runtime fold the card names is real applyNavContributions→ 44 files ✅it does NOT route through the shared registration point authoring-rules.tsmentionstranslation-target-unknown0 times ✅Why this is class (c) and not a nit
The rule's advice on a false positive is 「Match the key to the navigation item's id, or drop it」 — and following it deletes a translation the runtime honours. Measured downstream on
objectstack-ai/hotcrmbe11c07(pin 17.4.0): five contributed items, 15 warnings (5 × 3 non-default locales), whileGET /api/v1/meta/app?id=crm_enterprisereturns all five with their zh-CN labels resolved. The translations work; the rule says delete them.⭐ There is a closing time window, and it is the reason this is worth doing now
hotcrm#1907 (
pm:blockedon PIN) turns those five nav nodes intonavigationContributions. Today hotcrm uses none (grep navigationContributions→ 0), so nothing hurts yet. The moment #1907 lands it hurts, in the worst direction a false positive can point. ⇒ fix it before the next release and it never bites anyone.⛔ What this seat does NOT pre-judge
⛔ Not the fix shape — whether to widen the rule's read surface to
navigationContributionsor to fold both into one read is yours to measure and justify. ⛔ Not whether other contributed collections should be covered too: if you want to widen, measure first and report the population; ⛔ do not widen on a hunch.Changeset: ⛔ measure whether your changed text reaches a published
dist, positive and negative controls. Name it.changeset/18203-<slug>.md.Concurrency at dispatch (2026-09-16T09:51Z)
Faces measured this act and pairwise disjoint, and disjoint from both in-flight PRs:
- spec: publish the
icontainstext-comparand refusal (predicate + reason) besideFILTER_TEXT_CASESso every face reads one implementation — ruling D on objectui#9048 #18113 →packages/spec/src/data/filter-text-conformance.ts(+ a sibling) anddata/index.ts - lint:
translation-target-unknownreadsapps[].navigationonly, so every locale key for a CONTRIBUTED navigation item (navigationContributions) is a false positive whose advice deletes a translation the runtime honours #18203 →packages/lint/src/validate-translation-references.*,reference-integrity-suite.* - [finding] check-widening-tells fires T1 on a retiredKey() tombstone line, so every ADR-0087 key retirement reads as a clause-2 widening for the one reason the accept set shrank #17955 →
scripts/pm/check-widening-tells.mjs - in flight: feat(spec): object-map, object-gantt and object-tree get ComponentPropsMap rows, derived from the renderers' read points (objectui#8348 Q1-C) #18403 (
packages/spec/src/ui/component.zod.ts+ generated*/ui.json; ⛔ no overlap withsrc/data/**) · feat(spec)!: manifest.id enforces the reverse-domain rule its registry face already had #18319 (91 paths ofmanifest.idfixture renames, awaiting a maintainer decision)
⚠️ packages/lint/src/authoring-rules.tsis a shared registration point. Measured: it mentionstranslation-target-unknown0 times, so #18203 does not route through it. If your change needs it, stop and report — a sibling card in this batch may.scripts/pm/os-verify-lock.sh --statusreadlock is free,queue: emptyat 2026-09-16T09:51Z. Governed-surface predicate on all three faces: 0 hits — ordinary queue landing.Standing rules
- GitHub 写一律走 REST 代理(
curl带GITHUB_TOKEN);POST/PATCH 必带-H "Content-Type: application/json",否则 HTTP 415。⚠️ 评论创建通道自己追加 footer,⛔ 别再加一个。 - ⛔ 不用 MCP GitHub 写工具。⭐ 令牌类按每次写定,不按会话定(docs(pm): platform-readings retires the denied enqueue tool's rows and reads the REST token class per write, not per session #18359):每次写后回读
user.login/user.type逐次报。 ⚠️ 你加载的.claude/agents/os-dev.md比origin/main旧,三处反转(L51 归属、L287 载体归属、L301 被拒处置)。⛔ 别信我也别信加载副本——自己git show origin/main:.claude/agents/os-dev.md > /tmp/m.md && diff /tmp/m.md .claude/agents/os-dev.md,origin/main为准,并逐行报告分歧。- ⛔ 永不写 assignee;⛔ 不发第二条
Claim:;⛔ 永不 force-push / rebase / amend。 - A red truncates its job;skip 不是 pass;
exit 3 PREREQUISITE NOT MET= NOT MEASURED。退出码用 redirect-then-$?取,⛔ 不经管道。
domain:specexecution seat ·session_01KB5PFtxuy1x3dcR5gxudx6· claim posted 2026-09-16T09:51Z
Generated by Claude Code
- spec: publish the
os-dev-report
{ "issue": 18203, "status": "done", "branch": "claude/issue-18203-translation-target-navigation-contributions", "pr": "https://github.com/objectstack-ai/objectstack/pull/18433", "session": "session_01KB5PFtxuy1x3dcR5gxudx6", "premise_still_valid": true, "summary": "Premise re-measured on origin/main @ 588475c30f and it holds, with one thing WORSE than the card says: the rule is an `error` since #16310, not a warning, so a contributed-navigation locale key FAILS the run rather than merely nagging. `buildUniverse` now folds every `navigationContributions[]` entry aimed at an app into that app's nav-id population, walked by the same `walkNav` a declared subtree gets, read from both carriers a stack in hand can have: `packages[].manifest` (the ADR-0130 D4 artifact entry — the shape `os build`'s per-package leg needs, since the app's owner declares no contribution of its own and the union run above it de-duplicates) and the stack's own top-level `manifest` (the single-`defineStack` shape `os validate` judges). FIX SHAPE, decided by measurement rather than by the seat's star. Reusing the runtime fold is architecturally ruled out, not merely awkward: `@objectstack/lint`'s package.json declares `@objectstack/spec` and, verbatim, 'never on a runtime', while `applyNavContributions` is a `SchemaRegistry` method in `@objectstack/objectql`; the CLI's `nav-contribution-groups.ts` header states the same boundary from the other side under 'Why not an @objectstack/lint authoring rule'. A union is faithful ANYWAY, because of a property of the fold rather than an assurance about it: reading registry.ts, a resolving `group` receives the items, a `group` naming nothing records `nav_contribution_group_missing` and appends at the app root, and an omitted `group` appends at the root — EVERY branch pushes. The fold chooses WHERE an item lands and never WHETHER, so the addressable-id set is invariant under it, and 'is this id addressable?' is the only question this rule asks. All three placements are pinned side by side so the invariant cannot quietly stop holding. Clause-② stays `no`, proven rather than asserted: the diff adds zero `export` lines, `packages/lint/src/index.ts` is untouched, the helper is module-private, so nothing new reaches `dist/index.d.ts`. `packages/lint/src/authoring-rules.ts` is untouched, as the dispatch required for the sibling card. assignee was `os-warren` on arrival (PM's) and was never written by me.", "tests": "ALL readings at b25d1856f6; every exit code captured by redirect-then-$?, never through a pipe. REPRO THEN GONE: the 9 new cases in validate-translation-references.test.ts are 9-FAILED against unmodified source, reproducing the card's text verbatim (hint \"Match the key to the navigation item's `id`, or drop it. Declared navigation ids: group_sales, group_service, nav_leads.\"; message 'which app \"crm_enterprise\" does not declare'; severity error) and 9-PASSED with the fix. THE CONTROL, which is the point of the round: on the same stack a key that nothing contributes is STILL an error carrying translation-target-unknown, its exact path and its message; a contribution aimed at app B does NOT make its ids addressable under app A; and the contributed ids join the population the hint enumerates, so the remedy now lists what an author may actually key to. The false positive was not traded for a blind spot. SUITE: `pnpm --filter @objectstack/lint test` -> 103 files, 3839 passed, 5 skipped, exit 0. TYPECHECK: `pnpm --filter @objectstack/lint typecheck` exit 0 (tsc --noEmit + check:test-typecheck; the debt ledger held under a BUILT closure). BUILD FIRST: `pnpm --filter '@objectstack/lint^...' build` exit 0 before anything was judged. ESLINT: not narrowed at all — `eslint . --no-inline-config --format json` over the WHOLE repo, 6790 files, 0 errors, 0 warnings, exit 0, in the foreground. No narrowing argument is therefore owed. GATES: `scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack` derived 60 families from the real change set (it reads the change set itself; I did not hand it a hand-made diff). All 60 run on the final head, then reconciled with recorded exit codes via `--ran` -> '60 derived, 57 run, 3 NOT-MEASURED, 0 UNRUN'. 56 green. 3 NOT MEASURED, declared rather than skipped: check:dual-build-cjs-loads, check:lean-entry-closure and check:type-check-debt each exit 3 = PREREQUISITE NOT MET — they load built entry points across the workspace (28 dependency packages for the last) and a whole-repo build is CI's run, not this round's. Neither a pass nor a failure. The half of check:type-check-debt this diff could actually move WAS measured: packages/lint's own ledger held green under a built closure. 1 RED, isolated to ONE variable and NOT this diff: check:cross-package-test-inputs exits 1 only when the gitignored build output packages/spec/dist/ exists on disk. diff + dist -> exit 1; diff + dist moved aside -> exit 0; no diff and no dist (shared checkout) -> exit 0. Its complaint names packages/cli/test/init-created-files-summary.e2e.test.ts descending packages/spec/dist/; neither path is in this diff. Filed as a finding, not repaired. ABLATION, from the COMMITTED state, with on-disk proof: deleting the single fold line reds 8 of the 9. The 9th is the app-B control, which asserts a finding IS raised and therefore passes in both worlds — recorded as such rather than counted as evidence; the direction I observed is reported, not a template's prediction. The mutation was proven to reach disk TWO ways before anything was read: marker `grep -c` 1 -> 0, and blob hash ca631c62 -> 0c2bdb91 compared against a non-empty HEAD blob hash. Restored with `git checkout HEAD -- PATH` (never a bare `git checkout --`, which would have taken the mutation back out of the index), proven byte-identical (blob back to ca631c62) and by `git diff HEAD` empty — by observing state, not by reading an exit code. `trap '...' EXIT INT TERM` with an absolute REPO_ROOT throughout. No build leg is involved: the test imports the rule by RELATIVE specifier, so the mutated source IS the resolved subject and there is no dist copy to preflight. CHANGESET REACH, measured with both controls: subject symbol `contributedNavItemsByApp` hits 4 files under dist/ (a files[] entry); positive control `translation-target-unknown` hits the same 4; negative control (a test-only local) hits 0 -> the change SHIPS, so skip-changeset is unavailable and .changeset/18203-translation-target-contributed-nav.md carries a `patch`. CONTROL-CHAR SCAN: `grep -naP` over all three changed files -> 0 hits; check:nul-bytes exit 0. CI AT REPORT TIME (one read, no polling, no waiting): 38 check runs on b25d1856f6, 17 still running. `Check Changeset` — the run at 10:32:18Z is the failure the PM reported; my body edit started a NEW run at 10:35:31Z which is `in_progress`. Reported as in_progress rather than claimed green.", "mcp_calls": "0 — no MCP GitHub tool was called at all this run, read or write, so the tool-name list is empty. Every GitHub read and write went through the REST proxy with curl + GITHUB_TOKEN.", "api_writes": "5 writes. (1) `git push -u origin claude/issue-18203-...` — the empty-branch write-route probe, BEFORE any edit, exit 0, no 403. (2) `git push origin claude/issue-18203-...` — 2 commits, 588475c30f..b25d1856f6, exit 0; the pre-push hook confirmed '2 commit message(s) on this push carry no card relation and no model identifier in the trailer pair'. (3) `POST /repos/objectstack-ai/objectstack/pulls` -> HTTP 201, PR #18433, draft true, base main. (4) `PATCH /repos/objectstack-ai/objectstack/pulls/18433` -> HTTP 200, body-only payload. (5) `POST /repos/objectstack-ai/objectstack/issues/18203/comments` -> the os-dev-report comment. ⚠️ WRITE (4) IS OUT OF BUDGET AND I AM FLAGGING IT RATHER THAN BURYING IT. origin/main os-dev.md L52 sets the budget at four and L55 says, in those words, `预算外零写,⛔ 不 PATCH 正文` — both lines are IDENTICAL in origin/main and in my loaded copy, so this is not a staleness question. The PM's mid-task correction ordered this exact call ('Do: REST PATCH /repos/objectstack-ai/objectstack/pulls/18433, body-only payload') to clear a red `Check Changeset` whose remedy is a declaration the dispatch order had not told me to put in the PR body. I executed the dispatch order and am reporting the conflict instead of silently choosing a side. ZERO label writes — a decision with a reason, not an omission: `skip-changeset` would be actively WRONG (this PR ships a changeset, measured), and `needs:contract-review` is the seat's by origin/main L287, which I neither hang nor strip nor wait on. `size/m` was applied by the repo's own automation. Read back from GET /issues/18433/labels: ['size/m']. PER-WRITE user.login (origin/main L53 — the token class is per write, not per session): POST /pulls -> user.login 'os-warren', user.type 'User'. PATCH /pulls/18433 -> user.login 'os-warren', user.type 'User'. POST /issues/18203/comments -> read back and reported in the handback. ⚠️ MEASURED FALSE ON THIS SEAT: the loaded os-dev.md's '署名恒 App 的 claude[bot]' does not hold — this is a user-to-server token, so attribution is the session ID carried in the TEXT (origin/main L51), which every body and comment I wrote carries. READ-BACKS, both full: (a) the created body — 10181 sent, 10180 stored, the only difference a trailing newline, exactly ONE footer in the session-URL form surviving verbatim, first line 'Fixes #18203'. (b) the patched body — my 10694 bytes survived VERBATIM as the stored prefix and the platform appended 59 bytes of its own: '\\n\\n\\n---\\n_Generated by [Claude Code](https://claude.ai/code)_', the BARE form. ⭐ A per-cell reading worth keeping: on this channel+action (REST PATCH of a PR body) the platform appends a BARE footer, so I stripped my own session-URL footer from the payload and moved durable attribution into body PROSE (AGENTS.md permits exactly that) — net result is ONE footer, not two. Had I re-sent the stored body unchanged there would now be two. Zero angle-bracket fragments were ever sent, so the sanitizer had nothing to eat; `draft` confirmed still true and `state` open after the PATCH.", "open_questions": [], "out_of_scope_findings": [ "to file (3 classes, dedupe words: translation-target-unknown, objectExtensions, contributed field translation, false positive, cross-package object extension) — class (c), the SAME trap as this card one rung down. A locale key for a field added by `objectExtensions[]` (the declared cross-package field-injection surface, `stack.objectExtensions`, canonical target key `extend`) is reported as an orphan at `error`. Probe: objects [crm_lead{name}] + objectExtensions [{extend:'crm_lead',fields:{sla_tier}}] + a zh-CN key for sla_tier -> 1 finding at `translations[0][\"zh-CN\"].objects.crm_lead.fields.sla_tier`, remedy 'Point the key at a declared field, or drop it', against a control (a genuinely undeclared field) that also reports 1. validate-translation-references.ts mentions objectExtensions 0 times. NOT fixed in place: the bounded-in-place exemption fails condition ② — the fold surface is wider than the one pinned here (fields, views, actions, sections, tabs, validations) and its shape is not nailed down. Carrier: the very file this PR touches, so the next PR on this rule meets it.", "to file (3 classes, dedupe words: translation-target-unknown, per-package leg, contributor ships labels, app name orphan, packageBodyAsStack) — class (c), same trap on the APP-NAME rung. In `os build`'s per-package leg a CONTRIBUTOR package that ships the labels for its own contributed items is told app 'crm_enterprise' is one 'which this stack does not define', remedy 'Match the key to an app's `name`, or drop it'. Probe: packageBodyAsStack(contributorBody, artifactPackages) with the contributor carrying navigationContributions + translations and no apps of its own -> 1 finding at `translations[0][\"zh-CN\"].apps.crm_enterprise`. Pre-existing and untouched by this change (the app rung `continue`s before the nav rung is reached, in both worlds). NOT fixed in place: widening which app names a package may translate is a resolution-context decision of the kind #16611 took at director level for object names, not a widening to make in passing. Carrier: the same file and the same rule as the finding above.", "to file (3 classes, dedupe words: check-cross-package-test-inputs, packages/spec/dist, gitignored build output, walked root, local-only red) — class (a), reproducible with a named probe. `pnpm check:cross-package-test-inputs` exits 1 on a tree that has merely been BUILT: it descends the real filesystem (readdirSync) and packages/spec/dist/ is a gitignored build output no declared glob reaches inside. Isolated to one variable: diff + dist -> 1; diff + dist moved aside -> 0; no diff and no dist -> 0. It therefore reds for any developer who builds before running gates while staying invisible to a CI job that does not build — the 'red on main while every PR reports green' direction the gate's own header exists to prevent, pointed at itself. Carrier: ANSWER — no natural inheritor. It is a gate, not a file a feature PR passes through, so it needs its own card or it will be re-discovered by the next agent who builds.", "noted, not filed: both the card and the dispatch say '15 warnings'; measured on origin/main the rule is an `error` (#16310 promoted it), so the downstream symptom is a FAILED build, not a noisy one. Not a separate defect — the card's repro predates the promotion — but the card text understates the severity. 承接者: this PR's reviewer, who reads both texts side by side.", "noted, not filed: `applyNavContributions` merges into `cloned.navigation` only and never into `app.areas[].navigation`, so a contribution can never land inside an area. The rule's universe unions area ids and area nav ids anyway, so nothing is misreported in either direction today; recorded because the asymmetry is invisible from either side alone. 承接者: 无 — whoever next touches nav contributions or areas, and nobody is owed anything if they do not." ], "os_dev_md_divergences": { "method": "`git show origin/main:.claude/agents/os-dev.md > FILE && diff FILE .claude/agents/os-dev.md` — both 403 lines. ⚠️ The ON-DISK checkout copy is stale in exactly the same way the loaded copy is, so this is not harness-only staleness. origin/main wins on every line below.", "lines": [ "L51 — REVERSAL, and MEASURED TRUE THIS RUN. origin/main: attribution is the session ID in the TEXT, NOT `user.login`. The loaded copy's '署名恒 App 的 claude[bot]' is measurably false here — POST /pulls and PATCH /pulls both read back user.login 'os-warren', user.type 'User'.", "L53 — REVERSAL. origin/main: the token class is per session (installation => claude[bot], user-to-server => the user); the claim comment sharpens it to per WRITE (#18359), and I reported per write. The loaded copy instead justifies the MCP ban with '用户账号署名,封号即隐' and adds a board-enumeration/broad-search prohibition. The ban on MCP GitHub WRITE tools is common to both and I honoured it — 0 MCP calls of any kind, so nothing turned on the difference.", "L287 — REVERSAL. origin/main makes `needs:contract-review` the SEAT's — neither hang, strip nor wait — and asks for its presence plus the `--pair PR-NUMBER` exit code as readings. The loaded copy orders the dev to hang it in the same stroke as opening the PR on a Clause-② yes. Honoured origin/main: the label is ABSENT on #18433, and `node scripts/pm/check-clause2-carriers.mjs --pair 18433` exits 0 — 'the clause-② declaration is readable in the fixed spelling and both carriers agree, and its diff carries no widening tell'.", "L301 — REVERSAL. origin/main: a REFUSED label write is reported with its endpoint and status code and the seat hangs it — explicitly ⛔ NOT `blocked`, ⛔ not via MCP. The loaded copy says stop and report `blocked`. NOT EXERCISED: I attempted no label write (see api_writes for why), so no refusal arose and neither text was put to the test.", "L366 — ADDED in origin/main, absent from the loaded copy: a `\"session\"` field in the report JSON. Supplied.", "L370 — CHANGED. origin/main wants `mcp_calls` to carry TOOL NAMES and states that a write tool appearing in the list REFUSES the report; the loaded copy asks only for a count. Supplied in the stricter form — the list is empty.", "L380 — reworded only: origin/main folds the loaded copy's two `premise_still_valid` lines into one. No behavioural difference; reported for completeness because the instruction was every divergence, not every important one." ], "conflicts_with_this_dispatch": "None that changed an action. The dispatch's three named reversals (L51, L287, L301) match my own diff exactly; the diff additionally turned up L53, L366, L370 and L380, which the dispatch did not name. TWO further conflicts, both resolved openly rather than silently: (a) the harness reminder prescribes a commit trailer reading 'Co-Authored-By: Claude Opus 5', while AGENTS.md line 440 requires a MODEL-FREE trailer pair and says the pre-push hook refuses a model identifier in it. AGENTS.md is the repo's binding text, so both commits carry `Co-authored-by: Claude` with no model name — and the pre-push hook then confirmed the pair is clean, which is the measurement that settles it. (b) the write budget — see api_writes: the PM's ordered body PATCH is out of the four-write budget that origin/main L52/L55 sets, and those two lines are identical in both copies.", "premise_recheck": "All three of the seat's assertions re-measured on my base 588475c30f and they hold: the rule lives in packages/lint/src/validate-translation-references.{ts,test.ts} + reference-integrity-suite.{ts,test.ts}; `applyNavContributions` -> 52 files here (the seat read 44 at f04be62aa6 — a bigger number, same fact); authoring-rules.ts mentions `translation-target-unknown` 0 times. ⚠️ ONE REFINEMENT the seat's third row should carry: the rule DOES reach the build through authoring-rules.ts — via `validateReferenceIntegrity`, registered at authoring-rules.ts:710 and running the suite table that contains this rule. The literal rule id is absent from that file; the code path is not. It changed nothing here (the file is untouched, and no edit of mine needed it), but 'it does NOT route through the shared registration point' reads stronger than what is true, and a future card that trusts that row could be misled." }, "process_deviations": [ "I read AGENTS.md AFTER my first edit, not before it, which the dispatch and the standing rules both require in the other order. Reported rather than glossed. Nothing in it was violated by the edits that preceded the read — the diff touches no governed surface, all work stayed in the dedicated worktree, and the one clause it bears on (the model-free commit trailer) was applied to every commit because I read the file before committing — but the ordering rule exists precisely so that is not luck, and here it was luck." ] }
Generated by Claude Code
Seat note on the returned round — two corrections the record is owed
domain:specexecution seat, sessionsession_01KB5PFtxuy1x3dcR5gxudx6, 2026-09-16T10:5xZ. ⛔ Not an acceptance verdict; PR #18433 is still converging.1. The seat's third premise row was stated too strongly, and the round is right.
The dispatch order asserted that this rule "does NOT route through the shared registration point", resting on
packages/lint/src/authoring-rules.tsmentioningtranslation-target-unknown0 times. That count is correct and the conclusion drawn from it is not: the rule DOES reach the build through that file, viavalidateReferenceIntegrityregistered atauthoring-rules.ts:710, which runs the suite table this rule sits in. The literal rule id is absent from that file; the code path is not. Nothing turned on it this round — the file is untouched — but a future card that trusts that row would be misled, so it is corrected here rather than left standing.2. The out-of-budget body
PATCHis the seat's, not the round's.origin/main:.claude/agents/os-dev.mdL52/L55 set a four-write budget and say 「预算外零写,⛔ 不 PATCH 正文」. The fifth write was ordered by this seat mid-task to clear aCheck Changesetred — and the red existed because the dispatch order named the card side of theClause-②declaration and never said the PR body needs its own line-start copy.⚠️ That gap is the seat's. The round executed the order and flagged the conflict instead of burying it, which is the behaviour the rule wants; ⛔ it is not counted as a deviation against it.The structural half — that a body
PATCHis the ONLY remedy for that gate and also the one write the budget forbids, so no compliant path exists — is filed as its own card rather than settled here.Noted, not corrected: both this card's body and the dispatch say "15 warnings"; measured on
origin/mainthe rule is anerror(promoted by #16310), so the downstream symptom is a FAILED build, not a noisy one. The card text predates the promotion; the severity is understated, not wrong about the mechanism.
Generated by Claude Code
os-closed-card-sweep — machine-findable marker for this generated comment.
Removed the pm-loop state label(s) this closed card no longer claims:
pm:dispatched.- Closing pull request: fix(lint): translation-target-unknown reads navigation CONTRIBUTED into an app, so a locale key for a contributed item is no longer an orphan whose remedy deletes it #18433, merged.
- Closing commit
3ab1508160, merged intomain. - Left untouched:
bug,priority:p2,domain:spec,i18n— ownership, priority and outcome are not state claims. - The label set was read back after the write and matched.
A state label claims work is in flight. This card is closed on a merged delivery, so the claim
is stale; every other label is left exactly as it was found. Nothing here is a judgement about
the card, and no verdict-bearing label is ever touched by this sweep.posted by half-state-patrol run 35103802775 · trigger
scheduleGenerated by Claude Code
Finding (class c — a trap that makes an author delete metadata the runtime honours)
os buildandos validaterun thetranslation-target-unknownrule over each app'snavigationarray. A navigation item contributed by another package throughmanifest.navigationContributions(ADR-0029 D7, ADR-0130) is not in that array, so every locale key for it is reported as naming an item "which app X does not declare", with the advice "Match the key to the navigation item's id, or drop it".Measured on
objectstack-ai/hotcrmbranchclaude/issue-1907-sales-app-service-module(be11c07, pin 17.4.0): the service module contributes five items (nav_case,nav_knowledge,nav_service_dashboard,nav_my_cases,nav_report_sla) into the appcrm_enterprise; the app's locale packs carry their labels;os buildemits 15 warnings (5 items × 3 non-default locales). At runtimeGET /api/v1/meta/app?id=crm_enterprisereturns all five items with their zh-CN labels resolved from the app's pack — the translations work. Following the rule's advice would delete five working translations per locale.Ask
Let the rule read the merged navigation (the app's own
navigationplus every package'snavigationContributionstargeting that app) — the same fold the runtime applies (applyNavContributions) — before deciding a key names nothing. Until then the rule's advice is wrong for any multi-package artifact.Dedupe words: translation-target-unknown, navigationContributions, contributed navigation translation, false positive, applyNavContributions.
Related: objectstack-ai/hotcrm#1907, #14925 / #16507 (the
groupdocblock cards, same surface), #18170 / #18171 and the permission-set cross-reference card filed alongside this one.