memorix 1.2.1 → 1.2.2

package/src/cli/tui/Sidebar.tsx

@@ -1,179 +1,179 @@
-/**
- * Sidebar right panel with quick action hints and health snapshot.
- *
- * Modern design: Unicode separators, status dots, active indicator
- * with arrow symbol, brand-colored section headers.
- */
-
-import React from 'react';
-import { Box, Text, useInput } from 'ink';
-import { COLORS, SEP, STATUS_DOTS, SYMBOLS } from './theme.js';
-import type { HealthInfo, BackgroundInfo } from './data.js';
-import type { ViewType } from './theme.js';
-
-interface SidebarProps {
-  health: HealthInfo;
-  background: BackgroundInfo;
-  onAction: (cmd: string) => void;
-  activeView: ViewType;
-  /** When true, Sidebar captures shortcut keys and drives navigation. */
-  isFocused?: boolean;
-}
-
-const ACTIONS = [
-  { key: 's', label: 'Search', cmd: '/search' },
-  { key: 'r', label: 'Remember', cmd: '/remember' },
-  { key: 'v', label: 'Recent', cmd: '/recent' },
-  { key: 'd', label: 'Doctor', cmd: '/doctor' },
-  { key: 'b', label: 'Background', cmd: '/background' },
-  { key: 'w', label: 'Dashboard', cmd: '/dashboard' },
-  { key: 'p', label: 'Project', cmd: '/project' },
-  { key: 'c', label: 'Configure', cmd: '/configure' },
-  { key: 'i', label: 'Integrate', cmd: '/integrate' },
-  { key: 'h', label: 'Home', cmd: '/home' },
-];
-
-const SIDEBAR_WIDTH = 32;
-const INNER_WIDTH = SIDEBAR_WIDTH - 4;
-const ACTION_LABEL_WIDTH = INNER_WIDTH - 2;
-const VALUE_WIDTH = INNER_WIDTH - 8;
-
-function separator(width: number): string {
-  return SEP.thin.repeat(width);
-}
-
-function colorForMode(mode: string): string {
-  const normalized = mode.toLowerCase();
-  if (normalized.includes('hybrid')) return COLORS.success;
-  if (normalized.includes('vector')) return COLORS.accent;
-  return COLORS.warning;
-}
-
-function charWidth(ch: string): number {
-  const cp = ch.codePointAt(0)!;
-  return ((cp >= 0x4E00 && cp <= 0x9FFF) || (cp >= 0x3040 && cp <= 0x30FF) ||
-          (cp >= 0xAC00 && cp <= 0xD7AF) || (cp >= 0xFF01 && cp <= 0xFF60) ||
-          (cp >= 0x3400 && cp <= 0x4DBF)) ? 2 : 1;
-}
-
-function fit(text: string, max: number): string {
-  if (max <= 0) return '';
-  let width = 0;
-  for (const ch of text) {
-    width += charWidth(ch);
-    if (width > max) break;
-  }
-  if (width <= max) return text.padEnd(text.length + (max - width));
-  if (max === 1) return '…';
-  let result = '';
-  let used = 0;
-  for (const ch of text) {
-    const cw = charWidth(ch);
-    if (used + cw > max - 1) break;
-    result += ch;
-    used += cw;
-  }
-  const truncated = `${result}…`;
-  return truncated.padEnd(truncated.length + Math.max(0, max - (used + 1)));
-}
-
-// Build a key→cmd lookup from ACTIONS for O(1) dispatch
-const KEY_TO_CMD: Record<string, string> = {};
-for (const a of ACTIONS) KEY_TO_CMD[a.key] = a.cmd;
-
-export function Sidebar({ health, background, onAction, activeView, isFocused = false }: SidebarProps): React.ReactElement {
-  // ── Interactive navigation: Sidebar owns shortcut key dispatch ──
-  useInput((ch, key) => {
-    // Esc: return home from any secondary view
-    if (key.escape && activeView !== 'home') {
-      onAction('/home');
-      return;
-    }
-    const cmd = KEY_TO_CMD[ch];
-    if (cmd) {
-      onAction(cmd);
-    }
-  }, { isActive: isFocused });
-
-  // Map view types to sidebar action commands for highlight
-  const activeCmd = ACTIONS.find(a => {
-    const viewMap: Record<string, string> = {
-      '/search': 'search', '/recent': 'recent', '/doctor': 'doctor',
-      '/background': 'background', '/dashboard': 'dashboard',
-      '/project': 'project', '/configure': 'configure',
-      '/integrate': 'integrate', '/home': 'home',
-    };
-    return viewMap[a.cmd] === activeView;
-  })?.cmd;
-
-  return (
-    <Box
-      flexDirection="column"
-      width={SIDEBAR_WIDTH}
-      borderStyle="single"
-      borderColor={COLORS.border}
-      paddingX={1}
-    >
-      <Box flexDirection="column" marginBottom={1}>
-        <Text color={COLORS.brand} bold>Quick Actions</Text>
-        <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
-        {ACTIONS.map((action) => {
-          const isActive = action.cmd === activeCmd;
-          return (
-            <Box key={action.key}>
-              <Text color={isActive ? COLORS.brand : COLORS.muted}>
-                {`${isActive ? SYMBOLS.arrow : action.key} `}
-              </Text>
-              <Text color={isActive ? COLORS.accent : COLORS.text} bold={isActive}>{fit(action.label, ACTION_LABEL_WIDTH)}</Text>
-            </Box>
-          );
-        })}
-      </Box>
-
-      <Box flexDirection="column">
-        <Text color={COLORS.brand} bold>Health</Text>
-        <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
-
-        <Box>
-          <Text color={COLORS.muted}>{'Embed'.padEnd(8)}</Text>
-          <Text color={
-            health.embeddingProvider === 'ready' ? COLORS.success
-            : health.embeddingProvider === 'unavailable' ? COLORS.warning
-            : COLORS.muted
-          }>
-            {fit(`${health.embeddingProvider === 'ready' ? STATUS_DOTS.ok : health.embeddingProvider === 'unavailable' ? STATUS_DOTS.warn : STATUS_DOTS.off} ${health.embeddingLabel}`, VALUE_WIDTH)}
-          </Text>
-        </Box>
-        <Box>
-          <Text color={COLORS.muted}>{'Search'.padEnd(8)}</Text>
-          <Text color={colorForMode(health.searchModeLabel)}>{fit(`${STATUS_DOTS.ok} ${health.searchModeLabel}`, VALUE_WIDTH)}</Text>
-        </Box>
-        <Box>
-          <Text color={COLORS.muted}>{'Sess'.padEnd(8)}</Text>
-          <Text color={COLORS.text}>{fit(`${health.sessions}`, VALUE_WIDTH)}</Text>
-        </Box>
-
-        <Box marginTop={1} flexDirection="column">
-          <Text color={COLORS.brand} bold>Background</Text>
-          <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
-          <Box>
-            <Text color={COLORS.muted}>{'Status'.padEnd(8)}</Text>
-            <Text color={background.healthy ? COLORS.success : background.running ? COLORS.warning : COLORS.muted}>
-              {fit(`${background.healthy ? STATUS_DOTS.running : background.running ? STATUS_DOTS.warn : STATUS_DOTS.stopped} ${background.healthy ? 'Running' : background.running ? 'Warning' : 'Stopped'}`, VALUE_WIDTH)}
-            </Text>
-          </Box>
-          {background.port && (
-            <Box>
-              <Text color={COLORS.muted}>{'Port'.padEnd(8)}</Text>
-              <Text color={COLORS.text}>{fit(`${background.port}`, VALUE_WIDTH)}</Text>
-            </Box>
-          )}
-        </Box>
-      </Box>
-
-      <Box marginTop={1}>
-        <Text color={COLORS.muted} italic>Keys or /cmd</Text>
-      </Box>
-    </Box>
-  );
-}
+/**
+ * Sidebar right panel with quick action hints and health snapshot.
+ *
+ * Modern design: Unicode separators, status dots, active indicator
+ * with arrow symbol, brand-colored section headers.
+ */
+
+import React from 'react';
+import { Box, Text, useInput } from 'ink';
+import { COLORS, SEP, STATUS_DOTS, SYMBOLS } from './theme.js';
+import type { HealthInfo, BackgroundInfo } from './data.js';
+import type { ViewType } from './theme.js';
+
+interface SidebarProps {
+  health: HealthInfo;
+  background: BackgroundInfo;
+  onAction: (cmd: string) => void;
+  activeView: ViewType;
+  /** When true, Sidebar captures shortcut keys and drives navigation. */
+  isFocused?: boolean;
+}
+
+const ACTIONS = [
+  { key: 's', label: 'Search', cmd: '/search' },
+  { key: 'r', label: 'Remember', cmd: '/remember' },
+  { key: 'v', label: 'Recent', cmd: '/recent' },
+  { key: 'd', label: 'Doctor', cmd: '/doctor' },
+  { key: 'b', label: 'Background', cmd: '/background' },
+  { key: 'w', label: 'Dashboard', cmd: '/dashboard' },
+  { key: 'p', label: 'Project', cmd: '/project' },
+  { key: 'c', label: 'Configure', cmd: '/configure' },
+  { key: 'i', label: 'Integrate', cmd: '/integrate' },
+  { key: 'h', label: 'Home', cmd: '/home' },
+];
+
+const SIDEBAR_WIDTH = 32;
+const INNER_WIDTH = SIDEBAR_WIDTH - 4;
+const ACTION_LABEL_WIDTH = INNER_WIDTH - 2;
+const VALUE_WIDTH = INNER_WIDTH - 8;
+
+function separator(width: number): string {
+  return SEP.thin.repeat(width);
+}
+
+function colorForMode(mode: string): string {
+  const normalized = mode.toLowerCase();
+  if (normalized.includes('hybrid')) return COLORS.success;
+  if (normalized.includes('vector')) return COLORS.accent;
+  return COLORS.warning;
+}
+
+function charWidth(ch: string): number {
+  const cp = ch.codePointAt(0)!;
+  return ((cp >= 0x4E00 && cp <= 0x9FFF) || (cp >= 0x3040 && cp <= 0x30FF) ||
+          (cp >= 0xAC00 && cp <= 0xD7AF) || (cp >= 0xFF01 && cp <= 0xFF60) ||
+          (cp >= 0x3400 && cp <= 0x4DBF)) ? 2 : 1;
+}
+
+function fit(text: string, max: number): string {
+  if (max <= 0) return '';
+  let width = 0;
+  for (const ch of text) {
+    width += charWidth(ch);
+    if (width > max) break;
+  }
+  if (width <= max) return text.padEnd(text.length + (max - width));
+  if (max === 1) return '…';
+  let result = '';
+  let used = 0;
+  for (const ch of text) {
+    const cw = charWidth(ch);
+    if (used + cw > max - 1) break;
+    result += ch;
+    used += cw;
+  }
+  const truncated = `${result}…`;
+  return truncated.padEnd(truncated.length + Math.max(0, max - (used + 1)));
+}
+
+// Build a key→cmd lookup from ACTIONS for O(1) dispatch
+const KEY_TO_CMD: Record<string, string> = {};
+for (const a of ACTIONS) KEY_TO_CMD[a.key] = a.cmd;
+
+export function Sidebar({ health, background, onAction, activeView, isFocused = false }: SidebarProps): React.ReactElement {
+  // ── Interactive navigation: Sidebar owns shortcut key dispatch ──
+  useInput((ch, key) => {
+    // Esc: return home from any secondary view
+    if (key.escape && activeView !== 'home') {
+      onAction('/home');
+      return;
+    }
+    const cmd = KEY_TO_CMD[ch];
+    if (cmd) {
+      onAction(cmd);
+    }
+  }, { isActive: isFocused });
+
+  // Map view types to sidebar action commands for highlight
+  const activeCmd = ACTIONS.find(a => {
+    const viewMap: Record<string, string> = {
+      '/search': 'search', '/recent': 'recent', '/doctor': 'doctor',
+      '/background': 'background', '/dashboard': 'dashboard',
+      '/project': 'project', '/configure': 'configure',
+      '/integrate': 'integrate', '/home': 'home',
+    };
+    return viewMap[a.cmd] === activeView;
+  })?.cmd;
+
+  return (
+    <Box
+      flexDirection="column"
+      width={SIDEBAR_WIDTH}
+      borderStyle="single"
+      borderColor={COLORS.border}
+      paddingX={1}
+    >
+      <Box flexDirection="column" marginBottom={1}>
+        <Text color={COLORS.brand} bold>Quick Actions</Text>
+        <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
+        {ACTIONS.map((action) => {
+          const isActive = action.cmd === activeCmd;
+          return (
+            <Box key={action.key}>
+              <Text color={isActive ? COLORS.brand : COLORS.muted}>
+                {`${isActive ? SYMBOLS.arrow : action.key} `}
+              </Text>
+              <Text color={isActive ? COLORS.accent : COLORS.text} bold={isActive}>{fit(action.label, ACTION_LABEL_WIDTH)}</Text>
+            </Box>
+          );
+        })}
+      </Box>
+
+      <Box flexDirection="column">
+        <Text color={COLORS.brand} bold>Health</Text>
+        <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
+
+        <Box>
+          <Text color={COLORS.muted}>{'Embed'.padEnd(8)}</Text>
+          <Text color={
+            health.embeddingProvider === 'ready' ? COLORS.success
+            : health.embeddingProvider === 'unavailable' ? COLORS.warning
+            : COLORS.muted
+          }>
+            {fit(`${health.embeddingProvider === 'ready' ? STATUS_DOTS.ok : health.embeddingProvider === 'unavailable' ? STATUS_DOTS.warn : STATUS_DOTS.off} ${health.embeddingLabel}`, VALUE_WIDTH)}
+          </Text>
+        </Box>
+        <Box>
+          <Text color={COLORS.muted}>{'Search'.padEnd(8)}</Text>
+          <Text color={colorForMode(health.searchModeLabel)}>{fit(`${STATUS_DOTS.ok} ${health.searchModeLabel}`, VALUE_WIDTH)}</Text>
+        </Box>
+        <Box>
+          <Text color={COLORS.muted}>{'Sess'.padEnd(8)}</Text>
+          <Text color={COLORS.text}>{fit(`${health.sessions}`, VALUE_WIDTH)}</Text>
+        </Box>
+
+        <Box marginTop={1} flexDirection="column">
+          <Text color={COLORS.brand} bold>Background</Text>
+          <Text color={COLORS.border}>{separator(INNER_WIDTH)}</Text>
+          <Box>
+            <Text color={COLORS.muted}>{'Status'.padEnd(8)}</Text>
+            <Text color={background.healthy ? COLORS.success : background.running ? COLORS.warning : COLORS.muted}>
+              {fit(`${background.healthy ? STATUS_DOTS.running : background.running ? STATUS_DOTS.warn : STATUS_DOTS.stopped} ${background.healthy ? 'Running' : background.running ? 'Warning' : 'Stopped'}`, VALUE_WIDTH)}
+            </Text>
+          </Box>
+          {background.port && (
+            <Box>
+              <Text color={COLORS.muted}>{'Port'.padEnd(8)}</Text>
+              <Text color={COLORS.text}>{fit(`${background.port}`, VALUE_WIDTH)}</Text>
+            </Box>
+          )}
+        </Box>
+      </Box>
+
+      <Box marginTop={1}>
+        <Text color={COLORS.muted} italic>Keys or /cmd</Text>
+      </Box>
+    </Box>
+  );
+}

