Move shared error classes

src/error.ts

@@ -1,3 +1,5 @@
+import { quote } from "./shell.js";
+
 export function normalize(error: unknown): Error {
   return error instanceof Error ? error : new Error(String(error));
 }
@@ -10,3 +12,22 @@ export class SpecError extends Error {
     super(`specification for ${name} is invalid: ${cause.message}`);
   }
 }
+
+export class ValueError extends Error {
+  constructor(
+    public readonly name: string,
+    public readonly isSensitive: boolean,
+    public readonly value: string,
+    public readonly cause: Error,
+  ) {
+    const renderedValue = isSensitive ? "<sensitive value>" : quote(value);
+
+    super(`value of ${name} (${renderedValue}) is invalid: ${cause.message}`);
+  }
+}
+
+export class NotSetError extends Error {
+  constructor(public readonly name: string) {
+    super(`${name} is not set and does not have a default value`);
+  }
+}

src/summary.ts

@@ -1,8 +1,9 @@
+import { ValueError } from "./error.js";
 import { Visitor } from "./schema.js";
 import { quote } from "./shell.js";
 import { create as createTable } from "./table.js";
 import { Result, Results } from "./validation.js";
-import { ValueError, Variable } from "./variable.js";
+import { Variable } from "./variable.js";
 
 const ATTENTION = "❯";
 const INVALID = "✗";

src/variable.ts

@@ -1,10 +1,9 @@
 import { applyConstraints } from "./constraint.js";
 import { readVariable } from "./environment.js";
-import { SpecError, normalize } from "./error.js";
+import { NotSetError, SpecError, ValueError, normalize } from "./error.js";
 import { type Example } from "./example.js";
 import { Maybe, definedValue, map, undefinedValue } from "./maybe.js";
 import { Schema } from "./schema.js";
-import { quote } from "./shell.js";
 
 export type VariableSpec<T> = {
   readonly name: string;
@@ -141,22 +140,3 @@ export function create<T>(spec: VariableSpec<T>): Variable<T> {
     }
   }
 }
-
-export class ValueError extends Error {
-  constructor(
-    public readonly name: string,
-    public readonly isSensitive: boolean,
-    public readonly value: string,
-    public readonly cause: Error,
-  ) {
-    const renderedValue = isSensitive ? "<sensitive value>" : quote(value);
-
-    super(`value of ${name} (${renderedValue}) is invalid: ${cause.message}`);
-  }
-}
-
-export class NotSetError extends Error {
-  constructor(public readonly name: string) {
-    super(`${name} is not set and does not have a default value`);
-  }
-}

test/suite/initialize.spec.ts

@@ -1,8 +1,8 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import { Declaration, Options } from "../../src/declaration.js";
+import { NotSetError } from "../../src/error.js";
 import { OnInvalid, boolean, initialize, string } from "../../src/index.js";
 import { Results } from "../../src/validation.js";
-import { NotSetError } from "../../src/variable.js";
 import { MockConsole, Mocked, createMockConsole, mockFn } from "../helpers.js";
 
 describe("initialize()", () => {