comisai 1.0.37 → 1.0.40

package/README.md

@@ -138,19 +138,21 @@ comis sessions           # List and manage conversations
 
 ## Architecture
 
-Hexagonal (ports and adapters). Core defines port interfaces -- adapters implement them. Swap Discord for Matrix, SQLite for Postgres, or OpenAI for Ollama without touching core logic. **19 ports, 30+ adapters, 13 packages.** Every function returns `Result<T, E>` -- zero thrown exceptions.
+Hexagonal (ports and adapters). Core defines port interfaces -- adapters implement them. Swap Discord for Matrix, SQLite for Postgres, or OpenAI for Ollama without touching core logic. **25 ports, 30+ adapters, 14 packages.** Every function returns `Result<T, E>` -- zero thrown exceptions.
 
 ```
 shared (Result type, utilities)
  └── core (domain types, ports, event bus, security, config)
+      ├── infra (Pino structured logging runtime)
       ├── memory (SQLite + FTS5 + vector search)
       ├── gateway (HTTP, JSON-RPC, WebSocket, mTLS)
       ├── skills (tools, MCP, media processing)
       ├── scheduler (cron, heartbeats)
       ├── agent (executor, budget, RAG, sessions)
       ├── channels (9 platform adapters)
+      ├── orchestrator (inbound pipeline, channel manager, routing)
       ├── cli (commands, RPC client)
-      └── daemon (orchestrator, observability)
+      └── daemon (process orchestrator, observability)
 ```
 
 ---

package/dist/cli-entry.js

@@ -9,8 +9,9 @@
  * @module
  */
 import { fileURLToPath } from "node:url";
-import { dirname, join } from "node:path";
+import { dirname } from "node:path";
+import { safePath } from "@comis/core";
 // Resolve @comis/cli index via ESM, then derive cli.js from the same dist/ dir
 const cliIndexUrl = import.meta.resolve("@comis/cli");
 const cliDistDir = dirname(fileURLToPath(cliIndexUrl));
-await import(join(cliDistDir, "cli.js"));
+await import(safePath(cliDistDir, "cli.js"));

package/dist/index.d.ts

@@ -7,6 +7,7 @@ import * as skills from "@comis/skills";
 import * as scheduler from "@comis/scheduler";
 import * as agent from "@comis/agent";
 import * as channels from "@comis/channels";
+import * as orchestrator from "@comis/orchestrator";
 import * as cli from "@comis/cli";
 import * as daemon from "@comis/daemon";
-export { shared, core, infra, memory, gateway, skills, scheduler, agent, channels, cli, daemon, };
+export { shared, core, infra, memory, gateway, skills, scheduler, agent, channels, orchestrator, cli, daemon, };

package/dist/index.js

@@ -10,6 +10,7 @@ import * as skills from "@comis/skills";
 import * as scheduler from "@comis/scheduler";
 import * as agent from "@comis/agent";
 import * as channels from "@comis/channels";
+import * as orchestrator from "@comis/orchestrator";
 import * as cli from "@comis/cli";
 import * as daemon from "@comis/daemon";
-export { shared, core, infra, memory, gateway, skills, scheduler, agent, channels, cli, daemon, };
+export { shared, core, infra, memory, gateway, skills, scheduler, agent, channels, orchestrator, cli, daemon, };

package/dist/orchestrator.d.ts

@@ -0,0 +1 @@
+export * from "@comis/orchestrator";

package/dist/orchestrator.js

@@ -0,0 +1,2 @@
+// SPDX-License-Identifier: Apache-2.0
+export * from "@comis/orchestrator";

package/node_modules/@comis/agent/dist/background/auto-background-middleware.d.ts

@@ -1,6 +1,6 @@
 import type { BackgroundTasksConfig } from "@comis/core";
-import type { AgentToolResult } from "@mariozechner/pi-agent-core";
-import type { BackgroundTaskManager, NotifyFn } from "./background-task-manager.js";
+import type { AgentToolResult } from "@earendil-works/pi-agent-core";
+import type { BackgroundTaskManager } from "./background-task-manager.js";
 import type { BackgroundTaskOrigin } from "./background-task-types.js";
 /**
  * Tool definition interface matching pi-agent-core ToolDefinition.
@@ -34,4 +34,4 @@ export interface ToolDefinition {
  * If the tool exceeds the timeout, promotes to background via manager.promote().
  * If promotion fails (concurrency limit), awaits the tool normally (foreground fallback).
  */
