Skip to content

fix: remove should accept the same repo formats as fetch#22

Open
rexdotsh wants to merge 1 commit intovercel-labs:mainfrom
rexdotsh:fix/normalize-repo-urls-in-remove
Open

fix: remove should accept the same repo formats as fetch#22
rexdotsh wants to merge 1 commit intovercel-labs:mainfrom
rexdotsh:fix/normalize-repo-urls-in-remove

Conversation

@rexdotsh
Copy link

The remove command had a manual heuristic (split("/").length === 2) to convert owner/repo to github.com/owner/repo. Any other format (full URLs, github: prefix, etc.) was passed through as-is and couldn't match the stored key in sources.json.

Replaced it with parseRepoSpec() which already handles all input formats and is what fetch uses.

Fixes #21

@vercel
Copy link

vercel bot commented Feb 26, 2026

@rexdotsh is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

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.

opensrc remove fails with full repo URLs that work fine with fetch

1 participant