docs: reconcile landed repairs and current acceptance evidence - #43
Merged
Merged
Conversation
Refs #31. Distinguish independently landed Coven fixes, verified draft evidence isolation, successful native observations and unresolved startup/human gates. Align remediation statuses with Beads without closing them. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Val Alexander <val@opencoven.ai>
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The diff is documentation-only and the updated ledger/status language is internally consistent with the surrounding evidence sections and stated boundaries.
Pull request overview
Updates the Phase 5 delivery/acceptance ledger in coven-threads to more precisely separate (a) draft downstream integration evidence from (b) fixes that have landed on downstream main, while keeping Phase 5’s outstanding remediation gates clearly marked as still blocked pending reviewed integration and acceptance.
Changes:
- Refines the “as of” Phase 5 status summary to reflect the latest recorded draft checkpoint while explicitly preserving unresolved Windows startup causation.
- Reclassifies the four Phase 5 remediation beads (
threads-okc,threads-980,threads-dgg,threads-zav) as blocked on reviewed integration, and ties them to the shared downstream draft integration checkpoint. - Adds an explicit “independently landed production repairs” section documenting downstream PRs that are now on
covenmain, while clarifying they do not close the Phase 5 remediation gates.
File summaries
| File | Description |
|---|---|
| docs/phases.md | Reconciles Phase 5 documentation to distinguish landed downstream repairs from draft integration evidence and to align remediation-gate status language with current acceptance blocking state. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Objective
Refs #31; bead threads-8pz.22. Update only docs/phases.md, separating landed OpenCoven/coven#998 and OpenCoven/coven#1003 from draft integration, operational evidence isolation and unresolved startup causation. Align four remediation rows with their current blocked-on-reviewed-integration status.
Evidence and boundaries
Consulted agent/manifest.yaml, existing delivery ledger, live Beads statuses, exact downstream PR/merge records, native CI34489425644 logs and artifact10157780426. Latest native target20/20, including15 daemon journeys and5 artifact regressions;28 exact clean-merge manifests. Older failures remain explicitly preserved. No normative spec, code, authority, migration, compatibility pin, or human gate changes.
Validation and rollback
Documentation-only diff; git diff --check passes. All cited new PR/run/manifest facts were retrieved directly. No code tests required. Revert this explanatory documentation commit to roll back; full acceptance and human decisions remain open.