docs/providers/postgres.md cites code by line number in 18 places. Nothing measures those numbers, so a citation is true only until the next insertion above it, and nothing goes red when it stops being true. A method name is greppable and survives an insertion, so it needs no correction round at all.
Measured 2026-09-06. Of the 8 citations that name a method beside the line, 8 point at the wrong line today:
escapeIdentifier() cited :33, declared at :49shouldEnableSSL() cited :75, declared at :72getDefaultSchema() cited :91, declared at :88
Four files: postgres.ts (12), sql-base.ts (4), query-limiter.ts (1), factory.ts (1). Note that two of the misses point backwards - the numbers were not merely outgrown, they were wrong when written.
Fix. Rewrite each citation to name the declaration instead of its coordinate, keeping the link to the file:
`getSchema()` ([`postgres.ts`](../../src/lib/db/providers/sql/postgres.ts))
docs/providers/mongodb.md (#581), mssql.md, oracle.md and trino.md are already in this shape and are the model to copy. Then add the doc to the NAMED_CITATIONS list in tests/unit/provider-docs-monitoring-citations.test.ts, with src/lib/db/providers/sql/postgres.ts as its source and the method names the doc now uses. That list's two tests then assert the doc carries no .ts:<line> anywhere and that every name it cites is really declared, so this cannot rot again. Read that file's header comment first: it explains the policy and why it is pinned rather than the coordinates.
Done when grep -oE '\.ts:[0-9]+' docs/providers/postgres.md returns nothing, the doc appears in NAMED_CITATIONS, and the suite is green. One doc plus one list entry, no product code.
Tracked in docs/BACKLOG.md as DOC4.
Curated for Hacktoberfest 2026. Comment to claim it before you start so two people do not work on the same issue. A PR must reference this issue and land with its tests in the same change; see CONTRIBUTING.md. Repo rules that apply: run bun run test (never bare bun test), and the 100% line-coverage gate must stay green. Local bun run test also needs the helm binary on PATH for unrelated chart tests - see #570 if it bites.
docs/providers/postgres.mdcites code by line number in 18 places. Nothing measures those numbers, so a citation is true only until the next insertion above it, and nothing goes red when it stops being true. A method name is greppable and survives an insertion, so it needs no correction round at all.Measured 2026-09-06. Of the 8 citations that name a method beside the line, 8 point at the wrong line today:
escapeIdentifier()cited:33, declared at:49shouldEnableSSL()cited:75, declared at:72getDefaultSchema()cited:91, declared at:88Four files:
postgres.ts(12),sql-base.ts(4),query-limiter.ts(1),factory.ts(1). Note that two of the misses point backwards - the numbers were not merely outgrown, they were wrong when written.Fix. Rewrite each citation to name the declaration instead of its coordinate, keeping the link to the file:
docs/providers/mongodb.md(#581),mssql.md,oracle.mdandtrino.mdare already in this shape and are the model to copy. Then add the doc to theNAMED_CITATIONSlist intests/unit/provider-docs-monitoring-citations.test.ts, withsrc/lib/db/providers/sql/postgres.tsas its source and the method names the doc now uses. That list's two tests then assert the doc carries no.ts:<line>anywhere and that every name it cites is really declared, so this cannot rot again. Read that file's header comment first: it explains the policy and why it is pinned rather than the coordinates.Done when
grep -oE '\.ts:[0-9]+' docs/providers/postgres.mdreturns nothing, the doc appears inNAMED_CITATIONS, and the suite is green. One doc plus one list entry, no product code.Tracked in
docs/BACKLOG.mdas DOC4.Curated for Hacktoberfest 2026. Comment to claim it before you start so two people do not work on the same issue. A PR must reference this issue and land with its tests in the same change; see CONTRIBUTING.md. Repo rules that apply: run
bun run test(never barebun test), and the 100% line-coverage gate must stay green. Localbun run testalso needs thehelmbinary on PATH for unrelated chart tests - see #570 if it bites.