WIP

src/declaration/kubernetes-address.ts

@@ -7,7 +7,7 @@ import {
   defaultFromOptions,
   type ExactOptions,
 } from "../declaration.js";
-import { registerVariable } from "../environment.js";
+import { registerComposite, registerVariable } from "../environment.js";
 import { SpecError, normalize } from "../error.js";
 import { resolveExamples, type Example } from "../example.js";
 import { Maybe, map, resolve } from "../maybe.js";
@@ -51,7 +51,8 @@ export function kubernetesAddress<O extends Options>(
   const portVar = registerPort(name, portExamples, isSensitive, def, portName);
   const pName = portVar.spec.name;
 
-  return {
+  const composite = registerComposite({
+    variables: [hostVar, portVar],
     value() {
       const host = resolve(hostVar.nativeValue());
       const port = resolve(portVar.nativeValue());
@@ -63,6 +64,12 @@ export function kubernetesAddress<O extends Options>(
 
       return undefined as Value<KubernetesAddress, O>;
     },
+  });
+
+  return {
+    value() {
+      return composite.value();
+    },
   };
 }
 

src/environment.ts

@@ -6,6 +6,7 @@ import {
   Variable,
   VariableSpec,
   create as createVariable,
+  type VariableComposite,
 } from "./variable.js";
 
 let state: State = createInitialState();
@@ -22,7 +23,7 @@ export function initialize(options: InitializeOptions = {}): void {
     process.exit(0);
   } else {
     const { onInvalid = defaultOnInvalid } = options;
-    const [isValid, results] = validate(variablesByName());
+    const [isValid, results] = validate(variablesByName(), state.composites);
 
     if (!isValid) {
       onInvalid({
@@ -42,6 +43,14 @@ export function registerVariable<T>(spec: VariableSpec<T>): Variable<T> {
   return variable;
 }
 
+export function registerComposite<T>(
+  composite: VariableComposite<T>,
+): VariableComposite<T> {
+  state.composites.push(composite);
+
+  return composite;
+}
+
 export function readVariable<T>(spec: VariableSpec<T>): string {
   return process.env[spec.name] ?? "";
 }
@@ -54,11 +63,13 @@ type State = {
   // TODO: WTF TypeScript? Why can't I use unknown here?
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   readonly variables: Record<string, Variable<any>>;
+  readonly composites: VariableComposite<unknown>[];
 };
 
 function createInitialState(): State {
   return {
     variables: {},
+    composites: [],
   };
 }
 

src/error.ts

@@ -26,6 +26,15 @@ export class ValueError extends Error {
   }
 }
 
+export class CompositeError extends Error {
+  constructor(
+    public readonly name: string,
+    public readonly cause: Error,
+  ) {
+    super(`${name} 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,4 +1,4 @@
-import { ValueError } from "./error.js";
+import { CompositeError, ValueError } from "./error.js";
 import { Visitor } from "./schema.js";
 import { quote } from "./shell.js";
 import { create as createTable } from "./table.js";
@@ -84,9 +84,14 @@ function renderResult(
 }
 
 function describeError(isSensitive: boolean, error: Error) {
-  if (!(error instanceof ValueError)) return "not set";
+  if (error instanceof ValueError) {
+    return (
+      `set to ${quoteAndSuppress(isSensitive, error.value)}, ` +
+      `${error.cause.message}`
+    );
+  }
 
-  return `set to ${quoteAndSuppress(isSensitive, error.value)}, ${error.cause.message}`;
+  return error instanceof CompositeError ? error.message : "not set";
 }
 
 function quoteAndSuppress(isSensitive: boolean, value: string) {

src/validation.ts

@@ -1,23 +1,50 @@
-import { normalize } from "./error.js";
+import { CompositeError, normalize } from "./error.js";
 import { Maybe } from "./maybe.js";
-import { Value, Variable } from "./variable.js";
+import { Value, Variable, type VariableComposite } from "./variable.js";
 
-export function validate(variables: Variable<unknown>[]): [boolean, Results] {
-  const results: Results = [];
+export function validate(
+  variables: Variable<unknown>[],
+  composites: VariableComposite<unknown>[],
+): [boolean, Results] {
+  const resultMap = new Map<Variable<unknown>, Result>();
   let isValid = true;
 
   for (const variable of variables) {
     try {
-      results.push({
-        variable,
-        result: { maybe: variable.value() },
-      });
+      resultMap.set(variable, { maybe: variable.value() });
     } catch (error) {
       isValid = false;
-      results.push({ variable, result: { error: normalize(error) } });
+      resultMap.set(variable, { error: normalize(error) });
     }
   }
 
+  for (const composite of composites) {
+    const shouldSkip = composite.variables.some(
+      (variable) => resultMap.get(variable)?.error,
+    );
+
+    if (shouldSkip) continue;
+
+    try {
+      composite.value();
+    } catch (error) {
+      isValid = false;
+
+      for (const variable of composite.variables) {
+        resultMap.set(variable, {
+          error: new CompositeError(variable.spec.name, normalize(error)),
+        });
+      }
+    }
+  }
+
+  const results: Results = [];
+
+  for (const variable of variables) {
+    const result = resultMap.get(variable);
+    if (result) results.push({ variable, result });
+  }
+
   return [isValid, results];
 }
 

src/variable.ts

@@ -22,6 +22,12 @@ export type Variable<T> = {
   readonly unmarshal: (value: string) => T;
 };
 
+export type VariableComposite<T> = {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  readonly variables: Variable<any>[];
+  readonly value: () => T;
+};
+
 export type Value<T> = {
   readonly verbatim: string;
   readonly canonical: string;

test/fixture/summary/invalid-composite.ansi

@@ -0,0 +1,4 @@
+Environment Variables:
+
+❯ AUSTENITE_SVC_SERVICE_HOST  kubernetes `austenite-svc` service host  [ <hostname> ]     ✗ AUSTENITE_SVC_SERVICE_HOST is invalid: AUSTENITE_SVC_SERVICE_HOST is defined but AUSTENITE_SVC_SERVICE_PORT is not, define both or neither
+❯ AUSTENITE_SVC_SERVICE_PORT  kubernetes `austenite-svc` service port  [ <port number> ]  ✗ AUSTENITE_SVC_SERVICE_PORT is invalid: AUSTENITE_SVC_SERVICE_HOST is defined but AUSTENITE_SVC_SERVICE_PORT is not, define both or neither

test/suite/summary.spec.ts

@@ -311,6 +311,21 @@ describe("Validation summary", () => {
     expect(exitCode).toBeGreaterThan(0);
   });
 
+  it("summarizes invalid composites", async () => {
+    Object.assign(process.env, {
+      AUSTENITE_SVC_SERVICE_HOST: "host.example.org",
+    });
+
+    kubernetesAddress("austenite-svc", { default: undefined });
+
+    initialize();
+
+    await expect(mockConsole.readStderr()).toMatchFileSnapshot(
+      fixturePath("invalid-composite"),
+    );
+    expect(exitCode).toBeGreaterThan(0);
+  });
+
   it("summarizes variables that violate constraints", async () => {
     Object.assign(process.env, {
       AUSTENITE_STRING: "hello, world!",