[mikepitre]

What & why

• The CLI read the team Platform API key from CLERK_PLATFORM_API_KEY. That's also the name clerk/javascript's web integration suite uses for a different key, so a shell set up for those tests would send the mobile CLI's throwaway apps to the wrong workspace.
• The three ways to supply the key now match the MOBILE_VERIFICATION_PLATFORM_API_KEY secret that consumer CI already reads: MOBILE_VERIFICATION_PLATFORM_API_KEY, MOBILE_VERIFICATION_PLATFORM_API_KEY_FILE, and MOBILE_VERIFICATION_PLATFORM_API_KEY_REFERENCE (was VERIFY_PLATFORM_KEY_REFERENCE). No fallback to the old names.

Testing

• npm run build, then node --test test/*.test.ts: 736 pass. tsc -p . is clean.

Risk

• Anyone who sets the old variables locally gets KEYS_MISSING until they rename them. The error's fix line names the new variables.
• Cloud sessions aren't affected: their key is added to requests by host and path, not read from a variable.
• Consumers pick this up only when they bump their pin, in the clerk-ios, clerk-android, and clerk/javascript PRs that follow.

How to revert

• A plain revert, plus reverting the consumer pin bumps.

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The Platform API credential variables now use the MOBILE_VERIFICATION_PLATFORM_* names. Runtime lookup, launch configuration, documentation, and test fixtures use the updated names. Credential precedence, file checks, and tested behavior remain unchanged.

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

Suggested reviewers: wobsoriano

Merge Risk: Minimal · up to 0889a

The credential rename is ready to merge with the dependent workflow update; legacy variable users must still migrate their environment names.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly identifies the main change: renaming the Platform API key variables for mobile verification.
Description check		The description explains the variable renames, the reason for the change, compatibility impact, testing, risk, and revert plan. It directly relates to the changeset.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 6 functions across 5 files.

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Linked repositories: Some configured repositories could not be included in this review. Cross-repository analysis may be incomplete. Please check linked-repository access and settings.

CodeRabbit Verify

• Run CodeRabbit Verify to test the changes in this pull request.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: ASSERTIVE
• Plan: Team
• Run ID: 4a0559f4-a00a-4727-a9e9-c1c8d3143c08

Commits

Reviewing files that changed from the base of the PR and between b70b190 and 0889a2d.

Files selected for processing: 2

• src/core/launch.mjs
• test/instances.test.ts

Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

• clerk/clerk_go (manual)
• clerk/dashboard (manual)
• clerk/accounts (manual)
• clerk/backoffice (manual)
• clerk/clerk (manual)
• clerk/clerk-docs (manual)
• clerk/cloudflare-workers (manual)
• clerk/javascript (auto-detected) → reviewed against open PR #10616 mobile-verification-key-name instead of the default branch

Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 6 reviews per hour.

[coderabbitai]

• Fix comments: Fix CodeRabbit comments on this PR