march-cli 0.1.2 → 0.1.4

package/src/cli/tui/output-buffer.mjs

@@ -1,59 +1,13 @@
-import { visibleWidth } from "@earendil-works/pi-tui";
-import { R, brightBlack, dim } from "./ui-theme.mjs";
+import { brightBlack, dim } from "./ui-theme.mjs";
 import { renderToolCardBlock } from "./output/tool-card-renderer.mjs";
 import { renderMarkdown, renderStreamingMarkdown } from "./markdown-renderer.mjs";
 import { renderEditDiffBlock } from "./tui-diff-rendering.mjs";
 import { OutputScrollState } from "./output/scroll-state.mjs";
+import { appendTextLines, wrapLine } from "./output/text-line-renderer.mjs";
 
 const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
 
-function wrapLine(text, maxWidth) {
-  if (maxWidth <= 0) return [""];
-  const result = [];
-  let cur = "";
-  let curW = 0;
-  let activeSgr = "";
-  for (let i = 0; i < text.length;) {
-    if (text[i] === "\x1b") {
-      const match = text.slice(i).match(/^\x1b\[[0-?]*[ -/]*[@-~]/);
-      if (match) {
-        const seq = match[0];
-        cur += seq;
-        activeSgr = updateActiveSgr(activeSgr, seq);
-        i += seq.length;
-        continue;
-      }
-    }
-    const ch = text[i];
-    const w = visibleWidth(ch);
-    if (curW + w > maxWidth) {
-      result.push(activeSgr ? `${cur}${R}` : cur);
-      cur = activeSgr + ch;
-      curW = w;
-    } else {
-      cur += ch;
-      curW += w;
-    }
-    i += 1;
-  }
-  if (cur) result.push(cur);
-  return result.length > 0 ? result : [""];
-}
-
-function updateActiveSgr(activeSgr, seq) {
-  if (!seq.endsWith("m")) return activeSgr;
-  const body = seq.slice(2, -1);
-  if (body === "" || body.split(";").includes("0")) return "";
-  return seq;
-}
 
-function appendTextLines(lines, textLines, width) {
-  for (const line of textLines) {
-    for (const part of String(line ?? "").split(/\r?\n/)) {
-      for (const wrapped of wrapLine(part, width)) lines.push(wrapped);
-    }
-  }
-}
 
 function currentTextToBlocks(textLines, sealed, cache = null) {
   const blocks = [];
@@ -117,6 +71,7 @@ export class OutputBuffer {
     this._activeThinking = null;
     this.overlayStatus = null;
     this.scrollState = new OutputScrollState();
+    this._segmentLinesCache = new Map();
   }
 
   get scrollOffset() {
@@ -133,6 +88,7 @@ export class OutputBuffer {
     this._activeThinking = null;
     this.overlayStatus = null;
     this.scrollState.clear();
+    this._segmentLinesCache = new Map();
   }
 
   write(text) {
@@ -174,6 +130,7 @@ export class OutputBuffer {
     this._flushText();
     const seg = { type: "thinking", tokens: 0, content: [] };
     this.segments.push(seg);
+    this._invalidateSegmentLines();
     this._activeThinking = seg;
   }
 
@@ -197,12 +154,14 @@ export class OutputBuffer {
     this.overlayStatus = null;
     this._flushText();
     this.segments.push({ type: "thinking", tokens, content: content.split("\n") });
+    this._invalidateSegmentLines();
   }
 
   addBlock(block) {
     this.overlayStatus = null;
     this._flushText();
     this.segments.push(block);
+    this._invalidateSegmentLines();
   }
 
   setOverlayStatus(lines) {
@@ -220,6 +179,7 @@ export class OutputBuffer {
   _flushText() {
     if (this.currentText.length <= 1 && this.currentText[0].text === "") return false;
     this.segments.push(...currentTextToBlocks(this.currentText, true));
+    this._invalidateSegmentLines();
     this.currentText = [{ text: "", markdown: false }];
     this.currentTextCache = new Map();
     return true;
@@ -234,8 +194,8 @@ export class OutputBuffer {
     this.spinnerIdx = (this.spinnerIdx + 1) % SPINNER_FRAMES.length;
   }
 
-  scroll(delta) {
-    return this.scrollState.scroll(delta);
+  scroll(delta, options) {
+    return this.scrollState.scroll(delta, options);
   }
 
   getScrollStep() {
@@ -262,10 +222,17 @@ export class OutputBuffer {
       seg.expanded = expanded;
       changed = true;
     }
+    if (changed) this._invalidateSegmentLines();
     return changed;
   }
 
-  invalidate() {}
+  invalidate() {
+    this._invalidateSegmentLines();
+  }
+
+  _invalidateSegmentLines() {
+    this._segmentLinesCache.clear();
+  }
 
   render(width) {
     const allLines = this._computeLines(width);
@@ -278,8 +245,9 @@ export class OutputBuffer {
   }
 
   _computeLines(width) {
-    const lines = [];
-    for (const seg of this.segments) {
+    const lines = [...this._renderCachedSegmentLines(width)];
+    const dynamicStart = this._cachedSegmentPrefixCount();
+    for (const seg of this.segments.slice(dynamicStart)) {
       for (const line of renderBlock(seg, width)) lines.push(line);
     }
     for (const block of currentTextToBlocks(this.currentText, false, this.currentTextCache)) {
@@ -294,4 +262,23 @@ export class OutputBuffer {
     }
     return lines;
   }
+
+  _renderCachedSegmentLines(width) {
+    const prefixCount = this._cachedSegmentPrefixCount();
+    const cached = this._segmentLinesCache.get(width);
+    if (cached?.prefixCount === prefixCount) return cached.lines;
+
+    const lines = [];
+    for (let i = 0; i < prefixCount; i += 1) {
+      for (const line of renderBlock(this.segments[i], width)) lines.push(line);
+    }
+    this._segmentLinesCache.set(width, { prefixCount, lines });
+    return lines;
+  }
+
+  _cachedSegmentPrefixCount() {
+    if (!this._activeThinking) return this.segments.length;
+    const index = this.segments.indexOf(this._activeThinking);
+    return index < 0 ? this.segments.length : index;
+  }
 }

package/src/cli/tui/render/render-scheduler.mjs

@@ -0,0 +1,26 @@
+export function createRenderScheduler({ requestRender, delayMs = 50 }) {
+  let timer = null;
+
+  function renderNow() {
+    clearPending();
+    requestRender();
+  }
+
+  function renderSoon() {
+    if (timer) return;
+    // Streaming deltas are append-only, so coalesce them without delaying input-driven renders.
+    timer = setTimeout(() => {
+      timer = null;
+      requestRender();
+    }, delayMs);
+    timer.unref?.();
+  }
+
+  function clearPending() {
+    if (!timer) return;
+    clearTimeout(timer);
+    timer = null;
+  }
+
+  return { renderNow, renderSoon, clearPending };
+}

package/src/cli/tui/tool-rendering.mjs

@@ -90,6 +90,9 @@ export function formatToolSuccessSummary(name, result, out = "") {
     const matches = result?.details?.count ?? countNonEmptyLines(out);
     return `${matches} file${matches === 1 ? "" : "s"}`;
   }
+  if (name === "memory_open") {
+    return compactText(result?.details?.entry?.name ?? compactPath(result?.details?.path ?? ""));
+  }
   return "";
 }
 

package/src/cli/ui.mjs

@@ -23,6 +23,7 @@ import { createTuiInputController } from "./tui/tui-input-controller.mjs";
 import { writeMemoryHint } from "./tui/recall-rendering.mjs";
 import { writeToolEnd, writeToolStart } from "./tui/tool-rendering.mjs";
 import { EDITOR_THEME, brightBlack } from "./tui/ui-theme.mjs";
+import { createRenderScheduler } from "./tui/render/render-scheduler.mjs";
 import { writeTranscriptToOutput } from "../session/transcript.mjs";
 
 export { buildMarchCommands, MarchAutocompleteProvider } from "./input/autocomplete.mjs";
@@ -59,10 +60,8 @@ export function createTuiUI({
   let mouseOn = true;
   let toolsExpanded = false;
   const activeToolBlocks = [];
-
-  function requestRender() {
-    tui.requestRender();
-  }
+  const renderScheduler = createRenderScheduler({ requestRender: () => tui.requestRender() });
+  const requestRender = renderScheduler.renderNow;
 
   const spinnerStatus = createSpinnerStatusController({ output, requestRender });
   const retryStatus = createRetryStatusController({ output, requestRender, stopSpinner: spinnerStatus.stop });
@@ -172,7 +171,7 @@ export function createTuiUI({
 
     thinkingDelta: (delta) => {
       output.appendThinking(delta);
-      requestRender();
+      renderScheduler.renderSoon();
     },
 
     thinkingEnd: (tokens) => {
@@ -197,7 +196,7 @@ export function createTuiUI({
     textDelta: (delta) => {
       ensureStarted(); retryStatus.stop(); spinnerStatus.stop();
       output.writeMarkdown(delta);
-      requestRender();
+      renderScheduler.renderSoon();
     },
     assistantReplyEnd: () => {
       ensureStarted();
@@ -278,8 +277,8 @@ export function createTuiUI({
     toggleToolOutput,
     toggleShellDrawer: () => shellDrawerControls.toggle(),
     requestExit: () => inputController.requestExit(),
-
     close: async () => {
+      renderScheduler.clearPending();
       spinnerStatus.stop();
       retryStatus.stop();
       if (started) {

package/src/config/loader.mjs

@@ -49,7 +49,7 @@ function mergeLayers(layers) {
     maxTurns: null,
     trimBatch: null,
     memoryRoot: null,
-    notifications: { turnEnd: false },
+    notifications: { turnEnd: true, desktop: true, bell: false, command: null, minDurationMs: 0, sound: true },
   };
 
   for (const layer of layers) {

package/src/context/center-memory.mjs

@@ -0,0 +1,14 @@
+import { existsSync, readFileSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
+
+export function defaultCenterMemoryPath() {
+  return join(homedir(), ".march", "memory", "center.md");
+}
+
+export function buildCenterMemory(path = defaultCenterMemoryPath()) {
+  if (!path || !existsSync(path)) return null;
+  const content = readFileSync(path, "utf8").trimEnd();
+  if (!content.trim()) return null;
+  return `[center_memory]\n--- ${path} ---\n${content}`;
+}

package/src/context/engine.mjs

@@ -3,16 +3,20 @@ import { buildSessionIdentity } from "./session-status.mjs";
 import { buildSystemCore, resolveSystemCorePromptKey } from "./system-core.mjs";
 import { buildInjectionsLayer } from "./injections.mjs";
 import { buildProjectContext } from "./project-context.mjs";
+import { buildCenterMemory } from "./center-memory.mjs";
 import { formatRecallHints } from "../memory/markdown-store.mjs";
 
 export class ContextEngine {
-  constructor({ cwd, modelId, provider = "deepseek", thinkingLevel = "medium", namespace = "", memoryRoot = null, shellRuntime = null, lspService = null, injections = [], maxTurns, trimBatch }) {
+  constructor({ cwd, modelId, provider = "deepseek", thinkingLevel = "medium", namespace = "", memoryRoot = null, centerMemoryPath = null, shellRuntime = null, lspService = null, injections = [], maxTurns, trimBatch }) {
     this.cwd = cwd;
     this.memoryRoot = memoryRoot;
+    this.centerMemoryPath = centerMemoryPath;
     this.modelId = modelId;
     this.provider = provider;
     this.thinkingLevel = thinkingLevel;
     this.turns = [];
+    this.pendingAssistantRecallHints = [];
+    this.pendingAssistantRecallHintsRendered = false;
     this.sessionName = "";
     this.toolDefs = [];
     this.namespace = namespace;
@@ -58,6 +62,9 @@ export class ContextEngine {
     const projectCtx = buildProjectContext(this.cwd);
     if (projectCtx) layers.push({ name: "project_context", text: projectCtx });
 
+    const centerMemory = buildCenterMemory(this.centerMemoryPath);
+    if (centerMemory) layers.push({ name: "center_memory", text: centerMemory });
+
     layers.push({ name: "recent_chat", text: this.#buildRecentChat() });
 
     return layers;
@@ -86,6 +93,30 @@ export class ContextEngine {
     return ids;
   }
 
+  setPendingAssistantRecallHints(hints = []) {
+    this.pendingAssistantRecallHints = uniqueHints(hints);
+    this.pendingAssistantRecallHintsRendered = false;
+  }
+
+  peekPendingAssistantRecallHints() {
+    return this.pendingAssistantRecallHints;
+  }
+
+  hasRenderedPendingAssistantRecallHints() {
+    return this.pendingAssistantRecallHintsRendered;
+  }
+
+  markPendingAssistantRecallHintsRendered() {
+    if (this.pendingAssistantRecallHints.length > 0) this.pendingAssistantRecallHintsRendered = true;
+  }
+
+  takePendingAssistantRecallHints() {
+    const hints = this.pendingAssistantRecallHints;
+    this.pendingAssistantRecallHints = [];
+    this.pendingAssistantRecallHintsRendered = false;
+    return hints;
+  }
+
   resolvePath(raw) {
     return resolve(this.cwd, raw);
   }
@@ -107,9 +138,15 @@ export class ContextEngine {
   restoreSession(data, _pool, { replace = false } = {}) {
     if (replace) {
       this.turns = [];
+      this.pendingAssistantRecallHints = [];
+      this.pendingAssistantRecallHintsRendered = false;
       this.sessionName = "";
     }
     if (data.turns) this.turns = data.turns;
+    if (Array.isArray(data.pendingAssistantRecallHints)) {
+      this.pendingAssistantRecallHints = uniqueHints(data.pendingAssistantRecallHints);
+      this.pendingAssistantRecallHintsRendered = false;
+    }
     if (typeof data.sessionName === "string") this.sessionName = data.sessionName;
     this.setRuntimeState(data);
   }
@@ -147,3 +184,14 @@ function appendCurrentUser(recentChat, userMessage) {
   const currentUser = String(userMessage ?? "").trimEnd();
   return `${recentChat}\n\n[current_user]\n${currentUser}`;
 }
+
+function uniqueHints(hints = []) {
+  const seen = new Set();
+  const unique = [];
+  for (const hint of hints) {
+    if (!hint?.id || seen.has(hint.id)) continue;
+    seen.add(hint.id);
+    unique.push(hint);
+  }
+  return unique;
+}

package/src/context/system-core/base.md

@@ -7,7 +7,7 @@ The user primarily asks for software engineering work: fixing bugs, adding behav
 - Be concise and direct. Match the response shape to the task; simple questions get simple answers.
 - Assume users may not see tool calls. Before the first tool call, say in one sentence what you are about to do. While working, give brief updates when you find something important, change direction, or hit a blocker.
 - Don't narrate hidden reasoning. State decisions, results, and relevant next steps.
-- End with one or two sentences: what changed, verification status, and what's next if anything.
+- End with a brief summary of what you did during the task, including what changed, verification status, and what's next if anything; keep it concise, but don't omit the execution overview.
 - Report outcomes truthfully. If tests fail or a step was skipped, say so plainly with the relevant output or reason.
 </communication_contract>
 
@@ -57,4 +57,8 @@ The user primarily asks for software engineering work: fixing bugs, adding behav
 - Use memory_search(query) for full-text search across all memories.
 - To edit an existing memory, use memory_open(id) to get its path, then edit_file with mode="patch" for targeted edits.
 - Use memory_save() to create memories or update whole fields. Before creating a new memory, first search/open related memories and merge updates into an existing memory when they share the same topic, project, or decision thread; prefer modifying the existing memory file over creating a scattered new one. Tags are the primary retrieval key for future recall. Prefer lowercase kebab-case tags like 'march-cli', 'tooling', 'permissions'.
+- When learning multiple related external workflows or skills, maintain memory as an evolving domain library: start with the specific source name when only one item exists, then rename and rewrite the memory title/description as the scope grows; merge new related learnings into the same memory, preserving each source's unique traits while distilling reusable principles.
+- Distinguish "migrating a Skill to memory" from "learning a Skill": migration preserves the complete Skill folder under memory_root/skills/ and creates a memory index with purpose, source, entry files, and local path; learning only reads and internalizes the Skill's methods, scenarios, and principles into ordinary memory without copying source files. Infer the action from the user's wording, and ask when ambiguous.
+- Unlike memory hints, this system-core center is always visible in every model call. Only update the center for instructions that must always be followed; use memory for contextual, project-specific, or recall-dependent knowledge.
+- If execution takes a meaningful detour, create or update a memory after the task. A detour means the initial plan or assumption failed, multiple approaches were tried, and the final successful path contains reusable project knowledge. Record the failed assumption, what was tried, and the successful approach. Prefer updating an existing related memory over creating a new one.
 </memory_system>

package/src/main.mjs

@@ -28,6 +28,7 @@ import { formatStartupBanner } from "./cli/startup/startup-banner.mjs";
 import { initializeMcp } from "./mcp/index.mjs";
 import { createWebToolsFromConfig } from "./web/tools.mjs";
 import { createModelContextDumper } from "./debug/model-context-dumper.mjs";
+import { defaultCenterMemoryPath } from "./context/center-memory.mjs";
 import { runProviderConfigCommand } from "./provider/config-command.mjs";
 import { runWebSearchConfigCommand } from "./web/config-command.mjs";
 import { createDesktopTurnNotifier } from "./notification/desktop-notifier.mjs";
@@ -91,6 +92,7 @@ export async function run(argv) {
   const modeState = createModeState();
   const namespace = loadOrCreateProjectId(projectMarchDir);
   const memoryRoot = resolveMemoryRoot(config.memoryRoot, stateRoot);
+  const centerMemoryPath = defaultCenterMemoryPath();
   const memoryStore = new MarkdownMemoryStore({ root: memoryRoot });
   const memoryTools = createMarkdownMemoryTools(memoryStore);
   const currentProject = basename(cwd);
@@ -111,6 +113,7 @@ export async function run(argv) {
   const webTools = createWebToolsFromConfig(config);
   const turnNotifier = createDesktopTurnNotifier({
     enabled: Boolean(config.notifications?.turnEnd),
+    config: config.notifications,
   });
 
   // Permission controller
@@ -156,6 +159,7 @@ export async function run(argv) {
     stateRoot,
     ui,
     memoryRoot,
+    centerMemoryPath,
     memoryStore,
     memoryTools,
     shellRuntime,