eslint-plugin-package-json 0.59.0 → 0.60.0

package/CHANGELOG.md

@@ -1,11 +1,23 @@
 # Changelog
 
-# [0.59.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.58.0...v0.59.0) (2025-10-25)
+# [0.60.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.59.1...v0.60.0) (2025-11-03)
+
+
+### Features
+
+* **valid-author:** create more precise reports ([#1337](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1337)) ([e79ff9a](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/e79ff9ac602d85bf999686b018de6ace3a7bee36)), closes [#000](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/000)
 
+## [0.59.1](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.59.0...v0.59.1) (2025-10-30)
+
+### Bug Fixes
+
+- **deps:** update dependency validate-npm-package-name to v7 ([#1353](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1353)) ([be9970a](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/be9970a18c73521906d1564ca461d94159e88807))
+
+# [0.59.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.58.0...v0.59.0) (2025-10-25)
 
 ### Features
 
-* **exports-subpaths-style:** add new rule ([#1328](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1328)) ([d1a82ed](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/d1a82edf78b7b4bd446018f1ea3bb77e1bc9c772)), closes [#1322](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1322)
+- **exports-subpaths-style:** add new rule ([#1328](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1328)) ([d1a82ed](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/d1a82edf78b7b4bd446018f1ea3bb77e1bc9c772)), closes [#1322](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1322)
 
 # [0.58.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.57.0...v0.58.0) (2025-10-22)
 

package/README.md

@@ -262,7 +262,7 @@ Thanks! 🗂
       <td align="center" valign="top" width="14.28%"><a href="https://davidlj95.com"><img src="https://avatars.githubusercontent.com/u/8050648?v=4?s=100" width="100px;" alt="David LJ"/><br /><sub><b>David LJ</b></sub></a><br /><a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=davidlj95" title="Documentation">📖</a></td>
       <td align="center" valign="top" width="14.28%"><a href="http://lishaduck.github.io"><img src="https://avatars.githubusercontent.com/u/88557639?v=4?s=100" width="100px;" alt="Eli"/><br /><sub><b>Eli</b></sub></a><br /><a href="#ideas-lishaduck" title="Ideas, Planning, & Feedback">🤔</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues?q=author%3Alishaduck" title="Bug reports">🐛</a></td>
       <td align="center" valign="top" width="14.28%"><a href="http://heggria.site"><img src="https://avatars.githubusercontent.com/u/34475327?v=4?s=100" width="100px;" alt="Heggria"/><br /><sub><b>Heggria</b></sub></a><br /><a href="#ideas-heggria" title="Ideas, Planning, & Feedback">🤔</a></td>
-      <td align="center" valign="top" width="14.28%"><a href="https://hirok.io"><img src="https://avatars.githubusercontent.com/u/1075694?v=4?s=100" width="100px;" alt="Hiroki Osame"/><br /><sub><b>Hiroki Osame</b></sub></a><br /><a href="#ideas-privatenumber" title="Ideas, Planning, & Feedback">🤔</a></td>
+      <td align="center" valign="top" width="14.28%"><a href="https://hirok.io"><img src="https://avatars.githubusercontent.com/u/1075694?v=4?s=100" width="100px;" alt="Hiroki Osame"/><br /><sub><b>Hiroki Osame</b></sub></a><br /><a href="#ideas-privatenumber" title="Ideas, Planning, & Feedback">🤔</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=privatenumber" title="Code">💻</a></td>
       <td align="center" valign="top" width="14.28%"><a href="https://github.com/Zamiell"><img src="https://avatars.githubusercontent.com/u/5511220?v=4?s=100" width="100px;" alt="James"/><br /><sub><b>James</b></sub></a><br /><a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=Zamiell" title="Code">💻</a> <a href="#ideas-Zamiell" title="Ideas, Planning, & Feedback">🤔</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues?q=author%3AZamiell" title="Bug reports">🐛</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=Zamiell" title="Documentation">📖</a></td>
       <td align="center" valign="top" width="14.28%"><a href="https://github.com/zetlen"><img src="https://avatars.githubusercontent.com/u/1643758?v=4?s=100" width="100px;" alt="James Zetlen"/><br /><sub><b>James Zetlen</b></sub></a><br /><a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=zetlen" title="Code">💻</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues?q=author%3Azetlen" title="Bug reports">🐛</a> <a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=zetlen" title="Documentation">📖</a> <a href="#infra-zetlen" title="Infrastructure (Hosting, Build-Tools, etc)">🚇</a> <a href="#maintenance-zetlen" title="Maintenance">🚧</a> <a href="#tool-zetlen" title="Tools">🔧</a></td>
       <td align="center" valign="top" width="14.28%"><a href="https://piranna.github.io/"><img src="https://avatars.githubusercontent.com/u/532414?v=4?s=100" width="100px;" alt="Jesús Leganés-Combarro"/><br /><sub><b>Jesús Leganés-Combarro</b></sub></a><br /><a href="https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commits?author=piranna" title="Code">💻</a></td>

package/lib/plugin.js

@@ -7,13 +7,14 @@ import { rules } from "./rules/require-properties.js";
 import { rule as rule$5 } from "./rules/restrict-dependency-ranges.js";
 import { rule as rule$6 } from "./rules/sort-collections.js";
 import { rule as rule$7 } from "./rules/unique-dependencies.js";
-import { rule as rule$8 } from "./rules/valid-bin.js";
-import { rule as rule$9 } from "./rules/valid-local-dependency.js";
-import { rule as rule$10 } from "./rules/valid-name.js";
-import { rule as rule$11 } from "./rules/valid-package-definition.js";
+import { rule as rule$8 } from "./rules/valid-author.js";
+import { rule as rule$9 } from "./rules/valid-bin.js";
+import { rule as rule$10 } from "./rules/valid-local-dependency.js";
+import { rule as rule$11 } from "./rules/valid-name.js";
+import { rule as rule$12 } from "./rules/valid-package-definition.js";
 import { rules as rules$1 } from "./rules/valid-properties.js";
-import { rule as rule$12 } from "./rules/valid-repository-directory.js";
-import { rule as rule$13 } from "./rules/valid-version.js";
+import { rule as rule$13 } from "./rules/valid-repository-directory.js";
+import { rule as rule$14 } from "./rules/valid-version.js";
 import { createRequire } from "node:module";
 import * as parserJsonc from "jsonc-eslint-parser";
 
@@ -30,14 +31,15 @@ const rules$2 = {
 	"sort-collections": rule$6,
 	"unique-dependencies": rule$7,
 	...rules$1,
-	"valid-bin": rule$8,
-	"valid-local-dependency": rule$9,
-	"valid-name": rule$10,
-	"valid-package-definition": rule$11,
-	"valid-repository-directory": rule$12,
-	"valid-version": rule$13
+	"valid-author": rule$8,
+	"valid-bin": rule$9,
+	"valid-local-dependency": rule$10,
+	"valid-name": rule$11,
+	"valid-package-definition": rule$12,
+	"valid-repository-directory": rule$13,
+	"valid-version": rule$14
 };
-const baseRecommendedRules = { ...Object.fromEntries(Object.entries(rules$2).filter(([, rule$14]) => rule$14.meta.docs?.recommended).map(([name$1]) => ["package-json/" + name$1, "error"])) };
+const baseRecommendedRules = { ...Object.fromEntries(Object.entries(rules$2).filter(([, rule$15]) => rule$15.meta.docs?.recommended).map(([name$1]) => ["package-json/" + name$1, "error"])) };
 const recommendedRules = {
 	...baseRecommendedRules,
 	"package-json/valid-package-definition": ["error", { ignoreProperties: Object.entries(baseRecommendedRules).filter(([name$1]) => name$1.startsWith("package-json/valid-") && name$1 !== "package-json/valid-package-definition").map(([name$1]) => name$1.replace("package-json/valid-", "")) }]

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

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

package/lib/rules/valid-author.js

@@ -0,0 +1,39 @@
+import { createRule } from "../createRule.js";
+import { validateAuthor } from "package-json-validator";
+
+//#region src/rules/valid-author.ts
+const rule = createRule({
+	create(context) {
+		const reportIssues = (result, node) => {
+			if (result.errorMessages.length === 0) return;
+			if (result.issues.length) for (const issue of result.issues) context.report({
+				data: { error: issue.message },
+				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];
+				reportIssues(childResult, childNode);
+			}
+		};
+		return { "Program > JSONExpressionStatement > JSONObjectExpression > JSONProperty[key.value=author]"(node) {
+			const valueNode = node.value;
+			reportIssues(validateAuthor(JSON.parse(context.sourceCode.getText(valueNode))), valueNode);
+		} };
+	},
+	meta: {
+		docs: {
+			category: "Best Practices",
+			description: "Enforce that the `author` property is valid.",
+			recommended: true
+		},
+		messages: { validationError: `Invalid author: {{ error }}` },
+		schema: [],
+		type: "problem"
+	},
+	name: "valid-author"
+});
+
+//#endregion
+export { rule };

package/lib/rules/valid-bin.js

@@ -1,7 +1,7 @@
 import { createRule } from "../createRule.js";
 import { formatErrors } from "../utils/formatErrors.js";
-import { kebabCase } from "change-case";
 import { validateBin } from "package-json-validator";
+import { kebabCase } from "change-case";
 
 //#region src/rules/valid-bin.ts
 const rule = createRule({

package/lib/rules/valid-properties.js

@@ -1,9 +1,8 @@
 import { createSimpleValidPropertyRule } from "../utils/createSimpleValidPropertyRule.js";
-import { validateAuthor, validateBundleDependencies, validateConfig, validateCpu, validateDependencies, validateDescription, validateDirectories, validateExports, validateLicense, validateScripts, validateType } from "package-json-validator";
+import { validateBundleDependencies, validateConfig, validateCpu, validateDependencies, validateDescription, validateDirectories, validateExports, validateLicense, validateScripts, validateType } from "package-json-validator";
 
 //#region src/rules/valid-properties.ts
 const properties = [
-	["author", validateAuthor],
 	["bundleDependencies", {
 		aliases: ["bundledDependencies"],
 		validator: validateBundleDependencies

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-plugin-package-json",
-	"version": "0.59.0",
+	"version": "0.60.0",
 	"description": "Rules for consistent, readable, and valid package.json files. 🗂️",
 	"homepage": "https://github.com/JoshuaKGoldberg/eslint-plugin-package-json#readme",
 	"bugs": {
@@ -48,26 +48,26 @@
 		"detect-indent": "^7.0.2",
 		"detect-newline": "^4.0.1",
 		"eslint-fix-utils": "~0.4.0",
-		"package-json-validator": "~0.31.0",
+		"package-json-validator": "~0.32.1",
 		"semver": "^7.7.3",
 		"sort-object-keys": "^2.0.0",
 		"sort-package-json": "^3.4.0",
-		"validate-npm-package-name": "^6.0.2"
+		"validate-npm-package-name": "^7.0.0"
 	},
 	"devDependencies": {
 		"@eslint-community/eslint-plugin-eslint-comments": "4.5.0",
 		"@eslint/js": "9.38.0",
 		"@release-it/conventional-changelog": "10.0.0",
 		"@types/estree": "1.0.7",
-		"@types/node": "22.18.0",
+		"@types/node": "24.9.1",
 		"@types/semver": "7.7.0",
 		"@types/validate-npm-package-name": "4.0.2",
-		"@vitest/coverage-v8": "3.2.0",
+		"@vitest/coverage-v8": "4.0.4",
 		"@vitest/eslint-plugin": "1.3.3",
 		"console-fail-test": "0.5.0",
 		"eslint": "9.38.0",
 		"eslint-doc-generator": "2.3.0",
-		"eslint-plugin-eslint-plugin": "7.0.0",
+		"eslint-plugin-eslint-plugin": "7.2.0",
 		"eslint-plugin-jsdoc": "61.1.0",
 		"eslint-plugin-jsonc": "2.21.0",
 		"eslint-plugin-n": "17.23.1",
@@ -83,7 +83,7 @@
 		"markdownlint": "0.39.0",
 		"markdownlint-cli": "0.45.0",
 		"prettier": "3.6.0",
-		"prettier-plugin-curly": "0.3.1",
+		"prettier-plugin-curly": "0.4.0",
 		"prettier-plugin-packagejson": "2.5.10",
 		"prettier-plugin-sh": "0.18.0",
 		"release-it": "19.0.1",
@@ -91,13 +91,13 @@
 		"tsdown": "0.15.0",
 		"typescript": "5.9.2",
 		"typescript-eslint": "8.46.0",
-		"vitest": "3.2.0"
+		"vitest": "4.0.4"
 	},
 	"peerDependencies": {
 		"eslint": ">=8.0.0",
 		"jsonc-eslint-parser": "^2.0.0"
 	},
-	"packageManager": "pnpm@10.18.0",
+	"packageManager": "pnpm@10.19.0",
 	"engines": {
 		"node": "^20.19.0 || >=22.12.0"
 	},