@monorepolint/rules 0.6.0-alpha.2 → 0.6.0-alpha.3

package/coverage/src/util/packageDependencyGraphService.ts.html

@@ -23,9 +23,9 @@
         <div class='clearfix'>
             
             <div class='fl pad1y space-right2'>
-                <span class="strong">87.09% </span>
+                <span class="strong">87.17% </span>
                 <span class="quiet">Statements</span>
-                <span class='fraction'>54/62</span>
+                <span class='fraction'>68/78</span>
             </div>
         
             
@@ -44,9 +44,9 @@
         
             
             <div class='fl pad1y space-right2'>
-                <span class="strong">87.09% </span>
+                <span class="strong">87.17% </span>
                 <span class="quiet">Lines</span>
-                <span class='fraction'>54/62</span>
+                <span class='fraction'>68/78</span>
             </div>
         
             
@@ -177,7 +177,41 @@
 <a name='L112'></a><a href='#L112'>112</a>
 <a name='L113'></a><a href='#L113'>113</a>
 <a name='L114'></a><a href='#L114'>114</a>
-<a name='L115'></a><a href='#L115'>115</a></td><td class="line-coverage quiet"><span class="cline-any cline-yes">1x</span>
+<a name='L115'></a><a href='#L115'>115</a>
+<a name='L116'></a><a href='#L116'>116</a>
+<a name='L117'></a><a href='#L117'>117</a>
+<a name='L118'></a><a href='#L118'>118</a>
+<a name='L119'></a><a href='#L119'>119</a>
+<a name='L120'></a><a href='#L120'>120</a>
+<a name='L121'></a><a href='#L121'>121</a>
+<a name='L122'></a><a href='#L122'>122</a>
+<a name='L123'></a><a href='#L123'>123</a>
+<a name='L124'></a><a href='#L124'>124</a>
+<a name='L125'></a><a href='#L125'>125</a>
+<a name='L126'></a><a href='#L126'>126</a>
+<a name='L127'></a><a href='#L127'>127</a>
+<a name='L128'></a><a href='#L128'>128</a>
+<a name='L129'></a><a href='#L129'>129</a>
+<a name='L130'></a><a href='#L130'>130</a>
+<a name='L131'></a><a href='#L131'>131</a>
+<a name='L132'></a><a href='#L132'>132</a>
+<a name='L133'></a><a href='#L133'>133</a>
+<a name='L134'></a><a href='#L134'>134</a>
+<a name='L135'></a><a href='#L135'>135</a>
+<a name='L136'></a><a href='#L136'>136</a>
+<a name='L137'></a><a href='#L137'>137</a>
+<a name='L138'></a><a href='#L138'>138</a>
+<a name='L139'></a><a href='#L139'>139</a>
+<a name='L140'></a><a href='#L140'>140</a>
+<a name='L141'></a><a href='#L141'>141</a>
+<a name='L142'></a><a href='#L142'>142</a></td><td class="line-coverage quiet"><span class="cline-any cline-yes">1x</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-yes">1x</span>
+<span class="cline-any cline-yes">1x</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -210,6 +244,7 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -221,6 +256,9 @@
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">5x</span>
+<span class="cline-any cline-yes">5x</span>
+<span class="cline-any cline-yes">5x</span>
+<span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -240,14 +278,24 @@
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-yes">5x</span>
+<span class="cline-any cline-yes">3x</span>
+<span class="cline-any cline-yes">2x</span>
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-yes">4x</span>
 <span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">4x</span>
 <span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
+<span class="cline-any cline-yes">4x</span>
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">5x</span>
@@ -260,12 +308,18 @@
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">1x</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">1x</span>
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-yes">5x</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">5x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">5x</span>
@@ -312,7 +366,11 @@ export interface IPackageDependencyGraphNode {
 /** 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;
+  buildDependencyGraph(
+    packageJsonPath: string,
+    host: Host,
+    maxDepth?: number,
+  ): IPackageDependencyGraphNode;
 &nbsp;
   /** Traverse a package dependency graph. */
   traverse(
@@ -320,20 +378,27 @@ export interface IPackageDependencyGraphService {
     opts?: {
       /** Traverse each unique path to a given package (potentially slow). */
       traverseAllPaths?: boolean;
-    }
-  ): IterableIterator&lt;IPackageDependencyGraphNode &amp; { importPath: IPackageDependencyGraphNode[] }&gt;;
+    },
+  ): IterableIterator&lt;
+    IPackageDependencyGraphNode &amp; { importPath: IPackageDependencyGraphNode[] }
+  &gt;;
 }
 &nbsp;
 /** Default implementation of the package dependency graph service. */
