docs: record batch two triage fixes - #1501
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
@clawsweeper re-review Documentation-only head The linked code PRs contain their real runtime proof. Land this PR only after #1483, #1446, #1382, and #1499. CI for this exact head: https://github.com/openclaw/clawsweeper/actions/runs/34248097685. No release, tag or deployment has been performed. |
|
🦞👀 Re-review progress:
|
|
@clawsweeper re-review The prior review request could not reach review because main's early sparse checkout hid the setup-pnpm local action. #1498 has now reverted that regression. Please review this unchanged prepared head and current body; the runtime proof and clean independent review are already recorded there. This requests review only. |
|
🦞👀 Re-review progress:
|
|
Codex review: needs changes before merge. Reviewed September 8, 2026, 12:18 PM ET / 16:18 UTC. ClawSweeper reviewWhat this changesUpdates the Unreleased highlights and adds four credited notes covering paired-close checks, repair timeouts, and review-revision display. Merge readiness⛔ Needs changes before merge - 1 item remains The release-note contribution remains useful and has no actionable patch defects. The updated documentation proof resolves the prior proof concern; the four implementation dependencies remain open. Priority: P3 Review scores
Verification
How this fits togetherClawSweeper’s changelog collects operator-visible changes for release readers. This PR documents four separately reviewed implementations without changing automation or OpenClaw Bay. flowchart LR
A[Four implementation PRs] --> B[Confirm implementations landed]
B --> C[Unreleased changelog]
C --> D[Markdown rendering]
D --> E[Release readers]
Before merge
Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Keep the consolidated, credited release notes aligned with the four landed implementations while preserving released history. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR edits release notes rather than repairing runtime behavior; its exact diff and recorded rendering results are directly inspectable. Is this the best way to solve the issue? Yes: consolidating the batch notes avoids competing changelog edits, provided the documented implementations land first. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 2690dafa8c33. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
History |
|
@clawsweeper re-review The main body now contains after-change documentation proof: native Git supplied exact-head CHANGELOG.md to markdown-it 15.0.1, producing standalone before/after HTML. The Unreleased heading and all four contributor-credited list items render intact; the complete released suffix is byte-identical to the parent. A merge-tree check proves this PR does not restore the reverted retry note. Exact-head CI passed: https://github.com/openclaw/clawsweeper/actions/runs/34248097685. Local and branch autoreviews remain clean. This is proof of the changed documentation artifact; the four linked runtime fixes still must land first. No implementation, release, or deployment is claimed by these notes. |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
🦞👀 Re-review progress:
|
What Problem This Solves
Collect the second triage batch's release notes in one PR so the independently prepared code PRs do not conflict in CHANGELOG.md.
Changes
Update Highlights and record the paired-close repair, both repair timeout fixes, and review-revision display, retaining contributor credit. Released sections are unchanged.
Land after these code PRs:
The queue-retry entry already on main is not edited here. Its revert and corrected re-land are owned by #1498 and that repair lane.
Evidence
git diff --checkandpnpm run check:docspass. Local and committed-branch Codex autoreview are clean through P2. Exact head:a8fa43d0c538978a457aa7999eedcb2f446a613f; CI: https://github.com/openclaw/clawsweeper/actions/runs/34248097685. This is documentation-only; executable runtime proofs belong to the linked code PRs.pnpm outdated --format jsonfound only the coordinated Oxc/tsgolint tooling refresh already tracked in #1318 and newer Node typings outside the Node 24 runtime floor. No additional dependency bump is included. The 2,880-minute release-age policy is retained.Release and OpenClaw Bay Impact
These batch changes fit a patch. This PR does not authorize a tag, release, publication or deployment. Release preparation remains gated on the remaining autonomous queue and restoration of exact-review workflows after #1497. No Bay data or observer contract changes.
Real Behavior Proof: Rendered Documentation
The exact committed changelog was rendered with the installed
markdown-it15.0.1, the repository's documentation parser, on Node 24.20.0. This exercises the changed documentation artifact; runtime behavior remains proved in the four linked code PRs.Reproduction from an installed checkout:
Observed after rendering: the heading is
<h2>0.3.1 - Unreleased</h2>. All four added notes render as list items with intact contributor credit: paired-close revalidation (@vincentkoc), dispatch/clone deadlines (@SebTardif), selector deadlines (@SebTardif), and revision labels (@elijahfriedman). No note text is lost or interpreted as markup. Comparing the entire released suffix starting at## 0.3.0 -with parent9da27d076a542b06751a879eace475ccc134fbffis byte-identical.Before/after standalone HTML files were generated locally. A separate Git merge-tree check against the post-revert main is conflict-free and does not restore the removed queue-retry entry. Exact-head CI passed: https://github.com/openclaw/clawsweeper/actions/runs/34248097685.
Limits: this proves the release-note artifact renders correctly; it does not claim the four implementation PRs have already merged. The explicit notes-last landing order remains required. No public website deployment, release or publication occurred.