Skip to content

feat(skills): per-mind skill discovery + IPC#397

Open
patschmittdev wants to merge 1 commit into
refactor/ui-foundationfrom
feat/mind-skills
Open

feat(skills): per-mind skill discovery + IPC#397
patschmittdev wants to merge 1 commit into
refactor/ui-foundationfrom
feat/mind-skills

Conversation

@patschmittdev

Copy link
Copy Markdown
Collaborator

What

Per-mind skill discovery + IPC, split off feat/webgl-ambient-background onto the refactor/ui-foundation base (#389). Base of the agent-profile slice, which stacks on this.

Contents (11 files, +258)

  • Skill discovery service for per-mind skills, surfaced via a new IPC channel.
  • main.ts / preload.ts / ipc-channels.ts: new skills IPC (thin adapter pattern).

Security note for review: adds a new IPC channel. Inputs validated at the adapter boundary; no new persistence or credential surface.

Stack

Sibling on refactor/ui-foundation (#389); feat/agent-profile stacks on this.

Validation

  • npm run lint -- green (tsc, eslint, dependency-cruiser, yaml, md).
  • Slice's own tests pass.

Split the mind-skill discovery vertical out of feat/webgl-ambient-background
onto the ui-foundation base. Backend-only and purely additive:

- MindSkillDiscovery service: lists skills installed under
  <mindPath>/.github/skills/ (read-only filesystem scan)
- SkillManifest shared type + SKILLS.LIST_FOR_MIND ipc channel
- ElectronAPI.skills.listForMind across preload + browser shim
- skills:listForMind main-process handler (validates mindId, resolves the
  mind's path via MindManager)

The lone renderer consumer (AgentSurface) is deferred to the
genesis-marketplace branch; this slice carries only the data path.

Note: the original decomposition plan suggested moving this handler into a
main/ipc/skills.ts adapter. Kept inline to match the source faithfully; the
adapter refactor is a separate, optional follow-up.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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