Skip to content

fix(dev): refresh generated interception rewrites#94088

Open
bobu-putheeckal wants to merge 1 commit into
vercel:canaryfrom
bobu-putheeckal:codex/interception-rewrites-hmr
Open

fix(dev): refresh generated interception rewrites#94088
bobu-putheeckal wants to merge 1 commit into
vercel:canaryfrom
bobu-putheeckal:codex/interception-rewrites-hmr

Conversation

@bobu-putheeckal
Copy link
Copy Markdown

Closes #73803

Summary

  • Remove stale generated interception rewrites before appending the current generated set in dev.
  • Add a dev HMR regression test that renames an intercepted route and verifies the old rewrite no longer wins.

Testing

  • PATH=./node_modules/.bin:/Users/c543982/.npm/_npx/179a5fd3f63fede5/node_modules/.bin:$PATH scripts/run-jest.sh --mode=dev --bundler=webpack --headless -- test/development/app-dir/interception-routes-rewrites-hmr/interception-routes-rewrites-hmr.test.ts
  • git diff --check
  • ./node_modules/.bin/prettier --check packages/next/src/server/lib/router-utils/setup-dev-bundler.ts test/development/app-dir/interception-routes-rewrites-hmr/interception-routes-rewrites-hmr.test.ts test/development/app-dir/interception-routes-rewrites-hmr/app/layout.js test/development/app-dir/interception-routes-rewrites-hmr/app/page.js 'test/development/app-dir/interception-routes-rewrites-hmr/app/@slot/(.)photo/page.js' test/development/app-dir/interception-routes-rewrites-hmr/app/@slot/default.js test/development/app-dir/interception-routes-rewrites-hmr/app/photo/page.js test/development/app-dir/interception-routes-rewrites-hmr/app/picture/page.js

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.

[bug]: parallel/intercepting routes requires server reboot to work

1 participant