chore: update sdks 1.7.x #10202
chore: update sdks 1.7.x #10202
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (12)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the 📝 Walkthrough""" WalkthroughThis change updates the version numbers of multiple SDKs in the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 minutes Possibly related PRs
Suggested reviewers
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Security Scan Results for PRDocker Image Scan Results
Source Code Scan Results🎉 No vulnerabilities found! |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🔭 Outside diff range comments (1)
app/config/platforms.php (1)
3-475: Fix PSR-12 array-indentation – linter currently failsThe CI “Linter” job reports
array_indentation style issue detected.
Runvendor/bin/pint(orcomposer pint) locally; it will auto-format the file and keep the pipeline green.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (8)
composer.lockis excluded by!**/*.lockdocs/sdks/android/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/apple/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/dart/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/flutter/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/nodejs/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/react-native/CHANGELOG.mdis excluded by!docs/sdks/**docs/sdks/web/CHANGELOG.mdis excluded by!docs/sdks/**
📒 Files selected for processing (1)
app/config/platforms.php(8 hunks)
🧰 Additional context used
🪛 GitHub Actions: Linter
app/config/platforms.php
[error] 1-1: PSR 12 linting failed: array_indentation style issue detected. Run 'vendor/bin/pint' to fix code style issues.
⏰ 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). (3)
- GitHub Check: Setup & Build Appwrite Image
- GitHub Check: Setup & Build Appwrite Image
- GitHub Check: scan
| 'version' => '18.2.0', | ||
| 'url' => 'https://github.com/appwrite/sdk-for-web', |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Verify that every bumped version actually exists on the respective package registry / GitHub release before merging
Configuration is sound, but if any of these tags haven’t been published yet (or were yanked), package-managers and the website will display a broken version badge.
Run a quick check:
Also applies to: 60-63, 80-81, 115-116, 137-138, 202-203, 254-255, 380-381
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
declare -A repos=(
["web"]="appwrite/[email protected]"
["flutter"]="appwrite/[email protected]"
["apple"]="appwrite/[email protected]"
["android"]="appwrite/[email protected]"
["react-native"]="appwrite/[email protected]"
["console"]="appwrite/[email protected]"
["node"]="appwrite/[email protected]"
["dart"]="appwrite/[email protected]"
)
for key in "${!repos[@]}"; do
ref=${repos[$key]}
echo -n "$ref -> "
gh release view "$ref" --repo "${ref%%@*}" >/dev/null 2>&1 && echo "✔ exists" || echo "✖ missing"
doneLength of output: 2437
I noticed the previous GH CLI invocation incorrectly passed the full “owner/repo@tag” as the release identifier. Let’s retry with the correct usage, passing just the tag to gh release view:
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
# Parallel arrays of repos and their expected versions
repos=(
appwrite/sdk-for-web
appwrite/sdk-for-flutter
appwrite/sdk-for-apple
appwrite/sdk-for-android
appwrite/sdk-for-react-native
appwrite/sdk-for-console
appwrite/sdk-for-node
appwrite/sdk-for-dart
)
versions=(
18.2.0
17.1.0
10.2.0
8.2.0
0.10.2
1.9.0
17.2.0
16.2.0
)
for i in "${!repos[@]}"; do
repo=${repos[$i]}
version=${versions[$i]}
echo -n "$repo@$version -> "
gh release view "$version" --repo "$repo" >/dev/null 2>&1 && echo "✔ exists" || echo "✖ missing"
doneLength of output: 2199
Action Required: Publish Missing GitHub Releases for Bumped SDK Versions
A number of the newly bumped tags are not yet available on GitHub, which will break version badges and package installs. Please publish the following releases (or adjust the versions) before merging:
- appwrite/sdk-for-flutter @ 17.1.0 (lines 60–63) – missing
- appwrite/sdk-for-apple @ 10.2.0 (lines 80–81) – missing
- appwrite/sdk-for-react-native @ 0.10.2 (lines 137–138) – missing
- appwrite/sdk-for-node @ 17.2.0 (lines 254–255) – missing
- appwrite/sdk-for-dart @ 16.2.0 (lines 380–381) – missing
Confirmed existing releases (no action needed):
- appwrite/sdk-for-web @ 18.2.0 (lines 14–15) – ✔ exists
- appwrite/sdk-for-android @ 8.2.0 (lines 115–116) – ✔ exists
- appwrite/sdk-for-console @ 1.9.0 (lines 202–203) – ✔ exists
🤖 Prompt for AI Agents
In app/config/platforms.php around lines 14 to 15 and other specified lines,
several SDK versions have been bumped but the corresponding GitHub releases are
missing. To fix this, publish the missing GitHub releases for the following SDKs
and versions before merging: appwrite/sdk-for-flutter at 17.1.0 (lines 60–63),
appwrite/sdk-for-apple at 10.2.0 (lines 80–81), appwrite/sdk-for-react-native at
0.10.2 (lines 137–138), appwrite/sdk-for-node at 17.2.0 (lines 254–255), and
appwrite/sdk-for-dart at 16.2.0 (lines 380–381). Alternatively, adjust the
version numbers in the file to match existing published releases to avoid
breaking version badges and package installs.
✨ Benchmark results
⚡ Benchmark Comparison
|
chore: add docs for nestedType, encode, from and toMap
What does this PR do?
updates sdks and changelogs
Test Plan
Related PRs and Issues
Checklist