docs(agents): configure Sol Terra and Luna - #2730
Open
CarlosCastro77 wants to merge 2 commits into
Open
CarlosCastro77 wants to merge 2 commits into
CarlosCastro77 wants to merge 2 commits into
Conversation
Co-authored-by: Copilot App <[email protected]>
Contributor
Reviewer's GuideConfigures Sol, Terra, and Luna as specialized GitHub Copilot custom agents and extends the root AGENTS.md with their responsibilities, handoff workflow, validation expectations, and conflict-resolution rules. Flow diagram for the Sol Terra Luna agent workflowflowchart LR
Sol[Sol: read-only planning] --> Terra[Terra: implementation and validation]
Terra --> Luna[Luna: independent review]
Luna -->|APPROVE| Complete[Complete]
Luna -->|REQUEST CHANGES| Terra
Luna -->|BLOCKED| Blocked[Document blocker]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path=".github/agents/luna.agent.md" line_range="4" />
<code_context>
+---
+name: Luna
+description: Independently reviews changes, reports severity-ranked findings, and issues a merge-readiness decision.
+model: gpt-5.6-luna
+user-invocable: true
+---
</code_context>
<issue_to_address>
**issue (bug_risk):** The three agent profiles specify `gpt-5.6-luna`, `gpt-5.6-sol`, and `gpt-5.6-terra`, which are not valid GitHub Copilot model identifiers. Copilot cannot resolve these profiles to an available model, so invoking the agents fails or the profiles are rejected despite the YAML parsing check passing.
**Triggers:** When these profiles are loaded or invoked through GitHub Copilot.
**Suggested fix:** Use model identifiers supported by the target Copilot environment, or omit `model` and let Copilot use its configured default.
```suggestion
```
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: .github/agents/luna.agent.md:4
| --- | ||
| name: Luna | ||
| description: Independently reviews changes, reports severity-ranked findings, and issues a merge-readiness decision. | ||
| model: gpt-5.6-luna |
Contributor
There was a problem hiding this comment.
issue (bug_risk): The three agent profiles specify gpt-5.6-luna, gpt-5.6-sol, and gpt-5.6-terra, which are not valid GitHub Copilot model identifiers. Copilot cannot resolve these profiles to an available model, so invoking the agents fails or the profiles are rejected despite the YAML parsing check passing.
Triggers: When these profiles are loaded or invoked through GitHub Copilot.
Suggested fix: Use model identifiers supported by the target Copilot environment, or omit model and let Copilot use its configured default.
Suggested change
| model: gpt-5.6-luna |
There was a problem hiding this comment.
🟡 Changes recommended
Enforced read-only restrictions are still needed for Sol and Luna.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Configures three GitHub Copilot custom agents for planning, implementation, and independent review, with shared workflow documentation.
Changes:
- Adds Sol, Terra, and Luna agent definitions.
- Documents responsibilities, handoffs, and conflict resolution.
- Extends the root agent guidance.
File summaries
| File | Description |
|---|---|
AGENTS.md |
Documents the agent workflow and boundaries. |
.github/agents/terra.agent.md |
Defines the implementation agent. |
.github/agents/sol.agent.md |
Defines the planning agent. |
.github/agents/luna.agent.md |
Defines the independent review agent. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| name: Luna | ||
| description: Independently reviews changes, reports severity-ranked findings, and issues a merge-readiness decision. | ||
| model: gpt-5.6-luna | ||
| user-invocable: true |
| name: Sol | ||
| description: Plans repository changes with read-only investigation and explicit implementation handoffs. | ||
| model: gpt-5.6-sol | ||
| user-invocable: true |
Co-authored-by: Copilot App <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Description
Configures three user-invocable GitHub Copilot custom agents:
gpt-5.6-sol) is the single-entry planner and coordinator: it plans, invokes Terra as an implementation subagent, invokes Luna for independent review, and routes actionable findings back through Terra -> Luna until resolution.gpt-5.6-terra) performs complete implementation, focused validation, and related documentation.gpt-5.6-luna) performs independent, severity-ranked review and returns an explicit merge-readiness decision.Extends the existing root
AGENTS.mdwithout replacing project instructions. It documents shared principles, strict role boundaries, handoff criteria, conflict resolution, a maximum of three correction cycles to prevent unlimited loops, and ready-to-run manual Terra/Luna prompts when the current Copilot surface cannot invoke custom-agent subagents.🔗 Related Issue
N/A
🧪 Type of Change
🧪 Testing
Validation performed:
toolsis omitted from every agent definition.git diff --checkand reviewed the complete diff.📸 Screenshots (if applicable)
N/A
✅ Checklist
📝 Additional Notes
Direct push access to
EvolutionAPI/evolution-apiwas unavailable, so the branch is published from the authenticatedCarlosCastro77/evolution-apifork.Summary by Sourcery
Configure complementary Copilot agents and document their coordinated planning, implementation, and review workflow.
New Features:
Enhancements:
Documentation: