Skip to content

fix: pass all spawn fields through SDK client to daemon#403

Merged
khaliqgant merged 1 commit into
mainfrom
fix/sdk-spawn-interactive
Feb 11, 2026
Merged

fix: pass all spawn fields through SDK client to daemon#403
khaliqgant merged 1 commit into
mainfrom
fix/sdk-spawn-interactive

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Feb 11, 2026

Summary

  • The SDK client's spawn() method was only forwarding name, cli, task, model, cwd, and spawnerName to the daemon protocol payload
  • All other SpawnPayload fields (interactive, team, userId, shadowMode, shadowOf, shadowAgent, shadowTriggers, shadowSpeakOn, includeWorkflowConventions) were silently dropped
  • This caused --dangerously-skip-permissions to always be added for Claude agents in cloud workspaces, because interactive: true from the cloud API never reached the bridge spawner — the spawner saw interactive: undefined, treated it as falsy, and added the flag

Test plan

  • Deploy to staging workspace and verify Claude agents spawn without the bypass permissions prompt
  • Verify shadow agent spawning still works (shadowMode, shadowOf, etc. now forwarded)
  • Verify team assignment works for daemon-routed spawns

🤖 Generated with Claude Code


Open with Devin

The SDK client's spawn() method was only forwarding name, cli, task,
model, cwd, and spawnerName to the daemon protocol payload. All other
SpawnPayload fields (interactive, team, userId, shadowMode, etc.) were
silently dropped.

This caused --dangerously-skip-permissions to always be added for
Claude agents in cloud workspaces, because interactive:true from the
cloud API never reached the bridge spawner.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@khaliqgant khaliqgant merged commit f674f70 into main Feb 11, 2026
32 checks passed
@khaliqgant khaliqgant deleted the fix/sdk-spawn-interactive branch February 11, 2026 19:07
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