@eslint/json 1.1.0 → 2.0.0

package/README.md

@@ -296,7 +296,7 @@ to get your logo on our READMEs and [website](https://eslint.org/sponsors).
 <p><a href="https://automattic.com"><img src="https://images.opencollective.com/automattic/d0ef3e1/logo.png" alt="Automattic" height="128"></a></p><h3>Gold Sponsors</h3>
 <p><a href="https://qlty.sh/"><img src="https://images.opencollective.com/qltysh/33d157d/logo.png" alt="Qlty Software" height="96"></a></p><h3>Silver Sponsors</h3>
 <p><a href="https://vite.dev/"><img src="https://images.opencollective.com/vite/d472863/logo.png" alt="Vite" height="64"></a> <a href="https://liftoff.io/"><img src="https://images.opencollective.com/liftoff/2d6c3b6/logo.png" alt="Liftoff" height="64"></a> <a href="https://stackblitz.com"><img src="https://avatars.githubusercontent.com/u/28635252" alt="StackBlitz" height="64"></a></p><h3>Bronze Sponsors</h3>
-<p><a href="https://cybozu.co.jp/"><img src="https://images.opencollective.com/cybozu/933e46d/logo.png" alt="Cybozu" height="32"></a> <a href="https://opensource.sap.com"><img src="https://avatars.githubusercontent.com/u/2531208" alt="SAP" height="32"></a> <a href="https://www.crawljobs.com/"><img src="https://images.opencollective.com/crawljobs-poland/fa43a17/logo.png" alt="CrawlJobs" height="32"></a> <a href="https://depot.dev"><img src="https://images.opencollective.com/depot/39125a1/logo.png" alt="Depot" height="32"></a> <a href="https://www.n-ix.com/"><img src="https://images.opencollective.com/n-ix-ltd/575a7a5/logo.png" alt="N-iX Ltd" height="32"></a> <a href="https://icons8.com/"><img src="https://images.opencollective.com/icons8/7fa1641/logo.png" alt="Icons8" height="32"></a> <a href="https://discord.com"><img src="https://images.opencollective.com/discordapp/f9645d9/logo.png" alt="Discord" height="32"></a> <a href="https://www.gitbook.com"><img src="https://avatars.githubusercontent.com/u/7111340" alt="GitBook" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://www.lambdatest.com"><img src="https://avatars.githubusercontent.com/u/171592363" alt="TestMu AI Open Source Office (Formerly LambdaTest)" height="32"></a></p>
+<p><a href="https://cybozu.co.jp/"><img src="https://images.opencollective.com/cybozu/933e46d/logo.png" alt="Cybozu" height="32"></a> <a href="https://opensource.sap.com"><img src="https://avatars.githubusercontent.com/u/2531208" alt="SAP" height="32"></a> <a href="https://www.crawljobs.com/"><img src="https://images.opencollective.com/crawljobs-poland/fa43a17/logo.png" alt="CrawlJobs" height="32"></a> <a href="https://syntax.fm"><img src="https://github.com/syntaxfm.png" alt="Syntax" height="32"></a> <a href="https://depot.dev"><img src="https://images.opencollective.com/depot/39125a1/logo.png" alt="Depot" height="32"></a> <a href="https://icons8.com/"><img src="https://images.opencollective.com/icons8/7fa1641/logo.png" alt="Icons8" height="32"></a> <a href="https://discord.com"><img src="https://images.opencollective.com/discordapp/f9645d9/logo.png" alt="Discord" height="32"></a> <a href="https://www.gitbook.com"><img src="https://avatars.githubusercontent.com/u/7111340" alt="GitBook" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://citadel.co.jp"><img src="https://avatars.githubusercontent.com/u/75781367" alt="Citadel AI" height="32"></a></p>
 <h3>Technology Sponsors</h3>
 Technology sponsors allow us to use their products and services for free as part of a contribution to the open source ecosystem and our work.
 <p><a href="https://netlify.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/netlify-icon.svg" alt="Netlify" height="32"></a> <a href="https://algolia.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/algolia-icon.svg" alt="Algolia" height="32"></a> <a href="https://1password.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/1password-icon.svg" alt="1Password" height="32"></a></p>

package/dist/index.d.ts

@@ -5,6 +5,7 @@ export * from "./types.js";
 declare namespace plugin {
     export namespace meta {
         let name: string;
+        let namespace: string;
         let version: string;
     }
     export namespace languages {

package/dist/index.js

@@ -15,7 +15,8 @@ import rules from "./build/rules.js";
 const plugin = {
     meta: {
         name: "@eslint/json",
-        version: "1.1.0", // x-release-please-version
+        namespace: "json",
+        version: "2.0.0", // x-release-please-version
     },
     languages: {
         json: new JSONLanguage({ mode: "json" }),
@@ -31,10 +32,7 @@ const plugin = {
         },
     },
 };
-// eslint-disable-next-line no-lone-blocks -- The block syntax { ... } ensures that TypeScript does not get confused about the type of `plugin`.
-{
-    plugin.configs.recommended.plugins.json = plugin;
-}
+Object.assign(plugin.configs.recommended.plugins, { json: plugin });
 export default plugin;
 export { JSONSourceCode };
 export * from "./languages/json-language.js";

package/dist/rules/no-duplicate-keys.js

@@ -22,9 +22,11 @@ import { getKey, getRawKey } from "../util.js";
 const rule = {
     meta: {
         type: "problem",
+        languages: ["json/json", "json/jsonc", "json/json5"],
         docs: {
             recommended: true,
             description: "Disallow duplicate keys in JSON objects",
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/no-duplicate-keys.md",
         },
         messages: {

package/dist/rules/no-empty-keys.js

@@ -21,9 +21,11 @@ import { getKey } from "../util.js";
 const rule = {
     meta: {
         type: "problem",
+        languages: ["json/json", "json/jsonc", "json/json5"],
         docs: {
             recommended: true,
             description: "Disallow empty keys in JSON objects",
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/no-empty-keys.md",
         },
         messages: {

package/dist/rules/no-unnormalized-keys.js

@@ -22,10 +22,12 @@ import { getKey, getRawKey } from "../util.js";
 const rule = {
     meta: {
         type: "problem",
+        languages: ["json/json", "json/jsonc", "json/json5"],
         fixable: "code",
         docs: {
             recommended: true,
             description: "Disallow JSON keys that are not normalized",
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/no-unnormalized-keys.md",
         },
         messages: {

package/dist/rules/no-unsafe-values.js

@@ -32,9 +32,11 @@ const NON_ZERO = /[1-9]/u;
 const rule = {
     meta: {
         type: "problem",
+        languages: ["json/json", "json/jsonc", "json/json5"],
         docs: {
             recommended: true,
             description: "Disallow JSON values that are unsafe for interchange",
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/no-unsafe-values.md",
         },
         messages: {

package/dist/rules/sort-keys.js

@@ -33,6 +33,7 @@ import { getKey, getRawKey } from "../util.js";
 // Helpers
 //-----------------------------------------------------------------------------
 const hasNonWhitespace = /\S/u;
+const commentTypes = new Set(["LineComment", "BlockComment"]);
 /** @type {ComparatorMap} */
 const comparators = {
     ascending: {
@@ -63,6 +64,8 @@ const comparators = {
 const rule = {
     meta: {
         type: "suggestion",
+        languages: ["json/json", "json/jsonc", "json/json5"],
+        fixable: "code",
         defaultOptions: [
             "asc",
             {
@@ -75,6 +78,7 @@ const rule = {
         docs: {
             recommended: false,
             description: `Require JSON object keys to be sorted`,
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/sort-keys.md",
         },
         messages: {
@@ -144,6 +148,25 @@ const rule = {
             }
             return false;
         }
+        /**
+         * Checks if a member has a comment before or after it.
+         * @param {MemberNode} member The member to check.
+         * @returns {boolean} True if a comment is adjacent to the member.
+         */
+        function hasAdjacentComment(member) {
+            const before = sourceCode.getTokenBefore(member, {
+                includeComments: true,
+            });
+            let after = sourceCode.getTokenAfter(member, {
+                includeComments: true,
+            });
+            if (after.type === "Comma") {
+                after = sourceCode.getTokenAfter(after, {
+                    includeComments: true,
+                });
+            }
+            return (commentTypes.has(before.type) || commentTypes.has(after.type));
+        }
         return {
             Object(node) {
                 /** @type {MemberNode} */
@@ -158,6 +181,9 @@ const rule = {
                 for (const member of node.members) {
                     const thisName = getKey(member);
                     const thisRawName = getRawKey(member, sourceCode);
+                    // Capture `prevMember` for this iteration so the fixer closure uses the
+                    // intended node even though `prevMember` is reassigned in the loop.
+                    const prevMemberNode = prevMember;
                     if (prevMember &&
                         !isValidOrder(prevName, thisName) &&
                         (!allowLineSeparatedGroups ||
@@ -172,6 +198,16 @@ const rule = {
                                 sensitivity,
                                 sortName,
                             },
+                            fix(fixer) {
+                                if (hasAdjacentComment(member) ||
+                                    hasAdjacentComment(prevMemberNode)) {
+                                    return null;
+                                }
+                                return [
+                                    fixer.replaceText(member, sourceCode.getText(prevMemberNode)),
+                                    fixer.replaceText(prevMemberNode, sourceCode.getText(member)),
+                                ];
+                            },
                         });
                     }
                     prevMember = member;

package/dist/rules/top-level-interop.js

@@ -17,9 +17,11 @@
 const rule = {
     meta: {
         type: "problem",
+        languages: ["json/json", "json/jsonc", "json/json5"],
         docs: {
             recommended: false,
             description: "Require the JSON top-level value to be an array or object",
+            dialects: ["JSON", "JSONC", "JSON5"],
             url: "https://github.com/eslint/json/tree/main/docs/rules/top-level-interop.md",
         },
         messages: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint/json",
-  "version": "1.1.0",
+  "version": "2.0.0",
   "description": "JSON linting plugin for ESLint",
   "author": "Nicholas C. Zakas",
   "type": "module",
@@ -50,12 +50,17 @@
     "lint": "eslint",
     "lint:fix": "eslint --fix",
     "lint:types": "attw --pack --profile esm-only",
+    "lint:unused": "knip",
     "fmt": "prettier --write .",
     "fmt:check": "prettier --check .",
     "test": "mocha \"tests/**/*.test.js\"",
     "test:coverage": "c8 npm test",
     "test:jsr": "npx -y jsr@latest publish --dry-run",
-    "test:types": "tsc -p tests/types/tsconfig.json"
+    "test:types": "tsc -p tests/types/tsconfig.json",
+    "test:types:5.3": "npx -p typescript@5.3 -y -- tsc -p tests/types/tsconfig.legacy.json",
+    "test:types:5.x": "npx -p typescript@5.x -y -- tsc -p tests/types/tsconfig.json",
+    "test:types:7.x": "npx -p @typescript/native-preview@latest -y -- tsgo -p tests/types/tsconfig.json",
+    "test:types:all": "npm run test:types && npm run test:types:5.3 && npm run test:types:5.x && npm run test:types:7.x"
   },
   "keywords": [
     "eslint",
@@ -66,26 +71,34 @@
   ],
   "license": "Apache-2.0",
   "dependencies": {
-    "@eslint/core": "^1.1.1",
-    "@eslint/plugin-kit": "^0.6.1",
+    "@eslint/core": "^1.2.1",
+    "@eslint/plugin-kit": "^0.7.1",
     "@humanwhocodes/momoa": "^3.3.10",
     "natural-compare": "^1.4.0"
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.18.2",
+    "@types/natural-compare": "^1.4.3",
+    "@types/node": "^20.19.0",
     "c8": "^11.0.0",
     "dedent": "^1.5.3",
-    "eslint": "^10.0.0",
+    "eslint": "^10.4.0",
     "eslint-config-eslint": "^14.0.0",
     "eslint-plugin-eslint-plugin": "^7.3.2",
     "globals": "^17.0.0",
+    "knip": "^6.0.0",
     "lint-staged": "^16.0.0",
     "mdast-util-from-markdown": "^2.0.2",
     "mocha": "^11.3.0",
-    "prettier": "3.8.1",
-    "typescript": "^5.9.2",
+    "prettier": "3.8.3",
+    "typescript": "^6.0.2",
     "yorkie": "^2.0.0"
   },
+  "overrides": {
+    "@arethetypeswrong/core": {
+      "fflate": "0.8.2"
+    }
+  },
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24"
   }