Skip to content

feat: add DeepSeek Harness (DSH) support - #315

Open
cdxiaodong wants to merge 1 commit into
wxtsky:mainfrom
cdxiaodong:feat/dsh-support
Open

feat: add DeepSeek Harness (DSH) support#315
cdxiaodong wants to merge 1 commit into
wxtsky:mainfrom
cdxiaodong:feat/dsh-support

Conversation

@cdxiaodong

Copy link
Copy Markdown

Summary

Adds DeepSeek Harness (DSH) — the plugin-native agent runtime from DeepSeek — to CodeIsland's supported tools.

DSH ships no shell-hook config; it is bridged by the dsh-island cordis plugin, which listens to DSH's built-in events (session/created, tools/pre-execute, approval/request, …) and writes the same JSON over the CodeIsland Unix socket. This PR only makes CodeIsland recognize the dsh source so those events render as their own session card.

Changes

  • SessionSnapshot.supportedSources: add dsh — previously events with _source: dsh were dropped (HookServer returns {} for unknown sources).
  • CLIProcessResolver.sourceMatchesExecutablePath: match the dsh binary in process-ancestry resolution (session-id fallback / source inference).
  • ConfigInstaller: add a plugin-style CLIConfig for dsh (no shell hooks), skip hook install/uninstall, and detect the harness via ~/.dsh or $DSH_HOME.
  • NotchPanelView.cliIconFiles: map dsh → new pixel dsh.png cli-icon.
  • README / README.zh-CN: add DSH to the supported-tools table (15 tools) and document the dsh plugin add install step.

Verification

  • CodeIslandCore target builds clean with the changes.
  • The counterpart plugin dsh-island ships 8 passing tests and a live demo panel.

Screenshot

DSH → CodeIsland 面板演示

DSH events rendered as a CodeIsland-style panel, including a PermissionRequest approval card.

- Recognize dsh source in supportedSources so DSH events are not dropped
- Match the dsh binary in process-ancestry resolution
- Add a plugin-style CLIConfig (no shell hooks; dsh-island bridges the socket)
- Skip hook install/uninstall for dsh; detect harness via ~/.dsh or $DSH_HOME
- Add a pixel dsh cli-icon and document the integration in both READMEs
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.

1 participant