feat: add a warning message when users enable allow dangerously skip …

[brrock]

…permissions in sudo

Summary by CodeRabbit

• New Features
  • Added security confirmation dialog for the permission bypass setting. When users attempt to enable this sensitive feature, a warning message now appears with explicit guidance. Confirmation requires pressing Enter, while Escape cancels the action.

[coderabbitai]

📝 Walkthrough

Walkthrough

The change enhances the "Allow bypassing permissions in sudo" setting in MiscView by adding a security confirmation workflow. When toggling the setting on, a warning modal now appears requiring explicit confirmation via Enter key before enabling, while toggling off remains immediate. Disabling shows the warning state conditionally.

Changes

Cohort / File(s)	Summary
Security Warning UI for Sudo Permissions src/ui/components/MiscView.tsx	Adds showSudoWarning state to trigger security warning panel when enabling sudo bypass. Modifies toggle behavior: on → shows warning (requires Enter to confirm); off → disables immediately. Implements conditional rendering between warning card and normal item list. Updates input handling to commit setting or cancel based on Enter/Escape keys.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~20 minutes

Possibly related PRs

• Add support for dangerously bypassing permissions in sudo with new setting and patch #478: Introduces the initial allowBypassPermissionsInSudo setting and toggle, which this PR now wraps with a security confirmation workflow.

Suggested reviewers

• bl-ue

Poem

🐰 A warning appears with cautious care,
When sudo's walls might disappear,
Press Enter thrice to cross the gate,
Or Escape if you hesitate,
Security whispers, guards held dear!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.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 describes the main change: adding a warning message when users enable the 'allow dangerously skip permissions' feature in sudo, which matches the file changes and PR objectives.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[brrock]

can i get a review on this please

[bl-ue]

LGTM! Cool dialog. I'm going to make a few modifications to it after I merge it.

[brrock]

LGTM! Cool dialog. I'm going to make a few modifications to it after I merge it.

Ok nice