Skip to content

Add diff comment exports and lifecycle events - #14577

Open
austinywang wants to merge 7 commits into
mainfrom
issue-9508-cli-events-surface-for
Open

austinywang wants to merge 7 commits into
mainfrom
issue-9508-cli-events-surface-for

Conversation

@austinywang

@austinywang austinywang commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Agents can export every saved diff comment with cmux comments export --format json|markdown and watch comment.created, comment.updated, comment.deleted, and comment.consumed through cmux events --category comment. This completes the pull workflow started by #9604.

Events come from the shared DiffCommentStore, covering the viewer and TextBox delivery/dismissal, and redact message text. Export reuses the read-only comments.list method; no remote relay access is added. The CLI and event contracts are documented, with help/errors localized across all nine supported locales, including English and Japanese.

Validation: all 11 static preflight checks passed (python3 scripts/verify-local.py --swift-changed origin/main --timeout 300), including localization parity and Swift test wiring. Added store lifecycle/replay tests and built-CLI export tests. Hosted regression proof, final CI, and tagged-app E2E are pending; fleet access currently returns HTTP 401, and tagged backend provisioning hit its 320-instance capacity limit before a build could be submitted.

Closes #9508 (#9508).

Mergeability: head 6e3f14383bc441b9581e50a1f81f628fa30c6c7c; main merged without conflicts. The hosted conflict-only gate is awaiting GitHub API rate-limit recovery.

— MapleQuorum9508 (registration pending)
run: run_9508_6fd0bf358625; session: 01a0d83b-60a4-71a3-a1b5-982f88ebb119

@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 22 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 65ff4f62-043a-4fb3-9ec5-2b6068069c03

📥 Commits

Reviewing files that changed from the base of the PR and between e196ab6 and d28ddd7.

📒 Files selected for processing (9)
  • CLI/CMUXCLI+Comments.swift
  • CLI/CMUXCLI+TaskHelp.swift
  • Resources/Localizable.xcstrings
  • Sources/DiffCommentStore.swift
  • cmuxTests/DiffCommentStoreTests.swift
  • docs/cli-contract.md
  • docs/events.md
  • tests/test-execution.toml
  • tests/test_cli_comments_export.py

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@teamleaderleo

Copy link
Copy Markdown
Collaborator

This remains the follow-up for #9508; the current head conflicts with main, so it is left open for a refresh.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI + events surface for diff review comments (agent-driven collection)

2 participants