@agentprojectcontext/apx 1.33.0 → 1.33.1

package/package.json

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

package/skills/apc-context/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: apc-context
-description: "Activate whenever the project has a .apc/ directory or AGENTS.md — read .apc/ before assuming anything about agents, memory, or structure. If .apc/migrate.md exists, open with a migration offer first; if the user declines, delete it. Triggers: .apc/, AGENTS.md, 'which agents', 'list agents', 'agent context', 'who are the agents'."
+description: "ALWAYS activate when the project has a .apc/ directory or AGENTS.md file. Do not wait to be asked. Read .apc/ before making any assumption about agents, memory, or project structure. Activate on: .apc/, AGENTS.md, 'which agents', 'list agents', 'agent context', 'who are the agents', any question about agents or memory in this project. IMPORTANT: if .apc/migrate.md exists, open the conversation with a migration offer before answering anything else. If the user declines, delete .apc/migrate.md immediately so it is not shown again."
 homepage: https://github.com/agentprojectcontext/agentprojectcontext
 ---
 
@@ -66,13 +66,11 @@ After migration:
 ```text
 AGENTS.md                        ← root project contract
 .apc/
-  project.json                   ← project metadata (may carry an `apx` field)
-  config.json                    ← project-only config overrides (e.g. super_agent.model)
+  project.json                   ← project metadata
   .gitignore                     ← safety guard
   agents/<name>.md               ← agent definition
   agents/<name>/memory.md        ← optional curated project memory
   skills/<name>.md               ← reusable project instructions
-  commands/                      ← custom slash-commands (optional)
   mcps.json                      ← MCP hints without secrets
 ```
 
@@ -83,7 +81,6 @@ Do not store:
 .apc/sessions/
 .apc/conversations/
 .apc/messages/
-.apc/project.db
 .apc/cache/
 .apc/tmp/
 .apc/private/

package/src/core/apc/parser.js

@@ -123,7 +123,7 @@ import { fileURLToPath } from "node:url";
 const __parserDir = path.dirname(fileURLToPath(import.meta.url));
 
 export const VAULT_DIR = path.join(os.homedir(), ".apx", "agents");
-export const BUNDLED_VAULT_DIR = path.resolve(__parserDir, "../../assets/agent-vault-defaults");
+export const BUNDLED_VAULT_DIR = path.resolve(__parserDir, "../../../assets/agent-vault-defaults");
 export const VAULT_TOMBSTONE_PATH = path.join(VAULT_DIR, ".removed.json");
 
 function readVaultDirRaw(dir) {

package/src/core/apc/scaffold.js

@@ -16,7 +16,9 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url));
 // Now under src/core/apc/ — one more "../" to escape than before.
 const PACKAGE_ROOT = path.resolve(__dirname, "..", "..", "..");
 const BUNDLED_SKILLS_DIR = path.join(PACKAGE_ROOT, "skills");
-const RUNTIME_SKILLS_DIR = path.join(__dirname, "runtime-skills");
+// runtime-skills lives at src/core/runtime-skills/, one level up from this
+// file's new home in src/core/apc/ (was a sibling before the Phase 3 move).
+const RUNTIME_SKILLS_DIR = path.join(__dirname, "..", "runtime-skills");
 
 export const SPEC_VERSION = "0.1.0";
 

package/src/core/apc/skill-sync.js

@@ -4,7 +4,9 @@ import path from "node:path";
 import { fileURLToPath } from "node:url";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
-export const PACKAGE_ROOT = path.resolve(__dirname, "..", "..");
+// __dirname is src/core/apc/ after the Phase 3 move (was src/core/ before).
+// Repo root is three levels up, not two.
+export const PACKAGE_ROOT = path.resolve(__dirname, "..", "..", "..");
 
 export const APC_SKILL_REL = path.join("skills", "apc-context", "SKILL.md");
 export const APC_SKILL_REMOTE =

package/src/core/engines/gemini.js

