Skip to content

feat(mcp): wire availableStrategies into parse_mode tool response#724

Merged
JeremyDev87 merged 6 commits intomasterfrom
feat/available-strategies-wire-710
Mar 20, 2026
Merged

feat(mcp): wire availableStrategies into parse_mode tool response#724
JeremyDev87 merged 6 commits intomasterfrom
feat/available-strategies-wire-710

Conversation

@JeremyDev87
Copy link
Owner

Summary

  • Verify availableStrategies and taskmaestroInstallHint are included in parse_mode MCP tool response
  • No production code changes needed — mode.handler.ts already uses ...result spread from ParseModeResult
  • Add 2 verification tests confirming the fields pass through correctly

Test plan

  • 2 new tests: availableStrategies array passthrough, taskmaestroInstallHint passthrough
  • All 4830 existing tests pass
  • Full CI: lint, format, typecheck, test:coverage, circular, build — all pass

Closes #710

- Add TaskmaestroAssignment and TaskmaestroDispatch interfaces
- Extend DispatchAgentsInput with executionStrategy field
- Extend DispatchResult with taskmaestro and executionStrategy fields
- Extend ParseModeResult with availableStrategies and taskmaestroInstallHint
- Add type tests for new interfaces

Closes #706
…tegies

- Add taskmaestro-detector.ts utility with isTaskmaestroAvailable()
- Wire detection into buildParseModeResult() as step 10
- Populate availableStrategies and taskmaestroInstallHint in ParseModeResult
- Add 2 tests for skill present/absent scenarios

Closes #707
- Add executionStrategy enum ('subagent' | 'taskmaestro') to inputSchema
- Extract and default to 'subagent' in handleDispatchAgents handler
- Add 3 tests for subagent, taskmaestro, and default scenarios

Closes #708
- Add taskmaestro branch in dispatchAgents() for executionStrategy === 'taskmaestro'
- Build per-agent prompts with task description and target files context
- Return TaskmaestroDispatch with sessionName, paneCount, and assignments
- Add buildTaskmaestroPrompt and buildTaskmaestroHint helper methods
- Set executionStrategy: 'subagent' on existing return path
- Add 6 tests covering all taskmaestro dispatch scenarios

Closes #709
- Verify availableStrategies is included in parse_mode response via ...result spread
- Add 2 tests: availableStrategies array and taskmaestroInstallHint passthrough

Closes #710
@vercel
Copy link

vercel bot commented Mar 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
codingbuddy-landing Ready Ready Preview, Comment Mar 20, 2026 7:59am

@JeremyDev87 JeremyDev87 merged commit a332016 into master Mar 20, 2026
25 checks passed
@JeremyDev87 JeremyDev87 deleted the feat/available-strategies-wire-710 branch March 20, 2026 08:02
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.

feat(mcp): wire availableStrategies into parse_mode tool response

1 participant