Skip to content

Wire shaft-skills for external installation (#4350) - #4369

Open
MohabMohie wants to merge 10 commits into
mainfrom
ChaosEngine/shaft-skills-distribution-4350
Open

Wire shaft-skills for external installation (#4350)#4369
MohabMohie wants to merge 10 commits into
mainfrom
ChaosEngine/shaft-skills-distribution-4350

Conversation

@MohabMohie

Copy link
Copy Markdown
Contributor

This PR wires shaft-skills for external installation (marketplace.json + README). Created to finalize agent work and preserve changes for review.

MohabMohie and others added 3 commits July 29, 2026 16:48
Add .claude-plugin/marketplace.json so the existing shaft-skills/ pack
installs via Claude Code (/plugin marketplace add ShaftHQ/SHAFT_ENGINE)
and is discovered by the cross-agent skills CLI (npx skills add), plus a
README section pointing external users at both paths.

The skills field lists the seven skill directories individually: the
skills CLI resolves each entry's parent as the scan container, while
Claude Code loads each listed directory as one skill - the explicit list
is the only shape both tools resolve. strict:false keeps the marketplace
entry as the single manifest (no per-plugin plugin.json needed).

Verified end-to-end locally: claude plugin validate passes; a local
marketplace add + install exposes exactly the 7 skills (~605 always-on
tokens) with references/ shipped in the plugin cache; npx skills add
--list discovers all 7 (previously 0 from shaft-skills/); a real CLI
install of two skills landed in .claude/skills/.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PkZJgRNK97QeuGHK3aVULT
…kill dirs

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PkZJgRNK97QeuGHK3aVULT
…kill dirs

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PkZJgRNK97QeuGHK3aVULT
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

MohabMohie and others added 7 commits July 29, 2026 18:59
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…aft-skills paths)\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
… shaft-skills stay in sync\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
… is a helper dir)\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…low-timeout guard\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@MohabMohie

Copy link
Copy Markdown
Contributor Author

Merged changes from PR #4368 into this branch and pushed. I reviewed related issues (4350, 4366) — this branch now includes the marketplace.json wiring, README install docs, shaft-skills/* SKILL.md content, and CI validation workflow to keep manifest and skill dirs consistent. Key files changed: .claude-plugin/marketplace.json, README.md, .github/workflows/validate-shaft-skills.yml, shaft-skills/* and per-skill SKILL.md files.\n\nI closed PR #4368 (its contents are now included here). Build and static checks passed locally and the PR validation jobs ran green in CI; remaining analyzers (CodeQL) are running. If anything from 4350/4366 appears missing, reply here and it will be addressed in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant