chore: release 0.8.1 - #216
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs real behavior proof before merge. Reviewed September 13, 2026, 9:39 PM ET / September 14, 2026, 01:39 UTC (Revision 2). ClawSweeper reviewWhat this changesPrepares version 0.8.1 with dated release notes and adds three bounded retries when removing temporary test repositories. Merge readiness⛔ Blocked before merge - 2 items remain The release preparation remains necessary: current main and the latest release still identify 0.8.0. No actionable patch defect was found, but the previously requested packaged-version proof remains outstanding. Priority: P3 Review scores
Verification
How this fits togetherClawpatch packages its code-review CLI for npm; package metadata supplies the CLI version and is checked against the release tag before publishing. Shared test fixtures create temporary repositories and remove them after each test. flowchart TD
A[Package version] --> B[CLI version output]
A --> C[Release metadata checks]
D[Dated changelog] --> C
C --> E[npm release]
F[Temporary test repositories] --> G[Cleanup with bounded retries]
Before merge
Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Ship consistent 0.8.1 package metadata and release notes through the existing validated release workflow, with bounded fixture cleanup remaining test-only. Do we have a high-confidence way to reproduce the issue? Not applicable to the release metadata update. The reported cleanup race was assessed from source and supplied CI context; it was not independently reproduced. Is this the best way to solve the issue? Yes: updating the existing release metadata and using the filesystem API's bounded cleanup retries is a narrow solution without introducing a competing release or cleanup mechanism. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 0885c9372069. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
Prepare clawpatch 0.8.1 by updating the package version and finalizing the September 13 changelog. The patch fixes nested-project repair scoping, retains provider edits after failed patch attempts, and includes the doctor, deadline, provider-validation, tooling, and documentation fixes already on main.
A Node 24 CI run also exposed a fixture teardown race: removing a temporary Git repository returned ENOTEMPTY for .git/objects/pack. Shared fixture cleanup now uses three bounded filesystem retries; persistent errors still fail the test. This changes test cleanup only.
The release will use the existing signed-tag workflow and npm trusted publishing after CI passes on the merged release commit. Independent review of the complete candidate found no actionable P0–P2 findings.
Validation: the initial local gate passed frozen install, typecheck, lint, formatting, all 957 tests (two platform skips) with coverage, build, package-content checks, packaged CLI smoke (13 features, including three CUDA), and website smoke. The full local gate passed again with the cleanup fix: 957 tests passed, two platform skips, 89.45% line coverage, build, package smoke, and website smoke. All GitHub checks passed on fd0fd10, including Node 22/24/26, Windows, CodeQL, dependency review, and secret scanning.