Skip to content

feat(compatibility): discover DeepSeek Harness and PI skills and hooks - #2912

Merged
kev1n77 merged 1 commit into
GCWing:mainfrom
kev1n77:fmy/compatibility
Sep 9, 2026
Merged

kev1n77 merged 1 commit into
GCWing:mainfrom
kev1n77:fmy/compatibility

Conversation

@kev1n77

@kev1n77 kev1n77 commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add standard-directory Skill discovery for DeepSeek Harness and PI, including flat Markdown files, nested PI groups, and explicit entry files.
  • Discover DSH Cordis Hook bridge configurations and static PI extension events.
  • Enable the corresponding compatibility-page entries and display read-only Hook catalogs, states, and diagnostics.
  • Update English and Chinese documentation and UI strings across three locales.

Type and Areas

Type: Feature

Areas: Rust adapters, agent runtime, product assembly, web UI, i18n, docs

Motivation / Impact

Users can discover DeepSeek Harness and PI Skills and Hooks from the compatibility page.

Skills retain stable identifiers and their actual entry files. Hook discovery inspects configuration and source code without importing or executing external extensions.

Verification

Implementation-stage verification:

  • cargo test --locked -p openbitfun-pi-adapter --lib: 3 passed.
  • cargo test --locked -p openbitfun-dsh-adapter --lib hook_source::tests: 3 passed.
  • cargo test --locked -p openbitfun-dsh-adapter --test dsh_source_adapter: 15 passed.
  • cargo test --locked -p openbitfun-agent-runtime --no-default-features --features agent-runtime --test agent_definition_contracts skill_contracts: 34 passed.
  • Focused Core Skill discovery tests: 4 passed.
  • Focused external Hook tests: 5 passed.
  • Focused frontend tests: 28 passed.
  • pnpm run check:web: passed.
  • pnpm run i18n:audit: passed.
  • pnpm run check:repo-hygiene: passed.
  • Core boundary check: passed in a clean copy with the same changes. The original workspace check was blocked by an existing broken Harmony symlink.

Pre-commit verification:

  • git diff --cached --check: passed.

Reviewer Notes

  • The new entry-file field is optional; legacy payloads continue to default to SKILL.md.
  • Skill discovery covers standard directories and environment-variable overrides. Custom, package-provided, and runtime-published directories, as well as native ancestor-directory resolution, remain outside this scope.
  • Hook support is static discovery only. Dynamic registrations produce diagnostics, and remote-workspace Hook discovery remains explicitly unsupported.
  • Tests cover local and simulated remote-filesystem Skill discovery, plus remote Hook rejection.
  • Real SSH, Remote Control, Peer Device, and Detached Dispatch end-to-end scenarios were not exercised. Manual GUI verification was not performed.
  • AI-assisted. Testing level: focused automated verification completed; full end-to-end verification remains outstanding.

Checklist

  • This PR is focused and does not include secrets, temporary prompts, generated scratch files, or unrelated artifacts.
  • Relevant verification is recorded above, or skipped checks are explained.
  • User-facing strings, docs, and locales are updated where applicable.

@kev1n77
kev1n77 merged commit 9f54c5d into GCWing:main Sep 9, 2026
14 checks passed
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