@agentprojectcontext/apx 1.11.0 → 1.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentprojectcontext/apx",
-  "version": "1.11.0",
+  "version": "1.12.0",
   "description": "APX — unified CLI + daemon for the Agent Project Context (APC) standard.",
   "publishConfig": {
     "access": "public"

package/src/cli/commands/sys.js

@@ -359,9 +359,14 @@ async function runPrompt(pid, state, previousMessages, renderScreen, text, userI
   const startTime = Date.now();
 
   try {
+    const cwd = process.cwd();
     const body = {
       prompt: `[Mode: ${MODES[state.currentModeIdx]}]\n${text}`,
-      contextNote: "Channel: terminal. Format freely using markdown, but keep it readable. Use code diffs when editing.",
+      contextNote: [
+        "Channel: terminal. Format freely using markdown, but keep it readable. Use code diffs when editing.",
+        `CWD: ${cwd}`,
+        "When the user says \"este directorio\", \"este proyecto\", \"acá\", \"aquí\", \"this directory\", \"current dir\" or any equivalent reference without naming a path, they mean exactly the CWD above. Use it as the path argument directly — don't ask the user to provide it.",
+      ].join("\n"),
       previousMessages,
       model: state.activeModel,
     };

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

@@ -20,8 +20,9 @@ import setIdentity from "./tools/set-identity.js";
 import setPermissionMode from "./tools/set-permission-mode.js";
 import searchFiles from "./tools/search-files.js";
 import { createPermissionGuard } from "./helpers.js";
+import { buildBridgedTools, DEFAULT_CATEGORIES } from "./registry-bridge.js";
 
-const TOOLS = [
+const NATIVE_TOOLS = [
   listProjects,
   listAgents,
   listVaultAgents,
@@ -45,6 +46,18 @@ const TOOLS = [
   searchFiles,
 ];
 
+// 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);
 
 export function makeToolHandlers(ctx) {
@@ -56,3 +69,8 @@ export function makeToolHandlers(ctx) {
   };
   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/super-agent-tools/registry-bridge.js

@@ -0,0 +1,122 @@
+// daemon/super-agent-tools/registry-bridge.js
+//
+// Generic bridge that exposes registry-backed HTTP tools (browser, fetch,
+// search, glob, grep, etc.) to the super-agent — no per-tool import boilerplate.
+//
+// How it works:
+//   1. Read TOOL_DEFINITIONS from daemon/tools/registry.js
+//   2. Drop entries whose names collide with native super-agent tools (those
+//      win — they touch in-process state directly).
+//   3. For each remaining entry, produce { name, schema, makeHandler } in the
+//      exact shape index.js expects, so they slot into TOOL_SCHEMAS alongside
+//      the native ones.
+//   4. The generated handler POSTs/GETs to the daemon's own HTTP server on
+//      127.0.0.1:<port>. Yes, the super-agent talks to its own daemon — that
+//      keeps the bridge dead-simple, lets the engine adapter format tool
+//      schemas uniformly, and reuses the exact code path external callers hit.
+//
+// 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";
+
+// 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
+// native one with possibly different semantics.
+const NATIVE_NAMES = new Set([
+  "list_projects", "list_agents", "list_vault_agents", "import_agent",
+  "add_project", "list_mcps", "read_agent_memory",
+  "list_files", "read_file", "write_file", "edit_file", "search_files",
+  "run_shell", "tail_messages", "search_messages",
+  "call_agent", "call_mcp", "call_runtime",
+  "send_telegram", "set_identity", "set_permission_mode",
+]);
+
+// Default allow-list of categories the bridge will expose. The NATIVE_NAMES
+// filter handles duplicates inside these categories (e.g. "file" contains
+// both read_file [native] and glob [bridged]). Anything outside is ignored
+// — "shell"/"mcp"/"memory"/"session" have different semantics handled
+// natively. Override with env APX_BRIDGE_CATEGORIES.
+const DEFAULT_CATEGORIES = new Set(["browser", "fetch", "search", "file"]);
+
+function buildSchema(entry) {
+  return {
+    type: "function",
+    function: {
+      name: entry.name,
+      description: entry.description,
+      parameters: entry.parameters || { type: "object", properties: {} },
+    },
+  };
+}
+
+function buildHandler(entry) {
+  return ({ globalConfig }) => async (args = {}) => {
+    const port = globalConfig?.port || process.env.APX_PORT || 7430;
+    const method = String(entry.endpoint?.method || "POST").toUpperCase();
+    let url = `http://127.0.0.1:${port}${entry.endpoint?.path || ""}`;
+
+    const opts = {
+      method,
+      headers: { "content-type": "application/json" },
+    };
+
+    if (method === "GET" || method === "HEAD") {
+      const qs = new URLSearchParams();
+      for (const [k, v] of Object.entries(args)) {
+        if (v === undefined || v === null) continue;
+        qs.set(k, typeof v === "object" ? JSON.stringify(v) : String(v));
+      }
+      const q = qs.toString();
+      if (q) url += (url.includes("?") ? "&" : "?") + q;
+    } else {
+      opts.body = JSON.stringify(args);
+    }
+
+    let res, text;
+    try {
+      res = await fetch(url, opts);
+      text = await res.text();
+    } catch (e) {
+      return { error: `bridge fetch failed: ${e.message}`, url };
+    }
+
+    let parsed;
+    try { parsed = JSON.parse(text); }
+    catch { parsed = { raw: text }; }
+
+    if (!res.ok) {
+      return {
+        error: parsed?.error || `HTTP ${res.status}`,
+        status: res.status,
+        ...(typeof parsed === "object" ? parsed : {}),
+      };
+    }
+    return parsed;
+  };
+}
+
+/**
+ * Returns an array of tool objects in the shape super-agent-tools/index.js
+ * expects: { name, schema, makeHandler }.
+ *
+ * @param {object} opts
+ * @param {Set<string>=} opts.categories   override DEFAULT_CATEGORIES
+ * @param {Set<string>=} opts.skipNames    extra names to skip in addition to NATIVE_NAMES
+ */
+export function buildBridgedTools(opts = {}) {
+  const categories = opts.categories instanceof Set ? opts.categories : DEFAULT_CATEGORIES;
+  const skipNames = opts.skipNames instanceof Set ? opts.skipNames : new Set();
+
+  return TOOL_DEFINITIONS
+    .filter(e => categories.has(e.category))
+    .filter(e => !NATIVE_NAMES.has(e.name) && !skipNames.has(e.name))
+    .filter(e => e.endpoint?.path)
+    .map(entry => ({
+      name: entry.name,
+      schema: buildSchema(entry),
+      makeHandler: buildHandler(entry),
+    }));
+}
+
+export { NATIVE_NAMES, DEFAULT_CATEGORIES };

package/src/daemon/super-agent.js

@@ -61,7 +61,8 @@ HARD RULES (do not deviate):
 15. NO-PENDING RULE: never say "give me a second", "I will do it", or "I will try later" as a final answer. Either call the tool in this same turn or say what blocks you.
 16. IDENTITY RULE: when the user asks you to change your name, call yourself something, or update your personality/language, call set_identity and persist the change. Then confirm with your new name.
 17. ROUTINES RULE: NEVER create a routine in the default project (id=0). Routines MUST be tied to a specific registered project. Before adding a routine, call list_projects to find the correct project id or name. Then pass --project <id|name> to apx routine add. If no project fits, ask the user which project to use. Creating routines in project 0/default mixes unrelated projects' schedules and corrupts state.
-18. **NO EMPTY RESPONSES**: Never respond with only text when you have tools available and the user is asking you to DO something. Call the tool FIRST, then explain. Never say "I'll do X" without immediately calling the tool. Empty acknowledgments ("ok", "entendido", "dame un minuto", "voy", "checking", "stand by") without a tool call are invalid responses — they will be re-prompted and waste a turn.`;
+18. **NO EMPTY RESPONSES**: Never respond with only text when you have tools available and the user is asking you to DO something. Call the tool FIRST, then explain. Never say "I'll do X" without immediately calling the tool. Empty acknowledgments ("ok", "entendido", "dame un minuto", "voy", "checking", "stand by") without a tool call are invalid responses — they will be re-prompted and waste a turn.
+19. **CWD RULE**: When the channel context includes a "CWD: <path>" line, that is the user's current working directory. References to "este directorio", "este proyecto", "esta carpeta", "acá", "aquí", "this directory", "this project", "current dir/folder" all mean that exact CWD path. Use it as the path argument directly — DO NOT ask the user "what's the path?" when CWD is already given. Example: if user says "agregá este proyecto a la lista", call add_project({path: <CWD>}) immediately.`;
 
 function isShortConfirmation(text) {
   return /^(yes|y|si|si dale|dale|ok|okay|confirm|confirmed|go|proceed|do it)\b/i