@monorepolint/rules 0.5.0-beta.1 → 0.5.0-beta.10

package/build/types/packageEntry.d.ts

@@ -0,0 +1,43 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+import * as r from "runtypes";
+export declare const Options: r.Union<[r.Intersect<[r.Record<{
+    entries: r.Dictionary<r.Unknown, string | number | symbol>;
+}, false>, r.Partial<{
+    entriesExist: r.Literal<undefined>;
+}, false>]>, r.Intersect<[r.Record<{
+    entriesExist: r.Array<r.String, false>;
+}, false>, r.Partial<{
+    entries: r.Literal<undefined>;
+}, false>]>, r.Record<{
+    entries: r.Dictionary<r.Unknown, string | number | symbol>;
+    entriesExist: r.Array<r.String, false>;
+}, false>]>;
+export type Options = r.Static<typeof Options>;
+export declare const packageEntry: import("./util/createRuleFactory.js").RuleFactoryFn<({
+    entries: {
+        [x: string]: unknown;
+        [x: number]: unknown;
+        [x: symbol]: unknown;
+    };
+} & {
+    entriesExist?: undefined;
+}) | ({
+    entriesExist: string[];
+} & {
+    entries?: undefined;
+}) | {
+    entries: {
+        [x: string]: unknown;
+        [x: number]: unknown;
+        [x: symbol]: unknown;
+    };
+    entriesExist: string[];
+}>;
+export declare function createStandardizedEntryErrorMessage(key: string): string;
+export declare function createExpectedEntryErrorMessage(key: string): string;
+//# sourceMappingURL=packageEntry.d.ts.map

package/build/types/packageEntry.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"packageEntry.d.ts","sourceRoot":"","sources":["../../src/packageEntry.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,CAAC,MAAM,UAAU,CAAC;AAG9B,eAAO,MAAM,OAAO;;;;;;;;;;;WAuBnB,CAAC;AAEF,MAAM,MAAM,OAAO,GAAG,CAAC,CAAC,MAAM,CAAC,OAAO,OAAO,CAAC,CAAC;AAE/C,eAAO,MAAM,YAAY;;;;;;;;;;;;;;;;;;;EA0CvB,CAAC;AAEH,wBAAgB,mCAAmC,CAAC,GAAG,EAAE,MAAM,UAE9D;AAED,wBAAgB,+BAA+B,CAAC,GAAG,EAAE,MAAM,UAE1D"}

package/build/types/packageOrder.d.ts

@@ -0,0 +1,10 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+export declare const packageOrder: import("./util/createRuleFactory.js").RuleFactoryFn<{
+    order: string[] | ((...args: any[]) => any);
+} | undefined>;
+//# sourceMappingURL=packageOrder.d.ts.map

package/build/types/packageOrder.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"packageOrder.d.ts","sourceRoot":"","sources":["../../src/packageOrder.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAgDH,eAAO,MAAM,YAAY;;cAgCvB,CAAC"}

package/build/types/packageScript.d.ts

@@ -0,0 +1,32 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+import * as r from "runtypes";
+export declare const Options: r.Record<{
+    scripts: r.Dictionary<r.Union<[r.String, r.Record<{
+        options: r.Array<r.Union<[r.String, r.Literal<undefined>]>, false>;
+        fixValue: r.Optional<r.Union<[r.String, r.Literal<undefined>, r.Literal<false>]>>;
+    }, false>]>, string | number | symbol>;
+}, false>;
+export type Options = r.Static<typeof Options>;
+export declare const MSG_NO_SCRIPTS_BLOCK = "No scripts block in package.json";
+export declare const packageScript: import("./util/createRuleFactory.js").RuleFactoryFn<{
+    scripts: {
+        [x: string]: string | {
+            options: (string | undefined)[];
+            fixValue?: string | false | undefined;
+        };
+        [x: number]: string | {
+            options: (string | undefined)[];
+            fixValue?: string | false | undefined;
+        };
+        [x: symbol]: string | {
+            options: (string | undefined)[];
+            fixValue?: string | false | undefined;
+        };
+    };
+}>;
+//# sourceMappingURL=packageScript.d.ts.map

