@glassmkr/crucible 0.6.1 → 0.6.2

package/src/collect/__tests__/zfs.test.ts

@@ -0,0 +1,72 @@
+import { describe, it, expect } from "vitest";
+import { parseZpoolStatus } from "../zfs.js";
+
+describe("parseZpoolStatus", () => {
+  it("parses a healthy pool", () => {
+    const raw = `  pool: tank
+ state: ONLINE
+  scan: scrub repaired 0B in 01:23:45 with 0 errors on Sun Apr  5 12:34:56 2026
+config:
+
+        NAME        STATE     READ WRITE CKSUM
+        tank        ONLINE       0     0     0
+          mirror-0  ONLINE       0     0     0
+
+errors: No known data errors
+`;
+    const pools = parseZpoolStatus(raw);
+    expect(pools).toHaveLength(1);
+    expect(pools[0]).toMatchObject({
+      name: "tank",
+      state: "ONLINE",
+      errors_text: "No known data errors",
+      scrub_errors: 0,
+      scrub_repaired: "0B",
+    });
+    expect(pools[0].last_scrub_date).toContain("2026");
+  });
+
+  it("parses a DEGRADED pool", () => {
+    const raw = `  pool: tank
+ state: DEGRADED
+  scan: scrub repaired 16K in 02:00:00 with 3 errors on Sun Apr  5 12:34:56 2026
+
+errors: 3 data errors, use '-v' for a list
+`;
+    const [p] = parseZpoolStatus(raw);
+    expect(p.state).toBe("DEGRADED");
+    expect(p.scrub_errors).toBe(3);
+    expect(p.scrub_repaired).toBe("16K");
+  });
+
+  it("flags never-scrubbed pools", () => {
+    const raw = `  pool: tank
+ state: ONLINE
+  scan: none requested
+
+errors: No known data errors
+`;
+    const [p] = parseZpoolStatus(raw);
+    expect(p.scrub_never_run).toBe(true);
+    expect(p.scrub_errors).toBeUndefined();
+  });
+
+  it("returns empty for no pools", () => {
+    expect(parseZpoolStatus("no pools available")).toEqual([]);
+  });
+
+  it("parses multiple pools", () => {
+    const raw = `  pool: tank
+ state: ONLINE
+  scan: none requested
+errors: No known data errors
+  pool: data
+ state: FAULTED
+  scan: none requested
+errors: 2 data errors
+`;
+    const pools = parseZpoolStatus(raw);
+    expect(pools.map((p) => p.name)).toEqual(["tank", "data"]);
+    expect(pools[1].state).toBe("FAULTED");
+  });
+});

package/src/collect/ipmi.ts

@@ -104,7 +104,7 @@ async function collectSelEvents(): Promise<SelEvent[]> {
   return events.slice(-20).reverse();
 }
 
