@jhizzard/termdeck 0.7.0 → 0.7.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jhizzard/termdeck",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "Browser-based terminal multiplexer with metadata overlays, panel flashback memory recall, and AI-aware session management",
   "bin": {
     "termdeck": "./packages/cli/src/index.js"

package/packages/cli/src/doctor.js

@@ -58,7 +58,7 @@ async function _detectInstalled(pkg) {
       child = spawn('npm', ['ls', '-g', pkg, '--depth=0', '--json'], {
         stdio: ['ignore', 'pipe', 'pipe'],
       });
-    } catch {
+    } catch (_err) {
       return resolve(null);
     }
 
@@ -80,7 +80,7 @@ async function _detectInstalled(pkg) {
         const dep = parsed && parsed.dependencies && parsed.dependencies[pkg];
         if (dep && typeof dep.version === 'string') return resolve(dep.version);
         return resolve(null);
-      } catch {
+      } catch (_err) {
         return resolve(null);
       }
     });
@@ -118,13 +118,13 @@ async function _fetchLatest(pkg) {
             const parsed = JSON.parse(body);
             if (parsed && typeof parsed.latest === 'string') return done(parsed.latest);
             return done(null);
-          } catch {
+          } catch (_err) {
             return done(null);
           }
         });
         res.on('error', () => done(null));
       });
-    } catch {
+    } catch (_err) {
       return done(null);
     }
     req.on('timeout', () => {

package/packages/cli/src/update-check.js

@@ -38,7 +38,7 @@ function defaultPackageVersion() {
   try {
     const pkg = require(path.join(__dirname, '..', '..', '..', 'package.json'));
     return pkg && pkg.version ? String(pkg.version) : null;
-  } catch {
+  } catch (_err) {
     return null;
   }
 }
@@ -68,7 +68,7 @@ function readCache(cachePath) {
     if (!parsed || typeof parsed !== 'object') return null;
     if (parsed.version !== CACHE_VERSION) return null;
     return parsed;
-  } catch {
+  } catch (_err) {
     return null;
   }
 }
@@ -77,7 +77,7 @@ function writeCache(cachePath, data) {
   try {
     fs.mkdirSync(path.dirname(cachePath), { recursive: true });
     fs.writeFileSync(cachePath, JSON.stringify(data, null, 2), 'utf8');
-  } catch {
+  } catch (_err) {
     // Read-only home, ENOSPC, race with another process — all benign here.
   }
 }
@@ -91,7 +91,7 @@ async function fetchLatest(registryUrl) {
     const json = await res.json();
     const latest = json && json.latest;
     return isValidSemver(latest) ? latest : null;
-  } catch {
+  } catch (_err) {
     return null;
   } finally {
     clearTimeout(timeout);
@@ -144,7 +144,7 @@ async function checkAndPrintHint(_config, opts) {
       '       Or run `termdeck doctor` for the whole stack. ' +
         'Suppress with TERMDECK_NO_UPDATE_CHECK=1.'
     );
-  } catch {
+  } catch (_err) {
     // Never throw from a fire-and-forget hook.
   }
 }

package/packages/server/src/mnestra-bridge/index.js

