Skip to content

Add DeepSeek Harness support - #36

Open
LeonHibnik wants to merge 1 commit into
mainfrom
feat/deepseek-harness
Open

LeonHibnik wants to merge 1 commit into
mainfrom
feat/deepseek-harness

Conversation

@LeonHibnik

Copy link
Copy Markdown
Collaborator

No description provided.

@zro-inference zro-inference left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds a DeepSeek Harness (dsh) tool adapter: a new src/engine/tools/dsh.ts that writes an isolated cordis.patch.yml profile patch (Zro provider, default model, and an MCP client entry that pulls the bearer token from process.env.ZRO_API_KEY rather than embedding it), plus catalog/SUPPORTED_TOOLS/run.ts wiring, changelog/README docs, and tests covering the patch contents, gateway override, no-auth mode, and DSH_HOME isolation. The diff is additive and follows the existing adapter shape (mirrors opencode's baseUrl/mcpUrl/noAuth options and env-key indirection), and the secret is correctly kept out of generated files.

Note: all four analysis specialties (correctness, security, test-coverage, adversarial) timed out before producing findings, so this approval is a summary-level pass over the diff rather than the usual deep specialist review. Nothing here should be read as positive confirmation of correctness beyond what is visible in the diff; a follow-up pass is worth running once the specialists can complete.

JenClaude command reference

Reply with @MoonMathJen on this PR to trigger commands:

Command Example What it does
Re-review @MoonMathJen rereview Re-review after changes (full or scoped — see below)
Stop @MoonMathJen stop Cancel in-progress review
Stop + summarize @MoonMathJen stop and summarize Post partial findings so far
Focus specialty @MoonMathJen only security Review only the named specialty
Skip specialty @MoonMathJen skip test-coverage Review everything except the named specialty
Change model @MoonMathJen pr-review-model: kimi-k2.7-coder Run this review with a specific model
Add context @MoonMathJen focus on auth edge cases Free-text guidance passed to reviewers

Specialties: correctness, security, test-coverage, adversarial

Any text in the comment beyond a recognized command is passed to the reviewers as context. Use it to point them at areas that matter, explain what changed since the last review, or call out constraints (e.g. "skip generated files", "the auth flow is the riskiest part here"). Hard review rules still win over reviewer guidance.


JenClaude metrics: cost $0.000, time 40m 31s, tokens 73,941.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants