chore: update sdks add response models by ChiragAgg5k · Pull Request #10554 · appwrite/appwrite

ChiragAgg5k · 2025-09-26T10:03:51Z

No description provided.

coderabbitai · 2025-09-26T10:03:58Z

📝 Walkthrough

Walkthrough

The PR bumps multiple SDK version numbers in app/config/platforms.php across Client, Console, and Server platforms (e.g., Web 20.1.0 → 21.0.0, Flutter 19.1.0 → 20.0.0, Node.js 19.1.0 → 20.0.0, Ruby 18.1.0 → 19.0.0, etc.). It updates src/Appwrite/Platform/Tasks/SDKs.php to ensure write permissions before removal (using chmod -R u+w), to clear target contents while preserving .git using find . -mindepth 1 ! -path "./.git*" -delete, and to handle existing GitHub PRs by editing the PR title/body with gh pr edit when a PR already exists, including success/failure handling and adjusted cleanup commands.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

feat: automatic pr creation in sdk release script #10457 — Modifies src/Appwrite/Platform/Tasks/SDKs.php to add/extend GitHub PR automation for SDK pushes (strong overlap with PR-edit logic).
chore: update dart 16.1.0, flutter 17.0.2 and cli 8.2.2 #10161 — Changes both app/config/platforms.php and the SDKs task script (directly related to SDK version and push-flow edits).
update cli to 10.0.0 #10511 — Updates the CLI SDK version in app/config/platforms.php (overlaps with CLI version bump here).

Suggested reviewers

loks0n
abnegate
christyjacob4

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	⚠️ Warning	The title claims both “update sdks” and “add response models,” but while the version bumps across SDKs are present, there are no changes adding response models in the diff, so the title is misleading and does not accurately reflect the modifications.	Please revise the pull request title to describe the actual changes, such as the SDK version bumps and the script enhancements for PR updates and cleanup, and remove the incorrect reference to adding response models.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.
Description Check	❓ Inconclusive	The pull request description is empty and provides no insight into the changes, so there is insufficient context to determine how it relates to the modifications in this branch.	Please add a brief description summarizing the key updates in this pull request, such as the SDK version increments and the enhancements to the cleanup and PR tooling scripts, to help reviewers quickly understand the scope of these changes.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch update-sdks-add-response-models

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cf92c25 and a0e8711.

⛔ Files ignored due to path filters (1)

composer.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

