@oh-my-pi/pi-utils 17.2.12 → 17.2.14

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## [Unreleased]
 
+## [17.2.13] - 2026-08-11
+
+### Changed
+
+- Changed stale process-log retention from the newest five files globally to one newest file per completed process and day within the current and previous four local calendar days. This preserves bounded daily diagnostic coverage while continuing to remove one-use audit files.
+- Changed outbound User-Agent consumers to share the versioned `USER_AGENT` constant (`omp/<version>`).
+
+### Fixed
+
+- Fixed Mermaid ASCII state pseudostates rendering empty boxes, miscoloring final-state borders, and inverting rounded corners in bottom-to-top diagrams.
+
 ## [17.2.11] - 2026-08-07
 
 ### Added

package/dist/types/dirs.d.ts

@@ -18,6 +18,8 @@ export declare const CONFIG_DIR_NAME: string;
 export declare const MAIN_CONFIG_FILENAMES: readonly ["config.yml", "config.yaml"];
 /** Version (e.g. "1.0.0") */
 export declare const VERSION: string;
+/** Default User-Agent header string (e.g. "omp/17.2.12") */
+export declare const USER_AGENT: string;
 /** Minimum Bun version */
 export declare const MIN_BUN_VERSION: string;
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "17.2.12",
+	"version": "17.2.14",
 	"description": "Shared utilities for pi packages",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -31,7 +31,7 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "17.2.12"
+		"@oh-my-pi/pi-natives": "17.2.14"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14"

package/src/dirs.ts

@@ -28,6 +28,9 @@ export const MAIN_CONFIG_FILENAMES = ["config.yml", "config.yaml"] as const;
 /** Version (e.g. "1.0.0") */
 export const VERSION: string = version;
 
+/** Default User-Agent header string (e.g. "omp/17.2.12") */
+export const USER_AGENT = `omp/${VERSION}`;
+
 /** Minimum Bun version */
 export const MIN_BUN_VERSION: string = engines.bun.replace(/[^0-9.]/g, "");
 

package/src/logger.ts

@@ -53,9 +53,11 @@ function emitToSinks(level: LogLevel, message: string, context: Record<string, u
 	}
 }
 
-const PROCESS_LOG_PATTERN = /^omp\.\d{4}-\d{2}-\d{2}\.(\d+)\.log(?:\.\d+)?$/;
+const PROCESS_LOG_PATTERN = /^omp\.(\d{4}-\d{2}-\d{2})\.(\d+)\.log(?:\.(\d+))?$/;
 const PROCESS_AUDIT_PATTERN = /^\.omp\.(\d+)-audit\.json$/;