-export declare function wrapToolForAutoBackground(tool: ToolDefinition, manager: BackgroundTaskManager, config: BackgroundTasksConfig, notifyFn: NotifyFn, originResolver: () => BackgroundTaskOrigin | undefined): ToolDefinition;
+export declare function wrapToolForAutoBackground(tool: ToolDefinition, manager: BackgroundTaskManager, config: BackgroundTasksConfig, originResolver: () => BackgroundTaskOrigin | undefined): ToolDefinition;

package/node_modules/@comis/agent/dist/background/auto-background-middleware.js

@@ -9,6 +9,7 @@
  * @module
  */
 import { suppressError } from "@comis/shared";
+import { systemSetTimeout, systemClearTimeout } from "@comis/core";
 /**
  * Wrap a tool's execute() with auto-background promotion on timeout.
  *
@@ -26,7 +27,7 @@ import { suppressError } from "@comis/shared";
  * If the tool exceeds the timeout, promotes to background via manager.promote().
  * If promotion fails (concurrency limit), awaits the tool normally (foreground fallback).
  */
-export function wrapToolForAutoBackground(tool, manager, config, notifyFn, originResolver) {
+export function wrapToolForAutoBackground(tool, manager, config, originResolver) {
     // `exec` opts out of the generic auto-background wrapper to enforce
     // single-owner backgrounding. The exec-tool's own internal escalation path
     // (packages/skills/src/builtin/exec-tool.ts:613-668) is the SOLE
@@ -64,9 +65,9 @@ export function wrapToolForAutoBackground(tool, manager, config, notifyFn, origi
             const taskPromise = origExecute(toolCallId, params, ac.signal, gatedOnUpdate, ctx);
             // Race: tool result vs. timeout
             const timeoutPromise = new Promise((resolve) => {
-                const timer = setTimeout(() => resolve("timeout"), config.autoBackgroundMs);
+                const timer = systemSetTimeout(() => resolve("timeout"), config.autoBackgroundMs);
                 // Clean up timer if tool finishes first (prevents leak)
-                taskPromise.then(() => clearTimeout(timer), () => clearTimeout(timer));
+                taskPromise.then(() => systemClearTimeout(timer), () => systemClearTimeout(timer));
             });
             const raceResult = await Promise.race([
                 taskPromise.then((value) => ({ kind: "result", value })),
@@ -93,7 +94,7 @@ export function wrapToolForAutoBackground(tool, manager, config, notifyFn, origi
             onUpdateActive = false;
             const taskId = promoteResult.value;
             // Wire completion/failure handlers (fire-and-forget)
-            suppressError(taskPromise.then((result) => manager.complete(taskId, result, notifyFn), (error) => manager.fail(taskId, error, notifyFn)), "background task completion handler");
+            suppressError(taskPromise.then((result) => manager.complete(taskId, result), (error) => manager.fail(taskId, error)), "background task completion handler");
             // Return a well-formed AgentToolResult so the SDK's emitToolCallOutcome
             // produces a non-empty toolResult message. Returning a string here
             // collapses to `content: undefined` and triggers the silent-LLM-failure

package/node_modules/@comis/agent/dist/background/background-task-manager.d.ts

@@ -1,5 +1,5 @@
 import { type Result } from "@comis/shared";
-import type { TypedEventBus } from "@comis/core";
+import type { TypedEventBus, ClockPort, TimerPort } from "@comis/core";
 import type { BackgroundTask, BackgroundTaskOrigin, BackgroundSessionState, BackgroundTaskNotificationPolicy } from "./background-task-types.js";
 /** Notification callback fired when background task completes or fails. */
 export type NotifyFn = (opts: {
@@ -16,6 +16,10 @@ export interface BackgroundTaskManagerOpts {
         warn(obj: Record<string, unknown>, msg: string): void;
         debug(obj: Record<string, unknown>, msg: string): void;
     };
+    /** Wall-clock + monotonic time reads. */
+    clock: ClockPort;
+    /** Timer scheduling. Hard-timeout setTimeout uses .unref(). */
+    timers: TimerPort;
     maxPerAgent?: number;
     maxTotal?: number;
     maxBackgroundDurationMs?: number;
@@ -25,18 +29,16 @@ export interface BackgroundTaskManager {
     /**
      * Mark a task as completed.
      *
-     * The legacy `notifyFn` argument is preserved for backward compatibility but
-     * unused — the completion-dispatcher subscribes to the
-     * `background_task:completed` event emitted here and decides whether to fire
-     * the user-visible fallback notification. Single-owner contract eliminates
-     * double-notify.
+     * The completion-dispatcher subscribes to the `background_task:completed`
+     * event emitted here and decides whether to fire the user-visible fallback
+     * notification. Single-owner contract eliminates double-notify.
      */
-    complete(taskId: string, result: unknown, notifyFn?: NotifyFn): void;
+    complete(taskId: string, result: unknown): void;
     /**
-     * Mark a task as failed. See `complete` for the single-owner note — the
-     * `notifyFn` argument is unused; the dispatcher routes notification.
+     * Mark a task as failed. See `complete` for the single-owner note —
+     * the dispatcher routes notification via the emitted event.
      */
-    fail(taskId: string, error: unknown, notifyFn?: NotifyFn): void;
+    fail(taskId: string, error: unknown): void;
     cancel(taskId: string): Result<void, Error>;
     getTask(taskId: string): BackgroundTask | undefined;
     getTasks(agentId: string): BackgroundTask[];

package/node_modules/@comis/agent/dist/background/background-task-manager.js

@@ -12,7 +12,7 @@ import { ok, err } from "@comis/shared";
 import { persistTaskSync, recoverTasks, removeTaskFile } from "./background-task-persistence.js";
 const MAX_RESULT_CHARS = 102_400; // 100KB
 export function createBackgroundTaskManager(opts) {
-    const { dataDir, eventBus, logger, maxPerAgent = 5, maxTotal = 20, maxBackgroundDurationMs = 300_000, } = opts;
+    const { dataDir, eventBus, logger, clock, timers, maxPerAgent = 5, maxTotal = 20, maxBackgroundDurationMs = 300_000, } = opts;
     const tasks = new Map();
     const perAgentCount = new Map();
     let totalCount = 0;
@@ -67,7 +67,7 @@ export function createBackgroundTaskManager(opts) {
                 id: taskId,
                 toolName,
                 status: "running",
-                startedAt: Date.now(),
+                startedAt: clock.now(),
                 origin,
                 // Seed the dispatch state machine. Default policy is "deferred" —
                 // the dispatcher inspects dispatchState before firing fallback notify
@@ -78,13 +78,13 @@ export function createBackgroundTaskManager(opts) {
                 _abortController: ac,
             };
             // Hard-timeout abort
-            const timer = setTimeout(() => {
+            const timer = timers.setTimeout(() => {
                 if (task.status === "running") {
                     ac.abort();
                     manager.fail(taskId, new Error("Hard timeout exceeded"));
                 }
             }, maxBackgroundDurationMs);
-            timer.unref();
+            timer.unref(); // TimerHandle exposes .unref() by contract.
             task._hardTimeoutTimer = timer;
             tasks.set(taskId, task);
             incrementCounters(agentId);
@@ -93,19 +93,19 @@ export function createBackgroundTaskManager(opts) {
                 agentId,
                 taskId,
                 toolName,
-                timestamp: Date.now(),
+                timestamp: clock.now(),
             });
             return ok(taskId);
         },
-        complete(taskId, result, _notifyFn) {
+        complete(taskId, result) {
             const task = tasks.get(taskId);
             if (!task || task.status !== "running")
                 return;
             task.status = "completed";
-            task.completedAt = Date.now();
+            task.completedAt = clock.now();
             task.result = truncateResult(result);
             if (task._hardTimeoutTimer)
-                clearTimeout(task._hardTimeoutTimer);
+                task._hardTimeoutTimer.cancel();
             decrementCounters(task.origin.agentId);
             persistTaskSync(dataDir, task);
             const durationMs = task.completedAt - task.startedAt;
@@ -115,24 +115,23 @@ export function createBackgroundTaskManager(opts) {
                 toolName: task.toolName,
                 durationMs,
                 origin: task.origin,
-                timestamp: Date.now(),
+                timestamp: clock.now(),
             });
             // Notification routing lives in the completion-dispatcher (subscribed
-            // to background_task:completed above). The legacy `notifyFn` argument
-            // is kept for backward compatibility but unused here — the dispatcher
-            // inspects task.dispatchState before firing the user-visible fallback,
-            // and the runner skips when state is "notified" (single-owner contract,
-            // zero spurious outbound).
+            // to background_task:completed above). The dispatcher inspects
+            // task.dispatchState before firing the user-visible fallback, and the
+            // runner skips when state is "notified" (single-owner contract, zero
+            // spurious outbound).
         },
-        fail(taskId, error, _notifyFn) {
+        fail(taskId, error) {
             const task = tasks.get(taskId);
             if (!task || task.status !== "running")
                 return;
             task.status = "failed";
-            task.completedAt = Date.now();
+            task.completedAt = clock.now();
             task.error = error instanceof Error ? error.message : String(error);
             if (task._hardTimeoutTimer)
-                clearTimeout(task._hardTimeoutTimer);
+                task._hardTimeoutTimer.cancel();
             decrementCounters(task.origin.agentId);
             persistTaskSync(dataDir, task);
             const durationMs = task.completedAt - task.startedAt;
@@ -143,9 +142,9 @@ export function createBackgroundTaskManager(opts) {
                 error: task.error,
                 durationMs,
                 origin: task.origin,
-                timestamp: Date.now(),
+                timestamp: clock.now(),
             });
-            // See complete() above — notifyFn arg is unused; dispatcher owns routing.
+            // See complete() above — the dispatcher owns notification routing.
         },
         cancel(taskId) {
             const task = tasks.get(taskId);
@@ -154,18 +153,18 @@ export function createBackgroundTaskManager(opts) {
             if (task.status !== "running")
                 return err(new Error(`Task ${taskId} is not running (status: ${task.status})`));
             task.status = "cancelled";
-            task.completedAt = Date.now();
+            task.completedAt = clock.now();
             if (task._abortController)
                 task._abortController.abort();
             if (task._hardTimeoutTimer)
-                clearTimeout(task._hardTimeoutTimer);
+                task._hardTimeoutTimer.cancel();
             decrementCounters(task.origin.agentId);
             persistTaskSync(dataDir, task);
             eventBus.emit("background_task:cancelled", {
                 agentId: task.origin.agentId,
                 taskId,
                 toolName: task.toolName,
-                timestamp: Date.now(),
+                timestamp: clock.now(),
             });
             return ok(undefined);
         },
@@ -200,7 +199,7 @@ export function createBackgroundTaskManager(opts) {
                         logger.debug({
                             taskId: task.id,
                             dispatchState: task.dispatchState,
-                            hint: "Pre-restart dispatch state preserved; skipping re-emit (D-S2 recovery-without-events)",
+                            hint: "Pre-restart dispatch state preserved; skipping re-emit (recovery-without-events)",
                         }, "Recovery: skipped re-emit");
                         continue;
                     }
@@ -210,9 +209,9 @@ export function createBackgroundTaskManager(opts) {
                         taskId: task.id,
                         toolName: task.toolName,
                         error: persisted.error,
-                        durationMs: (persisted.completedAt ?? Date.now()) - persisted.startedAt,
+                        durationMs: (persisted.completedAt ?? clock.now()) - persisted.startedAt,
                         origin: task.origin,
-                        timestamp: Date.now(),
+                        timestamp: clock.now(),
                     });
                 }
             }
@@ -234,7 +233,7 @@ export function createBackgroundTaskManager(opts) {
             return true;
         },
         cleanup(maxAgeMs = 86_400_000) {
-            const cutoff = Date.now() - maxAgeMs;
+            const cutoff = clock.now() - maxAgeMs;
             for (const [taskId, task] of tasks) {
                 if (task.status !== "running" && (task.completedAt ?? task.startedAt) < cutoff) {
                     tasks.delete(taskId);

package/node_modules/@comis/agent/dist/background/background-task-persistence.js

@@ -1,4 +1,5 @@
 // SPDX-License-Identifier: Apache-2.0
+// @allow-throw: background task persistence re-raise (line 147) inside try/catch wrapper; outer caller (executor) catches at PiExecutor boundary which is itself consumed by daemon RPC handlers.
 /**
  * File-based persistence for background tasks.
  *
@@ -8,7 +9,7 @@
  * @module
  */
 import { mkdirSync, writeFileSync, readFileSync, readdirSync, statSync, unlinkSync, existsSync } from "node:fs";
-import { safePath } from "@comis/core";
+import { safePath, systemNowMs } from "@comis/core";
 /** Directory name under data dir for background task state files. */
 export const TASK_DIR_NAME = "background-tasks";
 /**
@@ -119,7 +120,7 @@ export function recoverTasks(dataDir) {
                 if (task.status === "running") {
                     task.status = "failed";
                     task.error = "Daemon restarted while task was running";
-                    task.completedAt = Date.now();
+                    task.completedAt = systemNowMs();
                     writeFileSync(filePath, JSON.stringify(task, null, 2), "utf-8");
                 }
                 recovered.push(task);

package/node_modules/@comis/agent/dist/background/background-task-types.d.ts

@@ -3,7 +3,7 @@
  *
  * @module
  */
-import type { BackgroundTaskOrigin } from "@comis/core";
+import type { BackgroundTaskOrigin, TimerHandle } from "@comis/core";
 export type { BackgroundTaskOrigin };
 export type BackgroundTaskStatus = "running" | "completed" | "failed" | "cancelled";
 /**
@@ -59,7 +59,8 @@ export interface BackgroundTask {
     dispatchState?: BackgroundSessionState;
     _promise?: Promise<unknown>;
     _abortController?: AbortController;
-    _hardTimeoutTimer?: ReturnType<typeof setTimeout>;
+    /** TimerHandle replaces NodeJS.Timeout for cancel-safe shutdown. */
+    _hardTimeoutTimer?: TimerHandle;
 }
 /** Serializable subset of BackgroundTask for file persistence. */
 export interface PersistedTaskState {

package/node_modules/@comis/agent/dist/background/completion-dispatcher.d.ts

@@ -32,7 +32,7 @@
  * @module
  */
 import type { TypedEventBus } from "@comis/core";
-import type { ComisLogger } from "@comis/infra";
+import type { ComisLogger } from "@comis/core";
 import type { BackgroundTask, BackgroundSessionState, BackgroundTaskNotificationPolicy as NotificationPolicyType } from "./background-task-types.js";
 import type { NotifyFn } from "./background-task-manager.js";
 /**

package/node_modules/@comis/agent/dist/background/completion-runner.d.ts

@@ -24,7 +24,7 @@
  * @module
  */
 import type { TypedEventBus } from "@comis/core";
-import type { ComisLogger } from "@comis/infra";
+import type { ComisLogger } from "@comis/core";
 import type { AgentExecutor } from "../executor/types.js";
 import type { BackgroundTaskManager, NotifyFn } from "./background-task-manager.js";
 /** Public-facing handle on the runner returned by createBackgroundCompletionRunner. */

package/node_modules/@comis/agent/dist/background/completion-runner.js

@@ -26,6 +26,7 @@
  */
 import { randomUUID } from "node:crypto";
 import { suppressError } from "@comis/shared";
+import { systemNowMs } from "@comis/core";
 import { parseFormattedSessionKey } from "@comis/core";
 import { formatCompletionAnnouncement } from "./completion-formatter.js";
 /**
@@ -135,7 +136,7 @@ export function createBackgroundCompletionRunner(deps) {
             channelType: "background_task",
             senderId: "background-task-runner",
             text: announcement,
-            timestamp: Date.now(),
+            timestamp: systemNowMs(),
             attachments: [],
             metadata: {
                 backgroundHopCount: nextHopCount,
@@ -166,7 +167,7 @@ export function createBackgroundCompletionRunner(deps) {
             sessionKey: origin.sessionKey,
             hopCount: nextHopCount,
             traceId: origin.traceId ?? null,
-            timestamp: Date.now(),
+            timestamp: systemNowMs(),
         });
         // One turn per event. Existing session lock orders concurrent calls.
         try {

package/node_modules/@comis/agent/dist/background/session-resolver.js

@@ -1,4 +1,5 @@
 // SPDX-License-Identifier: Apache-2.0
+// @allow-throw: session-resolver session-not-found guard; consumed by daemon RPC handlers (subagent-handlers / session-handlers @allow-throw).
 /**
  * BackgroundSessionResolver: composite-key wrapper around ActiveRunRegistry.
  *

package/node_modules/@comis/agent/dist/bootstrap/sections/core-sections.js

@@ -3,6 +3,7 @@
  * Core section builders: identity, safety, date/time, runtime metadata,
  * inbound metadata, and reasoning.
  */
+import { systemNowDate } from "@comis/core";
 // ---------------------------------------------------------------------------
 // 1. Identity (always included)
 // ---------------------------------------------------------------------------
@@ -56,7 +57,7 @@ export function buildLanguageSection(userLanguage) {
 // 14. Date/Time (always included)
 // ---------------------------------------------------------------------------
 export function buildDateTimeSection() {
-    const now = new Date();
+    const now = systemNowDate();
     const isoTimestamp = now.toISOString();
     const localTime = now.toLocaleString("en-US", {
         dateStyle: "full",

package/node_modules/@comis/agent/dist/bootstrap/sections/tool-descriptions.js

@@ -1,4 +1,5 @@
 // SPDX-License-Identifier: Apache-2.0
+// @allow-throw: bootstrap-time invariant assertion (LEAN_TOOL_DESCRIPTIONS / TOOL_SUMMARIES / NATIVE_TOOLS keys must match); consumed at daemon.ts bootstrap catch boundary.
 /**
  * Lean tool descriptions (LEAN_TOOL_DESCRIPTIONS) and summaries (TOOL_SUMMARIES)
  * for the dual-summary description layer.
@@ -18,7 +19,7 @@
  * @module
  */
 import { getToolMetadata } from "@comis/core";
-import { getProviders } from "@mariozechner/pi-ai";
+import { getProviders } from "@earendil-works/pi-ai";
 // ---------------------------------------------------------------------------
 // Live native-catalog provider list
 //

package/node_modules/@comis/agent/dist/bootstrap/sections/tooling-sections.d.ts

@@ -5,35 +5,20 @@
 /** Model size tier — determines prompt verbosity for tool descriptions. */
 export type ModelTier = "small" | "medium" | "large";
 /**
- * Build the static `## Available Tools` block (legacy path, gate-off) OR a
- * single residual one-liner pointing the model at the per-turn `## Capabilities`
- * block (gate-on path).
+ * Build the residual one-liner pointing the model at the per-turn
+ * `## Capabilities` block. The capability-index is now the only path.
  *
- * @param capabilityIndexEnabled - When `true`, emits ONLY the residual
- *   one-liner. When `false` or undefined, emits the legacy flat block
- *   BYTE-IDENTICALLY to the pre-feature baseline. The two paths are MUTUALLY
- *   EXCLUSIVE.
- *
- *   Restart-required: this gate selects between two cached system-prompt
- *   shapes; toggling at runtime is forbidden. Operator-facing constraint is
- *   documented in config docs.
+ * The wording below is normative — the per-turn `## Capabilities` block is
+ * rendered into the dynamic preamble by `executor-prompt-runner.ts`.
  */
-export declare function buildToolingSection(toolNames: string[], _modelTier: ModelTier, toolSummaries?: Record<string, string>, capabilityIndexEnabled?: boolean): string[];
+export declare function buildToolingSection(toolNames: string[], _modelTier: ModelTier, _toolSummaries?: Record<string, string>): string[];
 /**
  * Build the static "## Tool Call Style" section with conditional coding guidelines.
  *
- * @param capabilityIndexEnabled - When `true`, AND when `exec` is in
- *   `toolNames`, the rendered output includes the dual-gated "Tool-first
- *   principle" bullet immediately before the existing Python-virtualenv rule.
- *   When `false` or `undefined`, the bullet is omitted; the existing venv
- *   rule emits unchanged when `exec` is present.
- *
- *   Restart-required: the gate value flows from `tooling.capabilityIndex.enabled`
- *   via `AssemblerParams.capabilityIndexEnabled`, populated at the prompt
- *   assembly site from `port.isCapabilityIndexEnabled()`. The value is
- *   config-derived and stable per session — SAFE inside the cache fence.
+ * The "Tool-first principle" bullet emits unconditionally when `exec` is in
+ * `toolNames` — the capability-index is always on.
  */
-export declare function buildToolCallStyleSection(isMinimal: boolean, toolNames?: string[], capabilityIndexEnabled?: boolean): string[];
+export declare function buildToolCallStyleSection(isMinimal: boolean, toolNames?: string[]): string[];
 /** Tool names that trigger the confirmation flow / self-update gating section. */
 export declare const CONFIRMATION_TOOL_NAMES: string[];
 /**

package/node_modules/@comis/agent/dist/bootstrap/sections/tooling-sections.js

@@ -3,49 +3,22 @@
  * Tooling section builders: tool listing, tool call style, self-update gating,
  * and compacted output recovery.
  */
-import { TOOL_SUMMARIES, TOOL_ORDER } from "./tool-descriptions.js";
-import { getProviders } from "@mariozechner/pi-ai";
+import { getProviders } from "@earendil-works/pi-ai";
 /**
- * Build the static `## Available Tools` block (legacy path, gate-off) OR a
- * single residual one-liner pointing the model at the per-turn `## Capabilities`
- * block (gate-on path).
+ * Build the residual one-liner pointing the model at the per-turn
+ * `## Capabilities` block. The capability-index is now the only path.
  *
- * @param capabilityIndexEnabled - When `true`, emits ONLY the residual
- *   one-liner. When `false` or undefined, emits the legacy flat block
- *   BYTE-IDENTICALLY to the pre-feature baseline. The two paths are MUTUALLY
- *   EXCLUSIVE.
- *
- *   Restart-required: this gate selects between two cached system-prompt
- *   shapes; toggling at runtime is forbidden. Operator-facing constraint is
- *   documented in config docs.
+ * The wording below is normative — the per-turn `## Capabilities` block is
+ * rendered into the dynamic preamble by `executor-prompt-runner.ts`.
  */
-export function buildToolingSection(toolNames, _modelTier, toolSummaries, capabilityIndexEnabled) {
+export function buildToolingSection(toolNames, _modelTier, _toolSummaries) {
     if (toolNames.length === 0)
         return [];
-    // Gate-on path: residual one-liner only. The per-turn `## Capabilities`
-    // block is rendered into the dynamic preamble by `executor-prompt-runner.ts`.
-    // The wording below is normative.
-    if (capabilityIndexEnabled === true) {
-        return [
-            "When this turn includes a `Capabilities` context, refer to it for grouped tool guidance " +
-                "before invoking tools or running installs. Tool schemas in your active toolspace are " +
-                "authoritative for parameter shapes.",
-        ];
-    }
-    // Gate-off path: legacy flat block BYTE-IDENTICAL to pre-feature baseline.
-    // DO NOT modify the body below — the byte-identity assertion in
-    // tooling-sections.test.ts depends on this shape staying exact.
-    const summaries = { ...TOOL_SUMMARIES, ...toolSummaries };
-    const ordered = TOOL_ORDER.filter((t) => toolNames.includes(t));
-    const extras = toolNames.filter((t) => !TOOL_ORDER.includes(t)).sort();
-    const allTools = [...ordered, ...extras];
-    const lines = ["## Available Tools"];
-    for (const name of allTools) {
-        const desc = summaries[name];
-        lines.push(desc ? `- ${name}: ${desc}` : `- ${name}`);
-    }
-    lines.push("", "Always use tools to gather real data before answering. Never guess or fabricate tool results.", "Refer to each tool's schema for full parameter details.");
-    return lines;
+    return [
+        "When this turn includes a `Capabilities` context, refer to it for grouped tool guidance " +
+            "before invoking tools or running installs. Tool schemas in your active toolspace are " +
+            "authoritative for parameter shapes.",
+    ];
 }
 // ---------------------------------------------------------------------------
 // 4. Tool Call Style (skip if minimal)
@@ -53,18 +26,10 @@ export function buildToolingSection(toolNames, _modelTier, toolSummaries, capabi
 /**
  * Build the static "## Tool Call Style" section with conditional coding guidelines.
  *
- * @param capabilityIndexEnabled - When `true`, AND when `exec` is in
- *   `toolNames`, the rendered output includes the dual-gated "Tool-first
- *   principle" bullet immediately before the existing Python-virtualenv rule.
- *   When `false` or `undefined`, the bullet is omitted; the existing venv
- *   rule emits unchanged when `exec` is present.
- *
- *   Restart-required: the gate value flows from `tooling.capabilityIndex.enabled`
- *   via `AssemblerParams.capabilityIndexEnabled`, populated at the prompt
- *   assembly site from `port.isCapabilityIndexEnabled()`. The value is
- *   config-derived and stable per session — SAFE inside the cache fence.
+ * The "Tool-first principle" bullet emits unconditionally when `exec` is in
+ * `toolNames` — the capability-index is always on.
  */
-export function buildToolCallStyleSection(isMinimal, toolNames = [], capabilityIndexEnabled) {
+export function buildToolCallStyleSection(isMinimal, toolNames = []) {
     if (isMinimal)
         return [];
     const lines = [
@@ -105,14 +70,10 @@ export function buildToolCallStyleSection(isMinimal, toolNames = [], capabilityI
         guidelines.push("- Show file paths clearly when working with files.");
     }
     if (has("exec")) {
-        // Dual-gated counterweight to the venv rule below. The bullet precedes
-        // the venv rule because the first read sets the default; the second is
-        // the install fallback. Restart-required: capabilityIndexEnabled flows
-        // from `tooling.capabilityIndex.enabled` via AssemblerParams →
-        // SECTIONS["tool-call-style"].
-        if (capabilityIndexEnabled === true) {
-            guidelines.push("- **Tool-first principle.** When this turn includes a `Capabilities` context and the task can be satisfied by a connected tool or available skill, prefer that capability over installing a Python or Node package. Use installs only for capabilities not covered by active tools, deferred tools, or visible prompt skills.");
-        }
+        // Counterweight to the venv rule below. The bullet precedes the venv rule
+        // because the first read sets the default; the second is the install
+        // fallback. Capability-index is always on.
+        guidelines.push("- **Tool-first principle.** When this turn includes a `Capabilities` context and the task can be satisfied by a connected tool or available skill, prefer that capability over installing a Python or Node package. Use installs only for capabilities not covered by active tools, deferred tools, or visible prompt skills.");
         guidelines.push("- **Python projects:** Always create a virtualenv per project (`python3 -m venv .venv`). "
             + "Install packages into the project venv (`source .venv/bin/activate && pip install ...`). "
             + "Never use `--break-system-packages` — it pollutes the system Python. "

package/node_modules/@comis/agent/dist/bootstrap/system-prompt-assembler.d.ts

@@ -79,28 +79,6 @@ export interface AssemblerParams {
     excludeBootstrapFromContext?: boolean;
     /** Workspace profile controlling platform instruction verbosity ('full' or 'specialist'). */
     workspaceProfile?: "full" | "specialist";
-    /**
-     * Capability index gate.
-     *
-     * When `true`, `buildToolingSection` emits the residual one-liner instead
-     * of the legacy `## Available Tools` flat block; the per-turn
-     * `## Capabilities` block is rendered into the dynamic preamble in
-     * `executor-prompt-runner.ts`.
-     * When `false` or `undefined`, the static prompt is byte-identical to
-     * the pre-feature baseline.
-     *
-     * RESTART-REQUIRED: this flag selects between two cached system-prompt
-     * shapes. Daemon wiring documents the operator-facing constraint.
-     *
-     * SAFE INSIDE THE CACHE FENCE: this value is config-derived
-     * (operator-only, restart-required) and stable across all turns of a
-     * session. Plumbing it through `assemblerParams` does NOT create a
-     * cache-thrash regression. Live-runtime accessors
-     * (getPromptSkillCapabilities, getConnectedMcpServers) are forbidden
-     * inside this interface and enforced by an architecture-grep test.
-     * Adding a config-derived flag to the cache fence is safe.
-     */
-    capabilityIndexEnabled?: boolean;
     /** Whether Silent Execution Planner (SEP) is enabled for this agent. */
     sepEnabled?: boolean;
 }