docs(changelog): collect Claude menu and Bedrock disclosure fixes - #3499
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 26781dd590
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| ### Fixed | ||
| - Antigravity: render each CLI quota bucket once, keep unknown or disabled usage unavailable in text and full/brief cards, and apply shared idle-family visibility (#3489). Thanks @urda! | ||
| - AWS Bedrock: disclose monitoring charges in both authentication modes, link current Cost Explorer pricing, and explain the shared refresh controls and informational budget (#3496, related to #3387). Thanks @kyen99! |
There was a problem hiding this comment.
Land the Bedrock disclosure before advertising it
In the reviewed tree, BedrockProviderImplementation.settingsPickers and settingsFields still contain only the authentication and credential descriptions, with no monitoring-charge disclosure or pricing link. The repo-wide search finds those details only in docs/bedrock.md, which was already recorded under 0.56.4, so releasing 0.56.9 from this tree would either repeat an old documentation change or claim the absent #3496 in-app fix; land that implementation first or omit this entry.
Useful? React with 👍 / 👎.
| ### Fixed | ||
| - Antigravity: render each CLI quota bucket once, keep unknown or disabled usage unavailable in text and full/brief cards, and apply shared idle-family visibility (#3489). Thanks @urda! | ||
| - AWS Bedrock: disclose monitoring charges in both authentication modes, link current Cost Explorer pricing, and explain the shared refresh controls and informational budget (#3496, related to #3387). Thanks @kyen99! | ||
| - Claude: honor the segmented multi-account menu preference for claude-swap, preserve source-owned switching and unavailable-account diagnostics, and show stable slot numbers when personal information is hidden (#3498, #3382). Thanks @thatlev! |
There was a problem hiding this comment.
Land the Claude menu fix before advertising it
When claude-swap has at least four accounts, StatusItemController+ClaudeSwapMenu.swift still selects compactAccountPlan without consulting settings.multiAccountMenuLayout, while PersonalInfoRedactor.redactEmail still turns an email label into an empty string rather than a stable slot label. Thus the reviewed tree does not provide either the promised segmented-preference behavior or privacy-safe slot numbering; land #3498 before adding its 0.56.9 release note.
Useful? React with 👍 / 👎.
|
Codex review: blocked before merge. Reviewed September 8, 2026, 4:14 AM ET / 08:14 UTC. ClawSweeper reviewWhat this changesAdd Unreleased notes for Bedrock monitoring-charge disclosures and Claude account-menu fixes, and consolidate the duplicate Antigravity entry. Merge readiness⛔ Blocked before merge - 2 items remain Keep open: this is a useful, narrowly scoped notes update whose two implementation prerequisites remain unmerged. The owner-authored PR is also protected from automatic closure. Priority: P3 Review scores
Verification
How this fits togetherCodexBar’s changelog collects user-facing changes for upcoming releases. Release tooling turns the selected version section into GitHub release notes and Sparkle update notes. flowchart LR
A[Provider fixes] --> B[Unreleased changelog]
B --> C[Release preparation]
C --> D[GitHub release notes]
C --> E[Sparkle update notes]
Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Keep one accurate Unreleased entry per change, synchronized with the implementations that actually land. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes release-note prose; the relevant checks are wording, historical preservation, and implementation landing status. Is this the best way to solve the issue? Yes: a small consolidated changelog update is appropriate, provided the author’s explicit implementation-first ordering is honored. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against b1e27d266a8e. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
|
Verified notes head 855f136. CI is green: https://github.com/steipete/CodexBar/actions/runs/34202919824. Independent branch review is clean through P2. The diff against current main is CHANGELOG.md only. Released sections are byte-for-byte unchanged. The notes retain the concurrently landed #3486 entries, add this batch's two fixes, and consolidate the duplicated #3489 entry without dropping behavior or contributor credit. The two review comments correctly identify the dependency order: merge #3496 and #3498 first, then this notes PR. Its independent branch intentionally does not carry the sibling implementations, in accordance with this sweep's one-final-notes-PR rule. No release is being published from this branch. |
Collect the release notes for #3496 (Bedrock monitoring charges) and #3498 (Claude segmented account menus) in one independent notes PR. Consolidate the duplicated pending Antigravity #3489 entry, retaining its quota, reset, filtering, and contributor details.
Land after both code PRs. Released sections are byte-for-byte unchanged, and the existing 0.56.9 Unreleased version remains. Dependency and action pins were checked against upstream tags; no new compatible update was found. KeyboardShortcuts 3 and Swift Crypto 4 remain separate migration work.
Validation: released-history comparison, duplicate-reference check, diff checks, and independent review through P2. Runtime and full-suite proof belong to the linked code PRs. This PR does not authorize a release or publish artifacts.