Skip to content

Conversation

@adamdehaven
Copy link
Contributor

πŸ”— Linked issue

#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.

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

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

coderabbitai bot commented Jan 23, 2026

πŸ“ 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.
Description check βœ… Passed The description explains the bug fix, references the linked issue #1562, and clarifies that the import was removed in #1522.
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.

✏️ 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
Contributor Author

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Calling rerender from renderSuspended throws an error nextTick is not defined

2 participants