@hicaru/pi-rlm 0.2.1 → 0.3.0

package/src/tool/repl-result.ts

@@ -0,0 +1,70 @@
+/**
+ * Model-visible text assembly for a repl() result, plus the advisory diagnostics derived from
+ * its sub-calls. Split out of repl-tool.ts: this is pure string/array work with no sandbox,
+ * emitter, or TUI dependency, and both halves are asserted directly by test/phase-guards.ts.
+ */
+
+import type { RlmSubcall } from "./rlm-details.ts";
+import { capReplResultText, replDelegationNudge } from "../mode/native-guards.ts";
+
+/** Model-visible text assembled from a repl() result. */
+export interface ReplResultText {
+  readonly text: string;
+}
+
+/**
+ * Assemble the model-visible text for a repl() result: cap stdout, append a zero-subcall
+ * delegation nudge when a bulk read went undelegated, and report tasks still running.
+ *
+ * The pending line is the model's only signal that `spawn()`ed work is outstanding — without
+ * it a model that spawned and moved on has no way to know it should still collect.
+ *
+ * `varNames` covers the opposite failure: a block that stores its results in `answers` and
+ * prints nothing reads as a bare "(no output)", so the model concludes the block did nothing
+ * and re-runs it — paying twice for the same sub-calls. The headless engine already answers
+ * this with the same hint (core/answer.ts); native mode was the only path missing it.
+ */
+export function buildReplResultText(
+  stdout: string,
+  finalAnswer: string | undefined,
+  subcalls: readonly RlmSubcall[],
+  backgroundPending = 0,
+  varNames: readonly string[] = [],
+): ReplResultText {
+  const answerSubmitted = finalAnswer !== undefined;
+  const noOutput = !answerSubmitted && !stdout;
+  const varsHint = noOutput && varNames.length > 0
+    ? ` — the block ran fine and these REPL vars are defined: ${varNames.join(", ")}. `
+      + "Do NOT re-run it; read them in the next block."
+    : "";
+  const rawText = answerSubmitted
+    ? `ANSWER_SUBMITTED (${finalAnswer.length} chars) — delivered to user. Do not restate it.`
+    : stdout || `(no output)${varsHint}`;
+  // Model-visible text is capped; the caller keeps full stdout in `details` for the TUI.
+  const cappedText = capReplResultText(rawText) ?? rawText;
+  const delegated = subcalls.some((s) => s.kind === "llm" || s.kind === "batch" || s.kind === "rlm");
+  const nudge = answerSubmitted ? undefined : replDelegationNudge(rawText.length, delegated);
+  const failedBg = subcalls.filter((s) => s.id.startsWith("bg") && s.status === "error").length;
+  const pendingLine = backgroundPending > 0
+    ? `\n\n[rlm] ${backgroundPending} background task(s) still running — rlm_await_all(tasks) to collect.`
+    : "";
+  const failedLine = failedBg > 0
+    ? `\n[rlm] ${failedBg} background sub-call(s) FAILED — their rlm_await value is an "Error: …" string, not data.`
+    : "";
+  return { text: cappedText + (nudge ?? "") + pendingLine + failedLine };
+}
+
+/** Advisory diagnostics derived from a completed invocation's sub-calls. */
+export function collectReplWarnings(subcalls: readonly RlmSubcall[]): readonly string[] | undefined {
+  let failed = 0;
+  let total = 0;
+  for (let i = 0; i < subcalls.length; i++) {
+    const call = subcalls[i];
+    if (call.status !== "error") continue;
+    // A batch subcall stands for many prompts; a single call stands for one.
+    failed += call.failedCount ?? 1;
+    total += call.totalCount ?? 1;
+  }
+  if (failed === 0) return undefined;
+  return Object.freeze([`${failed}/${total} sub-call(s) failed — results may be incomplete`]);
+}

package/src/tool/repl-tool.ts

@@ -16,16 +16,15 @@
  */
 
 import { Type } from "typebox";
