Skip to content

fix(review,work): omit mode parameter in subagent dispatch to respect user permissions#522

Merged
tmchow merged 2 commits intomainfrom
tmchow/fix-agent-mode-override
Apr 6, 2026
Merged

fix(review,work): omit mode parameter in subagent dispatch to respect user permissions#522
tmchow merged 2 commits intomainfrom
tmchow/fix-agent-mode-override

Conversation

@tmchow
Copy link
Copy Markdown
Collaborator

@tmchow tmchow commented Apr 6, 2026

Summary

Two fixes for permission/sandbox issues when dispatching subagents:

1. Omit mode parameter in subagent dispatch

Claude Code will sometimes automatically add mode: "auto" to Agent tool calls even when not asked to. This overrides the user's permission settings (e.g., bypassPermissions configured in ~/.claude/settings.json), forcing interactive permission prompts on every subagent. This significantly impacts the experience — parallel background agents get blocked waiting for approval, slowing execution substantially because it then forces abandoning subagents to do it in the main session. It also impacts harnesses like Conductor that rely on user-level permission settings flowing through to subagents.

The fix: instruct all agent-dispatching skills to explicitly omit the mode parameter so the user's configured defaultMode applies naturally.

2. Remove version lookup from git-commit-push-pr badge

The jq pre-resolution call against ${CLAUDE_PLUGIN_ROOT}/.claude-plugin/plugin.json fails in sandboxed environments (e.g., Conductor) where the plugin install path is outside the allowed working directory. Replaced with a single static badge.

Files changed

  • AGENTS.md — New "Sub-Agent Permission Mode" section in Skill Compliance Checklist
  • ce-work, ce-work-beta — Permission mode note in subagent dispatch section
  • ce-review — Note in Stage 4 Spawning
  • document-review — Inline note in dispatch instructions
  • ce-plan/deepening-workflow — Inline note at agent launch
  • ce-compound-refresh — Added to "when spawning any subagent" section
  • ce-ideate — Inline note at ideation sub-agent dispatch
  • orchestrating-swarms — Note after Task tool example
  • git-commit-push-pr — Removed jq version lookup, static badge

Test plan

  • Verify subagent dispatch no longer passes mode: "auto"
  • Verify user-level bypassPermissions setting is respected by dispatched subagents
  • Verify git-commit-push-pr works in sandboxed environments without the jq error

tmchow and others added 2 commits April 6, 2026 12:00
… user permissions

Passing mode: "auto" on Agent tool calls overrides user-level permission
settings like bypassPermissions. Added instructions across all
agent-dispatching skills to omit the mode parameter so the user's
configured defaultMode applies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…environments

Drop the pre-resolved `jq ... plugin.json` version lookup and
versioned/versionless badge branching. Replace with a single static
badge. The jq call fails in sandboxed environments (e.g., Conductor)
where CLAUDE_PLUGIN_ROOT resolves outside the allowed working directory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tmchow tmchow merged commit 949bdef into main Apr 6, 2026
2 checks passed
@github-actions github-actions bot mentioned this pull request Apr 6, 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