docs: refresh weekly fork retirement evidence - #120
Merged
Merged
Conversation
joeshull
commented
Sep 13, 2026
joeshull
left a comment
Author
There was a problem hiding this comment.
Focused governance review of 87f5f9ca7 (Downstream feature, docs-only).
Verdict: PASS. No high-risk findings. No advisory findings. Confidence 0.99.
- Diff limited to
docs/fork-retirement.md; body has exactly oneDownstream feature: trueline, one non-emptyDownstream rationale:line, no conflicting mode or upstream-provenance markers, and the## Weekly resolver reportas the final section. - Evidence spot-checks against fetched refs: upstream
d1b705a0chas zerooptionDetailsand zero plugin-surfaceSvgXml/addMarkdownExtension/MarkdownSource; upstreamcompleteTurnAfterProviderIdleis the described state-only 10 ms loop inproviders/omp/agent.ts; forkc9f842ff6/f88e21fc3retains the optionDetails,acceptsPromptDuringAutonomousTurn, and bounded child-aware gate markers. Quoted SHAs (f88e21fc3,d1b705a0c,bbe56f99d) resolve as described. npm run format:checkpasses on the refreshed file.
This was referenced Sep 13, 2026
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.
Downstream feature: true
Downstream rationale: keeps the fork retirement ledger auditable — refreshed baseline SHAs after the #119 sync, re-verified upstream evidence for all four waiting capabilities at code level, and recorded the sync's union resolution of the plugin markdown surface; the path is not on the governance allowlist.
Weekly maintenance of
docs/fork-retirement.md. No capability status changed: all four remainwaiting. Evidence refreshed against live upstreammainatd1b705a0cand fork integration atf88e21fc3(post-sync).Changes:
upstream-candidate, noretired).origin/internal/mainatf88e21fc3(the Downstream sync: merge live main into internal/main #119 sync merge), mirror/upstream atd1b705a0c; review-period merges docs: refresh weekly fork retirement evidence (2026-09-12) #118 (docs) and Downstream sync: merge live main into internal/main #119 (sync resolution) land no product capability.f22a37e613→d1b705a0c) and verified at code level that none of the upstream commits touch the four capabilities.@types/markdown-it^10.0.3→^14.1.2reconciliation, lockfile hoisting, and Nix-hash refresh. Capability 1 behavior unchanged.SvgXmlimports (e.g.provider-icons.ts) are host-internal rendering, not the plugin surface; zero plugin-surface matches atd1b705a0c.providers/omp/agent.ts), theOmpProviderIdleSchedulerdetail (default 10 ms retry, no budget, no child awareness), and the re-verified interrupt path (interrupt()→terminalizeActiveWork()→subagentIndex.terminalizeRunning(...)still terminalizes running children).f8e7b93da); zerooptionDetailsmatches upstream.agent-manager/claude provider butagent-prompt.tsandagent-run-state.tsremain absent; fix(server): forward steerActiveTurn through wrapSessionProvider getpaseo/paseo#4633 head unchanged; upstreamtask_notification/admission behavior re-verified.Verification:
npm run format:checkclean on the refreshed file; ledger claims re-derived from fetched refs andghPR inspection this run (dates and heads quoted in the document). Docs-only change — no runtime behavior.Weekly resolver report
waitingagainst upstreammainatd1b705a0c(code-level checks, not PR titles alone); no new capability added; no capability promoted toupstream-candidate.head main→base internal/main) foundCONFLICTING; no existingDownstream sync: trueresolver, so Downstream sync: merge live main into internal/main #119 was created and merged (merge commitf88e21fc3, first parent frominternal/main, second parent livemaind1b705a0c); Sync upstream main into internal/main #114 merged automatically afterward. Conflict:packages/app/src/components/message.tsx(upstream fix(app): keep incomplete Markdown formatted while streaming getpaseo/paseo#4742 streaming parser vs fork plugin extensions) resolved as a union; semantically required@types/markdown-it/lockfile/Nix-hash reconciliation included; focused tests (127 app + 77 plugin), typecheck, lint, format green; one playwright flake (transport timeouts, two unrelated specs) passed on re-run.Downstream feature: true); verification = format clean, evidence re-derived from fetched refs and liveghinspection.