@next/eslint-plugin-next 16.3.0-canary.2 → 16.3.0-canary.3

package/dist/index.d.ts

@@ -18,6 +18,7 @@ declare const plugin: {
         'no-head-import-in-document': Rule.RuleModule;
         'no-html-link-for-pages': Rule.RuleModule;
         'no-img-element': Rule.RuleModule;
+        'no-location-assign-relative-destination': Rule.RuleModule;
         'no-page-custom-font': Rule.RuleModule;
         'no-script-component-in-head': Rule.RuleModule;
         'no-styled-jsx-in-document': Rule.RuleModule;
@@ -50,6 +51,7 @@ export declare const rules: {
     'no-head-import-in-document': Rule.RuleModule;
     'no-html-link-for-pages': Rule.RuleModule;
     'no-img-element': Rule.RuleModule;
+    'no-location-assign-relative-destination': Rule.RuleModule;
     'no-page-custom-font': Rule.RuleModule;
     'no-script-component-in-head': Rule.RuleModule;
     'no-styled-jsx-in-document': Rule.RuleModule;

package/dist/index.js

@@ -33,6 +33,7 @@ var _noheadelement = /*#__PURE__*/ _interop_require_default(require("./rules/no-
 var _noheadimportindocument = /*#__PURE__*/ _interop_require_default(require("./rules/no-head-import-in-document"));
 var _nohtmllinkforpages = /*#__PURE__*/ _interop_require_default(require("./rules/no-html-link-for-pages"));
 var _noimgelement = /*#__PURE__*/ _interop_require_default(require("./rules/no-img-element"));
+var _nolocationassignrelativedestination = /*#__PURE__*/ _interop_require_default(require("./rules/no-location-assign-relative-destination"));
 var _nopagecustomfont = /*#__PURE__*/ _interop_require_default(require("./rules/no-page-custom-font"));
 var _noscriptcomponentinhead = /*#__PURE__*/ _interop_require_default(require("./rules/no-script-component-in-head"));
 var _nostyledjsxindocument = /*#__PURE__*/ _interop_require_default(require("./rules/no-styled-jsx-in-document"));
@@ -84,6 +85,7 @@ var recommendedRules = {
     '@next/next/no-head-element': 'warn',
     '@next/next/no-html-link-for-pages': 'warn',
     '@next/next/no-img-element': 'warn',
+    '@next/next/no-location-assign-relative-destination': 'warn',
     '@next/next/no-page-custom-font': 'warn',
     '@next/next/no-styled-jsx-in-document': 'warn',
     '@next/next/no-sync-scripts': 'warn',
@@ -121,6 +123,7 @@ var plugin = {
         'no-head-import-in-document': _noheadimportindocument.default,
         'no-html-link-for-pages': _nohtmllinkforpages.default,
         'no-img-element': _noimgelement.default,
+        'no-location-assign-relative-destination': _nolocationassignrelativedestination.default,
         'no-page-custom-font': _nopagecustomfont.default,
         'no-script-component-in-head': _noscriptcomponentinhead.default,
         'no-styled-jsx-in-document': _nostyledjsxindocument.default,

package/dist/rules/no-location-assign-relative-destination.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("eslint").Rule.RuleModule;
+export default _default;

package/dist/rules/no-location-assign-relative-destination.js

@@ -0,0 +1,92 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", {
+    value: true
+});
+Object.defineProperty(exports, "default", {
+    enumerable: true,
+    get: function() {
+        return _default;
+    }
+});
+var _definerule = require("../utils/define-rule");
+var url = 'https://nextjs.org/docs/messages/no-location-assign-relative-destination';
+var LOCATION_GLOBALS = new Set([
+    'window',
+    'globalThis'
+]);
+function isLocationObject(node) {
+    // `location`
+    if (node.type === 'Identifier' && node.name === 'location') {
+        return true;
+    }
+    // `window.location` / `globalThis.location` (dot or bracket notation)
+    if (node.type === 'MemberExpression' && node.object.type === 'Identifier' && LOCATION_GLOBALS.has(node.object.name) && isPropertyNamed(node, 'location')) {
+        return true;
+    }
+    return false;
+}
+function isPropertyNamed(memberNode, name) {
+    return memberNode.computed === false && memberNode.property.type === 'Identifier' && memberNode.property.name === name || memberNode.computed === true && memberNode.property.type === 'Literal' && memberNode.property.value === name;
+}
+/** Returns true when the node is a string literal containing "://" (absolute URL). */ function isAbsoluteUrlLiteral(node) {
+    return node != null && node.type === 'Literal' && typeof node.value === 'string' && node.value.includes('://');
+}
+var _default = (0, _definerule.defineRule)({
+    meta: {
+        docs: {
+            description: 'Prevent usage of `location.assign` or `location.href` assignment to navigate to internal Next.js pages.',
+            recommended: true,
+            url: url
+        },
+        type: 'problem',
+        schema: [],
+        messages: {
+            noLocationAssign: "Do not use `{{expression}}` to navigate to internal Next.js pages. Use `redirect()` in the render phase, or `useRouter().push()` in Client Components' event handlers instead. See: " + url
+        }
+    },
+    create: function create(context) {
+        var sourceCode = context.sourceCode;
+        return {
+            // location.assign(...) / location['assign'](...)
+            // window.location.assign(...) / window.location['assign'](...)
+            // globalThis.location.assign(...) / globalThis.location['assign'](...)
+            CallExpression: function CallExpression(node) {
+                var callee = node.callee;
+                if (callee.type === 'MemberExpression' && isPropertyNamed(callee, 'assign') && isLocationObject(callee.object)) {
+                    // Allow calls where the first argument is an absolute URL string literal
+                    if (isAbsoluteUrlLiteral(node.arguments[0])) {
+                        return;
+                    }
+                    var expression = sourceCode.getText(callee);
+                    context.report({
+                        node: node,
+                        messageId: 'noLocationAssign',
+                        data: {
+                            expression: expression + '()'
+                        }
+                    });
+                }
+            },
+            // location.href = ... / location['href'] = ...
+            // window.location.href = ... / window.location['href'] = ...
+            // globalThis.location.href = ... / globalThis.location['href'] = ...
+            AssignmentExpression: function AssignmentExpression(node) {
+                var left = node.left;
+                if (left.type === 'MemberExpression' && isPropertyNamed(left, 'href') && isLocationObject(left.object)) {
+                    // Allow assignments where the right-hand side is an absolute URL string literal
+                    if (isAbsoluteUrlLiteral(node.right)) {
+                        return;
+                    }
+                    var expression = sourceCode.getText(left);
+                    context.report({
+                        node: node,
+                        messageId: 'noLocationAssign',
+                        data: {
+                            expression: expression
+                        }
+                    });
+                }
+            }
+        };
+    }
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@next/eslint-plugin-next",
-  "version": "16.3.0-canary.2",
+  "version": "16.3.0-canary.3",
   "description": "ESLint plugin for Next.js.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -25,4 +25,4 @@
     "types": "tsc --project tsconfig.json --skipLibCheck --declaration --emitDeclarationOnly --esModuleInterop --declarationDir dist",
     "prepublishOnly": "cd ../../ && turbo run build"
   }
-}
+}