Add repo map, MCP baseline verification, and repo-specific agent skills - #126
Open
groupthinking with Copilot wants to merge 2 commits into
Open
Add repo map, MCP baseline verification, and repo-specific agent skills#126groupthinking with Copilot wants to merge 2 commits into
groupthinking with Copilot wants to merge 2 commits into
Conversation
…kills Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update repo assist with specific examples and mapping resources
Add repo map, MCP baseline verification, and repo-specific agent skills
Sep 8, 2026
groupthinking
approved these changes
Sep 8, 2026
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
A few newly added documentation/skill statements are objectively inaccurate or reference non-existent tooling and should be corrected to avoid misleading future contributors/agents.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds repository-orientation documentation, repo-specific agent skills, and a new pytest baseline that pins down the current JSON-RPC MCP server handshake behavior for capability negotiation.
Changes:
- Added
docs/REPO_MAP.mdwith a top-level repo map, Mermaid architecture diagram, and an agent-run checklist mapped to repo artifacts. - Added
tests/test_mcp_baseline.pyto verifyinitialize,tools/list,tools/call, and unknown-method JSON-RPC error behavior formcp_server/main.py. - Added two repo-specific agent skills under
.github/skills/for MCP protocol debugging and quantum connector testing guidance.
File summaries
| File | Description |
|---|---|
tests/test_mcp_baseline.py |
Adds baseline tests that exercise MCP JSON-RPC handshake and tool execution paths. |
docs/REPO_MAP.md |
Provides a repo map + checklist + MCP baseline documentation for agents and contributors. |
.github/skills/quantum-connector-testing/SKILL.md |
Adds guidance for testing D-Wave integration without introducing simulation/mocks in production paths. |
.github/skills/mcp-protocol-debugging/SKILL.md |
Adds a step-by-step guide for driving and debugging mcp_server/main.py MCP negotiation and tool calls. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 4
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+37
to
+41
| this is explicitly checked by | ||
| `tests/test_mcp_compliance.py::test_no_placeholder_code_in_production` | ||
| and `test_data_processor_no_simulation`, which scan for | ||
| `mock`/`simulated`/`placeholder` strings in production directories | ||
| (`agents`, `connectors`, `mcp_server`, `protocols`). |
Comment on lines
+73
to
+76
| M2 --> C1 | ||
| M2 --> C2 | ||
| M2 --> C3 | ||
| M2 --> C4 |
Comment on lines
+95
to
+97
| - [ ] **Pull / Push / Commit / Merge** — Use `engine-tools-report_progress` | ||
| (or normal `git`/PR flow for humans); never push directly from an | ||
| agent sandbox. See `CONTRIBUTING.md`. |
Comment on lines
+123
to
+127
| | `resources/list` / `resources/read` | Lists/reads MCP resources | | ||
| | `notifications/list` / `notifications/subscribe` | Baseline notification support | | ||
|
|
||
| This baseline is exercised live and asserted in | ||
| `tests/test_mcp_baseline.py`. Run it with: |
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
The issue requested a repo tree + mermaid diagram, a checklist covering agents/tools/MCP/PRs/deps/actions, live proof that MCP capability negotiation works for this repo, and 2+ skills tailored to the repo.
Repo map & checklist
docs/REPO_MAP.md: top-level repo tree, Mermaid architecture diagram (host → MCP servers → tools/connectors/protocols), and an agentic-workflow checklist mapped to actual repo artifacts (agents/,mcp_server/main.py,.github/workflows/python-ci.yml,dependabot.yml, etc.)MCP baseline verification
mcp_server/main.py) live:initialize,tools/list,tools/call, and unknown-method error handling all behave correctlytests/test_mcp_baseline.pyto pin down this baseline capability negotiation so it stays verified going forwardRepo-specific skills
.github/skills/mcp-protocol-debugging/— exercising/debugging the MCP JSON-RPC server.github/skills/quantum-connector-testing/— testing the D-Wave quantum connector without a live QPU token, consistent with this repo's existing "no mocks/simulation in production" conventionNo production code was modified; changes are additive (docs, skills, tests).
Type of change
Checklist
Screenshots (if applicable)
Additional context
Pre-existing unrelated test failures (
tests/test_pattern_detector.pymissingpsycopg2,tests/test_dependabot_test.pyimport issue) were observed but are out of scope for this change.