eslint-plugin-package-json 0.50.0 → 0.51.0

package/CHANGELOG.md

@@ -1,11 +1,17 @@
 # Changelog
 
-# [0.50.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.49.0...v0.50.0) (2025-08-05)
+# [0.51.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.50.0...v0.51.0) (2025-08-05)
+
+
+### Features
+
+* add `ignorePrivate` option to all `require-*` rules ([#1158](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1158)) ([055009b](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/055009bf864c9f8db153bf0c5bb9568d023abe12)), closes [#1092](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1092)
 
+# [0.50.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.49.0...v0.50.0) (2025-08-05)
 
 ### Features
 
-* add new `require-` rules for `bugs`, `bundleDependencies`, `dependencies`, and more ([#1197](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1197)) ([0a06664](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/0a0666404d5659ea8eb8717516aa3c3b6374af26)), closes [#862](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/862) [#863](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/863) [#811](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/811) [#809](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/809) [#801](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/801) [#797](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/797)
+- add new `require-` rules for `bugs`, `bundleDependencies`, `dependencies`, and more ([#1197](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/1197)) ([0a06664](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/commit/0a0666404d5659ea8eb8717516aa3c3b6374af26)), closes [#862](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/862) [#863](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/863) [#811](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/811) [#809](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/809) [#801](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/801) [#797](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/797)
 
 # [0.49.0](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/compare/v0.48.0...v0.49.0) (2025-08-05)
 

package/README.md

@@ -101,6 +101,40 @@ module.exports = {
 };
 ```
 
+### Settings
+
+Some rules can be configured in ESLint shared settings.
+You can set them in `settings.packageJson` in an ESLint flat config.
+
+Example:
+
+```ts
+// eslint.config.ts
+import packageJson from "eslint-plugin-package-json";
+
+export default {
+	plugins: {
+		"package-json": packageJson,
+	},
+	rules: {
+		// `description` won't be required in package.json with `"private": true`
+		"package-json/require-description": "error",
+	},
+	settings: {
+		packageJson: {
+			enforceForPrivate: false,
+		},
+	},
+};
+```
+
+#### `enforceForPrivate`
+
+**Type:** `boolean`
+
+Determines whether `require-*` rules, if used, should enforce the presence of the corresponding property in package.json files with `"private": true`.
+By default, all `require-*` rules except for [`require-name`](docs/rules/require-name.md) and [`require-version`](docs/rules/require-version.md) will report if the corresponding property is missing in package.json with `"private": true`.
+
 ### Usage Alongside Prettier
 
 **[`prettier-plugin-packagejson`](https://github.com/matzkoh/prettier-plugin-packagejson)** is a [Prettier plugin](https://prettier.io/docs/en/plugins) that enforces the same `package.json` keys ordering as the [`order-properties`](docs/rules/order-properties.md) and [sort-collections](docs/rules/sort-collections.md) rules with default options.

package/lib/createRule.d.ts

@@ -14,8 +14,20 @@ interface JsonAstBodyStatement extends ESTree.ExpressionStatement {
 interface PackageJsonAst extends AST$1.Program {
     body: [JsonAstBodyStatement];
 }
+interface PackageJsonPluginSettings {
+    /**
+     * Whether `require-*` rules, if used, should enforce the presence of
+     * the corresponding property *in package.json files with `"private": true`*.
+     *
+     * If not specified, it will not enforce the presence only of `name` and `version` properties.
+     */
+    enforceForPrivate?: boolean;
+}
 interface PackageJsonRuleContext<Options extends unknown[] = unknown[]> extends Rule.RuleContext {
     options: Options;
+    settings: {
+        packageJson?: PackageJsonPluginSettings;
+    };
     sourceCode: PackageJsonSourceCode;
 }
 interface PackageJsonRuleModule<Options extends unknown[] = unknown[]> {
@@ -30,4 +42,4 @@ declare function createRule<Options extends unknown[]>(rule: PackageJsonRuleModu
     meta: Rule.RuleMetaData;
 };
 
-export { type JsonAstBodyExpression, type JsonAstBodyProperty, type JsonAstBodyStatement, type PackageJsonAst, type PackageJsonRuleContext, type PackageJsonRuleModule, type PackageJsonSourceCode, createRule };
+export { type JsonAstBodyExpression, type JsonAstBodyProperty, type JsonAstBodyStatement, type PackageJsonAst, type PackageJsonPluginSettings, type PackageJsonRuleContext, type PackageJsonRuleModule, type PackageJsonSourceCode, createRule };

package/lib/index.d.ts

@@ -1,5 +1,6 @@
 import * as jsonc_eslint_parser from 'jsonc-eslint-parser';
 import { PackageJsonRuleModule } from './createRule.js';
+export { PackageJsonPluginSettings } from './createRule.js';
 import { plugin } from './plugin.js';
 import 'estree';
 import 'eslint';

package/lib/rules/require-properties.d.ts

@@ -1,9 +1,15 @@
-import { PackageJsonRuleModule } from '../createRule.js';
+import * as eslint from 'eslint';
+import * as jsonc_eslint_parser from 'jsonc-eslint-parser';
+import { PackageJsonRuleContext } from '../createRule.js';
 import 'estree';
-import 'eslint';
-import 'jsonc-eslint-parser';
 
-/** All require- flavor rules */
-declare const rules: Record<string, PackageJsonRuleModule<unknown[]>>;
+declare const rules: {
+    [k: string]: {
+        create(context: PackageJsonRuleContext<[({
+            ignorePrivate?: boolean;
+        } | undefined)?]>): jsonc_eslint_parser.RuleListener;
+        meta: eslint.Rule.RuleMetaData;
+    };
+};
 
 export { rules };

package/lib/rules/require-properties.js

@@ -1,31 +1,28 @@
-import { createSimpleRequirePropertyRule } from "../utils/createSimpleRequirePropertyRule.js";
+import {
+  createSimpleRequirePropertyRule
+} from "../utils/createSimpleRequirePropertyRule.js";
 const properties = [
-  ["author", false],
-  ["bugs", false],
-  ["bundleDependencies", false],
-  ["dependencies", false],
-  ["devDependencies", false],
-  ["description", true],
-  ["engines", false],
-  ["files", false],
-  ["keywords", false],
-  ["name", true],
-  ["optionalDependencies", false],
-  ["peerDependencies", false],
-  ["type", true],
-  ["types", false],
-  ["version", true]
-  // TODO: More to come!
+  ["author"],
+  ["bugs"],
+  ["bundleDependencies"],
+  ["dependencies"],
+  ["description", { isRecommended: true }],
+  ["devDependencies"],
+  ["engines"],
+  ["files"],
+  ["keywords"],
+  ["name", { ignorePrivateDefault: true, isRecommended: true }],
+  ["optionalDependencies"],
+  ["peerDependencies"],
+  ["type", { isRecommended: true }],
+  ["types"],
+  ["version", { ignorePrivateDefault: true, isRecommended: true }]
 ];
-const rules = properties.reduce(
-  (acc, [propertyName, isRecommended]) => {
-    acc[`require-${propertyName}`] = createSimpleRequirePropertyRule(
-      propertyName,
-      isRecommended
-    );
-    return acc;
-  },
-  {}
+const rules = Object.fromEntries(
+  properties.map(([propertyName, options]) => [
+    `require-${propertyName}`,
+    createSimpleRequirePropertyRule(propertyName, options)
+  ])
 );
 export {
   rules

package/lib/utils/createSimpleRequirePropertyRule.d.ts

@@ -3,15 +3,28 @@ import * as jsonc_eslint_parser from 'jsonc-eslint-parser';
 import { PackageJsonRuleContext } from '../createRule.js';
 import 'estree';
 
+interface CreateRequirePropertyRuleOptions {
+    /**
+     * The default value of `ignorePrivate` rule option.
+     */
+    ignorePrivateDefault?: boolean;
+    /**
+     * Whether the rule should be included in the recommended config.
+     */
+    isRecommended?: boolean;
+}
+type Options = [{
+    ignorePrivate?: boolean;
+}?];
 /**
  * Given a top-level property name, create a rule that requires that property to be present.
  * Optionally, include it in the recommended config.
  * Note: this will only create a basic require rule, with no options.  If you need
  * to create a more complex rule, create it in its own file.
  */
-declare const createSimpleRequirePropertyRule: (propertyName: string, isRecommended?: boolean) => {
-    create(context: PackageJsonRuleContext<unknown[]>): jsonc_eslint_parser.RuleListener;
+declare const createSimpleRequirePropertyRule: (propertyName: string, { ignorePrivateDefault, isRecommended, }?: CreateRequirePropertyRuleOptions) => {
+    create(context: PackageJsonRuleContext<Options>): jsonc_eslint_parser.RuleListener;
     meta: eslint.Rule.RuleMetaData;
 };
 
-export { createSimpleRequirePropertyRule };
+export { type CreateRequirePropertyRuleOptions, createSimpleRequirePropertyRule };

package/lib/utils/createSimpleRequirePropertyRule.js

@@ -1,10 +1,20 @@
 import { createRule } from "../createRule.js";
 import { isJSONStringLiteral } from "./predicates.js";
-const createSimpleRequirePropertyRule = (propertyName, isRecommended = false) => {
+const createSimpleRequirePropertyRule = (propertyName, {
+  ignorePrivateDefault = false,
+  isRecommended
+} = {}) => {
   return createRule({
     create(context) {
+      const enforceForPrivate = context.settings.packageJson?.enforceForPrivate;
+      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
           )) {
@@ -25,7 +35,18 @@ const createSimpleRequirePropertyRule = (propertyName, isRecommended = false) =>
       messages: {
         missing: "Property '{{property}}' is required."
       },
-      schema: [],
+      schema: [
+        {
+          additionalProperties: false,
+          properties: {
+            ignorePrivate: {
+              default: ignorePrivateDefault,
+              type: "boolean"
+            }
+          },
+          type: "object"
+        }
+      ],
       type: "suggestion"
     }
   });

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-plugin-package-json",
-	"version": "0.50.0",
+	"version": "0.51.0",
 	"description": "Rules for consistent, readable, and valid package.json files. 🗂️",
 	"homepage": "https://github.com/JoshuaKGoldberg/eslint-plugin-package-json#readme",
 	"bugs": {