Skip to content

feat: add skills command group for use with LLMs#810

Merged
mcncl merged 1 commit intomainfrom
feat/agent-skills-commands
Apr 30, 2026
Merged

feat: add skills command group for use with LLMs#810
mcncl merged 1 commit intomainfrom
feat/agent-skills-commands

Conversation

@mcncl
Copy link
Copy Markdown
Contributor

@mcncl mcncl commented Apr 29, 2026

Description

Adds a command set for skill to enable adding Buildkite relevant skills to an LLM config

Changes

  • adds a skill command set with add, update and delete options

Usage

# Install a skill locally (current dir)
bk skill add buildkite-cli

# Install globally; will look for ~/.cursor, ~/.claude, or both
bk skill add buildkite-cli --global

# Install to non-Claude/Cursor
bk skill add buildkite-cli --[path/to/location] ~/.pi/agents/skills

Testing

  • Tests have run locally (with go test ./...)
  • Code is formatted (with go fmt ./...)

@mcncl mcncl force-pushed the feat/agent-skills-commands branch from 7d6d60f to 161ddcb Compare April 30, 2026 03:03
@mcncl mcncl marked this pull request as ready for review April 30, 2026 03:08
@mcncl mcncl requested review from a team as code owners April 30, 2026 03:08
Comment thread cmd/skill/skill.go
Comment thread cmd/skill/skill.go
Comment thread cmd/skill/skill.go
Comment thread cmd/skill/skill.go Outdated
Comment thread cmd/skill/skill.go Outdated
@mcncl mcncl force-pushed the feat/agent-skills-commands branch from 11cd84e to 8b6c6bf Compare April 30, 2026 04:22
Comment thread cmd/skill/skill.go
}
}
if !found {
return fmt.Errorf("skill %q not found in github.com/%s", skillName, defaultRepo)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The error message hardcodes defaultRepo but should use the actual repository that was specified. When a user provides a custom repo via the --repo flag, and the skill is not found, the error message will incorrectly reference github.com/buildkite/skills instead of the actual repository used.

// extractSkill needs to accept the repo parameter
func extractSkill(archive, skillName, dest, repo string) error {
    // ...
    if !found {
        return fmt.Errorf("skill %q not found in github.com/%s", skillName, repo)
    }
    return nil
}

Then update the call sites (lines 313, etc.) to pass the repo parameter through the call chain.

Spotted by Graphite

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We only want to use the buildkite/skills repo

@mcncl mcncl requested a review from matthewborden April 30, 2026 04:37
@mcncl mcncl merged commit ba1d5b7 into main Apr 30, 2026
4 checks passed
@mcncl mcncl deleted the feat/agent-skills-commands branch April 30, 2026 04: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.

3 participants