Deepagent-code integration demo by michalkulakowski · Pull Request #4527 · openvinotoolkit/model_server

michalkulakowski · 2026-09-09T09:53:29Z

🛠 Summary

https://openvino-doc.iotg.sclab.intel.com/deepagents-integration-demo/model-server/ovms_demos_integration_with_deepagents_code.html

🧪 Checklist

Unit tests added.
The documentation updated.
Change follows security best practices.
``

Copilot

🟡 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 .gitignore for 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.

Copilot

🟡 Changes recommended

One or more issues must be addressed before approval.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (4)

demos/integration_with_deepagents_code/.gitignore:3

The validation instructions in AGENTS.md run python -m py_compile, which creates __pycache__ under the generated server directory. This ignore file does not exclude that bytecode, so each validation run adds generated artifacts to the demo's git working tree; add __pycache__/.

outputs/

demos/integration_with_deepagents_code/README.md:252

This MCP-task example repeats invalid dcode filesystem tool names (list_dir, grep_search, and file_search), so the command fails argument validation instead of starting. Use the actual names exposed by dcode's filesystem middleware.

  --allow-fs-tools read_file,list_dir,grep_search,file_search

demos/integration_with_deepagents_code/README.md:116

execute is enabled and the shell allow-list includes python, python3, and timeout; these can run arbitrary Python or wrap another executable. As a result, this is not a safety boundary for an agent running on the host and can bypass the intended filesystem/tool restrictions. Run dcode in a real sandbox/container, or explicitly document that this command grants arbitrary host-code execution instead of presenting the list as command safety.

  -S python,python3,timeout,cat,grep,ls \

demos/integration_with_deepagents_code/README.md:118

--trust-project-mcp skips DeepAgents Code's project-MCP approval gate and automatically trusts the committed stdio command, which executes the repo-controlled mcp_server/time_mcp_server.py. Running this from an untrusted checkout can therefore execute arbitrary Python before the user reviews the server; omit this bypass and have the user approve only time-server, or add a prominent warning to audit the project config and generated file first.

  --trust-project-mcp

Files reviewed: 6/17 changed files
Comments generated: 5
Review effort level: Lite

Copilot

🟡 Changes recommended

Moderate issues remain with runtime validation, environment setup, and OVMS version compatibility.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (3)

demos/integration_with_deepagents_code/.deepagents/agents/mcp-tester/AGENTS.md:19

The only runtime check starts the script and waits for up to two seconds; it never performs MCP initialization or calls tools/list/tools/call. The 124 rule can therefore report PASS for a server that is hung, slow to initialize, or unable to serve tools, so this does not verify the runtime behavior claimed by the demo. Add a small MCP stdio readiness/tool-call probe, or narrow the claim to process startup.

- Run a short startup check: `timeout 2s python <server_path>`.

demos/integration_with_deepagents_code/.deepagents/skills/python-mcp-sdk-skill/SKILL.md:54

The required time tool returns the current time, but this quality rule requires all outputs to be deterministic. Those instructions conflict and can lead the agent to freeze or mock the timestamp. Require a stable machine-readable format/schema instead, while explicitly allowing the time value to vary.

- Keep tool outputs deterministic and machine-readable.

demos/integration_with_deepagents_code/README.md:43

The Qwen3.8-27B model used here is documented in demos/continuous_batching/speculative_decoding/README.md:29 as requiring OVMS 2026.4 or a weekly build, but these commands use the floating latest-gpu tag. On hosts whose tag predates that support, the model deployment will fail; use an explicit compatible tag or state and enforce the required OVMS version consistently for all five commands.

  -v ${HOME}/models:/models openvino/model_server:latest-gpu \
  --pull --source_model OpenVINO/Qwen3.8-27B-int4-ov --task text_generation --model_repository_path /models

Files reviewed: 6/17 changed files
Comments generated: 3
Review effort level: Lite

@@ -0,0 +1,3 @@
+.env/
+logs/
+outputs/


dtrawins · 2026-09-18T13:10:06Z

+:sync: Linux
+```bash
+mkdir -p ${HOME}/models
+export GPU_ARGS=$(if ls /dev/dri/render* >/dev/null 2>&1; then echo "--device /dev/dri --group-add $(stat -c '%g' /dev/dri/render* | head -n1)"; fi)


this should be a part of model deployment command

dtrawins · 2026-09-18T13:10:16Z

+:::
+:::{tab-item} Windows
+:sync: Windows
+```powershell


