@llblab/pi-actors 0.22.5 → 0.24.0

package/lib/runtime.ts

@@ -98,9 +98,27 @@ export function createAutoToolsRuntime(
     runtimeTools.add(cfg.name);
     runtimeToolFingerprints.set(cfg.name, fingerprint);
   }
+  function isStartupActionableRegistryWarning(warning: string): boolean {
+    if (
+      warning.includes("invokes bash;") &&
+      warning.includes("trusted executable content")
+    )
+      return false;
+    if (
+      warning.includes("invokes bash;") &&
+      warning.includes("shell scripts are trusted executable content")
+    )
+      return false;
+    return true;
+  }
+
   function formatRecipeToolWarnings(warnings: string[]): string {
-    const shadowed = warnings.filter((warning) => warning.includes(" shadows "));
-    const skipped = warnings.filter((warning) => warning.includes(" could not be exposed as a tool:"));
+    const shadowed = warnings.filter((warning) =>
+      warning.includes(" shadows "),
+    );
+    const skipped = warnings.filter((warning) =>
+      warning.includes(" could not be exposed as a tool:"),
+    );
     const other = warnings.filter(
       (warning) => !shadowed.includes(warning) && !skipped.includes(warning),
     );
@@ -122,7 +140,8 @@ export function createAutoToolsRuntime(
   function loadTools(ctx: RuntimeContext) {
     const warnings: string[] = [];
     const recipeRoot = deps.recipeRoot ?? Paths.getRecipeRoot();
-    const packagedRecipeRoot = deps.packagedRecipeRoot ?? Paths.getPackagedRecipeRoot();
+    const packagedRecipeRoot =
+      deps.packagedRecipeRoot ?? Paths.getPackagedRecipeRoot();
     const migration = RecipeMigration.migrateLegacyToolRegistry({
       configPath: deps.configPath,
       recipeRoot,
@@ -130,9 +149,13 @@ export function createAutoToolsRuntime(
     });
     warnings.push(...migration.warnings);
     if (migration.conflicts.length > 0)
-      warnings.push(`Recipe migration conflicts: ${migration.conflicts.join(", ")}`);
+      warnings.push(
+        `Recipe migration conflicts: ${migration.conflicts.join(", ")}`,
+      );
     if (migration.invalid.length > 0)
-      warnings.push(`Recipe migration invalid entries: ${migration.invalid.join(", ")}`);
+      warnings.push(
+        `Recipe migration invalid entries: ${migration.invalid.join(", ")}`,
+      );
     const discovered = RecipeDiscovery.discoverRecipeSources([
       { root: recipeRoot, defaultTool: true, mutableUsage: true },
       { root: packagedRecipeRoot },
@@ -158,8 +181,9 @@ export function createAutoToolsRuntime(
       }
       registerRuntimeTool(cfg);
     }
-    if (warnings.length > 0) {
-      notify(ctx, formatRecipeToolWarnings(warnings), "warning");
+    const startupWarnings = warnings.filter(isStartupActionableRegistryWarning);
+    if (startupWarnings.length > 0) {
+      notify(ctx, formatRecipeToolWarnings(startupWarnings), "warning");
     }
   }
   return {

package/lib/state-readers.ts

@@ -0,0 +1,93 @@
+/**
+ * Resilient state readers.
+ * Zones: file-backed actor state, JSON/JSONL diagnostics, inspect safety
+ * Owns best-effort JSON and JSONL parsing helpers for operator-facing state reads.
+ */
+
+import { readFileSync } from "node:fs";
+
+export interface StateReadDiagnostic {
+  line?: number;
+  message: string;
+  path: string;
+}
+
+export interface JsonReadResult<T> {
+  diagnostics: StateReadDiagnostic[];
+  value: T;
+}
+
+export interface JsonlReadResult<T> {
+  diagnostics: StateReadDiagnostic[];
+  records: T[];
+}
+
+function isEnoent(error: unknown): boolean {
+  return (error as NodeJS.ErrnoException).code === "ENOENT";
+}
+
+function diagnosticMessage(error: unknown): string {
+  return error instanceof Error ? error.message : String(error);
+}
+
+export function readJsonFileResilient<T>(path: string, fallback: T): JsonReadResult<T> {
+  try {
+    return {
+      diagnostics: [],
+      value: JSON.parse(readFileSync(path, "utf8")) as T,
+    };
+  } catch (error) {
+    if (isEnoent(error)) return { diagnostics: [], value: fallback };
+    return {
+      diagnostics: [
+        {
+          message: diagnosticMessage(error),
+          path,
+        },
+      ],
+      value: fallback,
+    };
+  }
+}
+
+export function readJsonlFileResilient<T>(path: string): JsonlReadResult<T> {
+  try {
+    const records: T[] = [];
+    const diagnostics: StateReadDiagnostic[] = [];
+    const lines = readFileSync(path, "utf8").split("\n");
+    for (const [index, line] of lines.entries()) {
+      if (!line.trim()) continue;
+      try {
+        records.push(JSON.parse(line) as T);
+      } catch (error) {
+        diagnostics.push({
+          line: index + 1,
+          message: diagnosticMessage(error),
+          path,
+        });
+      }
+    }
+    return { diagnostics, records };
+  } catch (error) {
+    if (isEnoent(error)) return { diagnostics: [], records: [] };
+    return {
+      diagnostics: [
+        {
+          message: diagnosticMessage(error),
+          path,
+        },
+      ],
+      records: [],
+    };
+  }
+}
+
+export function formatStateReadDiagnostics(
+  diagnostics: StateReadDiagnostic[],
+  limit = 5,
+): string[] {
+  return diagnostics.slice(0, limit).map((diagnostic) => {
+    const line = diagnostic.line === undefined ? "" : `:${diagnostic.line}`;
+    return `${diagnostic.path}${line}: ${diagnostic.message}`;
+  });
+}