Deepagent-code integration demo - #4527
Open
michalkulakowski wants to merge 2 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The new README is not merge-ready due to multiple placeholder blocks and inconsistent step numbering, and the skill metadata license field should be aligned with the repo license to avoid confusion.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds a new demo directory illustrating how to run short “DeepAgents Code” scenarios against OpenVINO Model Server, including local DeepAgents project assets for MCP SDK guidance and a verification subagent profile.
Changes:
- Introduces a new demo README with end-to-end setup instructions and scenario prompts.
- Adds project-local DeepAgents assets: an MCP SDK skill, an MCP tester subagent profile, and an MCP server config stub.
- Adds a demo-scoped
.gitignorefor common local artifacts.
File summaries
| File | Description |
|---|---|
| demos/integration_with_deepagents_code/README.md | Documents OVMS + dcode setup and a series of demo scenarios/prompts (currently includes placeholders and inconsistent step numbering). |
| demos/integration_with_deepagents_code/.gitignore | Ignores local virtualenv/log/output folders for the demo directory. |
| demos/integration_with_deepagents_code/.deepagents/skills/python-mcp-sdk-skill/SKILL.md | Adds guidance/template for implementing a Python MCP stdio server with FastMCP. |
| demos/integration_with_deepagents_code/.deepagents/agents/mcp-tester/AGENTS.md | Defines a focused subagent profile to validate an MCP server via static and short runtime checks. |
| demos/integration_with_deepagents_code/.deepagents/.mcp.json | Adds a local MCP server config entry pointing to mcp_server/time_mcp_server.py. |
Review details
Suppressed comments (9)
demos/integration_with_deepagents_code/README.md:80
- This line references Step 1/3/4, but those step headings don’t exist in the document, which makes the instructions ambiguous.
Use this profile for non-MCP steps (Step 1, Step 3, Step 4):
demos/integration_with_deepagents_code/README.md:89
- This line references “Step 6”, but there is no Step 6 section in the document, so readers can’t tell when to use this profile.
Use this profile for MCP steps (Step 2 and Step 6):
demos/integration_with_deepagents_code/README.md:111
- This section is titled “Step 2” even though the earlier numbered steps aren’t present (and “Step 3” appears above), so the document’s step ordering is inconsistent.
## Step 2: MCP server creation
demos/integration_with_deepagents_code/README.md:109
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
demos/integration_with_deepagents_code/README.md:115
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
demos/integration_with_deepagents_code/README.md:126
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
demos/integration_with_deepagents_code/README.md:157
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
demos/integration_with_deepagents_code/README.md:164
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
demos/integration_with_deepagents_code/README.md:188
- User-facing documentation shouldn’t ship with placeholder text; it makes the demo incomplete and harder to follow.
PLACEHOLDER FOR SCREENSHOT
- Files reviewed: 5/5 changed files
- Comments generated: 5
- 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: python-mcp-sdk-skill | ||
| description: "Guidance for implementing Python MCP SDK stdio servers with FastMCP, robust validation, and deterministic tool output." | ||
| license: MIT |
| export DEEPAGENTS_CODE_PRICES_AUTO_UPDATE=0 | ||
| ``` | ||
|
|
||
| ### Step 3: Install dependencies |
| Git init is required because dcode treats the Git root as the project root. | ||
|
|
||
|
|
||
| PLACEHOLDER FOR SCREENSHOT |
Comment on lines
+136
to
+138
| PLACEHOLDER FOR SCREENSHOT acceptance criteria | ||
| PLACEHOLDER FOR SCREENSHOT WIP | ||
| PLACEHOLDER FOR SCREENSHOT Grader |
Comment on lines
+176
to
+177
| PLACEHOLDER FOR SCREENSHOT | ||
| before / after |
michalkulakowski
force-pushed
the
mkulakow/deepagents_integration
branch
from
September 9, 2026 10:03
5bef66e to
8f82257
Compare
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.
🛠 Summary
JIRA/Issue if applicable.
Describe the changes.
🧪 Checklist
``