eslint-plugin-package-json 1.7.1 → 1.8.1

package/README.md

@@ -46,6 +46,7 @@ See [Getting Started](https://eslint-plugin-package-json.dev/getting-started) fo
 | [no-redundant-files](https://eslint-plugin-package-json.dev/rules/no-redundant-files)                                           | Prevents adding unnecessary / redundant files.                                                                                                                                               | ✅  |     | 💡  |
 | [no-redundant-publishConfig](https://eslint-plugin-package-json.dev/rules/no-redundant-publishConfig)                           | Warns when publishConfig.access is used in unscoped packages.                                                                                                                                | ✅  |     | 💡  |
 | [order-properties](https://eslint-plugin-package-json.dev/rules/order-properties)                                               | Enforces that package properties are declared in a consistent order.                                                                                                                         | 🎨  | 🔧  |     |
+| [prefer-rolling-workspace-spec](https://eslint-plugin-package-json.dev/rules/prefer-rolling-workspace-spec)                     | Require that dependencies declared with workspace protocol use a rolling workspace spec instead of a specific semver range.                                                                  |     |     | 💡  |
 | [repository-shorthand](https://eslint-plugin-package-json.dev/rules/repository-shorthand)                                       | Enforce either object or shorthand declaration for repository.                                                                                                                               | ✅  | 🔧  |     |
 | [require-attribution](https://eslint-plugin-package-json.dev/rules/require-attribution)                                         | Ensures that proper attribution is included, requiring that either `author` or `contributors` is defined, and that if `contributors` is present, it should include at least one contributor. | ✅  |     | 💡  |
 | [restrict-dependency-ranges](https://eslint-plugin-package-json.dev/rules/restrict-dependency-ranges)                           | Restricts the range of dependencies to allow or disallow specific types of ranges.                                                                                                           |     |     | 💡  |

package/dist/{createRule-29pC9RoP.d.mts → createRule-CSNxd7i2.d.mts}

@@ -1,7 +1,6 @@
 import { AST, RuleListener } from "jsonc-eslint-parser";
 import { AST as AST$1, Rule, SourceCode } from "eslint";
 import * as ESTree from "estree";
-
 //#region node_modules/.pnpm/json-schema-to-ts@3.1.1/node_modules/json-schema-to-ts/lib/types/definitions/jsonSchema.d.ts
 declare const $JSONSchema: unique symbol;
 type $JSONSchema = typeof $JSONSchema;
@@ -92,7 +91,8 @@ interface PackageJsonRuleModule<Options extends unknown[] = unknown[], Schema ex
   meta: Omit<Rule.RuleMetaData, 'defaultOptions' | 'docs' | 'schema'> & {
     defaultOptions?: NoInfer<Options>;
     docs?: Rule.RuleMetaData['docs'] & {
-      category?: string; /** An optional route segment to group this rule under */
+      category?: string;
+      /** An optional route segment to group this rule under */
       ruleGroup?: string;
     };
     schema?: Schema;

package/dist/experimental/index.d.mts

@@ -1,6 +1,5 @@
-import { n as PackageJsonRuleModule, r as JSONSchema, t as PackageJsonPluginSettings } from "../createRule-29pC9RoP.mjs";
-import { ESLint } from "eslint";
-
+import { n as PackageJsonRuleModule, r as JSONSchema, t as PackageJsonPluginSettings } from "../createRule-CSNxd7i2.mjs";
+import { ESLint as ESLint$1 } from "eslint";
 //#region src/experimental/plugin.d.ts
 declare const plugin: {
   configs: {
@@ -9,7 +8,7 @@ declare const plugin: {
       language: string;
       name: string;
       plugins: {
-        readonly 'package-json': ESLint.Plugin;
+        readonly 'package-json': ESLint$1.Plugin;
       };
       rules: {
         [k: string]: "error";
@@ -20,7 +19,7 @@ declare const plugin: {
       language: string;
       name: string;
       plugins: {
-        readonly 'package-json': ESLint.Plugin;
+        readonly 'package-json': ESLint$1.Plugin;
       };
       rules: {
         [k: string]: "error";
@@ -37,7 +36,7 @@ declare const plugin: {
 };
 //#endregion
 //#region src/experimental/index.d.ts
-declare const configs: {
+export declare const configs: {
   recommended: {
     files: string[];
     language: string;
@@ -61,8 +60,8 @@ declare const configs: {
     };
   };
 };
-declare const rules: {
+export declare const rules: {
   [k: string]: PackageJsonRuleModule<unknown[], JSONSchema[]>;
 };
 //#endregion
-export { type PackageJsonPluginSettings, configs, plugin as default, rules };
+export { type PackageJsonPluginSettings, plugin as default };

package/dist/experimental/index.mjs

@@ -1,4 +1,4 @@
-import { t as plugin$1 } from "../plugin-BG90Pf7y.mjs";
+import { t as plugin$1 } from "../plugin-DlF-r5D9.mjs";
 import { toCompatRule } from "eslint-json-compat-utils";
 //#region src/experimental/plugin.ts
 const language = "json/json";

package/dist/index.d.mts

@@ -1,7 +1,6 @@
-import { n as PackageJsonRuleModule, r as JSONSchema, t as PackageJsonPluginSettings } from "./createRule-29pC9RoP.mjs";
+import { n as PackageJsonRuleModule, r as JSONSchema, t as PackageJsonPluginSettings } from "./createRule-CSNxd7i2.mjs";
 import * as parserJsonc from "jsonc-eslint-parser";
-import { ESLint } from "eslint";
-
+import { ESLint as ESLint$1 } from "eslint";
 //#region src/plugin.d.ts
 declare const plugin: {
   configs: {
@@ -12,12 +11,13 @@ declare const plugin: {
       };
       name: string;
       plugins: {
-        readonly 'package-json': ESLint.Plugin;
+        readonly 'package-json': ESLint$1.Plugin;
       };
       rules: {
         [k: string]: "error";
       };
-    }; /** @deprecated use the recommended config instead. this config will be removed in a future major release */
+    };
+    /** @deprecated use the recommended config instead. this config will be removed in a future major release */
     'recommended-publishable': {
       files: string[];
       languageOptions: {
@@ -25,7 +25,7 @@ declare const plugin: {
       };
       name: string;
       plugins: {
-        readonly 'package-json': ESLint.Plugin;
+        readonly 'package-json': ESLint$1.Plugin;
       };
       rules: {
         [x: string]: "error";
@@ -38,7 +38,7 @@ declare const plugin: {
       };
       name: string;
       plugins: {
-        readonly 'package-json': ESLint.Plugin;
+        readonly 'package-json': ESLint$1.Plugin;
       };
       rules: {
         [k: string]: "error";
@@ -53,8 +53,8 @@ declare const plugin: {
 };
 //#endregion
 //#region src/index.d.ts
-declare const rules: Record<string, PackageJsonRuleModule<unknown[], JSONSchema[]>>;
-declare const configs: {
+export declare const rules: Record<string, PackageJsonRuleModule<unknown[], JSONSchema[]>>;
+export declare const configs: {
   recommended: {
     files: string[];
     languageOptions: {
@@ -96,4 +96,4 @@ declare const configs: {
   };
 };
 //#endregion
-export { type PackageJsonPluginSettings, configs, plugin as default, rules };
+export { type PackageJsonPluginSettings, plugin as default };

package/dist/index.mjs

@@ -1,4 +1,4 @@
-import { t as plugin } from "./plugin-BG90Pf7y.mjs";
+import { t as plugin } from "./plugin-DlF-r5D9.mjs";
 //#region src/index.ts
 const rules = plugin.rules;
 const configs = plugin.configs;

package/dist/{plugin-BG90Pf7y.mjs → plugin-DlF-r5D9.mjs}

@@ -6,14 +6,19 @@ import detectIndent from "detect-indent";
 import { detectNewlineGraceful } from "detect-newline";
 import sortObjectKeys from "sort-object-keys";
 import sortPackageJson, { sortOrder } from "sort-package-json";
-import { execFileSync } from "node:child_process";
 import semver from "semver";
+import { execFileSync } from "node:child_process";
 import { validateAuthor, validateBin, validateBrowser, validateBugs, validateBundleDependencies, validateConfig, validateContributors, validateCpu, validateDependencies, validateDescription, validateDevEngines, validateDirectories, validateEngines, validateExports, validateFiles, validateFunding, validateGypfile, validateHomepage, validateKeywords, validateLibc, validateLicense, validateMain, validateMan, validateName, validateOs, validatePackageManager, validatePeerDependenciesMeta, validatePrivate, validatePublishConfig, validateRepository, validateScripts, validateSideEffects, validateType, validateVersion, validateWorkspaces } from "package-json-validator";
 import fs from "node:fs";
 import path from "node:path";
 import process from "node:process";
 import { sep } from "node:path/posix";
 import { findRootSync } from "@altano/repository-tools";
+//#region src/utils/predicates/isJSONNullLiteral.ts
+function isJSONNullLiteral(node) {
+	return node.type === "JSONLiteral" && node.value === null;
+}
+//#endregion
 //#region src/utils/predicates/isJSONStringLiteral.ts
 function isJSONStringLiteral(node) {
 	return node.type === "JSONLiteral" && typeof node.value === "string";
@@ -50,7 +55,7 @@ function createRule(rule) {
 }
 //#endregion
 //#region src/rules/bin-name-casing.ts
-const rule$18 = createRule({
+const rule$19 = createRule({
 	create(context) {
 		return { "Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=bin]"(node) {
 			if (node.value.type === "JSONObjectExpression") for (const property of node.value.properties) {
@@ -92,11 +97,12 @@ function isImplicitFormat(node) {
 	if (node.type === "JSONLiteral") return true;
 	return node.properties.every((property) => !isJSONStringLiteral(property.key) || !property.key.value.startsWith("."));
 }
-const rule$17 = createRule({
+const rule$18 = createRule({
 	create(context) {
 		const [{ prefer = "explicit" } = {}] = context.options;
 		function validateForExplicit(node) {
 			const { value } = node;
+			if (isJSONNullLiteral(value)) return;
 			if (value.type !== "JSONLiteral" && value.type !== "JSONObjectExpression" || !isImplicitFormat(value)) return;
 			context.report({
 				fix(fixer) {
@@ -113,6 +119,7 @@ const rule$17 = createRule({
 			if (value.type !== "JSONObjectExpression") return;
 			if (value.properties.length !== 1 || !isJSONStringLiteral(value.properties[0].key) || value.properties[0].key.value !== ".") return;
 			const dotProperty = value.properties[0];
+			if (isJSONNullLiteral(dotProperty.value)) return;
 			context.report({
 				fix(fixer) {
 					const valueText = context.sourceCode.getText(dotProperty.value);
@@ -193,7 +200,7 @@ const getTopLevelProperty = (node) => {
 	return n.type === "JSONProperty" ? n.key : void 0;
 };
 const defaultIgnoreProperties = ["files"];
-const rule$16 = createRule({
+const rule$17 = createRule({
 	create(context) {
 		const ignoreProperties = new Set(context.options[0]?.ignoreProperties ?? defaultIgnoreProperties);
 		return {
@@ -249,7 +256,7 @@ const getBundledDependencyNames = (value) => {
 	if (value?.type === "JSONArrayExpression") for (const element of value.elements.filter(isNotNullish).filter(isJSONStringLiteral)) names.add(element.value);
 	return names;
 };
-const rule$15 = createRule({
+const rule$16 = createRule({
 	create(context) {
 		const ignorePrivate = context.options[0]?.ignorePrivate ?? true;
 		let isPrivate = false;
@@ -320,7 +327,7 @@ const getCachedLocalFileRegex = (filename) => {
 		return regex;
 	}
 };
-const rule$14 = createRule({
+const rule$15 = createRule({
 	create(context) {
 		const entryCache = {
 			bin: [],
@@ -397,7 +404,7 @@ const rule$14 = createRule({
 });
 //#endregion
 //#region src/rules/no-redundant-publishConfig.ts
-const rule$13 = createRule({
+const rule$14 = createRule({
 	create(context) {
 		let packageName;
 		let publishConfigAccessProperty;
@@ -443,7 +450,7 @@ const rule$13 = createRule({
 });
 //#endregion
 //#region src/rules/order-properties.ts
-const rule$12 = createRule({
+const rule$13 = createRule({
 	create(context) {
 		return { "Program:exit"() {
 			const { ast, text } = context.sourceCode;
@@ -500,6 +507,73 @@ const rule$12 = createRule({
 	name: "order-properties"
 });
 //#endregion
+//#region src/rules/prefer-rolling-workspace-spec.ts
+const rollingWorkspaceSpecRegex = /^workspace:[*^~]?$/;
+const convertibleSpecRegex = /^(workspace:[~^])/;
+const rule$12 = createRule({
+	create(context) {
+		const { ignoreDependencies = [], ignorePatterns = [] } = context.options[0] ?? {};
+		const ignoreRegexes = ignorePatterns.map((pattern) => new RegExp(pattern));
+		const isIgnored = (name) => ignoreDependencies.includes(name) || ignoreRegexes.some((regex) => regex.test(name));
+		return { "Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.type=JSONLiteral]:matches([key.value=dependencies], [key.value=devDependencies])"(node) {
+			if (node.value.type === "JSONObjectExpression") for (const property of node.value.properties) {
+				const key = property.key.value;
+				if (isIgnored(key)) continue;
+				const valueNode = property.value;
+				if (!isJSONStringLiteral(valueNode) || !valueNode.value.startsWith("workspace:")) continue;
+				const dependencySpec = valueNode.value;
+				if (!rollingWorkspaceSpecRegex.test(dependencySpec)) {
+					let conversion;
+					const convertibleMatch = convertibleSpecRegex.exec(dependencySpec);
+					const rawVersion = dependencySpec.replace("workspace:", "");
+					if (convertibleMatch) conversion = convertibleMatch[1];
+					else if (semver.parse(rawVersion)) conversion = "workspace:*";
+					context.report({
+						messageId: "nonRollingWorkspaceSpec",
+						node: valueNode,
+						suggest: conversion ? [{
+							fix: (fixer) => {
+								return fixer.replaceText(valueNode, JSON.stringify(conversion));
+							},
+							messageId: "convertToRolling"
+						}] : void 0
+					});
+				}
+			}
+		} };
+	},
+	meta: {
+		defaultOptions: [{}],
+		docs: {
+			category: "Best Practices",
+			description: "Require that dependencies declared with workspace protocol use a rolling workspace spec instead of a specific semver range."
+		},
+		hasSuggestions: true,
+		messages: {
+			convertToRolling: "Convert to rolling workspace spec",
+			nonRollingWorkspaceSpec: "Workspace spec should not include specific semver range. Use a rolling workspace spec instead."
+		},
+		schema: [{
+			additionalProperties: false,
+			properties: {
+				ignoreDependencies: {
+					items: { type: ["string"] },
+					type: ["array"],
+					description: "Specific dependencies to ignore."
+				},
+				ignorePatterns: {
+					items: { type: ["string"] },
+					type: ["array"],
+					description: "Regex patterns for dependency names to ignore."
+				}
+			},
+			type: "object"
+		}],
+		type: "suggestion"
+	},
+	name: "prefer-rolling-workspace-spec"
+});
+//#endregion
 //#region src/utils/findPropertyWithKeyValue.ts
 function findPropertyWithKeyValue(properties, value) {
 	return properties.find((property) => property.key.type === "JSONLiteral" && property.key.value === value);
@@ -1239,7 +1313,8 @@ const rule$5 = createRule({
 			let naturalCompare;
 			if (isScripts) {
 				const scriptsSource = context.sourceCode.getText(node);
-				const { scripts: sortedScripts } = sortPackageJson(JSON.parse(`{${scriptsSource}}`));
+				const minimalJson = JSON.parse(`{${scriptsSource}}`);
+				const { scripts: sortedScripts } = sortPackageJson(minimalJson);
 				const lifecycleIndex = new Map(Object.keys(sortedScripts).map((k, i) => [k, i]));
 				naturalCompare = (a, b) => (lifecycleIndex.get(a) ?? 0) - (lifecycleIndex.get(b) ?? 0);
 			} else naturalCompare = (a, b) => a > b ? 1 : -1;
@@ -1494,7 +1569,8 @@ const rule$2 = createRule({
 		};
 		return { "Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=author]"(node) {
 			const valueNode = node.value;
-			const result = validateAuthor(JSON.parse(context.sourceCode.getText(valueNode)));
+			const value = JSON.parse(context.sourceCode.getText(valueNode));
+			const result = validateAuthor(value);
 			reportIssues(result, valueNode, "author");
 		} };
 	},
@@ -1864,13 +1940,14 @@ const rule = createRule({
 //#region src/plugin.ts
 const require = createRequire(import.meta.url);
 const rules = {
-	"bin-name-casing": rule$18,
-	"exports-subpaths-style": rule$17,
-	"no-empty-fields": rule$16,
-	"no-local-dependencies": rule$15,
-	"no-redundant-files": rule$14,
-	"no-redundant-publishConfig": rule$13,
-	"order-properties": rule$12,
+	"bin-name-casing": rule$19,
+	"exports-subpaths-style": rule$18,
+	"no-empty-fields": rule$17,
+	"no-local-dependencies": rule$16,
+	"no-redundant-files": rule$15,
+	"no-redundant-publishConfig": rule$14,
+	"order-properties": rule$13,
+	"prefer-rolling-workspace-spec": rule$12,
 	"require-attribution": rule$10,
 	...rules$2,
 	"repository-shorthand": rule$11,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-package-json",
-  "version": "1.7.1",
+  "version": "1.8.1",
   "description": "Rules for consistent, readable, and valid package.json files. 🗂️",
   "homepage": "https://github.com/michaelfaith/eslint-plugin-package-json#readme",
   "bugs": {
@@ -57,50 +57,51 @@
     "sort-package-json": "^4.0.0"
   },
   "devDependencies": {
-    "@astrojs/check": "0.9.9",
-    "@astrojs/markdown-satteri": "0.3.3",
-    "@astrojs/starlight": "0.41.3",
+    "@astrojs/check": "0.9.10",
+    "@astrojs/markdown-satteri": "0.4.1",
+    "@astrojs/starlight": "0.42.0",
     "@catppuccin/starlight": "2.1.0",
-    "@eslint-community/eslint-plugin-eslint-comments": "4.7.2",
+    "@eslint-community/eslint-plugin-eslint-comments": "4.8.0",
     "@eslint/js": "10.0.1",
-    "@eslint/json": "2.0.0",
-    "@eslint/markdown": "8.0.1",
+    "@eslint/json": "2.1.0",
+    "@eslint/markdown": "8.0.3",
     "@ianvs/prettier-plugin-sort-imports": "4.7.1",
-    "@types/node": "24.13.0",
+    "@types/node": "24.13.4",
     "@types/semver": "7.8.0",
     "@typescript/native": "npm:typescript@7.0.2",
-    "@vitest/coverage-v8": "4.1.8",
-    "@vitest/eslint-plugin": "1.6.20",
-    "astro": "7.2.0",
-    "astro-og-canvas": "0.13.0",
-    "canvaskit-wasm": "0.41.1",
-    "console-fail-test": "0.6.0",
-    "eslint": "10.8.0",
-    "eslint-doc-generator": "3.7.0",
-    "eslint-plugin-eslint-plugin": "7.6.0",
-    "eslint-plugin-jsdoc": "64.0.0",
-    "eslint-plugin-jsonc": "3.4.0",
-    "eslint-plugin-n": "18.2.1",
+    "@vitest/coverage-v8": "5.0.0",
+    "@vitest/eslint-plugin": "1.6.27",
+    "astro": "7.3.2",
+    "astro-og-canvas": "0.13.1",
+    "canvaskit-wasm": "0.42.0",
+    "console-fail-test": "0.6.1",
+    "eslint": "10.10.0",
+    "eslint-doc-generator": "3.7.1",
+    "eslint-plugin-eslint-plugin": "7.6.2",
+    "eslint-plugin-jsdoc": "64.3.8",
+    "eslint-plugin-jsonc": "3.4.2",
+    "eslint-plugin-n": "18.3.0",
     "eslint-plugin-node-dependencies": "2.2.0",
-    "eslint-plugin-perfectionist": "5.10.0",
-    "eslint-plugin-regexp": "3.1.0",
-    "eslint-plugin-unicorn": "73.0.0",
-    "eslint-plugin-yml": "3.8.0",
+    "eslint-plugin-perfectionist": "5.11.0",
+    "eslint-plugin-regexp": "3.3.0",
+    "eslint-plugin-unicorn": "74.0.0",
+    "eslint-plugin-yml": "3.8.1",
     "jiti": "2.7.0",
     "json-schema-to-ts": "3.1.1",
-    "knip": "6.32.0",
-    "prettier": "3.9.0",
+    "knip": "6.35.1",
+    "pkg-pr-new": "0.0.88",
+    "prettier": "3.9.6",
     "prettier-plugin-curly": "0.4.1",
     "prettier-plugin-packagejson": "3.0.2",
     "prettier-plugin-sh": "0.19.0",
     "pretty-quick": "4.2.2",
-    "sharp": "0.35.0",
-    "simple-git-hooks": "2.13.1",
-    "starlight-auto-sidebar": "0.4.0",
-    "tsdown": "0.22.2",
+    "sharp": "0.35.4",
+    "simple-git-hooks": "2.14.0",
+    "starlight-auto-sidebar": "0.4.1",
+    "tsdown": "0.23.0",
     "typescript": "npm:@typescript/typescript6@6.0.2",
-    "typescript-eslint": "8.66.0",
-    "vitest": "4.1.8"
+    "typescript-eslint": "8.70.0",
+    "vitest": "5.0.0"
   },
   "peerDependencies": {
     "@eslint/json": ">=1.0.0",