Skip to content

chore(deps): update Clerk SDKs and reconcile native patch - #20359

Merged
juliusmarminge merged 1 commit into
mainfrom
chore/clerk-sdk58-refresh
Oct 16, 2026
Merged

juliusmarminge merged 1 commit into
mainfrom
chore/clerk-sdk58-refresh

Conversation

@juliusmarminge

@juliusmarminge juliusmarminge commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Problem

Change

Scope and approval

Verification


Devin Review

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Oct 16, 2026
@juliusmarminge juliusmarminge added the macroscope-review Opt PRs made by unvouched contributors in for Macroscope review. Vouched contributors auto-reviews label Oct 16, 2026
@github-actions github-actions Bot added size:L 100-499 changed lines (additions + deletions). 📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. labels Oct 16, 2026
@macroscopeapp

macroscopeapp Bot commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — This is more than a lockfile refresh: it replaces a patched Clerk native component and changes iOS authentication/profile navigation behavior in the shipped mobile app. Because authentication-sensitive native runtime behavior is involved, the change warrants human review.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 5.0 KiB 5.0 KiB +7 B (+0.1%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB +7 B (+0.6%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 20.9 KiB 20.9 KiB 0 B (0.0%) 29.3 KiB ✅
Codex Live turn messages 2 2 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.2 KiB 21.2 KiB 0 B (0.0%) 29.3 KiB ✅
Claude Live turn messages 2 2 0 (0.0%) 8 ✅

Baseline: a54a44c · PR result: 876f692 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@coderabbitai

coderabbitai Bot commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Path: .coderabbit.config.ts
  • Review profile: CHILL
  • Plan: Team
  • Run ID: 42424dd6-22de-41ae-813b-8d1bc93234f2
📥 Commits

Reviewing files that changed from the base of the PR and between a54a44c and 876f692.

⛔ Files ignored due to path filters: 1
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 4
  • apps/mobile/package.json
  • patches/@clerk__expo-native-components@0.1.1.patch
  • patches/@clerk__expo-native-components@0.2.0.patch
  • pnpm-workspace.yaml
💤 Files with no reviewable changes: 1

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📝 Walkthrough

The mobile app upgrades @clerk/expo-native-components to 0.2.0 and updates workspace package versions and patch configuration. The replacement patch adds guarded navigation-route popping to auth and user-profile back actions, with onHostBack as a fallback.

Changes: Clerk Expo Native Components

Layer / File(s) Summary
Upgrade dependency and patch selection
apps/mobile/package.json, pnpm-workspace.yaml, patches/@clerk__expo-native-components@0.1.1.patch
The mobile dependency and workspace patch configuration now select @clerk/expo-native-components 0.2.0. The workspace catalog also updates the Clerk backend, clerk-js, Electron, and Expo versions. The 0.1.1 patch file is removed.
Add guarded navigation-route popping
patches/@clerk__expo-native-components@0.2.0.patch
Auth and user-profile back actions attempt to pop an eligible enclosing navigation route before calling onHostBack. The host tracks pop progress and uses a shared responder-chain controller lookup.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~12 minutes

Change: Other

Suggested reviewers: t3dotgg

Merge Risk: ⚪ Minimal · up to 876f6

The changed back action reaches the expected settings screen in the ordinary flow. No actionable merge risk was established.

Pre-merge checks: Passed 3 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description contains only the template headings and comments. It does not explain the problem, the dependency and patch changes, scope approval, verification results, or any UI evidence required f… Complete each required section with the problem, the specific Clerk SDK and native patch changes, issue or maintainer approval details, focused verification results, and before/after UI evidence if applicable. State any checks that were not…
Passed Passed checks: 3 passed
Check name Status Explanation
Title check Passed The title clearly identifies the dependency updates and native patch reconciliation, which match the main changes.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description contains only the template headings and comments. It does not explain the problem, the dependency and patch changes, scope approval, verification results, or any UI evidence required for the native navigation changes.

Resolution

Complete each required section with the problem, the specific Clerk SDK and native patch changes, issue or maintainer approval details, focused verification results, and before/after UI evidence if applicable. State any checks that were not run.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@juliusmarminge
juliusmarminge merged commit a3e8cf6 into main Oct 16, 2026
31 of 32 checks passed
@juliusmarminge
juliusmarminge deleted the chore/clerk-sdk58-refresh branch October 16, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

macroscope-review Opt PRs made by unvouched contributors in for Macroscope review. Vouched contributors auto-reviews 📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant