Skip to content

Conversation

@tmilewski
Copy link
Member

@tmilewski tmilewski commented Dec 10, 2025

Description

When testing against local APIs the ports were conflicting with one another. This change simply uses a different port.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • Chores
    • Updated development server configuration to use port 4011 across Playwright, build tools, and development scripts. This affects the local development environment setup and testing infrastructure.

✏️ Tip: You can customize this high-level summary in your review settings.

@tmilewski tmilewski self-assigned this Dec 10, 2025
@changeset-bot
Copy link

changeset-bot bot commented Dec 10, 2025

🦋 Changeset detected

Latest commit: 331050b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When 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

@vercel
Copy link

vercel bot commented Dec 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clerk-js-sandbox Ready Ready Preview Comment Dec 10, 2025 8:29pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 10, 2025

Walkthrough

This changeset updates the default development server port from 4001 to 4011 across multiple configuration and source files in the clerk-js and ui packages, including Playwright configuration, rspack build configs, package.json scripts, and testing constants.

Changes

Cohort / File(s) Change Summary
Changeset
.changeset/loose-trams-look.md
Empty changeset file placeholder added
Clerk-JS Configuration
packages/clerk-js/playwright.config.ts, packages/clerk-js/rspack.config.js
Updated development server port from 4001 to 4011 in Playwright baseURL/webServer configuration and sandbox UI script injection URL
UI Package Configuration
packages/ui/package.json, packages/ui/rspack.config.js
Updated dev origin scripts and devConfig port derivation from 4001 to 4011
UI Testing Constants
packages/ui/src/internal/index.ts
Updated localUiForTesting constant URL from localhost:4001 to localhost:4011

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

The changes are homogeneous and repetitive—consistent port number updates applied across configuration files with no logic alterations, error handling changes, or functional modifications.

Poem

🐰 Port four-oh-oh-one hops away,
Four-oh-one-one bounds into play!
Dev servers dance on digits new,
Configuration tweaks through and through—
A rabbit's port protocol debut! 🎉

Pre-merge checks and finishing touches

✅ 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 'fix(js,ui): Conflicting ports' directly relates to the main change—updating ports from 4001 to 4011 across multiple packages to resolve port conflicts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tom/fix-sandbox-ports

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

@tmilewski tmilewski changed the title fix: Conflicting ports fix(js,ui): Conflicting ports Dec 10, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 10, 2025

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@7433

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7433

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7433

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7433

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7433

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7433

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@7433

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7433

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7433

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7433

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7433

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7433

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7433

@clerk/react

npm i https://pkg.pr.new/@clerk/react@7433

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7433

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7433

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7433

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7433

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7433

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7433

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7433

commit: 331050b

coderabbitai[bot]

This comment was marked as resolved.

@tmilewski tmilewski merged commit fce2ac5 into main Dec 10, 2025
43 checks passed
@tmilewski tmilewski deleted the tom/fix-sandbox-ports branch December 10, 2025 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants