@compilr-dev/sdk 0.10.26 → 0.10.28

package/dist/presets/coding.js

@@ -45,7 +45,7 @@ export const webTools = [webFetchTool];
  */
 const todoStore = getDefaultTodoStore();
 const todoTools = createTodoTools(todoStore);
-const lenientTodoWrite = createLenientTodoWriteTool(todoTools.todoWrite);
+const lenientTodoWrite = createLenientTodoWriteTool(todoTools.todoWrite, todoStore);
 const lenientTodoClaim = createLenientTodoClaimTool(todoTools.todoClaim, todoStore);
 const lenientTodoHandoff = createLenientTodoHandoffTool(todoTools.todoHandoff, todoStore);
 /**

package/dist/presets/general.js

@@ -24,7 +24,7 @@ Guidelines:
 // drop when the LLM picks the wrong key.
 const todoStore = getDefaultTodoStore();
 const todoTools = createTodoTools(todoStore);
-const lenientTodoWrite = createLenientTodoWriteTool(todoTools.todoWrite);
+const lenientTodoWrite = createLenientTodoWriteTool(todoTools.todoWrite, todoStore);
 const lenientTodoClaim = createLenientTodoClaimTool(todoTools.todoClaim, todoStore);
 const lenientTodoHandoff = createLenientTodoHandoffTool(todoTools.todoHandoff, todoStore);
 export const generalPreset = {

package/dist/presets/lenient-todo.d.ts

@@ -44,14 +44,25 @@ interface StrictTodoInput {
     }>;
 }
 /**
- * Wrap a strict `todo_write` tool with input-key normalisation.
+ * Wrap a strict `todo_write` tool with input-key normalisation AND
+ * response enrichment.
  *
- * Pass the `todoWrite` from `createTodoTools(store)` (or the standalone
- * `todoWriteTool` export). Returns a Tool with the same name (`todo_write`)
- * and the same schema, but with an `execute` that normalises input
- * before delegating.
+ * Two improvements over the strict tool:
+ *
+ * 1. **Input normalisation**: accepts alternative property aliases LLMs
+ *    commonly emit (`title`, `description`, `task`, `text` → `content`).
+ *    Prevents silent todo drops when the LLM picks the wrong key.
+ *
+ * 2. **Response enrichment**: after writing, returns the full todo list
+ *    *with ids* so subsequent `todo_claim` / `todo_handoff` calls have
+ *    the right `todoId` without needing an extra `todo_read` round trip.
+ *    Per agent feedback: "if todo_write returned the ids, that would
+ *    streamline the process."
+ *
+ * Pass the `todoWrite` from `createTodoTools(store)` and the same store
+ * instance — the wrapper reads the store after writing to surface ids.
  */
