Repository navigation
Conversation
…lerk-ios clerk-ios's skill has a reference for what the CLI prints: the JSON shapes, the exit codes, and a table of error codes. The Expo skill had one sentence on exit codes. `references/output.md` now has the same content, with the Expo meaning of each code: two kinds of lanes, the steps of the test app's build, and the run on the other platform that holds the application. `SKILL.md` now says that `screen` right after `up` does not show what a spec sees, and how to read a missing node. The feature map gets the baseline preconditions that clerk-ios lists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mikepitre
added this pull request to stack #10100
October 14, 2026 03:45
🦋 Changeset detectedLatest commit: 442a0fa The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Comment |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-biometrics
@clerk/expo-google-signin
@clerk/expo-native-components
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/mosaic
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
down --platform keeps the application while the other platform holds a lease. Only the tunnel and Clerk egress checks print blocked:, and a 403 on push means this machine's credential may not push. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The allowed-domains fix applies to the two egress checks. The github-rest check's own fix is to make api.github.com reachable. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The check reports any failed request as api.github.com did not answer, timeouts included, so an allowed domain is the first thing to check, not the only one. Same change as clerk/clerk-ios#696. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 of 9 tasks
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR sits on #10227 and must merge after it. clerk/clerk-android#1093 is the same change for clerk-android.
The clerk-ios verification skill has text that the Expo skill lacked. This PR adds it, written for the Expo package, and checked against the code in
integration/expo-native/. It changes three files of.claude/skills/verify-clerk-expo/and adds one.What came over from clerk-ios:
references/output.mdhas what each verb prints, the exit code table, and the error code table.SKILL.mdhad one sentence on exit codes, and now points at the file.SKILL.mdsays whatscreenshows right afterup, and how a missing node fails.features/README.mdhas the Baseline preconditions section.What is different here, and why:
POOL_FULLcovers four iOS lanes and two Android lanes.DEVICE_BUSYalso covers a run on the other platform that holds the worktree's application.BUILD_FAILEDnames the step of the test app's build that failed, such asexpo prebuildorxcodebuild, with its last 15 lines.SKILL.mdalready says.down --platformthat keeps the application for the other lane lists no application in its JSON.screenright afterupshows the host's error screen. Herescreenopens the test app without the verify inputs, and a local dev client and a Release build show different things then. The text says only that the screen is not one a spec sees, which holds for both.Two facts differ from the clerk-ios text on every platform.
NOT_TEST_IDENTITYis in the list of error codes, but no command raises it, and the table says so.doctor --jsonprints its report withok: falseand noerrorwhen a check fails.references/remote.mdhad two sentences from clerk-ios that the code does not support. Only the two egress checks ofdoctorprintblocked:, and a 403 on push means that this machine's credential may not push. The file now says that, and clerk/clerk-ios#696 makes the same fix there.Not brought over: the clerk-ios line that the screen tree leaves out what a sheet covers. Whether that holds for the modal AuthView of the test app needs a device to see. The cloud environment text and the driving conventions were already here.
The changeset is empty, as in the other PRs of the stack.
npm testinintegration/expo-native/passes 797 of 797. An earlier run timed out once intest/session-agent.test.tswhile it waited for a tunnel file, and passed when run again. This PR changes no code, so that test is not about this change.Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
🤖 Generated with Claude Code