Skip to content

feat: distribute selected skills from their owning packages - #260

Merged
LadyBluenotes merged 8 commits into
feat-focused-authoring-reviewfrom
feat-skill-distribution
Sep 11, 2026
Merged

feat: distribute selected skills from their owning packages#260
LadyBluenotes merged 8 commits into
feat-focused-authoring-reviewfrom
feat-skill-distribution

Conversation

@LadyBluenotes

Copy link
Copy Markdown
Member

🎯 Changes

Stacked on #258. Adds a saved repository-distribution choice to the maintainer workflow so package-owned skills can also reach consumers through GitHub skill installers and native plugins.

maintainer setup records explicitly selected skills or an opt-out in skill_tree.yaml. maintainer sync generates Claude/Cursor plugin metadata and copyable npx skills add and gh skill add commands pointing to the existing directories. maintainer check catches an unconfigured choice and stale generated files. Adding a skill never expands the selection automatically.

Skills stay alongside their code. Existing plugin fields and other marketplace entries are preserved; conflicting identities, source roots, and missing selected prerequisites are reported before synchronization writes files. The authoring procedure explains this choice during setup and distinguishes pre-install discovery guidance from installed-version implementation guidance.

The selection controls generated metadata and suggested commands; third-party installers retain their own repository discovery behavior. Publication stays with the library's release process. Documentation is in #259, which remains open for the maintainer release.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 7518dd5b-5c7d-47d2-822e-d04595756cde

📥 Commits

Reviewing files that changed from the base of the PR and between 44e2540 and 6da7880.

📒 Files selected for processing (14)
  • .changeset/selected-skill-distribution.md
  • CONTRIBUTING.md
  • packages/intent/meta/generate-skill/SKILL.md
  • packages/intent/meta/generate-skill/references/distribution.md
  • packages/intent/meta/generate-skill/references/maintainer-commands.md
  • packages/intent/package.json
  • packages/intent/src/cli.ts
  • packages/intent/src/commands/maintainer.ts
  • packages/intent/src/maintainer/distribution.ts
  • packages/intent/src/maintainer/sync.ts
  • packages/intent/tests/cli.test.ts
  • packages/intent/tests/distribution.test.ts
  • packages/intent/tests/integration/distribution-installers.test.ts
  • packages/intent/tests/maintainer.test.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@nx-cloud

nx-cloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 8133d37

Command Status Duration Result
nx affected --targets=test:eslint,test:sherif,t... ✅ Succeeded 46s View ↗
nx run-many --targets=build ✅ Succeeded 3s View ↗

☁️ Nx Cloud last updated this comment at 2026-09-11 04:24:43 UTC

@pkg-pr-new

pkg-pr-new Bot commented Sep 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@tanstack/intent@260

commit: 8133d37

@codspeed-hq

codspeed-hq Bot commented Sep 9, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing feat-skill-distribution (6da7880) with feat-focused-authoring-review (44e2540)

Open in CodSpeed

@LadyBluenotes
LadyBluenotes marked this pull request as ready for review September 11, 2026 02:54
@LadyBluenotes
LadyBluenotes added this pull request to stack #274 September 11, 2026 22:02
@LadyBluenotes
LadyBluenotes requested a review from a team as a code owner September 11, 2026 22:17
@LadyBluenotes
LadyBluenotes force-pushed the feat-skill-distribution branch from 8133d37 to c11c7a7 Compare September 11, 2026 22:22
@LadyBluenotes
LadyBluenotes force-pushed the feat-skill-distribution branch from 657aee5 to ad1e250 Compare September 11, 2026 22:25
@LadyBluenotes
LadyBluenotes removed the request for review from a team September 11, 2026 22:26
@LadyBluenotes
LadyBluenotes merged commit 3437a6c into main Sep 11, 2026
7 of 8 checks passed
@LadyBluenotes
LadyBluenotes deleted the feat-skill-distribution branch September 11, 2026 22:32
@github-actions github-actions Bot mentioned this pull request Sep 12, 2026
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