Skip to content

fix(init): hide ambient plugin from multiselect#150

Merged
dean0x merged 1 commit intomainfrom
fix/hide-ambient-from-multiselect
Mar 19, 2026
Merged

fix(init): hide ambient plugin from multiselect#150
dean0x merged 1 commit intomainfrom
fix/hide-ambient-from-multiselect

Conversation

@dean0x
Copy link
Owner

@dean0x dean0x commented Mar 19, 2026

Summary

  • Filter devflow-ambient from the plugin multiselect and preSelected defaults (follows devflow-core-skills precedent)
  • Auto-include devflow-ambient when the "Enable ambient mode?" prompt is answered yes
  • Removes redundancy: the ambient prompt now controls both hook installation AND plugin inclusion

Context

The init wizard previously showed devflow-ambient in the plugin multiselect (pre-selected) AND had a separate "Enable ambient mode?" prompt. Since /ambient was removed and ambient is hook-only, the plugin's sole purpose is distributing skills/agents the hook needs — it shouldn't be a user-facing selection.

Test plan

  • npm run build passes
  • npm test — all 250 tests pass
  • Manual: node dist/cli.js init in TTY — confirm ambient NOT in plugin list, ambient prompt still appears
  • Manual: node dist/cli.js init --no-ambient — confirm ambient plugin NOT installed
  • Manual: node dist/cli.js init --ambient — confirm ambient plugin IS installed (auto-included)

…nt prompt

The init wizard showed devflow-ambient in the plugin multiselect (pre-selected)
AND had a separate "Enable ambient mode?" prompt, which was redundant. Since the
/ambient command was removed (ambient is hook-only now), the plugin's only purpose
is distributing skills/agents the hook needs.

Now follows the devflow-core-skills precedent: filtered from multiselect, auto-
included when the ambient prompt answer is yes.
@dean0x dean0x merged commit 2a22e58 into main Mar 19, 2026
3 checks passed
@dean0x dean0x deleted the fix/hide-ambient-from-multiselect branch March 19, 2026 23:02
dean0x pushed a commit that referenced this pull request Mar 20, 2026
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.

1 participant