feat: add DeepSeek TUI harness support#1586
Closed
EmiyaKiritsugu3 wants to merge 1 commit into
Closed
Conversation
Adds DeepSeek TUI as a supported harness. AGENTS.md now has bootstrap, DEEPSEEK.md created, deepseek-tools.md for tool mapping, SKILL.md and README.md updated.
Author
|
Closes #1587 |
Owner
|
Hi, This looks like it was not really tested or designed. |
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.
What problem are you trying to solve?
DeepSeek TUI (Hmbown/DeepSeek-TUI) already auto-discovers superpowers skills from the workspace
skills/directory and lists them in the## Skillssection of the system prompt. However, theusing-superpowersbootstrap -- the mandate that forces the model to check for relevant skills before taking any action -- was never injected into the context at session start.Without the bootstrap, skills are passive metadata: the model sees them listed but has no reason to invoke them. The acceptance test ("Let's make a react todo list" triggers brainstorming) fails.
What does this PR change?
Adds DeepSeek TUI as a supported harness, matching the existing pattern for Gemini CLI (GEMINI.md), OpenCode (plugin), and Claude Code (hooks).
using-superpowersbootstrap before the existing contributor guidelines. DeepSeek TUI loads AGENTS.md asproject_instructionsat session start, so the bootstrap is now auto-injected.Is this change appropriate for the core library?
Yes. This adds support for a new harness (IDE/CLI tool), which is explicitly listed as acceptable in the contributor guidelines. No third-party dependencies are introduced.
What alternatives did you consider?
~/.deepseek/instructions.mdfile is auto-generated and can be overwritten; unreliable as a bootstrap anchor.Does this PR contain multiple unrelated changes?
No. All changes serve the same goal: adding DeepSeek TUI harness support.
Existing PRs
Environment tested
New harness support
Acceptance test performed: the user ran
/skill using-superpowersinside a DeepSeek TUI session in this workspace. The full SKILL.md content loaded into context, including the mandate ("IF A SKILL APPLIES, YOU MUST USE IT"), Red Flags, and skill flow diagram.The bootstrap is also auto-loaded from AGENTS.md as
project_instructionsat every session start -- no manual/skillinvocation needed for normal use.Evaluation
Rigor
Note: No existing skills were modified in content -- only new platform references were added.
Human review