Skip to content

docs(adr): date ADR-0019 brood-manifest literals as snapshots (#375) - #380

Merged
brenpike merged 2 commits into
mainfrom
docs/375-adr-0019-dated-snapshot-rule
Sep 25, 2026
Merged

brenpike merged 2 commits into
mainfrom
docs/375-adr-0019-dated-snapshot-rule

Conversation

@brenpike

Copy link
Copy Markdown
Owner

Summary

Closes #375.

ADR-0019's 2026-06-01 brood-manifest amendment ("JSON manifest root fix") still states two facts that no longer hold: the manifest pinned at manifest_version: 3, and a jq read in spawn-brood's liveness guard, which has since been removed.

This PR applies the same pointer-form approach used for ADR-0018 in #376:

  • Inline callout under the 2026-06-01 amendment heading. It flags that amendment as superseded in part and names the owning live docs. The original wording is untouched.
  • New 2026-09-25 amendment at the end of the ADR. It:
    • names the two stale literals as superseded, without restating any current value;
    • states a standing rule: every path, schema version, count, membership list, and named mechanism in any note or amendment of this ADR is a dated snapshot, not authority;
    • explicitly exempts owner pointers from that rule, so the rule does not cancel its own pointers;
    • points brood state layout and manifest schema at ADR-0021 and plugin/references/brood-ledger-model.md, and the inputs-file transport at plugin/governance/security-policy.md ("Inert Inputs-File Navigator Pattern").

Append-only: 14 insertions, 0 deletions.

Scope decisions

Validation

  • bash tools/validate.sh --changed: PASS. policy_check.sh --strict passed 72 of 73 checks, with 29 allowlisted findings and 0 new (baseline).
  • Local Codex review (gpt-5.5): approve, 0 findings.

Versioning

No version bump. This is a docs-only change outside plugin/, so the plugin stays at 4.1.0.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 51e46a4d50

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/adr/0019-trust-boundary-discipline-ledger-engine.md Outdated
Codex P2: the new amendment named ADR-0021 as the live owner of the
manifest reader mechanism, but ADR-0021 is itself append-only and dated
and its reader-mechanism section is stale, so readers were sent to two
mutually exclusive current contracts.

Removes ADR-0021 from every live-source pointer this amendment adds and
states the eliminated class in item 3: no ADR is ever named as a live
source, because an ADR is append-only and dated and its content is a
snapshot by this amendment's own standing rule. A live source is a
maintained document edited in place when the thing it describes changes.

No live value is added; all pointers stay pointer-form.
@brenpike
brenpike merged commit 0fff951 into main Sep 25, 2026
1 check passed
@brenpike
brenpike deleted the docs/375-adr-0019-dated-snapshot-rule branch September 25, 2026 18:59
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.

ADR-0019: brood-manifest amendment still names manifest_version 3 and the removed liveness guard

1 participant