docs(adr): accept ADR-0044 - #253
Merged
Merged
Conversation
The frontmatter change is the maintainer's own `adr accept 0044 --by @mbeacom` run (decidedAt 2026-10-01T02:01:16Z), carried over unchanged from the main checkout: status, provenance.ratifiedBy, and review.decidedAt, and no other line. It is the command's first real use and its rung-1 functional evidence (item 6). Item 7: the four proposed-state qualifiers it lists now read accepted: three in AGENTS.md and one in site/src/content/docs/commands.mdx. The CHANGELOG's Unreleased entry carried a fifth that item 7 missed. It has not shipped yet, so it would have been released saying "proposed"; it is updated too. MANIFEST.md regenerated by `bun run emit:manifest`. `adr queue` is now empty. Signed-off-by: Mark Beacom <m@beacom.dev>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The acceptance metadata, inventory totals, action items, and public qualifiers are consistent.
Review effort: Balanced
Findings: None
What changed in this PR
Accepts ADR-0044 and synchronizes its status across public documentation and generated inventory.
Changes:
- Records ratification metadata and completes ADR-0044 action items.
- Updates all public status qualifiers and regenerates the manifest.
| File | Description |
|---|---|
AGENTS.md |
Marks ADR-0044 accepted. |
CHANGELOG.md |
Updates the unreleased status qualifier. |
MANIFEST.md |
Regenerates ADR status counts and inventory. |
docs/adr/0044-…md |
Records acceptance and ratification evidence. |
site/src/content/docs/commands.mdx |
Updates command documentation status. |
Decisions governing this change
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
Accepts ADR-0044. I ratified it with the command it introduces,
adr accept 0044 --by @mbeacom. That is the command's first real use, and the evidence item 6 asks for.The command changed exactly the three fields it owns and no other line of the frontmatter.
Item 7: the four proposed-state qualifiers it lists now read accepted:
AGENTS.md(theadr queue--formatbullet, theadr acceptsection, and the write-surface sentence);site/src/content/docs/commands.mdx.The
CHANGELOG.mdUnreleased entry carried a fifth that item 7 didn't list. It hasn't shipped, so it would have been released saying "proposed"; it reads accepted now too.MANIFEST.mdis regenerated withbun run emit:manifest, andadr queueis now empty.Checklist
packages/ci/dist: n/a (docs only)adr lint(44 records),check:stale-refs, and thescripts/suite pass.Notes for reviewers
With items 6 and 7 done here, every ADR-0044 action item is ticked. Items 1–5 were verified against the tree before #250 merged.