Add repo-specific repo assist map, MCP baseline verification, and project skills - #127
groupthinking with Copilot wants to merge 3 commits into
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Repository: groupthinking/self-correcting-executor/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Portability and MCP verification documentation issues remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 5
Open (5)
What changed in this PR
Adds repo-specific agent guidance, MCP verification documentation, project skills, README wiring, and guardrail tests.
Changes:
- Adds repository mapping and MCP guidance.
- Adds CI-debugging and codebase-summary skills.
- Links documentation from
README.md. - Adds tests validating the new materials.
| File | Description |
|---|---|
tests/test_repo_assist_docs.py |
Guardrail checks for documentation and skills |
README.md |
Links to the repository map |
docs/REPO_MAP.md |
Documents structure, dependencies, workflows, and MCP verification |
.github/skills/github-actions-failure-debugging/SKILL.md |
Adds CI debugging guidance |
.github/skills/codebase-summary/SKILL.md |
Adds repository summarization guidance |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| When asked to summarize or orient an agent in this repository, use the current | ||
| repo structure instead of generic MCP examples. | ||
|
|
||
| 1. Start from `/home/runner/work/self-correcting-executor/self-correcting-executor`. |
| 4. Reproduce only the relevant local command: | ||
|
|
||
| ```bash | ||
| cd /home/runner/work/self-correcting-executor/self-correcting-executor |
| `/home/runner/work/self-correcting-executor/self-correcting-executor/docs/REPO_MAP.md` | ||
| (also tracked in the repository as [`docs/REPO_MAP.md`](docs/REPO_MAP.md)). No newline at end of file |
| 1. A custom JSON-RPC server in `/home/runner/work/self-correcting-executor/self-correcting-executor/mcp_server/main.py` | ||
| with explicit MCP methods such as `initialize`, `tools/list`, `tools/call`, | ||
| `resources/list`, and `resources/read`. | ||
| 2. A FastMCP-based server in `/home/runner/work/self-correcting-executor/self-correcting-executor/mcp_server/real_mcp_server.py` | ||
| built on the `mcp` package. |
| }, | ||
| }, | ||
| ): | ||
| print(json.dumps(await server.handle_request(request), indent=2)) |

Pull Request
Description
Adds repo-specific guidance for agent runs in
self-correcting-executor: a current repo map, dependency snapshot, MCP host/client/server framing, and a minimal live capability-negotiation proof. Also adds two project-scoped skills so CI debugging and repo summarization use this repo’s actual structure and workflows instead of generic examples.Repo assist doc
docs/REPO_MAP.mdwith:initialize,tools/list, andtools/callflake8/pytestfailures in current workflow behaviorProject skills
.github/skills/github-actions-failure-debugging/SKILL.mdpython-ci.ymlandfrontend-ci.yml.github/skills/codebase-summary/SKILL.mdDocumentation wiring
README.mdto point to the repo assist documentGuardrail test
tests/test_repo_assist_docs.pyto assert the new repo map sections and project skills remain present and repo-specificType of change
Checklist
Screenshots (if applicable)
Additional context
The issue asked for repo-specific examples, at least two useful skills, and proof that baseline MCP negotiation works live for this repo. The new materials are intentionally scoped to the existing repository layout and workflows rather than introducing new runtime behavior.