chore: consolidate pending notes and refresh dependency patches - #470
Conversation
Collect the complete ordered Unreleased notes and contributor credit for the prepared browser/model repairs. Update Sweet Cookie and TokenTally within the existing two-day release-age and Node 24 policies. Merge after the implementation PRs; no version bump or publication.
|
🦞👀 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: blocked before merge. Reviewed September 7, 2026, 10:22 PM ET / September 8, 2026, 02:22 UTC. ClawSweeper reviewWhat this changesConsolidates Unreleased notes and contributor credits, and updates Sweet Cookie to 0.4.3 and TokenTally to 0.1.5. Merge readiness⛔ Blocked before merge - 3 items remain Keep open: this is useful, owner-directed release preparation with no identified patch defect. The explicit prerequisite merge order still applies; current main and v0.19.0 retain the older dependencies. Priority: P3 Review scores
Verification
How this fits togetherOracle uses Sweet Cookie to read browser cookies and TokenTally to calculate consultation costs. Its manifest and lockfile select these runtime dependencies, while the changelog describes user-facing changes. flowchart TD
A[Package manifest] --> B[Locked dependency versions]
B --> C[Browser cookie reading]
B --> D[Consultation cost calculation]
E[Implementation PRs] --> F[Unreleased notes]
F --> G[User release information]
Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Keep the dependency refresh synchronized and publish only Unreleased entries whose implementations have landed, retaining contributor credit. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR prepares release notes and dependency updates rather than reporting a reproducible defect. Is this the best way to solve the issue? Yes: the bounded manifest/lockfile refresh and consolidated notes fit the existing repository pattern, provided the stated implementation sequence is respected. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against fe2c8d990562. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
|
Final-head proof for
Merge only after #428, #465, #468, #469, and #444. Adjust the notes if any implementation is held. No version, tag, release, or publication action is included. |
Collect the complete ordered Unreleased notes and contributor credit for the prepared model, Web Search, browser recovery, Deep Research plan, and localization changes. Merge this PR after the implementation PRs.
Update Sweet Cookie 0.4.2 → 0.4.3 and TokenTally 0.1.4 → 0.1.5 consistently in the manifest and lockfile. Both releases were more than 48 hours old when installed; Node >=24 and the two-day release-age policy remain intact. A fresh outdated check reports no remaining updates.
Validation: build and check passed; the full suite passed 2,230 tests / 45 skipped after building the CLI. The built CLI with the updated dependencies completed a real API request and returned
ORACLE_DEPS_OK. Local and final branch P0–P2 autoreview are clean. Exact-head CI is green: https://github.com/steipete/oracle/actions/runs/34178931052.No version bump, tag, release, or publication is included. The implementation entries describe the proposed merged state and must not land ahead of their code.