-export declare function createLenientTodoWriteTool(strictTool: Tool<StrictTodoInput>): Tool<LenientTodoInput>;
+export declare function createLenientTodoWriteTool(strictTool: Tool<StrictTodoInput>, store: TodoStore): Tool<LenientTodoInput>;
 interface LenientTodoClaimInput {
     todoId: string | number;
     agentId: string;

package/dist/presets/lenient-todo.js

@@ -20,17 +20,29 @@
  */
 import { defineTool, } from '@compilr-dev/agents';
 /**
- * Wrap a strict `todo_write` tool with input-key normalisation.
+ * Wrap a strict `todo_write` tool with input-key normalisation AND
+ * response enrichment.
  *
- * Pass the `todoWrite` from `createTodoTools(store)` (or the standalone
- * `todoWriteTool` export). Returns a Tool with the same name (`todo_write`)
- * and the same schema, but with an `execute` that normalises input
- * before delegating.
+ * Two improvements over the strict tool:
+ *
+ * 1. **Input normalisation**: accepts alternative property aliases LLMs
+ *    commonly emit (`title`, `description`, `task`, `text` → `content`).
+ *    Prevents silent todo drops when the LLM picks the wrong key.
+ *
+ * 2. **Response enrichment**: after writing, returns the full todo list
+ *    *with ids* so subsequent `todo_claim` / `todo_handoff` calls have
+ *    the right `todoId` without needing an extra `todo_read` round trip.
+ *    Per agent feedback: "if todo_write returned the ids, that would
+ *    streamline the process."
+ *
+ * Pass the `todoWrite` from `createTodoTools(store)` and the same store
+ * instance — the wrapper reads the store after writing to surface ids.
  */
-export function createLenientTodoWriteTool(strictTool) {
+export function createLenientTodoWriteTool(strictTool, store) {
     return defineTool({
         name: 'todo_write',
-        description: strictTool.definition.description,
+        description: `${strictTool.definition.description} ` +
+            'Returns the resulting todo list with ids — use those ids for any subsequent todo_claim / todo_handoff calls.',
         inputSchema: strictTool.definition.inputSchema,
         execute: async (input) => {
             // Normalise — accept alternative property names that LLMs commonly emit.
@@ -53,38 +65,59 @@ export function createLenientTodoWriteTool(strictTool) {
                 ...todo,
                 content: todo.content || 'Untitled task',
             }));
-            return strictTool.execute({ todos: valid });
+            const result = await strictTool.execute({ todos: valid });
+            // Enrich the success result with the actual stored todos (incl. ids)
+            // so the LLM has them in context immediately — no todo_read needed.
+            if (result.success) {
+                const todos = store.getAll().map((t) => ({
+                    id: t.id,
+                    content: t.content,
+                    status: t.status,
+                    owner: t.owner,
+                    activeForm: t.activeForm,
+                    priority: t.priority,
+                }));
+                const inner = result.result && typeof result.result === 'object'
+                    ? result.result
+                    : {};
+                return {
+                    success: true,
+                    result: { ...inner, todos },
+                };
+            }
+            return result;
         },
     });
 }
 /**
  * Resolve a lenient todoId reference to a concrete store key.
  * Returns null if no plausible match exists.
+ *
+ * Strategy intentionally narrow — only unambiguous one-to-one mappings.
+ * No content/prefix/substring matching: fuzzy resolution risks silent
+ * wrong matches in edge cases (e.g. "Fix login" matching the wrong of
+ * two similar todos). The agent should either pass the full id from a
+ * prior `todo_write` / `todo_read` response, or use the numeric position
+ * shown in the footer. Anything else gets a clear error listing the
+ * available ids, which the agent can recover from in one extra call.
  */
 function resolveTodoId(rawId, store) {
-    const all = store.getAll();
-    const ids = all.map((t) => t.id);
-    // String coercion + trim.
+    const ids = store.getAll().map((t) => t.id);
     const id = String(rawId).trim();
     // Direct match — handles "todo-1" as-is.
     if (ids.includes(id))
         return id;
-    // Bare positional reference: "1" or 1 → "todo-1".
+    // Bare positional reference: "1" or 1 → "todo-1". Matches the footer's
+    // "#1 ☐ task" rendering.
     if (/^\d+$/.test(id)) {
         const candidate = `todo-${id}`;
         if (ids.includes(candidate))
             return candidate;
     }
-    // Allow "todo_1" variant.
+    // "todo_1" variant — LLMs sometimes use underscores.
     const underscoreToHyphen = id.replace(/^todo_/, 'todo-');
     if (ids.includes(underscoreToHyphen))
         return underscoreToHyphen;
-    // Last resort: content-prefix match (case-insensitive). Only commits to
-    // a hit if exactly one todo's content starts with the given string.
-    const lower = id.toLowerCase();
-    const contentHits = all.filter((t) => t.content.toLowerCase().startsWith(lower));
-    if (contentHits.length === 1)
-        return contentHits[0].id;
     return null;
 }
 /**

package/dist/team/team-agent.js

@@ -188,6 +188,24 @@ export class TeamAgent {
         }
         // Build system prompt addition with shared context
         let finalSystemPromptAddition = this.systemPromptAddition ?? '';
+        // Identity preamble — every team agent gets a stable self-reference
+        // block at the top of its prompt addition. Without this, the agent
+        // knows it's "the PM" from the role prompt but doesn't connect that
+        // to `agentId: "pm"` when tools take an agentId parameter. User test
+        // 2026-06-04 caught this: $pm called `todo_read(owner="default")`
+        // because it didn't know its id was "pm".
+        //
+        // Uses runtime fields (this.id, this.displayName, this.mascot) so
+        // collision-renamed ids (e.g. "pm-2") are correctly stated.
+        const identityBlock = `# YOUR IDENTITY IN THIS TEAM\n\n` +
+            `You are agent **${this.displayName}** ${this.mascot}.\n\n` +
+            `- Your agent ID: \`${this.id}\`\n` +
+            `- When tools have an \`agentId\` (or \`owner\`, \`toAgentId\`) parameter referring to YOU, use \`"${this.id}"\`.\n` +
+            `- When users mention \`$${this.id}\`, they mean you.\n` +
+            `- When asked "who are you?" or "what is your role?", state both your role and your agent ID.`;
+        finalSystemPromptAddition = finalSystemPromptAddition
+            ? `${identityBlock}\n\n${finalSystemPromptAddition}`
+            : identityBlock;
         // Inject shared context if provided (excluding roster — that goes via anchor for live updates)
         if (sharedContext) {
             const sharedContextBlock = sharedContext.format({ excludeRoster: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compilr-dev/sdk",
-  "version": "0.10.26",
+  "version": "0.10.28",
   "description": "Universal agent runtime for building AI-powered applications",
   "type": "module",
   "main": "dist/index.js",