@teamscala/service-boot 0.23.337 → 0.23.339

package/package.json

@@ -6,7 +6,7 @@
 	},
 	"name": "@teamscala/service-boot",
 	"type": "module",
-	"version": "0.23.337",
+	"version": "0.23.339",
 	"sideEffects": false,
 	"files": [
 		"src"

package/src/handlers/sweep-host-cache.d.ts

@@ -19,3 +19,6 @@ export declare function checkDirStale(dir: string, cutoffMs: number): Promise<{
     bytes: number;
     fileCount: number;
 }>;
+/** Parse the root-fs Use% from `df --output=pcent,target` output (pure, for tests).
+ *  Returns null if no root-mount line is found. */
+export declare function parseRootDiskPct(out: string): number | null;

package/src/handlers/sweep-host-cache.test.ts

@@ -14,7 +14,7 @@ import { describe, expect, test, beforeEach, afterEach } from "bun:test";
 import { mkdtempSync, mkdirSync, writeFileSync, rmSync, utimesSync } from "node:fs";
 import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { expandGlob, checkDirStale } from "./sweep-host-cache.ts";
+import { expandGlob, checkDirStale, parseRootDiskPct } from "./sweep-host-cache.ts";
 
 const DAY = 86_400_000;
 const now = Date.now();
@@ -83,3 +83,28 @@ describe("checkDirStale", () => {
 		expect(res.fileCount).toBe(0);
 	});
 });
+
+describe("parseRootDiskPct", () => {
+	test("extracts root-fs Use% from df --output=pcent,target output", () => {
+		const df =
+			"Use% Mounted on\n" +
+			"  97% /\n" +
+			"  45% /home\n";
+		expect(parseRootDiskPct(df)).toBe(97);
+	});
+	test("picks the root mount (/) over nested mounts", () => {
+		const df =
+			"Use% Mounted on\n" +
+			"  10% /proc/x\n" +
+			"  76% /\n";
+		expect(parseRootDiskPct(df)).toBe(76);
+	});
+	test("returns null when no root-mount line present", () => {
+		const df = "Use% Mounted on\n  40% /mnt/other\n";
+		expect(parseRootDiskPct(df)).toBeNull();
+	});
+	test("returns null on empty / malformed output", () => {
+		expect(parseRootDiskPct("")).toBeNull();
+		expect(parseRootDiskPct("Use% Mounted on")).toBeNull();
+	});
+});

package/src/handlers/sweep-host-cache.ts

@@ -200,6 +200,14 @@ export async function checkDirStale(
 	return { stale: true, bytes, fileCount };
 }
 
+/** Parse the root-fs Use% from `df --output=pcent,target` output (pure, for tests).
+ *  Returns null if no root-mount line is found. */
+export function parseRootDiskPct(out: string): number | null {
+	const rootLine = out.split("\n").find((l) => l.trim().endsWith("/"));
+	const m = rootLine?.match(/(\d+)%/);
+	return m ? Number(m[1]) : null;
+}
+
 async function sweepHostCache(ctx: {
 	config: { retention?: Record<string, string>; retention_dirs?: Record<string, string> };
 }): Promise<void> {
@@ -290,32 +298,27 @@ async function sweepHostCache(ctx: {
 		},
 	});
 
-	// Disk-threshold monitor — piggybacks on this nightly sweep event (no separate
-	// interval/infra: `guards-never-run-on-interval`). If `/` usage crosses the
-	// configured threshold, emit `host.disk.high` (-> event-log/GlitchTip) so a fill
-	// the sweep can't absorb (external dump, runaway log) is caught within a day.
-	const highPercent = 90;
+	// Disk-threshold observability — rides this periodic sweep (NOT an interval:
+	// this job is genuinely periodic application-DATA work — deletion — and the
+	// df read piggybacks once per run). Emits through the event-driven surface:
+	// logger.warn -> GlitchTip (the paging path, same as host.pressure) +
+	// recordEvent for the durable event_log record. Catches a fill faster than
+	// the nightly prune can absorb (a cache with no retention entry, an external
+	// dump, a runaway log) so the operator learns it within a day, not at 100%.
+	const diskHighPct = 90;
 	try {
-		const dfRes = spawnSync({ name: "df:/", command: ["df", "--output=pcent,target"], timeoutMs: 5000 });
+		const dfRes = spawnSync({ name: "df:/", command: ["df", "--output=pcent,target"], timeoutMs: 5_000 });
 		const dfOut = (dfRes.stdout ?? "").trim();
-		const rootLine = dfOut
-			.split("\n")
-			.find((l) => l.trim().endsWith("/"));
-		const pctMatch = rootLine?.match(/(\d+)%/);
-		if (pctMatch) {
-			const usePct = Number(pctMatch[1]);
-			if (usePct >= highPercent) {
-				log.warn(
-					`[host-cache-sweep] disk usage ${usePct}% >= ${highPercent}% threshold — emitting host.disk.high`,
-				);
-				recordEvent({
-					kind: "host.disk.high",
-					payload: { usePct, threshold: highPercent, df: dfOut, completedAt: new Date().toISOString() },
-				});
-			}
+		const usePct = parseRootDiskPct(dfOut);
+		if (usePct !== null && usePct >= diskHighPct) {
+			log.warn("host.disk.high", { usePct, threshold: diskHighPct, df: dfOut });
+			recordEvent({
+				kind: "host.disk.high",
+				payload: { usePct, threshold: diskHighPct, df: dfOut, completedAt: new Date().toISOString() },
+			});
 		}
 	} catch (error) {
-		log.warn(`[host-cache-sweep] disk-threshold check failed: ${String(error)}`);
+		log.warn("[host-cache-sweep] disk-threshold read failed: " + String(error));
 	}
 }