@next/eslint-plugin-next 15.2.0-canary.2 → 15.2.0-canary.20

package/dist/rules/no-html-link-for-pages.js

@@ -109,6 +109,21 @@ var pagesDirWarning = (0, _url.execOnce)(function(pagesDirs) {
 // Cache for fs.existsSync lookup.
 // Prevent multiple blocking IO requests that have already been calculated.
 var fsExistsSyncCache = {};
+var memoize = function(fn) {
+    var cache = {};
+    return function() {
+        for(var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++){
+            args[_key] = arguments[_key];
+        }
+        var key = JSON.stringify(args);
+        if (cache[key] === undefined) {
+            cache[key] = fn.apply(void 0, _to_consumable_array(args));
+        }
+        return cache[key];
+    };
+};
+var cachedGetUrlFromPagesDirectories = memoize(_url.getUrlFromPagesDirectories);
+var cachedGetUrlFromAppDirectory = memoize(_url.getUrlFromAppDirectory);
 var url = 'https://nextjs.org/docs/messages/no-html-link-for-pages';
 module.exports = (0, _definerule.defineRule)({
     meta: {
@@ -173,9 +188,9 @@ module.exports = (0, _definerule.defineRule)({
             pagesDirWarning(pagesDirs);
             return {};
         }
-        var pageUrls = (0, _url.getUrlFromPagesDirectories)('/', foundPagesDirs);
-        var appDirUrls = (0, _url.getUrlFromAppDirectory)('/', foundAppDirs);
-        var allUrls = _to_consumable_array(pageUrls).concat(_to_consumable_array(appDirUrls));
+        var pageUrls = cachedGetUrlFromPagesDirectories('/', foundPagesDirs);
+        var appDirUrls = cachedGetUrlFromAppDirectory('/', foundAppDirs);
+        var allUrlRegex = _to_consumable_array(pageUrls).concat(_to_consumable_array(appDirUrls));
         return {
             JSXOpeningElement: function JSXOpeningElement(node) {
                 if (node.name.name !== 'a') {
@@ -207,7 +222,7 @@ module.exports = (0, _definerule.defineRule)({
                 if (/^(https?:\/\/|\/\/)/.test(hrefPath)) {
                     return;
                 }
-                allUrls.forEach(function(foundUrl) {
+                allUrlRegex.forEach(function(foundUrl) {
                     if (foundUrl.test((0, _url.normalizeURL)(hrefPath))) {
                         context.report({
                             node: node,

package/dist/rules/no-img-element.js

@@ -1,5 +1,6 @@
 "use strict";
 var _definerule = require("../utils/define-rule");
+var path = require("path");
 var url = 'https://nextjs.org/docs/messages/no-img-element';
 module.exports = (0, _definerule.defineRule)({
     meta: {
@@ -13,6 +14,9 @@ module.exports = (0, _definerule.defineRule)({
         schema: []
     },
     create: function create(context) {
+        // Get relative path of the file
+        var relativePath = context.filename.replace(path.sep, '/').replace(context.cwd, '').replace(/^\//, '');
+        var isAppDir = /^(src\/)?app\//.test(relativePath);
         return {
             JSXOpeningElement: function JSXOpeningElement(node) {
                 var _node_parent_parent_openingElement_name, _node_parent_parent_openingElement, _node_parent_parent, _node_parent;
@@ -25,6 +29,9 @@ module.exports = (0, _definerule.defineRule)({
                 if (((_node_parent = node.parent) === null || _node_parent === void 0 ? void 0 : (_node_parent_parent = _node_parent.parent) === null || _node_parent_parent === void 0 ? void 0 : (_node_parent_parent_openingElement = _node_parent_parent.openingElement) === null || _node_parent_parent_openingElement === void 0 ? void 0 : (_node_parent_parent_openingElement_name = _node_parent_parent_openingElement.name) === null || _node_parent_parent_openingElement_name === void 0 ? void 0 : _node_parent_parent_openingElement_name.name) === 'picture') {
                     return;
                 }
+                // If is metadata route files, ignore
+                // e.g. opengraph-image.js, twitter-image.js, icon.js
+                if (isAppDir && /\/opengraph-image|twitter-image|icon\.\w+$/.test(relativePath)) return;
                 context.report({
                     node: node,
                     message: "Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` or a custom image loader to automatically optimize images. This may incur additional usage or cost from your provider. See: ".concat(url)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@next/eslint-plugin-next",
-  "version": "15.2.0-canary.2",
+  "version": "15.2.0-canary.20",
   "description": "ESLint plugin for Next.js.",
   "main": "dist/index.js",
   "license": "MIT",