pi-crew 0.9.16 → 0.9.18

package/src/observability/exporters/otlp-exporter.ts

@@ -1,5 +1,5 @@
-import { gzip } from "node:zlib";
 import { promisify } from "node:util";
+import { gzip } from "node:zlib";
 import { logInternalError } from "../../utils/internal-error.ts";
 import { redactSecrets } from "../../utils/redaction.ts";
 import type { MetricRegistry } from "../metric-registry.ts";
@@ -25,9 +25,7 @@ export function validateEndpoint(endpoint: string): void {
 
 	// Only allow http and https protocols
 	if (url.protocol !== "http:" && url.protocol !== "https:") {
-		throw new Error(
-			`OTLP endpoint must use http:// or https:// protocol, got: ${url.protocol}`,
-		);
+		throw new Error(`OTLP endpoint must use http:// or https:// protocol, got: ${url.protocol}`);
 	}
 
 	const hostname = url.hostname.toLowerCase();
@@ -101,7 +99,12 @@ function pointValues(snapshot: MetricSnapshot): unknown[] {
 			attributes: Object.entries(value.labels).map(([key, item]) => {
 				const redacted = redactSecrets({ [key]: item }) as Record<string, string>;
 				const val = String(redacted[key] ?? item);
-				return { key, value: { stringValue: val.length > MAX_LABEL_LENGTH ? val.slice(0, MAX_LABEL_LENGTH) : val } };
+				return {
+					key,
+					value: {
+						stringValue: val.length > MAX_LABEL_LENGTH ? val.slice(0, MAX_LABEL_LENGTH) : val,
+					},
+				};
 			}),
 			count: "count" in value ? value.count : undefined,
 			sum: "sum" in value ? value.sum : undefined,
@@ -113,7 +116,12 @@ function pointValues(snapshot: MetricSnapshot): unknown[] {
 		attributes: Object.entries(value.labels).map(([key, item]) => {
 			const redacted = redactSecrets({ [key]: item }) as Record<string, string>;
 			const val = String(redacted[key] ?? item);
-			return { key, value: { stringValue: val.length > MAX_LABEL_LENGTH ? val.slice(0, MAX_LABEL_LENGTH) : val } };
+			return {
+				key,
+				value: {
+					stringValue: val.length > MAX_LABEL_LENGTH ? val.slice(0, MAX_LABEL_LENGTH) : val,
+				},
+			};
 		}),
 		asDouble: "value" in value ? value.value : undefined,
 		count: "count" in value ? value.count : undefined,
@@ -123,13 +131,30 @@ function pointValues(snapshot: MetricSnapshot): unknown[] {
 
 export function convertToOTLP(snapshots: MetricSnapshot[]): unknown {
 	return {
-		resourceMetrics: [{
-			resource: { attributes: [{ key: "service.name", value: { stringValue: "pi-crew" } }] },
-			scopeMetrics: [{
-				scope: { name: "pi-crew" },
-				metrics: snapshots.map((snapshot) => ({ name: snapshot.name, description: snapshot.description, [snapshot.type === "histogram" ? "histogram" : snapshot.type === "gauge" ? "gauge" : "sum"]: { dataPoints: pointValues(snapshot) } })),
-			}],
-		}],
+		resourceMetrics: [
+			{
+				resource: {
+					attributes: [
+						{
+							key: "service.name",
+							value: { stringValue: "pi-crew" },
+						},
+					],
+				},
+				scopeMetrics: [
+					{
+						scope: { name: "pi-crew" },
+						metrics: snapshots.map((snapshot) => ({
+							name: snapshot.name,
+							description: snapshot.description,
+							[snapshot.type === "histogram" ? "histogram" : snapshot.type === "gauge" ? "gauge" : "sum"]: {
+								dataPoints: pointValues(snapshot),
+							},
+						})),
+					},
+				],
+			},
+		],
 	};
 }
 
@@ -154,7 +179,9 @@ export class OTLPExporter implements MetricExporter {
 			// Skip if a previous push is still running; the next tick will retry.
 			if (this.inFlight) return;
 			const snap = this.registry.snapshot();
-			this.inFlight = this.push(snap).finally(() => { this.inFlight = null; });
+			this.inFlight = this.push(snap).finally(() => {
+				this.inFlight = null;
+			});
 		}, this.opts.intervalMs ?? 60_000);
 		this.timer.unref();
 	}
@@ -182,12 +209,20 @@ export class OTLPExporter implements MetricExporter {
 				const body = await gzipAsync(Buffer.from(json));
 				const response = await fetch(this.opts.endpoint, {
 					method: "POST",
-					headers: { "content-type": "application/json", "content-encoding": "gzip", ...(this.opts.headers ?? {}) },
+					headers: {
+						"content-type": "application/json",
+						"content-encoding": "gzip",
+						...(this.opts.headers ?? {}),
+					},
 					body,
 					signal: controller.signal,
 				});
 				if (!response.ok) {
-					logInternalError("otlp-export-http", new Error(`HTTP ${response.status}: ${response.statusText}`), `endpoint=${this.opts.endpoint}`);
+					logInternalError(
+						"otlp-export-http",
+						new Error(`HTTP ${response.status}: ${response.statusText}`),
+						`endpoint=${this.opts.endpoint}`,
+					);
 				}
 			} finally {
 				clearTimeout(timer);
@@ -208,7 +243,11 @@ export class OTLPExporter implements MetricExporter {
 		if (this.timer) clearInterval(this.timer);
 		this.timer = undefined;
 		if (this.inFlight) {
-			try { await this.inFlight; } catch { /* swallow — push() already logs errors */ }
+			try {
+				await this.inFlight;
+			} catch {
+				/* swallow — push() already logs errors */
+			}
 		}
 	}
 }

package/src/observability/exporters/prometheus-exporter.ts

@@ -5,7 +5,7 @@ function prometheusName(name: string): string {
 }
 
 function escapeLabel(value: string): string {
-	return value.replace(/\\/g, "\\\\").replace(/\n/g, "\\n").replace(/"/g, "\\\"");
+	return value.replace(/\\/g, "\\\\").replace(/\n/g, "\\n").replace(/"/g, '\\"');
 }
 
 function labelsText(labels: MetricLabels): string {

package/src/observability/metric-registry.ts

@@ -1,5 +1,5 @@
-import { Counter, Gauge, Histogram, type Metric, type MetricSnapshot } from "./metrics-primitives.ts";
 import { logInternalError } from "../utils/internal-error.ts";
+import { Counter, Gauge, Histogram, type Metric, type MetricSnapshot } from "./metrics-primitives.ts";
 
 const METRIC_NAME_PATTERN = /^crew\.[a-z]+\.[a-z][a-z_]*$/;
 
@@ -38,7 +38,11 @@ export class MetricRegistry {
 		const existing = this.metrics.get(name);
 		if (existing instanceof Counter) {
 			if (existing.description !== description) {
-				logInternalError("metric-registry.counter", new Error("description mismatch"), `name='${name}' original='${existing.description}'`);
+				logInternalError(
+					"metric-registry.counter",
+					new Error("description mismatch"),
+					`name='${name}' original='${existing.description}'`,
+				);
 			}
 			return existing;
 		}
@@ -50,7 +54,11 @@ export class MetricRegistry {
 		const existing = this.metrics.get(name);
 		if (existing instanceof Gauge) {
 			if (existing.description !== description) {
-				logInternalError("metric-registry.gauge", new Error("description mismatch"), `name='${name}' original='${existing.description}'`);
+				logInternalError(
+					"metric-registry.gauge",
+					new Error("description mismatch"),
+					`name='${name}' original='${existing.description}'`,
+				);
 			}
 			return existing;
 		}
@@ -62,7 +70,11 @@ export class MetricRegistry {
 		const existing = this.metrics.get(name);
 		if (existing instanceof Histogram) {
 			if (existing.description !== description) {
-				logInternalError("metric-registry.histogram", new Error("description mismatch"), `name='${name}' original='${existing.description}'`);
+				logInternalError(
+					"metric-registry.histogram",
+					new Error("description mismatch"),
+					`name='${name}' original='${existing.description}'`,
+				);
 			}
 			return existing;
 		}

package/src/observability/metric-retention.ts

@@ -21,7 +21,11 @@ export class TimeWindowedCounter {
 		if (!Number.isFinite(delta)) return;
 		// Cap the event array to prevent unbounded memory growth.
 		if (this.events.length >= TimeWindowedCounter.MAX_EVENTS) this.prune();
-		this.events.push({ timestamp: this.now(), labels: { ...labels }, delta });
+		this.events.push({
+			timestamp: this.now(),
+			labels: { ...labels },
+			delta,
+		});
 		this.prune();
 	}
 
@@ -30,7 +34,9 @@ export class TimeWindowedCounter {
 		this.pruneAt(now);
 		const key = labelKey(labels);
 		const cutoff = now - durationMs;
-		return this.events.filter((event) => event.timestamp >= cutoff && labelKey(event.labels) === key).reduce((sum, event) => sum + event.delta, 0);
+		return this.events
+			.filter((event) => event.timestamp >= cutoff && labelKey(event.labels) === key)
+			.reduce((sum, event) => sum + event.delta, 0);
 	}
 
 	rate(labels: MetricLabels = {}, durationMs = this.windowMs): number {

package/src/observability/metric-sink.ts

@@ -1,7 +1,7 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { redactSecrets } from "../utils/redaction.ts";
 import { logInternalError } from "../utils/internal-error.ts";
+import { redactSecrets } from "../utils/redaction.ts";
 import type { MetricRegistry } from "./metric-registry.ts";
 import type { MetricSnapshot } from "./metrics-primitives.ts";
 
@@ -42,7 +42,11 @@ export function createMetricFileSink(opts: MetricFileSinkOptions): MetricSink {
 	const ensureFd = (date: string): number => {
 		if (fd !== undefined && fdDate === date) return fd;
 		if (fd !== undefined) {
-			try { fs.closeSync(fd); } catch (error) { logInternalError("metric-sink.closeFd", error); }
+			try {
+				fs.closeSync(fd);
+			} catch (error) {
+				logInternalError("metric-sink.closeFd", error);
+			}
 		}
 		fs.mkdirSync(dir, { recursive: true });
 		rotateOldFiles(dir, retentionDays);
@@ -72,7 +76,11 @@ export function createMetricFileSink(opts: MetricFileSinkOptions): MetricSink {
 		dispose: () => {
 			clearInterval(timer);
 			if (fd !== undefined) {
-				try { fs.closeSync(fd); } catch (error) { logInternalError("metric-sink.dispose", error); }
+				try {
+					fs.closeSync(fd);
+				} catch (error) {
+					logInternalError("metric-sink.dispose", error);
+				}
 				fd = undefined;
 				fdDate = undefined;
 			}

package/src/observability/metrics-primitives.ts

@@ -78,11 +78,17 @@ export class Counter extends Metric {
 	inc(labels: MetricLabels = {}, delta = 1): void {
 		if (!Number.isFinite(delta) || delta < 0) return;
 		const key = labelKey(labels);
-		const current = this.values.get(key) ?? { labels: normalizeLabels(labels), value: 0 };
+		const current = this.values.get(key) ?? {
+			labels: normalizeLabels(labels),
+			value: 0,
+		};
 		// Delete before set to move key to end of insertion order (MRU).
 		// Without this, enforceLabelCap could evict an actively-used entry.
 		this.values.delete(key);
-		this.values.set(key, { labels: current.labels, value: current.value + delta });
+		this.values.set(key, {
+			labels: current.labels,
+			value: current.value + delta,
+		});
 		enforceLabelCap(this.values, this.name);
 	}
 
@@ -91,7 +97,15 @@ export class Counter extends Metric {
 	}
 
 	snapshot(): MetricSnapshot {
-		return { type: "counter", name: this.name, description: this.description, values: [...this.values.values()].map((entry) => ({ labels: cloneLabels(entry.labels), value: entry.value })) };
+		return {
+			type: "counter",
+			name: this.name,
+			description: this.description,
+			values: [...this.values.values()].map((entry) => ({
+				labels: cloneLabels(entry.labels),
+				value: entry.value,
+			})),
+		};
 	}
 }
 
@@ -117,7 +131,15 @@ export class Gauge extends Metric {
 	}
 
 	snapshot(): MetricSnapshot {
-		return { type: "gauge", name: this.name, description: this.description, values: [...this.values.values()].map((entry) => ({ labels: cloneLabels(entry.labels), value: entry.value })) };
+		return {
+			type: "gauge",
+			name: this.name,
+			description: this.description,
+			values: [...this.values.values()].map((entry) => ({
+				labels: cloneLabels(entry.labels),
+				value: entry.value,
+			})),
+		};
 	}
 }
 
@@ -135,9 +157,15 @@ export class Histogram extends Metric {
 		if (!Number.isFinite(value)) return;
 		const key = labelKey(labels);
 		const existing = this.observations.get(key);
-		const current = existing ?? { labels: normalizeLabels(labels), counts: new Array(this.buckets.length + 1).fill(0) as number[], sum: 0, count: 0 };
+		const current = existing ?? {
+			labels: normalizeLabels(labels),
+			counts: new Array(this.buckets.length + 1).fill(0) as number[],
+			sum: 0,
+			count: 0,
+		};
 		const bucketIndex = this.buckets.findIndex((bucket) => value <= bucket);
-		current.counts[bucketIndex === -1 ? this.buckets.length : bucketIndex] = (current.counts[bucketIndex === -1 ? this.buckets.length : bucketIndex] ?? 0) + 1;
+		current.counts[bucketIndex === -1 ? this.buckets.length : bucketIndex] =
+			(current.counts[bucketIndex === -1 ? this.buckets.length : bucketIndex] ?? 0) + 1;
 		current.sum += value;
 		current.count += 1;
 		if (!existing) this.observations.set(key, current);
@@ -155,8 +183,9 @@ export class Histogram extends Metric {
 			cumulative += count;
 			if (cumulative >= target) {
 				const previous = cumulative - count;
-				const lower = index === 0 ? 0 : this.buckets[index - 1] ?? this.buckets.at(-1) ?? 0;
-				const upper = index < this.buckets.length ? this.buckets[index] ?? lower : Math.max(lower, obs.sum / Math.max(1, obs.count));
+				const lower = index === 0 ? 0 : (this.buckets[index - 1] ?? this.buckets.at(-1) ?? 0);
+				const upper =
+					index < this.buckets.length ? (this.buckets[index] ?? lower) : Math.max(lower, obs.sum / Math.max(1, obs.count));
 				const fraction = count === 0 ? 0 : (target - previous) / Math.max(1, count);
 				return lower + fraction * (upper - lower);
 			}
@@ -179,7 +208,11 @@ export class Histogram extends Metric {
 				counts: [...entry.counts],
 				sum: entry.sum,
 				count: entry.count,
-				quantiles: { p50: this.quantile(entry.labels, 0.5), p95: this.quantile(entry.labels, 0.95), p99: this.quantile(entry.labels, 0.99) },
+				quantiles: {
+					p50: this.quantile(entry.labels, 0.5),
+					p95: this.quantile(entry.labels, 0.95),
+					p99: this.quantile(entry.labels, 0.99),
+				},
 			})),
 		};
 	}

package/src/plugins/plugin-define.ts

@@ -2,5 +2,5 @@
  * Simplifies plugin struct definition.
  */
 export function definePlugin<T extends { name: string; enablers: readonly string[] }>(spec: T): T {
-  return spec;
-}
+	return spec;
+}

package/src/plugins/plugin-registry.ts

@@ -1,32 +1,32 @@
 export interface Plugin {
-  readonly name: string;
-  readonly enablers: readonly string[];
-  readonly entryPatterns?: readonly string[];
-  readonly configPatterns?: readonly string[];
-  readonly toolingDependencies?: readonly string[];
-  readonly pathAliases?: readonly string[][];
-  readonly virtualModulePrefixes?: readonly string[];
+	readonly name: string;
+	readonly enablers: readonly string[];
+	readonly entryPatterns?: readonly string[];
+	readonly configPatterns?: readonly string[];
+	readonly toolingDependencies?: readonly string[];
+	readonly pathAliases?: readonly string[][];
+	readonly virtualModulePrefixes?: readonly string[];
 }
 
 export class PluginRegistry {
-  private plugins: Plugin[] = [];
+	private plugins: Plugin[] = [];
 
-  register(plugin: Plugin): void {
-    this.plugins.push(plugin);
-  }
+	register(plugin: Plugin): void {
+		this.plugins.push(plugin);
+	}
 
-  activePlugins(allDeps: string[]): Plugin[] {
-    return this.plugins.filter((p) =>
-      p.enablers.some((enabler) => {
-        if (enabler.endsWith("/")) {
-          return allDeps.some((d) => d.startsWith(enabler));
-        }
-        return allDeps.includes(enabler);
-      }),
-    );
-  }
+	activePlugins(allDeps: string[]): Plugin[] {
+		return this.plugins.filter((p) =>
+			p.enablers.some((enabler) => {
+				if (enabler.endsWith("/")) {
+					return allDeps.some((d) => d.startsWith(enabler));
+				}
+				return allDeps.includes(enabler);
+			}),
+		);
+	}
 
-  allPlugins(): Plugin[] {
-    return [...this.plugins];
-  }
-}
+	allPlugins(): Plugin[] {
+		return [...this.plugins];
+	}
+}

package/src/plugins/plugins/index.ts

@@ -1,3 +1,3 @@
 export { NextJsPlugin } from "./nextjs.ts";
+export { VitePlugin } from "./vite.ts";
 export { VitestPlugin } from "./vitest.ts";
-export { VitePlugin } from "./vite.ts";

package/src/plugins/plugins/nextjs.ts

@@ -1,19 +1,19 @@
 import { definePlugin } from "../plugin-define.ts";
 
 export const NextJsPlugin = definePlugin({
-  name: "nextjs",
-  enablers: ["next"],
-  entryPatterns: [
-    "src/app/**/*.{ts,tsx}",
-    "src/pages/**/*.{ts,tsx}",
-    "src/app/**/page.{ts,tsx}",
-    "src/app/**/layout.{ts,tsx}",
-    "src/app/**/route.{ts,tsx}",
-    "middleware.{ts,js}",
-    "next.config.{ts,js,mjs}",
-  ],
-  configPatterns: ["next.config.{ts,js,mjs}"],
-  toolingDependencies: ["next", "@next/font", "@next/mdx"],
-  pathAliases: [["~", "src"]],
-  virtualModulePrefixes: ["next:"],
-});
+	name: "nextjs",
+	enablers: ["next"],
+	entryPatterns: [
+		"src/app/**/*.{ts,tsx}",
+		"src/pages/**/*.{ts,tsx}",
+		"src/app/**/page.{ts,tsx}",
+		"src/app/**/layout.{ts,tsx}",
+		"src/app/**/route.{ts,tsx}",
+		"middleware.{ts,js}",
+		"next.config.{ts,js,mjs}",
+	],
+	configPatterns: ["next.config.{ts,js,mjs}"],
+	toolingDependencies: ["next", "@next/font", "@next/mdx"],
+	pathAliases: [["~", "src"]],
+	virtualModulePrefixes: ["next:"],
+});

package/src/plugins/plugins/vite.ts

@@ -1,14 +1,10 @@
 import { definePlugin } from "../plugin-define.ts";
 
 export const VitePlugin = definePlugin({
-  name: "vite",
-  enablers: ["vite", "rolldown-vite"],
-  entryPatterns: [
-    "src/main.{ts,tsx,js,jsx}",
-    "src/index.{ts,tsx,js,jsx}",
-    "index.html",
-  ],
-  configPatterns: ["vite.config.{ts,js,mts,mjs}"],
-  toolingDependencies: ["vite"],
-  virtualModulePrefixes: ["virtual:"],
-});
+	name: "vite",
+	enablers: ["vite", "rolldown-vite"],
+	entryPatterns: ["src/main.{ts,tsx,js,jsx}", "src/index.{ts,tsx,js,jsx}", "index.html"],
+	configPatterns: ["vite.config.{ts,js,mts,mjs}"],
+	toolingDependencies: ["vite"],
+	virtualModulePrefixes: ["virtual:"],
+});

package/src/plugins/plugins/vitest.ts

@@ -1,14 +1,9 @@
 import { definePlugin } from "../plugin-define.ts";
 
 export const VitestPlugin = definePlugin({
-  name: "vitest",
-  enablers: ["vitest"],
-  entryPatterns: [
-    "**/*.test.{ts,tsx}",
-    "**/*.spec.{ts,tsx}",
-    "src/**/*.test.{ts,tsx}",
-    "src/**/*.spec.{ts,tsx}",
-  ],
-  configPatterns: ["vitest.config.{ts,js,mjs}", "vite.config.ts"],
-  toolingDependencies: ["vitest"],
-});
+	name: "vitest",
+	enablers: ["vitest"],
+	entryPatterns: ["**/*.test.{ts,tsx}", "**/*.spec.{ts,tsx}", "src/**/*.test.{ts,tsx}", "src/**/*.spec.{ts,tsx}"],
+	configPatterns: ["vitest.config.{ts,js,mjs}", "vite.config.ts"],
+	toolingDependencies: ["vitest"],
+});