pdurlej/platform

Fork 0

docs(governance): renumber wloczykij ADR #775

Merged

pdurlej merged 1 commit from codex/issues/730-renumber-wloczykij-adr into main

2026-06-10 21:49:11 +02:00

codex commented

2026-06-10 21:46:02 +02:00

Collaborator

Canary status: missing — fire canary 3+3 manually before merge

Canary Context Pack

Product story

ADR numbers should be unique and stable so agents and humans can follow governance references without resolving the wrong decision. Duplicate ADR-0023 made Wloczykij and Gemini references ambiguous.

What changed

Renamed the Wloczykij ADR from decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md to decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md.
Updated the ADR title/date and added a Renumeration note matching the existing ADR-0010/0011 pattern.
Updated Wloczykij/Things references to ADR-0026.
Corrected two stale ADR-0026 Agent Access Plane references in the M10 closure plan to ADR-0003 Agent Access Plane, because decisions/0003-agent-access-plane.md is the actual accepted Agent Access Plane ADR.

Why it changed

Issue #730 identified two distinct accepted ADR files numbered 0023. While implementing the requested renumber, a repo scan found that assigning Wloczykij to 0026 would otherwise make old ADR-0026 Agent Access Plane text falsely point at Wloczykij.

Files touched

decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md
state/roadmap/wloczykij-gold-vault-implementation-plan.md
modules/obsidian-headless-sync/runbook.md
modules/iskra-things-sync/runbook.md
docs/iskra-things-roadmap.md
state/strategy/m10-closure-plan-2026-05-30.md

Relevant context

Issue #730 scope and acceptance criteria.
ADR-0010/0011 Renumeration note format.
Ollama GLM scout reviewed the hidden ADR-0026 Agent Access Plane coupling and recommended correcting it to ADR-0003 before creating Wloczykij as ADR-0026.
AntiGravity scout was attempted with a sanitized packet but blocked by approval review as private repo context egress; no workaround was used.

Runtime evidence

No runtime mutation. Docs/governance-only change.

Known constraints

Historical wave artifacts under state/waves/2026-06-08-openai-saver-deslop/ still describe issue #730 itself and mention the planned renumber. They are not current inbound ADR references and were left intact.

Explicit out-of-scope

Renumbering future ADRs beyond this duplicate collision.
Editing historical scout transcripts or wave packet artifacts.
Changing runtime, secrets, deploy, or Forgejo policy behavior.

Requested decision

Full-tier review because this touches decisions/ and governance references. Merge if duplicate ADR numbering and reference checks hold.

Merge blockers

Any duplicate numeric ADR files remain under decisions/.
Any current Wloczykij/Things docs still point at ADR-0023.
Any current Agent Access Plane docs still point at ADR-0026.

Spec sources read

Forgejo issue #730: task scope and acceptance criteria.
decisions/0010-cousin-role-taxonomy.md: renumeration note format.
decisions/0011-domain-promotion-policy.md: renumeration note format.
decisions/0023-gemini-3-5-flash-reviewer-cousin.md: kept as ADR-0023.
decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md: renamed target.
state/roadmap/wloczykij-gold-vault-implementation-plan.md: requested inbound ref.
modules/obsidian-headless-sync/runbook.md: requested inbound ref.
modules/iskra-things-sync/runbook.md: requested inbound ref.
docs/iskra-things-roadmap.md: additional hidden Wloczykij/Things inbound ref found by repo scan.
state/strategy/m10-closure-plan-2026-05-30.md: additional hidden stale ADR-0026 Agent Access Plane refs found by repo scan.
decisions/0003-agent-access-plane.md: source proving Agent Access Plane is ADR-0003.

Verification

Duplicate ADR number scan under decisions/ -> no output.
rg --files decisions | rg '(^|/)0023-|(^|/)0026-' -> only Gemini ADR-0023 and Wloczykij ADR-0026.
rg -n -i 'ADR-0026.*Agent Access Plane|Agent Access Plane.*ADR-0026' decisions state modules contexts policies docs AGENTS.md REVIEW.md PLATFORM_CHARTER.md PLATFORM_CONSTITUTION.md -> no matches.
git diff --check -> passed.
test -f decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md -> passed.
test ! -f decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md -> passed.

Closes #730

