lsm / neokai / 24912906123

82%

push

github

fix(daemon): wrap builtin skills as SDK plugins so /playwright works

* fix(daemon): wrap builtin skills as SDK plugins so /playwright works

The Claude Agent SDK's `plugins: [{ type: 'local', path }]` loader
silently drops entries whose directories lack `.claude-plugin/plugin.json`.
Our builtin skills live at `~/.neokai/skills/<name>/` in the agent-skills
layout (SKILL.md at root), not the plugin layout, so the SDK was skipping
them and `/playwright` returned "Unknown command".

Materialise a wrapper plugin at `~/.neokai/skill-plugins/<name>/` with the
manifest the SDK requires plus a `skills/<name>/` symlink back to the real
skill directory. Point `buildPluginsFromBuiltinSkills()` at the wrapper.
User-visible skill directories are unchanged.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* fix(daemon): use isBuiltinSkillConfig guard in ensureBuiltinPluginWrappers

Replace the manual `as { type: 'builtin'; commandName: string }` cast with
the existing `isBuiltinSkillConfig` type guard from @neokai/shared. The
guard both narrows the config properly and pairs with the sourceType check
to defend against any hypothetical drift between sourceType and config.type.

Addresses review feedback on PR #1627.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

9142 of 12422 branches covered (73.6%)

Branch coverage included in aggregate %.

103 of 145 new or added lines in 4 files covered. (71.03%)

66911 of 79392 relevant lines covered (84.28%)

297.37 hits per line