@sema-agent/core 5.0.1 → 5.2.0

package/dist/stores/file/session-store.js

@@ -54,12 +54,21 @@ class FileSessionStorage extends BaseSessionStorage {
 export class FileSessionRepo {
     dir;
     tmpDir;
+    onCorruptRead;
     joined = new Map();
-    constructor(root) {
+    constructor(root, opts) {
         this.dir = join(root, "sessions");
         this.tmpDir = join(root, "tmp");
         ensureDir(this.dir);
         ensureDir(this.tmpDir);
+        this.onCorruptRead = opts?.onCorruptRead;
+    }
+    disclose(path, reason) {
+        try {
+            this.onCorruptRead?.({ path, reason });
+        }
+        catch {
+        }
     }
     pathFor(id) {
         return join(this.dir, `${sanitizePathComponent(id)}${SUFFIX}`);
@@ -69,7 +78,7 @@ export class FileSessionRepo {
         if (!existsSync(path)) {
             throw new SessionError("not_found", `Session not found: ${id}`);
         }
-        const lines = readJsonlRecords(path);
+        const lines = readJsonlRecords(path, (info) => this.disclose(info.path, info.reason));
         let createdAt = "";
         let forkedFrom;
         const entries = [];
@@ -132,7 +141,10 @@ export class FileSessionRepo {
         try {
             names = readdirSync(this.dir);
         }
-        catch {
+        catch (err) {
+            if (err.code !== "ENOENT") {
+                this.disclose(this.dir, `session directory unreadable (${err.code ?? "unknown"}) — listed as empty`);
+            }
             return [];
         }
         const out = [];
@@ -144,7 +156,10 @@ export class FileSessionRepo {
                 const { createdAt, forkedFrom } = this.read(id);
                 out.push({ id, createdAt, ...(forkedFrom !== undefined ? { forkedFrom } : {}) });
             }
-            catch {
+            catch (err) {
+                if (!(err instanceof SessionError && err.code === "not_found")) {
+                    this.disclose(this.pathFor(id), `session log unreadable (${err instanceof Error ? err.message : String(err)}) — skipped from the listing`);
+                }
             }
         }
         return out.sort((a, b) => a.createdAt.localeCompare(b.createdAt));

package/dist/tools/fs/fs-search-tools.js

@@ -37,10 +37,18 @@ export function createGrepTool(env, rootCanonical, additionalRoots) {
                 description: 'Limit output to first N lines/entries, equivalent to "| head -N". Works across all output modes: content (limits output lines), files_with_matches (limits file paths), count (limits count entries). Defaults to 250 when unspecified. Pass 0 for unlimited (use sparingly — large result sets waste context).',
             })),
             offset: Type.Optional(Type.Number({ description: "Skip the first N output lines (pagination partner of head_limit)." })),
-            max_results: Type.Optional(Type.Number({ description: "Deprecated alias for head_limit." })),
         }),
         effect: "read",
         execute: async (args, ctx) => {
+            const retired = args;
+            for (const [name, repair] of [
+                ["regex", "the `pattern` is ALWAYS evaluated as a regular expression (ripgrep semantics) — drop the key; to match text literally, escape the regex metacharacters in `pattern`"],
+                ["max_results", "cap the result count with `head_limit` (0 = unlimited), paginating with `offset`"],
+            ]) {
+                if (retired[name] !== undefined) {
+                    return errorResult(`Error (Grep): \`${name}\` is not a Grep parameter (retired) — ${repair}. Nothing was searched.`);
+                }
+            }
             const a = args;
             for (const [name, value] of [
                 ["head_limit", a.head_limit],
@@ -69,7 +77,7 @@ export function createGrepTool(env, rootCanonical, additionalRoots) {
             const grepRun = await runGrepDetailed(env, rootCanonical, {
                 ...a,
                 path: scoped,
-                head_limit: a.head_limit ?? a.max_results,
+                head_limit: a.head_limit,
                 context: a.context ?? a["-C"],
                 context_after: a.context_after ?? a["-A"],
                 context_before: a.context_before ?? a["-B"],

package/dist/tools/fs/fs-shared.d.ts

@@ -42,8 +42,7 @@ export declare function bashTimeoutCapsSec(caps: {
 };
 export declare function bashMaxOutputChars(): number;
 export declare const FILE_PATH_PARAMS: {
-    file_path: Type.TOptional<Type.TString>;
-    path: Type.TOptional<Type.TString>;
+    file_path: Type.TString;
 };
 export declare function clipShellOutput(s: string): string;
 export declare function writeShellOverflowFile(env: ExecutionEnv, stdout: string, stderr: string): Promise<string | undefined>;

package/dist/tools/fs/fs-shared.js

@@ -76,8 +76,7 @@ export function bashMaxOutputChars() {
     return Math.min(Math.floor(raw), BASH_MAX_OUTPUT_CHARS_CEILING);
 }
 export const FILE_PATH_PARAMS = {
-    file_path: Type.Optional(Type.String({ description: "File path (within the configured root)." })),
-    path: Type.Optional(Type.String({ description: "Deprecated alias for `file_path` (back-compat; prefer file_path)." })),
+    file_path: Type.String({ description: "File path (within the configured root)." }),
 };
 export function clipShellOutput(s) {
     return clipWithFilePointer(s, bashMaxOutputChars());

package/dist/tools/todo.js

@@ -1,4 +1,5 @@
 import { Type } from "typebox";
+import { errorResult } from "../core/tools.js";
 export function createTodoWriteTool() {
     let todos = [];
     return {
@@ -6,7 +7,7 @@ export function createTodoWriteTool() {
         contract: { contractId: "core.todo_write@1", implementationRevision: "1" },
         description: "Create and update a task list for the current session. The list is rendered to the user as your working plan.\n" +
             "\n" +
-            '- Each todo has `content`, `status` ("pending" | "in_progress" | "completed"), and optionally `activeForm` (present-tense label shown while in progress).\n' +
+            '- Each todo has `content`, `status` ("pending" | "in_progress" | "completed"), and `activeForm` (present-tense label shown while in progress).\n' +
             "- Send the full list each call; it replaces the previous one.\n" +
             "- Keep one item `in_progress` at a time and mark it `completed` when done.",
         descriptionClassic: "Use this tool to create and manage a structured task list for your current coding session. This helps you track progress, organize complex tasks, and demonstrate thoroughness to the user.\n" +
@@ -76,18 +77,24 @@ export function createTodoWriteTool() {
             todos: Type.Array(Type.Object({
                 content: Type.String({ description: "The task, in imperative form (e.g. \"Run the tests\")." }),
                 status: Type.Union([Type.Literal("pending"), Type.Literal("in_progress"), Type.Literal("completed")]),
-                activeForm: Type.Optional(Type.String({ description: "Present-continuous form shown while in progress (e.g. \"Running the tests\")." })),
+                activeForm: Type.String({ minLength: 1, description: "Present-continuous form shown while in progress (e.g. \"Running the tests\")." }),
             })),
         }),
         effect: "idempotent",
         execute: (args) => {
-            const shape = (t) => ({
-                content: t.content,
-                status: t.status,
-                activeForm: t.activeForm && t.activeForm.length > 0 ? t.activeForm : t.content,
-            });
+            const shape = (t) => ({ content: t.content, status: t.status, activeForm: t.activeForm });
             const oldTodos = todos.map(shape);
             const written = args.todos;
+            for (let i = 0; i < written.length; i++) {
+                const activeForm = written[i].activeForm;
+                if (typeof activeForm !== "string" || activeForm.length === 0) {
+                    const what = activeForm === undefined ? "is missing" : "has an empty or non-string";
+                    return errorResult(`Error (TodoWrite): todo ${i + 1} of ${written.length} ${what} \`activeForm\` — every todo needs the ` +
+                        `present-continuous label shown while it runs (e.g. content "Run the tests" → activeForm "Running the ` +
+                        `tests"), alongside its imperative \`content\`. Nothing was stored; resend the COMPLETE list with the ` +
+                        `label filled in on every item.`);
+                }
+            }
             const allDone = written.every((t) => t.status === "completed");
             todos = allDone ? [] : written;
             const newTodos = written.map(shape);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sema-agent/core",
-  "version": "5.0.1",
+  "version": "5.2.0",
   "description": "Stateless, task-oriented AI agent core",
   "type": "module",
   "license": "BUSL-1.1",