principles-disciple 1.190.0 → 1.192.0

package/dist/core/system-logger.d.ts

@@ -3,6 +3,22 @@ export declare const SystemLogger: {
     /**
      * Force refresh of the cached log file path.
      * Call this at midnight or when the date changes to ensure proper rotation.
+     *
+     * PRI-504: clears log-file and log-date caches for ALL workspaces (preserves
+     * the original "force refresh" semantics). Does NOT clear lastCleanupDates
+     * to avoid re-triggering cleanup for every workspace on the same day.
      */
     refreshCache(): void;
 };
+/**
+ * PRI-504: dispose the SystemLogger cache for a single workspace.
+ * Useful for tests and for workspace teardown in multi-workspace processes.
+ * Pattern follows evolution-engine.ts:564-570 (disposeEvolutionEngine).
+ */
+export declare function disposeSystemLogger(workspaceDir: string): void;
+/**
+ * PRI-504: dispose ALL SystemLogger caches. Call this in test afterEach
+ * hooks to prevent state leakage between tests. Pattern follows
+ * evolution-engine.ts:572-577 (disposeAllEvolutionEngines).
+ */
+export declare function disposeAllSystemLoggers(): void;

package/dist/core/system-logger.js

@@ -11,9 +11,13 @@ import { resolvePdPath, PD_FILES } from './paths.js';
  * Automatically rotates to a new file at midnight.
  * Old log files are automatically cleaned up based on retention policy.
  */
-// Cache for current log file path, invalidated when date changes
-let cachedLogFile;
-let cachedLogDate;
+// PRI-504: per-workspace caches keyed by path.resolve(workspaceDir).
+// Previously these were module-level `let` variables, so the first workspace
+// to call log() would pin the log file path for ALL subsequent workspaces in
+// the same process — causing cross-workspace log leakage (ERR-092).
+// Pattern follows evolution-engine.ts:551-577 (Map + path.resolve + dispose).
+const cachedLogFiles = new Map();
+const cachedLogDates = new Map();
 /**
  * Log retention: delete SYSTEM logs older than this many days.
  * Set to 0 to disable cleanup.
@@ -64,28 +68,33 @@ function cleanupOldLogs(workspaceDir) {
         // Silently fail cleanup - don't crash logging for cleanup failures
     }
 }
-// Track if cleanup has run today
-let lastCleanupDate;
+// PRI-504: per-workspace cleanup tracker (was module-level `let`).
+// Keyed by path.resolve(workspaceDir) so each workspace runs its own
+// daily cleanup independently.
+const lastCleanupDates = new Map();
 export const SystemLogger = {
     log(workspaceDir, eventType, message) {
         if (!workspaceDir)
             return;
         try {
+            const resolved = path.resolve(workspaceDir);
             const today = getTodayStr();
-            // Check if date changed - invalidate cache and run cleanup
-            if (cachedLogDate !== today) {
-                cachedLogDate = today;
-                cachedLogFile = undefined;
-                // Run cleanup once per day when date changes
-                if (lastCleanupDate !== today) {
-                    lastCleanupDate = today;
+            // Check if date changed for THIS workspace - invalidate its cache and run cleanup
+            if (cachedLogDates.get(resolved) !== today) {
+                cachedLogDates.set(resolved, today);
+                cachedLogFiles.delete(resolved);
+                // Run cleanup once per day per workspace when date changes
+                if (lastCleanupDates.get(resolved) !== today) {
+                    lastCleanupDates.set(resolved, today);
                     cleanupOldLogs(workspaceDir);
                 }
             }
-            // Get or create log file path
-            if (!cachedLogFile) {
-                cachedLogFile = getSystemLogPath(workspaceDir, new Date());
-                const logDir = path.dirname(cachedLogFile);
+            // Get or create log file path for this workspace
+            let logFile = cachedLogFiles.get(resolved);
+            if (!logFile) {
+                logFile = getSystemLogPath(workspaceDir, new Date());
+                cachedLogFiles.set(resolved, logFile);
+                const logDir = path.dirname(logFile);
                 if (!fs.existsSync(logDir)) {
                     fs.mkdirSync(logDir, { recursive: true });
                 }
@@ -94,7 +103,7 @@ export const SystemLogger = {
             // Format: [YYYY-MM-DDTHH:mm:ss.sssZ] [EVENT_TYPE] Message
             const logEntry = `[${timestamp}] [${eventType.padEnd(15)}] ${message}\n`;
             // Use fire-and-forget async append to prevent blocking
-            fs.appendFile(cachedLogFile, logEntry, 'utf8', (_err) => {
+            fs.appendFile(logFile, logEntry, 'utf8', (_err) => {
                 // Silently drop errors (e.g. disk full) to not crash the gateway
             });
         }
@@ -105,9 +114,34 @@ export const SystemLogger = {
     /**
      * Force refresh of the cached log file path.
      * Call this at midnight or when the date changes to ensure proper rotation.
+     *
+     * PRI-504: clears log-file and log-date caches for ALL workspaces (preserves
+     * the original "force refresh" semantics). Does NOT clear lastCleanupDates
+     * to avoid re-triggering cleanup for every workspace on the same day.
      */
     refreshCache() {
-        cachedLogDate = undefined;
-        cachedLogFile = undefined;
+        cachedLogFiles.clear();
+        cachedLogDates.clear();
     }
 };
+/**
+ * PRI-504: dispose the SystemLogger cache for a single workspace.
+ * Useful for tests and for workspace teardown in multi-workspace processes.
+ * Pattern follows evolution-engine.ts:564-570 (disposeEvolutionEngine).
+ */
+export function disposeSystemLogger(workspaceDir) {
+    const resolved = path.resolve(workspaceDir);
+    cachedLogFiles.delete(resolved);
+    cachedLogDates.delete(resolved);
+    lastCleanupDates.delete(resolved);
+}
+/**
+ * PRI-504: dispose ALL SystemLogger caches. Call this in test afterEach
+ * hooks to prevent state leakage between tests. Pattern follows
+ * evolution-engine.ts:572-577 (disposeAllEvolutionEngines).
+ */
+export function disposeAllSystemLoggers() {
+    cachedLogFiles.clear();
+    cachedLogDates.clear();
+    lastCleanupDates.clear();
+}

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "principles-disciple",
   "name": "Principles Disciple",
   "description": "Evolutionary programming agent framework with strategic guardrails and reflection loops.",
-  "version": "1.190.0",
+  "version": "1.192.0",
   "activation": {
     "onCapabilities": [
       "hook"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "principles-disciple",
-  "version": "1.190.0",
+  "version": "1.192.0",
   "description": "Native OpenClaw plugin for Principles Disciple",
   "type": "module",
   "main": "./dist/index.js",