Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing intermediate states during navigation" #6928

wobsoriano · 2025-10-05T19:53:59Z

Reverts #6802

The original intent of the PR is to fix issues with setTransitiveState() in non-React apps, but looks like it introduced another issue (#6904). We have plans to kill setTransitiveState() so I guess it's better to wait for that than to patch it like this.

Summary by CodeRabbit

Refactor
- Simplified client-side navigation by removing an unnecessary asynchronous wrapper, making routing calls more direct.
Performance
- Slight improvement to navigation responsiveness by eliminating extra scheduling overhead.
Chores
- Added a changeset noting this minor routing update.

…diate st…" This reverts commit e2dd40a.

changeset-bot · 2025-10-05T19:54:03Z

🦋 Changeset detected

Latest commit: 4114153

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

This PR includes changesets to release 1 package

Name	Type
@clerk/nuxt	Minor

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 · 2025-10-05T19:54:04Z

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

Project	Deployment	Preview	Comments	Updated (UTC)
clerk-js-sandbox	Ready	Preview	Comment	Oct 5, 2025 7:57pm

coderabbitai · 2025-10-05T19:54:10Z

Walkthrough

Removed nextTick wrapping from Nuxt runtime navigation helpers. routerPush now directly calls navigateTo(to) and routerReplace calls navigateTo(to, { replace: true }); unused nextTick import and a related ts-expect-error comment were removed.

Changes

Cohort / File(s)	Summary of Changes
Nuxt runtime plugin navigation `packages/nuxt/src/runtime/plugin.ts`	Removed `nextTick` wrapper around `routerPush`/`routerReplace`; now directly call `navigateTo` and `navigateTo(..., { replace: true })`. Removed unused `nextTick` import and a `ts-expect-error` comment.
Changeset `.changeset/ten-poets-agree.md`	Added changeset noting removal of `nextTick` calls in Clerk/Nuxt routing helpers.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant App
  participant Router

  rect rgba(220, 248, 207, 0.6)
  note over App: New flow (direct)
  User->>App: routerPush(to) / routerReplace(to)
  App->>Router: navigateTo(to) / navigateTo(to, { replace: true })
  Router-->>User: Navigation handled
  end

  rect rgba(255, 243, 207, 0.6)
  note over App: Prior flow (queued)
  User->>App: routerPush(to) / routerReplace(to)
  App->>App: nextTick(() => ...)
  App->>Router: navigateTo(...)
  Router-->>User: Navigation handled
  end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hop the path without delay,
No ticking pause along the way.
A gentle push, a swapping stride,
Direct to carrots on the ride.
Small change, big hop—let's glide! 🥕🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title clearly states that this pull request reverts the previous Nuxt routing fix and directly corresponds to the changes being rolled back, specifying the original commit’s description without extraneous detail.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ 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 revert-6802-rob/nuxt-fix-transition

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

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 12e36c4 and 4114153.

📒 Files selected for processing (1)

.changeset/ten-poets-agree.md (1 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

.changeset/**

📄 CodeRabbit inference engine (.cursor/rules/monorepo.mdc)

Automated releases must use Changesets.

Files:

.changeset/ten-poets-agree.md

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (29)

GitHub Check: Integration Tests (machine, chrome)
GitHub Check: Integration Tests (nextjs, chrome, 15)
GitHub Check: Integration Tests (custom, chrome)
GitHub Check: Integration Tests (react-router, chrome)
GitHub Check: Integration Tests (nuxt, chrome)
GitHub Check: Integration Tests (nextjs, chrome, 14)
GitHub Check: Integration Tests (billing, chrome)
GitHub Check: Integration Tests (vue, chrome)
GitHub Check: Integration Tests (expo-web, chrome)
GitHub Check: Integration Tests (tanstack-react-start, chrome)
GitHub Check: Integration Tests (handshake, chrome)
GitHub Check: Integration Tests (handshake:staging, chrome)
GitHub Check: Integration Tests (tanstack-react-router, chrome)
GitHub Check: Integration Tests (astro, chrome)
GitHub Check: Integration Tests (sessions, chrome)
GitHub Check: Integration Tests (elements, chrome)
GitHub Check: Integration Tests (sessions:staging, chrome)
GitHub Check: Integration Tests (quickstart, chrome)
GitHub Check: Integration Tests (ap-flows, chrome)
GitHub Check: Integration Tests (generic, chrome)
GitHub Check: Integration Tests (localhost, chrome)
GitHub Check: Integration Tests (express, chrome)
GitHub Check: Unit Tests (18, --filter=@clerk/astro --filter=@clerk/backend --filter=@clerk/express --filter=@c...
GitHub Check: Unit Tests (22, **)
GitHub Check: Publish with pkg-pr-new
GitHub Check: Static analysis
GitHub Check: Analyze (javascript-typescript)
GitHub Check: semgrep-cloud-platform/scan
GitHub Check: semgrep-cloud-platform/scan

coderabbitai · 2025-10-05T19:58:33Z

.changeset/ten-poets-agree.md

+---
+"@clerk/nuxt": minor
+---
+
+Remove added `nextTick()` calls when invoking Clerk routing functions.


⚠️ Potential issue | 🟠 Major

Changeset summary contradicts the revert.

This PR restores the nextTick wrapper, yet the changeset still advertises removing it. Shipping this note would mislead downstream consumers and trigger an incorrect minor release. Please delete this changeset or replace it with one that accurately states the revert (likely a patch note explaining that the previous change was rolled back).

🤖 Prompt for AI Agents

.changeset/ten-poets-agree.md lines 1-5: the changeset message advertises removing nextTick and marks a minor bump but this PR actually reverts that change; either delete this changeset or update it to accurately describe the revert and use a patch release. Replace the body summary to state that the previous removal of nextTick was reverted (restored behavior), change the release type from "minor" to "patch" (or remove the file if you prefer no release note), and ensure the changelog text clearly explains the rollback.

pkg-pr-new · 2025-10-05T19:58:56Z

Open in StackBlitz

@clerk/agent-toolkit

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

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

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

@clerk/elements

npm i https://pkg.pr.new/@clerk/elements@6928

@clerk/clerk-expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/clerk-react

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

@clerk/react-router

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

@clerk/remix

npm i https://pkg.pr.new/@clerk/remix@6928

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/themes

npm i https://pkg.pr.new/@clerk/themes@6928

@clerk/types

npm i https://pkg.pr.new/@clerk/types@6928

@clerk/upgrade

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

@clerk/vue

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

commit: 4114153

Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing interme…

12e36c4

…diate st…" This reverts commit e2dd40a.

clerk-cookie added the nuxt label Oct 5, 2025

chore: add changeset

4114153

vercel bot deployed to Preview October 5, 2025 19:57 View deployment

wobsoriano mentioned this pull request Oct 5, 2025

Call to useSession().signOut doesn't redirect to login on @clerk/nuxt^1.9.0 #6904

Closed

4 tasks

coderabbitai bot reviewed Oct 5, 2025

View reviewed changes

wobsoriano requested a review from tmilewski October 6, 2025 16:45

This comment was marked as resolved.

Sign in to view

wobsoriano merged commit 6dfb3ec into main Oct 6, 2025
74 of 76 checks passed

wobsoriano deleted the revert-6802-rob/nuxt-fix-transition branch October 6, 2025 17:07

clerk-cookie mentioned this pull request Oct 6, 2025

ci(repo): Version packages #6929

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing intermediate states during navigation" #6928

Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing intermediate states during navigation" #6928

Uh oh!

wobsoriano commented Oct 5, 2025 •

edited

Loading

Uh oh!

changeset-bot bot commented Oct 5, 2025 •

edited

Loading

Uh oh!

vercel bot commented Oct 5, 2025 •

edited

Loading

Uh oh!

coderabbitai bot commented Oct 5, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Oct 5, 2025

Uh oh!

pkg-pr-new bot commented Oct 5, 2025

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing intermediate states during navigation" #6928

Revert "fix(nuxt): Prevent Nuxt route middlewares from seeing intermediate states during navigation" #6928

Uh oh!

Conversation

wobsoriano commented Oct 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

changeset-bot bot commented Oct 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

vercel bot commented Oct 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Oct 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Oct 5, 2025

Choose a reason for hiding this comment

Uh oh!

pkg-pr-new bot commented Oct 5, 2025

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

wobsoriano commented Oct 5, 2025 •

edited

Loading

changeset-bot bot commented Oct 5, 2025 •

edited

Loading

vercel bot commented Oct 5, 2025 •

edited

Loading

coderabbitai bot commented Oct 5, 2025 •

edited

Loading