@@ -51,15 +51,24 @@ function toGeminiContents(messages) {
     if (m.role === "assistant" && Array.isArray(m.tool_calls) && m.tool_calls.length > 0) {
       out.push({
         role: "model",
-        parts: m.tool_calls.map((tc) => ({
-          functionCall: {
-            name: tc.function?.name || tc.name,
-            args:
-              typeof tc.function?.arguments === "string"
-                ? safeParseJson(tc.function.arguments)
-                : tc.function?.arguments || tc.arguments || {},
-          },
-        })),
+        parts: m.tool_calls.map((tc) => {
+          const part = {
+            functionCall: {
+              name: tc.function?.name || tc.name,
+              args:
+                typeof tc.function?.arguments === "string"
+                  ? safeParseJson(tc.function.arguments)
+                  : tc.function?.arguments || tc.arguments || {},
+            },
+          };
+          // Gemini 3.x thinking models require us to echo back the
+          // thoughtSignature that came attached to the original functionCall
+          // part, or the API rejects the next turn with 400. We captured it
+          // in the response parser; replay it verbatim when present.
+          const sig = tc._thoughtSignature || tc.thought_signature;
+          if (sig) part.thoughtSignature = sig;
+          return part;
+        }),
       });
       continue;
     }
@@ -143,14 +152,22 @@ export default {
     for (const p of parts) {
       const fc = p.functionCall || p.function_call;
       if (fc?.name) {
-        toolCalls.push({
+        const tc = {
           id: `gemini_${randomUUID().slice(0, 8)}`,
           type: "function",
           function: {
             name: fc.name,
             arguments: typeof fc.args === "string" ? fc.args : JSON.stringify(fc.args || {}),
           },
-        });
+        };
+        // Thinking models (Gemini 3.x) attach a thoughtSignature to the part
+        // alongside the functionCall. We must replay it on the next request
+        // or the API 400s. Carry it on the tool_call so the next call to
+        // toGeminiContents() can put it back. Underscore prefix marks it as
+        // adapter-private metadata other engines should ignore.
+        const sig = p.thoughtSignature || p.thought_signature;
+        if (sig) tc._thoughtSignature = sig;
+        toolCalls.push(tc);
       }
     }
 

package/src/core/engines/index.js

@@ -52,12 +52,22 @@ export async function callEngine({ modelId, system, messages, config, temperatur
   const { provider, model } = resolveProvider(modelId);
   const adapter = getAdapter(provider);
   const providerCfg = (config && config.engines && config.engines[provider]) || {};
+  // The per-provider `default_max_tokens` set in the web admin (Provider modal
+  // slider) acts as a floor: callers may ask for more, but never less. This
+  // matters for "thinking" models (e.g. Gemini 3.x) whose internal reasoning
+  // tokens count against maxOutputTokens — too low a cap and the visible reply
+  // gets truncated mid-sentence. Fallback chain:
+  //   caller value → provider cfg → 2048 (safe baseline that survives thinking
+  //   models without truncating; non-thinking models just don't fill it).
+  const providerCap = Number(providerCfg.default_max_tokens) || 0;
+  const callerCap = Number(maxTokens) || 0;
+  const effectiveMaxTokens = Math.max(callerCap, providerCap) || 2048;
   return adapter.chat({
     system,
     messages,
     model,
     temperature,
-    maxTokens,
+    maxTokens: effectiveMaxTokens,
     tools,
     toolChoice,
     config: providerCfg,

package/src/host/daemon/api/engines.js

@@ -13,6 +13,11 @@ const DEFAULT_BASE = {
   ollama:     "http://localhost:11434",
 };
 
+// Gemini's native models endpoint returns a much richer catalog than the
+// OpenAI-compat shim (which only echoes back a handful). We always query the
+// native URL regardless of the user's configured base_url.
+const GEMINI_NATIVE_BASE = "https://generativelanguage.googleapis.com/v1beta";
+
 // Returns { models } or { error }. Reads the right /models endpoint per engine.
 async function listModels(engine, baseUrl, apiKey) {
   const base = String(baseUrl || DEFAULT_BASE[engine] || "").replace(/\/$/, "");
@@ -37,7 +42,32 @@ async function listModels(engine, baseUrl, apiKey) {
     return { models: data.map((m) => m?.id).filter(Boolean) };
   }
 
-  // openai-compatible family: openai, groq, openrouter, gemini, azure, custom
+  if (engine === "gemini") {
+    if (!apiKey) return { error: "falta api_key" };
+    // Native Gemini API: returns a `models` array with rich metadata, including
+    // `supportedGenerationMethods` so we can drop embeddings/vision-only entries.
+    // Names come back as "models/<id>"; strip the prefix for display.
+    const r = await fetchJsonWithTimeout(
+      `${GEMINI_NATIVE_BASE}/models?key=${encodeURIComponent(apiKey)}&pageSize=200`,
+      { timeoutMs: 5000 },
+    );
+    if (!r.ok) return { error: r.reason || `HTTP ${r.status}` };
+    const data = Array.isArray(r.json?.models) ? r.json.models : [];
+    const models = data
+      .filter((m) => {
+        const methods = m?.supportedGenerationMethods;
+        if (!Array.isArray(methods)) return true;
+        return methods.includes("generateContent");
+      })
+      .map((m) => {
+        const name = typeof m?.name === "string" ? m.name : "";
+        return name.startsWith("models/") ? name.slice("models/".length) : name;
+      })
+      .filter(Boolean);
+    return { models };
+  }
+
+  // openai-compatible family: openai, groq, openrouter, azure, custom
   if (!apiKey) return { error: "falta api_key" };
   if (!base) return { error: "falta base_url" };
   const r = await fetchJsonWithTimeout(`${base}/models`, {