Repository navigation
chore: prepare 0.16.3 release - #250
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 SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex review: needs maintainer review before merge. What this changesPrepare version 0.16.3 with dated release notes, backup-fix highlights, and a new 0.16.4 Unreleased section. Example: Read the package version and release notes.
Review scores
ProductKind: Maintenance · Worth it: Yes · Fix scope: Complete Merge readiness✅ Ready for maintainer review Keep open: this owner-authored release PR is focused, follows the established release pattern, and introduces no actionable defect. Priority: P3 Before mergeNone. FindingsNone. Agent review detailsHow this fits togetherThe package manifest supplies CLI and publication version metadata, while the release preflight extracts the matching dated changelog section for release artifacts. flowchart LR
A[Package manifest] --> C[Release preflight]
B[Dated changelog] --> C
C --> D[Package build and smoke]
D --> E[Release artifacts]
E --> F[Signed tag publication]
Technical reviewBest possible solution: Land the focused release preparation through the existing release process. Do we have a high-confidence way to reproduce the issue? This is release preparation rather than a bug report; source inspection confirms the intended metadata changes without executing target code. Is this the best way to solve the issue? The two-file patch is the smallest complete release preparation and preserves the established preflight and publication owners. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 639ed3cf61b8. Provenance checked
TestingProof path: unit tests only. SecurityNone. EvidenceWhat I checked:
Likely related people:
LabelsLabel changes:
Label justifications:
Rating scale6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior. WorkflowClawSweeper edits this one comment on every review. Comment Reviewed October 24, 2026, 6:56 PM ET / 22:56 UTC. |
Prepare 0.16.3: date the changelog section, add highlights, bump
package.json, and restore a numberedUnreleasedsection.Includes #248 (backup merge-import folds legacy profile rows into canonical profiles; backup validation ignores regular macOS
.DS_Storemetadata). Verified before landing #248 by merge-importing a real production backup into a copy of the production database with.DS_Storepresent: validation and import succeeded, no legacy rows or references remained, integrity and foreign-key checks clean.Release-preflight unit tests pass locally; the full preflight runs in the release workflow dry run on
main.