Skip to content

Support bundled skills - #69

Merged
adampoit merged 1 commit into
mainfrom
bundled-skills
Aug 6, 2026
Merged

Support bundled skills#69
adampoit merged 1 commit into
mainfrom
bundled-skills

Conversation

@adampoit

@adampoit adampoit commented Aug 6, 2026

Copy link
Copy Markdown
Owner

No description provided.

@not-adam

not-adam Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Mira PR Walkthrough

This PR adds support for bundled skills in the Patchlane CLI. Instead of always fetching skills from GitHub, the agents command now falls back to reading skills from a local skills/ directory shipped with the package. Bundled skills are used when no --ref flag or remote source environment variable is explicitly set, avoiding network access and making local development easier.

graph LR
    cli["src/cli.ts"]
    agents["src/agents-install.ts"]
    skills["skills/ (bundled)"]
    github["GitHub raw (remote)"]
    cli -->|"agents command"| agents
    agents -->|"useBundled? read locally"| skills
    agents -->|"else: fetch"| github
Loading
Confidence: 4/5   ◉◉◉◉○   Safe with minor fixes
  • The logic is sound and well-structured, but the one new integration test only covers the bundled path; no test validates the existing remote-fetch fallback path still works correctly with the new branching logic.

Blast Radius — 2 dependent repositories, 3 total references

adampoit/mira — 1 reference
adampoit/wezterm — 2 references

5 files reviewed


Comment @not-adam help to get the list of available commands and usage tips.

@adampoit
adampoit merged commit 712935c into main Aug 6, 2026
2 checks passed
@adampoit
adampoit deleted the bundled-skills branch August 6, 2026 05:18
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.

1 participant