Skip to content

feat: Web SDK update for version 24.1.0#164

Merged
ChiragAgg5k merged 4 commits into
mainfrom
dev
Mar 31, 2026
Merged

feat: Web SDK update for version 24.1.0#164
ChiragAgg5k merged 4 commits into
mainfrom
dev

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

This PR contains updates to the Web SDK for version 24.1.0.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 31, 2026

Warning

Rate limit exceeded

@ChiragAgg5k has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 15 minutes and 56 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 15 minutes and 56 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: eaa52f62-ded4-4d75-8388-0271f5c563eb

📥 Commits

Reviewing files that changed from the base of the PR and between 10c6a81 and 9ab9c84.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (6)
  • .github/workflows/publish.yml
  • .gitignore
  • CHANGELOG.md
  • README.md
  • package.json
  • src/client.ts
✨ Finishing Touches
🧪 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 bumps the Web SDK to version 24.1.0. The changes are straightforward: the version string is updated in package.json, src/client.ts, and README.md; a package-lock.json is added to the repository (with correct root metadata); a .gitignore is introduced; the publish workflow is updated from npm install to npm ci (appropriate now that the lockfile is committed); and a new public getHeaders() method is added to Client that returns a shallow copy of the request headers.

Key changes:

  • getHeaders() in src/client.ts: Returns { ...this.headers }, a correctly shallow-copied plain object typed against the local Headers alias. The implementation is sound, but the Headers type itself is not exported — see inline comment.
  • npm ci in publish workflow: A correct and intentional improvement now that package-lock.json is tracked; ensures reproducible builds.
  • package-lock.json: Root metadata (name, version, license) matches package.json. One transitive devDependency (glob@8.1.0) carries a deprecation notice in the lockfile, but as a build-time-only dependency it does not affect the published package.

Confidence Score: 5/5

Safe to merge — all remaining findings are P2 suggestions that do not affect runtime correctness or security.

All functional changes are correct: the lockfile matches package.json, npm ci is appropriate now that the lockfile is committed, and getHeaders() returns an accurate shallow copy. The only outstanding issue is the unexported Headers type, which is a developer-experience improvement rather than a defect.

src/client.ts — minor: the Headers type returned by the new getHeaders() method should be exported for caller convenience.

Important Files Changed

Filename Overview
src/client.ts Adds getHeaders() method returning a shallow copy of this.headers; the locally-defined Headers type is not exported, making it harder for callers to explicitly type the return value
.github/workflows/publish.yml Switches install step from npm install to npm ci — correct change now that a package-lock.json is committed
package-lock.json New lockfile committed with correct name, version, and license metadata matching package.json
package.json Version bumped from 24.0.0 to 24.1.0 — straightforward semver minor bump
.gitignore New .gitignore file added with sensible exclusions for node_modules, build outputs, and OS files
CHANGELOG.md 24.1.0 section added; 24.0.0 section expanded with full list of changes from that release
README.md CDN script tag URL updated from appwrite@24.0.0 to appwrite@24.1.0

Reviews (2): Last reviewed commit: "chore: update Web SDK to 24.1.0" | Re-trigger Greptile

Comment thread package-lock.json Outdated
Comment thread src/client.ts
@ChiragAgg5k ChiragAgg5k merged commit 5853287 into main Mar 31, 2026
2 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