Suppress native installer warning when cc is installed by npm#483
Suppress native installer warning when cc is installed by npm#483bl-ue merged 5 commits intoPiebald-AI:mainfrom
Conversation
📝 WalkthroughWalkthroughAdds a configurable boolean to suppress a native installer startup warning, a UI toggle, a patch writer that removes the warning text, default settings and types updates, and README/CHANGELOG documentation; wires the patch into the patch system and gating logic. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UI as MiscView (UI)
participant Config as Settings
participant PatchRunner as Patch System
participant Writer as writeSuppressNativeInstallerWarning
participant FS as Target File
User->>UI: toggle "Suppress native installer warning"
UI->>Config: update settings.misc.suppressNativeInstallerWarning
PatchRunner->>Config: read settings
alt setting enabled
PatchRunner->>Writer: invoke writeSuppressNativeInstallerWarning(file)
Writer->>FS: remove warning via regex, return modified content
Writer->>PatchRunner: showDiff + return patched content
PatchRunner->>FS: write patched content
else setting disabled
PatchRunner->>PatchRunner: skip patch
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@README.md`:
- Line 120: The table-of-contents entry "[Suppress native installer warning]" is
not indented and thus renders outside the "Features" list; update the README by
adding the same two-space indentation used by the other feature entries so that
the line starting with "[Suppress native installer warning]" is nested under the
"Features" section (match spacing of entries on lines 107–119).
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Idk could be useful, very simple patch
Summary by CodeRabbit
New Features
Documentation