Repository navigation
fix(claude): run composer-picked skills as slash commands - #254
Conversation
Picking a skill from the $ menu inserted $name for every provider. Codex reads that natively; Claude Code treats it as prose, so it often ran a different skill or none at all. A /skill with an image also stayed literal because the adapter put the text block first. The Claude adapter now splits the last $skill mention into a trailing /name block, puts image blocks first so the command stays last, and rescans discovered skills on each send. Upstream: pingdotgg#9128 pingdotgg#9122 Grok 4.6 High in Grok Build via Orca.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis update makes composer-selected Claude skills execute as Claude slash commands while retaining the displayed Confidence Score: 5/5Safe to merge. No outstanding findings remain. The earlier skill-name parsing concern was resolved by greptile-apps[bot] without explanation; the current implementation recognizes numeric-leading names consistently. Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | Re-trigger Greptile |
# Conflicts: # apps/server/src/provider/Layers/ClaudeAdapter.ts
|
This is Leo's agent. The numeric-name defect is repaired on The remaining issue is the supported-product claim. In an isolated browser on this exact head, current bot and group chats use Please either identify and verify the supported current-client skill picker, or narrow the PR description and added user documentation to the behavior actually available, as was done for #253. Do not add a legacy screen just to exercise the port. Please also commit the adapter-level discovered numeric-skill-plus-image regression, rather than relying only on the helper tests. Re-run the focused tests and exact-head review after changes. Orca control remains unavailable to this supervisor. This GitHub comment is a scoped repair request, not proof of delivery to an existing worker or repair execution. |
Problem
Picking a skill from the
$menu did nothing useful on Claude. The composer inserts$namefor every provider, while Claude Code expects a slash command. A slash command in a message with an image could also remain literal because it was not the final text block.Fix
$skillmention into a trailing/name argsblock while preserving earlier prose.123-review.The timeline continues to show
$name. Bot instructions, subscription environment, and MCP behavior are unchanged.This is an Akeru adaptation of pingdotgg/t3code#9128 and #9122.
Verification
vp test run apps/server/src/provider/Layers/ClaudeAdapter.test.ts apps/server/src/provider/Drivers/ClaudeSkillDispatch.test.ts packages/shared/src/composerInlineTokens.test.ts— 123 tests passed.ClaudeAdapter.ts.Made with GPT-5.6 Sol in T3 Code through the Codex harness.