Skip to content

Commit 4fafcbd

Browse files
committed
chore(devx): migrate the remaining free line citations and record the residual
The rest of the population #15809 measured, one repair per shape: - TRACKED TARGETS the surrounding prose already names become file-level anchors, and where the prose names the symbol too they become symbol anchors the gate now resolves — `packages/objectql/src/plugin.ts#runGovernanceInventory`, `packages/plugins/plugin-audit/src/audit-writers.ts#resolveWriteLocale`, `scripts/check-system-context-census.mjs#UNENFORCED_TEXT_COUNTS`. A bare `engine.ts` was never ambiguous where the prose said which package. - THIRD-PARTY AND DEPENDENCY SOURCES (`@oclif/core`, `@changesets/pre`) name the package, the file and the pinned line as prose. Nothing in this tree can resolve them, and a pinned upstream line is provenance, not a pointer. - ILLUSTRATIONS take the angle-bracket placeholder spelling. ⛔ Again, no number is repaired or repointed: every digit is still on its page. The residual is 15 citations across six files, and every one of them is a file another lane holds OPEN — three by PR #16215, one by PR #16202, three by the in-flight #15776, one by #12511. NOT ONE is a genuinely ambiguous abbreviation. ⚠️ `judgeUntrackedLineAnchors` therefore stays `false`. The fence this work was dispatched under flips it only when the residual is ZERO and a self-test pins the flip; flipping it over 15 citations in files this PR may not touch would make the gate permanently red for the length of somebody else's pull request — the exact shape the card argued against. `--list-unresolvable` now prints that residual, so the next author inherits a worklist rather than a count. Part of #15809. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vbw3RPgdtqesx4azk9SbW8
1 parent c4b9850 commit 4fafcbd

18 files changed

Lines changed: 73 additions & 37 deletions

scripts/audits/14096-tenancy-ledger-namespace-regexp-census.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,8 @@ const REPO_ROOT = resolve(HERE, '../..');
9494
// ---------------------------------------------------------------------------
9595
// §1. Discover every registered platform-namespace object by AST-lite census
9696
// of `ObjectSchema.create(` calls — the same method the ledger's own
97-
// header (`platform-object-tenancy.ts:32-37`) documents using.
97+
// header (`packages/objectql/src/tenancy/platform-object-tenancy.ts`, its
98+
// opening comment — lines 32-37 as measured) documents using.
9899
// ---------------------------------------------------------------------------
99100