-const RETAINED_STALE_LOG_FILES = 5;
+const RETAINED_STALE_LOGS_PER_PROCESS_DAY = 1;
+const RETAINED_STALE_AUDIT_FILES = 0;
+const RETAINED_STALE_LOG_DAYS = 5;
 
 function processIsRunning(pid: number): boolean {
 	try {
@@ -67,8 +69,10 @@ function processIsRunning(pid: number): boolean {
 }
 
 /**
- * Retain the newest completed-process logs globally and remove their one-use
- * audit files. Live PID namespaces are never touched.
+ * Retain one newest completed-process log per process/day within the current
+ * and previous four local calendar days, and remove one-use audit files. Live
+ * PID namespaces are never touched. The calendar-day boundary preserves daily
+ * diagnostic coverage while bounding completed-process storage and scans.
  */
 function pruneStaleProcessLogs(dir: string): void {
 	let entries: fs.Dirent[];
@@ -77,38 +81,69 @@ function pruneStaleProcessLogs(dir: string): void {
 	} catch {
 		return;
 	}
-
-	const staleLogs: Array<{ path: string; mtimeMs: number }> = [];
+	const current = new Date();
+	const currentDate =
+		`${current.getFullYear()}-${String(current.getMonth() + 1).padStart(2, "0")}-` +
+		String(current.getDate()).padStart(2, "0");
+	const cutoff = new Date(current);
+	cutoff.setDate(cutoff.getDate() - (RETAINED_STALE_LOG_DAYS - 1));
+	const cutoffDate =
+		`${cutoff.getFullYear()}-${String(cutoff.getMonth() + 1).padStart(2, "0")}-` +
+		String(cutoff.getDate()).padStart(2, "0");
+
+	const staleLogsByProcessDay = new Map<string, Array<{ path: string; mtimeMs: number; rollover: number }>>();
 	for (const entry of entries) {
 		if (!entry.isFile()) continue;
 		const logMatch = PROCESS_LOG_PATTERN.exec(entry.name);
 		const auditMatch = PROCESS_AUDIT_PATTERN.exec(entry.name);
-		const pidText = logMatch?.[1] ?? auditMatch?.[1];
+		const pidText = logMatch?.[2] ?? auditMatch?.[1];
 		if (!pidText || processIsRunning(Number(pidText))) continue;
 		const entryPath = path.join(dir, entry.name);
 
 		if (auditMatch) {
+			if (RETAINED_STALE_AUDIT_FILES === 0) {
+				try {
+					fs.rmSync(entryPath, { force: true });
+				} catch {
+					// Retention is best-effort; logging must still initialize.
+				}
+			}
+			continue;
+		}
+		if (!logMatch?.[1]) continue;
+		if (logMatch[1] < cutoffDate || logMatch[1] > currentDate) {
 			try {
 				fs.rmSync(entryPath, { force: true });
 			} catch {
-				// Retention is best-effort; logging must still initialize.
+				// Another process may have pruned the same stale namespace.
 			}
 			continue;
 		}
 
 		try {
-			staleLogs.push({ path: entryPath, mtimeMs: fs.statSync(entryPath).mtimeMs });
+			const key = `${pidText}:${logMatch[1]}`;
+			const staleLogs = staleLogsByProcessDay.get(key) ?? [];
+			staleLogs.push({
+				path: entryPath,
+				mtimeMs: fs.statSync(entryPath).mtimeMs,
+				rollover: Number(logMatch[3] ?? 0),
+			});
+			staleLogsByProcessDay.set(key, staleLogs);
 		} catch {
 			// Another process may have pruned the same stale namespace.
 		}
 	}
 
-	staleLogs.sort((a, b) => b.mtimeMs - a.mtimeMs);
-	for (const stale of staleLogs.slice(RETAINED_STALE_LOG_FILES)) {
-		try {
-			fs.rmSync(stale.path, { force: true });
-		} catch {
-			// Another process may have pruned the same stale namespace.
+	for (const staleLogs of staleLogsByProcessDay.values()) {
+		staleLogs.sort(
+			(a, b) => b.mtimeMs - a.mtimeMs || b.rollover - a.rollover || (a.path < b.path ? -1 : a.path > b.path ? 1 : 0),
+		);
+		for (const stale of staleLogs.slice(RETAINED_STALE_LOGS_PER_PROCESS_DAY)) {
+			try {
+				fs.rmSync(stale.path, { force: true });
+			} catch {
+				// Another process may have pruned the same stale namespace.
+			}
 		}
 	}
 }

package/src/vendor/mermaid-ascii/ascii/canvas.ts

@@ -370,6 +370,8 @@ const VERTICAL_FLIP_MAP: Record<string, string> = {
   // Unicode corners
   '┌': '└', '└': '┌',
   '┐': '┘', '┘': '┐',
+  '╭': '╰', '╰': '╭',
+  '╮': '╯', '╯': '╮',
   // Unicode junctions (T-pieces flip vertically)
   '┬': '┴', '┴': '┬',
   // Box-start junctions (exit points from node boxes)

package/src/vendor/mermaid-ascii/ascii/draw.ts

@@ -76,16 +76,18 @@ function drawBoxWithGridDimensions(node: AsciiNode, graph: AsciiGraph): Canvas {
   // Get corner characters for this shape type
   const corners = getCorners(node.shape, useAscii)
 
-  // State-end uses double border to differentiate from state-start
-  const isDoubleBox = node.shape === 'state-end'
-  const hChar = useAscii ? (isDoubleBox ? '=' : '-') : (isDoubleBox ? '═' : '─')
-  const vChar = useAscii ? (isDoubleBox ? '‖' : '|') : (isDoubleBox ? '║' : '│')
-
-  // Double-box corners (for state-end)
-  const doubleCorners = useAscii
+  const isStateStart = node.shape === 'state-start'
+  const isStateEnd = node.shape === 'state-end'
+  const hChar = useAscii ? (isStateEnd ? '=' : '-') : (isStateEnd ? '═' : '─')
+  const vChar = useAscii ? (isStateEnd ? '‖' : '|') : (isStateEnd ? '║' : '│')
+
+  const stateStartCorners = useAscii
+    ? { tl: '+', tr: '+', bl: '+', br: '+' }
+    : { tl: '╭', tr: '╮', bl: '╰', br: '╯' }
+  const stateEndCorners = useAscii
     ? { tl: '#', tr: '#', bl: '#', br: '#' }
     : { tl: '╔', tr: '╗', bl: '╚', br: '╝' }
-  const effectiveCorners = isDoubleBox ? doubleCorners : corners
+  const effectiveCorners = isStateEnd ? stateEndCorners : isStateStart ? stateStartCorners : corners
 
   // Draw box border with shape-specific corners
   for (let x = from.x + 1; x < to.x; x++) box[x]![from.y] = hChar
@@ -97,8 +99,8 @@ function drawBoxWithGridDimensions(node: AsciiNode, graph: AsciiGraph): Canvas {
   box[from.x]![to.y] = effectiveCorners.bl
   box[to.x]![to.y] = effectiveCorners.br
 
-  // Center the multi-line display label inside the box
-  const label = node.displayLabel
+  // Pseudostates have no source label; restore their UML marker explicitly.
+  const label = node.displayLabel || (isStateStart ? (useAscii ? '*' : '●') : isStateEnd ? (useAscii ? '*' : '◎') : '')
   const lines = splitLines(label)
   const textCenterY = from.y + Math.floor(h / 2)
   const startY = textCenterY - Math.floor((lines.length - 1) / 2)
@@ -1223,7 +1225,8 @@ function fillRolesFromCanvases(
 
 /**
  * Special handling for node boxes: border chars get 'border' role, text gets 'text' role.
- * Detects text by checking if character is alphanumeric or common punctuation.
+ * Common final-state border characters (`#` and `=`) count only on the outer edge,
+ * so identical characters inside ordinary node labels retain the text role.
  */
 function fillRolesForNodeBox(
   roleCanvas: RoleCanvas,
@@ -1231,7 +1234,9 @@ function fillRolesForNodeBox(
   offset: DrawingCoord,
 ): void {
   const isBorderChar = (c: string) => /^[┌┐└┘├┤┬┴┼│─╭╮╰╯+\-|.':]$/.test(c)
-
+  const isStateEndBorderChar = (c: string) => /^[╔╗╚╝═║#=‖]$/.test(c)
+  const maxX = canvas.length - 1
+  const maxY = (canvas[0]?.length ?? 1) - 1
   for (let x = 0; x < canvas.length; x++) {
     for (let y = 0; y < (canvas[0]?.length ?? 0); y++) {
       const char = canvas[x]?.[y]
@@ -1240,7 +1245,9 @@ function fillRolesForNodeBox(
         const ry = y + offset.y
         // Use setRole which auto-expands the role canvas if needed
         if (rx >= 0 && ry >= 0) {
-          setRole(roleCanvas, rx, ry, isBorderChar(char) ? 'border' : 'text')
+          const isOuterEdge = x === 0 || x === maxX || y === 0 || y === maxY
+          const role = isBorderChar(char) || (isOuterEdge && isStateEndBorderChar(char)) ? 'border' : 'text'
+          setRole(roleCanvas, rx, ry, role)
         }
       }
     }