package/build/types/packageScript.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"packageScript.d.ts","sourceRoot":"","sources":["../../src/packageScript.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,CAAC,MAAM,UAAU,CAAC;AAG9B,eAAO,MAAM,OAAO;;;;;SAUlB,CAAC;AAEH,MAAM,MAAM,OAAO,GAAG,CAAC,CAAC,MAAM,CAAC,OAAO,OAAO,CAAC,CAAC;AAE/C,eAAO,MAAM,oBAAoB,qCAAqC,CAAC;AAEvE,eAAO,MAAM,aAAa;;;;;;;;;;;;;;;EAsExB,CAAC"}

package/build/types/requireDependency.d.ts

@@ -0,0 +1,29 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+export declare const requireDependency: import("./util/createRuleFactory.js").RuleFactoryFn<{
+    dependencies?: {
+        [x: string]: string;
+        [x: number]: string;
+        [x: symbol]: string;
+    } | undefined;
+    devDependencies?: {
+        [x: string]: string;
+        [x: number]: string;
+        [x: symbol]: string;
+    } | undefined;
+    peerDependencies?: {
+        [x: string]: string;
+        [x: number]: string;
+        [x: symbol]: string;
+    } | undefined;
+    optionalDependencies?: {
+        [x: string]: string;
+        [x: number]: string;
+        [x: symbol]: string;
+    } | undefined;
+}>;
+//# sourceMappingURL=requireDependency.d.ts.map

package/build/types/requireDependency.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"requireDependency.d.ts","sourceRoot":"","sources":["../../src/requireDependency.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAiBH,eAAO,MAAM,iBAAiB;;;;;;;;;;;;;;;;;;;;;EAgD5B,CAAC"}

package/build/types/standardTsconfig.d.ts

@@ -0,0 +1,29 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+import * as r from "runtypes";
+declare const Options: r.Constraint<r.Partial<{
+    file: r.String;
+    generator: r.Function;
+    tsconfigReferenceFile: r.String;
+    template: r.Union<[r.Record<{}, false>, r.String]>;
+    templateFile: r.String;
+    excludedReferences: r.Union<[r.Array<r.String, false>, r.Literal<undefined>]>;
+    additionalReferences: r.Union<[r.Array<r.String, false>, r.Literal<undefined>]>;
+}, false>, {
+    file?: string | undefined;
+    generator?: ((...args: any[]) => any) | undefined;
+    tsconfigReferenceFile?: string | undefined;
+    template?: string | {} | undefined;
+    templateFile?: string | undefined;
+    excludedReferences?: string[] | undefined;
+    additionalReferences?: string[] | undefined;
+}, unknown>;
+export interface Options extends r.Static<typeof Options> {
+}
+export declare const standardTsconfig: import("./util/createRuleFactory.js").RuleFactoryFn<Options>;
+export {};
+//# sourceMappingURL=standardTsconfig.d.ts.map

package/build/types/standardTsconfig.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"standardTsconfig.d.ts","sourceRoot":"","sources":["../../src/standardTsconfig.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,OAAO,KAAK,CAAC,MAAM,UAAU,CAAC;AAK9B,QAAA,MAAM,OAAO;;;;;;;;;;;;;;;;WAuBT,CAAC;AAEL,MAAM,WAAW,OAAQ,SAAQ,CAAC,CAAC,MAAM,CAAC,OAAO,OAAO,CAAC;CAAG;AAE5D,eAAO,MAAM,gBAAgB,8DAkC3B,CAAC"}

package/build/types/util/checkAlpha.d.ts

@@ -0,0 +1,10 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+import { Context } from "@monorepolint/config";
+export declare function checkAlpha(context: Context, block: "dependencies" | "devDependencies" | "peerDependencies" | "scripts"): void;
+export declare function createIncorrectOrderErrorMessage(block: string, packageName: string): string;
+//# sourceMappingURL=checkAlpha.d.ts.map

