Skip to content

Update PHP SDK to 26#194

Merged
loks0n merged 3 commits into
mainfrom
codex/update-php-sdk-26
Jun 17, 2026
Merged

Update PHP SDK to 26#194
loks0n merged 3 commits into
mainfrom
codex/update-php-sdk-26

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Summary

  • Update appwrite/appwrite dependency constraint to ^26.0 and lock to 26.0.0.
  • Adjust redirect status code enum calls for PHP SDK 26 compatibility.

Testing

  • composer lint
  • composer check
  • composer test (fails in NHost/Supabase E2E setup because DB was offline after 5 tries)

@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

  • Updates the Appwrite PHP SDK dependency to 26.0.0.
  • Adjusts redirect status enum usage for SDK 26 compatibility.
  • Adds project headers to direct multipart upload calls.
  • Refactors Appwrite database reader filtering and per-database row access.

Confidence Score: 3/5

The dependency and SDK compatibility changes are scoped, but some migration paths depend on external Appwrite database behavior that was not fully exercised.

Static checks and most tests were reported, while the end-to-end database setup did not complete because the database was unavailable.

No specific files require follow-up from this review.

T-Rex T-Rex Logs

What T-Rex did

  • Reviewed the SDK 26 redirect enum compatibility before and after artifacts to capture environment state and blockers.
  • Noted that the PHP enum probe could not run in this environment, so conclusive verification about SDK 26 factories invokability could not be achieved.
  • Examined the appwrite-upload-project-header logs before and after to verify header handling and fake API contract behavior.
  • Examined the callback inputs before and after to confirm updated document metadata, per-database counts, and routing through the docs-route backend.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (3): Last reviewed commit: "Pass project header on direct Appwrite u..." | Re-trigger Greptile

Comment on lines +3382 to +3386
301 => StatusCode::MOVEDPERMANENTLY(),
302 => StatusCode::FOUND(),
307 => StatusCode::TEMPORARYREDIRECT(),
308 => StatusCode::PERMANENTREDIRECT(),
default => StatusCode::MOVEDPERMANENTLY(),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Update runtime mappings The SDK bump brings new Runtime and BuildRuntime enum values, but the import mappings below still only accept the older runtime set. When a source project has a function or site using a newer Appwrite 1.9 runtime such as php-8.4, node-23, go-1.24, or newer Flutter build runtimes, import reaches the default branch and fails with Invalid Runtime or Invalid Build Runtime instead of recreating the resource. Please extend the function and site runtime match expressions for the SDK 26 enum values as part of this compatibility update.

Comment thread composer.json
"ext-openssl": "*",

"appwrite/appwrite": "24.*",
"appwrite/appwrite": "^26.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Migrate new policies SDK 26 exposes additional project policy IDs for denying aliased, disposable, free, and corporate emails, but this dependency bump does not add those fields to the migration resource. A project with those email restrictions enabled can be exported and imported without carrying the settings into the destination, reopening sign-up paths that were blocked on the source. Please extend the policies export/resource/import path to preserve the new policy values introduced by this SDK version.

Artifacts

Repro: focused PHP policy migration harness

  • Contains supporting evidence from the run (text/x-php; charset=utf-8).

Repro: harness output showing SDK 26 email restriction policies dropped

  • Keeps the command output available without making the summary code-heavy.

View artifacts

T-Rex Ran code and verified through T-Rex

Comment thread src/Migration/Sources/Appwrite/Reader/Database.php
@loks0n loks0n merged commit d0b37d2 into main Jun 17, 2026
4 checks passed
@loks0n loks0n deleted the codex/update-php-sdk-26 branch June 17, 2026 08:52
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.

3 participants