eslint-plugin-package-json 1.3.0 → 1.5.0

package/README.md

@@ -46,7 +46,7 @@ See [Getting Started](https://eslint-plugin-package-json.dev/getting-started) fo
 | [no-local-dependencies](https://eslint-plugin-package-json.dev/rules/no-local-dependencies)                                     | Requires that dependencies do not use local file paths, which will likely result in errors when installing from a registry.                                                                  |       |     |     |
 | [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)                                               | Package properties should be declared in standard order                                                                                                                                      | 🎨    | 🔧  |     |
+| [order-properties](https://eslint-plugin-package-json.dev/rules/order-properties)                                               | Enforces that package properties are declared in a consistent order.                                                                                                                         | 🎨    | 🔧  |     |
 | [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.                                                                                                           |       |     | 💡  |
@@ -65,7 +65,7 @@ This group of rules allows you to require that the associated top-level property
 
 | Name                                                                                                                         | Description                                                 | 💼    | 🔧  | 💡  |
 | :--------------------------------------------------------------------------------------------------------------------------- | :---------------------------------------------------------- | :---- | :-- | :-- |
-| [require-author](https://eslint-plugin-package-json.dev/rules/require-properties/require-author)                             | Requires the `author` property to be present.               |       |     |     |
+| [require-author](https://eslint-plugin-package-json.dev/rules/require-properties/require-author)                             | Requires the `author` property to be present.               |       | 🔧  |     |
 | [require-bin](https://eslint-plugin-package-json.dev/rules/require-properties/require-bin)                                   | Requires the `bin` property to be present.                  |       |     |     |
 | [require-browser](https://eslint-plugin-package-json.dev/rules/require-properties/require-browser)                           | Requires the `browser` property to be present.              |       |     |     |
 | [require-bugs](https://eslint-plugin-package-json.dev/rules/require-properties/require-bugs)                                 | Requires the `bugs` property to be present.                 |       |     |     |
@@ -111,7 +111,7 @@ This group of rules allows you to enforce that the value of the associated top-l
 
 | Name                                                                                                                   | Description                                                                           | 💼    | 🔧  | 💡  |
 | :--------------------------------------------------------------------------------------------------------------------- | :------------------------------------------------------------------------------------ | :---- | :-- | :-- |
-| [valid-author](https://eslint-plugin-package-json.dev/rules/valid-properties/valid-author)                             | Enforce that the `author` property is valid.                                          | ✅ 📦 |     |     |
+| [valid-author](https://eslint-plugin-package-json.dev/rules/valid-properties/valid-author)                             | Enforce that the `author` property is valid.                                          | ✅ 📦 | 🔧  |     |
 | [valid-bin](https://eslint-plugin-package-json.dev/rules/valid-properties/valid-bin)                                   | Enforce that the `bin` property is valid.                                             | ✅ 📦 |     |     |
 | [valid-browser](https://eslint-plugin-package-json.dev/rules/valid-properties/valid-browser)                           | Enforce that the `browser` property is valid.                                         | ✅ 📦 |     |     |
 | [valid-bugs](https://eslint-plugin-package-json.dev/rules/valid-properties/valid-bugs)                                 | Enforce that the `bugs` property is valid.                                            | ✅ 📦 |     |     |

package/lib/createRule.d.mts

@@ -5,17 +5,17 @@ import { AST, RuleListener } from "jsonc-eslint-parser";
 import { AST as AST$1, Rule, SourceCode } from "eslint";
 
 //#region src/createRule.d.ts
-type JsonAstBodyExpression = Expression & {
-  properties: JsonAstBodyProperty[];
+type ASTBodyExpression = Expression & {
+  properties: ASTBodyProperty[];
 };
-type JsonAstBodyProperty = AST.JSONProperty & {
+type ASTBodyProperty = AST.JSONProperty & {
   value: string;
 };
-interface JsonAstBodyStatement extends ExpressionStatement {
-  expression: JsonAstBodyExpression;
+interface ASTBodyStatement extends ExpressionStatement {
+  expression: ASTBodyExpression;
 }
-interface PackageJsonAst extends AST$1.Program {
-  body: [JsonAstBodyStatement];
+interface PackageAST extends AST$1.Program {
+  body: [ASTBodyStatement];
 }
 interface PackageJsonPluginSettings {
   /**
@@ -45,7 +45,7 @@ interface PackageJsonRuleModule<Options extends unknown[] = unknown[], Schema ex
   };
 }
 interface PackageJsonSourceCode extends SourceCode {
-  ast: PackageJsonAst;
+  ast: PackageAST;
 }
 type InferJsonSchemasTupleType<T extends JSONSchema[]> = { [K in keyof T]?: FromSchema<T[K]> };
 /**
@@ -56,4 +56,4 @@ declare function createRule<OptionsOverride extends unknown[] = never, const Sch
   name: string;
 }): PackageJsonRuleModule<_OptionsResolved, Schema>;
 //#endregion
-export { JsonAstBodyExpression, JsonAstBodyProperty, JsonAstBodyStatement, PackageJsonAst, PackageJsonPluginSettings, PackageJsonRuleContext, PackageJsonRuleModule, PackageJsonSourceCode, createRule };
+export { ASTBodyExpression, ASTBodyProperty, ASTBodyStatement, PackageAST, PackageJsonPluginSettings, PackageJsonRuleContext, PackageJsonRuleModule, PackageJsonSourceCode, createRule };

package/lib/createRule.mjs

@@ -1,4 +1,4 @@
-import { isPackageJson } from "./utils/isPackageJson.mjs";
+import { isPackageJson } from "./utils/predicates/isPackageJson.mjs";
 //#region src/createRule.ts
 /**
 * Rule options type is inferred from the JSON schema by [json-schema-to-ts](https://www.npmjs.com/package/json-schema-to-ts).

package/lib/plugin.mjs

@@ -15,9 +15,10 @@ import { rule as rule$12 } from "./rules/scripts-name-casing.mjs";
 import { rule as rule$13 } from "./rules/sort-collections.mjs";
 import { rule as rule$14 } from "./rules/specify-peers-locally.mjs";
 import { rule as rule$15 } from "./rules/unique-dependencies.mjs";
-import { rule as rule$16 } from "./rules/valid-peerDependenciesMeta-relationship.mjs";
+import { rule as rule$16 } from "./rules/valid-author.mjs";
+import { rule as rule$17 } from "./rules/valid-peerDependenciesMeta-relationship.mjs";
 import { rules as rules$2 } from "./rules/valid-properties.mjs";
-import { rule as rule$17 } from "./rules/valid-repository-directory.mjs";
+import { rule as rule$18 } from "./rules/valid-repository-directory.mjs";
 import { createRequire } from "node:module";
 import * as parserJsonc from "jsonc-eslint-parser";
 //#region src/plugin.ts
@@ -40,9 +41,10 @@ const rules = {
 	"sort-collections": rule$13,
 	"specify-peers-locally": rule$14,
 	"unique-dependencies": rule$15,
+	"valid-author": rule$16,
 	...rules$2,
-	"valid-peerDependenciesMeta-relationship": rule$16,
-	"valid-repository-directory": rule$17
+	"valid-peerDependenciesMeta-relationship": rule$17,
+	"valid-repository-directory": rule$18
 };
 const recommendedRules = { ...Object.fromEntries(Object.entries(rules).filter(([, rule]) => rule.meta.docs?.recommended).map(([name]) => ["package-json/" + name, "error"])) };
 const recommendedPublishableRules = {

package/lib/rules/exports-subpaths-style.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 //#region src/rules/exports-subpaths-style.ts
 function isImplicitFormat(node) {
 	if (node.type === "JSONLiteral") return true;

package/lib/rules/no-local-dependencies.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 //#region src/rules/no-local-dependencies.ts
 const isLocalDependency = (value) => value.startsWith("file:") || value.startsWith("link:") || value.startsWith("./") || value.startsWith("../") || value.startsWith(".\\") || value.startsWith("..\\");
 const rule = createRule({

package/lib/rules/no-redundant-files.mjs

@@ -1,5 +1,6 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
+import { isNotNullish } from "../utils/predicates/isNotNullish.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral, isNotNullish } from "../utils/predicates.mjs";
 import { fixRemoveArrayElement } from "eslint-fix-utils";
 //#region src/rules/no-redundant-files.ts
 const defaultFiles = [
@@ -47,15 +48,14 @@ const rule = createRule({
 				}
 			},
 			"Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=files]"(node) {
-				if (node.value.type === "JSONArrayExpression") {
-					const seen = /* @__PURE__ */ new Set();
-					const elements = node.value.elements;
-					entryCache.files = elements;
-					for (const [index, element] of elements.entries()) if (isNotNullish(element) && isJSONStringLiteral(element)) {
-						if (seen.has(element.value)) report(elements, index, "duplicate");
-						else seen.add(element.value);
-						for (const defaultFile of defaultFiles) if (defaultFile.test(element.value)) report(elements, index, "unnecessaryDefault");
-					}
+				if (node.value.type !== "JSONArrayExpression") return;
+				const seen = /* @__PURE__ */ new Set();
+				const elements = node.value.elements;
+				entryCache.files = elements;
+				for (const [index, element] of elements.entries()) if (isNotNullish(element) && isJSONStringLiteral(element)) {
+					if (seen.has(element.value)) report(elements, index, "duplicate");
+					else seen.add(element.value);
+					for (const defaultFile of defaultFiles) if (defaultFile.test(element.value)) report(elements, index, "unnecessaryDefault");
 				}
 			},
 			"Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=main]"(node) {

package/lib/rules/no-redundant-publishConfig.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import { fixRemoveObjectProperty } from "eslint-fix-utils";
 //#region src/rules/no-redundant-publishConfig.ts
 const rule = createRule({

package/lib/rules/order-properties.mjs

@@ -38,10 +38,10 @@ const rule = createRule({
 		defaultOptions: [{ order: "sort-package-json" }],
 		docs: {
 			category: "Stylistic",
-			description: "Package properties should be declared in standard order"
+			description: "Enforces that package properties are declared in a consistent order."
 		},
 		fixable: "code",
-		messages: { incorrectOrder: "Top-level property \"{{property}}\" is not ordered in the standard way." },
+		messages: { incorrectOrder: "Top-level property `{{property}}` is not ordered in the standard way." },
 		schema: [{
 			additionalProperties: false,
 			properties: { order: {

package/lib/rules/repository-shorthand.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import { findPropertyWithKeyValue } from "../utils/findPropertyWithKeyValue.mjs";
 //#region src/rules/repository-shorthand.ts
 const providerRegexes = {

package/lib/rules/require-attribution.mjs

@@ -15,7 +15,7 @@ const rule = createRule({
 			"Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=contributors]"(node) {
 				contributorsPropertyNode = node;
 				const contributorsValue = node.value;
-				if (contributorsValue.type !== "JSONArrayExpression" || !contributorsValue.elements.some((element) => !!element)) context.report({
+				if (contributorsValue.type !== "JSONArrayExpression" || contributorsValue.elements.every((element) => !element)) context.report({
 					messageId: "noContributors",
 					node
 				});

package/lib/rules/require-properties.mjs

@@ -1,7 +1,8 @@
 import { createSimpleRequirePropertyRule } from "../utils/createSimpleRequirePropertyRule.mjs";
+import { getGitAuthor } from "../utils/git/getGitAuthor.mjs";
 //#region src/rules/require-properties.ts
 const propertyConfig = [
-	["author"],
+	["author", { fixValue: getGitAuthor }],
 	["bin"],
 	["browser"],
 	["bugs", { ignorePrivateDefault: true }],

package/lib/rules/restrict-dependency-ranges.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import semver from "semver";
 //#region src/rules/restrict-dependency-ranges.ts
 const DEPENDENCY_TYPES = [
@@ -102,7 +102,7 @@ const rule = createRule({
 						});
 						break;
 					}
-					if (!rangeTypes.find((rangeType) => {
+					if (!rangeTypes.some((rangeType) => {
 						switch (rangeType) {
 							case "caret": return isCaretRange;
 							case "pin": return isPinned;

package/lib/rules/restrict-private-properties.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import { fixRemoveObjectProperty } from "eslint-fix-utils";
 //#region src/rules/restrict-private-properties.ts
 const defaultBlockedProperties = ["files", "publishConfig"];
@@ -7,7 +7,7 @@ const rule = createRule({
 	create(context) {
 		const blockedProperties = context.options[0]?.blockedProperties ?? defaultBlockedProperties;
 		return { "Program > JSONExpressionStatement > JSONObjectExpression"(node) {
-			if (!node.properties.some((property) => isJSONStringLiteral(property.key) && property.key.value === "private" && property.value.type === "JSONLiteral" && property.value.value === true)) return;
+			if (node.properties.every((property) => !(isJSONStringLiteral(property.key) && property.key.value === "private" && property.value.type === "JSONLiteral" && property.value.value === true))) return;
 			for (const property of node.properties) if (isJSONStringLiteral(property.key) && blockedProperties.includes(property.key.value)) {
 				const isEmpty = property.value.type === "JSONArrayExpression" && property.value.elements.length === 0 || property.value.type === "JSONObjectExpression" && property.value.properties.length === 0;
 				context.report({

package/lib/rules/restrict-top-level-properties.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import { fixRemoveObjectProperty } from "eslint-fix-utils";
 //#region src/rules/restrict-top-level-properties.ts
 const rule = createRule({

package/lib/rules/sort-collections.d.mts

@@ -1,10 +1,33 @@
 import { PackageJsonRuleModule } from "../createRule.mjs";
 
 //#region src/rules/sort-collections.d.ts
-declare const rule: PackageJsonRuleModule<[(string[] | undefined)?], [{
-  readonly description: "Array of package properties to require sorting.";
+declare const rule: PackageJsonRuleModule<[((string | {
+  order: string[];
+  key: string;
+})[] | undefined)?], [{
+  readonly description: "Array of package properties to require sorting. Provide a string to sort that collection lexicographically (lifecycle-aware for `scripts`), or an object to sort it by a specified order.";
   readonly items: {
-    readonly type: "string";
+    readonly anyOf: readonly [{
+      readonly type: "string";
+    }, {
+      readonly additionalProperties: false;
+      readonly properties: {
+        readonly key: {
+          readonly description: "The collection property to sort.";
+          readonly type: "string";
+        };
+        readonly order: {
+          readonly description: "The order to sort the collection by. Keys not listed are appended in lexicographical order.";
+          readonly items: {
+            readonly type: "string";
+          };
+          readonly type: "array";
+          readonly uniqueItems: true;
+        };
+      };
+      readonly required: readonly ["key", "order"];
+      readonly type: "object";
+    }];
   };
   readonly type: "array";
 }]>;

package/lib/rules/sort-collections.mjs

@@ -1,4 +1,6 @@
 import { createRule } from "../createRule.mjs";
+import detectIndent from "detect-indent";
+import { detectNewlineGraceful } from "detect-newline";
 import sortPackageJson from "sort-package-json";
 //#region src/rules/sort-collections.ts
 const defaultCollections = new Set([
@@ -14,7 +16,7 @@ const defaultCollections = new Set([
 ]);
 const rule = createRule({
 	create(context) {
-		const toSort = context.options[0] ? new Set(context.options[0]) : defaultCollections;
+		const toSort = new Map((context.options[0] ?? Array.from(defaultCollections)).map((entry) => typeof entry === "string" ? [entry, null] : [entry.key, entry.order]));
 		return { "JSONProperty:exit"(node) {
 			const { key: nodeKey, value: collection } = node;
 			if (nodeKey.type !== "JSONLiteral" || collection.type !== "JSONObjectExpression") return;
@@ -22,27 +24,47 @@ const rule = createRule({
 			for (let currNode = node.parent; currNode; currNode = currNode.parent) if (currNode.type === "JSONProperty" && currNode.key.type === "JSONLiteral") keyPartsReversed.push(currNode.key.value);
 			else if (currNode.type === "JSONArrayExpression") return;
 			const key = keyPartsReversed.reverse().join(".");
-			if (!toSort.has(key)) return;
+			const customOrder = toSort.get(key);
+			if (customOrder === void 0) return;
 			const currentOrder = collection.properties;
-			let desiredOrder;
-			if (keyPartsReversed.at(-1) === "scripts") {
+			const isScripts = keyPartsReversed.at(-1) === "scripts";
+			let naturalCompare;
+			if (isScripts) {
 				const scriptsSource = context.sourceCode.getText(node);
 				const { scripts: sortedScripts } = sortPackageJson(JSON.parse(`{${scriptsSource}}`));
-				const propertyNodeMap = Object.fromEntries(collection.properties.map((prop) => [prop.key.value, prop]));
-				desiredOrder = Object.keys(sortedScripts).map((prop) => propertyNodeMap[prop]);
-			} else desiredOrder = currentOrder.toSorted((a, b) => {
-				return a.key.value > b.key.value ? 1 : -1;
+				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;
+			const orderIndex = new Map((customOrder ?? []).map((k, i) => [k, i]));
+			const rank = (k) => orderIndex.get(k) ?? orderIndex.size;
+			const desiredOrder = currentOrder.toSorted((a, b) => {
+				const aKey = a.key.value;
+				const bKey = b.key.value;
+				const ai = rank(aKey);
+				const bi = rank(bKey);
+				if (ai !== bi) return ai - bi;
+				return naturalCompare(aKey, bKey);
 			});
 			if (currentOrder.some((property, i) => desiredOrder[i] !== property)) context.report({
 				data: { key },
 				fix(fixer) {
-					return fixer.replaceText(collection, JSON.stringify(desiredOrder.reduce((out, property) => {
+					const { text } = context.sourceCode;
+					const { indent, type } = detectIndent(text);
+					const newline = detectNewlineGraceful(text);
+					const indentUnit = type === "tab" ? "	" : indent || "  ";
+					const jsonLines = JSON.stringify(desiredOrder.reduce((out, property) => {
 						out[property.key.value] = JSON.parse(context.sourceCode.getText(property.value));
 						return out;
-					}, {}), null, 2).split("\n").join("\n  "));
+					}, {}), null, indentUnit).split("\n");
+					const collectionStartLine = collection.loc.start.line;
+					const lineText = context.sourceCode.lines[collectionStartLine - 1];
+					const leadingWhitespaceMatch = /^\s*/.exec(lineText);
+					const leadingWhitespace = leadingWhitespaceMatch ? leadingWhitespaceMatch[0] : "";
+					const result = jsonLines.map((l, i) => i === 0 ? l : leadingWhitespace + l).join(newline);
+					return fixer.replaceText(collection, result);
 				},
 				loc: collection.loc,
-				messageId: keyPartsReversed.at(-1) === "scripts" ? "unsortedScripts" : "unsortedKeys",
+				messageId: customOrder ? "unsortedOrder" : isScripts ? "unsortedScripts" : "unsortedKeys",
 				node
 			});
 		} };
@@ -57,11 +79,28 @@ const rule = createRule({
 		fixable: "code",
 		messages: {
 			unsortedKeys: "Entries in '{{ key }}' are not in lexicographical order",
+			unsortedOrder: "Entries in '{{ key }}' are not in the specified order",
 			unsortedScripts: "Entries in 'scripts' are not in lexicographical order and grouped by lifecycles"
 		},
 		schema: [{
-			description: "Array of package properties to require sorting.",
-			items: { type: "string" },
+			description: "Array of package properties to require sorting. Provide a string to sort that collection lexicographically (lifecycle-aware for `scripts`), or an object to sort it by a specified order.",
+			items: { anyOf: [{ type: "string" }, {
+				additionalProperties: false,
+				properties: {
+					key: {
+						description: "The collection property to sort.",
+						type: "string"
+					},
+					order: {
+						description: "The order to sort the collection by. Keys not listed are appended in lexicographical order.",
+						items: { type: "string" },
+						type: "array",
+						uniqueItems: true
+					}
+				},
+				required: ["key", "order"],
+				type: "object"
+			}] },
 			type: "array"
 		}],
 		type: "layout"

package/lib/rules/specify-peers-locally.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 //#region src/rules/specify-peers-locally.ts
 const rule = createRule({
 	create(context) {

package/lib/rules/unique-dependencies.mjs

@@ -1,5 +1,6 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
+import { isNotNullish } from "../utils/predicates/isNotNullish.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral, isNotNullish } from "../utils/predicates.mjs";
 import { fixRemoveArrayElement, fixRemoveObjectProperty } from "eslint-fix-utils";
 //#region src/rules/unique-dependencies.ts
 const dependencyPropertyNames = new Set([

package/lib/rules/valid-author.d.mts

@@ -0,0 +1,6 @@
+import { PackageJsonRuleModule } from "../createRule.mjs";
+
+//#region src/rules/valid-author.d.ts
+declare const rule: PackageJsonRuleModule<[], []>;
+//#endregion
+export { rule };

package/lib/rules/valid-author.mjs

@@ -0,0 +1,59 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
+import { createRule } from "../createRule.mjs";
+import { getGitAuthor } from "../utils/git/getGitAuthor.mjs";
+import { validateAuthor } from "package-json-validator";
+//#region src/rules/valid-author.ts
+const rule = createRule({
+	create(context) {
+		const createFixer = (propertyName, node) => {
+			const author = getGitAuthor();
+			if (!author) return;
+			let mergedAuthor = author;
+			const originalValue = JSON.parse(context.sourceCode.getText(node));
+			if (originalValue && typeof originalValue === "object" && !Array.isArray(originalValue)) mergedAuthor = {
+				name: author.name,
+				...originalValue
+			};
+			switch (propertyName) {
+				case "author": return (fixer) => fixer.replaceText(node, JSON.stringify(mergedAuthor, null, 2).split("\n").join("\n  "));
+				case "email": return (fixer) => fixer.replaceText(node, JSON.stringify(author.email));
+				case "name": return (fixer) => fixer.replaceText(node, JSON.stringify(author.name));
+				default: return;
+			}
+		};
+		const reportIssues = (result, node, propertyName) => {
+			if (result.errorMessages.length === 0) return;
+			if (result.issues.length) for (const issue of result.issues) context.report({
+				data: { error: issue.message },
+				fix: propertyName === void 0 ? void 0 : createFixer(propertyName, node),
+				messageId: "validationError",
+				node
+			});
+			const childrenWithIssues = result.childResults.filter((childResult) => childResult.errorMessages.length);
+			if (node.type === "JSONObjectExpression" && childrenWithIssues.length) for (const childResult of childrenWithIssues) {
+				const childNode = node.properties[childResult.index];
+				const childPropertyName = childNode.key;
+				reportIssues(childResult, childNode.value, isJSONStringLiteral(childPropertyName) ? childPropertyName.value : void 0);
+			}
+		};
+		return { "Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=author]"(node) {
+			const valueNode = node.value;
+			reportIssues(validateAuthor(JSON.parse(context.sourceCode.getText(valueNode))), valueNode, "author");
+		} };
+	},
+	meta: {
+		docs: {
+			category: "Best Practices",
+			description: "Enforce that the `author` property is valid.",
+			recommended: true,
+			ruleGroup: "valid-properties"
+		},
+		fixable: "code",
+		messages: { validationError: `Invalid author: {{ error }}` },
+		schema: [],
+		type: "problem"
+	},
+	name: "valid-author"
+});
+//#endregion
+export { rule };

package/lib/rules/valid-peerDependenciesMeta-relationship.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "../utils/predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "../utils/predicates.mjs";
 import { fixRemoveObjectProperty } from "eslint-fix-utils";
 //#region src/rules/valid-peerDependenciesMeta-relationship.ts
 const rule = createRule({

package/lib/rules/valid-properties.mjs

@@ -1,8 +1,7 @@
 import { createSimpleValidPropertyRule } from "../utils/createSimpleValidPropertyRule.mjs";
-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 { 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";
 /** All basic valid- flavor rules */
 const rules = Object.fromEntries([
-	["author", validateAuthor],
 	["bin", validateBin],
 	["browser", validateBrowser],
 	["bugs", validateBugs],

package/lib/rules/valid-repository-directory.mjs

@@ -12,8 +12,8 @@ import { findRootSync } from "@altano/repository-tools";
 * @example '/a/b/c', 'd' => false
 */
 const pathEndsWith = (parent, child) => {
-	const segments = parent.split(path.sep);
 	if (parent === child) return true;
+	const segments = parent.split(path.sep);
 	let pathToCheck = "";
 	return segments.reverse().some((segment) => {
 		pathToCheck = path.join(segment, pathToCheck);

package/lib/utils/createSimpleRequirePropertyRule.mjs

@@ -1,5 +1,5 @@
+import { isJSONStringLiteral } from "./predicates/isJSONStringLiteral.mjs";
 import { createRule } from "../createRule.mjs";
-import { isJSONStringLiteral } from "./predicates.mjs";
 //#region src/utils/createSimpleRequirePropertyRule.ts
 /**
 * Given a top-level property name, create a rule that requires that property to be present.
@@ -16,18 +16,21 @@ const createSimpleRequirePropertyRule = (propertyName, { category, fixValue, ign
 				const ignorePrivate = context.options[0]?.ignorePrivate ?? (typeof enforceForPrivate === "boolean" ? !enforceForPrivate : ignorePrivateDefault);
 				return { "Program > JSONExpressionStatement > JSONObjectExpression"(node) {
 					if (ignorePrivate && node.properties.some((property) => isJSONStringLiteral(property.key) && property.key.value === "private" && property.value.type === "JSONLiteral" && property.value.value === true)) return;
-					if (!node.properties.some((property) => isJSONStringLiteral(property.key) && property.key.value === propertyName)) context.report({
-						data: { property: propertyName },
-						fix: fixValue === void 0 ? void 0 : function* (fixer) {
-							yield fixer.insertTextAfterRange([0, 1], `\n  "${propertyName}": ${JSON.stringify(fixValue)}`);
-							yield node.properties.length > 0 ? fixer.insertTextAfterRange([0, 1], ",") : fixer.insertTextAfterRange([0, 1], "\n");
-						},
-						loc: {
-							column: 0,
-							line: 1
-						},
-						messageId: "missing"
-					});
+					if (node.properties.every((property) => !(isJSONStringLiteral(property.key) && property.key.value === propertyName))) {
+						const resolvedValue = typeof fixValue === "function" ? fixValue() : fixValue;
+						context.report({
+							data: { property: propertyName },
+							fix: resolvedValue === void 0 ? void 0 : function* (fixer) {
+								yield fixer.insertTextAfterRange([0, 1], `\n  "${propertyName}": ${JSON.stringify(resolvedValue, null, 2).split("\n").join("\n  ")}`);
+								yield node.properties.length > 0 ? fixer.insertTextAfterRange([0, 1], ",") : fixer.insertTextAfterRange([0, 1], "\n");
+							},
+							loc: {
+								column: 0,
+								line: 1
+							},
+							messageId: "missing"
+						});
+					}
 				} };
 			},
 			meta: {

package/lib/utils/git/getGitAuthor.d.mts

@@ -0,0 +1,8 @@
+//#region src/utils/git/getGitAuthor.d.ts
+interface Author {
+  email?: string;
+  name: string;
+}
+declare const getGitAuthor: () => Author | undefined;
+//#endregion
+export { getGitAuthor };

package/lib/utils/git/getGitAuthor.mjs

@@ -0,0 +1,29 @@
+import { execFileSync } from "node:child_process";
+//#region src/utils/git/getGitAuthor.ts
+let cachedAuthor;
+const getGitAuthor = () => {
+	if (cachedAuthor === null) return;
+	else if (cachedAuthor !== void 0) return cachedAuthor;
+	try {
+		const name = execFileSync("git", [
+			"config",
+			"--get",
+			"user.name"
+		], { encoding: "utf8" }).trim();
+		const email = execFileSync("git", [
+			"config",
+			"--get",
+			"user.email"
+		], { encoding: "utf8" }).trim();
+		cachedAuthor = {
+			name,
+			...email && { email }
+		};
+	} catch {
+		cachedAuthor = null;
+		return;
+	}
+	return cachedAuthor;
+};
+//#endregion
+export { getGitAuthor };

package/lib/utils/git/index.d.mts

@@ -0,0 +1,2 @@
+import { getGitAuthor } from "./getGitAuthor.mjs";
+export { getGitAuthor };

package/lib/utils/git/index.mjs

@@ -0,0 +1,2 @@
+import { getGitAuthor } from "./getGitAuthor.mjs";
+export { getGitAuthor };

package/lib/utils/predicates/index.d.mts

@@ -0,0 +1,4 @@
+import { isJSONStringLiteral } from "./isJSONStringLiteral.mjs";
+import { isNotNullish } from "./isNotNullish.mjs";
+import { isPackageJson } from "./isPackageJson.mjs";
+export { isJSONStringLiteral, isNotNullish, isPackageJson };

package/lib/utils/predicates/index.mjs

@@ -0,0 +1,4 @@
+import { isJSONStringLiteral } from "./isJSONStringLiteral.mjs";
+import { isNotNullish } from "./isNotNullish.mjs";
+import { isPackageJson } from "./isPackageJson.mjs";
+export { isJSONStringLiteral, isNotNullish, isPackageJson };

package/lib/utils/predicates/isJSONStringLiteral.d.mts

@@ -0,0 +1,6 @@
+import { AST } from "jsonc-eslint-parser";
+
+//#region src/utils/predicates/isJSONStringLiteral.d.ts
+declare function isJSONStringLiteral(node: AST.JSONNode): node is AST.JSONStringLiteral;
+//#endregion
+export { isJSONStringLiteral };

package/lib/utils/predicates/isJSONStringLiteral.mjs

@@ -0,0 +1,6 @@
+//#region src/utils/predicates/isJSONStringLiteral.ts
+function isJSONStringLiteral(node) {
+	return node.type === "JSONLiteral" && typeof node.value === "string";
+}
+//#endregion
+export { isJSONStringLiteral };

package/lib/utils/predicates/isNotNullish.d.mts

@@ -0,0 +1,4 @@
+//#region src/utils/predicates/isNotNullish.d.ts
+declare function isNotNullish<T extends NonNullable<unknown>>(value: null | T | undefined): value is T;
+//#endregion
+export { isNotNullish };

package/lib/utils/predicates/isNotNullish.mjs

@@ -0,0 +1,6 @@
+//#region src/utils/predicates/isNotNullish.ts
+function isNotNullish(value) {
+	return value !== null && value !== void 0;
+}
+//#endregion
+export { isNotNullish };

package/lib/utils/predicates/isPackageJson.d.mts

@@ -0,0 +1,4 @@
+//#region src/utils/predicates/isPackageJson.d.ts
+declare const isPackageJson: (filePath: string) => boolean;
+//#endregion
+export { isPackageJson };

package/lib/utils/{isPackageJson.mjs → predicates/isPackageJson.mjs}

@@ -1,4 +1,4 @@
-//#region src/utils/isPackageJson.ts
+//#region src/utils/predicates/isPackageJson.ts
 const isPackageJson = (filePath) => /(?:^|[/\\])package.json$/.test(filePath);
 //#endregion
 export { isPackageJson };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-package-json",
-  "version": "1.3.0",
+  "version": "1.5.0",
   "description": "Rules for consistent, readable, and valid package.json files. 🗂️",
   "homepage": "https://github.com/michaelfaith/eslint-plugin-package-json#readme",
   "bugs": {
@@ -53,51 +53,52 @@
     "package-json-validator": "^1.5.0",
     "semver": "^7.7.3",
     "sort-object-keys": "^2.0.0",
-    "sort-package-json": "^3.4.0"
+    "sort-package-json": "^4.0.0"
   },
   "devDependencies": {
-    "@astrojs/starlight": "0.39.0",
+    "@astrojs/check": "0.9.9",
+    "@astrojs/starlight": "0.40.0",
     "@catppuccin/starlight": "2.0.1",
-    "@eslint-community/eslint-plugin-eslint-comments": "4.7.0",
+    "@eslint-community/eslint-plugin-eslint-comments": "4.7.2",
     "@eslint/js": "10.0.1",
     "@eslint/json": "2.0.0",
     "@eslint/markdown": "8.0.1",
     "@ianvs/prettier-plugin-sort-imports": "4.7.1",
     "@types/estree": "1.0.8",
-    "@types/node": "24.12.0",
+    "@types/node": "24.13.0",
     "@types/semver": "7.7.1",
-    "@vitest/coverage-v8": "4.1.0",
-    "@vitest/eslint-plugin": "1.6.1",
+    "@vitest/coverage-v8": "4.1.8",
+    "@vitest/eslint-plugin": "1.6.20",
     "astro": "6.4.2",
     "astro-og-canvas": "0.11.1",
     "canvaskit-wasm": "0.41.1",
     "console-fail-test": "0.6.0",
-    "eslint": "10.4.0",
+    "eslint": "10.5.0",
     "eslint-doc-generator": "3.6.0",
-    "eslint-plugin-eslint-plugin": "7.3.1",
-    "eslint-plugin-jsdoc": "63.0.0",
+    "eslint-plugin-eslint-plugin": "7.4.0",
+    "eslint-plugin-jsdoc": "63.0.2",
     "eslint-plugin-jsonc": "3.2.0",
-    "eslint-plugin-n": "18.0.0",
+    "eslint-plugin-n": "18.1.0",
     "eslint-plugin-node-dependencies": "2.2.0",
     "eslint-plugin-perfectionist": "5.9.0",
     "eslint-plugin-regexp": "3.1.0",
-    "eslint-plugin-unicorn": "64.0.0",
+    "eslint-plugin-unicorn": "67.0.0",
     "eslint-plugin-yml": "3.4.0",
     "jiti": "2.7.0",
     "json-schema-to-ts": "3.1.1",
-    "knip": "6.14.0",
-    "prettier": "3.8.0",
-    "prettier-plugin-curly": "0.4.0",
-    "prettier-plugin-packagejson": "3.0.0",
-    "prettier-plugin-sh": "0.18.0",
+    "knip": "6.17.1",
+    "prettier": "3.8.4",
+    "prettier-plugin-curly": "0.4.1",
+    "prettier-plugin-packagejson": "3.0.2",
+    "prettier-plugin-sh": "0.18.1",
     "pretty-quick": "4.2.2",
-    "sharp": "0.34.5",
+    "sharp": "0.35.0",
     "simple-git-hooks": "2.13.1",
     "starlight-auto-sidebar": "0.4.0",
-    "tsdown": "0.22.0",
-    "typescript": "6.0.2",
-    "typescript-eslint": "8.60.0",
-    "vitest": "4.1.0"
+    "tsdown": "0.22.2",
+    "typescript": "6.0.3",
+    "typescript-eslint": "8.61.0",
+    "vitest": "4.1.8"
   },
   "peerDependencies": {
     "@eslint/json": ">=1.0.0",
@@ -118,10 +119,10 @@
     "lint": "eslint . --max-warnings 0",
     "lint:knip": "knip",
     "site:build": "astro build",
+    "site:check": "astro check --root site",
     "site:dev": "astro dev",
     "site:preview": "astro preview",
     "site:sync": "astro sync",
-    "site:typecheck": "pnpm site:sync && tsc --project site/tsconfig.json",
     "test": "vitest",
     "typecheck": "tsc"
   }

package/lib/utils/isPackageJson.d.mts

@@ -1,4 +0,0 @@
-//#region src/utils/isPackageJson.d.ts
-declare const isPackageJson: (filePath: string) => boolean;
-//#endregion
-export { isPackageJson };

package/lib/utils/predicates.d.mts

@@ -1,7 +0,0 @@
-import { AST } from "jsonc-eslint-parser";
-
-//#region src/utils/predicates.d.ts
-declare function isJSONStringLiteral(node: AST.JSONNode): node is AST.JSONStringLiteral;
-declare function isNotNullish<T extends NonNullable<unknown>>(value: null | T | undefined): value is T;
-//#endregion
-export { isJSONStringLiteral, isNotNullish };

package/lib/utils/predicates.mjs

@@ -1,9 +0,0 @@
-//#region src/utils/predicates.ts
-function isJSONStringLiteral(node) {
-	return node.type === "JSONLiteral" && typeof node.value === "string";
-}
-function isNotNullish(value) {
-	return value !== null && value !== void 0;
-}
-//#endregion
-export { isJSONStringLiteral, isNotNullish };