@agentprojectcontext/apx 1.25.0 → 1.27.1

package/src/{daemon → host/daemon}/routines.js

@@ -15,11 +15,13 @@ import { execFile } from "node:child_process";
 import os from "node:os";
 import fs from "node:fs";
 import path from "node:path";
-import { callEngine } from "./engines/index.js";
+import { callEngine } from "../../core/engines/index.js";
 import { runSuperAgent } from "./super-agent.js";
-import { readAgents } from "../core/parser.js";
-import { buildAgentSystem } from "../core/agent-system.js";
-import { resolveArtifactRef, ARTIFACTS_SKIP_SIGNAL } from "../core/artifacts-store.js";
+import { computeSuppressedTools } from "../../core/agent/index.js";
+import { readAgents } from "../../core/parser.js";
+import { buildAgentSystem } from "../../core/agent-system.js";
+import { resolveAgentName, SUPERAGENT_ACTOR_ID } from "../../core/identity.js";
+import { resolveArtifactRef, ARTIFACTS_SKIP_SIGNAL } from "../../core/artifacts-store.js";
 import {
   listRoutines,
   getRoutine,
@@ -30,7 +32,7 @@ import {
   getDueRoutines,
   parseSchedule,
   computeNextRun,
-} from "../core/routines-store.js";
+} from "../../core/routines-store.js";
 
 export {
   listRoutines,
@@ -91,6 +93,7 @@ async function handleExecAgent(ctx, routine) {
     direction: "out",
     type: "agent",
     actor_id: slug,
+    actor_kind: "agent",
     author: slug,
     body: result.text,
     meta: { routine: routine.name, usage: result.usage },
@@ -110,21 +113,37 @@ async function handleSuperAgent(ctx, routine) {
     ...(Array.isArray(routine.allowed_tools) ? { allowed_tools: routine.allowed_tools } : {}),
   };
 
+  // Auto-suppress tools whose output would duplicate post_commands.
+  // Example: a routine with `apx telegram send "$APX_LLM_OUTPUT"` in post_commands
+  // shouldn't also let the agent call send_telegram inside the loop.
+  // See spec/backlog/01-routine-output-coherence.md.
+  const autoSuppress = computeSuppressedTools(routine.post_commands);
+  const explicitSuppress = Array.isArray(routine.spec?.suppress_tools)
+    ? routine.spec.suppress_tools.filter((s) => typeof s === "string")
+    : [];
+  const suppressTools = [...new Set([...autoSuppress, ...explicitSuppress])];
+
   const result = await runSuperAgent({
     globalConfig: cfg,
     projects,
     plugins,
     registries,
     prompt,
-    contextNote: `You were invoked by APX routine "${routine.name}" in project ${project.path}. This is an autonomous scheduled run, not an interactive Telegram reply.`,
+    channel: "routine",
+    channelMeta: {
+      routineName: routine.name,
+      projectPath: project.path,
+    },
+    suppressTools: suppressTools.length > 0 ? suppressTools : null,
   });
 
   project.logMessage({
     channel: "routine",
     direction: "out",
     type: "agent",
-    actor_id: result.name || "super_agent",
-    author: result.name || "super_agent",
+    actor_id: SUPERAGENT_ACTOR_ID,
+    actor_kind: "superagent",
+    author: result.name || resolveAgentName(globalConfig),
     body: result.text || "",
     meta: { routine: routine.name, tool_trace: result.trace, usage: result.usage },
   });

package/src/{daemon → host/daemon}/skills-loader.js

@@ -21,7 +21,7 @@
 // but the bundled copy stays in the package as a safety net.
 //
 // Note: the bundled `apc-context` skill is REFRESHED from the canonical apc
-// repo on every npm install / update (see src/cli/postinstall.js). APC is a
+// repo on every npm install / update (see src/interfaces/cli/postinstall.js). APC is a
 // living standard, so its skill content is not pinned to an apx version.
 
 import fs from "node:fs";
@@ -31,7 +31,9 @@ import { fileURLToPath } from "node:url";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname  = path.dirname(__filename);
-const PACKAGE_ROOT = path.resolve(__dirname, "..", "..");
+// Three levels up: __dirname = src/host/daemon/ → src/host/ → src/ → repo root.
+// Used to find the bundled skills/ folder at the repo root.
+const PACKAGE_ROOT = path.resolve(__dirname, "..", "..", "..");
 
 const RUNTIME_SKILLS_DIR = path.join(PACKAGE_ROOT, "src", "core", "runtime-skills");
 const BUNDLED_SKILLS_DIR = path.join(PACKAGE_ROOT, "skills");

package/src/{daemon → host/daemon}/smoke.js

@@ -8,15 +8,19 @@ import path from "node:path";
 import fs from "node:fs";
 import { fileURLToPath } from "node:url";
 import { ProjectManager } from "./db.js";
-import { McpRegistry } from "./mcp-runner.js";
-import { readAgents } from "../core/parser.js";
+import { McpRegistry } from "../../core/mcp/runner.js";
+import { readAgents } from "../../core/parser.js";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
+// Smoke now lives at src/host/daemon/smoke.js — one level deeper than the
+// legacy src/daemon/smoke.js. Adjust the example-project lookup accordingly:
+//   ../../../examples/my-first-project           ← repo-local examples (if any)
+//   ../../../../apc/examples/my-first-project    ← sibling apc/ repo
 const EXAMPLE_CANDIDATES = [
-  path.resolve(__dirname, "..", "..", "examples", "my-first-project"),
-  path.resolve(__dirname, "..", "..", "..", "apc", "examples", "my-first-project"),
+  path.resolve(__dirname, "..", "..", "..", "examples", "my-first-project"),
+  path.resolve(__dirname, "..", "..", "..", "..", "apc", "examples", "my-first-project"),
 ];
 const EXAMPLE = EXAMPLE_CANDIDATES.find((p) =>
   fs.existsSync(path.join(p, "AGENTS.md")) &&
@@ -49,7 +53,7 @@ const sofiaSessions = (() => {
 })();
 console.log("sofia sessions:", sofiaSessions);
 
-const reg = new McpRegistry(entry.path);
+const reg = new McpRegistry({ projectPath: entry.path, storagePath: entry.storagePath });
 const list = reg.list();
 console.log("mcps:", list.map((m) => `${m.name} (${m.source})`));
 assert(list.find((m) => m.name === "filesystem" && m.source === "apc"), "filesystem MCP missing");

package/src/{daemon → host/daemon}/super-agent-tools/helpers.js

@@ -1,5 +1,5 @@
 import path from "node:path";
-import { agentSkills, buildAgentSystem as buildCoreAgentSystem } from "../../core/agent-system.js";
+import { agentSkills, buildAgentSystem as buildCoreAgentSystem } from "../../../core/agent-system.js";
 
 export function projectMeta(projects, entry) {
   const meta = projects.list().find((p) => p.id === entry.id);

package/src/host/daemon/super-agent-tools/index.js

@@ -0,0 +1,157 @@
+import listProjects from "./tools/list-projects.js";
+import listAgents from "./tools/list-agents.js";
+import listVaultAgents from "./tools/list-vault-agents.js";
+import importAgent from "./tools/import-agent.js";
+import addProject from "./tools/add-project.js";
+import listMcps from "./tools/list-mcps.js";
+import readAgentMemory from "./tools/read-agent-memory.js";
+import remember from "./tools/remember.js";
+import readSelfMemory from "./tools/read-self-memory.js";
+import listFiles from "./tools/list-files.js";
+import readFile from "./tools/read-file.js";
+import writeFile from "./tools/write-file.js";
+import editFile from "./tools/edit-file.js";
+import runShell from "./tools/run-shell.js";
+import tailMessages from "./tools/tail-messages.js";
+import searchMessages from "./tools/search-messages.js";
+import searchSessions from "./tools/search-sessions.js";
+import callAgent from "./tools/call-agent.js";
+import callMcp from "./tools/call-mcp.js";
+import callRuntime from "./tools/call-runtime.js";
+import sendTelegram from "./tools/send-telegram.js";
+import setIdentity from "./tools/set-identity.js";
+import setPermissionMode from "./tools/set-permission-mode.js";
+import searchFiles from "./tools/search-files.js";
+import listSkills from "./tools/list-skills.js";
+import loadSkill from "./tools/load-skill.js";
+import transcribeAudio from "./tools/transcribe-audio.js";
+import askQuestions from "./tools/ask-questions.js";
+import createTask from "./tools/create-task.js";
+import listTasks from "./tools/list-tasks.js";
+import { createPermissionGuard } from "./helpers.js";
+import { buildBridgedTools, DEFAULT_CATEGORIES } from "./registry-bridge.js";
+
+const NATIVE_TOOLS = [
+  listProjects,
+  listAgents,
+  listVaultAgents,
+  importAgent,
+  addProject,
+  listMcps,
+  readAgentMemory,
+  remember,
+  readSelfMemory,
+  listFiles,
+  readFile,
+  writeFile,
+  editFile,
+  runShell,
+  tailMessages,
+  searchMessages,
+  searchSessions,
+  callAgent,
+  callMcp,
+  callRuntime,
+  sendTelegram,
+  setIdentity,
+  setPermissionMode,
+  searchFiles,
+  listSkills,
+  loadSkill,
+  transcribeAudio,
+  askQuestions,
+  createTask,
+  listTasks,
+];
+
+// Registry-backed bridges. Categories can be overridden per-process via env
+// APX_BRIDGE_CATEGORIES (comma-separated), e.g. "browser,fetch,search".
+// Default: browser, fetch, search, glob, grep (see registry-bridge.js).
+function resolveBridgeCategories() {
+  const env = (process.env.APX_BRIDGE_CATEGORIES || "").trim();
+  if (!env) return DEFAULT_CATEGORIES;
+  return new Set(env.split(",").map(s => s.trim()).filter(Boolean));
+}
+
+const BRIDGED_TOOLS = buildBridgedTools({ categories: resolveBridgeCategories() });
+const TOOLS = [...NATIVE_TOOLS, ...BRIDGED_TOOLS];
+
+export const TOOL_SCHEMAS = TOOLS.map((tool) => tool.schema);
+
+// "Core" tools always sent to the model. The rest are pulled in on-demand via
+// load_skill or by switching to a heavier channel. Picked to fit cheap cloud
+// tiers: full TOOL_SCHEMAS is ~22 KB / ~5.5 K tokens — too much when Groq
+// free tier caps you at 6-12 K TPM. CORE_TOOL_NAMES is ~3 KB / ~700 tokens.
+// See spec/done/backlog item 12 for the underlying motivation.
+const CORE_TOOL_NAMES = new Set([
+  // Inventory — the model NEEDS to call these to know what's there.
+  "list_projects",
+  "list_agents",
+  "list_mcps",
+  "list_skills",
+  // Memory + identity — used during identity / config conversations.
+  "read_agent_memory",
+  "set_identity",
+  // Self-memory: jot durable facts so they survive across sessions.
+  "remember",
+  // Self-recall: "what did we do / last session" must work on every channel.
+  "search_sessions",
+  // Conversation control.
+  "ask_questions",
+  // On-demand expansion: this is how the model loads the rest of the surface.
+  "load_skill",
+  // Channels the user expects out of any super-agent turn.
+  "send_telegram",
+  // Lightweight delegation (no spawn).
+  "call_agent",
+  // Routine creation (very common ask via chat).
+  "create_task",
+  "list_tasks",
+]);
+
+export const CORE_TOOL_SCHEMAS = TOOLS
+  .filter((t) => CORE_TOOL_NAMES.has(t.name))
+  .map((t) => t.schema);
+
+/**
+ * Choose the tool schema list for a given channel. Telegram / desktop / api
+ * (chit-chat) get the "core" subset to stay under cheap-tier TPM limits;
+ * routines get the full list because they're deliberate, scheduled, and the
+ * user has chosen the model. Override with the explicit `full: true` opt.
+ */
+export function schemasForChannel(channel, { full = false } = {}) {
+  if (full) return TOOL_SCHEMAS;
+  // Full registry for deliberate, local surfaces running on a user-picked model
+  // (not subject to the cheap-tier TPM caps that motivate the "core" subset):
+  //   routine — scheduled/autonomous · api — generic HTTP / `apx exec`
+  //   web     — the big web chat (long-form workspace)
+  //   code    — the web Code module (needs read/write/edit/run_shell/grep/glob)
+  //   terminal — the `apx code`/`apx sys` TUI: same coding surface as web Code,
+  //              so it needs the full read/write/edit/run_shell registry too.
+  if (
+    channel === "routine" ||
+    channel === "api" ||
+    channel === "web" ||
+    channel === "code" ||
+    channel === "terminal"
+  )
+    return TOOL_SCHEMAS;
+  // Lightweight surfaces stay on the small subset to fit cheap cloud TPM limits
+  // and keep replies snappy: telegram, web_sidebar, deck, desktop.
+  return CORE_TOOL_SCHEMAS;
+}
+
+export function makeToolHandlers(ctx) {
+  const toolCtx = {
+    ...ctx,
+    requirePermission: createPermissionGuard(ctx.globalConfig || {}, {
+      implicitConfirmation: !!ctx.implicitConfirmation,
+    }),
+  };
+  return Object.fromEntries(TOOLS.map((tool) => [tool.name, tool.makeHandler(toolCtx)]));
+}
+
+// Diagnostic helper — useful for `apx daemon status` or debug logging.
+export function listBridgedToolNames() {
+  return BRIDGED_TOOLS.map(t => t.name);
+}

package/src/{daemon → host/daemon}/super-agent-tools/registry-bridge.js

@@ -18,7 +18,7 @@
 // Net result: adding a tool = adding one entry to registry.js. No file in
 // super-agent-tools/tools/, no import in index.js.
 
-import { TOOL_DEFINITIONS } from "../tools/registry.js";
+import { TOOL_DEFINITIONS } from "../../../core/tools/registry.js";
 
 // Native handlers in super-agent-tools/tools/ that own these names. The bridge
 // MUST skip them or the registry version (HTTP roundtrip) would shadow the

package/src/{daemon → host/daemon}/super-agent-tools/tools/add-project.js

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
-import { readConfig, addProject as addProjectInConfig } from "../../../core/config.js";
-import { initApf } from "../../../core/scaffold.js";
+import { readConfig, addProject as addProjectInConfig } from "../../../../core/config.js";
+import { initApf } from "../../../../core/scaffold.js";
 import { confirmedProperty, projectMeta } from "../helpers.js";
 
 function isApcProject(absPath) {

package/src/{daemon → host/daemon}/super-agent-tools/tools/ask-questions.js

@@ -1,6 +1,10 @@
 export default {
   name: "ask_questions",
   schema: {
+    // type: "function" is REQUIRED at the top level — OpenAI and Groq reject
+    // schemas without it (Groq → 400 'tools.N.type': property 'type' is missing).
+    // Anthropic / Ollama tolerate its absence but the contract is clearer with it.
+    type: "function",
     function: {
       name: "ask_questions",
       description: "Ask the user one or more specific questions to clarify the task or gather requirements.",

package/src/{daemon → host/daemon}/super-agent-tools/tools/call-agent.js

@@ -1,5 +1,5 @@
-import { callEngine } from "../../engines/index.js";
-import { readAgents } from "../../../core/parser.js";
+import { callEngine } from "../../../../core/engines/index.js";
+import { readAgents } from "../../../../core/parser.js";
 import { buildAgentSystem, resolveProject } from "../helpers.js";
 
 export default {
@@ -39,6 +39,9 @@ export default {
       agent_slug: slug,
       channel: "engine",
       direction: "out",
+      type: "agent",
+      actor_id: slug,
+      actor_kind: "agent",
       author: slug,
       body: result.text,
       meta: { invoked_by: "super_agent_tool", usage: result.usage },

package/src/{daemon → host/daemon}/super-agent-tools/tools/call-runtime.js

@@ -1,6 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
-import { readAgents } from "../../../core/parser.js";
+import { loggerFor } from "../../../../core/logging.js";
+import { readAgents } from "../../../../core/parser.js";
 import {
   buildApfHint,
   closeRuntimeSession,
@@ -8,10 +9,65 @@ import {
   extractApfResult,
 } from "../../apc-runtime-context.js";
 import { detectAll } from "../../env-detect.js";
+import {
+  findEngineSessionById,
+  readEngineSessionContext,
+} from "../../engine-sessions.js";
 import { runProcess } from "../../runtimes/_spawn.js";
 import { getRuntime, RUNTIME_IDS } from "../../runtimes/index.js";
 import { buildAgentSystem, confirmedProperty, resolveProject } from "../helpers.js";
 
+const log = loggerFor("call_runtime");
+
+// Decide if the runtime actually did the work the model asked for. A spawn
+// failure (-1), a non-zero exit, or a clean exit with no captured output and
+// no transcript path all point at "process didn't really run" — exactly the
+// false-positive scenario this guard exists to catch.
+function runtimeLooksLikeFailure(r) {
+  if (!r) return { failed: true, reason: "no runtime result" };
+  if (r.exitCode === -1 || r.error) {
+    return { failed: true, reason: r.error || "spawn error" };
+  }
+  if (typeof r.exitCode === "number" && r.exitCode !== 0) {
+    return { failed: true, reason: `exit ${r.exitCode}` };
+  }
+  if (r.killed) return { failed: true, reason: "killed (timeout)" };
+  const out = String(r.output || "").trim();
+  const stderr = String(r.stderr || "").trim();
+  if (!out && !r.externalSessionPath && !r.sessionId) {
+    return {
+      failed: true,
+      reason: stderr ? `empty output (stderr: ${stderr.slice(0, 120)})` : "empty output",
+    };
+  }
+  return { failed: false };
+}
+
+// If the model passed resume_session_id, look the session up (claude / codex /
+// apx) and prepend a tiny context block so the runtime knows what the prior
+// session was about. The model's prompt still wins; this is preamble only.
+function buildResumePreamble(sessionId) {
+  if (!sessionId) return { text: "", meta: null };
+  let meta;
+  try {
+    meta = findEngineSessionById(sessionId);
+  } catch {
+    return { text: "", meta: null };
+  }
+  if (!meta) return { text: "", meta: null };
+  const ctx = readEngineSessionContext(meta);
+  const title = ctx?.title || meta.title || null;
+  const lastPrompt = ctx?.lastPrompt || null;
+  if (!title && !lastPrompt) return { text: "", meta };
+  const parts = [
+    `## Context from prior session (${meta.engine}:${meta.id})`,
+  ];
+  if (title) parts.push(`Title: ${title}`);
+  if (lastPrompt) parts.push(`Last prompt: ${lastPrompt}`);
+  parts.push("---");
+  return { text: parts.join("\n") + "\n\n", meta };
+}
+
 function resolveProjectForAgent(projects, project, slug) {
   if (project) return resolveProject(projects, project);
 
@@ -106,6 +162,10 @@ export default {
             description: "external CLI runtime",
           },
           prompt: { type: "string" },
+          resume_session_id: {
+            type: "string",
+            description: "Optional prior session id (claude/codex/apx) — APX prepends that session's title + last prompt to the prompt so the runtime has context.",
+          },
           timeout_s: { type: "integer", description: "seconds before SIGTERM; default 300" },
           confirmed: confirmedProperty("true only after explicit user confirmation for this exact runtime command"),
         },
@@ -113,7 +173,7 @@ export default {
       },
     },
   },
-  makeHandler: ({ projects, requirePermission }) => async ({ project, agent: slug, runtime, prompt, timeout_s = 300, confirmed = false }) => {
+  makeHandler: ({ projects, requirePermission }) => async ({ project, agent: slug, runtime, prompt, resume_session_id = null, timeout_s = 300, confirmed = false }) => {
     requirePermission("call_runtime", { dangerous: true, confirmed });
 
     const p = slug ? resolveProjectForAgent(projects, project, slug) : resolveProject(projects, project);
@@ -157,20 +217,33 @@ export default {
       title: `Runtime: ${runtime}${agent ? ` (${agent.slug})` : ""}`,
     });
 
+    const resume = buildResumePreamble(resume_session_id);
+    const effectivePrompt = resume.text ? resume.text + prompt : prompt;
+
+    log.info(`spawn ${runtime}`, {
+      apc_session: session.id,
+      agent: actor,
+      project: p.path,
+      resume_session_id: resume_session_id || null,
+      resume_resolved: resume.meta ? `${resume.meta.engine}:${resume.meta.id}` : null,
+      timeout_s,
+    });
+
     try {
       const r = await rt.run({
         system: buildRuntimeSystem(p, agent, runtime, session.id, "super_agent_tool"),
-        prompt,
+        prompt: effectivePrompt,
         cwd: p.path,
         timeoutMs: timeout_s * 1000,
       });
 
+      const failure = runtimeLooksLikeFailure(r);
       const result = extractApfResult(r.output) || (r.output || "").slice(0, 200);
       closeRuntimeSession({
         filePath: session.path,
         externalSessionPath: r.externalSessionPath || null,
-        exitCode: r.exitCode,
-        result,
+        exitCode: failure.failed && r.exitCode === 0 ? -1 : r.exitCode,
+        result: failure.failed ? `failed: ${failure.reason}` : result,
       });
 
       p.logMessage({
@@ -178,14 +251,17 @@ export default {
         channel: "runtime",
         direction: "in",
         author: "user",
-        body: prompt,
-        meta: { runtime, invoked_by: "super_agent_tool", apc_session: session.id },
+        body: effectivePrompt,
+        meta: { runtime, invoked_by: "super_agent_tool", apc_session: session.id, resume_session_id: resume_session_id || null },
       });
       p.logMessage({
         agent_slug: actor,
         channel: "runtime",
         direction: "out",
-        author: actor,
+        type: "agent",
+        actor_id: agent?.slug || runtime,
+        actor_kind: agent?.slug ? "agent" : "engine",
+        author: agent?.slug || runtime,
         body: r.output || "",
         meta: {
           runtime,
@@ -194,9 +270,39 @@ export default {
           session_id: r.sessionId || null,
           apc_session: session.id,
           invoked_by: "super_agent_tool",
+          failed: failure.failed || false,
+          failure_reason: failure.failed ? failure.reason : null,
         },
       });
 
+      if (failure.failed) {
+        log.error(`${runtime} run failed: ${failure.reason}`, {
+          apc_session: session.id,
+          exit_code: r.exitCode,
+          stderr: String(r.stderr || "").slice(0, 500),
+          external_session_path: r.externalSessionPath || null,
+        });
+        return {
+          error: `runtime "${runtime}" did not complete successfully: ${failure.reason}`,
+          runtime,
+          agent: agent?.slug || null,
+          apc_session: session.id,
+          exit_code: r.exitCode,
+          stderr: (r.stderr || "").slice(0, 2000),
+          output: (r.output || "").slice(0, 2000),
+          external_session_path: r.externalSessionPath || null,
+          session_id: r.sessionId || null,
+        };
+      }
+
+      log.info(`${runtime} run ok`, {
+        apc_session: session.id,
+        exit_code: r.exitCode,
+        external_session_path: r.externalSessionPath || null,
+        session_id: r.sessionId || null,
+        output_bytes: (r.output || "").length,
+      });
+
       return {
         runtime,
         agent: agent?.slug || null,
@@ -209,6 +315,9 @@ export default {
         session_id: r.sessionId || null,
       };
     } catch (e) {
+      log.error(`${runtime} run threw: ${e.message}`, {
+        apc_session: session.id,
+      });
       try {
         closeRuntimeSession({
           filePath: session.path,

package/src/host/daemon/super-agent-tools/tools/create-task.js

@@ -0,0 +1,52 @@
+import { createTask } from "../../../../core/tasks-store.js";
+
+export default {
+  name: "create_task",
+  schema: {
+    type: "function",
+    function: {
+      name: "create_task",
+      description:
+        "Create a TODO task in a project. Use when the user asks you to remember something to do later, take note of a follow-up, or track a pending item. Project resolves by id, name or absolute path — call list_projects first if unsure.",
+      parameters: {
+        type: "object",
+        required: ["project", "title"],
+        properties: {
+          project: { type: "string", description: "Project id, name, or path." },
+          title:   { type: "string", description: "Short imperative title for the task." },
+          body:    { type: "string", description: "Optional longer description." },
+          tags:    { type: "array", items: { type: "string" }, description: "Optional tags." },
+          due:     { type: "string", description: "Optional ISO date (YYYY-MM-DD) the task is due by." },
+          agent:   { type: "string", description: "Optional agent slug responsible for the task." },
+          source:  { type: "string", description: "Where the task came from (telegram, desktop, …). Defaults to the calling channel." },
+        },
+      },
+    },
+  },
+  makeHandler: ({ projects }) => async ({ project: ref, title, body, tags, due, agent, source }) => {
+    if (!ref) return { error: "project required" };
+    if (!title) return { error: "title required" };
+    const all = projects.list();
+    const r = String(ref);
+    const found = all.find((p) =>
+      String(p.id) === r || p.name === r || p.path === r
+    );
+    if (!found) return { error: `project not found: ${ref}` };
+    const proj = projects.get(found.id);
+    if (!proj) return { error: `project storage not loaded: ${ref}` };
+    const task = createTask(proj.storagePath, {
+      title,
+      body: body || null,
+      tags: Array.isArray(tags) ? tags : [],
+      due: due || null,
+      agent: agent || null,
+      source: source || "super-agent",
+    });
+    return {
+      id: task.id,
+      project: { id: proj.id, name: proj.name },
+      title: task.title,
+      state: task.state,
+    };
+  },
+};

package/src/{daemon → host/daemon}/super-agent-tools/tools/import-agent.js

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
-import { readVaultAgents, VAULT_DIR } from "../../../core/parser.js";
-import { addImportedAgent, ensureAgentDir, regenerateAgentsMd } from "../../../core/scaffold.js";
+import { readVaultAgents, VAULT_DIR } from "../../../../core/parser.js";
+import { addImportedAgent, ensureAgentDir } from "../../../../core/scaffold.js";
 import { confirmedProperty, projectMeta, resolveProject } from "../helpers.js";
 
 export default {
@@ -35,7 +35,6 @@ export default {
     const p = resolveProject(projects, project || "default");
     addImportedAgent(p.path, slug);
     ensureAgentDir(p.path, slug);
-    regenerateAgentsMd(p.path);
     projects.rebuild(p.id);
 
     return {

package/src/{daemon → host/daemon}/super-agent-tools/tools/list-agents.js

@@ -1,4 +1,4 @@
-import { readAgents } from "../../../core/parser.js";
+import { readAgents } from "../../../../core/parser.js";
 import { agentRow, resolveProject } from "../helpers.js";
 
 export default {