-
Notifications
You must be signed in to change notification settings - Fork 3.5k
feat(cli): persistent opt-out for install hooks #1857
Copy link
Copy link
Open
Labels
editor/integrationEditor compatibility and CLI integrationEditor compatibility and CLI integrationenhancementNew feature or requestNew feature or requestpriority/normalStandard review queue; useful PR with ordinary maintainer urgency.Standard review queue; useful PR with ordinary maintainer urgency.ux/behaviorDisplay bugs, docs, adoption UXDisplay bugs, docs, adoption UX
Description
Activity
Metadata
Metadata
Assignees
Labels
editor/integrationEditor compatibility and CLI integrationEditor compatibility and CLI integrationenhancementNew feature or requestNew feature or requestpriority/normalStandard review queue; useful PR with ordinary maintainer urgency.Standard review queue; useful PR with ordinary maintainer urgency.ux/behaviorDisplay bugs, docs, adoption UXDisplay bugs, docs, adoption UX
Problem
PR #753 identified measurable per-call hook process-spawn overhead. Maintainer direction there was to keep hooks enabled by default, add an opt-out, and persist the user's choice so later installs and updates respect it.
Proposed behavior
install --no-hooksdisables CBM-managed lifecycle/tool hooks.install --hooksre-enables them.installkeeps hooks enabled by default and reads the persisted preference.install_hooksin the existing CLI config database.This follows the maintainer-approved direction in #753.