-export class PackageDependencyGraphService implements IPackageDependencyGraphService {
+export class PackageDependencyGraphService
+  implements IPackageDependencyGraphService
+{
   /** Construct a graph of package dependencies and return the root node. */
   public buildDependencyGraph(
     startPackageJsonPath: string,
     host: Host,
-    maxDepth?: number
+    maxDepth?: number,
   ): IPackageDependencyGraphNode {
     const nodes = new Map&lt;string, IPackageDependencyGraphNode&gt;();
-    const visit = (packageJsonPath: string, currentDepth: number): IPackageDependencyGraphNode =&gt; {
+    const visit = (
+      packageJsonPath: string,
+      currentDepth: number,
+    ): IPackageDependencyGraphNode =&gt; {
       if (nodes.has(packageJsonPath)) <span class="branch-0 cbranch-no" title="branch not covered" >{</span>
 <span class="cstat-no" title="statement not covered" >        return nodes.get(packageJsonPath)!;</span>
 <span class="cstat-no" title="statement not covered" >      }</span>
@@ -353,14 +418,24 @@ export class PackageDependencyGraphService implements IPackageDependencyGraphSer
 &nbsp;
       const nextDepth = currentDepth + 1;
       if (maxDepth == <span class="branch-0 cbranch-no" title="branch not covered" >null || nextDepth &lt;= maxDepth) {</span>
-        const dependencies = packageJson.dependencies != null ? Object.keys(packageJson.dependencies) : [];
+        const dependencies = packageJson.dependencies != null
+          ? Object.keys(packageJson.dependencies)
+          : [];
         for (const dependency of dependencies) {
-          const dependencyPackageJsonPath = resolvePackagePath(dependency, node.paths.rootDirectory);
+          const dependencyPackageJsonPath = resolvePackagePath(
+            dependency,
+            node.paths.rootDirectory,
+          );
           if (dependencyPackageJsonPath == null) <span class="branch-0 cbranch-no" title="branch not covered" >{</span>
-<span class="cstat-no" title="statement not covered" >            throw new Error(`Could not resolve ${dependency} from ${node.paths.rootDirectory}`);</span>
+<span class="cstat-no" title="statement not covered" >            throw new Error(</span>
+<span class="cstat-no" title="statement not covered" >              `Could not resolve ${dependency} from ${node.paths.rootDirectory}`,</span>
+<span class="cstat-no" title="statement not covered" >            );</span>
 <span class="cstat-no" title="statement not covered" >          }</span>
 &nbsp;
-          node.dependencies.set(dependency, visit(dependencyPackageJsonPath, nextDepth));
+          node.dependencies.set(
+            dependency,
+            visit(dependencyPackageJsonPath, nextDepth),
+          );
         }
       }
 &nbsp;
@@ -373,14 +448,20 @@ export class PackageDependencyGraphService implements IPackageDependencyGraphSer
   /** Traverse a package dependency graph with an iterator. */
   public *traverse(
     root: IPackageDependencyGraphNode,
-    opts = { traverseAllPaths: false }
-  ): IterableIterator&lt;IPackageDependencyGraphNode &amp; { importPath: IPackageDependencyGraphNode[] }&gt; {
+    opts = { traverseAllPaths: false },
+  ): IterableIterator&lt;
+    IPackageDependencyGraphNode &amp; { importPath: IPackageDependencyGraphNode[] }
+  &gt; {
     const visited = new Set&lt;IPackageDependencyGraphNode&gt;();
 &nbsp;
     function* visit(
       node: IPackageDependencyGraphNode,
-      importPath: IPackageDependencyGraphNode[] = []
-    ): IterableIterator&lt;IPackageDependencyGraphNode &amp; { importPath: IPackageDependencyGraphNode[] }&gt; {
+      importPath: IPackageDependencyGraphNode[] = [],
+    ): IterableIterator&lt;
+      IPackageDependencyGraphNode &amp; {
+        importPath: IPackageDependencyGraphNode[];
+      }
+    &gt; {
       // Don't visit a package more than once unless explicitly asked to traverse all paths
       if (!opts.traverseAllPaths &amp;&amp; visited.has(node)) <span class="branch-0 cbranch-no" title="branch not covered" >{</span>
 <span class="cstat-no" title="statement not covered" >        return;</span>
@@ -412,7 +493,7 @@ export class PackageDependencyGraphService implements IPackageDependencyGraphSer
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2025-01-17T16:18:41.406Z
+                at 2025-01-22T17:39:11.886Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@monorepolint/rules",
-  "version": "0.6.0-alpha.2",
+  "version": "0.6.0-alpha.3",
   "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.3",
     "tslib": "^2.8.1",
-    "@monorepolint/core": "^0.6.0-alpha.2",
-    "@monorepolint/utils": "^0.6.0-alpha.2",
-    "@monorepolint/config": "^0.6.0-alpha.2"
+    "@monorepolint/core": "^0.6.0-alpha.3",
+    "@monorepolint/utils": "^0.6.0-alpha.3",
+    "@monorepolint/config": "^0.6.0-alpha.3"
   },
   "devDependencies": {
     "@types/semver": "^7.5.8",
@@ -38,7 +38,6 @@
     "@typescript-eslint/eslint-plugin": "^8.20.0",
     "@typescript-eslint/parser": "^8.20.0",
     "eslint": "^9.18.0",
-    "prettier": "^3.4.2",
     "tmp": "^0.2.3",
     "tslib": "^2.8.1",
     "tsup": "^8.3.5",

package/src/__tests__/alphabeticalScripts.spec.ts

@@ -6,11 +6,17 @@
  */
 
 // tslint:disable:no-console
-import { describe, expect, it, beforeEach, vi } from "vitest";
-import { AddErrorSpy, createTestingWorkspace, HOST_FACTORIES, jsonToString, TestingWorkspace } from "./utils.js";
 import { Context, Failure } from "@monorepolint/config";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { alphabeticalScripts } from "../alphabeticalScripts.js";
 import { createIncorrectOrderErrorMessage } from "../util/checkAlpha.js";
+import {
+  AddErrorSpy,
+  createTestingWorkspace,
+  HOST_FACTORIES,
+  jsonToString,
+  TestingWorkspace,
+} from "./utils.js";
 
 const PACKAGE_SCRIPTS_SORTED = jsonToString({
   name: "foo-lib",
@@ -59,10 +65,12 @@ describe.each(HOST_FACTORIES)("alphabeticalScripts ($name)", (hostFactory) => {
           file: "package.json",
           hasFixer: true,
           message: createIncorrectOrderErrorMessage("scripts", "foo-lib"),
-        })
+        }),
       );
 
-      expect(workspace.readFile("package.json")).toEqual(PACKAGE_SCRIPTS_SORTED);
+      expect(workspace.readFile("package.json")).toEqual(
+        PACKAGE_SCRIPTS_SORTED,
+      );
     });
 
     it("does nothing if already sorted", () => {

package/src/__tests__/bannedDependencies.spec.ts

@@ -4,12 +4,12 @@
  * Licensed under the MIT license. See LICENSE file in the project root for details.
  *
  */
-import { describe, expect, it, beforeEach, vi, afterEach } from "vitest";
 import { WorkspaceContextImpl } from "@monorepolint/core";
 import { SimpleHost } from "@monorepolint/utils";
 import { writeFileSync } from "fs";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { bannedDependencies, Options } from "../bannedDependencies.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
 import { jsonToString } from "./utils.js";
@@ -46,7 +46,7 @@ describe("bannedDependencies", () => {
         verbose: false,
         silent: true,
       },
-      new SimpleHost()
+      new SimpleHost(),
     );
 
     function checkAndSpy(options: Options) {
@@ -77,15 +77,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: ["ccc"] });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: ["ccc"],
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: ["ddd"] });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: ["ddd"],
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: ["ccc", "ddd"] });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: ["ccc", "ddd"],
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -100,15 +106,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: ["c*c"] });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: ["c*c"],
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: ["d*d"] });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: ["d*d"],
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: ["c*c", "d*d"] });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: ["c*c", "d*d"],
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -123,15 +135,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: { glob: ["c*c"] } });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: { glob: ["c*c"] },
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: { glob: ["d*d"] } });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: { glob: ["d*d"] },
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: { glob: ["c*c", "d*d"] } });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: { glob: ["c*c", "d*d"] },
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -146,15 +164,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: { exact: ["ccc"] } });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: { exact: ["ccc"] },
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: { exact: ["ddd"] } });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: { exact: ["ddd"] },
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: { exact: ["ccc", "ddd"] } });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: { exact: ["ccc", "ddd"] },
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -181,9 +205,14 @@ describe("bannedDependencies", () => {
       },
     });
     addFile("./node_modules/aaa/node_modules/bbb/package.json", bbbPackageJson);