-import type { Theme, ToolDefinition } from "@earendil-works/pi-coding-agent";
-import { Container, Spacer, Text } from "@earendil-works/pi-tui";
+import type { ToolDefinition } from "@earendil-works/pi-coding-agent";
+import { Text } from "@earendil-works/pi-tui";
 import type { Model, Usage, Api } from "@earendil-works/pi-ai";
 import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
-import { buildInteractiveHandlers } from "../bridge/interactive.ts";
-import { buildLibraryHandler } from "../bridge/library.ts";
-import { createPiInteractiveDeps } from "../bridge/pi-interactive.ts";
+import { buildAddContextHandler, type AddContextHandlerBundle } from "../bridge/add-context.ts";
+import { contextPrefixesIn } from "../context/namespace.ts";
 import type { SubcallGates } from "../util/concurrency.ts";
 import { LimitGuard, limitsFromConfig } from "../core/limits.ts";
-import type { InteractiveDeps, RlmConfig, RlmInput, RlmResult } from "../core/types.ts";
+import type { RlmConfig, RlmInput, RlmResult } from "../core/types.ts";
 import { SandboxManager } from "../sandbox/sandbox-manager.ts";
 import type { SubcallOpts } from "../sandbox/sandbox.ts";
 import { createSubcallHandlers, type Invocation } from "../bridge/subcall-handlers.ts";
@@ -39,26 +38,13 @@ import { createEngine } from "../core/engine.ts";
 import { spinnerFrame } from "../ui/theme.ts";
 import { previewText } from "../text/preview.ts";
 import { errorMessage } from "../util/errors.ts";
-import {
-  cardHeader,
-  cardStatsLine,
-  renderCollapsedCard,
-  renderExpandedSubcallTree,
-} from "./subcall-render.ts";
 import { createProgressNotifier, validateToolParams } from "./tool-utils.ts";
-import { capReplResultText, replDelegationNudge } from "../mode/native-guards.ts";
+import { buildReplResultText, collectReplWarnings } from "./repl-result.ts";
+import { renderReplCollapsed, renderReplExpanded } from "./repl-render.ts";
 import { attachTracer, trace, traceEnabled } from "../util/trace.ts";
 
-/** Chars of code shown on the tool call line, and of stdout in the expanded view. */
+/** Chars of code shown on the tool call line. */
 const CALL_PREVIEW_CHARS = 80;