100101
/** Blank out `//` and `/* *\/` comments and string contents (chars only,

scripts/audits/14423-unbound-declarations-before-count.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,8 @@ const loadStandaloneActions = async () => [...namelessStandaloneOrphans, namedSt
5959

6060
const { warnings, logger } = silentLogger();
6161

62-
// Exactly as ObjectQLPlugin.runGovernanceInventory calls it (plugin.ts:2516),
62+
// Exactly as ObjectQLPlugin.runGovernanceInventory calls it
63+
// (`packages/objectql/src/plugin.ts#runGovernanceInventory`),
6364
// with an empty registered-handler set and no object-embedded actions/registry
6465
// rung, so the ENTIRE declaration set comes from `loadStandaloneActions`.
6566
await runActionGovernanceInventory({

scripts/check-adr-links.mjs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,9 @@
1111
// checked by nothing, and records link each other heavily: most carry a
1212
// `**Builds on**:` line with 3-10 relative links in it.
1313
//
14-
// That is not a theoretical hole. `0057-erp-authorization-core-business-units-
15-
// and-scope-depth.md:5` pointed at `./0010-metadata-protection.md`, a file that
14+
// That is not a theoretical hole.
15+
// `docs/adr/0057-erp-authorization-core-business-units-and-scope-depth.md`
16+
// (line 5 as measured) pointed at `./0010-metadata-protection.md`, a file that
1617
// has never existed (the record is `0010-metadata-protection-model.md`). It sat
1718
// there long enough that a triage comment inherited the wrong attribution and
1819
// repeated it. PD #13 sends the next author to "go read that decision"; a dead

scripts/check-changeset-no-major.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -687,7 +687,8 @@ export function render(result) {
687687
// nothing final ships until `changeset pre exit`. Surface the introduced
688688
// majors for the RC curator, but do not fail. The guard re-arms once
689689
// pre-mode exits: `changeset pre exit` rewrites pre.json's mode to `"exit"`
690-
// (@changesets/pre@2.0.2, changesets-pre.cjs.js:117), which is not `pre`.
690+
// (@changesets/pre@2.0.2, its `changesets-pre.cjs.js` bundle, line 117 as
691+
// pinned — a dependency file, outside this tree), which is not `pre`.
691692
case 'exempt':
692693
stdout.push(
693694
`✓ Changesets is in pre-release mode (tag: ${result.tag}) — ` +

scripts/check-cli-test-child-env.mjs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,8 @@
185185
* suite stays green, and the comment claiming the built entry stays plausible.
186186
*
187187
* `@oclif/core@4.13.3` skips its TypeScript path lookup only when `isProd()`,
188-
* which `lib/util/util.js:66` defines as a NEGATED membership test -- a leading
188+
* which its `lib/util/util.js` (line 66 as pinned -- a dependency file, outside
189+
* this tree) defines as a NEGATED membership test -- a leading
189190
* logical-NOT over `['development', 'test'].includes(process.env.NODE_ENV ?? '')`.
190191
* Measured directly against `Config.load()` on `packages/cli`, with
191192
* `packages/cli/dist` present, reading back the root plugin's `commandsDir`:
@@ -1056,7 +1057,8 @@ const BUILT_ENTRYPOINT = 'bin/run.js';
10561057
* The `NODE_ENV` values that send `@oclif/core`'s command lookup to `src/`.
10571058
*
10581059
* `@oclif/core@4.13.3` skips its TypeScript path lookup only when `isProd()`,
1059-
* which `lib/util/util.js:66` defines as a NEGATED membership test:
1060+
* which its `lib/util/util.js` (line 66 as pinned -- a dependency file, outside
1061+
* this tree) defines as a NEGATED membership test:
10601062
* `!['development', 'test'].includes(process.env.NODE_ENV ?? '')`. So these two
10611063
* values -- and only these two -- turn the reroute ON. `production` and unset
10621064
* both leave it off, which is why the rule below refuses a SET of values rather

scripts/check-console-injection.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,8 @@
9999
* esbuild re-emits `"…definition's…"`. Same characters, different bytes, so a
100100
* literal substring search over source text misses it.
101101
* - 19/44 constant-folded concatenation. Source splits a long description as
102-
* `'… declares ' + '`_packageId`.'` (api/protocol.zod.ts:341-342 is one);
102+
* `'… declares ' + '`_packageId`.'` (`packages/spec/src/api/protocol.zod.ts`
103+
* is one, lines 341-342 as measured);
103104
* the bundler folds it to one literal that exists in no source file.
104105
*
105106
* A missed probe here reads as "not expired" — a SILENT PASS, the same failure

scripts/check-dispatcher-error-vocabulary.mjs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -925,7 +925,7 @@ export const OBJECT_LITERAL_CODE_HELPER_BLINDNESS = Object.freeze({
925925
* for the D6 field-addressed validation catalog in `domains/automation.ts`, and
926926
* [#14626]'s two newly reached positions in that same file owe NO further rows:
927927
* they are the identical catalog ternary with the identical values, so they
928-
* derive the site keys the `:1125` instance already derives.
928+
* derive the site keys the instance at line 1125 of that file already derives.
929929
*
930930
* ## [#14742] What moved this census, separated from what merely drifted
931931
*
@@ -1196,7 +1196,8 @@ export const INLINE_LITERAL_EXPRESSION_CENSUS = Object.freeze({
11961196
}),
11971197
// [#14626] The two the card predicted, recovered by the template-literal
11981198
// mode. Same file, same genre, same two values — so they derive the site
1199-
// keys `:1125` already derives and owe NO further verdict rows. That is the
1199+
// keys the instance at line 1125 of that file already derives, and owe NO
1200+
// further verdict rows. That is the
12001201
// measurement, not a convenience: `rowsOwed: 0` here is what says the
12011202
// recovered positions surfaced nothing unregistered.
12021203
Object.freeze({

scripts/check-doc-anchors.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@
1616
// heading lives in one file, the link in another, and until this script nothing
1717
// in CI related them: a rename that updates the heading and not the inbound
1818
// `#anchor` shipped green. #7465 exists because exactly that pairing had already
19-
// drifted once — `cli.mdx:458` pointed at a heading whose text had been correct
19+
// drifted once — `content/docs/deployment/cli.mdx` (line 458 as measured then)
20+
// pointed at a heading whose text had been correct
2021
// in 2024 — and PR #7483 could only catch its own rename by hand, with a slug
2122
// computed in a REPL and a repo-wide grep. Neither runs on anyone else's PR.
2223
//

scripts/check-engine-double-contract.mjs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,9 @@
101101
// NO on #7620 for these doubles specifically. So it asks each independent
102102
// double a behavioural question instead of handing it an implementation.
103103
// - a scoped repository that declares NO repository-only member. Measured on
104-
// the corpus this landed against: `packages/runtime/src/action-body-identity
105-
// .test.ts:71` is a real scoped facade (`createContext().object(name)`)
104+
// the corpus this landed against:
105+
// `packages/runtime/src/action-body-identity.test.ts` (line 71 as measured)
106+
// is a real scoped facade (`createContext().object(name)`)
106107
// spelling only `find`/`count`/`insert`/`update`/`delete`, and it stays in
107108
// the ledger. Seeing it would mean reading its parameter NAMES, and `o` is
108109
// ambiguous in exactly this repo: `o: string` is the object name in twelve

scripts/check-live-db-isolation.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,8 +214,8 @@ const SOLE_INTERPOLATION = /^\$\{\s*([A-Za-z_$][\w$]*)\s*\}$/;
214214
* Source with block and line comments BLANKED -- replaced space-for-space
215215
* rather than deleted, so prose is never a hit and the line numbers this gate
216216
* reports are still the line numbers in the real file. Deleting the comments
217-
* was the first spelling and it reported `…live-mysql.test.ts:45` for a
218-
* statement that lives on line 81; a gate that points at the wrong line is a
217+
* was the first spelling and it reported the live-mysql isolation test at line
218+
* 45 for a statement that lives on line 81; a gate that points at the wrong line is a
219219
* gate the next author stops believing.
220220
*/
221221
export function codeOf(source) {

0 commit comments

Comments
 (0)