cli: install only authored skills - #310
Closed
bensandler-stripe wants to merge 4 commits into
Closed
bensandler-stripe wants to merge 4 commits into
bensandler-stripe wants to merge 4 commits into
Conversation
bensandler-stripe
commented
Sep 8, 2026
bensandler-stripe
commented
Contributor
- Intercept Incur skill installation and delegate to the canonical repository installer, following Shopify's ucp-cli interception pattern.
- Preserve global and project-local installation while preventing generated command-reference skills.
- Add regression coverage for aliases, help handling, and installer delegation.
bensandler-stripe
enabled auto-merge (squash)
September 8, 2026 18:49
bensandler-stripe
marked this pull request as draft
September 9, 2026 17:56
auto-merge was automatically disabled
September 9, 2026 17:56
Pull request was converted to draft
bensandler-stripe
marked this pull request as ready for review
September 9, 2026 19:41
kreese-stripe
approved these changes
Sep 11, 2026
- Intercept Incur skill installation and delegate to the canonical repository installer, following Shopify's ucp-cli interception pattern. - Preserve global and project-local installation while preventing generated command-reference skills. - Add regression coverage for aliases, help handling, and installer delegation. Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com> Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com> Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com>
- Remove only Incur-recorded link-cli generated skills after authored-skill installation succeeds. - Clear stale Incur metadata so unrelated commands stop advertising a recurring sync CTA. - Force the child skills installer to resolve the public GitHub repository under Stripe GH_HOST settings. - Cover successful migration, failed installation, and authored-skill preservation. Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com>
- Normalize Incur global flags before intercepting skills add so generated skills cannot bypass the authored installer. - Remove legacy skills from every agent-specific path recorded by Incur while preserving authored skill directories. - Embed authored skill content and install it natively from standalone executables without requiring Node or npm. - Add regression coverage for flagged invocations, noncanonical cleanup paths, and native authored-only sync. Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com>
- Preserve main's MCP package-runner import alongside the authored-skills imports. - Apply Biome's canonical ordering after resolving the rebase conflict. Committed-By-Agent: codex Co-authored-by: codex <noreply@openai.com>
bensandler-stripe
force-pushed
the
bensandler/curated-link-cli-skills
branch
from
September 11, 2026 15:53
f611ddc to
93acaac
Compare
Contributor
Author
|
Closing in favor of the upstream Incur fix in wevm/incur#237, which adds the supported opt-out for generated skills. The replacement is a much smaller Link CLI change using sync: false. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.