use c:\models

dtrawins · 2026-09-18T13:12:57Z

+:::
+:::{tab-item} Windows
+:sync: Windows
+```powershell


on windows should be use binary deployment - not docker

dtrawins · 2026-09-18T13:14:37Z

+:::{tab-item} Linux
+:sync: Linux
+```bash
+docker run --rm ${GPU_ARGS} -u $(id -u):$(id -g) \


why using so many models in this demo? Maybe it should be set that this is example models and it can be swapped

dtrawins · 2026-09-18T13:15:56Z

+:::{tab-item} Linux
+:sync: Linux
+```bash
+export OPENAI_API_KEY=not_used


we should use a real api key like my-key also in ovms

dtrawins · 2026-09-18T13:21:39Z

+
+The MCP server script is missing. Ask the agent to create it using the project skill located at `.deepagents/skills/python-mcp-sdk-skill/SKILL.md`. Invoking a skill with `/skill:<name>` gives the agent a focused, tested recipe instead of relying on generic knowledge:
+
+```bash


this is not bash command...

dtrawins · 2026-09-18T13:23:16Z

+Non-MCP tasks:
+
+```bash
+dcode --model openai:OpenVINO/Qwen3.8-27B-int4-ov \


Can this be executed non interactively so it could be tested?
Otherwise it shouldn't be bash command here

dtrawins · 2026-09-18T13:24:47Z

why this error is included?

dtrawins · 2026-09-18T13:27:08Z

do we need so many screenshots? maybe short gif or some compressed video would be better? 12 images about ~1.5MB

dtrawins · 2026-09-18T13:27:46Z

+Requirements:
+- Linux or Windows
+- Python 3.12+
+- Docker (Docker Engine on Linux, Docker Desktop on Windows)


what HW is needed and how much RAM

Copilot

Copilot review overview

🟡 Changes recommended

Address the documented setup portability, repository-path, and validation issues before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 4 · 2

Open (6)

Use a portable timeout command for Windows validation · New
Provide platform-specific readiness check commands · New
Create the Linux virtual environment inside the demo directory · New
Running this sequence from the repository root creates .env at the repository root, while this…
This setup grants the agent execute and explicitly allows python/python3, so it can run…
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

Resolved since last review (2)

The setup creates the virtual environment as .env, but the new .gitignore only excludes…
This skill file declares license: MIT, but the repository is Apache-2.0 licensed (see LICENSE in…

Copilot

Copilot review overview

🟡 Changes recommended

Unresolved endpoint exposure and readiness-check issues need to be addressed before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 · 1

Open (2)

Bind the REST endpoint to loopback · New
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

Resolved since last review (5)

Create the Linux virtual environment inside the demo directory
Provide platform-specific readiness check commands
Use a portable timeout command for Windows validation
Running this sequence from the repository root creates .env at the repository root, while this…
This setup grants the agent execute and explicitly allows python/python3, so it can run…

Previously missed (1)

In code that hasn't changed since last review

Link directly to the bare-metal deployment guide

demos/integration_with_deepagents_code/README.md:27

This link targets the generic deployment page with a fragment that does not exist; the bare-metal page is a separate generated page (docs/deploying_server_baremetal.md, anchor ovms_docs_deploying_server_baremetal). As written, the Windows installation prerequisite does not take users to the referenced guide. Link directly to https://docs.openvino.ai/2026/model-server/ovms_docs_deploying_server_baremetal.html.

+```bash
+export GPU_ARGS=$(if ls /dev/dri/render* >/dev/null 2>&1; then echo "--device /dev/dri --group-add $(stat -c '%g' /dev/dri/render* | head -n1)"; fi)
+docker run -d ${GPU_ARGS} -u $(id -u):$(id -g) \
+  -v ${HOME}/models:/models -p 8000:8000 openvino/model_server:latest-gpu \


Copilot

Copilot review overview

🟡 Changes recommended

Moderate validation, platform compatibility, setup, and model-readiness issues remain unresolved.

Review effort: Lite
Findings: 1 · 1 · 3

Open (5)

Bind the REST endpoint to loopback
Require ToolAnnotations in headless flow validation · New
Align expected output with AGENTS.md contract · New
Correct weekday from Tuesday to Wednesday · New
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

+
+What this subagent is responsible for:
+- Inspect the target server script.
+- Verify required elements exist: FastMCP usage, at least one @mcp.tool, and stdio run call.


+**time_mcp_server.py — MCP Test Results: All PASSED ✅**
+
+| Check | Result |
+|---|---|
+| Script Structure | ✅ FastMCP, 2 tools, `mcp.run(transport="stdio")` |


+
+Current UTC date: `2026-09-30`
+
+Full datetime: **Tuesday, September 30, 2026 at 14:00:12 UTC**


Copilot

Copilot review overview

🟡 Changes recommended

Unresolved moderate issues affect cross-platform usage, security boundaries, and MCP validation.

Review effort: Lite
Findings: 1 · 3 · 4

Open (8)

Bind the REST endpoint to loopback
Document Windows OVMS installation · New
Use correct PowerShell environment assignments · New
Require ToolAnnotations in headless flow validation
Use the correct DeepAgents Code branding · New
Correct weekday from Tuesday to Wednesday
Align expected output with AGENTS.md contract
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

+:sync: Windows
+```bat
+mkdir c:\models
+ovms.exe --source_model OpenVINO/Qwen3.8-27B-int4-ov --model_repository_path c:\models --model_name OpenVINO/Qwen3.8-27B-int4-ov --rest_port 8000


+```bat
+set OPENAI_API_KEY=not_used
+set OPENAI_BASE_URL=http://localhost:8000/v1
+set TAVILY_API_KEY=not_used
+set DEEPAGENTS_CODE_PRICES_AUTO_UPDATE=0
+set DEMO_DIR=%CD%


…de branding' Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Copilot review overview

🔵 Needs a closer look

Unresolved moderate findings affect platform compatibility, model startup, validation correctness, and documented workflow behavior.

Review effort: Lite
Findings: 1 · 3 · 3

Open (7)

Bind the REST endpoint to loopback
Use correct PowerShell environment assignments
Document Windows OVMS installation
Require ToolAnnotations in headless flow validation
Correct weekday from Tuesday to Wednesday
Align expected output with AGENTS.md contract
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

Resolved since last review (1)

Use the correct DeepAgents Code branding

Previously missed (1)

In code that hasn't changed since last review

Require read-only annotations for the new date tool

demos/integration_with_deepagents_code/README.md:302

The --rubric does not require the new date tool to carry the read-only annotations required by SKILL.md. A rubric-compliant implementation can add a plain @mcp.tool, pass this headless step, and then fail Step 7 with the documented approval error; require readOnlyHint=true and destructiveHint=false for the new tool here.

Copilot

🟡 Changes recommended

Unresolved security, portability, validation, and documentation issues remain.

8 open findings

