@ai-devkit/agent-manager 0.10.0 → 0.11.0

package/src/utils/session.ts

@@ -1,12 +1,12 @@
 /**
  * Session File Utilities
  *
- * Shell command wrappers for discovering session files and their birth times.
- * Uses `stat` to get exact epoch-second birth timestamps without reading file contents.
+ * Utilities for discovering session files and their birth times.
+ * Uses Node.js fs APIs to get birth timestamps without reading file contents.
  */
 
+import * as fs from 'fs';
 import * as path from 'path';
-import { execSync } from 'child_process';
 
 /**
  * Represents a session file with its birth time metadata.
@@ -29,63 +29,104 @@ export interface SessionFile {
 }
 
 /**
- * Get birth times for .jsonl session files across multiple directories in a single shell call.
- *
- * Combines all directory globs into one `stat` command to avoid per-directory exec overhead.
- * Returns empty array if no directories have .jsonl files or command fails.
- * resolvedCwd is left empty — the adapter must set it.
+ * Check whether a path exists and is a directory.
+ * Returns false on any error (missing path, permission denied, broken symlink, etc.).
  */
-export function batchGetSessionFileBirthtimes(dirs: string[]): SessionFile[] {
-    if (dirs.length === 0) return [];
+export function isDirectory(p: string): boolean {
+    return safeStat(p)?.isDirectory() ?? false;
+}
 
+/**
+ * `fs.statSync` that swallows errors and returns `undefined` on failure.
+ * Callers can pull whichever fields they need (mtime, birthtime, ...).
+ */
+export function safeStat(filePath: string): fs.Stats | undefined {
     try {
-        const isMacOS = process.platform === 'darwin';
-        const globs = dirs.map((d) => `"${d}"/*.jsonl`).join(' ');
-        // || true prevents non-zero exit when some globs have no .jsonl matches
-        const command = isMacOS
-            ? `stat -f '%B %N' ${globs} 2>/dev/null || true`
-            : `stat --format='%W %n' ${globs} 2>/dev/null || true`;
+        return fs.statSync(filePath);
+    } catch {
+        return undefined;
+    }
+}
 
-        const output = execSync(command, { encoding: 'utf-8' });
+/**
+ * `fs.readFileSync` (utf-8) that swallows errors and returns `undefined`
+ * on failure. Use when an unreadable file should be skipped rather than
+ * raised.
+ */
+export function safeReadFile(filePath: string): string | undefined {
+    try {
+        return fs.readFileSync(filePath, 'utf-8');
+    } catch {
+        return undefined;
+    }
+}
 
-        return parseStatOutput(output);
+/**
+ * `fs.readdirSync` that swallows errors and returns `[]` on failure.
+ * Useful when walking optional/transient directories where missing or
+ * unreadable entries should be skipped silently.
+ */
+export function safeReaddir(dir: string): string[] {
+    try {
+        return fs.readdirSync(dir);
     } catch {
         return [];
     }
 }
 
 /**
- * Parse stat output lines into SessionFile entries.
+ * List entries in a directory that end with `.jsonl`. Returns `[]` on
+ * read errors. The result preserves directory order (no sorting).
  */
-function parseStatOutput(output: string): SessionFile[] {
-    const results: SessionFile[] = [];
-
-    for (const rawLine of output.trim().split('\n')) {
-        const line = rawLine.trim();
-        if (!line) continue;
-
-        // Format: "<epoch_seconds> <filepath>"
-        const spaceIdx = line.indexOf(' ');
-        if (spaceIdx === -1) continue;
-
-        const epochStr = line.slice(0, spaceIdx);
-        const filePath = line.slice(spaceIdx + 1).trim();
-
-        const epochSeconds = parseInt(epochStr, 10);
-        if (!Number.isFinite(epochSeconds) || epochSeconds <= 0) continue;
+export function listJsonl(dir: string): string[] {
+    return safeReaddir(dir).filter((name) => name.endsWith('.jsonl'));
+}
 
-        const fileName = path.basename(filePath);
-        if (!fileName.endsWith('.jsonl')) continue;
+/**
+ * Get birth times for .jsonl session files across multiple directories.
+ *
+ * Enumerates each directory with readdirSync and stats each .jsonl file
+ * to get its birth time. No shell commands are used.
+ * Returns empty array if no directories have .jsonl files or reads fail.
+ * resolvedCwd is left empty — the adapter must set it.
+ */
+export function batchGetSessionFileBirthtimes(dirs: string[]): SessionFile[] {
+    if (dirs.length === 0) return [];
 
-        const sessionId = fileName.replace(/\.jsonl$/, '');
+    const results: SessionFile[] = [];
 
-        results.push({
-            sessionId,
-            filePath,
-            projectDir: path.dirname(filePath),
-            birthtimeMs: epochSeconds * 1000,
-            resolvedCwd: '',
-        });
+    for (const dir of dirs) {
+        let entries: string[];
+        try {
+            entries = fs.readdirSync(dir);
+        } catch {
+            continue;
+        }
+
+        for (const entry of entries) {
+            if (!entry.endsWith('.jsonl')) continue;
+
+            const filePath = path.join(dir, entry);
+
+            let birthtimeMs: number;
+            try {
+                birthtimeMs = fs.statSync(filePath).birthtimeMs;
+            } catch {
+                continue;
+            }
+
+            if (!Number.isFinite(birthtimeMs) || birthtimeMs <= 0) continue;
+
+            const sessionId = entry.replace(/\.jsonl$/, '');
+
+            results.push({
+                sessionId,
+                filePath,
+                projectDir: dir,
+                birthtimeMs,
+                resolvedCwd: '',
+            });
+        }
     }
 
     return results;