Skip to content

feat: Command Line SDK update for version 17.2.1#286

Merged
ChiragAgg5k merged 3 commits intomasterfrom
dev
Mar 31, 2026
Merged

feat: Command Line SDK update for version 17.2.1#286
ChiragAgg5k merged 3 commits intomasterfrom
dev

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

@ChiragAgg5k ChiragAgg5k commented Mar 31, 2026

This PR contains updates to the Command Line SDK for version 17.2.1.

Summary by CodeRabbit

  • New Features

    • Added a method to retrieve the current request headers from the client.
  • Chores

    • Bumped version to 17.2.1.
    • Updated CI/CD workflows to enforce lockfile consistency during dependency installation.
    • Updated installer scripts and package manifest to reflect the new version.
    • Moved lockfile into version control for improved reproducibility.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e88d81b7-7184-4088-a9d3-23b2305ed2ec

📥 Commits

Reviewing files that changed from the base of the PR and between fabc3b2 and 0641da8.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • lib/client.ts
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/ci.yml

Walkthrough

Bumps release from 17.2.0 to 17.2.1 across package metadata, docs, installers, and distribution manifests. CI and publish workflows change Bun install to bun install --frozen-lockfile. .gitignore was modified to track bun.lock. A new public method getHeaders(): Headers was added to Client in lib/client.ts. No other logic or exported API signatures were changed.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately describes the main change: a version update of the Command Line SDK to version 17.2.1, which aligns with all file modifications in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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.

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Mar 31, 2026

Greptile Summary

This PR is a patch version bump from 17.2.0 to 17.2.1 for the Appwrite CLI SDK. The primary substantive change is committing the bun.lock lockfile to source control (by removing it from .gitignore) and enforcing --frozen-lockfile in both CI and publish workflows for reproducible, deterministic builds.

Key changes:

  • Removes bun.lock from .gitignore and commits it; the workspace name (\"appwrite-cli\") correctly matches package.json
  • Adds --frozen-lockfile to bun install in both .github/workflows/ci.yml and .github/workflows/publish.yml
  • Bumps version to 17.2.1 consistently across package.json, lib/constants.ts, install.sh, install.ps1, scoop/appwrite.config.json, and README.md
  • Adds a JSDoc comment to getHeaders() in lib/client.ts for documentation clarity

Confidence Score: 5/5

This PR is safe to merge — all version references are updated consistently and the lockfile change is correct.

All version bumps are consistent across every relevant file. The bun.lock workspace name matches package.json, and enforcing --frozen-lockfile in CI/publish is the appropriate companion change. No functional logic was modified beyond the JSDoc addition. No issues found.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/ci.yml Adds --frozen-lockfile to bun install to enforce reproducible builds now that bun.lock is tracked in source control.
.github/workflows/publish.yml Mirrors the CI change — bun install --frozen-lockfile ensures the publish job uses exact locked dependency versions.
.gitignore Removes bun.lock from .gitignore so the lockfile is now tracked; also fixes a missing trailing newline.
bun.lock New lockfile committed to source control; workspace name is "appwrite-cli" matching package.json, and zod resolves correctly to 4.3.6.
lib/client.ts Adds a JSDoc block to getHeaders() describing the method's purpose and return type — no functional change.
lib/constants.ts Bumps SDK_VERSION from 17.2.0 to 17.2.1.
package.json Bumps package version from 17.2.0 to 17.2.1; no dependency changes.
install.sh Updates GITHUB_LATEST_VERSION to 17.2.1 so the install script fetches the correct release binary.
install.ps1 Updates both x64 and arm64 Windows download URLs to the 17.2.1 release.
scoop/appwrite.config.json Updates Scoop manifest version and both architecture download URLs from 17.2.0 to 17.2.1.

Reviews (3): Last reviewed commit: "chore: update Command Line SDK to 17.2.1" | Re-trigger Greptile

@ChiragAgg5k ChiragAgg5k merged commit 9253538 into master Mar 31, 2026
2 of 3 checks passed
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