package/build/types/util/checkAlpha.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"checkAlpha.d.ts","sourceRoot":"","sources":["../../../src/util/checkAlpha.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,sBAAsB,CAAC;AAG/C,wBAAgB,UAAU,CACxB,OAAO,EAAE,OAAO,EAChB,KAAK,EAAE,cAAc,GAAG,iBAAiB,GAAG,kBAAkB,GAAG,SAAS,QAgC3E;AAYD,wBAAgB,gCAAgC,CAAC,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,UAElF"}

package/build/types/util/createRuleFactory.d.ts

@@ -0,0 +1,14 @@
+import { Context, RuleEntry, RuleModule } from "@monorepolint/config";
+export type ValidateOptionsFn<X> = (options: unknown) => asserts options is X;
+export type RuleFactoryFn<T> = (ruleEntry: RuleEntry<T>) => RuleModule<T>;
+export type RuleCheckFn<O> = (context: Context, options: O, extra: {
+    id: string;
+}) => Promise<unknown> | unknown;
+export interface RuleFactoryOptions<X> {
+    name: string;
+    check: RuleCheckFn<X>;
+    printStats?: () => void;
+    validateOptions: ValidateOptionsFn<X>;
+}
+export declare function createRuleFactory<X>({ name, check, validateOptions, printStats, }: RuleFactoryOptions<X>): RuleFactoryFn<X>;
+//# sourceMappingURL=createRuleFactory.d.ts.map

package/build/types/util/createRuleFactory.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"createRuleFactory.d.ts","sourceRoot":"","sources":["../../../src/util/createRuleFactory.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAEtE,MAAM,MAAM,iBAAiB,CAAC,CAAC,IAAI,CAAC,OAAO,EAAE,OAAO,KAAK,OAAO,CAAC,OAAO,IAAI,CAAC,CAAC;AAC9E,MAAM,MAAM,aAAa,CAAC,CAAC,IAAI,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,KAAK,UAAU,CAAC,CAAC,CAAC,CAAC;AAC1E,MAAM,MAAM,WAAW,CAAC,CAAC,IAAI,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC,EAAE,KAAK,EAAE;IAAE,EAAE,EAAE,MAAM,CAAA;CAAE,KAAK,OAAO,CAAC,OAAO,CAAC,GAAG,OAAO,CAAC;AAEjH,MAAM,WAAW,kBAAkB,CAAC,CAAC;IACnC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,IAAI,CAAC;IACxB,eAAe,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC;CACvC;AAGD,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,EACnC,IAAI,EACJ,KAAK,EACL,eAAe,EACf,UAAU,GACX,EAAE,kBAAkB,CAAC,CAAC,CAAC,GAAG,aAAa,CAAC,CAAC,CAAC,CAa1C"}

package/build/types/util/makeDirectory.d.ts

@@ -0,0 +1,8 @@
+/*!
+ * Copyright 2019 Palantir Technologies, Inc.
+ *
+ * Licensed under the MIT license. See LICENSE file in the project root for details.
+ *
+ */
+export declare function makeDirectoryRecursively(directoryPath: string): void;
+//# sourceMappingURL=makeDirectory.d.ts.map

package/build/types/util/makeDirectory.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"makeDirectory.d.ts","sourceRoot":"","sources":["../../../src/util/makeDirectory.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,wBAAgB,wBAAwB,CAAC,aAAa,EAAE,MAAM,QAa7D"}

package/build/types/util/packageDependencyGraphService.d.ts

