Skip to content

docs(install): document the verified gh skill install route - #150

Merged
sajor2000 merged 2 commits into
mainfrom
docs/gh-skill-route
Oct 21, 2026
Merged

sajor2000 merged 2 commits into
mainfrom
docs/gh-skill-route

Conversation

@sajor2000

Copy link
Copy Markdown
Owner

What

Documents the GitHub CLI skills-only install route, verified end to end — MedSci audit gap #7.

  • docs/setup.md: new "Claude Code via GitHub CLI (skills only)" subsection under the easiest install path, with the --all and single-skill forms and the three caveats (skills only — no agents, commands, hooks, or auto-update; flat names can collide with a user's own skills; --all also picks up the sample skills from tests/fixtures/).
  • README.md: matching summary subsection plus rows in the platform install table and the update table (no auto-update; re-run after a new release).

Verification (performed on gh 2.100.0 against release tag methodsmith-v5.1.1)

  • gh skill install sajor2000/methodsmith --all --agent claude-code --scope project in a throwaway clone: succeeds, discovers the skills via the plugins/ convention, and pins to the latest release tag automatically. Caveat observed: the sweep also installs the fixture sample skills under tests/fixtures/ — documented, with the single-skill form preferred.
  • gh skill install sajor2000/methodsmith method-citations --agent claude-code --scope project --pin methodsmith-v5.1.1: installs exactly that one skill.
  • Tests were run before any doc claim was written, per the issue's rule.

Closes #145

Validation

  • bun run release:validate — in sync (55 agents, 67 skills, 0 MCP servers)
  • bun test tests/displayed-counts.test.ts — pass
  • Markdown-only change; no release-owned files touched.

MedSci audit gap #7. Verified on gh 2.100.0 against methodsmith-v5.1.1:
the plugins/ convention is discovered, --all sweeps every skill (plus the
tests/fixtures sample skills, now a documented caveat), a single skill
installs cleanly with an optional --pin, and the route carries no
auto-update. Setup guide gains the full subsection; README gains the
summary, install-table row, and update-table row.

Closes #145

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Next included review available in 56 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 69 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: d50f7802-5462-40c3-96d3-e333fde30445
📥 Commits

Reviewing files that changed from the base of the PR and between 1daea92 and b2c5168.

📒 Files selected for processing: 2
  • README.md
  • docs/setup.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@sajor2000
sajor2000 merged commit 8dae103 into main Oct 21, 2026
3 checks passed
@sajor2000
sajor2000 deleted the docs/gh-skill-route branch October 21, 2026 02:16
sajor2000 added a commit that referenced this pull request Oct 21, 2026
Gaps #3, #5, #7, #10 closed (PRs #150, #151, #153, #154); #1, #6, #8
tracked as issues #141, #144, #146 with scoping; #9 wontfix as #147.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
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.

Verify and document (or reject) the gh skill install route (MedSci audit gap #7)

1 participant