feat: add methods to fetch package info from new composer 2 api

[JellyBellyDev]

I add two new add methods to fetch package info from new composer 2 API.

I tried to add spec test but I'm not understand why not work.

@robbieaverill could you help me understand? Thanks

[robbieaverill]

Hey @JellyBellyDev, thank you for taking the time to make this pull request. I've had a quick look - do you think it's possible to change from Composer 1 to Composer 2 API results under the hood, while keeping the API surface of this package the same? If that's possible, that would be my suggested approach, even if it's only for the Versions data collection. It would be nice to avoid adding the new methods like getComposer2() etc.

[JellyBellyDev]

Hi @robbieaverill the new version of composer api expose two new endpoint:

• /p2/$vendor/$package.json (only tagged releases)
• /p2/$vendor/$package~dev.json (tagged releases and branches)

If we want change under the hood the method getComposer() we should use /p2/$vendor/$package~dev.json but we must apply this logic, because the new api don't return the name of the version as key of the object.

If we do this, wouldn't it be fair to also provide a method that uses the lite version (/p2/$vendor/$package.json)?

[robbieaverill]

Hi @JellyBellyDev, apologies for the delay in responding to you. I personally don't think it's important for this library to expose the underlying Packagist API responses. The fact that we have one public method that does this already is okay. If you want to add another protected method to expose the lite version, go for it. Hopefully we can avoid exposing this as a public method though.

Your logic change seems reasonable to retain compatibility with the existing data structure we provide here.

[JellyBellyDev]

Hi @robbieaverill, thanks for the reply, don't worry about the late! ;)
Forgive me, but it is not yet clear to me what you want me to do! Could you explain better? Thanks

[robbieaverill]

Hi @JellyBellyDev, apologies. My recommendation would be to change the getComposer() method to return the new information from Packagist and apply this logic, but not create a new method like getComposer2(). Let me know what you think 🙂

[robbieaverill]

Hey @JellyBellyDev, are you still keen to get this over the line? It'd be a great addition to get merged

[JellyBellyDev]

@robbieaverill forgive me, I completely forgot about this issue. Complete ASAP! ;)

[JellyBellyDev]

@robbieaverill PTAL

I override the method getComposer with new API /p2/%s~dev.json like you suggested and add new method getComposerLite for the lite version with only tagged releases!

[robbieaverill]

Thanks!

[JellyBellyDev]

it was a pleasure! 🚀