Repository navigation
Trim 0.64.0 changelog to shipped features - #3646
Conversation
Drop Feed, Vault, Cloud VMs, and Dock from the 0.64.0 changelog and docs highlights since they are not ready to ship. Keep session restore on quit (the part that did ship) as a top-line highlight. Update the release skill to require explicit user confirmation that each user-facing item actually shipped before drafting the changelog, and to keep the curated highlights in changelog-media.ts in sync with that scope.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR updates release documentation for v0.64.0: edits ChangesRelease documentation & changelog updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 13✅ Passed checks (13 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR trims the 0.64.0 changelog to only features that actually shipped by removing Feed, Vault, Cloud VMs, and Dock from both
Confidence Score: 5/5Documentation-only change with no app or runtime impact; safe to merge. All three files are documentation and tooling (changelog copy, docs media metadata, and a release-skill prompt). No Swift, TypeScript runtime, or data-layer code is touched. Both issues flagged by earlier review passes — the orphaned sessions-sidebar bullet and the orphaned Sessions panel CPU-loop fix — are correctly resolved in this diff. The remaining CHANGELOG.md content is internally consistent with changelog-media.ts. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Collect merged PRs] --> B[List candidate user-facing items]
B --> C{Confirm shipped scope\nwith user}
C -->|Feature NOT shipped| D[Drop from CHANGELOG.md\nDrop from changelog-media.ts]
C -->|Feature shipped| E[Write CHANGELOG.md bullets\nAdded / Changed / Fixed / Removed]
E --> F[Mirror curated highlights\nin changelog-media.ts]
F --> G[Bump version metadata]
G --> H[Tag & publish]
Reviews (3): Last reviewed commit: "Address review feedback on changelog tri..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| @@ -102,7 +97,6 @@ All notable changes to cmux are documented here. | |||
| - Select find text on repeated Cmd+F ([#3314](https://github.com/manaflow-ai/cmux/pull/3314)) | |||
| - Search Codex rollout content from the sessions sidebar ([#3396](https://github.com/manaflow-ai/cmux/pull/3396)) | |||
There was a problem hiding this comment.
This Changed bullet references functionality inside a "sessions sidebar" — the same conceptual surface that was just dropped under the Vault/Sessions panel removal. The parallel removal was applied to "Route Codex permission approvals through Feed" (Feed not shipping → entry removed), but the equivalent "Search Codex rollout content from the sessions sidebar" was left in. If PR #3396's feature depends on a sidebar panel that isn't in the shipped build, users will see a changelog claim they cannot exercise.
There was a problem hiding this comment.
Fixed - dropped the 'Search Codex rollout content from the sessions sidebar' bullet from 0.64.0 Changed and the 'Fix Sessions panel CPU loop' bullet from Fixed (kept only the unrelated cancel-drag-on-Escape part), since the Sessions panel/Vault surface is not shipping in 0.64.0.
— Claude Code
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/release/SKILL.md`:
- Around line 28-32: The rule in SKILL.md is contradictory: it currently forbids
editing a separate docs changelog file while later requiring updates to
web/app/[locale]/docs/changelog/changelog-media.ts. Edit the sentence that
begins "Do not edit a separate docs changelog file;
`web/app/docs/changelog/page.tsx` renders from `CHANGELOG.md`" to remove the
prohibition and make a single clear workflow (e.g., state that CHANGELOG.md is
the source of truth for the page but contributors must also mirror curated
highlights into web/app/[locale]/docs/changelog/changelog-media.ts when
applicable), and ensure the files `CHANGELOG.md`,
web/app/docs/changelog/page.tsx, and
web/app/[locale]/docs/changelog/changelog-media.ts are referenced consistently.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: eec3214c-ef18-4160-bf80-75269046a5ac
📒 Files selected for processing (3)
CHANGELOG.mdskills/release/SKILL.mdweb/app/[locale]/docs/changelog/changelog-media.ts
- Resolve contradictory rule in skills/release/SKILL.md: drop the "do not edit a separate docs changelog file" line and explicitly describe the two-file split (CHANGELOG.md for bullets, changelog- media.ts for hero title and feature highlights). - Drop the "Search Codex rollout content from the sessions sidebar" entry from 0.64.0 Changed since the sessions sidebar surface was pulled with Vault. - Drop the Sessions panel CPU loop fix from 0.64.0 Fixed for the same reason, keeping only the unrelated cancel-drag-on-Escape part.
This branch was successfully deployed
1 participant
Summary
CHANGELOG.mdand from the docs highlights inweb/app/[locale]/docs/changelog/changelog-media.ts. Those features are not ready to ship.releaseskill to require explicit user confirmation that each user-facing item actually shipped before drafting the changelog, and to keepchangelog-media.tsin sync with that scope.Testing
Related
Summary by cubic
Trims the 0.64.0 changelog/docs to only shipped features and promotes session restore on quit. Adds a release check to avoid preannouncing unshipped work and clarifies the docs vs. changelog split.
CHANGELOG.md: remove Feed, Vault, Cloud VMs, Dock and dependents (beta toggles, Feed routing, sessions sidebar search, Vault CPU loop fix); keep “Session Restore on Quit,” rewrite to stand alone, include Gemini and Rovo Dev, and note red X close restores panes/sessions.web/app/[locale]/docs/changelog/changelog-media.ts: retitle 0.64.0 highlights, drop Feed/Vault/Cloud VMs/Dock, and keep “Session Restore on Quit” with the red X detail.skills/release/SKILL.md: define the two-file split—CHANGELOG.mdis the source for bullets;changelog-media.tssupplies the hero title and highlights—and require explicit user confirmation of shipped scope; keep both files in sync and include only shipped, user-visible changes.Written for commit 83e6b41. Summary will update on new commits.
Summary by CodeRabbit
New Features
Documentation
Changed
Chores