@@ -0,0 +1,37 @@
+/**
+ * @license Copyright 2019 Palantir Technologies, Inc. All rights reserved.
+ */
+import { Host, PackageJson } from "@monorepolint/utils";
+/** Interface for a node in a package dependency graph. */
+export interface IPackageDependencyGraphNode {
+    packageJson: PackageJson;
+    dependencies: Map<string, IPackageDependencyGraphNode>;
+    paths: {
+        packageJsonPath: string;
+        rootDirectory: string;
+    };
+}
+/** Service abstraction for constructing and traversing package dependency graphs. */
+export interface IPackageDependencyGraphService {
+    /** Construct a graph of package dependencies. */
+    buildDependencyGraph(packageJsonPath: string, host: Host, maxDepth?: number): IPackageDependencyGraphNode;
+    /** Traverse a package dependency graph. */
+    traverse(root: IPackageDependencyGraphNode, opts?: {
+        /** Traverse each unique path to a given package (potentially slow). */
+        traverseAllPaths?: boolean;
+    }): IterableIterator<IPackageDependencyGraphNode & {
+        importPath: IPackageDependencyGraphNode[];
+    }>;
+}
+/** Default implementation of the package dependency graph service. */
+export declare class PackageDependencyGraphService implements IPackageDependencyGraphService {
+    /** Construct a graph of package dependencies and return the root node. */
+    buildDependencyGraph(startPackageJsonPath: string, host: Host, maxDepth?: number): IPackageDependencyGraphNode;
+    /** Traverse a package dependency graph with an iterator. */
+    traverse(root: IPackageDependencyGraphNode, opts?: {
+        traverseAllPaths: boolean;
+    }): IterableIterator<IPackageDependencyGraphNode & {
+        importPath: IPackageDependencyGraphNode[];
+    }>;
+}
+//# sourceMappingURL=packageDependencyGraphService.d.ts.map