-const EXPANDED_STDOUT_CHARS = 2_000;
-const EXPANDED_STDERR_CHARS = 500;
-
-// ── Parameter schema ──
-
-export const ReplToolParams = Object.freeze(Type.Object({
-  code: Type.String({ description: "Python code to execute in the persistent REPL sandbox" }),
-}));
 
 /** Last non-empty line of a Python traceback — the `TypeError: …` line, not the frames. */
 function lastLine(text: string): string {
@@ -70,67 +56,11 @@ function lastLine(text: string): string {
   return "";
 }
 
-/** Model-visible text assembled from a repl() result. */
-export interface ReplResultText {
-  readonly text: string;
-}
-
-/**
- * Assemble the model-visible text for a repl() result: cap stdout, append a zero-subcall
- * delegation nudge when a bulk read went undelegated, and report tasks still running.
- *
- * The pending line is the model's only signal that `spawn()`ed work is outstanding — without
- * it a model that spawned and moved on has no way to know it should still collect.
- *
- * `varNames` covers the opposite failure: a block that stores its results in `answers` and
- * prints nothing reads as a bare "(no output)", so the model concludes the block did nothing
- * and re-runs it — paying twice for the same sub-calls. The headless engine already answers
- * this with the same hint (core/answer.ts); native mode was the only path missing it.
- */
-export function buildReplResultText(
-  stdout: string,
-  finalAnswer: string | undefined,
-  subcalls: readonly RlmSubcall[],
-  backgroundPending = 0,
-  varNames: readonly string[] = [],
-): ReplResultText {
-  const answerSubmitted = finalAnswer !== undefined;
-  const noOutput = !answerSubmitted && !stdout;
-  const varsHint = noOutput && varNames.length > 0
-    ? ` — the block ran fine and these REPL vars are defined: ${varNames.join(", ")}. `
-      + "Do NOT re-run it; read them in the next block."
-    : "";
-  const rawText = answerSubmitted
-    ? `ANSWER_SUBMITTED (${finalAnswer.length} chars) — delivered to user. Do not restate it.`
-    : stdout || `(no output)${varsHint}`;
-  // Model-visible text is capped; the caller keeps full stdout in `details` for the TUI.
-  const cappedText = capReplResultText(rawText) ?? rawText;
-  const delegated = subcalls.some((s) => s.kind === "llm" || s.kind === "batch" || s.kind === "rlm");
-  const nudge = answerSubmitted ? undefined : replDelegationNudge(rawText.length, delegated);
-  const failedBg = subcalls.filter((s) => s.id.startsWith("bg") && s.status === "error").length;
-  const pendingLine = backgroundPending > 0
-    ? `\n\n[rlm] ${backgroundPending} background task(s) still running — rlm_await_all(tasks) to collect.`
-    : "";
-  const failedLine = failedBg > 0
-    ? `\n[rlm] ${failedBg} background sub-call(s) FAILED — their rlm_await value is an "Error: …" string, not data.`
-    : "";
-  return { text: cappedText + (nudge ?? "") + pendingLine + failedLine };
-}
+// ── Parameter schema ──
 
-/** Advisory diagnostics derived from a completed invocation's sub-calls. */
-export function collectReplWarnings(subcalls: readonly RlmSubcall[]): readonly string[] | undefined {
-  let failed = 0;
-  let total = 0;
-  for (let i = 0; i < subcalls.length; i++) {
-    const call = subcalls[i];
-    if (call.status !== "error") continue;
-    // A batch subcall stands for many prompts; a single call stands for one.
-    failed += call.failedCount ?? 1;
-    total += call.totalCount ?? 1;
-  }
-  if (failed === 0) return undefined;
-  return Object.freeze([`${failed}/${total} sub-call(s) failed — results may be incomplete`]);
-}
+export const ReplToolParams = Object.freeze(Type.Object({
+  code: Type.String({ description: "Python code to execute in the persistent REPL sandbox" }),
+}));
 
 // ── Mutable bridge state (handler indirection) ──
 
@@ -141,21 +71,18 @@ export function collectReplWarnings(subcalls: readonly RlmSubcall[]): readonly s
  * calls rather than rebuilding handlers (which would lose REPL variable state). Handlers
  * capture the Invocation synchronously at interrupt entry and never re-read it — with
  * spawn() a sub-call can outlive its exec, and a later read would attribute it to whichever
- * turn happened to be current when it resumed.
+ * turn happened to be current when it settled.
  *
  * Detached work resolves to the session-scoped background Invocation instead, whose emitter
  * and LimitGuard are not torn down at the end of a turn.
  */
 class NativeBridgeState {
   private current: Invocation | null = null;
-  /** Interactive callbacks for the turn in progress; child engines inherit them. */
-  interactive: InteractiveDeps | null = null;
 
   constructor(private readonly background: BackgroundTasks) {}
 
-  swap(inv: Invocation, interactive: InteractiveDeps): void {
+  swap(inv: Invocation): void {
     this.current = Object.freeze({ ...inv });
-    this.interactive = interactive;
   }
 
   /** Detached ⇒ session registry; otherwise the turn that is currently executing. */
@@ -175,9 +102,9 @@ class NativeBridgeState {
 export interface ReplToolDeps {
   readonly sandboxManager: SandboxManager;
   readonly model: Model<Api>;
-  readonly workerModel: Model<Api>;
+  readonly llmModel: Model<Api>;
   readonly getModel?: () => Model<Api> | undefined;
-  readonly getWorkerModel?: () => Model<Api> | undefined;
+  readonly getLlmModel?: () => Model<Api> | undefined;
   readonly registry: ModelRegistry;
   /** Live accessor — `/rlm-config` replaces the config object, so never capture the value. */
   readonly getConfig: () => RlmConfig;
@@ -188,18 +115,23 @@ export interface ReplToolDeps {
   readonly signal?: AbortSignal;
   readonly onUsage?: (usage: Usage, role: "sub") => void;
   readonly ensureContext?: () => Promise<void>;
-  /** Register a reset hook for sandbox death/dispose (e.g. load_library slot counter). */
+  /** Register a reset hook for sandbox death/dispose (e.g. add_context prefix cache). */
   readonly registerDiscardHook?: (reset: () => void) => void;
+  /**
+   * Hands the live add_context bundle to the extension so the cwd seed can
+   * markLoaded("") / markSeededCwd(abs) — without this, add_context(".") doubles the tree.
+   */
+  readonly registerContextBundle?: (bundle: AddContextHandlerBundle) => void;
 }
 
 export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplToolParams, ReplDetails> {
-  const { sandboxManager, workerModel, registry, getConfig, signal, onUsage, background } = deps;
+  const { sandboxManager, llmModel, registry, getConfig, signal, onUsage, background } = deps;
   const bridgeState = new NativeBridgeState(background);
 
   // Late-bound cwd — getOrCreate installs handlers only at spawn; never rebuild the closure.
   let sessionCwd = process.cwd();
 
-  const getWorkerModel = (): Model<Api> => deps.getWorkerModel?.() ?? workerModel;
+  const getLlmModel = (): Model<Api> => deps.getLlmModel?.() ?? llmModel;
   const getModel = (): Model<Api> => deps.getModel?.() ?? deps.model;
 
   // Each rlm_query spawns a child RLM with its own sandbox and turn loop, not a flat
@@ -207,7 +139,7 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
   // progress and cost deltas land on the emitter the parent invocation is using.
   const runChild = (input: RlmInput, inv: Invocation): Promise<RlmResult> => createEngine({
     model: getModel(),
-    workerModel: getWorkerModel(),
+    llmModel: getLlmModel(),
     registry,
     config: getConfig(),
     signal,
@@ -218,8 +150,6 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
     // the child's own root turns — so fold both roles into "sub" rather than casting.
     onUsage: onUsage === undefined ? undefined : (usage: Usage) => onUsage(usage, "sub"),
     limits: limitsFromConfig(getConfig()),
-    onTodo: bridgeState.interactive?.onTodo,
-    onAskUserQuestion: bridgeState.interactive?.onAskUserQuestion,
   })(input);
 
   // Built once: the same closures stay correct across repl() calls because everything
@@ -228,39 +158,60 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
     resolve: (opts) => bridgeState.resolve(opts),
     gates: deps.gates,
     registry,
-    getWorkerModel,
+    getLlmModel,
     getModel,
     getConfig,
     signal,
     onUsage,
     runChild,
+    // The session sandbox's context is the child's world. Read lazily so an add_context from an
+    // earlier repl() reaches a child spawned in a later one. Populated before any interrupt can
+    // fire: execute() awaits ensureContext() before getOrCreate().
+    getChildContext: () => sandboxManager.contextPayload ?? undefined,
     trackDetached: (task) => background.track(task),
   });
 
-  const libraryBundle = getConfig().libraryLoader
-    ? buildLibraryHandler({
+  const contextBundle = getConfig().contextLoader
+    ? buildAddContextHandler({
         getCwd: () => sessionCwd,
         getEmitter: () => bridgeState.currentEmitter,
+        // Refuse pre-flight whatever the worker would reject, so host idempotency is never
+        // committed for an append that did not happen.
+        getContext: () => sandboxManager.contextPayload,
         parentId: undefined,
         signal,
-        startIndex: 1,
+        // Keep the manager's replay copy in step with the worker's live `context`, and with it
+        // whatever a child spawned after this load will inherit.
+        onLoaded: (payload) => { sandboxManager.appendContext(payload); },
       })
     : undefined;
-  if (libraryBundle) deps.registerDiscardHook?.(libraryBundle.reset);
+  if (contextBundle) {
+    // Re-derive the loaded-prefix cache from the payload that will actually be replayed —
+    // clearing it outright would make the host re-clone a source the recreated worker already has.
+    // Re-plant the cwd sentinel if the seed is still in the payload (un-prefixed files).
+    const bundle = contextBundle;
+    deps.registerDiscardHook?.(() => {
+      const prefixes = contextPrefixesIn(sandboxManager.contextPayload);
+      bundle.reset(prefixes);
+      if (bundle.seededCwd() !== undefined) bundle.markLoaded("");
+    });
+    deps.registerContextBundle?.(contextBundle);
+  }
 
   return {
     name: "repl",
     label: "REPL",
     description:
       "PRIMARY tool for ALL repository reading and analysis (read/grep are disabled in RLM mode). " +
-      "Persistent Python sandbox with every file pre-loaded in `context`. Locate first with the " +
-      "free primitives search(query) / grep_context(pattern) / outline(path), then delegate the " +
-      "semantic reading to map_files / llm_query / llm_query_batched / llm_query_chunked " +
-      "(rlm_query for iterative sub-tasks) — stdout returned to you is hard-capped at 4K chars, " +
-      "so printing file bodies is useless. Variables, imports, and the `answers`/`plan` memo " +
-      "persist across calls. Also supports todo, ask_user_question, and load_library.",
+      "Persistent Python sandbox with loaded files in `context` (starts empty; cwd seeds on first " +
+      "call). Locate first with the free primitives search(query) / grep_context(pattern) / " +
+      "outline(path), then delegate the semantic reading to map_files / llm_query / " +
+      "llm_query_batched / llm_query_chunked (rlm_query for iterative sub-tasks) — stdout " +
+      "returned to you is hard-capped at 4K chars, so printing file bodies is useless. " +
+      "Variables, imports, and the `answers`/`plan` memo persist across calls. Also supports " +
+      "add_context for external dirs/files/git URLs and document conversion.",
     promptSnippet:
-      "repl: run Python in a persistent sandbox holding the whole repository in `context`; " +
+      "repl: run Python in a persistent sandbox holding loaded files in `context`; " +
       "search/grep_context/outline to locate, map_files/llm_query* to read.",
     promptGuidelines: [
       "In RLM mode, read the repository through `repl` only — `read`/`grep` and bash readers are blocked.",
@@ -327,25 +278,12 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
       }
 
       try {
-        // Build interactive handlers (session-stable callbacks)
-        const interactive = createPiInteractiveDeps(ctx);
-        const interactiveHandlers = buildInteractiveHandlers({
-          onAskUserQuestion: getConfig().askUserQuestion ? interactive.onAskUserQuestion : undefined,
-          onTodo: interactive.onTodo,
-          onTodoRow: undefined,
-          emitter,
-          depth: 0,
-          parentId: undefined,
-        });
-
         sessionCwd = ctx.cwd ?? process.cwd();
 
         await deps.ensureContext?.();
         await sandboxManager.getOrCreate({
           ...subcallHandlers,
-          askUserQuestion: interactiveHandlers.askUserQuestion,
-          todo: interactiveHandlers.todo,
-          ...(libraryBundle?.handlers ?? {}),
+          ...(contextBundle?.handlers ?? {}),
         });
 
         // Detect queue contention AFTER sandbox init (initPromise settled, isExecuting now accurate)
@@ -366,7 +304,7 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
           // Wire per-invocation mutable state only after the serialized exec slot
           // is active. Swapping earlier would let queued repl() calls overwrite
           // emitter/limits for the currently running REPL execution.
-          bridgeState.swap({ emitter, parentId: undefined, depth: 0, limits }, interactive);
+          bridgeState.swap({ emitter, parentId: undefined, depth: 0, limits });
         }, execSignal);
         const elapsed = Date.now() - start;
         capturedStdout = result.stdout;
@@ -480,51 +418,3 @@ export function createReplTool(deps: ReplToolDeps): ToolDefinition<typeof ReplTo
     },
   };
 }
-
-// ── Collapsed view ──
-
-function replStats(details: ReplDetails, theme: Theme): string {
-  const elapsed = details.executionTimeMs > 0 ? `${details.executionTimeMs}ms` : undefined;
-  return cardStatsLine(details.totals, theme, elapsed, details.backgroundPending);
-}
-
-function renderReplCollapsed(details: ReplDetails, theme: Theme): Text {
-  return renderCollapsedCard("REPL", details.status, replStats(details, theme), details.subcalls, theme);
-}
-
-// ── Expanded view ──
-
-function renderReplExpanded(details: ReplDetails, theme: Theme): Container {
-  const container = new Container();
-
-  container.addChild(new Text(cardHeader("REPL", details.status, replStats(details, theme), theme), 0, 0));
-
-  // Output
-  if (details.output) {
-    container.addChild(new Spacer(1));
-    const out = details.output.length > EXPANDED_STDOUT_CHARS
-      ? `${details.output.slice(0, EXPANDED_STDOUT_CHARS)}…`
-      : details.output;
-    container.addChild(new Text(out, 0, 0));
-  }
-
-  if (details.warnings && details.warnings.length > 0) {
-    container.addChild(new Spacer(1));
-    container.addChild(new Text(theme.fg("muted", details.warnings.join("\n")), 0, 0));
-  }
-
-  // Stderr
-  if (details.stderr) {
-    container.addChild(new Spacer(1));
-    container.addChild(new Text(theme.fg("error", details.stderr.slice(0, EXPANDED_STDERR_CHARS)), 0, 0));
-  }
-
-  // Sub-call tree
-  if (details.subcalls.length > 0) {
-    container.addChild(new Spacer(1));
-    container.addChild(new Text(theme.fg("muted", "─── Sub-calls ───"), 0, 0));
-    container.addChild(renderExpandedSubcallTree(details.subcalls, theme));
-  }
-
-  return container;
-}

package/src/tool/rlm-aggregator.ts

@@ -6,14 +6,14 @@
  * getState(): RlmDetails for direct access (spinner loop, final return).
  *
  * Subcall storage and totals are delegated to SubcallStore. Root-level state
- * (status, prompt, turns, answer, warnings) is kept in the aggregator.
+ * (status, prompt, turns, answer) is kept in the aggregator.
  *
  * Replaces RlmToolBridge's internal state accumulation. The emitter is pure
  * dispatch; the aggregator is pure state. Separated for independent testing.
  */
 
 import type { AgentToolUpdateCallback } from "@earendil-works/pi-agent-core";
-import type { RlmEmitter, TurnEvent, RootUsageEvent, AnswerEvent, StatusEvent, RootPromptEvent, WarningsEvent } from "./rlm-events.ts";
+import type { RlmEmitter, TurnEvent, RootUsageEvent, AnswerEvent, StatusEvent, RootPromptEvent } from "./rlm-events.ts";
 import type { RlmDetails, RlmRunStatus } from "./rlm-details.ts";
 import { EmitterListener } from "./emitter-listener.ts";
 import { SubcallStore } from "./subcall-store.ts";
@@ -27,7 +27,6 @@ export class RlmEventAggregator extends EmitterListener {
   private turnCurrent = 0;
   private turnMax = 0;
   private answer?: string;
-  private warnings?: readonly string[];
 
   constructor(
     emitter: RlmEmitter,
@@ -40,7 +39,6 @@ export class RlmEventAggregator extends EmitterListener {
       emitter.onTurn((e) => this.handleTurn(e)),
       emitter.onRootUsage((e) => this.handleRootUsage(e)),
       emitter.onAnswer((e) => this.handleAnswer(e)),
-      emitter.onWarnings((e) => this.handleWarnings(e)),
       emitter.onStatus((e) => this.handleStatus(e)),
       emitter.onRootPrompt((e) => this.handleRootPrompt(e)),
     ]);
@@ -64,11 +62,6 @@ export class RlmEventAggregator extends EmitterListener {
     this.notify();
   }
 
-  private handleWarnings(event: WarningsEvent): void {
-    this.warnings = event.warnings;
-    this.notify();
-  }
-
   private handleStatus(event: StatusEvent): void {
     this.rootStatus = event.status;
     this.notify();
@@ -90,7 +83,6 @@ export class RlmEventAggregator extends EmitterListener {
       subcalls: this.store.getSubcalls(),
       totals: this.store.getTotals(),
       answer: this.answer,
-      warnings: this.warnings,
     };
   }
 

package/src/tool/rlm-details.ts

@@ -40,7 +40,5 @@ export interface RlmDetails {
   readonly subcalls: readonly RlmSubcall[];
   readonly totals: { readonly costUsd: number; readonly tokens: number };
   readonly answer?: string;
-  /** Advisory diagnostics — surfaced to the user, never a failure. */
-  readonly warnings?: readonly string[];
 }
 

package/src/tool/rlm-events.ts

@@ -45,10 +45,6 @@ export interface SubcallUpdatedEvent {
   readonly totalCount?: number;
 }
 
-export interface WarningsEvent {
-  readonly warnings: readonly string[];
-}
-
 export interface TurnEvent {
   readonly current: number;
   readonly max: number;
@@ -121,11 +117,6 @@ export class RlmEmitter {
     this.ee.emit("answer", { text } satisfies AnswerEvent);
   }
 
-  /** Set advisory warnings (root-only; never a failure). */
-  emitWarnings(warnings: readonly string[]): void {
-    this.ee.emit("warnings", { warnings } satisfies WarningsEvent);
-  }
-
   /** Set the root run status (done/error/aborted). */
   emitStatus(status: RlmRunStatus): void {
     this.ee.emit("status", { status } satisfies StatusEvent);
@@ -163,11 +154,6 @@ export class RlmEmitter {
     return () => { this.ee.off("answer", handler); };
   }
 
-  onWarnings(handler: (event: WarningsEvent) => void): () => void {
-    this.ee.on("warnings", handler);
-    return () => { this.ee.off("warnings", handler); };
-  }
-
   onStatus(handler: (event: StatusEvent) => void): () => void {
     this.ee.on("status", handler);
     return () => { this.ee.off("status", handler); };

package/src/tool/rlm-tool.ts

@@ -8,7 +8,6 @@
 import { type Theme, type ToolDefinition } from "@earendil-works/pi-coding-agent";
 import { Container, Markdown, Spacer, Text, type Component } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
-import { createPiInteractiveDeps } from "../bridge/pi-interactive.ts";
 import type { RlmController, StartInput } from "../mode/rlm-mode.ts";
 import { spinnerFrame } from "../ui/theme.ts";
 import { markdownTheme } from "../ui/theme-adapter.ts";
@@ -32,7 +31,7 @@ const CALL_PREVIEW_CHARS = 80;
 
 export const RlmToolParams = Object.freeze(Type.Object({
   prompt: Type.String({ description: "The task or question for the RLM engine" }),
-  context: Type.Optional(Type.String({ description: "Optional context. If omitted, repo is auto-packed via repomix." })),
+  context: Type.Optional(Type.String({ description: "Optional context. If omitted, the working directory is packed into context." })),
 }));
 
 // ── Rendering helpers ──
@@ -82,15 +81,10 @@ export function createRlmTool(controller: RlmController): ToolDefinition<typeof
 
       try {
         const input: StartInput = {
-          kind: "fresh",
           rootPrompt: params.prompt,
           context: params.context ?? undefined,
         };
-        const interactive = createPiInteractiveDeps(ctx);
-        const { done } = controller.start(ctx, input, emitter, {
-          onAskUserQuestion: controller.config.askUserQuestion ? interactive.onAskUserQuestion : undefined,
-          onTodo: controller.config.todo ? interactive.onTodo : undefined,
-        });
+        const { done } = controller.start(ctx, input, emitter);
         const result = await done;
 
         emitter.emitAnswer(result.answer);
@@ -152,11 +146,6 @@ function renderExpanded(details: RlmDetails, theme: Theme): Component {
     container.addChild(new Markdown(details.answer, 0, 0, markdownTheme(theme)));
   }
 
-  if (details.warnings && details.warnings.length > 0) {
-    container.addChild(new Spacer(1));
-    container.addChild(new Text(theme.fg("muted", details.warnings.join("\n")), 0, 0));
-  }
-
   return container;
 }