cli: install only authored skills - #310
Open
bensandler-stripe wants to merge 3 commits into
Open
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.
- 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>
bensandler-stripe
enabled auto-merge (squash)
September 8, 2026 18:49
- 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>
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
- 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>
bensandler-stripe
marked this pull request as ready for review
September 9, 2026 19:41
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
1 participant
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.