hippo-memory 1.43.3 → 1.45.0

package/dist/capture.js

@@ -642,18 +642,17 @@ export function summariseTranscript(jsonl) {
 /**
  * Resolve a transcript path for `--last-session`.
  *
- * Priority:
+ * Priority, where the first source present is the only one tried:
  *   1. Explicit `transcriptPath` option (from `--transcript <path>`)
  *   2. Stdin JSON payload (Claude Code / OpenCode SessionEnd hook shape)
- *   3. Most recent `.jsonl` under `~/.claude/projects/<any>/`
+ *   3. Most recent `.jsonl` under `~/.claude/projects/<any>/`, only on a proven manual run (no path, no stdin text, no timed-out read), because this scan spans every project on the box
  *
- * Returns null when nothing resolves. Never throws.
+ * Returns null when nothing resolves, a named transcript or payload whose file is missing included. Never throws.
  */
-export function resolveLastSessionTranscript(explicit, stdinText) {
-    if (explicit && fs.existsSync(explicit))
-        return explicit;
-    // Try parsing stdin as the SessionEnd JSON payload
-    if (stdinText && stdinText.trim().startsWith('{')) {
+export function resolveLastSessionTranscript(explicit, stdinText, stdinTimedOut = false) {
+    if (explicit)
+        return fs.existsSync(explicit) ? explicit : null;
+    if (stdinText && stdinText.trim() !== '') {
         try {
             const payload = JSON.parse(stdinText);
             if (isObjectLike(payload) && 'transcript_path' in payload) {
@@ -663,10 +662,12 @@ export function resolveLastSessionTranscript(explicit, stdinText) {
             }
         }
         catch {
-            // not JSON - fall through
+            // not JSON, but still a payload, so no scan
         }
+        return null;
     }
-    // Auto-discover the most recent transcript
+    if (stdinTimedOut)
+        return null;
     const home = process.env.HOME || process.env.USERPROFILE;
     if (!home)
         return null;
@@ -803,21 +804,7 @@ function cmdCaptureCore(hippoRoot, options) {
             break;
         }
         case 'last-session': {
-            // Try to read stdin non-blockingly: SessionEnd hooks pass a JSON payload,
-            // but manual / test invocations have no piped stdin. fs.readFileSync(0)
-            // will block waiting for input when run interactively, so:
-            //   - skip entirely when caller passed an explicit --transcript path
-            //   - skip when stdin is a TTY (interactive shell)
-            let stdinText;
-            if (!options.transcriptPath && !process.stdin.isTTY) {
-                try {
-                    stdinText = fs.readFileSync(0, 'utf8');
-                }
-                catch {
-                    stdinText = undefined;
-                }
-            }
-            const resolved = resolveLastSessionTranscript(options.transcriptPath, stdinText);
+            const resolved = resolveLastSessionTranscript(options.transcriptPath, options.stdinText, options.stdinTimedOut);
             if (!resolved) {
                 console.log('No transcript found. Pass --transcript <path> or run from a SessionEnd hook.');
                 return;
@@ -1127,15 +1114,20 @@ function isReadableFile(filePath) {
  * off along the way (empty on every skip path) so `cmdPreCompact` can await
  * them, bounded, before it exits (X6).
  */
-function runPreCompact(hippoRoot, stdinText, logFile) {
+function runPreCompact(hippoRoot, stdinText, stdinTimedOut, logFile) {
     // X3: the PreCompact hook fires in every Claude Code project, including
-    // ones that never ran `hippo init`. Gate on the non-exiting isInitialized
-    // check BEFORE any store-opening call (saveActiveTaskSnapshot etc. all
-    // call initStore internally, which would silently create a store here).
+    // ones that never ran `hippo init`, so gate before any store-opening call
+    // (saveActiveTaskSnapshot etc. call initStore, which would create one).
     if (!isInitialized(hippoRoot)) {
         appendPreCompactLog(logFile, 'skip: store not initialized');
         return [];
     }
+    // Same hazard X4 guards below, different trigger: a read that timed out
+    // must not reach auto-discovery either, or it snapshots another session.
+    if (stdinTimedOut && (!stdinText || stdinText.trim() === '')) {
+        appendPreCompactLog(logFile, 'skip: no PreCompact payload arrived before the stdin wait window closed');
+        return [];
+    }
     // A true manual invocation has no stdin at all (TTY, or a non-TTY pipe
     // that yielded an empty read) — that's the ONLY case newest-transcript
     // auto-discovery is allowed to run. Any other non-empty stdin must
@@ -1333,7 +1325,7 @@ export async function cmdPreCompact(hippoRoot, options) {
     const logFile = options.logFile ?? defaultPreCompactLogPath();
     let embeds = [];
     try {
-        embeds = runPreCompact(hippoRoot, options.stdinText, logFile);
+        embeds = runPreCompact(hippoRoot, options.stdinText, options.stdinTimedOut ?? false, logFile);
     }
     catch (err) {
         appendPreCompactLog(logFile, `pre-compact failed: ${errorMessage(err)}`);

package/dist/cli.d.ts

@@ -33,6 +33,8 @@ import { MemoryEntry } from './memory.js';
 import * as api from './api.js';
 /** Test-only: reset the module-level recall-history Map. Call from beforeEach. */
 export declare function __resetSessionRecallHistoryCli(): void;
+export declare const BOOLEAN_FLAGS: ReadonlySet<string>;
+export declare const KNOWN_FLAGS: ReadonlySet<string>;
 export declare function parseArgs(argv: string[]): {
     command: string;
     args: string[];
@@ -49,6 +51,7 @@ export declare function learnFromMemoryMd(hippoRoot: string, homeDir?: string):
  */
 /** @internal — exported for snapshot tests (tests/cli-context-render-snapshot.test.ts). NOT a stable public API. */
 export declare function renderSleepResult(result: api.SleepResult): void;
+export declare function shouldAutoRepairCodexWrapper(currentCommand: string, flags: Record<string, string | boolean | string[]>): boolean;
 /** @internal — exported for snapshot tests (tests/cli-context-render-snapshot.test.ts). NOT a stable public API. */
 export declare function printContextMarkdown(items: Array<{
     entry: MemoryEntry;
@@ -56,4 +59,5 @@ export declare function printContextMarkdown(items: Array<{
     tokens: number;
     isGlobal: boolean;
 }>, totalTokens: number, framing?: string): void;
+export declare function runCli(argv?: string[]): Promise<void>;
 //# sourceMappingURL=cli.d.ts.map