interference-agent 0.2.2 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "interference-agent",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "The open-source coding agent that lives in your terminal.",
   "type": "module",
   "license": "MIT",

package/src/auth.ts

@@ -1,21 +1,19 @@
 import { readFile, writeFile, mkdir, chmod } from "node:fs/promises";
 import * as path from "node:path";
-
-const AUTH_DIR = path.join(
-  process.env.HOME ?? process.env.USERPROFILE ?? "/tmp",
-  ".interference",
-);
+import { interferenceDir } from "./paths.ts";
 
 interface ProviderAuth {
   label: string;
   envKey: string;
 }
 
-const AUTH_FILE = path.join(AUTH_DIR, "auth.json");
+// Risolti a runtime (non all'import) così INTERFERENCE_HOME isola i test.
+const authDir = (): string => interferenceDir();
+const authFile = (): string => path.join(authDir(), "auth.json");
 
 export async function loadAuth(): Promise<Record<string, string>> {
   try {
-    const raw = await readFile(AUTH_FILE, "utf-8");
+    const raw = await readFile(authFile(), "utf-8");
     return JSON.parse(raw) as Record<string, string>;
   } catch {
     return {};
@@ -23,9 +21,9 @@ export async function loadAuth(): Promise<Record<string, string>> {
 }
 
 export async function saveAuth(auth: Record<string, string>): Promise<void> {
-  await mkdir(AUTH_DIR, { recursive: true });
-  await writeFile(AUTH_FILE, JSON.stringify(auth, null, 2));
-  try { await chmod(AUTH_FILE, 0o600); } catch {}
+  await mkdir(authDir(), { recursive: true });
+  await writeFile(authFile(), JSON.stringify(auth, null, 2));
+  try { await chmod(authFile(), 0o600); } catch {}
 }
 
 export function applyAuthToEnv(auth: Record<string, string>, providers: Record<string, ProviderAuth>): void {

package/src/paths.ts

@@ -0,0 +1,25 @@
+import * as path from "node:path";
+
+/**
+ * Home base dei dati di interference (`~/.interference`).
+ *
+ * `INTERFERENCE_HOME` ridireziona l'intero store dell'app: usato dai test per
+ * isolarsi dalla directory reale dell'utente, così nessun test possa scrivere
+ * o cancellare dati veri (sessioni, snapshot, credenziali, skill, cache update).
+ *
+ * Tutti i consumatori di `~/.interference` DEVONO passare da qui — non
+ * ricalcolare la home a mano (vedi store/skills/auth/version).
+ */
+export function interferenceHome(): string {
+  return (
+    process.env.INTERFERENCE_HOME ??
+    process.env.HOME ??
+    process.env.USERPROFILE ??
+    "/tmp"
+  );
+}
+
+/** Path dentro `~/.interference` (o la home reindirizzata da INTERFERENCE_HOME). */
+export function interferenceDir(...segments: string[]): string {
+  return path.join(interferenceHome(), ".interference", ...segments);
+}

package/src/session/__tests__/session.test.ts

@@ -26,7 +26,14 @@ import type { ModelMessage } from "ai";
 
 const TMP = path.join(process.cwd(), ".test-tmp-session");
 
+// ISOLAMENTO STORE: reindirizza ~/.interference verso TMP così i test su
+// saveSession/cleanupSessions/deleteSession NON tocchino mai le sessioni reali
+// dell'utente. Senza questo override, `cleanupSessions(2)` cancellerebbe le
+// chat vere salvate in ~/.interference/<hash>/sessions.
+const PREV_HOME = process.env.INTERFERENCE_HOME;
+
 beforeAll(async () => {
+  process.env.INTERFERENCE_HOME = TMP;
   await rm(TMP, { recursive: true, force: true });
   await mkdir(TMP, { recursive: true });
   await writeFile(path.join(TMP, "a.txt"), "original a\n");
@@ -35,6 +42,8 @@ beforeAll(async () => {
 
 afterAll(async () => {
   await rm(TMP, { recursive: true, force: true });
+  if (PREV_HOME === undefined) delete process.env.INTERFERENCE_HOME;
+  else process.env.INTERFERENCE_HOME = PREV_HOME;
 });
 
 describe("session store", () => {

package/src/session/store.ts

@@ -3,9 +3,11 @@ import * as path from "node:path";
 import { createHash } from "node:crypto";
 import type { ModelMessage } from "ai";
 import type { Todo } from "../tools/todowrite.ts";
+import { interferenceDir } from "../paths.ts";
 
 export interface SessionMeta {
   id: string;
+  title?: string; // nome leggibile (auto dal primo messaggio o via /rename)
   workspace: string;
   startedAt: string;
   updatedAt: string;
@@ -26,11 +28,8 @@ function projectDir(): string {
     .update(process.cwd())
     .digest("hex")
     .slice(0, 12);
-  return path.join(
-    process.env.HOME ?? process.env.USERPROFILE ?? "/tmp",
-    ".interference",
-    hash,
-  );
+  // Home reindirizzabile via INTERFERENCE_HOME (isolamento test) — vedi paths.ts.
+  return interferenceDir(hash);
 }
 
 function sessionsDir(): string {

package/src/skills.ts

@@ -1,11 +1,11 @@
 import { readFile, readdir, mkdir, writeFile } from "node:fs/promises";
 import * as path from "node:path";
+import { interferenceDir } from "./paths.ts";
 
-export const SKILLS_DIR = path.join(
-  process.env.HOME ?? process.env.USERPROFILE ?? "/tmp",
-  ".interference",
-  "skills",
-);
+/** Directory delle skill (`~/.interference/skills`, reindirizzabile in test). */
+export function skillsDir(): string {
+  return interferenceDir("skills");
+}
 
 const BUNDLED_SKILLS: Record<string, string> = {
   "agents-setup": `---
@@ -90,10 +90,10 @@ export async function loadSkillRegistry(): Promise<SkillInfo[]> {
   if (registryCache) return registryCache;
   const list: SkillInfo[] = [];
   try {
-    const entries = await readdir(SKILLS_DIR, { withFileTypes: true });
+    const entries = await readdir(skillsDir(), { withFileTypes: true });
     for (const entry of entries) {
       if (!entry.isDirectory()) continue;
-      const skillFile = path.join(SKILLS_DIR, entry.name, "SKILL.md");
+      const skillFile = path.join(skillsDir(), entry.name, "SKILL.md");
       try {
         const content = await readFile(skillFile, "utf-8");
         const info = parseSkillFrontmatter(content);
@@ -119,7 +119,7 @@ function parseSkillFrontmatter(content: string): SkillInfo | null {
 }
 
 export async function loadSkillBody(name: string): Promise<string | null> {
-  const skillFile = path.join(SKILLS_DIR, name, "SKILL.md");
+  const skillFile = path.join(skillsDir(), name, "SKILL.md");
   try {
     return await readFile(skillFile, "utf-8");
   } catch {
@@ -165,7 +165,7 @@ const STOPWORDS = new Set([
 
 export async function bootstrapSkills(): Promise<void> {
   for (const [name, content] of Object.entries(BUNDLED_SKILLS)) {
-    const dir = path.join(SKILLS_DIR, name);
+    const dir = path.join(skillsDir(), name);
     try { await mkdir(dir, { recursive: true }); } catch {}
     const fp = path.join(dir, "SKILL.md");
     try {

package/src/tui/App.tsx

@@ -192,6 +192,12 @@ export default function App({ session }: { session: Session }) {
 
   const nextId = (): number => Date.now() + Math.random();
 
+  // Titolo leggibile dal primo messaggio (collassa spazi, cap ~40 char).
+  const deriveTitle = (text: string): string => {
+    const t = text.replace(/\s+/g, " ").trim();
+    return t.length > 40 ? t.slice(0, 40).trimEnd() + "…" : t;
+  };
+
   async function doCompact() {
     const pct = getUsagePercent(messagesRef.current);
     if (!shouldCompact(messagesRef.current)) {
@@ -226,6 +232,10 @@ export default function App({ session }: { session: Session }) {
     setHistory((h) => [...h, userMsg]);
 
     nextTurn();
+    // Auto-title alla prima interazione (se non già rinominata dall'utente).
+    if (!sessionRef.current.meta.title) {
+      sessionRef.current.meta.title = deriveTitle(userText);
+    }
     messagesRef.current.push({ role: "user", content: userText });
     aborterRef.current = new AbortController();
     let acc = "";
@@ -455,7 +465,7 @@ ${args ? `Additional context: ${args}` : ""}`;
             return "";
           },
           doRename: async (name) => {
-            sessionRef.current.meta.id = name;
+            sessionRef.current.meta.title = name;
             await saveSession(sessionRef.current);
             return `Session renamed to '${name}'.`;
           },
@@ -498,20 +508,36 @@ ${args ? `Additional context: ${args}` : ""}`;
               messagesRef.current = loaded.messages;
               sessionRef.current = loaded;
               setTodos(loaded.todos ?? []);
-              // Rebuild history from messages
+              // Ricostruisci la history dai messaggi salvati. Il content può essere
+              // una stringa o un ARRAY di parti ({type:"text"|"reasoning"|...}):
+              // estraiamo testo e reasoning invece di stringificare (era reso come JSON grezzo).
               const items: HistoryItem[] = [];
               let nid = Date.now();
               for (const m of loaded.messages) {
-                if (m.role === "user" || m.role === "assistant") {
-                  const content = typeof m.content === "string" ? m.content : JSON.stringify(m.content);
-                  items.push({ id: nid++, role: m.role as "user" | "assistant", content });
+                if (m.role !== "user" && m.role !== "assistant") continue;
+                let text = "";
+                let reasoning = "";
+                if (typeof m.content === "string") {
+                  text = m.content;
+                } else if (Array.isArray(m.content)) {
+                  for (const p of m.content as Array<{ type?: string; text?: string }>) {
+                    if (p?.type === "text" && p.text) text += p.text;
+                    else if (p?.type === "reasoning" && p.text) reasoning += p.text;
+                  }
                 }
+                if (!text && !reasoning) continue; // salta i messaggi solo-tool
+                items.push({
+                  id: nid++,
+                  role: m.role as "user" | "assistant",
+                  content: text,
+                  reasoning: reasoning || undefined,
+                });
               }
               setHistory(items);
               setStreaming("");
               setReasoning("");
               setToolSteps([]);
-              addToast(`Resumed session ${id.slice(0, 12)} (${loaded.meta.turnCount} turns)`, "success");
+              addToast(`Resumed '${loaded.meta.title || id.slice(0, 12)}' (${loaded.meta.turnCount} turns)`, "success");
             } else {
               addToast(`Session ${id.slice(0, 12)} not found`, "error");
             }
@@ -604,6 +630,13 @@ ${args ? `Additional context: ${args}` : ""}`;
             <SlashAutocomplete filter={draft.slice(1)} selected={acIdx} />
           )}
 
+          {/* Avviso comandi/stato: subito SOPRA l'input (stile opencode), non nel footer */}
+          {statusText && !confirmPreview && !questions && !showSessions && (
+            <Box paddingLeft={1}>
+              <Text dimColor>{statusText}</Text>
+            </Box>
+          )}
+
           {!confirmPreview && !questions && !showSessions && (
             <Box borderStyle="round" borderColor="gray" paddingX={1}>
               <Text color="white" bold>{"› "}</Text>
@@ -626,7 +659,7 @@ ${args ? `Additional context: ${args}` : ""}`;
             thinking={currentThinking()}
             contextPct={messagesRef.current.length > 0 ? getUsagePercent(messagesRef.current) : 0}
             busy={busy}
-            statusLine={statusText}
+            statusLine=""
             turnCount={sessionRef.current.meta.turnCount}
             cost={formatCost(getTotalCost())}
             gitBranch={gitBranch}

package/src/tui/SessionList.tsx

@@ -59,8 +59,7 @@ export const SessionList: FC<Props> = ({ onSelect, onCancel }) => {
       {sessions.slice(0, PAGE_SIZE).map((s, i) => (
         <SelectRow
           key={s.id}
-          label={s.id.slice(0, 12)}
-          meta={`${s.mode} · ${s.turnCount}t · ${s.updatedAt.slice(0, 10)}`}
+          label={s.title || `(untitled ${s.id.slice(0, 8)})`}
           selected={i === idx}
         />
       ))}

package/src/version.ts

@@ -1,6 +1,7 @@
 import { readFileSync } from "node:fs";
 import { readFile, writeFile, mkdir } from "node:fs/promises";
 import * as path from "node:path";
+import { interferenceDir } from "./paths.ts";
 
 // Versione corrente letta dal package.json del pacchetto (sync, funziona anche
 // installato globalmente: version.ts sta in <pkg>/src, package.json in <pkg>/).
@@ -20,8 +21,8 @@ const TTL = 24 * 60 * 60 * 1000; // 24h
 const REGISTRY = `https://registry.npmjs.org/${PKG}/latest`;
 
 function cachePath(): string {
-  const home = process.env.HOME ?? process.env.USERPROFILE ?? "/tmp";
-  return path.join(home, ".interference", "update-check.json");
+  // Reindirizzabile via INTERFERENCE_HOME (isolamento test) — vedi paths.ts.
+  return interferenceDir("update-check.json");
 }
 
 // Confronto semver "x.y.z" (con eventuale prefisso v). true se `latest` > `current`.