package/src/cli/tui/chat-service.ts

@@ -3,11 +3,11 @@ import { loadDotenv } from '../../config/dotenv-loader.js';
 import { initLLM, isLLMEnabled, getLLMConfig, callLLMWithTools } from '../../llm/provider.js';
 import type { ChatMessage, ToolDefinition, ToolCall } from '../../llm/provider.js';
 import { initObservations, prepareSearchIndex, storeObservation, resolveObservations, getObservation, getAllObservations } from '../../memory/observations.js';
-import type { ObservationType } from '../../types.js';
-import { detectProject } from '../../project/detector.js';
+import { canManageObservation, filterReadableObservations } from '../../memory/visibility.js';
+import type { ObservationReader, ObservationType } from '../../types.js';
 import { getLastSearchMode } from '../../store/orama-store.js';
-import { getProjectDataDir } from '../../store/persistence.js';
 import type { MemorixDocument } from '../../types.js';
+import { getTuiOperatorContext } from './operator-context.js';
 
 export interface ChatHistoryTurn {
   role: 'user' | 'assistant';
@@ -208,19 +208,29 @@ async function prepareProjectSearch(projectId: string, dataDir: string): Promise
 
 interface ToolExecutionContext {
   projectId: string;
+  reader: ObservationReader;
+  writerAgentId?: string;
   collectedSources: ChatSource[];
 }
 
+async function resolveTuiContext() {
+  try {
+    return await getTuiOperatorContext();
+  } catch {
+    return null;
+  }
+}
+
 function executeSearchMemories(args: { query: string; limit?: number }, ctx: ToolExecutionContext): Promise<string> {
   const limit = Math.min(args.limit ?? SEARCH_LIMIT, 10);
-  return compactSearch({ query: args.query, limit, projectId: ctx.projectId, status: 'active' })
+  return compactSearch({ query: args.query, limit, projectId: ctx.projectId, status: 'active', reader: ctx.reader })
     .then((result) => {
       const entries = result.entries.slice(0, DETAIL_LIMIT);
       if (entries.length === 0) return 'No memories found for that query.';
 
       // Collect sources for citation tracking
       const refs = entries.map((e) => ({ id: e.id, projectId: ctx.projectId }));
-      return compactDetail(refs).then((detail) => {
+      return compactDetail(refs, { reader: ctx.reader }).then((detail) => {
         for (let i = 0; i < detail.documents.length; i++) {
           const doc = detail.documents[i];
           const score = entries[i]?.score ?? 0;
@@ -235,7 +245,7 @@ function executeSearchMemories(args: { query: string; limit?: number }, ctx: Too
 }
 
 function executeGetMemoryDetail(args: { id: number }, ctx: ToolExecutionContext): Promise<string> {
-  return compactDetail([{ id: args.id, projectId: ctx.projectId }])
+  return compactDetail([{ id: args.id, projectId: ctx.projectId }], { reader: ctx.reader })
     .then((detail) => {
       if (detail.documents.length === 0) return `Observation ${args.id} not found.`;
       const doc = detail.documents[0];
@@ -295,6 +305,8 @@ function executeStoreMemory(
     facts: args.facts,
     projectId: ctx.projectId,
     source: 'agent',
+    visibilityReader: ctx.reader,
+    ...(ctx.writerAgentId ? { createdByAgentId: ctx.writerAgentId } : {}),
   }).then((result) => {
     const obs = result.observation;
     ctx.collectedSources.push({
@@ -314,7 +326,7 @@ function executeUpdateMemory(
   ctx: ToolExecutionContext,
 ): string {
   const obs = getObservation(args.id, ctx.projectId);
-  if (!obs) return `Observation ${args.id} not found.`;
+  if (!obs || !canManageObservation(obs, ctx.reader)) return `Observation ${args.id} not found.`;
 
   const shouldAppend = args.append !== false; // default true
   const newNarrative = shouldAppend
@@ -334,6 +346,7 @@ function executeUpdateMemory(
     projectId: ctx.projectId,
     topicKey: obs.topicKey,
     source: 'agent',
+    visibilityReader: ctx.reader,
   }).catch(() => {/* non-blocking */});
 
   return `Updated observation [obs:${args.id}] — ${shouldAppend ? 'appended to' : 'replaced'} narrative${args.facts?.length ? ` and added ${args.facts.length} facts` : ''}. Changes will be persisted asynchronously.`;
@@ -344,7 +357,7 @@ function executeDeleteMemory(
   ctx: ToolExecutionContext,
 ): Promise<string> {
   const obs = getObservation(args.id, ctx.projectId);
-  if (!obs) return Promise.resolve(`Observation ${args.id} not found.`);
+  if (!obs || !canManageObservation(obs, ctx.reader)) return Promise.resolve(`Observation ${args.id} not found.`);
 
   return resolveObservations([args.id], 'resolved')
     .then((result) => {
@@ -361,8 +374,10 @@ function executeListRecentMemories(
   ctx: ToolExecutionContext,
 ): string {
   const limit = Math.min(args.limit ?? 10, 20);
-  let allObs = getAllObservations()
-    .filter(o => o.projectId === ctx.projectId && o.status !== 'archived' && o.status !== 'resolved');
+  let allObs = filterReadableObservations(
+    getAllObservations().filter(o => o.projectId === ctx.projectId && o.status !== 'archived' && o.status !== 'resolved'),
+    ctx.reader,
+  );
 
   if (args.type) {
     allObs = allObs.filter(o => o.type === args.type);
@@ -399,10 +414,10 @@ export async function askMemoryQuestion(
     };
   }
 
-  const project = detectProject(process.cwd());
+  const operatorContext = await resolveTuiContext();
 
   // No project + no LLM → dead end
-  if (!project) {
+  if (!operatorContext) {
     loadDotenv(process.cwd());
     initLLM({ scope: 'agent' });
     if (!isLLMEnabled()) {
@@ -438,10 +453,11 @@ export async function askMemoryQuestion(
     };
   }
 
+  const { project, dataDir, reader, identity } = operatorContext;
+
   loadDotenv(project.rootPath);
   initLLM({ scope: 'agent' });
 
-  const dataDir = await getProjectDataDir(project.id);
   await prepareProjectSearch(project.id, dataDir);
 
   const searchMode = normalizeSearchMode(getLastSearchMode(project.id) || 'fulltext');
@@ -453,11 +469,12 @@ export async function askMemoryQuestion(
       limit: SEARCH_LIMIT,
       projectId: project.id,
       status: 'active',
+      reader,
     });
     const topEntries = searchResult.entries.slice(0, DETAIL_LIMIT);
     const detailRefs = topEntries.map((entry) => ({ id: entry.id, projectId: project.id }));
     const detailResult = detailRefs.length > 0
-      ? await compactDetail(detailRefs)
+      ? await compactDetail(detailRefs, { reader })
       : { documents: [], formatted: '', totalTokens: 0 };
     const sources = detailResult.documents.map((doc, index) => toSource(doc, topEntries[index]?.score ?? 0));
 
@@ -488,6 +505,8 @@ export async function askMemoryQuestion(
 
   const ctx: ToolExecutionContext = {
     projectId: project.id,
+    reader,
+    ...(identity ? { writerAgentId: identity.agentId } : {}),
     collectedSources: [],
   };
 
@@ -584,10 +603,10 @@ export async function askMemoryQuestionStream(
     };
   }
 
-  const project = detectProject(process.cwd());
+  const operatorContext = await resolveTuiContext();
 
   // No project + no LLM → dead end
-  if (!project) {
+  if (!operatorContext) {
     loadDotenv(process.cwd());
     initLLM({ scope: 'agent' });
     if (!isLLMEnabled()) {
@@ -626,10 +645,11 @@ export async function askMemoryQuestionStream(
     };
   }
 
+  const { project, dataDir, reader, identity } = operatorContext;
+
   loadDotenv(project.rootPath);
   initLLM({ scope: 'agent' });
 
-  const dataDir = await getProjectDataDir(project.id);
   await prepareProjectSearch(project.id, dataDir);
 
   const searchMode = normalizeSearchMode(getLastSearchMode(project.id) || 'fulltext');
@@ -641,11 +661,12 @@ export async function askMemoryQuestionStream(
       limit: SEARCH_LIMIT,
       projectId: project.id,
       status: 'active',
+      reader,
     });
     const topEntries = searchResult.entries.slice(0, DETAIL_LIMIT);
     const detailRefs = topEntries.map((entry) => ({ id: entry.id, projectId: project.id }));
     const detailResult = detailRefs.length > 0
-      ? await compactDetail(detailRefs)
+      ? await compactDetail(detailRefs, { reader })
       : { documents: [], formatted: '', totalTokens: 0 };
     const sources = detailResult.documents.map((doc, index) => toSource(doc, topEntries[index]?.score ?? 0));
 
@@ -672,6 +693,8 @@ export async function askMemoryQuestionStream(
 
   const ctx: ToolExecutionContext = {
     projectId: project.id,
+    reader,
+    ...(identity ? { writerAgentId: identity.agentId } : {}),
     collectedSources: [],
   };