Skip to content

Remove interactive apply in favor of more detailed --apply#434

Merged
bl-ue merged 3 commits intomainfrom
remove-interactive-apply
Jan 29, 2026
Merged

Remove interactive apply in favor of more detailed --apply#434
bl-ue merged 3 commits intomainfrom
remove-interactive-apply

Conversation

@bl-ue
Copy link
Member

@bl-ue bl-ue commented Jan 29, 2026 •
edited by coderabbitai bot
Loading

This PR removes the interactive *Apply customizations option in favor of --apply, which as of #433 is more detailed. --apply is likely used much more than interactive apply in the majority of cases.

Summary by CodeRabbit

  • New Features

    • Added CLI invocation detection to display the correct command for applying changes.
    • Added notification banner prompting users to use the --apply flag to apply pending changes.

  • Bug Fixes

    • Removed interactive apply option from main menu, consolidating patch application workflow to explicit --apply CLI flag.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026 •
edited
Loading

Warning

Rate limit exceeded

@bl-ue has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 30 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

This PR removes the interactive apply menu option and replaces it with a directed CLI approach. The changes detect how the CLI was invoked, pass this context through the app, and display a banner suggesting users explicitly use the --apply flag instead of interactively applying changes.

Changes

Cohort / File(s) Summary
Documentation
CHANGELOG.md		 Added entry documenting removal of interactive apply in favor of explicit --apply CLI option.
CLI Invocation Detection
src/index.tsx		 Added getInvocationCommand helper to detect how CLI was invoked (npx/pnpm/yarn/bun, etc.); passes detected command to App component; updated patch display message to note unchanged patches.
Type System Updates
src/types.ts		 Removed APPLY_CHANGES member from MainMenuItem enum, eliminating the "*Apply customizations" menu option.
UI Components
src/ui/App.tsx, src/ui/components/MainMenu.tsx, src/ui/components/MainView.tsx		 Removed interactive apply logic and applyCustomization imports from App; removed SettingsContext dependency from MainMenu; added invocationCommand and changesApplied props flowing to MainView; MainView now displays info banner suggesting --apply flag when changes haven't been applied.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 Hop along with CLI flags bright,
No more menus, just --apply in sight!
The bunny commands with precision and care,
Explicit and clear, the changes laid bare. 🚩

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: removing the interactive apply feature in favor of the --apply flag, which matches the primary modifications across the codebase.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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 and usage tips.

coderabbitai[bot]
coderabbitai bot requested changes Jan 29, 2026
View reviewed changes
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents 
In `@src/index.tsx`:
- Around line 93-95: The console.log message in src/index.tsx contains a typo:
change the string '"\nPatches applied (run with --show-unchaned to show all
patches):"' used in the top-level console.log to use the correct flag name
'--show-unchanged' so the message reads '"\nPatches applied (run with
--show-unchanged to show all patches):"'; locate the console.log call and update
the misspelled word only.

@bl-ue bl-ue merged commit 4aade7f into main Jan 29, 2026
2 checks passed
@bl-ue bl-ue deleted the remove-interactive-apply branch January 29, 2026 16:29
This was referenced Jan 30, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bl-ue