Skip to content

refactor(skills): streamline skill hierarchy with /install as primary entry point#15

Closed
pmTouchedTheCode wants to merge 2 commits into
mainfrom
refactor/skill-hierarchy
Closed

refactor(skills): streamline skill hierarchy with /install as primary entry point#15
pmTouchedTheCode wants to merge 2 commits into
mainfrom
refactor/skill-hierarchy

Conversation

@pmTouchedTheCode

Copy link
Copy Markdown

Summary

  • Hide advanced/specialized skills from slash command menu to reduce clutter for new users
  • Consolidate onboarding experience around /install as the primary SDK setup entry point
  • Enhance /doctor for comprehensive health checks and /ios-setup as a sub-skill

Changes

  • skills.ts: Add HIDDEN_PACKAGE_SKILLS and HIDDEN_LOCAL_SKILLS sets to control menu visibility
  • install/SKILL.md: Major rewrite as comprehensive orchestrator with visual progress tracking and iOS NSE support
  • doctor/SKILL.md: Expanded health check capabilities
  • ios-setup/SKILL.md: Refactored as sub-skill invoked by /install
  • Documentation: Updated README.md and llms.txt to reflect new skill hierarchy
  • Tests: Updated slash-command-menu tests for hidden skills

Test plan

  • Run bun test to verify all tests pass
  • Run bun run dev and verify hidden skills don't appear in / autocomplete
  • Verify hidden skills still work when typed directly (e.g., /integration)
  • Test /install flow on a sample iOS project
  • Test /doctor health check output

🤖 Generated with Claude Code

… entry point

- Hide advanced skills from slash command menu (integration, event-tracking,
  user-management, personalization, api-triggered-campaigns, ios-setup)
- Skills remain accessible by typing full command directly
- Enhance /install skill as comprehensive SDK installation orchestrator
  with visual progress tracking and iOS NSE setup support
- Expand /doctor skill for thorough SDK health checks
- Refactor ios-setup as sub-skill invoked by /install
- Update documentation (README.md, llms.txt) to reflect new hierarchy
- Update slash-command-menu tests for hidden skills

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

coderabbitai Bot commented Jan 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

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 and usage tips.

Update generateHelpText test to check for visible skills (/install, /doctor)
instead of hidden ones (/integration, /event-tracking). Also verify that
hidden skills do NOT appear in the help text.

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

Copy link
Copy Markdown
Contributor

Closing this draft PR — the changes are no longer applicable after the command-mode architecture refactor in #22.

What happened:

  • Package skills (@clix-so/clix-agent-skills) were removed entirely, making HIDDEN_PACKAGE_SKILLS unnecessary
  • LOCAL_SKILLS reduced to install/doctor only (debug removed)
  • install/SKILL.md and doctor/SKILL.md rewritten with PreparationContext-based architecture
  • ios-setup moved from skills to UI components
  • README/llms.txt updated differently by subsequent commits

The goals of this PR (simplify skill hierarchy, /install as primary entry) are now achieved through the preparation-based task flow in #22.

@pitzcarraldo pitzcarraldo deleted the refactor/skill-hierarchy branch February 24, 2026 05:57
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.

2 participants