@@ -231,13 +231,21 @@ function createBridge(config) {
     // back to resolving the session's cwd against config.projects so queries
     // don't leak into unrelated repos via basename collisions.
     let effectiveProject = project;
+    let projectSource = project ? 'explicit' : 'none';
     if (!effectiveProject) {
       const ctxCwd = cwd || (sessionContext && sessionContext.cwd);
       if (ctxCwd) {
         effectiveProject = resolveProjectName(ctxCwd, config);
+        projectSource = effectiveProject ? 'cwd' : 'none';
       }
     }
 
+    // Sprint 34 observability: every Flashback query announces its project tag
+    // and how it was resolved. If the writer chain is ever mis-emitting a tag
+    // (as happened pre-v0.7.2 with the `chopin-nashville` regression from the
+    // out-of-repo session-end hook), the mismatch surfaces here at query time.
+    console.log(`[mnestra-bridge] query project=${effectiveProject ?? 'ALL'} source=${searchAll ? 'searchAll' : projectSource} mode=${mode}`);
+
     switch (mode) {
       case 'webhook':
         return queryWebhook({ question, project: effectiveProject, searchAll });

package/packages/server/src/rag.js

@@ -97,53 +97,80 @@ class RAGIntegration {
 
   // Canonical project tag for a session. Prefers the explicit config.yaml name
   // (set at session creation), falls back to cwd → config.projects resolution.
+  // Returns { tag, source } so callers can audit which resolution path fired —
+  // explicit (session.meta.project), cwd (cwd matched a config.projects entry),
+  // fallback (cwd basename), or null (no cwd, no config). Sprint 34: the
+  // chopin-nashville mis-tag came from an out-of-repo writer, but source
+  // attribution here makes any future TermDeck-side regression visible in logs.
+  _resolveProjectAttribution(session) {
+    if (session.meta.project) return { tag: session.meta.project, source: 'explicit' };
+    const tag = resolveProjectName(session.meta.cwd, this.config);
+    if (!tag) return { tag: null, source: 'none' };
+    const cwdResolved = session.meta.cwd && path.resolve(String(session.meta.cwd).replace(/^~/, os.homedir()));
+    const matchedConfig = !!cwdResolved && Object.values((this.config && this.config.projects) || {}).some((def) => {
+      if (!def || typeof def.path !== 'string') return false;
+      const p = path.resolve(def.path.replace(/^~/, os.homedir()));
+      return cwdResolved === p || cwdResolved.startsWith(p + path.sep);
+    });
+    return { tag, source: matchedConfig ? 'cwd' : 'fallback' };
+  }
+
   _projectFor(session) {
-    if (session.meta.project) return session.meta.project;
-    return resolveProjectName(session.meta.cwd, this.config);
+    return this._resolveProjectAttribution(session).tag;
+  }
+
+  // Single attribution + observability point for session events. Logs once per
+  // record() so future drift in the project-resolution chain (e.g. a writer
+  // that bypasses _projectFor and stamps a raw path segment) is visible in
+  // stdout. Cheap: ~one log line per RAG event, off the hot path.
+  _recordForSession(session, eventType, payload) {
+    const { tag, source } = this._resolveProjectAttribution(session);
+    console.log(`[rag] write project=${tag ?? 'null'} source=${source} session=${session.id} event=${eventType}`);
+    this.record(session.id, eventType, payload, tag);
   }
 
   // Event types to record
   onSessionCreated(session) {
-    this.record(session.id, 'session_created', {
+    this._recordForSession(session, 'session_created', {
       type: session.meta.type,
       command: session.meta.command,
       cwd: session.meta.cwd,
       reason: session.meta.reason
-    }, this._projectFor(session));
+    });
   }
 
   onCommandExecuted(session, command, outputSnippet) {
-    this.record(session.id, 'command_executed', {
+    this._recordForSession(session, 'command_executed', {
       command,
       output_snippet: outputSnippet?.slice(0, 500), // Truncate for storage
       type: session.meta.type
-    }, this._projectFor(session));
+    });
   }
 
   onStatusChanged(session, oldStatus, newStatus) {
-    this.record(session.id, 'status_changed', {
+    this._recordForSession(session, 'status_changed', {
       from: oldStatus,
       to: newStatus,
       detail: session.meta.statusDetail,
       type: session.meta.type
-    }, this._projectFor(session));
+    });
   }
 
   onSessionEnded(session) {
-    this.record(session.id, 'session_ended', {
+    this._recordForSession(session, 'session_ended', {
       type: session.meta.type,
       duration_ms: Date.now() - new Date(session.meta.createdAt).getTime(),
       command_count: session.meta.lastCommands.length,
       exit_code: session.meta.exitCode
-    }, this._projectFor(session));
+    });
   }
 
   onFileEdited(session, filepath, editType) {
-    this.record(session.id, 'file_edited', {
+    this._recordForSession(session, 'file_edited', {
       filepath,
       edit_type: editType,
       type: session.meta.type
-    }, this._projectFor(session));
+    });
   }
 
   // Circuit breaker check — returns true if pushes to this table are disabled.

package/packages/server/src/session.js

@@ -69,7 +69,15 @@ const PATTERNS = {
   // Stricter line-anchored variant for Claude Code, whose tool output (grep
   // results, test logs, file contents) routinely mentions "Error" mid-line
   // without representing an actual failure of the agent itself.
-  errorLineStart: /^\s*(error|Error|ERROR|exception|Exception|Traceback|fatal|FATAL|segmentation fault|panic|EACCES|ECONNREFUSED|ENOENT|command not found|undefined reference|cannot find module|failed with exit code|No such file or directory|Permission denied)\b/m
+  errorLineStart: /^\s*(error|Error|ERROR|exception|Exception|Traceback|fatal|FATAL|segmentation fault|panic|EACCES|ECONNREFUSED|ENOENT|command not found|undefined reference|cannot find module|failed with exit code|No such file or directory|Permission denied)\b/m,
+  // Sprint 33: PATTERNS.error misses the most common Unix shell errors —
+  // `cat: /foo: No such file or directory`, `bash: foo: command not found`,
+  // `rm: cannot remove ...: Permission denied`. These have a colon-prefix
+  // shape (`<cmd>: ...: <phrase>`) that distinguishes them from prose
+  // mentioning the same words. Each branch requires either the colon-prefix
+  // structure or a stand-alone anchored keyword. Validated against an
+  // adversarial prose suite (see tests/analyzer-error-fixtures.test.js).
+  shellError: /(?:^|\n)(?:[^\n]*:\s+(?:.*?:\s+)?(?:No such file or directory|Permission denied|Is a directory|Not a directory|command not found)\b|[^\n]*?\(\d+\)\s+Could not resolve host\b|\s*ModuleNotFoundError:\s+\S|\s*Segmentation fault\b|\s*fatal:\s+\S)/m
 };
 
 class Session {
@@ -345,7 +353,11 @@ class Session {
     const pattern = this.meta.type === 'claude-code'
       ? PATTERNS.errorLineStart
       : PATTERNS.error;
-    if (!pattern.test(clean)) return;
+    // Sprint 33 fix: the structured patterns above miss `cat: /foo: No such
+    // file or directory` and friends — the most common Unix shell error
+    // shapes Josh hits day-to-day. Fall through to PATTERNS.shellError so
+    // the analyzer flips status='errored' and Flashback can fire.
+    if (!pattern.test(clean) && !PATTERNS.shellError.test(clean)) return;
 
     const oldStatus = this.meta.status;
     this.meta.status = 'errored';

package/packages/server/src/theme-resolver.js

@@ -40,7 +40,7 @@ function getCurrentConfig() {
     const { loadConfig } = require('./config');
     _configCache = { mtimeMs: stat.mtimeMs, value: loadConfig(), frozen: false };
     return _configCache.value;
-  } catch {
+  } catch (_err) {
     return _configCache.value || {};
   }
 }