Skip to content

feat: React Native SDK update for version 0.34.0#111

Merged
ChiragAgg5k merged 2 commits into
mainfrom
dev
Jul 13, 2026
Merged

feat: React Native SDK update for version 0.34.0#111
ChiragAgg5k merged 2 commits into
mainfrom
dev

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Jul 13, 2026

Copy link
Copy Markdown
Member

This PR contains updates to the SDK for version 0.34.0.

What's Changed

  • Added: Client.setBearer() method for OAuth access token authentication
  • Added: Query.vectorDot, Query.vectorCosine, and Query.vectorEuclidean vector similarity queries
  • Added: Appwrite value to the OAuthProvider enum
  • Added: optional geolocation fields (city, timeZone, latitude, isp, etc.) to Models.Locale

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown

Greptile Summary

This PR updates the React Native SDK to version 0.34.0, adding new authentication, query, and model features generated from the latest Appwrite API spec. Dependency tooling is also updated and npm overrides are added to pin transitive dependencies to secure versions.

  • Client.setBearer(): New method sets the Authorization: Bearer <token> header for OAuth access token authentication; the Bearer scheme prefix is correctly included.
  • Vector similarity queries: Adds Query.vectorDot, Query.vectorCosine, and Query.vectorEuclidean, following the same array-wrapping serialization pattern as existing geometric queries.
  • OAuthProvider.Appwrite enum value added; Models.Locale extended with optional geolocation fields (city, timeZone, latitude, isp, etc.), all backward-compatible.

Confidence Score: 5/5

This PR is safe to merge — all changes are additive (new methods, new enum value, new optional model fields) with no modifications to existing behavior.

The new setBearer method correctly includes the Bearer scheme prefix. The three vector query methods follow the established array-wrapping serialization pattern used consistently by all other geometric queries in the file. The new Locale fields are all optional, keeping the type change backward-compatible. Dependency overrides are development-only and do not affect downstream consumers of the SDK.

No files require special attention.

Important Files Changed

Filename Overview
src/client.ts Adds setBearer() method with correct Bearer ${value} Authorization header; updates SDK version to 0.34.0.
src/query.ts Adds three vector similarity query methods (vectorDot, vectorCosine, vectorEuclidean) following the same wrapping pattern as other geometric queries.
src/enums/o-auth-provider.ts Adds Appwrite = 'appwrite' as a new OAuthProvider enum value.
src/models.ts Extends Models.Locale with optional geolocation fields (city, timeZone, latitude, longitude, isp, etc.); all fields are optional, maintaining backward compatibility.
package.json Bumps version to 0.34.0, upgrades rollup and TypeScript, and adds npm overrides to pin transitive deps (glob, rimraf, uuid, etc.) to newer secure versions.
.github/workflows/publish.yml Adds npm audit --audit-level=high --omit=dev step before build; prior review thread covered the trade-offs of this approach.
CHANGELOG.md Adds 0.34.0 section documenting new features; removes the URL-encode path parameters entry from 0.33.0 (prior comment noted the ambiguity).
src/services/account.ts No functional changes visible in the diff; file updated as part of the SDK regeneration cycle.

Reviews (2): Last reviewed commit: "chore: update React Native SDK to 0.34.0" | Re-trigger Greptile

Comment thread src/client.ts
- name: Install dependencies, audit, and build
run: |
npm ci
npm audit --audit-level=high --omit=dev

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Failing audit blocks the entire publish workflow

Running npm audit --audit-level=high as a hard prerequisite of publishing means any newly-disclosed high-severity CVE in a transitive production dependency will block all releases until patched, even for unrelated emergency fixes. Consider running the audit as an informational step or in a separate job with continue-on-error: true.

Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/publish.yml
Line: 40

Comment:
**Failing audit blocks the entire publish workflow**

Running `npm audit --audit-level=high` as a hard prerequisite of publishing means any newly-disclosed high-severity CVE in a transitive production dependency will block all releases until patched, even for unrelated emergency fixes. Consider running the audit as an informational step or in a separate job with `continue-on-error: true`.

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Fix in Claude Code Fix in Codex

@ChiragAgg5k
ChiragAgg5k merged commit 52ada46 into main Jul 13, 2026
1 check 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