Skip to content

feat: add skill-build action for ToolHive skills#12

Merged
JAORMX merged 1 commit intomainfrom
feat/skill-build-action
Mar 10, 2026
Merged

feat: add skill-build action for ToolHive skills#12
JAORMX merged 1 commit intomainfrom
feat/skill-build-action

Conversation

@JAORMX
Copy link
Copy Markdown
Collaborator

@JAORMX JAORMX commented Mar 10, 2026

Summary

  • Add new skill-build composite action that builds ToolHive skills (SKILL.md directories) into OCI artifacts via thv skill build
  • Supports optional push to any OCI-compatible registry (ghcr.io, Docker Hub, etc.) — callers handle registry auth themselves via docker/login-action or similar
  • Automatically manages the thv serve daemon lifecycle, reusing an existing one if already running
  • All inputs passed via env: mappings (no expression injection), commands built with bash arrays (no eval)

Test plan

  • test-skill-build-action CI job: creates a test skill, builds with default tag, builds with custom tag, verifies output references
  • validate-action-metadata CI job: validates skill-build/action.yml with yq
  • Manual: verify push flow works with docker/login-action + ghcr.io (requires a ToolHive version with skill support)

🤖 Generated with Claude Code

…facts

Add a new composite action that builds ToolHive skills from a local
directory (containing SKILL.md) into OCI artifacts, with optional push
to a remote registry. The action manages the thv daemon lifecycle
automatically, reusing an existing daemon if one is already running.

Registry credentials are intentionally not handled by this action —
callers should use docker/login-action or similar beforehand, following
the same pattern as docker/build-push-action.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@rdimitrov rdimitrov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@JAORMX JAORMX merged commit 9fb5f21 into main Mar 10, 2026
7 checks passed
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