Skip to content

fix(pi): expose native skills through $ instead of / - #1286

Merged
tiann merged 1 commit into
tiann:mainfrom
swear01:fix/issue-1285-pi-skills
Aug 2, 2026
Merged

fix(pi): expose native skills through $ instead of /#1286
tiann merged 1 commit into
tiann:mainfrom
swear01:fix/issue-1285-pi-skills

Conversation

@swear01

@swear01 swear01 commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • split Pi get_commands inventory by native source: skills go to HAPI $, prompt/extension commands stay in /
  • normalize Pi's skill:<name> transport prefix to $<name> and translate HAPI invocations back to /skill:<name>
  • keep Pi skill commands first when attachments are present, so Pi performs native skill expansion
  • stop Pi and Kimi from falling back to Claude built-in slash commands

Pi RPC documents source: "skill" entries as /skill:<name> commands and expands them for prompt/steer input. Gemini legacy is intentionally out of scope.

Fixes #1285
Parent: #1284

Verification

  • bun typecheck — exit 0
  • bun run test — 4,243 passed; 3 skipped; 0 failed
  • focused Pi/web tests — 17 passed

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • No issues found in the added or modified lines.

Questions

  • None.

Summary

  • Review mode: initial. No Blocker, Major, Minor, or Nit findings. Residual risk: the Pi namespace and prompt rewrite are covered by helper tests, but not by an end-to-end test against a real Pi RPC process.

Testing

  • Not run (automation): Bun is unavailable in this review environment; the GitHub test check was pending at review time.

HAPI Bot

@tiann
tiann merged commit 545af9b into tiann:main Aug 2, 2026
2 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.

fix(pi): expose native skills through $ instead of /

2 participants