Skip to content

add packagist support#3

Open
xHeaven wants to merge 3 commits intovercel-labs:mainfrom
xHeaven:packagist
Open

add packagist support#3
xHeaven wants to merge 3 commits intovercel-labs:mainfrom
xHeaven:packagist

Conversation

@xHeaven
Copy link
Copy Markdown

@xHeaven xHeaven commented Jan 14, 2026

This PR adds support for PHP packages via Packagist.
It adds 3 aliases: packagist, composer and php

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 14, 2026

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

A member of the Team first needs to authorize it.

Comment thread src/lib/registries/packagist.ts
Removed dev filtering, untagged branches are available on the ~dev.json endpoint.
Fallback was flagged by vercel bot.
@ctate
Copy link
Copy Markdown
Collaborator

ctate commented Feb 2, 2026

Nice work - this is well-structured and follows the existing patterns cleanly

Two small fixes before merging:

  1. Remove the unused includeDev parameter from getAvailableVersions (line 143 in packagist.ts) - it's defined but never used.

  2. Add Bitbucket git URL normalization in extractRepoUrl:

.replace(/^git@bitbucket\.org:/, "https://bitbucket.org/")

You already handle GitHub and GitLab, just missing this one.

@xHeaven
Copy link
Copy Markdown
Author

xHeaven commented Feb 3, 2026

Nice work - this is well-structured and follows the existing patterns cleanly

Two small fixes before merging:

1. Remove the unused includeDev parameter from `getAvailableVersions` (line 143 in packagist.ts) - it's defined but never used.

2. Add Bitbucket git URL normalization in extractRepoUrl:
.replace(/^git@bitbucket\.org:/, "https://bitbucket.org/")

You already handle GitHub and GitLab, just missing this one.

I'll take a look at this, thanks!

@ctate
Copy link
Copy Markdown
Collaborator

ctate commented Apr 9, 2026

@xHeaven Hey! Just a heads up — the CLI was rewritten in Rust (v0.7.0), so this PR can't be merged as-is against the old Node.js codebase. Packagist support is still something we'd like to add though.

Feel free to re-submit against the Rust CLI (packages/opensrc/cli/), or I'm happy to implement it myself. Let me know!

@xHeaven
Copy link
Copy Markdown
Author

xHeaven commented Apr 15, 2026

@xHeaven Hey! Just a heads up — the CLI was rewritten in Rust (v0.7.0), so this PR can't be merged as-is against the old Node.js codebase. Packagist support is still something we'd like to add though.

Feel free to re-submit against the Rust CLI (packages/opensrc/cli/), or I'm happy to implement it myself. Let me know!

Hey @ctate! Thanks for the heads-up, glad to hear about the Rust rewrite. I probably won’t have enough free time to port this anytime soon, feel free to take a look at 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.

2 participants