Jinal-Sarvaiya opened a new pull request, #117:
URL: https://github.com/apache/camel-kit/pull/117

   ## Summary
   
   - **Bug 1:** Hide internal skills (`camel-design`, `camel-implement`, 
`camel-test`, `camel-verify`) from Claude Code via `user-invocable: false` and 
`disable-model-invocation: true` in `SkillResourceInstaller`
   - **Bug 2:** Add hyphenated `user-invocable` frontmatter mapping for Claude 
so Claude Code's skill picker can discover skills correctly
   - **Bug 3:** Rewrite all 9 user-facing SKILL.md descriptions as `"Use 
when..."` contextual triggers that match Claude Code's `when_to_use` discovery 
heuristic
   - **Bug 4:** Set `generatesCommandStubs: false` in `claude.yaml` to 
eliminate 9 duplicate command stubs — Claude Code discovers skills natively 
through frontmatter, not command files
   
   ## Files changed (15)
   
   | Area | Files |
   |------|-------|
   | Core | `SkillResourceInstaller.java` — added 
`addClaudeReadableInternalSkillMetadata()`, added CLAUDE to hyphenation block |
   | Config | `agents/registry/claude.yaml` — `generatesCommandStubs: false` |
   | Skills | 9 SKILL.md files — description rewrites to "Use when..." |
   | Template | `templates/claude/camel-ship.md` — native ship description 
rewrite |
   | Tests | `ClaudeGeneratorTest` (3 new tests), `ShippedAssetStructureTest`, 
`AgentRegistryTest` |
   
   ## Test plan
   
   - [x] 
`ClaudeGeneratorTest.marksInternalSkillsAsNotUserOrModelInvocableForClaude` — 
verifies Bug 1
   - [x] `ClaudeGeneratorTest.addsHyphenatedUserInvocableMetadataForClaude` — 
verifies Bug 2
   - [x] `ClaudeGeneratorTest.skillDescriptionsAreClaudeCodeDiscoverable` — 
verifies Bug 3
   - [x] `ClaudeGeneratorTest.preservesBaseSkillBehavior` — verifies Bug 4 (no 
command stubs)
   - [x] `ShippedAssetStructureTest` — full structural coherence across all 8 
agents
   - [x] `AgentRegistryTest` — registry reflects updated config
   - [x] `./mvnw -Psourcecheck validate -B` passes
   
   ## Summary by Sourcery
   
   Make Camel-Kit skills discoverable natively in Claude Code while safely 
migrating away from generated command stubs.
   
   New Features:
   - Enable Claude Code to discover Camel-Kit skills natively from 
`.claude/skills/` frontmatter without generated command stubs.
   
   Bug Fixes:
   - Hide internal workflow skills from Claude Code user invocation and model 
auto-selection.
   - Improve Claude Code skill discovery through hyphenated invocation metadata 
and contextual `Use when` descriptions.
   - Remove obsolete generated Claude command stubs while preserving 
user-created commands.
   - Prevent initialization from operating through a symlinked legacy Claude 
command directory and accept legacy configurations in Doctor.
   
   Documentation:
   - Update the changelog and agent asset documentation to reflect Claude 
Code's native skill layout.
   
   Tests:
   - Expand Claude generator, asset structure, registry, Doctor, and 
initialization coverage for native skill discovery, legacy stub cleanup, 
compatibility, and symlink safety.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to