Bind the Docker endpoint to localhost · New
Bind the REST endpoint to loopback
Use correct PowerShell environment assignments
Document Windows OVMS installation
Require ToolAnnotations in headless flow validation
Correct weekday from Tuesday to Wednesday
Align expected output with AGENTS.md contract
The tester explicitly runs python -m py_compile, which creates mcp_server/__pycache__/*.pyc. In…

🧠 Review effort: Lite

Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

+mkdir -p ${HOME}/models
+export GPU_ARGS=$(if ls /dev/dri/render* >/dev/null 2>&1; then echo "--device /dev/dri --group-add $(stat -c '%g' /dev/dri/render* | head -n1)"; fi)
+docker run -d ${GPU_ARGS} -u $(id -u):$(id -g) \
+  -v ${HOME}/models:/models -p 8000:8000 openvino/model_server:latest-gpu \


Copilot AI lite review requested due to automatic review settings September 9, 2026 09:53

Copilot started reviewing on behalf of michalkulakowski September 9, 2026 09:54 View session

Copilot AI reviewed Sep 9, 2026

View reviewed changes

michalkulakowski force-pushed the mkulakow/deepagents_integration branch from 5bef66e to 8f82257 Compare September 9, 2026 10:03

michalkulakowski requested a review from mzegla September 9, 2026 12:56

michalkulakowski force-pushed the mkulakow/deepagents_integration branch from 5e93009 to 7bff9b1 Compare September 11, 2026 12:24

michalkulakowski requested review from dtrawins and a lite review from Copilot September 11, 2026 12:25

Copilot started reviewing on behalf of michalkulakowski September 11, 2026 12:39 View session

Copilot AI reviewed Sep 11, 2026

View reviewed changes

michalkulakowski force-pushed the mkulakow/deepagents_integration branch from 88705bf to e776655 Compare September 11, 2026 13:07

michalkulakowski requested a lite review from Copilot September 11, 2026 13:08

Copilot started reviewing on behalf of michalkulakowski September 11, 2026 13:08 View session

mzegla approved these changes Sep 11, 2026

View reviewed changes

Copilot AI reviewed Sep 11, 2026

View reviewed changes

Comment thread demos/integration_with_deepagents_code/README.md Outdated

Comment thread demos/integration_with_deepagents_code/.gitignore Outdated

@@ -0,0 +1,3 @@

.env/

logs/

outputs/

Comment thread demos/integration_with_deepagents_code/README.md

przepeck requested a review from pgladkows September 18, 2026 08:06

pgladkows reviewed Sep 18, 2026

View reviewed changes

Comment thread demos/integration_with_deepagents_code/README.md Outdated

dtrawins requested changes Sep 18, 2026

View reviewed changes

michalkulakowski requested review from pgladkows and a lite review from Copilot September 25, 2026 11:57

Copilot started reviewing on behalf of michalkulakowski September 25, 2026 11:57 View session

Copilot AI reviewed Sep 25, 2026

View reviewed changes

Comment thread demos/integration_with_deepagents_code/.deepagents/agents/mcp-tester/AGENTS.md

Comment thread demos/integration_with_deepagents_code/README.md Outdated

Comment thread demos/integration_with_deepagents_code/README.md Outdated

michalkulakowski requested a lite review from Copilot September 25, 2026 12:11

Copilot started reviewing on behalf of michalkulakowski September 25, 2026 12:11 View session

Copilot AI reviewed Sep 25, 2026

View reviewed changes

michalkulakowski force-pushed the mkulakow/deepagents_integration branch from 9eb1a15 to 53f2396 Compare September 30, 2026 10:42

michalkulakowski requested review from dtrawins and a lite review from Copilot September 30, 2026 14:47

Copilot started reviewing on behalf of michalkulakowski September 30, 2026 14:49 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

michalkulakowski added 5 commits October 1, 2026 12:51

fix

5289d23

fixes

dfaf1fe

fixes

2ae01d6

fix

950403d

fix

e8c88d0

michalkulakowski force-pushed the mkulakow/deepagents_integration branch from 5a5774f to e8c88d0 Compare October 1, 2026 10:52

michalkulakowski and others added 3 commits October 1, 2026 16:22

fix

3a51dfa

fix

31c26c9

fix

859c05e

michalkulakowski requested a lite review from Copilot October 2, 2026 11:30

Copilot started reviewing on behalf of michalkulakowski October 2, 2026 11:31 View session

Copilot AI reviewed Oct 2, 2026

View reviewed changes

michalkulakowski and others added 2 commits October 2, 2026 15:24

Potential fix for pull request finding 'Use the correct DeepAgents Co…

dbd088d

…de branding' Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

fix

18b62f7

michalkulakowski requested a lite review from Copilot October 6, 2026 14:13

Copilot started reviewing on behalf of michalkulakowski October 6, 2026 14:14 View session

Copilot AI reviewed Oct 6, 2026

View reviewed changes

Michal Kulakowski added 4 commits October 7, 2026 12:15

refactor

c387240

fix

fcafa34

fix

05ef501

fix

e17b40f

dtrawins approved these changes Oct 7, 2026

View reviewed changes

michalkulakowski requested a lite review from Copilot October 8, 2026 08:37

Copilot started reviewing on behalf of michalkulakowski October 8, 2026 08:39 View session

Copilot AI reviewed Oct 8, 2026

View reviewed changes

Michal Kulakowski added 4 commits October 8, 2026 10:45

fix

8e4d156

fix

e550069

fix

e23e345

fix

26b2906

michalkulakowski merged commit a789486 into main Oct 8, 2026
7 checks passed


		The MCP server script is missing. Ask the agent to create it using the project skill located at `.deepagents/skills/python-mcp-sdk-skill/SKILL.md`. Invoking a skill with `/skill:<name>` gives the agent a focused, tested recipe instead of relying on generic knowledge:

		```bash


		Current UTC date: `2026-09-30`

		Full datetime: Tuesday, September 30, 2026 at 14:00:12 UTC

Conversation

michalkulakowski commented Sep 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🛠 Summary

🧪 Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Suppressed comments (9)

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Suppressed comments (4)

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Suppressed comments (3)

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

michalkulakowski commented Sep 9, 2026 •

edited

Loading