Skip to content

docs: address #166 review findings on ADR-016/020 mirror - #174

Open
christian-byrne wants to merge 1 commit into
mainfrom
docs/adr-016-020-mirror-followup
Open

docs: address #166 review findings on ADR-016/020 mirror#174
christian-byrne wants to merge 1 commit into
mainfrom
docs/adr-016-020-mirror-followup

Conversation

@christian-byrne

Copy link
Copy Markdown
Contributor

Docs-only follow-up addressing every non-blocking finding in the approve-with-comments review on #166 (review id 5095287625, @benceruleanlu, head 981785de).

Changes

  1. ADR-016 amendment miscite fixed. The references cited docs/multiplayer-schema.md Amendment A15 for DQ-11; the enacted incarnation-namespaced widget stamps amendment is A16 (A15 governs promoted subgraph host writes). Now cites A16 with a deep link and a one-line disambiguation.
  2. Unresolvable workspace-relative references fixed. ADR-016 (program/decision-queue.md, program/dq-11-primer.md, reports/spikes/spike-5-merge-suite.md, plus the inline spike/conflict-matrix paths) and ADR-020 (ADR-007-op-based-crdt-v1.md — which does not exist in this repo and shadowed the cmp ADR-007, ../reports/spikes/dq10-lamport-feasibility.md, ../AGENTS.md) now link to the source paths in christian-byrne/in-app-agent-program, and both ADRs carry a mirror note stating that program/… and reports/… paths are workspace-repo paths.
  3. ADR-020 supersession note added. A short mirror note records that ADR-021 (accepted) supersedes and narrows this ADR, while preserving the historical Proposed status and verification gate as record; current decision state is ADR-021 with KA-2, FC-2, FC-9.

Evidence

  • Review findings verified against main before this PR: ADR-016 still cited "Amendment A15" (schema ## Amendment A16 — 2026-08-28 — DQ-11 incarnation-namespaced widget stamps is the enacted one); grep '](\.\./' over both ADRs returned hits pre-change, zero post-change; ADR-020 Status line unchanged since docs: mirror workspace ADR-016 and ADR-020 (op-layer seams) #166 with no ADR-021 cross-reference.
  • All rewritten workspace links point at paths verified to exist on the workspace repo's main (git ls-tree origin/main).
  • Docs-only diff: 2 files, no code paths touched.
  • Post-change grep over both ADRs: no remaining ](../ links, no remaining bare "Amendment A15" citation in ADR-016.

- ADR-016: cite schema Amendment A16 (DQ-11 incarnation-namespaced widget
  stamps) instead of A15, which governs promoted subgraph host writes.
- ADR-016/020: replace workspace-relative paths that do not resolve in this
  repository with deep links to christian-byrne/in-app-agent-program and add
  a mirror note explaining the workspace path convention.
- ADR-020: add a supersession mirror note pointing to the accepted ADR-021
  (narrows this ADR) while preserving the historical Proposed record.

Follow-up to #166 (approve-with-comments review by @benceruleanlu).
@christian-byrne christian-byrne added documentation Improvements or additions to documentation risk:R1 PR risk grade (advisory shadow check; grader-owned) labels Sep 3, 2026
@christian-byrne christian-byrne added documentation Improvements or additions to documentation risk:R1 PR risk grade (advisory shadow check; grader-owned) labels Sep 3, 2026
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 128 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 0f3eb04d-b1eb-4983-b553-a8af4afc9920

📥 Commits

Reviewing files that changed from the base of the PR and between 84030a4 and b88a79e.

📒 Files selected for processing (2)
  • docs/decisions/ADR-016-dq11-incarnation-namespaced-stamps.md
  • docs/decisions/ADR-020-lamport-vs-vector-vs-base-version.md

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation risk:R1 PR risk grade (advisory shadow check; grader-owned)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants