Skip to content

update sdks#10694

Merged
loks0n merged 3 commits into1.8.xfrom
update-sdks
Oct 27, 2025
Merged

update sdks#10694
loks0n merged 3 commits into1.8.xfrom
update-sdks

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

@ChiragAgg5k ChiragAgg5k commented Oct 24, 2025

No description provided.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Oct 24, 2025 •
edited
Loading

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

This PR updates observable SDK version strings in app/config/platforms.php. The Web SDK version is bumped from 21.2.1 to 21.3.0, and the CLI SDK version is bumped from 10.2.2 to 10.2.3. These are configuration-only changes with no structural or behavioral modifications to the codebase.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

This change consists of two homogeneous version string updates in a single configuration file. The edits are straightforward value replacements with minimal scope and no complex logic to validate.

Possibly related PRs

Suggested reviewers

  • loks0n
  • abnegate

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description Check ⚠️ Warning No pull request description was provided by the author. While the check is lenient and only requires that a description be related to the changeset, a completely absent description fails to meet the basic pass criterion of being related to the changeset in some way. The missing context about why these specific SDK versions are being updated leaves reviewers without any explanation of the change's motivation or impact. Please add a pull request description explaining the purpose of these SDK version updates, such as what bug fixes, features, or improvements are included in the new versions, or any other relevant context for reviewers.
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.
✅ Passed checks (1 passed)
Check name Status Explanation
Title Check ✅ Passed The PR title "update sdks" is directly and accurately related to the main change in the changeset, which updates Observable SDK version strings for both the Web SDK (21.2.1 to 21.3.0) and CLI SDK (10.2.2 to 10.2.3) in the app/config/platforms.php file. While the title is concise and somewhat generic, it clearly communicates the primary purpose of the changeset without being misleading or off-topic.

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.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 24, 2025 •
edited
Loading

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
libxml2 2.13.8-r0 CVE-2025-49794 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49796 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49795 HIGH
libxml2 2.13.8-r0 CVE-2025-6021 HIGH
pcre2 10.43-r1 CVE-2025-58050 CRITICAL
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!

coderabbitai[bot]
coderabbitai bot reviewed Oct 24, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 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 046d119 and d90e262.

⛔ Files ignored due to path filters (3)
  • composer.lock is excluded by !**/*.lock
  • docs/sdks/cli/CHANGELOG.md is excluded by !docs/sdks/**
  • docs/sdks/web/CHANGELOG.md is excluded by !docs/sdks/**
📒 Files selected for processing (1)
  • app/config/platforms.php (2 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). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 24, 2025 •
edited
Loading

✨ Benchmark results

  • Requests per second: 975
  • Requests with 200 status code: 175,523
  • P99 latency: 0.195824207

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 975 988
200 175,523 177,796
P99 0.195824207 0.20892855

@loks0n loks0n merged commit e4f63ed into 1.8.x Oct 27, 2025
41 checks passed
This was referenced Nov 3, 2025
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@loks0n loks0n loks0n approved these changes

+1 more reviewer

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ChiragAgg5k @loks0n