shraga 0.1.26 → 0.1.27

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shraga",
-  "version": "0.1.26",
+  "version": "0.1.27",
   "description": "The teammate you delegate coding to — a self-hostable, multi-user AI coding agent web UI (Claude Code, with a pluggable engine seam).",
   "type": "module",
   "main": "./src/index.ts",

package/src/server/claude.ts

@@ -74,7 +74,7 @@ export type WsEvent =
   | { type: 'text_delta'; text: string }
   | { type: 'tool_use'; tool: string; toolUseId: string; input: unknown }
   | { type: 'tool_use_input'; toolUseId: string; input: unknown }
-  | { type: 'tool_result'; toolUseId: string; output: string }
+  | { type: 'tool_result'; toolUseId: string; output: string; isError?: boolean }
   | { type: 'tool_result_image'; toolUseId: string; dataUrl: string }
   | { type: 'permission_request'; id: string; tool: string; input: Record<string, unknown> }
   | { type: 'question_request'; id: string; questions: AskQuestion[] }

package/src/server/engine/claude-code.ts

@@ -446,7 +446,9 @@ export class ClaudeCodeEngine implements AgentEngine {
               const output = contentArr.length > 0
                 ? contentArr.filter((c: any) => c.type === 'text').map((c: any) => c.text ?? '').join('')
                 : String(block.content ?? '');
-              yield { type: 'tool_result', toolUseId: String(block.tool_use_id), output };
+              // `is_error` is the ONLY signal that a tool failed — the text alone is indistinguishable
+              // from a successful result. Scheduled `bash` tasks rely on it to notice a non-zero exit.
+              yield { type: 'tool_result', toolUseId: String(block.tool_use_id), output, isError: block.is_error === true };
 
               let foundImage = false;
               for (const c of contentArr) {

package/src/server/scheduler/runner.ts

@@ -194,12 +194,21 @@ export async function runSchedule(
   let status: ScheduleRunSummary['status'] = 'ok';
   let error: string | undefined;
 
+  // A `bash` task is not exec'd — it's handed to the agent as a prompt, so the command's exit code
+  // reaches nobody: the agent reports the failure in prose, its own turn succeeds, the run is stored
+  // `ok`, and the failure notifier never fires. Track the tool_result of the task's OWN Bash call and
+  // fail the run with it, so a broken scheduled script alerts like a `job` does.
+  const taskBashToolUseIds = new Set<string>();
+  let bashFailure: string | undefined;
+
   onEvent({ type: 'schedule:run_started', scheduleId: schedule.id, sessionId, at: now });
 
   try {
     for (let attempt = 1; attempt <= MAX_ATTEMPTS; attempt++) {
       status = 'ok';
       error = undefined;
+      bashFailure = undefined;
+      taskBashToolUseIds.clear();
       try {
         for await (const ev of streamChat({
           prompt,
@@ -217,10 +226,12 @@ export async function runSchedule(
           } else if (ev.type === 'tool_use') {
             if (assistantText) { assistantBlocks.push({ type: 'text', text: assistantText }); assistantText = ''; }
             assistantBlocks.push({ type: 'tool_use', tool: ev.tool, toolUseId: ev.toolUseId, input: ev.input });
+            if (allowedCmd && ev.tool === 'Bash' && (ev.input as any)?.command === allowedCmd) taskBashToolUseIds.add(ev.toolUseId);
           } else if (ev.type === 'thinking_delta') {
             producedThinking = true;
           } else if (ev.type === 'tool_result') {
             assistantBlocks.push({ type: 'tool_result', toolUseId: ev.toolUseId, output: ev.output });
+            if (ev.isError && taskBashToolUseIds.has(ev.toolUseId)) bashFailure = ev.output;
           } else if (ev.type === 'done') {
             break;
           } else if (ev.type === 'error') {
@@ -238,6 +249,12 @@ export async function runSchedule(
         }
       }
 
+      // The agent turn can succeed while the command it was asked to run failed — that's the run failing.
+      if (status === 'ok' && bashFailure) {
+        status = 'error';
+        error = `Scheduled command failed:\n${bashFailure.slice(0, 4000)}`;
+      }
+
       if (status !== 'error') break;
 
       // The side-effect boundary. This is NOT an exact `ttft=-1` test — the engine emits events we