@blogic-cz/agent-tools 0.1.0 → 0.2.1

package/src/session-tool/service.ts

@@ -51,8 +51,10 @@ const readJsonFilesInTree = (parentDir: string): Effect.Effect<FileEntry[], Sess
         const subPath = `${parentDir}/${subDir}`;
         let files: string[];
         try {
+          // eslint-disable-next-line eslint/no-await-in-loop -- sequential directory walk, each iteration may short-circuit
           files = await readdir(subPath);
         } catch {
+          /* ignore unreadable files */
           continue;
         }
 
@@ -63,8 +65,11 @@ const readJsonFilesInTree = (parentDir: string): Effect.Effect<FileEntry[], Sess
             try {
               const content = await Bun.file(filePath).text();
               results.push({ filePath, content });
-            } catch {}
+            } catch {
+              /* ignore unreadable files */
+            }
           });
+        // eslint-disable-next-line eslint/no-await-in-loop -- sequential directory walk, each iteration may short-circuit
         await Promise.all(reads);
       }
 
@@ -90,7 +95,9 @@ const readJsonFilesFlat = (dir: string): Effect.Effect<FileEntry[], SessionError
           try {
             const content = await Bun.file(filePath).text();
             results.push({ filePath, content });
-          } catch {}
+          } catch {
+            /* ignore unreadable files */
+          }
         });
       await Promise.all(reads);
 
@@ -191,7 +198,13 @@ export class SessionService extends ServiceMap.Service<
             }
           }
 
-          return summaries.sort((left, right) => right.created - left.created);
+          return (
+            summaries as MessageSummary[] & {
+              toSorted(
+                compareFn: (left: MessageSummary, right: MessageSummary) => number,
+              ): MessageSummary[];
+            }
+          ).toSorted((left, right) => right.created - left.created);
         }),
 
         searchSummaries: (summaries: MessageSummary[], query: string): MessageSummary[] => {

package/src/session-tool/types.ts

@@ -1,4 +1,4 @@
-import type { OutputFormat } from "../shared";
+import type { OutputFormat } from "#src/shared";
 
 export type { OutputFormat };
 

package/src/shared/bun.ts

@@ -51,7 +51,7 @@ export async function runCommand(
   };
 }
 
-export async function runShellCommand(
+export function runShellCommand(
   command: string,
   options: CommandOptions = {},
 ): Promise<CommandResult> {

package/src/shared/error-renderer.ts

@@ -11,12 +11,21 @@ const formatError = (error: unknown): string => {
   ) {
     const tag = (error as Record<string, unknown>)._tag as string;
     const message = (error as Record<string, unknown>).message;
-    if (typeof message === "string") return `${tag}: ${message}`;
-    const details = Object.entries(error as Record<string, unknown>)
-      .filter(([key, val]) => typeof val === "string" && key !== "_tag")
-      .map(([key, val]) => `${key}=${String(val)}`)
-      .join(", ");
-    return details ? `${tag}: ${details}` : tag;
+    const hint = (error as Record<string, unknown>).hint;
+    let result = "";
+    if (typeof message === "string") {
+      result = `${tag}: ${message}`;
+    } else {
+      const details = Object.entries(error as Record<string, unknown>)
+        .filter(([key, val]) => typeof val === "string" && key !== "_tag" && key !== "hint")
+        .map(([key, val]) => `${key}=${String(val)}`)
+        .join(", ");
+      result = details ? `${tag}: ${details}` : tag;
+    }
+    if (typeof hint === "string") {
+      result += `\n  Hint: ${hint}`;
+    }
+    return result;
   }
   if (error instanceof Error) return error.message;
   return String(error);
@@ -24,14 +33,15 @@ const formatError = (error: unknown): string => {
 
 const formatCause = (cause: Cause.Cause<unknown>): string => {
   const failures = cause.reasons.filter(Cause.isFailReason);
-  const firstFailure = Array.from(failures)[0];
-  if (firstFailure !== undefined) return formatError(firstFailure);
+  const firstFailure = failures[0];
+  if (firstFailure !== undefined) return formatError(firstFailure.error);
 
   const defects = cause.reasons.filter(Cause.isDieReason);
-  const firstDefect = Array.from(defects)[0];
+  const firstDefect = defects[0];
   if (firstDefect !== undefined) {
-    if (firstDefect instanceof Error) return `Unexpected error: ${firstDefect.message}`;
-    return `Unexpected error: ${String(firstDefect)}`;
+    if (firstDefect.defect instanceof Error)
+      return `Unexpected error: ${firstDefect.defect.message}`;
+    return `Unexpected error: ${String(firstDefect.defect)}`;
   }
 
   if (Cause.hasInterruptsOnly(cause)) return "Interrupted";

package/src/shared/index.ts

@@ -8,6 +8,7 @@ export { commonArgOptions, parseCommonArgs } from "./cli";
 
 export { renderCauseToStderr } from "./error-renderer";
 
+// eslint-disable-next-line import/no-relative-parent-imports -- package.json lives at project root, outside src/
 import pkg from "../../package.json" with { type: "json" };
 export const VERSION = pkg.version;
 

package/src/shared/types.ts

@@ -5,6 +5,9 @@ export type BaseResult = {
   success: boolean;
   error?: string;
   executionTimeMs: number;
+  nextCommand?: string;
+  retryable?: boolean;
+  hint?: string;
 };
 
 export type CommandOptions = {