Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a new “DeepSeek Harness (CLI)” backend to LongHorizon-Harness, wiring it through the model catalog, CLI/doctor, Web API meta payload, supervisor role bindings, and the Web UI so DeepSeek can be selected per role.
Changes:
- Introduces
DeepSeekHarnessAdapterplus adeepseek_runnerJSONL bridge that executesdsh --profile headlessand standardizes results asdsh.result. - Extends agent/model discovery and Web metadata to include
deepseek_harnesswith a default model (deepseek-v4-flash) and “suggested” discovery provenance. - Updates CLI, supervisor, snapshot sanitization, and frontend role pickers to accept and display the new backend; adds targeted tests.
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/test_model_catalog.py | Adds coverage for DeepSeek model catalog discovery and agent availability. |
| tests/test_deepseek_harness_adapter.py | New tests for binary resolution, adapter command quoting, runner behavior, JSONL parsing, end-to-end adapter run, and Web meta. |
| tests/supervisor/test_supervisor.py | Ensures the supervisor can create DeepSeek runs with per-role agent/model bindings. |
| src/lh_harness/webapi/snapshot.py | Allows deepseek_harness through role config sanitization. |
| src/lh_harness/webapi/server.py | Includes resolve_dsh_binary() in /api/meta model catalog discovery. |
| src/lh_harness/utils/agent_cli.py | Adds LH_HARNESS_DSH_BINARY support and resolve_dsh_binary(). |
| src/lh_harness/types.py | Defines DEFAULT_DEEPSEEK_HARNESS_MODEL. |
| src/lh_harness/supervisor/service.py | Permits DeepSeek in agent validation and default-model resolution for runs/roles. |
| src/lh_harness/model_catalog.py | Adds DeepSeek to cached discovery, agents list, and model discovery provenance. |
| src/lh_harness/config.py | Adds deepseek_harness to config agent choices. |
| src/lh_harness/cli.py | Registers DeepSeek as an agent option, excludes it from MCP/plugin flows, and builds the DeepSeek adapter. |
| src/lh_harness/agent_logs.py | Adds DeepSeek JSONL detection/parsing and trajectory support for dsh.result. |
| src/lh_harness/adapters/deepseek_runner.py | New bridge module that runs dsh headlessly and emits normalized JSONL. |
| src/lh_harness/adapters/deepseek_harness.py | New adapter integrating DeepSeek Harness with role-scoped permissions and isolated DSH_HOME. |
| src/lh_harness/adapters/init.py | Exposes DeepSeekHarnessAdapter via lazy exports and __all__. |
| README.zh-CN.md | Documents DeepSeek Harness phase-1 support, setup, and limitations (CN). |
| README.md | Documents DeepSeek Harness phase-1 support, setup, and limitations (EN). |
| pyproject.toml | Bumps project version to 0.1.5. |
| frontend/web/src/App.tsx | Adds DeepSeek to model presets, role runtime picker defaults, and UI notes. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Add DeepSeek Harness support
@Upper9527