-    addFile("./node_modules/aaa/node_modules/bbb/node_modules/ddd/package.json", EMPTY_PACKAGE);
+    addFile(
+      "./node_modules/aaa/node_modules/bbb/node_modules/ddd/package.json",
+      EMPTY_PACKAGE,
+    );
     addFile("./node_modules/aaa/node_modules/ccc/package.json", EMPTY_PACKAGE);
 
-    expect(checkAndSpy({ bannedTransitiveDependencies: ["ccc", "ddd"] }).addErrorSpy).toHaveBeenCalledTimes(2);
+    expect(
+      checkAndSpy({ bannedTransitiveDependencies: ["ccc", "ddd"] }).addErrorSpy,
+    ).toHaveBeenCalledTimes(2);
   });
 });

package/src/__tests__/consistentDependencies.spec.ts

@@ -9,10 +9,10 @@ import { SimpleHost } from "@monorepolint/utils";
 import { readFileSync, writeFileSync } from "fs";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, describe, expect, it, vi } from "vitest";
 import { consistentDependencies, Options } from "../consistentDependencies.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
 import { jsonToString } from "./utils.js";
-import { describe, expect, it, afterEach, vi } from "vitest";
 
 const PACKAGE_ROOT = jsonToString({
   workspaces: {
@@ -71,11 +71,13 @@ describe("consistentDependencies", () => {
         verbose: false,
         silent: true,
       },
-      new SimpleHost()
+      new SimpleHost(),
     );
 
     function checkAndSpy(q: string, opts?: Options) {
-      const context = workspaceContext.createChildContext(path.resolve(dir.name, q));
+      const context = workspaceContext.createChildContext(
+        path.resolve(dir.name, q),
+      );
       const addErrorSpy = vi.spyOn(context, "addError");
       consistentDependencies({ options: opts }).check(context);
       return { context, addErrorSpy };
@@ -97,7 +99,9 @@ describe("consistentDependencies", () => {
   }
 
   it("checks correctly", () => {
-    const { addFile, workspaceContext, checkAndSpy } = makeWorkspace({ fix: false });
+    const { addFile, workspaceContext, checkAndSpy } = makeWorkspace({
+      fix: false,
+    });
     addFile("./package.json", PACKAGE_ROOT);
     addFile("./packages/star/package.json", PACKAGE_CHILD_WITH_STAR);
     addFile("./packages/latest/package.json", PACKAGE_CHILD_WITH_LATEST);
@@ -136,7 +140,9 @@ describe("consistentDependencies", () => {
     addFile("./package.json", PACKAGE_ROOT);
     addFile("./packages/wrong/package.json", PACKAGE_CHILD_WITH_WRONG_VERSION);
 
-    const ignored = checkAndSpy("./packages/wrong", { ignoredDependencies: ["foo"] });
+    const ignored = checkAndSpy("./packages/wrong", {
+      ignoredDependencies: ["foo"],
+    });
     expect(ignored.addErrorSpy).toHaveBeenCalledTimes(0);
   });
 });

package/src/__tests__/consistentVersions.spec.ts

@@ -9,9 +9,9 @@ import { WorkspaceContextImpl } from "@monorepolint/core";
 import { Host, PackageJson, SimpleHost } from "@monorepolint/utils";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { consistentVersions, Options } from "../consistentVersions.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
-import { describe, expect, it, beforeEach, afterEach, vi } from "vitest";
 
 describe("consistentVersions", () => {
   tmp.setGracefulCleanup();
@@ -45,7 +45,7 @@ describe("consistentVersions", () => {
         verbose: false,
         silent: true,
       },
-      host
+      host,
     );
     const addErrorSpy = vi.spyOn(workspaceContext, "addError");
 
@@ -56,7 +56,11 @@ describe("consistentVersions", () => {
     return { addErrorSpy, check, host };
   }
 
-  function addPackageJson(host: Host, filePath: string, packageJson: PackageJson) {
+  function addPackageJson(
+    host: Host,
+    filePath: string,
+    packageJson: PackageJson,
+  ) {
     const dirPath = path.resolve(cwd!, path.dirname(filePath));
     const resolvedFilePath = path.resolve(cwd!, filePath);
 
@@ -99,15 +103,24 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect dependency version", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredGreatLibVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
       check({
-        matchDependencyVersions: { both: testPackageJson.dependencies!.both, greatLib: requiredGreatLibVersion },
+        matchDependencyVersions: {
+          both: testPackageJson.dependencies!.both,
+          greatLib: requiredGreatLibVersion,
+        },
       });
       expect(addErrorSpy).toHaveBeenCalledTimes(1);
-      expect(readTestPackageJson().dependencies!.greatLib).toEqual(requiredGreatLibVersion);
+      expect(readTestPackageJson().dependencies!.greatLib).toEqual(
+        requiredGreatLibVersion,
+      );
     });
 
     it("Ignores packages that have a correct dependency version", async () => {
@@ -126,13 +139,24 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect devDependency version", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredElseLibVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { both: testPackageJson.dependencies!.both, else: requiredElseLibVersion } });
+      check({
+        matchDependencyVersions: {
+          both: testPackageJson.dependencies!.both,
+          else: requiredElseLibVersion,
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(1);
-      expect(readTestPackageJson().devDependencies!.else).toEqual(requiredElseLibVersion);
+      expect(readTestPackageJson().devDependencies!.else).toEqual(
+        requiredElseLibVersion,
+      );
     });
 
     it("Ignores packages that have a correct devDependency version", async () => {
@@ -151,14 +175,22 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect dependency and devDependency versions", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredBothVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
       check({ matchDependencyVersions: { both: requiredBothVersion } });
       expect(addErrorSpy).toHaveBeenCalledTimes(2);
-      expect(readTestPackageJson().dependencies!.both).toEqual(requiredBothVersion);
-      expect(readTestPackageJson().devDependencies!.both).toEqual(requiredBothVersion);
+      expect(readTestPackageJson().dependencies!.both).toEqual(
+        requiredBothVersion,
+      );
+      expect(readTestPackageJson().devDependencies!.both).toEqual(
+        requiredBothVersion,
+      );
     });
   });
 
@@ -187,13 +219,35 @@ describe("consistentVersions", () => {
       addPackageJson(host, "./package.json", testPackageJson);
 
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: [testPackageJson.dependencies!.greatLib] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [testPackageJson.dependencies!.greatLib],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: ["1", "2", testPackageJson.dependencies!.greatLib] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: ["1", "2", testPackageJson.dependencies!.greatLib],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: ["1", "2", testPackageJson.dependencies!.greatLib, "99", "100"] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [
+            "1",
+            "2",
+            testPackageJson.dependencies!.greatLib,
+            "99",
+            "100",
+          ],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: [testPackageJson.dependencies!.greatLib, "99", "100"] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [testPackageJson.dependencies!.greatLib, "99", "100"],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
     });
 
@@ -212,12 +266,12 @@ describe("consistentVersions", () => {
       expect(addErrorSpy.mock.calls[0][0].message).toEqual(
         `Expected dependency on both to match one of '["99","100"]', got '${
           testPackageJson.dependencies!.both
-        }' instead.`
+        }' instead.`,
       );
       expect(addErrorSpy.mock.calls[1][0].message).toEqual(
         `Expected devDependency on both to match one of '["99","100"]', got '${
           testPackageJson.devDependencies!.both
-        }' instead.`
+        }' instead.`,
       );
     });
   });