-function parseSelTimestamp(date: string, time: string): string {
+export function parseSelTimestamp(date: string, time: string): string {
   if (!date || !time) return new Date().toISOString();
   // Format: "04/05/2026" and "14:23:05"
   const parts = date.split("/");
@@ -113,7 +113,7 @@ function parseSelTimestamp(date: string, time: string): string {
   return `${year}-${month.padStart(2, "0")}-${day.padStart(2, "0")}T${time}Z`;
 }
 
-function classifySensor(sensor: string): string {
+export function classifySensor(sensor: string): string {
   const lower = sensor.toLowerCase();
   if (lower.includes("memory") || lower.includes("dimm")) return "memory";
   if (lower.includes("power supply") || lower.includes("psu")) return "power";
@@ -127,7 +127,7 @@ function classifySensor(sensor: string): string {
   return "other";
 }
 
-function deriveSelSeverity(event: string, sensorType: string): string {
+export function deriveSelSeverity(event: string, sensorType: string): string {
   const lower = event.toLowerCase();
 
   // Critical events
@@ -163,7 +163,10 @@ function deriveSelSeverity(event: string, sensorType: string): string {
 async function collectFanStatus(): Promise<FanStatus[]> {
   const output = await run("ipmitool", ["sdr", "type", "Fan"]);
   if (!output) return [];
+  return parseFanStatus(output);
+}
 
+export function parseFanStatus(output: string): FanStatus[] {
   const fans: FanStatus[] = [];
   const lines = output.trim().split("\n");
 

package/src/collect/smart.ts

@@ -22,34 +22,7 @@ export async function collectSmart(): Promise<SmartInfo[]> {
     if (!output) continue;
 
     try {
-      const data = JSON.parse(output);
-      const info: SmartInfo = {
-        device,
-        model: data.model_name || data.model_family || "unknown",
-        health: data.smart_status?.passed ? "PASSED" : "FAILED",
-        temperature_c: data.temperature?.current,
-        power_on_hours: data.power_on_time?.hours,
-      };
-
-      // NVMe specific
-      if (data.nvme_smart_health_information_log) {
-        const nvme = data.nvme_smart_health_information_log;
-        info.percentage_used = nvme.percentage_used;
-        info.temperature_c = nvme.temperature;
-      }
-
-      // SATA specific
-      if (data.ata_smart_attributes?.table) {
-        for (const attr of data.ata_smart_attributes.table) {
-          if (attr.id === 5 || attr.name === "Reallocated_Sector_Ct") {
-            info.reallocated_sectors = attr.raw?.value || 0;
-          }
-          if (attr.id === 197 || attr.name === "Current_Pending_Sector") {
-            info.pending_sectors = attr.raw?.value || 0;
-          }
-        }
-      }
-
+      const info = parseSmartctlJson(JSON.parse(output), device);
       results.push(info);
     } catch {
       // Failed to parse, skip this device
@@ -58,3 +31,42 @@ export async function collectSmart(): Promise<SmartInfo[]> {
 
   return results;
 }
+
+export function parseSmartctlJson(data: Record<string, unknown> & {
+  model_name?: string;
+  model_family?: string;
+  smart_status?: { passed?: boolean };
+  temperature?: { current?: number };
+  power_on_time?: { hours?: number };
+  nvme_smart_health_information_log?: { percentage_used?: number; temperature?: number };
+  ata_smart_attributes?: { table?: Array<{ id?: number; name?: string; raw?: { value?: number } }> };
+}, device: string): SmartInfo {
+  const info: SmartInfo = {
+    device,
+    model: data.model_name || data.model_family || "unknown",
+    health: data.smart_status?.passed ? "PASSED" : "FAILED",
+    temperature_c: data.temperature?.current,
+    power_on_hours: data.power_on_time?.hours,
+  };
+
+  // NVMe specific
+  if (data.nvme_smart_health_information_log) {
+    const nvme = data.nvme_smart_health_information_log;
+    info.percentage_used = nvme.percentage_used;
+    info.temperature_c = nvme.temperature;
+  }
+
+  // SATA specific
+  if (data.ata_smart_attributes?.table) {
+    for (const attr of data.ata_smart_attributes.table) {
+      if (attr.id === 5 || attr.name === "Reallocated_Sector_Ct") {
+        info.reallocated_sectors = attr.raw?.value || 0;
+      }
+      if (attr.id === 197 || attr.name === "Current_Pending_Sector") {
+        info.pending_sectors = attr.raw?.value || 0;
+      }
+    }
+  }
+
+  return info;
+}

package/src/collect/zfs.ts

@@ -9,6 +9,12 @@ export async function collectZfs(): Promise<ZfsData | null> {
   const zpoolStatus = await run("zpool", ["status"], 10000);
   if (!zpoolStatus || !zpoolStatus.trim()) return null;
 
+  const pools = parseZpoolStatus(zpoolStatus);
+  if (pools.length === 0) return null;
+  return { pools };
+}
+
+export function parseZpoolStatus(zpoolStatus: string): ZfsPool[] {
   const pools: ZfsPool[] = [];
   let current: ZfsPool | null = null;
 
@@ -56,6 +62,5 @@ export async function collectZfs(): Promise<ZfsData | null> {
     }
   }
 
-  if (pools.length === 0) return null;
-  return { pools };
+  return pools;
 }

package/src/index.ts

@@ -3,7 +3,7 @@
 import { readFileSync } from "node:fs";
 import { fileURLToPath } from "node:url";
 import { dirname, join } from "node:path";
-import { loadConfig } from "./config.js";
+import { parseCliArgs } from "./cli.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const PKG_VERSION = (() => {
@@ -14,6 +14,17 @@ const PKG_VERSION = (() => {
     return "0.0.0";
   }
 })();
+
+// Handle --version and --help before importing collectors, loading config, or
+// starting the Prometheus server. This keeps the CLI responsive even on hosts
+// missing the config file or external tools.
+const { result: cliArgs, output: cliOutput } = parseCliArgs(process.argv.slice(2), PKG_VERSION);
+if (cliArgs.mode !== "run") {
+  console.log(cliOutput);
+  process.exit(0);
+}
+
+import { loadConfig } from "./config.js";
 import { checkForUpdates } from "./lib/version-check.js";
 import { startMetricsServer, updateMetrics } from "./metrics-server.js";
 import { collectSystem } from "./collect/system.js";
@@ -41,8 +52,7 @@ import { collectNtp } from "./collect/ntp.js";
 import { collectFileDescriptors } from "./collect/fd.js";
 import type { Snapshot, IpmiInfo } from "./lib/types.js";
 
-const configPath = process.argv[2] || "/etc/glassmkr/collector.yaml";
-const config = loadConfig(configPath);
+const config = loadConfig(cliArgs.configPath);
 
 console.log(`[collector] Starting. Server: ${config.server_name}. Interval: ${config.collection.interval_seconds}s`);
 console.log(`[collector] IPMI: ${config.collection.ipmi ? "enabled" : "disabled"}, SMART: ${config.collection.smart ? "enabled" : "disabled"}`);

package/src/lib/__tests__/parse.test.ts

@@ -0,0 +1,28 @@
+import { describe, it, expect } from "vitest";
+import { parseKeyValue, parseKb } from "../parse.js";
+
+describe("parseKeyValue", () => {
+  it("parses colon-delimited key/value lines", () => {
+    const out = parseKeyValue("Name: foo\nVersion: 1.2.3\n");
+    expect(out).toEqual({ Name: "foo", Version: "1.2.3" });
+  });
+  it("ignores lines with no colon", () => {
+    expect(parseKeyValue("no colon here\nA: 1\n")).toEqual({ A: "1" });
+  });
+  it("trims whitespace around keys and values", () => {
+    expect(parseKeyValue("   A   :    1   \n")).toEqual({ A: "1" });
+  });
+});
+
+describe("parseKb", () => {
+  it("parses a numeric kB value", () => {
+    expect(parseKb("16384 kB")).toBe(16384);
+  });
+  it("parses without unit", () => {
+    expect(parseKb("4096")).toBe(4096);
+  });
+  it("returns 0 for undefined/bad input", () => {
+    expect(parseKb(undefined)).toBe(0);
+    expect(parseKb("not a number")).toBe(0);
+  });
+});

package/vitest.config.ts

@@ -0,0 +1,12 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: {
+    include: ["src/**/*.test.ts"],
+    exclude: ["node_modules", "dist"],
+  },
+  resolve: {
+    // .js specifier inside TS (NodeNext) needs to resolve to .ts in tests
+    extensions: [".ts", ".js"],
+  },
+});