hippo-memory 1.61.0 → 1.62.0

package/dist/cli/context.js

@@ -86,16 +86,8 @@ async function renderContext(hippoRoot, args, flags, stdinText, rec) {
     if (!pinnedOnly) {
         requireInit(hippoRoot);
     }
-    // The session id bounds the active-task-snapshot read: the stdin hook payload wins, then hostSessionId();
-    // absent both, api.getContext applies the pure freshness bound.
-    const payload = readHookPayload(stdinText);
-    let payloadSessionId = payload.sessionId;
-    const currentSessionId = payloadSessionId ?? hostSessionId();
-    // A sub-agent's payload and env both carry its parent's session id, so it books no session and never skips a block.
-    const subagent = isSubagentPayload(stdinText);
-    const ledgerSessionId = subagent ? undefined : currentSessionId;
-    if (subagent)
-        payloadSessionId = undefined;
+    const session = resolveContextSession(stdinText);
+    const { currentSessionId, ledgerSessionId, payloadSessionId } = session;
     // The pilot arm is booked at the first hook call whatever the flags, so the holdout sees no budget or content branch.
     const resolvedTenant = resolveTenantId({});
     if (inPilotHoldout(hippoRoot, resolvedTenant, currentSessionId, payloadSessionId !== undefined)) {
@@ -112,38 +104,16 @@ async function renderContext(hippoRoot, args, flags, stdinText, rec) {
     if (!query && flags['auto']) {
         query = autoDetectContext();
     }
-    // Scope detection uses cwd, so it is resolved here and passed in via opts.scope.
-    const ctxExplicitScope = flags['scope'] !== undefined ? String(flags['scope']).trim() : null;
-    const ctxActiveScope = ctxExplicitScope || detectScope();
     const ctx = {
         hippoRoot,
         tenantId: resolvedTenant,
         actor: api.adminActor('cli'),
     };
-    // --cross-project re-includes other-project memories, rendered under their own section.
-    const crossProject = flags['cross-project'] === true;
     const format = String(flags['format'] ?? 'markdown');
     const framing = String(flags['framing'] ?? 'observe');
-    const opts = {
-        q: query,
-        budget,
-        limit: parseLimitFlag(flags['limit']),
-        pinnedOnly,
-        scope: ctxActiveScope ?? undefined,
-        includeRecent: parseCountFlag(flags['include-recent']),
-        crossProject,
-        currentSessionId,
-        prompt: payload.prompt,
-        // JSON is budgeted as the markdown it stands for, so one budget picks the same memories in every format.
-        cost: contextCost(format === 'additional-context' ? 'additional-context' : 'markdown', framing),
-        deliveryObserver: rec ?? undefined,
-    };
+    const opts = buildContextOpts(flags, { query, budget, pinnedOnly, format, framing, session, rec });
     const result = await api.getContext(ctx, opts);
-    const hasContextData = result.entries.length > 0 ||
-        result.activeSnapshot ||
-        result.sessionHandoff ||
-        (result.recentEvents && result.recentEvents.length > 0);
-    if (!hasContextData) {
+    if (!hasContextData(result)) {
         rec?.delivered({ state: 'empty' });
         return;
     }
@@ -158,6 +128,46 @@ async function renderContext(hippoRoot, args, flags, stdinText, rec) {
         renderContextMarkdown(view);
     }
 }
+function resolveContextSession(stdinText) {
+    // The session id bounds the active-task-snapshot read: the stdin hook payload wins, then hostSessionId();
+    // absent both, api.getContext applies the pure freshness bound.
+    const payload = readHookPayload(stdinText);
+    let payloadSessionId = payload.sessionId;
+    const currentSessionId = payloadSessionId ?? hostSessionId();
+    // A sub-agent's payload and env both carry its parent's session id, so it books no session and never skips a block.
+    const subagent = isSubagentPayload(stdinText);
+    const ledgerSessionId = subagent ? undefined : currentSessionId;
+    if (subagent)
+        payloadSessionId = undefined;
+    return { currentSessionId, ledgerSessionId, payloadSessionId, prompt: payload.prompt };
+}
+function buildContextOpts(flags, input) {
+    // Scope detection uses cwd, so it is resolved here and passed in via opts.scope.
+    const ctxExplicitScope = flags['scope'] !== undefined ? String(flags['scope']).trim() : null;
+    const ctxActiveScope = ctxExplicitScope || detectScope();
+    // --cross-project re-includes other-project memories, rendered under their own section.
+    const crossProject = flags['cross-project'] === true;
+    return {
+        q: input.query,
+        budget: input.budget,
+        limit: parseLimitFlag(flags['limit']),
+        pinnedOnly: input.pinnedOnly,
+        scope: ctxActiveScope ?? undefined,
+        includeRecent: parseCountFlag(flags['include-recent']),
+        crossProject,
+        currentSessionId: input.session.currentSessionId,
+        prompt: input.session.prompt,
+        // JSON is budgeted as the markdown it stands for, so one budget picks the same memories in every format.
+        cost: contextCost(input.format === 'additional-context' ? 'additional-context' : 'markdown', input.framing),
+        deliveryObserver: input.rec ?? undefined,
+    };
+}
+function hasContextData(result) {
+    return Boolean(result.entries.length > 0 ||
+        result.activeSnapshot ||
+        result.sessionHandoff ||
+        (result.recentEvents && result.recentEvents.length > 0));
+}
 function toRenderItems(entries) {
     return entries.map((r) => ({ entry: r.entry, score: r.score, tokens: r.tokens, isGlobal: r.isGlobal ?? false }));
 }