Skip to content

feat(skills): support repo and external commands#261

Merged
shiny-code-bot merged 1 commit into
mainfrom
feat/external-skill-commands
May 31, 2026
Merged

feat(skills): support repo and external commands#261
shiny-code-bot merged 1 commit into
mainfrom
feat/external-skill-commands

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • Add commands[].source for explicit skill, repo, and external command metadata.
  • Preserve strict resource_path validation for default source: skill bundled helper commands.
  • Render repo/external commands without pretending they are skill-bundled helper scripts.
  • Migrate .codex/skills/remote-tests and .codex/skills/test-tui onto structured repo/external command metadata.

Design Notes

Refs #260.

This keeps the existing bundled helper contract intact:

  • source: skill is the default for backwards compatibility.
  • source: skill still requires resource_path, a matching resources[] entry, and an existing helper file.
  • source: repo is allowed only for repo-scoped skills and must not declare resource_path.
  • source: external must not declare resource_path and does not perform filesystem checks.

The schema is intentionally small. It does not add command variants, option metadata, network/capability hints, or shared skill-library migration.

Agent Review

Design reviewed by Opus, Antigravity/Gemini-style, and GPT agents before implementation. All agreed on the key invariant: do not relax bundled resource_path validation. The final implementation uses one command list with an explicit source discriminator rather than fake resources for repo/external workflows.

Validation

  • cargo test -p code-core skills::
  • ./build-fast.sh

@shiny-code-bot shiny-code-bot merged commit f3af876 into main May 31, 2026
1 check passed
@shiny-code-bot shiny-code-bot deleted the feat/external-skill-commands branch May 31, 2026 12:48
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