Skip to content

refactor: use utopia-php/client 0.5's names - #18

Merged
ChiragAgg5k merged 1 commit into
mainfrom
refactor/client-0.5-names
Sep 24, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
refactor/client-0.5-names

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

utopia-php/client 0.5 moved Utopia\Client to Utopia\Client\Client and keeps the old name only as an alias until its next major. This PR requires ^0.5 and imports the new name in Deployments, Jobs, Sandboxes and the README, so client can drop the alias.

  • The only code changes are those imports, plus Pint's import re-sorting. Behaviour is identical, because the old name is an alias of the same class in 0.5.
  • The lock moves utopia-php/client from 0.4.3 to 0.5.1 and nothing else.
  • Locally, composer format:check, composer analyze and composer refactor:check pass, and composer test passes 43 tests.

Part of removing client's alias shim (appwrite/appwrite#13828). Appwrite needs an sdk-for-php release with this change before client's next major.

client 0.5 moved Utopia\Client under Utopia\Client\Client and keeps
the old name only as an alias until its next major. Require ^0.5 and
import the new name so client can drop the alias.
@greptile-apps

greptile-apps Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The PR appears safe to merge, with a non-blocking gap in observable test coverage for the upgraded HTTP client.

Fix All in Claude CodeFindings

  1. P2 Real client remains untested ▶
Fix with agent prompt
### Issue 1
composer.json:10
The upgrade changes the HTTP client used when a resource client is constructed without an argument, but the tests inject a PSR-18 test double instead. They therefore cannot catch a problem with constructing or configuring the upgraded client. Please add a test that makes an observable request with the configured client, rather than asserting version pins or source text.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR upgrades utopia-php/client to 0.5.1 and updates the three resource clients and README to import its new client name.

  • The changed real-client integration lacks an observable-behavior test; existing resource tests use a PSR-18 double.

Reviews (1) · Last reviewed commit: "refactor: use utopia-php/client 0.5's na..."

Comment thread composer.json
@ChiragAgg5k
ChiragAgg5k merged commit 5ee5ee3 into main Sep 24, 2026
5 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the refactor/client-0.5-names branch September 24, 2026 12:27
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.

1 participant