decouple #84529
Draft
decouple #84529
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Failing test suitesCommit: cfc26b0 | About building and testing Next.js
Expand output● should work with runtime-config in next.config.js › empty runtime-config ● should work with runtime-config in next.config.js › with runtime-config
Expand output● create-next-app with package manager bun › should use bun for --use-bun flag ● create-next-app with package manager bun › should use bun when user-agent is bun ● create-next-app with package manager bun › should use bun for --use-bun flag with example ● create-next-app with package manager bun › should use bun when user-agent is bun with example
Expand output● hydration-error-count › should have correct hydration error count for bad nesting ● hydration-error-count › should have correct hydration error count for html diff ● hydration-error-count › should display correct hydration info in each hydration error view ● hydration-error-count › should display runtime error separately from hydration errors
Expand output● segment-explorer - globals › should show global-error segment ● segment-explorer - globals › should display parallel routes default page when present
Expand output● Error overlay for hydration errors in App router › should show correct hydration error when client and server render different text ● Error overlay for hydration errors in App router › should show correct hydration error when client renders an extra element ● Error overlay for hydration errors in App router › should show correct hydration error when extra attributes set on server ● Error overlay for hydration errors in App router › should show correct hydration error when client renders an extra text node ● Error overlay for hydration errors in App router › should show correct hydration error when server renders an extra element ● Error overlay for hydration errors in App router › should show correct hydration error when server renders an extra text node ● Error overlay for hydration errors in App router › should show correct hydration error when server renders an extra text node in an invalid place ● Error overlay for hydration errors in App router › should show correct hydration error when server renders an extra whitespace in an invalid place ● Error overlay for hydration errors in App router › should show correct hydration error when client renders an extra node inside Suspense content ● Error overlay for hydration errors in App router › should only show one hydration error when bad nesting happened - p under p ● Error overlay for hydration errors in App router › should only show one hydration error when bad nesting happened - div under p ● Error overlay for hydration errors in App router › should only show one hydration error when bad nesting happened - div > tr ● Error overlay for hydration errors in App router › should show the highlighted bad nesting html snippet when bad nesting happened ● Error overlay for hydration errors in App router › should show error if script is directly placed under html instead of body
Expand output● segment-explorer › should render the segment explorer for parallel routes ● segment-explorer › should render the segment explorer for parallel routes in edge runtime ● segment-explorer › should render the segment explorer for nested routes ● segment-explorer › should cleanup on soft navigation ● segment-explorer › should cleanup on soft navigation ● segment-explorer › should handle show file segments in order ● segment-explorer › should handle special built-in not-found segments ● segment-explorer › should show navigation boundaries of the segment ● segment-explorer › should show navigation boundaries of the segment ● segment-explorer › should show navigation boundaries of the segment ● segment-explorer › should show the loading boundary when it is present ● segment-explorer › should show the custom error boundary when present ● segment-explorer › should display parallel routes default page when present ● segment-explorer › should display boundary selector when a segment has only boundary files ● segment-explorer › should render route for index page |
Stats from current PRDefault Build (Increase detected
|
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| buildDuration | 18s | 15.1s | N/A |
| buildDurationCached | 14.2s | 11.4s | N/A |
| nodeModulesSize | 448 MB | 447 MB | N/A |
| nextStartRea..uration (ms) | 702ms | 715ms | N/A |
Client Bundles (main, webpack)
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| 1916.HASH.js gzip | 169 B | 169 B | ✓ |
| 7322-HASH.js gzip | 46 kB | 45.8 kB | N/A |
| 9225-HASH.js gzip | 5.19 kB | 5.18 kB | N/A |
| c57d0559-HASH.js gzip | 55 kB | 55 kB | N/A |
| framework-HASH.js gzip | 59.8 kB | 59.8 kB | ✓ |
| main-app-HASH.js gzip | 260 B | 261 B | N/A |
| main-HASH.js gzip | 36.4 kB | 36.3 kB | N/A |
| webpack-HASH.js gzip | 1.69 kB | 1.69 kB | N/A |
| Overall change | 60 kB | 60 kB | ✓ |
Legacy Client Bundles (polyfills)
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| polyfills-HASH.js gzip | 39.4 kB | 39.4 kB | ✓ |
| Overall change | 39.4 kB | 39.4 kB | ✓ |
Client Pages
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| _app-HASH.js gzip | 193 B | 193 B | ✓ |
| _error-HASH.js gzip | 182 B | 182 B | ✓ |
| css-HASH.js gzip | 334 B | 334 B | ✓ |
| dynamic-HASH.js gzip | 1.8 kB | 1.79 kB | N/A |
| edge-ssr-HASH.js gzip | 256 B | 255 B | N/A |
| head-HASH.js gzip | 350 B | 352 B | N/A |
| hooks-HASH.js gzip | 384 B | 381 B | N/A |
| image-HASH.js gzip | 4.67 kB | 4.66 kB | N/A |
| index-HASH.js gzip | 259 B | 260 B | N/A |
| link-HASH.js gzip | 2.28 kB | 2.28 kB | N/A |
| routerDirect..HASH.js gzip | 318 B | 317 B | N/A |
| script-HASH.js gzip | 386 B | 386 B | ✓ |
| withRouter-HASH.js gzip | 315 B | 315 B | ✓ |
| 1afbb74e6ecf..834.css gzip | 106 B | 106 B | ✓ |
| Overall change | 1.52 kB | 1.52 kB | ✓ |
Client Build Manifests
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| _buildManifest.js gzip | 716 B | 719 B | N/A |
| Overall change | 0 B | 0 B | ✓ |
Rendered Page Sizes
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| index.html gzip | 523 B | 522 B | N/A |
| link.html gzip | 536 B | 537 B | N/A |
| withRouter.html gzip | 519 B | 518 B | N/A |
| Overall change | 0 B | 0 B | ✓ |
Edge SSR bundle Size Overall increase ⚠️
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| edge-ssr.js gzip | 129 kB | 129 kB | N/A |
| page.js gzip | 242 kB | 243 kB | |
| Overall change | 242 kB | 243 kB |
Middleware size
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| middleware-b..fest.js gzip | 635 B | 639 B | N/A |
| middleware-r..fest.js gzip | 156 B | 157 B | N/A |
| middleware.js gzip | 33.1 kB | 32.8 kB | N/A |
| edge-runtime..pack.js gzip | 846 B | 846 B | ✓ |
| Overall change | 846 B | 846 B | ✓ |
Next Runtimes
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| app-page-exp...dev.js gzip | 288 kB | 288 kB | N/A |
| app-page-exp..prod.js gzip | 158 kB | 158 kB | N/A |
| app-page-tur...dev.js gzip | 288 kB | 288 kB | N/A |
| app-page-tur..prod.js gzip | 158 kB | 158 kB | N/A |
| app-page-tur...dev.js gzip | 282 kB | 282 kB | N/A |
| app-page-tur..prod.js gzip | 154 kB | 154 kB | N/A |
| app-page.run...dev.js gzip | 282 kB | 282 kB | N/A |
| app-page.run..prod.js gzip | 154 kB | 154 kB | N/A |
| app-route-ex...dev.js gzip | 70.1 kB | 70.1 kB | ✓ |
| app-route-ex..prod.js gzip | 49.3 kB | 49.3 kB | ✓ |
| app-route-tu...dev.js gzip | 70.1 kB | 70.1 kB | ✓ |
| app-route-tu..prod.js gzip | 49.3 kB | 49.3 kB | ✓ |
| app-route-tu...dev.js gzip | 69.5 kB | 69.5 kB | ✓ |
| app-route-tu..prod.js gzip | 49 kB | 49 kB | ✓ |
| app-route.ru...dev.js gzip | 69.5 kB | 69.5 kB | ✓ |
| app-route.ru..prod.js gzip | 48.9 kB | 48.9 kB | ✓ |
| dist_client_...dev.js gzip | 326 B | 326 B | ✓ |
| dist_client_...dev.js gzip | 328 B | 328 B | ✓ |
| dist_client_...dev.js gzip | 320 B | 320 B | ✓ |
| dist_client_...dev.js gzip | 318 B | 318 B | ✓ |
| pages-api-tu...dev.js gzip | 43.1 kB | 43.1 kB | ✓ |
| pages-api-tu..prod.js gzip | 33.1 kB | 33.1 kB | ✓ |
| pages-api.ru...dev.js gzip | 43 kB | 43 kB | ✓ |
| pages-api.ru..prod.js gzip | 33.1 kB | 33.1 kB | ✓ |
| pages-turbo....dev.js gzip | 52.8 kB | 52.8 kB | ✓ |
| pages-turbo...prod.js gzip | 40.2 kB | 40.2 kB | ✓ |
| pages.runtim...dev.js gzip | 52.8 kB | 52.8 kB | ✓ |
| pages.runtim..prod.js gzip | 40.2 kB | 40.2 kB | ✓ |
| server.runti..prod.js gzip | 78.3 kB | 78.3 kB | ✓ |
| Overall change | 894 kB | 894 kB | ✓ |
build cache Overall increase ⚠️
| vercel/next.js canary | vercel/next.js jude/devtools-api-and-mcp | Change | |
|---|---|---|---|
| 0.pack gzip | 3.07 MB | 3.08 MB | |
| index.pack gzip | 92.5 kB | 92.5 kB | N/A |
| Overall change | 3.07 MB | 3.08 MB |
Diff details
Diff for page.js
Diff too large to display
Diff for middleware.js
Diff too large to display
Diff for edge-ssr.js
failed to diffDiff for css-HASH.js
@@ -1,7 +1,31 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[9813],
{
- /***/ 3593: /***/ (
+ /***/ 5832: /***/ (module) => {
+ // extracted by mini-css-extract-plugin
+ module.exports = { helloWorld: "css_helloWorld__aUdUq" };
+
+ /***/
+ },
+
+ /***/ 6471: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/css",
+ function () {
+ return __webpack_require__(7839);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
+
+ /***/ 7839: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -15,7 +39,7 @@
/* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
__webpack_require__(1503);
/* harmony import */ var _css_module_css__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(7634);
+ __webpack_require__(5832);
/* harmony import */ var _css_module_css__WEBPACK_IMPORTED_MODULE_1___default =
/*#__PURE__*/ __webpack_require__.n(
_css_module_css__WEBPACK_IMPORTED_MODULE_1__
@@ -34,37 +58,13 @@
/***/
},
-
- /***/ 4569: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/css",
- function () {
- return __webpack_require__(3593);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
-
- /***/ 7634: /***/ (module) => {
- // extracted by mini-css-extract-plugin
- module.exports = { helloWorld: "css_helloWorld__aUdUq" };
-
- /***/
- },
},
/******/ (__webpack_require__) => {
// webpackRuntimeModules
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(4569)
+ __webpack_exec__(6471)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for dynamic-HASH.js
@@ -1,24 +1,7 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[2291],
{
- /***/ 1033: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/dynamic",
- function () {
- return __webpack_require__(2283);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
-
- /***/ 2283: /***/ (
+ /***/ 133: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -33,7 +16,7 @@
/* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
__webpack_require__(1503);
/* harmony import */ var next_dynamic__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(4939);
+ __webpack_require__(7514);
/* harmony import */ var next_dynamic__WEBPACK_IMPORTED_MODULE_1___default =
/*#__PURE__*/ __webpack_require__.n(
next_dynamic__WEBPACK_IMPORTED_MODULE_1__
@@ -42,12 +25,12 @@
const DynamicHello = next_dynamic__WEBPACK_IMPORTED_MODULE_1___default()(
() =>
__webpack_require__
- .e(/* import() */ 1916)
- .then(__webpack_require__.bind(__webpack_require__, 1916))
+ .e(/* import() */ 3862)
+ .then(__webpack_require__.bind(__webpack_require__, 3862))
.then((mod) => mod.Hello),
{
loadableGenerated: {
- webpack: () => [/*require.resolve*/ 1916],
+ webpack: () => [/*require.resolve*/ 3862],
},
}
);
@@ -74,17 +57,24 @@
/***/
},
- /***/ 4939: /***/ (
- module,
+ /***/ 431: /***/ (
+ __unused_webpack_module,
__unused_webpack_exports,
__webpack_require__
) => {
- module.exports = __webpack_require__(5121);
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/dynamic",
+ function () {
+ return __webpack_require__(133);
+ },
+ ]);
+ if (false) {
+ }
/***/
},
- /***/ 5121: /***/ (module, exports, __webpack_require__) => {
+ /***/ 1709: /***/ (module, exports, __webpack_require__) => {
"use strict";
Object.defineProperty(exports, "__esModule", {
@@ -117,7 +107,7 @@
__webpack_require__(2223)
);
const _loadablesharedruntime = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(7622)
+ __webpack_require__(2522)
);
const isServerSide = "object" === "undefined";
// Normalize loader to return the module as form { default: Component } for `React.lazy`.
@@ -217,34 +207,7 @@
/***/
},
- /***/ 5160: /***/ (
- __unused_webpack_module,
- exports,
- __webpack_require__
- ) => {
- "use strict";
- /* __next_internal_client_entry_do_not_use__ cjs */
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "LoadableContext", {
- enumerable: true,
- get: function () {
- return LoadableContext;
- },
- });
- const _interop_require_default = __webpack_require__(1532);
- const _react = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(2223)
- );
- const LoadableContext = _react.default.createContext(null);
- if (false) {
- } //# sourceMappingURL=loadable-context.shared-runtime.js.map
-
- /***/
- },
-
- /***/ 7622: /***/ (
+ /***/ 2522: /***/ (
__unused_webpack_module,
exports,
__webpack_require__
@@ -286,7 +249,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE
const _react = /*#__PURE__*/ _interop_require_default._(
__webpack_require__(2223)
);
- const _loadablecontextsharedruntime = __webpack_require__(5160);
+ const _loadablecontextsharedruntime = __webpack_require__(4844);
function resolve(obj) {
return obj && obj.default ? obj.default : obj;
}
@@ -519,13 +482,50 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE
/***/
},
+
+ /***/ 4844: /***/ (
+ __unused_webpack_module,
+ exports,
+ __webpack_require__
+ ) => {
+ "use strict";
+ /* __next_internal_client_entry_do_not_use__ cjs */
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ Object.defineProperty(exports, "LoadableContext", {
+ enumerable: true,
+ get: function () {
+ return LoadableContext;
+ },
+ });
+ const _interop_require_default = __webpack_require__(1532);
+ const _react = /*#__PURE__*/ _interop_require_default._(
+ __webpack_require__(2223)
+ );
+ const LoadableContext = _react.default.createContext(null);
+ if (false) {
+ } //# sourceMappingURL=loadable-context.shared-runtime.js.map
+
+ /***/
+ },
+
+ /***/ 7514: /***/ (
+ module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ module.exports = __webpack_require__(1709);
+
+ /***/
+ },
},
/******/ (__webpack_require__) => {
// webpackRuntimeModules
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(1033)
+ __webpack_exec__(431)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for hooks-HASH.js
@@ -1,24 +1,7 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[9804],
{
- /***/ 1679: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/hooks",
- function () {
- return __webpack_require__(9198);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
-
- /***/ 9198: /***/ (
+ /***/ 2592: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -76,13 +59,30 @@
/***/
},
+
+ /***/ 3925: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/hooks",
+ function () {
+ return __webpack_require__(2592);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
},
/******/ (__webpack_require__) => {
// webpackRuntimeModules
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(1679)
+ __webpack_exec__(3925)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for image-HASH.js
Diff too large to display
Diff for index-HASH.js
@@ -1,7 +1,24 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[3332],
{
- /***/ 3454: /***/ (
+ /***/ 8431: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/",
+ function () {
+ return __webpack_require__(8972);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
+
+ /***/ 8972: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -19,30 +36,13 @@
/***/
},
-
- /***/ 9241: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/",
- function () {
- return __webpack_require__(3454);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
},
/******/ (__webpack_require__) => {
// webpackRuntimeModules
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(9241)
+ __webpack_exec__(8431)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for link-HASH.js
@@ -1,7 +1,24 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[4672],
{
- /***/ 3262: /***/ (module, exports, __webpack_require__) => {
+ /***/ 2025: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/link",
+ function () {
+ return __webpack_require__(3072);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
+
+ /***/ 3066: /***/ (module, exports, __webpack_require__) => {
"use strict";
/* __next_internal_client_entry_do_not_use__ cjs */
Object.defineProperty(exports, "__esModule", {
@@ -28,16 +45,16 @@
const _react = /*#__PURE__*/ _interop_require_wildcard._(
__webpack_require__(2223)
);
- const _resolvehref = __webpack_require__(7868);
- const _islocalurl = __webpack_require__(9350);
- const _formaturl = __webpack_require__(6319);
- const _utils = __webpack_require__(9889);
- const _addlocale = __webpack_require__(9466);
- const _routercontextsharedruntime = __webpack_require__(5691);
- const _useintersection = __webpack_require__(3981);
- const _getdomainlocale = __webpack_require__(4206);
- const _addbasepath = __webpack_require__(9339);
- const _usemergedref = __webpack_require__(6848);
+ const _resolvehref = __webpack_require__(2776);
+ const _islocalurl = __webpack_require__(5770);
+ const _formaturl = __webpack_require__(8531);
+ const _utils = __webpack_require__(3565);
+ const _addlocale = __webpack_require__(8878);
+ const _routercontextsharedruntime = __webpack_require__(5111);
+ const _useintersection = __webpack_require__(6873);
+ const _getdomainlocale = __webpack_require__(4554);
+ const _addbasepath = __webpack_require__(9567);
+ const _usemergedref = __webpack_require__(4308);
const prefetched = new Set();
function prefetch(router, href, as, options) {
if (false) {
@@ -351,7 +368,167 @@
/***/
},
- /***/ 3981: /***/ (module, exports, __webpack_require__) => {
+ /***/ 3072: /***/ (
+ __unused_webpack_module,
+ __webpack_exports__,
+ __webpack_require__
+ ) => {
+ "use strict";
+ __webpack_require__.r(__webpack_exports__);
+ /* harmony export */ __webpack_require__.d(__webpack_exports__, {
+ /* harmony export */ __N_SSP: () => /* binding */ __N_SSP,
+ /* harmony export */ default: () => __WEBPACK_DEFAULT_EXPORT__,
+ /* harmony export */
+ });
+ /* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
+ __webpack_require__(1503);
+ /* harmony import */ var next_link__WEBPACK_IMPORTED_MODULE_1__ =
+ __webpack_require__(6929);
+ /* harmony import */ var next_link__WEBPACK_IMPORTED_MODULE_1___default =
+ /*#__PURE__*/ __webpack_require__.n(
+ next_link__WEBPACK_IMPORTED_MODULE_1__
+ );
+
+ function aLink(props) {
+ return /*#__PURE__*/ (0,
+ react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsxs)("div", {
+ children: [
+ /*#__PURE__*/ (0,
+ react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)("h3", {
+ children: "A Link page!",
+ }),
+ /*#__PURE__*/ (0,
+ react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)(
+ next_link__WEBPACK_IMPORTED_MODULE_1___default(),
+ {
+ href: "/",
+ children: "Go to /",
+ }
+ ),
+ ],
+ });
+ }
+ var __N_SSP = true;
+ /* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = aLink;
+
+ /***/
+ },
+
+ /***/ 4308: /***/ (module, exports, __webpack_require__) => {
+ "use strict";
+
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ Object.defineProperty(exports, "useMergedRef", {
+ enumerable: true,
+ get: function () {
+ return useMergedRef;
+ },
+ });
+ const _react = __webpack_require__(2223);
+ function useMergedRef(refA, refB) {
+ const cleanupA = (0, _react.useRef)(null);
+ const cleanupB = (0, _react.useRef)(null);
+ // NOTE: In theory, we could skip the wrapping if only one of the refs is non-null.
+ // (this happens often if the user doesn't pass a ref to Link/Form/Image)
+ // But this can cause us to leak a cleanup-ref into user code (previously via `<Link legacyBehavior>`),
+ // and the user might pass that ref into ref-merging library that doesn't support cleanup refs
+ // (because it hasn't been updated for React 19)
+ // which can then cause things to blow up, because a cleanup-returning ref gets called with `null`.
+ // So in practice, it's safer to be defensive and always wrap the ref, even on React 19.
+ return (0, _react.useCallback)(
+ (current) => {
+ if (current === null) {
+ const cleanupFnA = cleanupA.current;
+ if (cleanupFnA) {
+ cleanupA.current = null;
+ cleanupFnA();
+ }
+ const cleanupFnB = cleanupB.current;
+ if (cleanupFnB) {
+ cleanupB.current = null;
+ cleanupFnB();
+ }
+ } else {
+ if (refA) {
+ cleanupA.current = applyRef(refA, current);
+ }
+ if (refB) {
+ cleanupB.current = applyRef(refB, current);
+ }
+ }
+ },
+ [refA, refB]
+ );
+ }
+ function applyRef(refA, current) {
+ if (typeof refA === "function") {
+ const cleanup = refA(current);
+ if (typeof cleanup === "function") {
+ return cleanup;
+ } else {
+ return () => refA(null);
+ }
+ } else {
+ refA.current = current;
+ return () => {
+ refA.current = null;
+ };
+ }
+ }
+ if (
+ (typeof exports.default === "function" ||
+ (typeof exports.default === "object" && exports.default !== null)) &&
+ typeof exports.default.__esModule === "undefined"
+ ) {
+ Object.defineProperty(exports.default, "__esModule", {
+ value: true,
+ });
+ Object.assign(exports.default, exports);
+ module.exports = exports.default;
+ } //# sourceMappingURL=use-merged-ref.js.map
+
+ /***/
+ },
+
+ /***/ 4554: /***/ (module, exports, __webpack_require__) => {
+ "use strict";
+
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ Object.defineProperty(exports, "getDomainLocale", {
+ enumerable: true,
+ get: function () {
+ return getDomainLocale;
+ },
+ });
+ const _normalizetrailingslash = __webpack_require__(7740);
+ const basePath =
+ /* unused pure expression or super */ null && (false || "");
+ function getDomainLocale(path, locale, locales, domainLocales) {
+ if (false) {
+ } else {
+ return false;
+ }
+ }
+ if (
+ (typeof exports.default === "function" ||
+ (typeof exports.default === "object" && exports.default !== null)) &&
+ typeof exports.default.__esModule === "undefined"
+ ) {
+ Object.defineProperty(exports.default, "__esModule", {
+ value: true,
+ });
+ Object.assign(exports.default, exports);
+ module.exports = exports.default;
+ } //# sourceMappingURL=get-domain-locale.js.map
+
+ /***/
+ },
+
+ /***/ 6873: /***/ (module, exports, __webpack_require__) => {
"use strict";
Object.defineProperty(exports, "__esModule", {
@@ -364,7 +541,7 @@
},
});
const _react = __webpack_require__(2223);
- const _requestidlecallback = __webpack_require__(1472);
+ const _requestidlecallback = __webpack_require__(6892);
const hasIntersectionObserver =
typeof IntersectionObserver === "function";
const observers = new Map();
@@ -476,189 +653,12 @@
/***/
},
- /***/ 4206: /***/ (module, exports, __webpack_require__) => {
- "use strict";
-
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "getDomainLocale", {
- enumerable: true,
- get: function () {
- return getDomainLocale;
- },
- });
- const _normalizetrailingslash = __webpack_require__(6704);
- const basePath =
- /* unused pure expression or super */ null && (false || "");
- function getDomainLocale(path, locale, locales, domainLocales) {
- if (false) {
- } else {
- return false;
- }
- }
- if (
- (typeof exports.default === "function" ||
- (typeof exports.default === "object" && exports.default !== null)) &&
- typeof exports.default.__esModule === "undefined"
- ) {
- Object.defineProperty(exports.default, "__esModule", {
- value: true,
- });
- Object.assign(exports.default, exports);
- module.exports = exports.default;
- } //# sourceMappingURL=get-domain-locale.js.map
-
- /***/
- },
-
- /***/ 6691: /***/ (
+ /***/ 6929: /***/ (
module,
__unused_webpack_exports,
__webpack_require__
) => {
- module.exports = __webpack_require__(3262);
-
- /***/
- },
-
- /***/ 6771: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/link",
- function () {
- return __webpack_require__(8178);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
-
- /***/ 6848: /***/ (module, exports, __webpack_require__) => {
- "use strict";
-
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "useMergedRef", {
- enumerable: true,
- get: function () {
- return useMergedRef;
- },
- });
- const _react = __webpack_require__(2223);
- function useMergedRef(refA, refB) {
- const cleanupA = (0, _react.useRef)(null);
- const cleanupB = (0, _react.useRef)(null);
- // NOTE: In theory, we could skip the wrapping if only one of the refs is non-null.
- // (this happens often if the user doesn't pass a ref to Link/Form/Image)
- // But this can cause us to leak a cleanup-ref into user code (previously via `<Link legacyBehavior>`),
- // and the user might pass that ref into ref-merging library that doesn't support cleanup refs
- // (because it hasn't been updated for React 19)
- // which can then cause things to blow up, because a cleanup-returning ref gets called with `null`.
- // So in practice, it's safer to be defensive and always wrap the ref, even on React 19.
- return (0, _react.useCallback)(
- (current) => {
- if (current === null) {
- const cleanupFnA = cleanupA.current;
- if (cleanupFnA) {
- cleanupA.current = null;
- cleanupFnA();
- }
- const cleanupFnB = cleanupB.current;
- if (cleanupFnB) {
- cleanupB.current = null;
- cleanupFnB();
- }
- } else {
- if (refA) {
- cleanupA.current = applyRef(refA, current);
- }
- if (refB) {
- cleanupB.current = applyRef(refB, current);
- }
- }
- },
- [refA, refB]
- );
- }
- function applyRef(refA, current) {
- if (typeof refA === "function") {
- const cleanup = refA(current);
- if (typeof cleanup === "function") {
- return cleanup;
- } else {
- return () => refA(null);
- }
- } else {
- refA.current = current;
- return () => {
- refA.current = null;
- };
- }
- }
- if (
- (typeof exports.default === "function" ||
- (typeof exports.default === "object" && exports.default !== null)) &&
- typeof exports.default.__esModule === "undefined"
- ) {
- Object.defineProperty(exports.default, "__esModule", {
- value: true,
- });
- Object.assign(exports.default, exports);
- module.exports = exports.default;
- } //# sourceMappingURL=use-merged-ref.js.map
-
- /***/
- },
-
- /***/ 8178: /***/ (
- __unused_webpack_module,
- __webpack_exports__,
- __webpack_require__
- ) => {
- "use strict";
- __webpack_require__.r(__webpack_exports__);
- /* harmony export */ __webpack_require__.d(__webpack_exports__, {
- /* harmony export */ __N_SSP: () => /* binding */ __N_SSP,
- /* harmony export */ default: () => __WEBPACK_DEFAULT_EXPORT__,
- /* harmony export */
- });
- /* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
- __webpack_require__(1503);
- /* harmony import */ var next_link__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(6691);
- /* harmony import */ var next_link__WEBPACK_IMPORTED_MODULE_1___default =
- /*#__PURE__*/ __webpack_require__.n(
- next_link__WEBPACK_IMPORTED_MODULE_1__
- );
-
- function aLink(props) {
- return /*#__PURE__*/ (0,
- react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsxs)("div", {
- children: [
- /*#__PURE__*/ (0,
- react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)("h3", {
- children: "A Link page!",
- }),
- /*#__PURE__*/ (0,
- react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__.jsx)(
- next_link__WEBPACK_IMPORTED_MODULE_1___default(),
- {
- href: "/",
- children: "Go to /",
- }
- ),
- ],
- });
- }
- var __N_SSP = true;
- /* harmony default export */ const __WEBPACK_DEFAULT_EXPORT__ = aLink;
+ module.exports = __webpack_require__(3066);
/***/
},
@@ -668,7 +668,7 @@
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(6771)
+ __webpack_exec__(2025)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for routerDirect-HASH.js
@@ -1,7 +1,24 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[188],
{
- /***/ 286: /***/ (
+ /***/ 417: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/routerDirect",
+ function () {
+ return __webpack_require__(504);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
+
+ /***/ 504: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -16,7 +33,7 @@
/* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
__webpack_require__(1503);
/* harmony import */ var next_router__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(7798);
+ __webpack_require__(1840);
/* harmony import */ var next_router__WEBPACK_IMPORTED_MODULE_1___default =
/*#__PURE__*/ __webpack_require__.n(
next_router__WEBPACK_IMPORTED_MODULE_1__
@@ -36,29 +53,12 @@
/***/
},
- /***/ 4283: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/routerDirect",
- function () {
- return __webpack_require__(286);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
-
- /***/ 7798: /***/ (
+ /***/ 1840: /***/ (
module,
__unused_webpack_exports,
__webpack_require__
) => {
- module.exports = __webpack_require__(9587);
+ module.exports = __webpack_require__(1903);
/***/
},
@@ -68,7 +68,7 @@
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(4283)
+ __webpack_exec__(417)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for script-HASH.js
@@ -1,7 +1,34 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[1209],
{
- /***/ 661: /***/ (
+ /***/ 2398: /***/ (
+ module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ module.exports = __webpack_require__(2397);
+
+ /***/
+ },
+
+ /***/ 4305: /***/ (
+ __unused_webpack_module,
+ __unused_webpack_exports,
+ __webpack_require__
+ ) => {
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/script",
+ function () {
+ return __webpack_require__(8543);
+ },
+ ]);
+ if (false) {
+ }
+
+ /***/
+ },
+
+ /***/ 8543: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -16,7 +43,7 @@
/* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
__webpack_require__(1503);
/* harmony import */ var next_script__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(5964);
+ __webpack_require__(2398);
/* harmony import */ var next_script__WEBPACK_IMPORTED_MODULE_1___default =
/*#__PURE__*/ __webpack_require__.n(
next_script__WEBPACK_IMPORTED_MODULE_1__
@@ -48,40 +75,13 @@
/***/
},
-
- /***/ 5964: /***/ (
- module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- module.exports = __webpack_require__(297);
-
- /***/
- },
-
- /***/ 8803: /***/ (
- __unused_webpack_module,
- __unused_webpack_exports,
- __webpack_require__
- ) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/script",
- function () {
- return __webpack_require__(661);
- },
- ]);
- if (false) {
- }
-
- /***/
- },
},
/******/ (__webpack_require__) => {
// webpackRuntimeModules
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(8803)
+ __webpack_exec__(4305)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for withRouter-HASH.js
@@ -1,24 +1,17 @@
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
[3263],
{
- /***/ 3163: /***/ (
- __unused_webpack_module,
+ /***/ 1840: /***/ (
+ module,
__unused_webpack_exports,
__webpack_require__
) => {
- (window.__NEXT_P = window.__NEXT_P || []).push([
- "/withRouter",
- function () {
- return __webpack_require__(3295);
- },
- ]);
- if (false) {
- }
+ module.exports = __webpack_require__(1903);
/***/
},
- /***/ 3295: /***/ (
+ /***/ 2037: /***/ (
__unused_webpack_module,
__webpack_exports__,
__webpack_require__
@@ -33,7 +26,7 @@
/* harmony import */ var react_jsx_runtime__WEBPACK_IMPORTED_MODULE_0__ =
__webpack_require__(1503);
/* harmony import */ var next_router__WEBPACK_IMPORTED_MODULE_1__ =
- __webpack_require__(7798);
+ __webpack_require__(1840);
/* harmony import */ var next_router__WEBPACK_IMPORTED_MODULE_1___default =
/*#__PURE__*/ __webpack_require__.n(
next_router__WEBPACK_IMPORTED_MODULE_1__
@@ -52,12 +45,19 @@
/***/
},
- /***/ 7798: /***/ (
- module,
+ /***/ 4041: /***/ (
+ __unused_webpack_module,
__unused_webpack_exports,
__webpack_require__
) => {
- module.exports = __webpack_require__(9587);
+ (window.__NEXT_P = window.__NEXT_P || []).push([
+ "/withRouter",
+ function () {
+ return __webpack_require__(2037);
+ },
+ ]);
+ if (false) {
+ }
/***/
},
@@ -67,7 +67,7 @@
/******/ var __webpack_exec__ = (moduleId) =>
__webpack_require__((__webpack_require__.s = moduleId));
/******/ __webpack_require__.O(0, [636, 6593, 8792], () =>
- __webpack_exec__(3163)
+ __webpack_exec__(4041)
);
/******/ var __webpack_exports__ = __webpack_require__.O();
/******/ _N_E = __webpack_exports__;Diff for 7322-HASH.js
Diff too large to display
Diff for 9225-HASH.js
@@ -1,32 +1,81 @@
"use strict";
(self["webpackChunk_N_E"] = self["webpackChunk_N_E"] || []).push([
- [9225],
+ [8439],
{
- /***/ 2: /***/ (__unused_webpack_module, exports, __webpack_require__) => {
+ /***/ 405: /***/ (
+ __unused_webpack_module,
+ exports,
+ __webpack_require__
+ ) => {
Object.defineProperty(exports, "__esModule", {
value: true,
});
- Object.defineProperty(exports, "ImageConfigContext", {
+ Object.defineProperty(exports, "default", {
enumerable: true,
get: function () {
- return ImageConfigContext;
+ return SideEffect;
},
});
- const _interop_require_default = __webpack_require__(9010);
- const _react = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(8879)
- );
- const _imageconfig = __webpack_require__(1686);
- const ImageConfigContext = _react.default.createContext(
- _imageconfig.imageConfigDefault
- );
- if (false) {
- } //# sourceMappingURL=image-config-context.shared-runtime.js.map
+ const _react = __webpack_require__(1774);
+ const isServer = "object" === "undefined";
+ const useClientOnlyLayoutEffect = isServer
+ ? () => {}
+ : _react.useLayoutEffect;
+ const useClientOnlyEffect = isServer ? () => {} : _react.useEffect;
+ function SideEffect(props) {
+ const { headManager, reduceComponentsToState } = props;
+ function emitChange() {
+ if (headManager && headManager.mountedInstances) {
+ const headElements = _react.Children.toArray(
+ Array.from(headManager.mountedInstances).filter(Boolean)
+ );
+ headManager.updateHead(reduceComponentsToState(headElements));
+ }
+ }
+ if (isServer) {
+ headManager?.mountedInstances?.add(props.children);
+ emitChange();
+ }
+ useClientOnlyLayoutEffect(() => {
+ headManager?.mountedInstances?.add(props.children);
+ return () => {
+ headManager?.mountedInstances?.delete(props.children);
+ };
+ });
+ // We need to call `updateHead` method whenever the `SideEffect` is trigger in all
+ // life-cycles: mount, update, unmount. However, if there are multiple `SideEffect`s
+ // being rendered, we only trigger the method from the last one.
+ // This is ensured by keeping the last unflushed `updateHead` in the `_pendingUpdate`
+ // singleton in the layout effect pass, and actually trigger it in the effect pass.
+ useClientOnlyLayoutEffect(() => {
+ if (headManager) {
+ headManager._pendingUpdate = emitChange;
+ }
+ return () => {
+ if (headManager) {
+ headManager._pendingUpdate = emitChange;
+ }
+ };
+ });
+ useClientOnlyEffect(() => {
+ if (headManager && headManager._pendingUpdate) {
+ headManager._pendingUpdate();
+ headManager._pendingUpdate = null;
+ }
+ return () => {
+ if (headManager && headManager._pendingUpdate) {
+ headManager._pendingUpdate();
+ headManager._pendingUpdate = null;
+ }
+ };
+ });
+ return null;
+ } //# sourceMappingURL=side-effect.js.map
/***/
},
- /***/ 1169: /***/ (
+ /***/ 1155: /***/ (
__unused_webpack_module,
exports,
__webpack_require__
@@ -40,9 +89,9 @@
return getImgProps;
},
});
- const _warnonce = __webpack_require__(4352);
- const _imageblursvg = __webpack_require__(2314);
- const _imageconfig = __webpack_require__(1686);
+ const _warnonce = __webpack_require__(5358);
+ const _imageblursvg = __webpack_require__(6924);
+ const _imageconfig = __webpack_require__(9712);
const VALID_LOADING_VALUES =
/* unused pure expression or super */ null && [
"lazy",
@@ -473,241 +522,40 @@
/***/
},
- /***/ 1666: /***/ (module, exports, __webpack_require__) => {
- /* __next_internal_client_entry_do_not_use__ cjs */
+ /***/ 1417: /***/ (
+ __unused_webpack_module,
+ exports,
+ __webpack_require__
+ ) => {
Object.defineProperty(exports, "__esModule", {
value: true,
});
- 0 && 0;
- function _export(target, all) {
- for (var name in all)
- Object.defineProperty(target, name, {
- enumerable: true,
- get: all[name],
- });
- }
- _export(exports, {
- default: function () {
+ Object.defineProperty(exports, "default", {
+ enumerable: true,
+ get: function () {
return _default;
},
- defaultHead: function () {
- return defaultHead;
- },
});
- const _interop_require_default = __webpack_require__(9010);
- const _interop_require_wildcard = __webpack_require__(2275);
- const _jsxruntime = __webpack_require__(1024);
- const _react = /*#__PURE__*/ _interop_require_wildcard._(
- __webpack_require__(8879)
- );
- const _sideeffect = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(2543)
- );
- const _headmanagercontextsharedruntime = __webpack_require__(5428);
- const _warnonce = __webpack_require__(4352);
- function defaultHead() {
- const head = [
- /*#__PURE__*/ (0, _jsxruntime.jsx)(
- "meta",
- {
- charSet: "utf-8",
- },
- "charset"
- ),
- /*#__PURE__*/ (0, _jsxruntime.jsx)(
- "meta",
- {
- name: "viewport",
- content: "width=device-width",
- },
- "viewport"
- ),
- ];
- return head;
- }
- function onlyReactElement(list, child) {
- // React children can be "string" or "number" in this case we ignore them for backwards compat
- if (typeof child === "string" || typeof child === "number") {
- return list;
- }
- // Adds support for React.Fragment
- if (child.type === _react.default.Fragment) {
- return list.concat(
- _react.default.Children.toArray(child.props.children).reduce(
- (fragmentList, fragmentChild) => {
- if (
- typeof fragmentChild === "string" ||
- typeof fragmentChild === "number"
- ) {
- return fragmentList;
- }
- return fragmentList.concat(fragmentChild);
- },
- []
- )
- );
+ const _findclosestquality = __webpack_require__(1639);
+ function defaultLoader({ config, src, width, quality }) {
+ if (false) {
}
- return list.concat(child);
- }
- const METATYPES = ["name", "httpEquiv", "charSet", "itemProp"];
- /*
- returns a function for filtering head child elements
- which shouldn't be duplicated, like <title/>
- Also adds support for deduplicated `key` properties
-*/ function unique() {
- const keys = new Set();
- const tags = new Set();
- const metaTypes = new Set();
- const metaCategories = {};
- return (h) => {
- let isUnique = true;
- let hasKey = false;
- if (h.key && typeof h.key !== "number" && h.key.indexOf("$") > 0) {
- hasKey = true;
- const key = h.key.slice(h.key.indexOf("$") + 1);
- if (keys.has(key)) {
- isUnique = false;
- } else {
- keys.add(key);
- }
- }
- // eslint-disable-next-line default-case
- switch (h.type) {
- case "title":
- case "base":
- if (tags.has(h.type)) {
- isUnique = false;
- } else {
- tags.add(h.type);
- }
- break;
- case "meta":
- for (let i = 0, len = METATYPES.length; i < len; i++) {
- const metatype = METATYPES[i];
- if (!h.props.hasOwnProperty(metatype)) continue;
- if (metatype === "charSet") {
- if (metaTypes.has(metatype)) {
- isUnique = false;
- } else {
- metaTypes.add(metatype);
- }
- } else {
- const category = h.props[metatype];
- const categories = metaCategories[metatype] || new Set();
- if (
- (metatype !== "name" || !hasKey) &&
- categories.has(category)
- ) {
- isUnique = false;
- } else {
- categories.add(category);
- metaCategories[metatype] = categories;
- }
- }
- }
- break;
- }
- return isUnique;
- };
- }
- /**
- *
- * @param headChildrenElements List of children of <Head>
- */ function reduceComponents(headChildrenElements) {
- return headChildrenElements
- .reduce(onlyReactElement, [])
- .reverse()
- .concat(defaultHead().reverse())
- .filter(unique())
- .reverse()
- .map((c, i) => {
- const key = c.key || i;
- if (false) {
- }
- return /*#__PURE__*/ _react.default.cloneElement(c, {
- key,
- });
- });
- }
- /**
- * This component injects elements to `<head>` of your page.
- * To avoid duplicated `tags` in `<head>` you can use the `key` property, which will make sure every tag is only rendered once.
- */ function Head({ children }) {
- const headManager = (0, _react.useContext)(
- _headmanagercontextsharedruntime.HeadManagerContext
- );
- return /*#__PURE__*/ (0, _jsxruntime.jsx)(_sideeffect.default, {
- reduceComponentsToState: reduceComponents,
- headManager: headManager,
- children: children,
- });
- }
- const _default = Head;
- if (
- (typeof exports.default === "function" ||
- (typeof exports.default === "object" && exports.default !== null)) &&
- typeof exports.default.__esModule === "undefined"
- ) {
- Object.defineProperty(exports.default, "__esModule", {
- value: true,
- });
- Object.assign(exports.default, exports);
- module.exports = exports.default;
- } //# sourceMappingURL=head.js.map
-
- /***/
- },
-
- /***/ 1686: /***/ (__unused_webpack_module, exports) => {
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- 0 && 0;
- function _export(target, all) {
- for (var name in all)
- Object.defineProperty(target, name, {
- enumerable: true,
- get: all[name],
- });
+ const q = (0, _findclosestquality.findClosestQuality)(quality, config);
+ return `${config.path}?url=${encodeURIComponent(
+ src
+ )}&w=${width}&q=${q}${
+ src.startsWith("/_next/static/media/") && false ? 0 : ""
+ }`;
}
- _export(exports, {
- VALID_LOADERS: function () {
- return VALID_LOADERS;
- },
- imageConfigDefault: function () {
- return imageConfigDefault;
- },
- });
- const VALID_LOADERS = [
- "default",
- "imgix",
- "cloudinary",
- "akamai",
- "custom",
- ];
- const imageConfigDefault = {
- deviceSizes: [640, 750, 828, 1080, 1200, 1920, 2048, 3840],
- imageSizes: [16, 32, 48, 64, 96, 128, 256, 384],
- path: "/_next/image",
- loader: "default",
- loaderFile: "",
- domains: [],
- disableStaticImages: false,
- minimumCacheTTL: 14400,
- formats: ["image/webp"],
- dangerouslyAllowSVG: false,
- contentSecurityPolicy: `script-src 'none'; frame-src 'none'; sandbox;`,
- contentDispositionType: "attachment",
- localPatterns: undefined,
- remotePatterns: [],
- qualities: [75],
- unoptimized: false,
- }; //# sourceMappingURL=image-config.js.map
+ // We use this to determine if the import is the default loader
+ // or a custom loader defined by the user in next.config.js
+ defaultLoader.__next_img_default = true;
+ const _default = defaultLoader; //# sourceMappingURL=image-loader.js.map
/***/
},
- /***/ 2285: /***/ (__unused_webpack_module, exports) => {
+ /***/ 1639: /***/ (__unused_webpack_module, exports) => {
Object.defineProperty(exports, "__esModule", {
value: true,
});
@@ -731,7 +579,35 @@
/***/
},
- /***/ 2314: /***/ (__unused_webpack_module, exports) => {
+ /***/ 2896: /***/ (
+ __unused_webpack_module,
+ exports,
+ __webpack_require__
+ ) => {
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ Object.defineProperty(exports, "ImageConfigContext", {
+ enumerable: true,
+ get: function () {
+ return ImageConfigContext;
+ },
+ });
+ const _interop_require_default = __webpack_require__(2570);
+ const _react = /*#__PURE__*/ _interop_require_default._(
+ __webpack_require__(1774)
+ );
+ const _imageconfig = __webpack_require__(9712);
+ const ImageConfigContext = _react.default.createContext(
+ _imageconfig.imageConfigDefault
+ );
+ if (false) {
+ } //# sourceMappingURL=image-config-context.shared-runtime.js.map
+
+ /***/
+ },
+
+ /***/ 6924: /***/ (__unused_webpack_module, exports) => {
/**
* A shared function, used on both client and server, to generate a SVG blur placeholder.
*/
@@ -770,113 +646,7 @@
/***/
},
- /***/ 2543: /***/ (
- __unused_webpack_module,
- exports,
- __webpack_require__
- ) => {
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "default", {
- enumerable: true,
- get: function () {
- return SideEffect;
- },
- });
- const _react = __webpack_require__(8879);
- const isServer = "object" === "undefined";
- const useClientOnlyLayoutEffect = isServer
- ? () => {}
- : _react.useLayoutEffect;
- const useClientOnlyEffect = isServer ? () => {} : _react.useEffect;
- function SideEffect(props) {
- const { headManager, reduceComponentsToState } = props;
- function emitChange() {
- if (headManager && headManager.mountedInstances) {
- const headElements = _react.Children.toArray(
- Array.from(headManager.mountedInstances).filter(Boolean)
- );
- headManager.updateHead(reduceComponentsToState(headElements));
- }
- }
- if (isServer) {
- headManager?.mountedInstances?.add(props.children);
- emitChange();
- }
- useClientOnlyLayoutEffect(() => {
- headManager?.mountedInstances?.add(props.children);
- return () => {
- headManager?.mountedInstances?.delete(props.children);
- };
- });
- // We need to call `updateHead` method whenever the `SideEffect` is trigger in all
- // life-cycles: mount, update, unmount. However, if there are multiple `SideEffect`s
- // being rendered, we only trigger the method from the last one.
- // This is ensured by keeping the last unflushed `updateHead` in the `_pendingUpdate`
- // singleton in the layout effect pass, and actually trigger it in the effect pass.
- useClientOnlyLayoutEffect(() => {
- if (headManager) {
- headManager._pendingUpdate = emitChange;
- }
- return () => {
- if (headManager) {
- headManager._pendingUpdate = emitChange;
- }
- };
- });
- useClientOnlyEffect(() => {
- if (headManager && headManager._pendingUpdate) {
- headManager._pendingUpdate();
- headManager._pendingUpdate = null;
- }
- return () => {
- if (headManager && headManager._pendingUpdate) {
- headManager._pendingUpdate();
- headManager._pendingUpdate = null;
- }
- };
- });
- return null;
- } //# sourceMappingURL=side-effect.js.map
-
- /***/
- },
-
- /***/ 8035: /***/ (
- __unused_webpack_module,
- exports,
- __webpack_require__
- ) => {
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "default", {
- enumerable: true,
- get: function () {
- return _default;
- },
- });
- const _findclosestquality = __webpack_require__(2285);
- function defaultLoader({ config, src, width, quality }) {
- if (false) {
- }
- const q = (0, _findclosestquality.findClosestQuality)(quality, config);
- return `${config.path}?url=${encodeURIComponent(
- src
- )}&w=${width}&q=${q}${
- src.startsWith("/_next/static/media/") && false ? 0 : ""
- }`;
- }
- // We use this to determine if the import is the default loader
- // or a custom loader defined by the user in next.config.js
- defaultLoader.__next_img_default = true;
- const _default = defaultLoader; //# sourceMappingURL=image-loader.js.map
-
- /***/
- },
-
- /***/ 8848: /***/ (module, exports, __webpack_require__) => {
+ /***/ 7614: /***/ (module, exports, __webpack_require__) => {
Object.defineProperty(exports, "__esModule", {
value: true,
});
@@ -886,7 +656,7 @@
return useMergedRef;
},
});
- const _react = __webpack_require__(8879);
+ const _react = __webpack_require__(1774);
function useMergedRef(refA, refB) {
const cleanupA = (0, _react.useRef)(null);
const cleanupB = (0, _react.useRef)(null);
@@ -952,32 +722,7 @@
/***/
},
- /***/ 9059: /***/ (
- __unused_webpack_module,
- exports,
- __webpack_require__
- ) => {
- Object.defineProperty(exports, "__esModule", {
- value: true,
- });
- Object.defineProperty(exports, "RouterContext", {
- enumerable: true,
- get: function () {
- return RouterContext;
- },
- });
- const _interop_require_default = __webpack_require__(9010);
- const _react = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(8879)
- );
- const RouterContext = _react.default.createContext(null);
- if (false) {
- } //# sourceMappingURL=router-context.shared-runtime.js.map
-
- /***/
- },
-
- /***/ 9225: /***/ (module, exports, __webpack_require__) => {
+ /***/ 8439: /***/ (module, exports, __webpack_require__) => {
/* __next_internal_client_entry_do_not_use__ cjs */
Object.defineProperty(exports, "__esModule", {
value: true,
@@ -988,27 +733,27 @@
return Image;
},
});
- const _interop_require_default = __webpack_require__(9010);
- const _interop_require_wildcard = __webpack_require__(2275);
- const _jsxruntime = __webpack_require__(1024);
+ const _interop_require_default = __webpack_require__(2570);
+ const _interop_require_wildcard = __webpack_require__(5643);
+ const _jsxruntime = __webpack_require__(2014);
const _react = /*#__PURE__*/ _interop_require_wildcard._(
- __webpack_require__(8879)
+ __webpack_require__(1774)
);
const _reactdom = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(3869)
+ __webpack_require__(7499)
);
const _head = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(1666)
+ __webpack_require__(9452)
);
- const _getimgprops = __webpack_require__(1169);
- const _imageconfig = __webpack_require__(1686);
- const _imageconfigcontextsharedruntime = __webpack_require__(2);
- const _warnonce = __webpack_require__(4352);
- const _routercontextsharedruntime = __webpack_require__(9059);
+ const _getimgprops = __webpack_require__(1155);
+ const _imageconfig = __webpack_require__(9712);
+ const _imageconfigcontextsharedruntime = __webpack_require__(2896);
+ const _warnonce = __webpack_require__(5358);
+ const _routercontextsharedruntime = __webpack_require__(9813);
const _imageloader = /*#__PURE__*/ _interop_require_default._(
- __webpack_require__(8035)
+ __webpack_require__(1417)
);
- const _usemergedref = __webpack_require__(8848);
+ const _usemergedref = __webpack_require__(7614);
// This is replaced by webpack define plugin
const configEnv = {
deviceSizes: [640, 750, 828, 1080, 1200, 1920, 2048, 3840],
@@ -1329,5 +1074,264 @@
/***/
},
+
+ /***/ 9452: /***/ (module, exports, __webpack_require__) => {
+ /* __next_internal_client_entry_do_not_use__ cjs */
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ 0 && 0;
+ function _export(target, all) {
+ for (var name in all)
+ Object.defineProperty(target, name, {
+ enumerable: true,
+ get: all[name],
+ });
+ }
+ _export(exports, {
+ default: function () {
+ return _default;
+ },
+ defaultHead: function () {
+ return defaultHead;
+ },
+ });
+ const _interop_require_default = __webpack_require__(2570);
+ const _interop_require_wildcard = __webpack_require__(5643);
+ const _jsxruntime = __webpack_require__(2014);
+ const _react = /*#__PURE__*/ _interop_require_wildcard._(
+ __webpack_require__(1774)
+ );
+ const _sideeffect = /*#__PURE__*/ _interop_require_default._(
+ __webpack_require__(405)
+ );
+ const _headmanagercontextsharedruntime = __webpack_require__(5326);
+ const _warnonce = __webpack_require__(5358);
+ function defaultHead() {
+ const head = [
+ /*#__PURE__*/ (0, _jsxruntime.jsx)(
+ "meta",
+ {
+ charSet: "utf-8",
+ },
+ "charset"
+ ),
+ /*#__PURE__*/ (0, _jsxruntime.jsx)(
+ "meta",
+ {
+ name: "viewport",
+ content: "width=device-width",
+ },
+ "viewport"
+ ),
+ ];
+ return head;
+ }
+ function onlyReactElement(list, child) {
+ // React children can be "string" or "number" in this case we ignore them for backwards compat
+ if (typeof child === "string" || typeof child === "number") {
+ return list;
+ }
+ // Adds support for React.Fragment
+ if (child.type === _react.default.Fragment) {
+ return list.concat(
+ _react.default.Children.toArray(child.props.children).reduce(
+ (fragmentList, fragmentChild) => {
+ if (
+ typeof fragmentChild === "string" ||
+ typeof fragmentChild === "number"
+ ) {
+ return fragmentList;
+ }
+ return fragmentList.concat(fragmentChild);
+ },
+ []
+ )
+ );
+ }
+ return list.concat(child);
+ }
+ const METATYPES = ["name", "httpEquiv", "charSet", "itemProp"];
+ /*
+ returns a function for filtering head child elements
+ which shouldn't be duplicated, like <title/>
+ Also adds support for deduplicated `key` properties
+*/ function unique() {
+ const keys = new Set();
+ const tags = new Set();
+ const metaTypes = new Set();
+ const metaCategories = {};
+ return (h) => {
+ let isUnique = true;
+ let hasKey = false;
+ if (h.key && typeof h.key !== "number" && h.key.indexOf("$") > 0) {
+ hasKey = true;
+ const key = h.key.slice(h.key.indexOf("$") + 1);
+ if (keys.has(key)) {
+ isUnique = false;
+ } else {
+ keys.add(key);
+ }
+ }
+ // eslint-disable-next-line default-case
+ switch (h.type) {
+ case "title":
+ case "base":
+ if (tags.has(h.type)) {
+ isUnique = false;
+ } else {
+ tags.add(h.type);
+ }
+ break;
+ case "meta":
+ for (let i = 0, len = METATYPES.length; i < len; i++) {
+ const metatype = METATYPES[i];
+ if (!h.props.hasOwnProperty(metatype)) continue;
+ if (metatype === "charSet") {
+ if (metaTypes.has(metatype)) {
+ isUnique = false;
+ } else {
+ metaTypes.add(metatype);
+ }
+ } else {
+ const category = h.props[metatype];
+ const categories = metaCategories[metatype] || new Set();
+ if (
+ (metatype !== "name" || !hasKey) &&
+ categories.has(category)
+ ) {
+ isUnique = false;
+ } else {
+ categories.add(category);
+ metaCategories[metatype] = categories;
+ }
+ }
+ }
+ break;
+ }
+ return isUnique;
+ };
+ }
+ /**
+ *
+ * @param headChildrenElements List of children of <Head>
+ */ function reduceComponents(headChildrenElements) {
+ return headChildrenElements
+ .reduce(onlyReactElement, [])
+ .reverse()
+ .concat(defaultHead().reverse())
+ .filter(unique())
+ .reverse()
+ .map((c, i) => {
+ const key = c.key || i;
+ if (false) {
+ }
+ return /*#__PURE__*/ _react.default.cloneElement(c, {
+ key,
+ });
+ });
+ }
+ /**
+ * This component injects elements to `<head>` of your page.
+ * To avoid duplicated `tags` in `<head>` you can use the `key` property, which will make sure every tag is only rendered once.
+ */ function Head({ children }) {
+ const headManager = (0, _react.useContext)(
+ _headmanagercontextsharedruntime.HeadManagerContext
+ );
+ return /*#__PURE__*/ (0, _jsxruntime.jsx)(_sideeffect.default, {
+ reduceComponentsToState: reduceComponents,
+ headManager: headManager,
+ children: children,
+ });
+ }
+ const _default = Head;
+ if (
+ (typeof exports.default === "function" ||
+ (typeof exports.default === "object" && exports.default !== null)) &&
+ typeof exports.default.__esModule === "undefined"
+ ) {
+ Object.defineProperty(exports.default, "__esModule", {
+ value: true,
+ });
+ Object.assign(exports.default, exports);
+ module.exports = exports.default;
+ } //# sourceMappingURL=head.js.map
+
+ /***/
+ },
+
+ /***/ 9712: /***/ (__unused_webpack_module, exports) => {
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ 0 && 0;
+ function _export(target, all) {
+ for (var name in all)
+ Object.defineProperty(target, name, {
+ enumerable: true,
+ get: all[name],
+ });
+ }
+ _export(exports, {
+ VALID_LOADERS: function () {
+ return VALID_LOADERS;
+ },
+ imageConfigDefault: function () {
+ return imageConfigDefault;
+ },
+ });
+ const VALID_LOADERS = [
+ "default",
+ "imgix",
+ "cloudinary",
+ "akamai",
+ "custom",
+ ];
+ const imageConfigDefault = {
+ deviceSizes: [640, 750, 828, 1080, 1200, 1920, 2048, 3840],
+ imageSizes: [16, 32, 48, 64, 96, 128, 256, 384],
+ path: "/_next/image",
+ loader: "default",
+ loaderFile: "",
+ domains: [],
+ disableStaticImages: false,
+ minimumCacheTTL: 14400,
+ formats: ["image/webp"],
+ dangerouslyAllowSVG: false,
+ contentSecurityPolicy: `script-src 'none'; frame-src 'none'; sandbox;`,
+ contentDispositionType: "attachment",
+ localPatterns: undefined,
+ remotePatterns: [],
+ qualities: [75],
+ unoptimized: false,
+ }; //# sourceMappingURL=image-config.js.map
+
+ /***/
+ },
+
+ /***/ 9813: /***/ (
+ __unused_webpack_module,
+ exports,
+ __webpack_require__
+ ) => {
+ Object.defineProperty(exports, "__esModule", {
+ value: true,
+ });
+ Object.defineProperty(exports, "RouterContext", {
+ enumerable: true,
+ get: function () {
+ return RouterContext;
+ },
+ });
+ const _interop_require_default = __webpack_require__(2570);
+ const _react = /*#__PURE__*/ _interop_require_default._(
+ __webpack_require__(1774)
+ );
+ const RouterContext = _react.default.createContext(null);
+ if (false) {
+ } //# sourceMappingURL=router-context.shared-runtime.js.map
+
+ /***/
+ },
},
]);Diff for main-HASH.js
Diff too large to display
Diff for app-page-exp..ntime.dev.js
Diff too large to display
Diff for app-page-exp..time.prod.js
Diff too large to display
Diff for app-page-tur..ntime.dev.js
Diff too large to display
Diff for app-page-tur..time.prod.js
Diff too large to display
Diff for app-page-tur..ntime.dev.js
Diff too large to display
Diff for app-page-tur..time.prod.js
Diff too large to display
Diff for app-page.runtime.dev.js
Diff too large to display
Diff for app-page.runtime.prod.js
Diff too large to display
gaojude
force-pushed
the
jude/devtools-api-and-mcp
branch
from
eb4ef34 to
0f14add
Compare
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
| return | ||
| } | ||
|
|
||
| const responses = await createBrowserRequest<PageMetadata>( |
There was a problem hiding this comment.
The API handler uses the wrong TypeScript type for page metadata responses. It uses PageMetadata (which has a segments field) when it should use SegmentTrieData (which has a segmentTrie field).
View Details
📝 Patch Details
diff --git a/packages/next/src/server/dev/devtools-api/page-metadata.ts b/packages/next/src/server/dev/devtools-api/page-metadata.ts
index ebd1641e00..28e4fdad30 100644
--- a/packages/next/src/server/dev/devtools-api/page-metadata.ts
+++ b/packages/next/src/server/dev/devtools-api/page-metadata.ts
@@ -1,6 +1,6 @@
import type { ServerResponse } from 'http'
import type { HmrMessageSentToBrowser } from '../hot-reloader-types'
-import type { PageMetadata } from '../../../shared/lib/devtools-api-types'
+import type { SegmentTrieData } from '../../../shared/lib/devtools-api-types'
import { HMR_MESSAGE_SENT_TO_BROWSER } from '../hot-reloader-types'
import { createBrowserRequest } from '../../lib/devtools-api-utils/tools/utils/browser-communication'
@@ -23,14 +23,14 @@ export async function handlePageMetadataRequest(
return
}
- const responses = await createBrowserRequest<PageMetadata>(
+ const responses = await createBrowserRequest<SegmentTrieData | null>(
HMR_MESSAGE_SENT_TO_BROWSER.REQUEST_PAGE_METADATA,
sendHmrMessage,
getActiveConnectionCount,
5000
)
- const metadataByUrl = new Map<string, PageMetadata>()
+ const metadataByUrl = new Map<string, SegmentTrieData | null>()
for (const response of responses) {
if (response.data) {
metadataByUrl.set(response.url, response.data)
Analysis
Type mismatch in handlePageMetadataRequest causes incorrect TypeScript types for API response
What fails: The handlePageMetadataRequest function in packages/next/src/server/dev/devtools-api/page-metadata.ts uses incorrect TypeScript type PageMetadata (with segments field) when browser clients actually send SegmentTrieData (with segmentTrie field).
How to reproduce:
- Start Next.js dev server with devtools API enabled
- Open browser dev tools and navigate to a page
- Call
/_next/devtools-api/page-metadataendpoint - Response contains
segmentTriefield at runtime, but TypeScript thinks it'ssegments
Result: TypeScript type annotations don't match runtime data structure. The browser sends DevtoolsPageMetadataResponse with segmentTrieData: SegmentTrieData | null (containing { segmentTrie: SegmentTrieNode | null, routerType: 'app' | 'pages' }), but the API handler incorrectly types it as PageMetadata (containing { segments: PageSegment[], routerType: 'app' | 'pages' }).
Expected: API handler should use SegmentTrieData type to match the actual data structure sent by the browser. This is confirmed by:
- Browser code in
hot-reloader-app.tsxandhot-reloader-pages.tssendingsegmentTrieData: SegmentTrieData - Server handler in
hot-reloader-turbopack.tscallinghandlePageMetadataResponse(requestId, segmentTrieData, url) - MCP server consumer in
packages/next-devtools-mcp-server/src/tools/get-page-metadata.tsexpectingSegmentTrieDatawithsegmentTriefield - Test file
test/development/devtools-api/devtools-api-get-page-metadata.test.tsassertingmetadata.segmentTrieexists
Fix: Changed import from PageMetadata to SegmentTrieData and updated type annotations in createBrowserRequest and metadataByUrl Map to use correct type.
gaojude
force-pushed
the
jude/devtools-api-and-mcp
branch
from
0f14add to
265eae7
Compare
Comment on lines
+5
to
+12
| interface ServerActionResponse { | ||
| id: string | ||
| name?: string | ||
| file?: string | ||
| line?: number | ||
| column?: number | ||
| source?: string | ||
| } |
There was a problem hiding this comment.
The ServerActionResponse interface has incorrect field names that don't match what the actual API returns, causing the tool to display only the action ID without any useful information about the function name or location.
View Details
📝 Patch Details
diff --git a/packages/next-devtools-mcp-server/src/tools/get-server-action-by-id.ts b/packages/next-devtools-mcp-server/src/tools/get-server-action-by-id.ts
index 27be9ba0f4..7c45c9ab06 100644
--- a/packages/next-devtools-mcp-server/src/tools/get-server-action-by-id.ts
+++ b/packages/next-devtools-mcp-server/src/tools/get-server-action-by-id.ts
@@ -4,11 +4,12 @@ import { makeDevtoolsRequest } from '../utils/api-client'
interface ServerActionResponse {
id: string
- name?: string
- file?: string
- line?: number
- column?: number
- source?: string
+ runtime?: 'node' | 'edge'
+ filename?: string
+ functionName?: string
+ layer?: Record<string, string>
+ workers?: Record<string, unknown>
+ error?: string
}
export function registerGetActionByIdTool(server: McpServer) {
@@ -33,27 +34,28 @@ export function registerGetActionByIdTool(server: McpServer) {
lines.push(`**Server Action ID:** ${response.id}`)
- if (response.name) {
- lines.push(`**Name:** ${response.name}`)
+ if (response.error) {
+ lines.push(`**Error:** ${response.error}`)
+ return {
+ content: [
+ {
+ type: 'text',
+ text: lines.join('\n'),
+ },
+ ],
+ }
}
- if (response.file) {
- let location = response.file
- if (response.line !== undefined) {
- location += `:${response.line}`
- if (response.column !== undefined) {
- location += `:${response.column}`
- }
- }
- lines.push(`**Location:** ${location}`)
+ if (response.functionName) {
+ lines.push(`**Function Name:** ${response.functionName}`)
+ }
+
+ if (response.filename) {
+ lines.push(`**File:** ${response.filename}`)
}
- if (response.source) {
- lines.push('')
- lines.push('**Source:**')
- lines.push('```typescript')
- lines.push(response.source)
- lines.push('```')
+ if (response.runtime) {
+ lines.push(`**Runtime:** ${response.runtime}`)
}
return {
Analysis
ServerActionResponse interface uses incorrect field names causing missing output
What fails: The get_server_action_by_id tool in packages/next-devtools-mcp-server/src/tools/get-server-action-by-id.ts defines interface fields (name, file, line, column, source) that don't match the actual API response from getServerActionInfo() in packages/next/src/server/lib/devtools-api-utils/tools/get-server-action-by-id.ts
How to reproduce: The API returns ServerActionInfo with fields: id, runtime, filename, functionName, layer, workers, error
The MCP server checks if (response.name) and if (response.file) which are always undefined, causing only the action ID to be displayed
Result: Tool output only shows **Server Action ID:** <id> and omits function name, file location, runtime, and error information
Expected: Tool should display function name (via response.functionName), file path (via response.filename), runtime (via response.runtime), and error messages (via response.error) when available, matching the actual structure returned by getServerActionInfo()
gaojude
force-pushed
the
jude/devtools-api-and-mcp
branch
2 times, most recently
from
b908549 to
d8798db
Compare
gaojude
force-pushed
the
jude/devtools-api-and-mcp
branch
from
d8798db to
cfc26b0
Compare
gaojude
commented
Oct 6, 2025
| *No segments found* | ||
| ---" |
There was a problem hiding this comment.
Add E2E for server action tools
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.