@geravant/sinain 1.23.1 → 1.23.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geravant/sinain",
-  "version": "1.23.1",
+  "version": "1.23.2",
   "description": "Ambient intelligence that sees what you see, hears what you hear, and acts on your behalf",
   "type": "module",
   "bin": {

package/sinain-core/src/audio/pipeline.ts

@@ -212,6 +212,8 @@ export class AudioPipeline extends EventEmitter {
 
     let headerSkipped = name !== "sox";
     let headerBuf = Buffer.alloc(0);
+    let stderrAccum = "";
+    const spawnTime = Date.now();
 
     proc.stdout?.on("data", (data: Buffer) => {
       if (!this.running) return;
@@ -237,6 +239,12 @@ export class AudioPipeline extends EventEmitter {
       if (msg && !/^In:.*Out:/.test(msg)) {
         log(TAG, `${name} stderr: ${msg.slice(0, 200)}`);
       }
+      // Accumulate stderr for TCC detection on exit
+      stderrAccum += data.toString();
+      // Cap accumulation to avoid unbounded growth (4KB is enough for any TCC message)
+      if (stderrAccum.length > 4096) {
+        stderrAccum = stderrAccum.slice(-4096);
+      }
     });
 
     proc.on("error", (err) => {
@@ -252,6 +260,45 @@ export class AudioPipeline extends EventEmitter {
       if (this.running && code !== 0) {
         this.errorCount++;
         this.profiler?.gauge("audio.errors", this.errorCount);
+
+        // Detect TCC (macOS Screen Recording / Microphone) permission denial.
+        // sck-capture logs "declined TCCs" to stderr when the entitlement is
+        // missing. The chicken-and-egg: clicking "Allow" on the prompt doesn't
+        // apply to a running process — the user must restart Terminal and
+        // re-run. We print a prominent banner and request graceful shutdown
+        // so users aren't left wondering why the agent never escalates.
+        const elapsedMs = Date.now() - spawnTime;
+        const isTccDenial = stderrAccum.includes("declined TCCs");
+        if (isTccDenial && elapsedMs < 5000) {
+          process.stdout.write([
+            "",
+            "=======================================================================",
+            "  WARNING: Screen Recording permission needed",
+            "=======================================================================",
+            "",
+            "  sck-capture cannot access screen capture and audio without",
+            "  TCC (Screen Recording) permission from macOS.",
+            "",
+            "  If you just clicked Allow -- that is normal! macOS does not apply",
+            "  the permission to processes that are already running. To fix:",
+            "",
+            "    1. Press Ctrl+C to stop sinain",
+            "    2. Quit and restart your Terminal app (Cmd+Q, then reopen)",
+            "    3. Run again: npx @geravant/sinain@latest start",
+            "",
+            "  Already declined? Re-grant permission:",
+            "    System Settings > Privacy & Security > Screen Recording",
+            "    > enable Terminal (or your terminal app)",
+            "",
+            "=======================================================================",
+            "",
+          ].join("\n"));
+
+          // Emit TCC-specific error so index.ts can initiate graceful shutdown
+          this.emit("tcc-denied");
+          return;
+        }
+
         warn(TAG, `${name} exited unexpectedly, stopping pipeline`);
         this.stop();
       }

package/sinain-core/src/index.ts

@@ -543,7 +543,14 @@ async function importKnowledgeToLocal(data: string): Promise<string> {
   const dbPath = `${localDir}/knowledge-graph.db`;
 
   const __dir = dirname(fileURLToPath(import.meta.url));
-  const scriptsDir = resolve(__dir, "..", "..", "sinain-hud-plugin", "sinain-memory");
+  // Two package layouts are supported:
+  //   dev/monorepo: <repo>/sinain-core/src/ → ../../sinain-hud-plugin/sinain-memory
+  //   npm-published flat: <pkg>/sinain-core/src/ → ../../sinain-memory
+  const scriptsDir = [
+    resolve(__dir, "..", "..", "sinain-hud-plugin", "sinain-memory"),  // dev/monorepo layout
+    resolve(__dir, "..", "..", "sinain-memory"),                         // npm-published flat layout
+    resolve(__dir, "..", "sinain-memory"),                               // legacy alt
+  ].find(p => existsSync(`${p}/triplestore.py`)) || resolve(__dir, "..", "..", "sinain-memory");
 
   // Convert facts to graph ops for knowledge_integrator
   const graphOps = facts.map((f: any) => ({
@@ -834,6 +841,12 @@ async function main() {
     wsHandler.updateState({ audio: "muted" });
   });
 
+  systemAudioPipeline.on("tcc-denied", () => {
+    // Banner already printed by pipeline.ts. Initiate graceful shutdown so
+    // sinain exits cleanly rather than continuing with audio dead.
+    shutdown("TCC-DENIED").catch(() => process.exit(1));
+  });
+
   systemAudioPipeline.on("muted", () => {
     log(TAG, "system audio muted (capture process still running)");
     wsHandler.updateState({ audio: "muted" });

package/sinain-core/src/learning/entity-cache.ts

@@ -55,10 +55,14 @@ export class EntityCache {
         return;
       }
 
-      // Query entity names directly via SQLite
+      // Query entity names directly via SQLite.
+      // Two package layouts are supported:
+      //   dev/monorepo: <repo>/sinain-core/src/learning/ → ../../../sinain-hud-plugin/sinain-memory
+      //   npm-published flat: <pkg>/sinain-core/src/learning/ → ../../../sinain-memory
       const scriptCandidates = [
-        resolve(__dir, "..", "..", "sinain-hud-plugin", "sinain-memory", "graph_query.py"),
-        resolve(__dir, "..", "sinain-memory", "graph_query.py"),
+        resolve(__dir, "..", "..", "..", "sinain-hud-plugin", "sinain-memory", "graph_query.py"),  // dev/monorepo layout
+        resolve(__dir, "..", "..", "..", "sinain-memory", "graph_query.py"),                        // npm-published flat layout
+        resolve(__dir, "..", "..", "sinain-memory", "graph_query.py"),                             // legacy alt
       ];
       const scriptPath = scriptCandidates.find(p => existsSync(p));
       if (!scriptPath) return;

package/sinain-core/src/learning/local-curation.ts

@@ -23,18 +23,26 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
 
 /** Resolve the sinain-memory Python scripts directory. */
 function resolveScriptsDir(): string {
-  // Look for sinain-memory scripts in known locations
+  // Look for sinain-memory scripts in known locations.
+  // Two package layouts are supported:
+  //   dev/monorepo: <repo>/sinain-core/src/learning/ → ../../../sinain-hud-plugin/sinain-memory
+  //   npm-published flat: <pkg>/sinain-core/src/learning/ → ../../../sinain-memory
   const candidates = [
-    resolve(__dirname, "..", "..", "..", "sinain-hud-plugin", "sinain-memory"),
-    resolve(__dirname, "..", "..", "sinain-memory"),
-    resolve(process.env.HOME || "", ".sinain", "sinain-memory"),
+    resolve(__dirname, "..", "..", "..", "sinain-hud-plugin", "sinain-memory"),  // dev/monorepo layout
+    resolve(__dirname, "..", "..", "..", "sinain-memory"),                         // npm-published flat layout
+    resolve(__dirname, "..", "..", "sinain-memory"),                               // legacy alt
+    resolve(process.env.HOME || "", ".sinain", "sinain-memory"),                  // user-local fallback
   ];
   for (const dir of candidates) {
     if (existsSync(resolve(dir, "session_distiller.py"))) {
       return dir;
     }
   }
-  return candidates[0]; // Fallback
+  error(TAG, `sinain-memory scripts not found. Searched ${candidates.length} locations:`);
+  for (const dir of candidates) {
+    error(TAG, `  - ${dir}`);
+  }
+  return candidates[candidates.length - 1]; // Return user-local path as sentinel
 }
 
 /** Resolve the local memory directory. */