Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@ChiragAgg5k
Copy link
Member

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 24, 2025

📝 Walkthrough

Walkthrough

The pull request updates the Console/Command Line SDK entry in the configuration file by changing the version number from 12.0.1 to 13.0.0-rc.2. This is a single-line modification to a version string within a SDK registry. No configuration URLs, enable flags, metadata, or other entries are affected by this change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess relevance to the changeset. Add a brief description explaining the purpose of updating the CLI SDK to version 13.0.0-rc.2 and any relevant context or testing performed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the CLI SDK version to an RC (release candidate) version in the configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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-cli-sdk

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

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
libpng 1.6.51-r0 CVE-2025-66293 HIGH
libpng-dev 1.6.51-r0 CVE-2025-66293 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

Copy link
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: 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 0e75793 and c3bc3dc.

⛔ Files ignored due to path filters (2)
  • composer.lock is excluded by !**/*.lock
  • docs/sdks/cli/CHANGELOG.md is excluded by !docs/sdks/**
📒 Files selected for processing (1)
  • app/config/sdks.php
⏰ 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

'key' => 'cli',
'name' => 'Command Line',
'version' => '12.0.1',
'version' => '13.0.0-rc.2',
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

# First, verify the file and line content
cat -n app/config/sdks.php | sed -n '225,235p'

Repository: appwrite/appwrite

Length of output: 561


🏁 Script executed:

# Check npm registry for version 13.0.0-rc.2 availability
curl -s https://registry.npmjs.org/appwrite-cli/13.0.0-rc.2 | head -20

Repository: appwrite/appwrite

Length of output: 93


🏁 Script executed:

# Search for hardcoded version references in codebase
rg "12\.0\.1|13\.0\.0" --type php --type js --type ts -n

Repository: appwrite/appwrite

Length of output: 128


Verify npm version exists before merging—version 13.0.0-rc.2 not found in npm registry.

The version 13.0.0-rc.2 referenced on line 229 does not exist in the npm registry. Either publish this version to npm or update the config to an existing version before this change can be merged.

🤖 Prompt for AI Agents
In app/config/sdks.php around line 229, the SDK version is set to '13.0.0-rc.2'
which does not exist in the npm registry; update this entry to a valid,
published npm version (e.g., an existing stable or RC version) or coordinate to
publish '13.0.0-rc.2' before merging. Replace the value with the confirmed
version string and run composer/npm install or lockfile update as appropriate to
ensure dependency resolution succeeds.

@github-actions
Copy link

✨ Benchmark results

  • Requests per second: 1,107
  • Requests with 200 status code: 199,390
  • P99 latency: 0.173570478

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,107 1,162
200 199,390 209,134
P99 0.173570478 0.178459437

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.