Skip to content

Fix typo#52

Merged
pedramamini merged 1 commit intoRunMaestro:mainfrom
cclauss:patch-1
Dec 20, 2025
Merged

Fix typo#52
pedramamini merged 1 commit intoRunMaestro:mainfrom
cclauss:patch-1

Conversation

@cclauss
Copy link
Copy Markdown
Contributor

@cclauss cclauss commented Dec 20, 2025

No description provided.

@pedramamini
Copy link
Copy Markdown
Collaborator

Good catches, thank you!

@pedramamini pedramamini reopened this Dec 20, 2025
@pedramamini pedramamini merged commit f0bf158 into RunMaestro:main Dec 20, 2025
@cclauss cclauss deleted the patch-1 branch December 20, 2025 20:04
denniswebb added a commit to denniswebb/Maestro that referenced this pull request Dec 28, 2025
Implement foundation support for GitHub Copilot CLI in Maestro:

- Add 'copilot-cli' to ToolType union in shared/types.ts
- Add copilot-cli agent definition to AGENT_DEFINITIONS with:
  - Prompt mode args (-p), batch mode args (--allow-all-tools -s)
  - Session resume, model selection, and detection paths
- Create CopilotOutputParser implementing AgentOutputParser:
  - Parse plain text output (JSON not yet supported - GitHub issue RunMaestro#52)
  - Extract session IDs and usage statistics
  - Handle error detection from output and exit codes
- Add COPILOT_ERROR_PATTERNS for auth, rate limit, network, and crash errors
- Add copilot-cli capabilities to AGENT_CAPABILITIES
- Update Record<ToolType> usages across codebase:
  - contextGroomer.ts: AGENT_ARTIFACTS, AGENT_TARGET_NOTES, display names
  - contextUsage.ts, usage-aggregator.ts: DEFAULT_CONTEXT_WINDOWS
  - sessionValidation.ts: getProviderDisplayName
- Update tests to expect 8 agents and 4 parsers

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
denniswebb added a commit to denniswebb/Maestro that referenced this pull request Dec 28, 2025
…d limitations

- Added copilot-cli to Available Agents table with Active status
- Documented batch mode flags: -p for batch, --allow-all-tools -s in Auto Run
- Noted JSON output limitation (GitHub issue RunMaestro#52) with plain text parsing fallback
- Listed key features: session resumption, streaming, model selection, MCP config
- Comprehensive test coverage already exists: 95 tests across 4 test files (all passing)
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.

2 participants