Skip to content

feat: React Native SDK update for version 0.27.0#96

Merged
ChiragAgg5k merged 4 commits intomainfrom
dev
Mar 31, 2026
Merged

feat: React Native SDK update for version 0.27.0#96
ChiragAgg5k merged 4 commits intomainfrom
dev

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

@ChiragAgg5k ChiragAgg5k commented Mar 31, 2026

This PR contains updates to the React Native SDK for version 0.27.0.

Summary by CodeRabbit

  • New Features
    • Added getHeaders() method to the Client class to retrieve current request headers.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

Important

Review skipped

Review was skipped due to path filters

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

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.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a1466b6a-22ea-4828-ad18-038f75743294

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

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This pull request represents a minor version release (0.26.0 → 0.27.0) that includes the addition of a new public getHeaders() method on the Client class for exposing current request headers. Supporting changes include updating the SDK version header value, bumping version metadata in package.json, adding a .gitignore file to exclude build artifacts and dependencies, updating the CI workflow to use npm ci for deterministic dependency installation, and documenting the changes in CHANGELOG.md.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 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 title accurately summarizes the main change: a version bump to 0.27.0 with associated updates (new getHeaders() method, workflow changes, and dependency lockfile tracking).
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 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 React Native SDK to version 0.27.0, introducing a getHeaders() method on the Client class and improving the repository's dependency management story by committing package-lock.json and adding a .gitignore.

Key changes:

  • src/client.ts — Adds getHeaders(): Headers which returns a shallow spread copy ({ ...this.headers }) of all current request headers. Because all header values are strings (primitives), the shallow copy is safe and callers cannot mutate internal state. The JSDoc correctly flags that the result may include sensitive auth tokens (JWT, session, dev key, impersonation headers).
  • package-lock.json — Committed for the first time (lockfileVersion 3), enabling deterministic installs for contributors and CI alike.
  • .github/workflows/publish.ymlnpm installnpm ci, the correct companion change now that the lockfile is tracked; this ensures the published package is always built from the exact pinned dependency tree.
  • .gitignore — New file; correctly excludes node_modules/, compiled dist/ artifacts (while preserving the dist/cjs/package.json and dist/esm/package.json stubs needed by some bundlers), types/, and .DS_Store.
  • CHANGELOG.md / package.json — Version and changelog are in sync at 0.27.0.

Confidence Score: 5/5

Safe to merge — all changes are additive, well-scoped, and mutually consistent.

No P0 or P1 issues found. The only non-trivial new surface is getHeaders(), which is correctly implemented (shallow copy, primitive values only) and appropriately documented. The CI improvement (npm ci) is a strictly better practice now that the lockfile is committed, and the .gitignore follows a standard pattern for NPM packages. Version numbers are consistent across package.json, client.ts, and CHANGELOG.md.

No files require special attention.

Important Files Changed

Filename Overview
src/client.ts Version header bumped to 0.27.0; new getHeaders() method added that returns a shallow copy of all current request headers, including any set auth tokens
.github/workflows/publish.yml npm install replaced with npm ci — a good improvement for deterministic builds, now that package-lock.json is committed
.gitignore New .gitignore ignores node_modules/, compiled dist/ artifacts (keeping only dist/cjs/package.json and dist/esm/package.json), types/, and .DS_Store
package.json Version bumped from 0.26.0 to 0.27.0, consistent with header and lockfile
package-lock.json New lockfile (lockfileVersion 3) committed for the first time; enables npm ci and reproducible builds in CI
CHANGELOG.md 0.27.0 section added, accurately documenting the new getHeaders() method and lockfile addition

Reviews (1): Last reviewed commit: "Merge main into dev, resolve package-loc..." | Re-trigger Greptile

@ChiragAgg5k ChiragAgg5k merged commit 0ad5bf7 into main Mar 31, 2026
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