Repository navigation
plan: AF-CTR-04 delivered (complete after the Windows reruns at the post-#71 main); start AF-CTR-05 with its P0 scope - #82
Merged
Conversation
…; widen and start AF-CTR-05 (P0) AF-CTR-04 delivered with every merged PR and merge SHA from GitHub, the Contracts cohorts and the Cloud deploys; the record correction for the files outside AF-CTR-04's paths (#63's tier workflow; the .NET repair PRs named); the acceptance register gains AR-24 (a real node sign-in outside test mode), the mobile part of AR-08 with H-02, and AF-CTR-04 on AR-23. AF-CTR-05 gets the P0 items (fakes, src/internal/dotnet/**, the shared Cloud and ArcForges files, acceptance bullets 1 and 3, the e-mail bound note), the routed items (D3, #71 N-8 to N-12) and its started line; AF-AND-01 carries the emulator checks; AF-PROD-01 the assetlinks rate limit (Cloud #141 B N-8) and the release identity. States and ledger lines by tools/plan.py. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01H6Cf6kHnz1ZkfURhVWvGjP
…on the Windows re-runs after #71 The coordinator's facts: ArcForges #71 merged 0708e1b and #70 merged 4bc33da (both rechecked: no file outside the paths); H3 PASSED on proof on 2026-11-05 (logs/evidence/h3-mobile-2026-11-05.md), its two tooling defects fixed in Cloud #142, nothing left to route. Decision: delivered; complete follows in a later state-only Plan PR once the Windows OS secret-store tier and X item 1 are re-run at ArcForges main 4bc33da or later. Graph, sequence and ledger rebuilt from the base by tools/plan.py (the ledger diff against main is again 7 appended lines); the register rebuilt the same way. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01H6Cf6kHnz1ZkfURhVWvGjP
…efresh limit, H-04, D0's code; the complete condition Review of #82 (APPROVE, 0 blocking): N-1, the delivered line records that acceptance bullet 6's real-browser part ran on proof (H3), with the local closed loop in Go tests only; N-2, N2-7 maps to bullet 4 (only its self-hosted center contacted) and X's limit that a refresh's success is inferred is stated; N-4, H-04 and AF-PROD-01's human_prerequisites name the upload certificate fingerprint of the asset links, and AR-08's section says the prerequisites, not AR-08, block AF-PROD-01's complete; N-5, AF-CTR-05 bullet 3 says D0 adds STATE/node_offline. The complete condition names the Windows tier (PASSED at 4bc33da, run 42061577006), X item 1 (PASSED at 4bc33da) and the keys check. Rebuilt from the base by tools/plan.py; the ledger diff against main stays 7 appended lines. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01H6Cf6kHnz1ZkfURhVWvGjP
Contributor
Author
|
Review record for Plan #82 (AF-CTR-04 delivered; AF-CTR-05 started with P0). Independent review at
Review round at
Legs at Generated by Claude Code |
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.
This PR records AF-CTR-04 as
delivered, widens AF-CTR-05's task record before its work starts (execution-policy §1.6, part P0 of the reviewed AF-CTR-05 plan) and starts AF-CTR-05. It also records the deferred real-world validations of AF-CTR-04 in the acceptance register and two hand-overs (AF-AND-01, AF-PROD-01). It changes graph content, so it is reviewed like a task (execution-policy §4). States,sequence.mdand every ledger line are written bytools/plan.py.AF-CTR-04 ->
deliveredgh api repos/ArcForges/<repo>/pulls/<n> --jq .merge_commit_sha):a5bc52f), #126 (.NET 10.0.14 repair,1e0a5d6), #127 (CM,5873dc2).602705f; already cited on AF-CTR-03's complete line as an AF-CTR-04 repair PR), #130 (E1a,2aaac73), #132 (E1b,3e2cf6b), #133 (.NET repair,b1c8c0d), #134 (E2,940cb97), #135 (E3,2686a73), #136 (vectors,41c8a72), #137 (the /tmp leak and the scratch helper,9085b12), #138 (the CI proof gate,98e7389), #140 (the center lane,3a13b97), #141 (M,c837860), #142 (M follow-up and D2,34d9e60).e105508), docs: record MR-99 durable attempt phase repair #63 (N1,fb9901c), docs: record MR-17 memory-notification recovery repair #69 (N2,978d25d), docs: remove hardcoded reviewer model requirements #71 (Windows-safe CloudLink private files and the sign-in reservation race fixes,0708e1b), docs: record MR-18 verification and stop after this item #70 (CloudSignInTests: real refreshes and the delivered center keys,4bc33da).7466181), Record maintenance handoff and renewed serial repair authorization #48 (D1,5da245e). Plan plan: [AF-CTR-04] Scope: the prerequisites of the first gated operation move to AF-CTR-05 #80 (99b7bc6), plan: [AF-CTR-04] Scope: MobileProbe as an allowed fake; AF-AND-01 redirect wording #81 (af0e36e): in the ledger line only (scope records).gh run view: success at the merge SHA):1.0.0-ci.424.1(run 41724651206),1.0.0-ci.426.1(42005833954),1.0.0-ci.430.1(42016941693).0.1.0-ci.517.1to0.1.0-ci.563.1. #132's own main push (41774844734) failed in the image build on GHSA-p3jp-9v83-76w9 and deployed nothing; #133 deployed both.0.1.0-ci.518.1to0.1.0-ci.558.1(since #138 the main-push artifact); surfacesidentity,pages,operations,fleet,mobile.deliveredledger line (as the evidence files state them):logs/evidence/x-ctr04-2026-11-05/), N2-7, the keys rerun (keys-rerun/), H1 (logs/evidence/e3-h1-2026-11-04.md), H3 (logs/evidence/h3-mobile-2026-11-05.md, Cloud #141 comment 6660944407: PASSED on proof, all four mobile cases, assetlinks, the callback, Google's Digital Asset Links check and the regression; its two tooling defects were fixed in Cloud #142), the Go integration tests on real PostgreSQL, the 120 mobile vectors.978d25d).completefollows in a later state-only Plan PR recording the re-runs at ArcForges main4bc33daor later: the Windows tier (PASSED at4bc33da, run 42061577006), X item 1 (PASSED at4bc33da,logs/evidence/x-ctr04-2026-11-05/rerun-4bc33da/) and the keys check.Record correction (ledger note on AF-CTR-04)
Computed from GitHub's file list of every PR against AF-CTR-04's
repositories[].paths:.github/workflows/os-secret-store-tier.yml(described in its body). No Plan record widened the paths beforehand; the note records it now.global.jsonand thepackages.lock.jsonof eightsrc/internal/dotnet/packages (the body counts the lock files without naming them); Cloud #133 and ArcForges maintenance: record MR-95 recovery repair and MR-12 publication result #61global.json.Acceptance register
--test-mode(OS opener, Credential Manager, CNG key, logout); §3 section with the procedure and the Credential Manager status of X (REAL in the hosted tier run 41994995309; the owner's local opt-in not run).human_prerequisitesgains H-04.acceptance_register: AR-08, AR-23, AR-24.AF-CTR-05 widened and started (P0)
fakes_allowed(was empty): the nodetest channel client and the FakeCenter channel peer (Q3); also the virtual WebAuthn authenticator and IsolatedSecretStore, as AF-CTR-04's list has them.src/internal/dotnet/**(Q4); Cloud.gitleaks.toml,.dockerignore,biome.json,tsconfig.json,deploy/**and ArcForges.github/**(the files AF-CTR-03's and AF-CTR-04's PRs changed outside their paths).STATE/node_offline(added by part D0; Q2).public/http/v1documents stay at 320 (Q12).activewith astartedline naming the part order P0, D0, C1, E0, C2, E1a, E1b, E2, N1, N2a, N2b, N3, E3, X, D.AF-AND-01, AF-PROD-01
/.well-known/assetlinks.jsonmust not refuse Google's or Android's verifiers (Cloud #141 review B N-8); the release identity (H-02, with H-04's upload key).Review round
Independent review at
0891264(plan82-verdict.md): APPROVE, 0 blocking, 6 non-blocking. Taken in a new commit, rebuilt from the base with the same scripts:human_prerequisitesname the upload certificate fingerprint of the asset links; AR-08's section 3 now says these prerequisites blockcomplete, not delivery, of AF-PROD-01, and that AR-08 itself blocks nothing.STATE/node_offline(added to ArcForges-Design errors.md by part D0)".Checks
python3 tools/plan.py validate --design <ArcForges-Design main 5da245e>: 0 errors; 63 tasks, 17 milestones.python3 <Design>/tools/check_design.py --plan delivery-graph.json: 0 errors, 3 notes.python3 -m unittest discover -s tests: 48 tests OK (QUIET-HOST absent).git diff --check: clean.sequence.mdand the ledger were written bytools/plan.pyonly (rebuilt from the base;build.pyandregister.pyin the coordinator's scratch directory).🤖 Generated with Claude Code
https://claude.ai/code/session_01H6Cf6kHnz1ZkfURhVWvGjP