refactor: update @mjackson/* packages to @remix-run/*#14930
Conversation
|
efae0a4 to
7659fc0
Compare
b58ba1e to
1a36ac0
Compare
There was a problem hiding this comment.
Pull request overview
Refactors the repo to replace usage of @mjackson/node-fetch-server with @remix-run/node-fetch-server, updating both runtime imports and workspace dependency declarations so Node/Express-based playgrounds, integration helpers, and published packages rely on the renamed package.
Changes:
- Replace
@mjackson/node-fetch-serverimports with@remix-run/node-fetch-serveracross playground, integration helpers, and package sources. - Update workspace
package.jsondependencies andpnpm-lock.yamlto resolve@remix-run/node-fetch-server@0.13.0. - Update a decision record reference/link (non-code documentation change).
Reviewed changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml | Lockfile updates swapping @mjackson/node-fetch-server to @remix-run/node-fetch-server@0.13.0 across importers and package snapshots. |
| playground/rsc-vite/server.js | Updates server import to @remix-run/node-fetch-server. |
| playground/rsc-vite/package.json | Updates dependency to @remix-run/node-fetch-server. |
| playground/rsc-vite-framework/start.js | Updates server import to @remix-run/node-fetch-server. |
| playground/rsc-vite-framework/start-vite-middleware.js | Updates server import to @remix-run/node-fetch-server. |
| playground/rsc-vite-framework/package.json | Updates dependency to @remix-run/node-fetch-server. |
| playground/rsc-vite-7-framework/start.js | Updates server import to @remix-run/node-fetch-server. |
| playground/rsc-vite-7-framework/start-vite-middleware.js | Updates server import to @remix-run/node-fetch-server. |
| playground/rsc-vite-7-framework/package.json | Updates dependency to @remix-run/node-fetch-server. |
| packages/react-router-serve/package.json | Updates published package dependency to @remix-run/node-fetch-server. |
| packages/react-router-serve/cli.ts | Updates CLI import to @remix-run/node-fetch-server. |
| packages/react-router-node/server.ts | Updates type/value imports to @remix-run/node-fetch-server. |
| packages/react-router-node/package.json | Updates published package dependency to @remix-run/node-fetch-server. |
| integration/vite-basename-test.ts | Updates embedded template server imports to @remix-run/node-fetch-server. |
| integration/helpers/vite.ts | Updates embedded template server import to @remix-run/node-fetch-server. |
| integration/helpers/rsc-vite/server.js | Updates helper server import to @remix-run/node-fetch-server. |
| integration/helpers/rsc-vite/package.json | Updates helper dependency to @remix-run/node-fetch-server. |
| integration/helpers/rsc-vite-framework/start.js | Updates helper server import to @remix-run/node-fetch-server. |
| integration/helpers/rsc-vite-framework/package.json | Updates helper dependency to @remix-run/node-fetch-server. |
| integration/helpers/express.ts | Updates embedded template server import to @remix-run/node-fetch-server. |
| decisions/0014-context-middleware.md | Updates a project reference/link in the decision record (non-package refactor change). |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1a36ac0 to
e68a9c7
Compare
Preview Build AvailablePreview builds have been created for this PR. You can install pnpm install "remix-run/react-router#preview/pr-14930&path:packages/react-router"And/or install other packages via: pnpm install "remix-run/react-router#preview/pr-14930&path:packages/react-router-dev"
pnpm install "remix-run/react-router#preview/pr-14930&path:packages/react-router-express"
pnpm install "remix-run/react-router#preview/pr-14930&path:packages/react-router-node"
pnpm install "remix-run/react-router#preview/pr-14930&path:packages/react-router-serve"These preview builds will be updated automatically as you push new commits. |
|
The preview branch |
#14468 (reply in thread)
Resubmission of #14070