app/config/platforms.php (14 hunks)
composer.json (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (15)

GitHub Check: E2E Service Test (Messaging)
GitHub Check: E2E Service Test (FunctionsSchedule)
GitHub Check: E2E Service Test (Webhooks)
GitHub Check: E2E Service Test (Functions)
GitHub Check: E2E Service Test (Databases/Legacy)
GitHub Check: E2E Service Test (Locale)
GitHub Check: E2E Service Test (Proxy)
GitHub Check: E2E Service Test (Realtime)
GitHub Check: E2E Service Test (Avatars)
GitHub Check: E2E Service Test (Dev Keys)
GitHub Check: E2E Service Test (Site Screenshots)
GitHub Check: Unit Test
GitHub Check: E2E General Test
GitHub Check: Setup & Build Appwrite Image
GitHub Check: scan

🔇 Additional comments (1)

app/config/platforms.php (1)

14-422: SDK matrix bumps look consistent.

All the version increments line up with the expected +0.1 cadence across the supported SDKs. Looks good.

composer.json

github-actions · 2025-09-26T10:06:30Z

Security Scan Results for PR

Docker Image Scan Results

Package	Version	Vulnerability	Severity
binutils	2.44-r2	CVE-2025-5244	HIGH
binutils	2.44-r2	CVE-2025-5245	HIGH
libexpat	2.7.1-r0	CVE-2025-59375	HIGH
golang.org/x/crypto	v0.31.0	CVE-2025-22869	HIGH
golang.org/x/oauth2	v0.24.0	CVE-2025-22868	HIGH
stdlib	1.22.10	CVE-2025-47907	HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

github-actions · 2025-09-26T10:17:07Z

✨ Benchmark results

Requests per second: 1,244
Requests with 200 status code: 223,915
P99 latency: 0.1563748

⚡ Benchmark Comparison

Metric	This PR	Latest version
RPS	1,244	1,033
200	223,915	185,919
P99	0.1563748	0.188107176

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f5af1ee and 0314547.

📒 Files selected for processing (1)

src/Appwrite/Platform/Tasks/SDKs.php (3 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (19)

GitHub Check: E2E Service Test (Messaging)
GitHub Check: E2E Service Test (Sites)
GitHub Check: E2E Service Test (Webhooks)
GitHub Check: E2E Service Test (Users)
GitHub Check: E2E Service Test (Health)
GitHub Check: E2E Service Test (Migrations)
GitHub Check: E2E Service Test (Projects)
GitHub Check: E2E Service Test (Locale)
GitHub Check: E2E Service Test (Databases/TablesDB)
GitHub Check: E2E Service Test (GraphQL)
GitHub Check: E2E Service Test (Functions)
GitHub Check: E2E Service Test (Databases/Legacy)
GitHub Check: E2E Service Test (Console)
GitHub Check: E2E Service Test (Account)
GitHub Check: E2E General Test
GitHub Check: E2E Service Test (Dev Keys)
GitHub Check: E2E Service Test (Site Screenshots)
GitHub Check: Unit Test
GitHub Check: scan

src/Appwrite/Platform/Tasks/SDKs.php

chore: update sdks add response models

a0e8711

coderabbitai bot reviewed Sep 26, 2025

View reviewed changes

composer.json Outdated Show resolved Hide resolved

ChiragAgg5k added 3 commits October 1, 2025 09:46

update cli

397c41e

add temp hack

3d43f85

update composer

b4dd1df

ChiragAgg5k force-pushed the update-sdks-add-response-models branch from 6fba44e to b4dd1df Compare October 1, 2025 06:26

ChiragAgg5k added 7 commits October 1, 2025 19:13

Merge branch '1.8.x' into update-sdks-add-response-models

2841c1b

Merge branch '1.8.x' into update-sdks-add-response-models

a3dff6a

update versions

a61385e

fix flutter version

848b932

update more

f5af1ee

Merge branch '1.8.x' into update-sdks-add-response-models

5aa9403

fix: permission issues

0314547

coderabbitai bot reviewed Oct 4, 2025

View reviewed changes

src/Appwrite/Platform/Tasks/SDKs.php Show resolved Hide resolved

abnegate approved these changes Oct 6, 2025

View reviewed changes

abnegate merged commit ee79179 into 1.8.x Oct 6, 2025
41 checks passed

abnegate deleted the update-sdks-add-response-models branch October 6, 2025 03:19

coderabbitai bot mentioned this pull request Oct 8, 2025

add automatic releases #10615

Merged

This was referenced Oct 22, 2025

chore: update apple sdk to 13.3.0 #10679

Merged

update sdks #10694

Merged

This was referenced Oct 29, 2025

update sdks #10729

Merged

Add operator support #10735

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: update sdks add response models#10554

chore: update sdks add response models#10554
abnegate merged 11 commits into1.8.xfrom
update-sdks-add-response-models

ChiragAgg5k commented Sep 26, 2025

Uh oh!

coderabbitai bot commented Sep 26, 2025 •

edited

Loading

Walkthrough

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

github-actions bot commented Sep 26, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Sep 26, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

ChiragAgg5k commented Sep 26, 2025

Uh oh!

coderabbitai bot commented Sep 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Estimated code review effort

Possibly related PRs

Suggested reviewers

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Sep 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Security Scan Results for PR

Docker Image Scan Results

Source Code Scan Results

Uh oh!

github-actions bot commented Sep 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✨ Benchmark results

⚡ Benchmark Comparison

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai bot commented Sep 26, 2025 •

edited

Loading

github-actions bot commented Sep 26, 2025 •

edited

Loading

github-actions bot commented Sep 26, 2025 •

edited

Loading