shraga 0.1.79 → 0.1.80

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shraga",
-  "version": "0.1.79",
+  "version": "0.1.80",
   "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/scheduler/runner.ts

@@ -208,7 +208,7 @@ export async function runSchedule(
   // Tell the run how to report its own truthful outcome (scheduler/outcome.ts). Prompt tasks only:
   // a `bash` task's permission handler allows nothing but the task's own command, so such a run
   // could not write the file even if it wanted to — its exit code is already the truth there.
-  if (task.kind === 'prompt') {
+  if ((task.kind ?? 'prompt') === 'prompt') {
     // Cleared on RESUME too: a resume reuses the interrupted run's session id, so a declaration left
     // by the attempt that crashed would be adopted as this attempt's verdict. The contract is
     // re-stated for the same reason — the resumed turn must be able to declare for itself.
@@ -401,7 +401,7 @@ export async function runSchedule(
   // The run's OWN verdict beats "the turn returned" — see scheduler/outcome.ts. Deliberately after
   // the finally: the session lock is released by now, so a run that declared `pending` can be closed
   // by a later turn in this session (a background job's wake, a follow-up message) while we wait.
-  if (status === 'ok' && task.kind === 'prompt') {
+  if (status === 'ok' && (task.kind ?? 'prompt') === 'prompt') {
     const declared = await resolveDeclaredOutcome(sessionId, abortController, schedule.id);
     if (declared) {
       status = declared.status;

package/src/server/scheduler/storage.ts

@@ -27,11 +27,28 @@ export function saveThrottleState(state: Record<string, number>): void {
   renameSync(tmp, THROTTLE_FILE);
 }
 
+/**
+ * Schedules persisted before `task.kind` existed carry only the payload fields. Everything
+ * downstream branches on the discriminant, so infer it from the shape once, on load — a task
+ * missing its kind silently skipped the outcome contract and reported every run as `ok`.
+ * Only `prompt` is inferable: `bash` and `job` are both `{ command }`, and guessing between them
+ * would hand a run the wrong permission handler, so a kindless command task is left alone.
+ */
+function normalizeTask(task: Record<string, unknown>): void {
+  if (task.kind) return;
+  if (typeof task.prompt === 'string' || typeof task.promptFile === 'string') task.kind = 'prompt';
+}
+
+function normalizeSchedule(s: Schedule): Schedule {
+  if (s.task && typeof s.task === 'object') normalizeTask(s.task as unknown as Record<string, unknown>);
+  return s;
+}
+
 export function loadSchedules(): Schedule[] {
   if (!existsSync(FILE)) return [];
   try {
     const parsed = JSON.parse(readFileSync(FILE, 'utf-8'));
-    return Array.isArray(parsed) ? parsed : [];
+    return Array.isArray(parsed) ? parsed.map((s) => normalizeSchedule(s as Schedule)) : [];
   } catch (err) {
     console.error('[scheduler] failed to parse schedules.json:', err);
     return [];