Skip to content

docs(catalog): adopt legacy.defining_adr; re-vendor agent-ways #133 doclint#536

Merged
aaronsb merged 1 commit into
mainfrom
docs/adr-908-defining-adr-backport
Jun 20, 2026
Merged

docs(catalog): adopt legacy.defining_adr; re-vendor agent-ways #133 doclint#536
aaronsb merged 1 commit into
mainfrom
docs/adr-908-defining-adr-backport

Conversation

@aaronsb

@aaronsb aaronsb commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Backports agent-ways #133, which un-hardcodes the retired-range scan's
defining-ADR exemption (was a baked-in ADR-900- prefix, now adr.yaml config).
KGS is a vacating repo, so it must declare its defining ADR before re-vendoring —
this adopts the config and the tool together.

  • adr.yaml: legacy.defining_adr: ADR-900 — the ADR that vacated the 1–99 range names retired numbers legitimately, so it's exempt; every other doc/ADR/source stays policed.
  • Re-vendor docs/scripts/{doclint.py,doc} verbatim from canonical (bug: search results not clickable in web UI after z-index fix #133). The canonical tools are now ADR-number-agnostic ("your project's documentation-catalog ADR"), which retires our earlier local citation patch — zero local delta, both byte-identical to canonical.
  • ADR-908: refresh the "Upstream" note (tools no longer hardcode an ADR number) and document the defining_adr exemption.
  • documentation way: note the defining-ADR exemption.

Verified

  • doclint.py --check --enforce-adrs0/0 (49 catalog pages + 120 ADRs).
  • Behavioral test: in-range ADR-042 injected into ADR-900 (defining) → exempt; the same ref in ADR-908 (non-defining) → flagged. The config is load-bearing.
  • Link check clean.

Closes the convergence-backport tail. Coordinated with the agent-ways session (Tamsin) that owns #133.

agent-ways #133 un-hardcodes the retired-range scan's defining-ADR exemption,
moving it from a baked-in `ADR-900-` prefix to `adr.yaml` config. Adopt the
config and re-vendor the tool together so the backport is clean.

- adr.yaml: add `legacy.defining_adr: ADR-900` — the ADR that vacated 1–99 and
  thus names retired numbers legitimately is exempt; everything else stays policed
- re-vendor docs/scripts/{doclint.py,doc} verbatim from canonical (#133). The
  canonical tools are now ADR-number-agnostic, so our prior local citation patch
  on `doc` is gone — zero local delta, both byte-identical to canonical
- ADR-908: refresh the "Upstream" note — tools no longer hardcode an ADR number;
  document the defining_adr exemption
- documentation way: note the defining-ADR exemption

Behavioral test: an in-range ref injected into ADR-900 is exempt; the same ref in
a non-defining ADR is flagged. Lint 0/0 (docs + ADRs enforced).
@aaronsb aaronsb merged commit 6b5aa4b into main Jun 20, 2026
6 checks passed
@aaronsb aaronsb deleted the docs/adr-908-defining-adr-backport branch June 20, 2026 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant