Skip to content

fix: add load commands to session skill catalogsfix hooks session catalog load guidance - #217

Merged
LadyBluenotes merged 2 commits into
mainfrom
feat/fixing-issue-215
Aug 30, 2026
Merged

fix: add load commands to session skill catalogsfix hooks session catalog load guidance#217
LadyBluenotes merged 2 commits into
mainfrom
feat/fixing-issue-215

Conversation

@LadyBluenotes

@LadyBluenotes LadyBluenotes commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Add the package-manager-specific Intent load command to the SessionStart skill catalog.
  • Clarify that catalog entries are Intent skills, not native agent skills.
  • Add regression coverage for Claude Code, Codex, and GitHub Copilot CLI.

Closes #215

Summary by CodeRabbit

  • New Features

    • SessionStart skill catalogs now include package-manager-specific commands for loading matching Intent skills.
    • Catalog guidance clarifies that Intent skills are not native agent skills and must be loaded explicitly.
  • Bug Fixes

    • Improved skill-loading instructions to provide the correct command for the detected package manager.
  • Release

    • Added a patch release entry for the Intent package.
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b29a8e4a-373a-4734-a557-0132ffbe9b6f

📥 Commits

Reviewing files that changed from the base of the PR and between f1b97bd and 5b8de99.

📒 Files selected for processing (3)
  • .changeset/kind-bears-visit.md
  • packages/intent/src/hooks/install.ts
  • packages/intent/tests/hooks-install.test.ts

📝 Walkthrough

Walkthrough

The hook installation flow now builds a package-manager-specific skill load command. The generated SessionStart catalog embeds this command and states that Intent skills are not native agent skills. Tests and a patch changeset cover the updated catalog text.

Changes

Session catalog load flow

Layer / File(s) Summary
Build and embed the load command
packages/intent/src/hooks/install.ts
installAgentHook detects the package manager once and passes catalog and load commands to buildHookRunnerScript. The generated script emits LOAD_COMMAND.
Describe and validate skill loading
packages/intent/src/hooks/install.ts, packages/intent/tests/hooks-install.test.ts, .changeset/kind-bears-visit.md
The catalog identifies Intent skills as non-native and shows the configured load command. Tests verify the wording and command. A patch changeset records the release.
Estimated code review effort: 2 (Simple) ~10 minutes
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/fixing-issue-215

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@LadyBluenotes
LadyBluenotes merged commit 481cfbb into main Aug 30, 2026
6 of 8 checks passed
@LadyBluenotes
LadyBluenotes deleted the feat/fixing-issue-215 branch August 30, 2026 05:28
@github-actions github-actions Bot mentioned this pull request Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant