Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

improvement: add allowed-tools configuration UI for Slash Command export#431

Merged
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom
improvement/424-allowed-tools-configbreaking-brake/cc-wf-studio:improvement/424-allowed-tools-configCopy head branch name to clipboard
Jan 12, 2026
Merged

improvement: add allowed-tools configuration UI for Slash Command export#431
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom
improvement/424-allowed-tools-configbreaking-brake/cc-wf-studio:improvement/424-allowed-tools-configCopy head branch name to clipboard

Conversation

@breaking-brake
Copy link
Copy Markdown
Owner

@breaking-brake breaking-brake commented Jan 12, 2026

Problem

The allowed-tools setting in Slash Command exports was hardcoded as Task,AskUserQuestion, with no way for users to customize which tools are allowed without permission prompts.

Current Behavior

  1. Export Slash Command
  2. allowed-tools: Task,AskUserQuestion is always included in frontmatter

Expected Behavior

  1. Export Slash Command
  2. allowed-tools is only included if explicitly configured by user
  3. ✅ Users can select which tools to include via UI

Solution

Added allowedTools configuration to SlashCommandOptions with full UI support.

Changes

Type Definition (src/shared/types/workflow-definition.ts)

  • Added allowedTools?: string property to SlashCommandOptions interface

Store (src/webview/src/stores/workflow-store.ts)

  • Added setSlashCommandAllowedTools action

UI (src/webview/src/components/toolbar/SlashCommandOptionsDropdown.tsx)

  • Added "Allowed Tools" sub-menu with checkbox selection
  • Uses AVAILABLE_TOOLS (18 tools) from refinement-store
  • Includes "Reset to Default" option
  • Reordered dropdown menu: Allowed Tools → Model → Hooks → Context
  • Moved description tooltips to bottom of each submenu for consistency
  • Unified submenu minWidth: 180px for consistent left-side display

Export Service (src/extension/services/export-service.ts)

  • Changed from hardcoded value to conditional output
  • Only outputs allowed-tools if explicitly configured

Serialization (src/webview/src/services/workflow-service.ts)

  • Added allowedTools to workflow serialization logic

i18n Refinements

  • Use static English text for technical terms (Allowed Tools, default, Reset to Default)
  • Use static text for hooks display (none, X hooks)
  • Removed unnecessary tooltips (Model, Allowed Tools)
  • Removed unused translation keys (toolbar.model.tooltip, toolbar.allowedTools.*, hooks.entryCount, hooks.noEntries)

Impact

  • Users can now configure which tools are allowed without permission prompts
  • No allowed-tools output when not configured (uses Claude Code default behavior)
  • Improved UI consistency: all submenus open on the same side (left)
  • Reduced cognitive load with consistent tooltip placement
  • Non-breaking change for existing workflows

Testing

  • Build passes (npm run build)
  • Lint passes (npm run lint && npm run check)
  • Manual E2E testing completed
    • Allowed Tools sub-menu appears in SlashCommandOptions dropdown
    • Tool selection works via checkboxes
    • Export includes allowed-tools only when configured
    • Export omits allowed-tools when not configured
    • Workflow save/load preserves allowedTools setting
    • All submenus display consistently on the left side

Related

🤖 Generated with Claude Code

@breaking-brake breaking-brake force-pushed the improvement/424-allowed-tools-config branch from 462385c to 75f3fb3 Compare January 12, 2026 14:30
- Add allowedTools property to SlashCommandOptions interface
- Add Allowed Tools submenu with tool checkboxes and reset button
- Use static English text for technical terms (Allowed Tools, default, Reset to Default)
- Remove unnecessary tooltips (Model, Allowed Tools)
- Use static text for hooks display (none, X hooks)
- Reorder dropdown: Allowed Tools first, Context last
- Move description tooltips to bottom of submenus for consistency
- Unify submenu minWidth to 180px for consistent left-side display
- Remove unused translation keys

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@breaking-brake breaking-brake force-pushed the improvement/424-allowed-tools-config branch from 75f3fb3 to 86b8f47 Compare January 12, 2026 14:33
@breaking-brake breaking-brake merged commit 0add0c3 into main Jan 12, 2026
3 checks passed
@breaking-brake breaking-brake deleted the improvement/424-allowed-tools-config branch January 12, 2026 14:35
@breaking-brake breaking-brake mentioned this pull request Jan 12, 2026
braking-brake-semantic-release Bot pushed a commit that referenced this pull request Jan 12, 2026
## [3.14.2](v3.14.1...v3.14.2) (2026-01-12)

### Improvements

* add allowed-tools configuration UI for Slash Command export ([#431](#431)) ([0add0c3](0add0c3))
@braking-brake-semantic-release
Copy link
Copy Markdown

🎉 This PR is included in version 3.14.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add allowed-tools configuration to slashCommandOptions

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.