Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

fix(render): add missing nextTick import#1563

Merged
danielroe merged 1 commit intonuxt:mainnuxt/test-utils:mainfrom
adamdehaven:fix/missing-importadamdehaven/test-utils:fix/missing-importCopy head branch name to clipboard
Jan 26, 2026
Merged

fix(render): add missing nextTick import#1563
danielroe merged 1 commit intonuxt:mainnuxt/test-utils:mainfrom
adamdehaven:fix/missing-importadamdehaven/test-utils:fix/missing-importCopy head branch name to clipboard

Conversation

@adamdehaven
Copy link
Copy Markdown
Contributor

@adamdehaven adamdehaven commented Jan 23, 2026 •
edited by danielroe
Loading

🔗 Linked issue

resolves #1562

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The nextTick import apepars to have been unintentionally removed in #1522. This pull request restores the missing import and fixes #1562.

@adamdehaven
fix(render): add missing nextTick import
5a7f145 
The nextTick import apepars to have been unintentionally removed in nuxt#1522. This pull request restores the missing import and fixes nuxt#1562.
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 23, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/test-utils@1563

commit: 5a7f145

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 23, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

This change adds nextTick to the Vue import statement in src/runtime-utils/render.ts. The modification addresses a missing dependency that was being referenced in the rerender logic but was not imported, causing a ReferenceError when rerender was called from renderSuspended. The adjustment consists of a single import line change with no modifications to function signatures or control flow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly addresses the main change: adding a missing nextTick import to the render.ts file.
Linked Issues check ✅ Passed The PR restores the missing nextTick import to fix the ReferenceError thrown when calling rerender after renderSuspended, directly addressing issue #1562.
Out of Scope Changes check ✅ Passed The change is minimal and scoped: only restoring a single missing import to render.ts with no other modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description clearly explains the issue being fixed: restoring a missing nextTick import that was unintentionally removed in PR #1522 to fix the ReferenceError when calling rerender from renderSuspended.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@adamdehaven
Copy link
Copy Markdown
Contributor Author

adamdehaven commented Jan 23, 2026

@danielroe I attempted to see where the type check should have caught this but ran into some other issues with the "moduleResolution": "Bundler" missing the module setting, so didn't include here.

It would probably be worth adding a type check somewhere in the release pipeline that would have (I believe) caught the missing import

@danielroe danielroe merged commit 07d6b4a into nuxt:main Jan 26, 2026
8 checks passed
@github-actions github-actions Bot mentioned this pull request Jan 26, 2026
Merged
@adamdehaven adamdehaven deleted the fix/missing-import branch January 26, 2026 15:14
@coderabbitai coderabbitai Bot mentioned this pull request Feb 11, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danielroe danielroe danielroe approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Calling rerender from renderSuspended throws an error nextTick is not defined

2 participants

@adamdehaven @danielroe
Morty Proxy This is a proxified and sanitized view of the page, visit original site.