Canary status: missing — fire canary 3+3 manually before merge ## Canary Context Pack ### Product story ADR numbers should be unique and stable so agents and humans can follow governance references without resolving the wrong decision. Duplicate ADR-0023 made Wloczykij and Gemini references ambiguous. ### What changed - Renamed the Wloczykij ADR from `decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md` to `decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md`. - Updated the ADR title/date and added a Renumeration note matching the existing ADR-0010/0011 pattern. - Updated Wloczykij/Things references to ADR-0026. - Corrected two stale `ADR-0026 Agent Access Plane` references in the M10 closure plan to `ADR-0003 Agent Access Plane`, because `decisions/0003-agent-access-plane.md` is the actual accepted Agent Access Plane ADR. ### Why it changed Issue #730 identified two distinct accepted ADR files numbered 0023. While implementing the requested renumber, a repo scan found that assigning Wloczykij to 0026 would otherwise make old `ADR-0026 Agent Access Plane` text falsely point at Wloczykij. ### Files touched - `decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md` - `state/roadmap/wloczykij-gold-vault-implementation-plan.md` - `modules/obsidian-headless-sync/runbook.md` - `modules/iskra-things-sync/runbook.md` - `docs/iskra-things-roadmap.md` - `state/strategy/m10-closure-plan-2026-05-30.md` ### Relevant context - Issue #730 scope and acceptance criteria. - ADR-0010/0011 Renumeration note format. - Ollama GLM scout reviewed the hidden `ADR-0026 Agent Access Plane` coupling and recommended correcting it to ADR-0003 before creating Wloczykij as ADR-0026. - AntiGravity scout was attempted with a sanitized packet but blocked by approval review as private repo context egress; no workaround was used. ### Runtime evidence No runtime mutation. Docs/governance-only change. ### Known constraints - Historical wave artifacts under `state/waves/2026-06-08-openai-saver-deslop/` still describe issue #730 itself and mention the planned renumber. They are not current inbound ADR references and were left intact. ### Explicit out-of-scope - Renumbering future ADRs beyond this duplicate collision. - Editing historical scout transcripts or wave packet artifacts. - Changing runtime, secrets, deploy, or Forgejo policy behavior. ### Requested decision Full-tier review because this touches `decisions/` and governance references. Merge if duplicate ADR numbering and reference checks hold. ### Merge blockers - Any duplicate numeric ADR files remain under `decisions/`. - Any current Wloczykij/Things docs still point at ADR-0023. - Any current Agent Access Plane docs still point at ADR-0026. ## Spec sources read - Forgejo issue #730: task scope and acceptance criteria. - `decisions/0010-cousin-role-taxonomy.md`: renumeration note format. - `decisions/0011-domain-promotion-policy.md`: renumeration note format. - `decisions/0023-gemini-3-5-flash-reviewer-cousin.md`: kept as ADR-0023. - `decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md`: renamed target. - `state/roadmap/wloczykij-gold-vault-implementation-plan.md`: requested inbound ref. - `modules/obsidian-headless-sync/runbook.md`: requested inbound ref. - `modules/iskra-things-sync/runbook.md`: requested inbound ref. - `docs/iskra-things-roadmap.md`: additional hidden Wloczykij/Things inbound ref found by repo scan. - `state/strategy/m10-closure-plan-2026-05-30.md`: additional hidden stale `ADR-0026 Agent Access Plane` refs found by repo scan. - `decisions/0003-agent-access-plane.md`: source proving Agent Access Plane is ADR-0003. ## Verification - Duplicate ADR number scan under `decisions/` -> no output. - `rg --files decisions | rg '(^|/)0023-|(^|/)0026-'` -> only Gemini ADR-0023 and Wloczykij ADR-0026. - `rg -n -i 'ADR-0026.*Agent Access Plane|Agent Access Plane.*ADR-0026' decisions state modules contexts policies docs AGENTS.md REVIEW.md PLATFORM_CHARTER.md PLATFORM_CONSTITUTION.md` -> no matches. - `git diff --check` -> passed. - `test -f decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md` -> passed. - `test ! -f decisions/0023-wloczykij-gold-vault-and-iskra-promotion-boundary.md` -> passed. Closes #730

codex added the

tier/full

label

2026-06-10 21:46:02 +02:00

codex added 1 commit

2026-06-10 21:46:02 +02:00

docs(governance): renumber wloczykij ADR

canary-required / collect-diff (pull_request) Successful in 4s

Details

platformctl plan / auto-apply scope (pull_request) Successful in 19s

Details

base-is-main / guard (pull_request) Successful in 1s

Details

patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s

Details

canary-required / canary (pull_request) Successful in 14s

Details

patchwarden-client-dry-run / dry-run (pull_request) Successful in 19s

Details

patchwarden-client-dry-run / collect-diff (pull_request) Successful in 3s

Details

patchwarden-pr-sanity / sanity (pull_request) Successful in 2m26s

Details

0f7cfa4b16

pdurlej merged commit 5dfeb2f8e9 into main

2026-06-10 21:49:11 +02:00

pdurlej deleted branch codex/issues/730-renumber-wloczykij-adr

2026-06-10 21:49:11 +02:00

pdurlej referenced this pull request from a commit

2026-06-10 21:49:12 +02:00

Merge pull request 'docs(governance): renumber wloczykij ADR' (#775) from codex/issues/730-renumber-wloczykij-adr into main

patchwarden commented

2026-06-10 21:49:25 +02:00

First-time contributor

Patchwarden PR sanity

Status: advisory_findings
PR: 775
Commit: 0f7cfa4b16e4119da42f00f13f6b09589245d9be
Security-sensitive label: missing
Authority: advisory model review plus deterministic blockers only
3+3 canary: still alive; this does not replace it

Deterministic findings

info sensitive-path-touched Sensitive path touched — decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md
- Evidence: decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md
- Next: Route through the existing 3+3/risk-tier process; model review remains advisory.

Model reviewers

`global-glm` / `glm-5.1:cloud`

Status: ok
Verdict: NOT_OK
high Canary not fired before merge
- Evidence: PR description states 'Canary status: missing — fire canary 3+3 manually before merge' but no evidence canary was executed
- Next: Fire canary 3+3 and document results before proceeding to merge
low Historical wave artifacts retain stale ADR-0023 Wloczykij references
- Evidence: PR description acknowledges 'state/waves/2026-06-08-openai-saver-deslop/' still mentions planned renumber but were left intact as historical artifacts
- Next: Acceptable per explicit out-of-scope declaration, but consider adding a footnote in the ADR renumeration note referencing these historical artifacts for future readers

`global-deepseek` / `deepseek-v4-pro:cloud`

Status: ok
Verdict: OK
Findings: none

`redteam` / `kimi-k2.6:cloud`

Status: ok
Verdict: OK
Findings: none

Policy notes

GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot.
Optional red-team model is enabled only when PLATFORMCTL_PR_SANITY_REDTEAM_MODEL is configured.
Auto-merge is not enabled here.

# Patchwarden PR sanity - Status: `advisory_findings` - PR: `775` - Commit: `0f7cfa4b16e4119da42f00f13f6b09589245d9be` - Security-sensitive label: `missing` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings - **`info` `sensitive-path-touched`** Sensitive path touched — `decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md` - Evidence: `decisions/0026-wloczykij-gold-vault-and-iskra-promotion-boundary.md` - Next: Route through the existing 3+3/risk-tier process; model review remains advisory. ## Model reviewers ### `global-glm` / `glm-5.1:cloud` - Status: `ok` - Verdict: `NOT_OK` - **`high`** Canary not fired before merge - Evidence: `PR description states 'Canary status: missing — fire canary 3+3 manually before merge' but no evidence canary was executed` - Next: Fire canary 3+3 and document results before proceeding to merge - **`low`** Historical wave artifacts retain stale ADR-0023 Wloczykij references - Evidence: `PR description acknowledges 'state/waves/2026-06-08-openai-saver-deslop/' still mentions planned renumber but were left intact as historical artifacts` - Next: Acceptable per explicit out-of-scope declaration, but consider adding a footnote in the ADR renumeration note referencing these historical artifacts for future readers ### `global-deepseek` / `deepseek-v4-pro:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `redteam` / `kimi-k2.6:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ## Policy notes - GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot. - Optional red-team model is enabled only when `PLATFORMCTL_PR_SANITY_REDTEAM_MODEL` is configured. - Auto-merge is not enabled here.