chore(deps): refresh compatible Swift packages and checkout action - #3487
Conversation
Update compatible Commander, Sparkle, swift-log and swift-asn1 releases, and pin checkout 7.0.1. Keep the existing major-version constraints. Sparkle includes installer hardening. Collect the timestamp fix from #3485 in the single notes PR; land that independent fix first.
|
🦞👀 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 maintainer review before merge. Reviewed September 7, 2026, 11:03 PM ET / September 8, 2026, 03:03 UTC. ClawSweeper reviewWhat this changesUpdates four existing Swift packages and nine checkout-action pins across three workflows while preserving dependency sources and major-version constraints. Merge readiness✅ Ready for maintainer review Still useful and ready for ordinary maintainer consideration: current main retains the older dependencies, and this review found no actionable defect. The recorded runtime validation remains applicable to the reviewed head. Priority: P2 Review scores
Verification
How this fits togetherCodexBar uses Swift packages for command parsing, logging, cryptography, and native app updates. Its GitHub workflows check out the repository to test and package the app and CLI. flowchart TD
A[Package manifest and lockfile] --> B[Swift package resolution]
B --> C[CLI parsing and shared services]
B --> D[Native app updater]
E[Pinned checkout action] --> F[CI and packaging workflows]
F --> G[Validated app and CLI artifacts]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Land the scoped dependency refresh in the owner's stated sequence, keeping release notes in #3488 and publication checks in the release process. Do we have a high-confidence way to reproduce the issue? Not applicable: this is dependency maintenance, with recorded CLI and native integration checks rather than a reported CodexBar failure. Is this the best way to solve the issue? Yes: refreshing the existing manifest floors and resolved pins is a focused approach, with major migrations kept separate and no competing implementation added. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 9810f24b0609. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
History |
|
Validation for
Live proof: the freshly built CLI executes Exact-head CI passed every required job on 7d2fe40, including both macOS shards, all three Linux builds, lint and the aggregate. This dependency PR is land-ready. Land #3485, this dependency PR, and then the separate final notes PR #3488. This is dependency maintenance and release-note preparation, not authorization to tag or publish. |
Update the existing compatible Swift dependencies: Commander 0.2.2 → 0.2.4, Sparkle 2.9.3 → 2.9.6, swift-log 1.13.2 → 1.15.0, and swift-asn1 1.7.1 → 1.7.2. Update all nine checkout action pins from 7.0.0 to 7.0.1. Manifest floors and lockfile agree; the current major-version constraints stay intact.
Sparkle 2.9.6 includes installer archive-moving and package-signature validation hardening. The remaining updates are compatible maintenance releases. KeyboardShortcuts 3 and Swift Crypto 4 remain separate migration work.
Release notes are collected only in #3488, after #3485 and this PR. This branch changes only the package manifest, lockfile and three workflow pins; it does not change CHANGELOG.md, a version or publication state.
Validation: the production, test, package and workflow tree is byte-for-byte identical to tested commit 3315604; the sole follow-up change moves changelog text to #3488. That tested tree passed all 1,031 selections / 86 groups without retries or timeouts, make check with zero violations, the built CLI 1,100-token/$0.0045 fixture, and Developer ID-signed app startup/Settings with mapped Sparkle 2.9.6. Independent P0–P2 review is clean. The final exact-head CI passes all jobs, including both macOS shards, all three Linux builds, lint and aggregate. This PR is land-ready.