chore: require utopia-php/client 0.4 - #5
Merged
Conversation
appwrite-labs/sdk-for-manager 5.0.0 requires utopia-php/client ^0.4.1, and consumers such as appwrite-labs/edge install feed alongside it. Client 0.4 adds withFollowRedirects() to the Adapter contract, so the FakeClient test double gains it; the suite passes against 0.4.3.
Greptile SummaryUpdates
Confidence Score: 4/5The PR should not merge until the client constraint is restricted to the validated 0.4 release line. The lock file now correctly selects client 0.4.3, so the prior manually resolved finding is fixed. However, the subsequent change from Files Needing Attention: composer.json Important Files Changed
Prompt To Fix All With AI### Issue 1
composer.json:48
**Constraint permits untested versions**
Using `^0.4` permits releases from 0.4.0 up to, but excluding, 1.0.0, including potentially incompatible 0.5+ releases of this pre-1.0 dependency. Compatibility was established only for the 0.4 contract, so future installs could select an untested minor version. Keep the dependency on the validated 0.4 release line.
```suggestion
"utopia-php/client": "0.4.*",
```
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (4): Last reviewed commit: "chore: use caret range for utopia-php/cl..." | Re-trigger Greptile |
A single range keeps the resolver's job simple; 0.3 support is dropped rather than kept as an alternative.
CI installs from the lock file, so the constraint change needs the matching lock update.
loks0n
approved these changes
Sep 6, 2026
loks0n
approved these changes
Sep 7, 2026
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
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.
appwrite-labs/sdk-for-manager 5.0.0 requires
utopia-php/client ^0.4.1. Consumers that install it next toutopia-php/feed(appwrite-labs/edge) cannot resolve because feed pins the client to 0.3.Client 0.4 only adds
withFollowRedirects()to theAdaptercontract (redirects stay off by default), so theFakeClienttest double gains that method.composer test,composer lint, and PHPStan pass against client 0.4.3.A 1.2.0 tag is needed once merged so edge can pick it up.