package/build/types/util/packageDependencyGraphService.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"packageDependencyGraphService.d.ts","sourceRoot":"","sources":["../../../src/util/packageDependencyGraphService.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAIxD,0DAA0D;AAC1D,MAAM,WAAW,2BAA2B;IAC1C,WAAW,EAAE,WAAW,CAAC;IACzB,YAAY,EAAE,GAAG,CAAC,MAAM,EAAE,2BAA2B,CAAC,CAAC;IACvD,KAAK,EAAE;QACL,eAAe,EAAE,MAAM,CAAC;QACxB,aAAa,EAAE,MAAM,CAAC;KACvB,CAAC;CACH;AAED,qFAAqF;AACrF,MAAM,WAAW,8BAA8B;IAC7C,iDAAiD;IACjD,oBAAoB,CAAC,eAAe,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,CAAC,EAAE,MAAM,GAAG,2BAA2B,CAAC;IAE1G,2CAA2C;IAC3C,QAAQ,CACN,IAAI,EAAE,2BAA2B,EACjC,IAAI,CAAC,EAAE;QACL,uEAAuE;QACvE,gBAAgB,CAAC,EAAE,OAAO,CAAC;KAC5B,GACA,gBAAgB,CAAC,2BAA2B,GAAG;QAAE,UAAU,EAAE,2BAA2B,EAAE,CAAA;KAAE,CAAC,CAAC;CAClG;AAED,sEAAsE;AACtE,qBAAa,6BAA8B,YAAW,8BAA8B;IAClF,0EAA0E;IACnE,oBAAoB,CACzB,oBAAoB,EAAE,MAAM,EAC5B,IAAI,EAAE,IAAI,EACV,QAAQ,CAAC,EAAE,MAAM,GAChB,2BAA2B;IAuC9B,4DAA4D;IACpD,QAAQ,CACd,IAAI,EAAE,2BAA2B,EACjC,IAAI;;KAA8B,GACjC,gBAAgB,CAAC,2BAA2B,GAAG;QAAE,UAAU,EAAE,2BAA2B,EAAE,CAAA;KAAE,CAAC;CA8BjG"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@monorepolint/rules",
-  "version": "0.5.0-beta.1",
+  "version": "0.5.0-beta.10",
   "author": "Eric L Anderson (https://github.com/ericanderson)",
   "contributors": [
     "Mac Lockard (https://github.com/maclockard)"
@@ -28,9 +28,9 @@
     "runtypes": "^6.7.0",
     "semver": "^7.6.0",
     "tslib": "^2.6.2",
-    "@monorepolint/core": "^0.5.0-beta.1",
-    "@monorepolint/config": "^0.5.0-beta.1",
-    "@monorepolint/utils": "^0.5.0-beta.1"
+    "@monorepolint/core": "^0.5.0-beta.10",
+    "@monorepolint/config": "^0.5.0-beta.10",
+    "@monorepolint/utils": "^0.5.0-beta.10"
   },
   "devDependencies": {
     "@jest/globals": "^29.7.0",

package/src/alphabeticalDependencies.ts

@@ -6,9 +6,9 @@
  */
 
 import { checkAlpha } from "./util/checkAlpha.js";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
-export const alphabeticalDependencies = makeRule<undefined>({
+export const alphabeticalDependencies = createRuleFactory<undefined>({
   name: "alphabeticalDependencies",
   check: (context) => {
     checkAlpha(context, "dependencies");

package/src/alphabeticalScripts.ts

@@ -6,9 +6,9 @@
  */
 
 import { checkAlpha } from "./util/checkAlpha.js";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
-export const alphabeticalScripts = makeRule<undefined>({
+export const alphabeticalScripts = createRuleFactory<undefined>({
   name: "alphabeticalScripts",
   check: (context) => {
     checkAlpha(context, "scripts");

package/src/bannedDependencies.ts

@@ -10,7 +10,7 @@ import { matchesAnyGlob } from "@monorepolint/utils";
 import { AggregateTiming } from "@monorepolint/utils";
 import * as path from "node:path";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 import { IPackageDependencyGraphNode, PackageDependencyGraphService } from "./util/packageDependencyGraphService.js";
 // FIXME: This rule is messed. bannedTransitiveDependencies doesnt glob
 
@@ -49,7 +49,7 @@ const setCache = new Map<ReadonlyArray<string>, Set<string>>();
 
 const aggregateTiming = new AggregateTiming(":bannedDependencies stats");
 
-export const bannedDependencies = makeRule<Options>({
+export const bannedDependencies = createRuleFactory<Options>({
   name: "bannedDependencies",
   check: (context, opts, extra) => {
     aggregateTiming.start(extra?.id ?? "unknown id");

package/src/consistentDependencies.ts

@@ -8,7 +8,7 @@
 import { Context } from "@monorepolint/config";
 import { diff } from "jest-diff";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 const Options = r
   .Record({
     ignoredDependencies: r.Array(r.String).Or(r.Undefined),
@@ -18,7 +18,7 @@ export type Options = r.Static<typeof Options>;
 
 const skippedVersions = ["*", "latest"];
 
-export const consistentDependencies = makeRule<Options>({
+export const consistentDependencies = createRuleFactory<Options>({
   name: "consistentDependencies",
   check: (context, args) => {
     checkDeps(context, args, "dependencies");

package/src/consistentVersions.ts

@@ -9,14 +9,14 @@ import { Context } from "@monorepolint/config";
 import { mutateJson, PackageJson } from "@monorepolint/utils";
 import * as r from "runtypes";
 import { coerce, SemVer } from "semver";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 export const Options = r.Record({
   matchDependencyVersions: r.Dictionary(r.Union(r.String, r.Array(r.String))),
 });
 
 export type Options = r.Static<typeof Options>;
 
-export const consistentVersions = makeRule({
+export const consistentVersions = createRuleFactory({
   name: "consistentVersions",
   check: checkConsistentVersions,
   validateOptions: Options.check,

package/src/fileContents.ts

@@ -9,7 +9,7 @@ import { Context } from "@monorepolint/config";
 import { diff } from "jest-diff";
 import * as path from "path";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 const Options = r.Union(
   r.Record({
     file: r.String,
@@ -35,7 +35,7 @@ const Options = r.Union(
 
 type Options = r.Static<typeof Options>;
 
-export const fileContents = makeRule<Options>({
+export const fileContents = createRuleFactory<Options>({
   name: "fileContents",
   check: async (context, opts) => {
     const fullPath = path.join(context.packageDir, opts.file);

package/src/index.ts

@@ -18,3 +18,11 @@ export { packageScript } from "./packageScript.js";
 export { standardTsconfig } from "./standardTsconfig.js";
 export { nestedWorkspaces } from "./nestedWorkspaces.js";
 export { requireDependency } from "./requireDependency.js";
+
+export {
+  createRuleFactory,
+  RuleCheckFn,
+  RuleFactoryFn,
+  RuleFactoryOptions,
+  ValidateOptionsFn,
+} from "./util/createRuleFactory.js";

package/src/mustSatisfyPeerDependencies.ts

@@ -11,7 +11,7 @@ import * as path from "node:path";
 import * as r from "runtypes";
 import { coerce } from "semver";
 import resolvePackagePath from "resolve-package-path";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 const Options = r.Union(
   r.Partial({
@@ -184,7 +184,7 @@ const Options = r.Union(
 
 export type Options = r.Static<typeof Options>;
 
-export const mustSatisfyPeerDependencies = makeRule({
+export const mustSatisfyPeerDependencies = createRuleFactory({
   name: "mustSatisfyPeerDependencies",
   check: checkSatisfyPeerDependencies,
   validateOptions: Options.check,

package/src/nestedWorkspaces.ts

@@ -8,14 +8,14 @@
 import * as globby from "globby";
 import * as path from "node:path";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 export const Options = r.Undefined;
 
 type Options = r.Static<typeof Options>;
 
 // Enforce that the root package.json contains all of the workspaces in the repo (including nested packages)
-export const nestedWorkspaces = makeRule({
+export const nestedWorkspaces = createRuleFactory({
   name: "nestedWorkspaces",
   check: (context) => {
     const rootPackageJson = context.getWorkspaceContext().getPackageJson();
@@ -26,8 +26,8 @@ export const nestedWorkspaces = makeRule({
     const workspaces = Array.isArray(rootPackageJson.workspaces)
       ? rootPackageJson.workspaces
       : rootPackageJson.workspaces !== undefined
-      ? rootPackageJson.workspaces.packages
-      : undefined;
+        ? rootPackageJson.workspaces.packages
+        : undefined;
 
     if (workspaces === undefined && packageJsonPaths.length > 0) {
       context.addError({

package/src/packageEntry.ts

@@ -8,7 +8,7 @@
 import { mutateJson, PackageJson } from "@monorepolint/utils";
 import { diff } from "jest-diff";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 export const Options = r.Union(
   r
@@ -37,7 +37,7 @@ export const Options = r.Union(
 
 export type Options = r.Static<typeof Options>;
 
-export const packageEntry = makeRule<Options>({
+export const packageEntry = createRuleFactory<Options>({
   name: "packageEntry",
   check: (context, options) => {
     const packageJson = context.getPackageJson();

package/src/packageOrder.ts

@@ -8,7 +8,7 @@
 import { Context } from "@monorepolint/config";
 import { diff } from "jest-diff";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 type OrderFunction = (context: Context) => (a: string, b: string) => number;
 
 const Options = r
@@ -51,7 +51,7 @@ const defaultKeyOrder = [
   "publishConfig",
 ];
 
-export const packageOrder = makeRule<Options>({
+export const packageOrder = createRuleFactory<Options>({
   name: "packageOrder",
   check: (context, opts) => {
     const packageJson = context.getPackageJson();

package/src/packageScript.ts

@@ -8,7 +8,7 @@
 import { mutateJson, PackageJson } from "@monorepolint/utils";
 import { diff } from "jest-diff";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 export const Options = r.Record({
   scripts: r.Dictionary(
@@ -26,7 +26,7 @@ export type Options = r.Static<typeof Options>;
 
 export const MSG_NO_SCRIPTS_BLOCK = "No scripts block in package.json";
 
-export const packageScript = makeRule<Options>({
+export const packageScript = createRuleFactory<Options>({
   name: "packageScript",
   check: (context, options) => {
     const packageJson = context.getPackageJson();

package/src/requireDependency.ts

@@ -9,7 +9,7 @@ import { Context } from "@monorepolint/config";
 import { mutateJson, PackageJson } from "@monorepolint/utils";
 import { diff } from "jest-diff";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 const Options = r.Partial({
   dependencies: r.Dictionary(r.String),
@@ -20,7 +20,7 @@ const Options = r.Partial({
 
 type Options = r.Static<typeof Options>;
 
-export const requireDependency = makeRule({
+export const requireDependency = createRuleFactory({
   name: "requireDependency",
   check: function expectPackageEntry(context: Context, options: Options) {
     const packageJson = context.getPackageJson();

package/src/standardTsconfig.ts

@@ -10,7 +10,7 @@ import { matchesAnyGlob } from "@monorepolint/utils";
 import { diff } from "jest-diff";
 import * as path from "path";
 import * as r from "runtypes";
-import { makeRule } from "./util/makeRule.js";
+import { createRuleFactory } from "./util/createRuleFactory.js";
 
 const DEFAULT_TSCONFIG_FILENAME = "tsconfig.json";
 
@@ -41,7 +41,7 @@ const Options = r
 
 export interface Options extends r.Static<typeof Options> {}
 
-export const standardTsconfig = makeRule<Options>({
+export const standardTsconfig = createRuleFactory<Options>({
   name: "standardTsconfig",
   check: async (context, opts) => {
     const tsconfigFileName = opts.file ?? DEFAULT_TSCONFIG_FILENAME;

package/src/util/createRuleFactory.ts

@@ -0,0 +1,33 @@
+import { Context, RuleEntry, RuleModule } from "@monorepolint/config";
+
+export type ValidateOptionsFn<X> = (options: unknown) => asserts options is X;
+export type RuleFactoryFn<T> = (ruleEntry: RuleEntry<T>) => RuleModule<T>;
+export type RuleCheckFn<O> = (context: Context, options: O, extra: { id: string }) => Promise<unknown> | unknown;
+
+export interface RuleFactoryOptions<X> {
+  name: string;
+  check: RuleCheckFn<X>;
+  printStats?: () => void;
+  validateOptions: ValidateOptionsFn<X>;
+}
+
+let globalId = 0;
+export function createRuleFactory<X>({
+  name,
+  check,
+  validateOptions,
+  printStats,
+}: RuleFactoryOptions<X>): RuleFactoryFn<X> {
+  return function (ruleEntry) {
+    const id = ruleEntry.id ?? `${name} :: ${globalId++}`;
+    return {
+      id,
+      // eslint-disable-next-line @typescript-eslint/no-extra-non-null-assertion
+      check: (context) => check(context, ruleEntry.options!!, { id }),
+      name,
+      validateOptions,
+      ruleEntry,
+      printStats,
+    };
+  };
+}

package/src/public/util.ts

@@ -1 +0,0 @@
-export { makeRule } from "../util/makeRule.js";

package/src/util/makeRule.ts

@@ -1,29 +0,0 @@
-import { Context, RuleEntry, RuleModule } from "@monorepolint/config";
-
-let globalId = 0;
-export function makeRule<X>({
-  name,
-  check,
-  validateOptions,
-  printStats,
-}: {
-  name: string;
-  check: (context: Context, options: X, extra: { id: string }) => Promise<unknown> | unknown;
-  printStats?: () => void;
-  validateOptions: (options: unknown) => asserts options is X;
-}): RuleFunction<X> {
-  return function (ruleEntry) {
-    const id = `${name} :: ${globalId++}`;
-    return {
-      id,
-      // eslint-disable-next-line @typescript-eslint/no-extra-non-null-assertion
-      check: (context) => check(context, ruleEntry.options!!, { id }),
-      name,
-      validateOptions,
-      ruleEntry,
-      printStats,
-    };
-  };
-}
-
-type RuleFunction<T> = (ruleEntry: RuleEntry<T>) => RuleModule<T>;