clementine-agent 1.18.75 → 1.18.77

package/dist/agent/advisor-rules/engine.js

@@ -70,14 +70,18 @@ export function evaluateWhen(c, ctx) {
             const lastRun = ctx.recentRuns[0];
             if (!lastRun)
                 return false;
-            const lastRunTime = new Date(lastRun.finishedAt).getTime();
+            // finishedAt is optional (in-progress runs are 'running' with no
+            // finishedAt). Fall back to startedAt so the rule still has a clock.
+            const lastRunTime = new Date(lastRun.finishedAt ?? lastRun.startedAt).getTime();
             return ctx.nowMs - lastRunTime > c.ms;
         }
         case 'lastRunWithinMs': {
             const lastRun = ctx.recentRuns[0];
             if (!lastRun)
                 return false;
-            const lastRunTime = new Date(lastRun.finishedAt).getTime();
+            // finishedAt is optional (in-progress runs are 'running' with no
+            // finishedAt). Fall back to startedAt so the rule still has a clock.
+            const lastRunTime = new Date(lastRun.finishedAt ?? lastRun.startedAt).getTime();
             return ctx.nowMs - lastRunTime <= c.ms;
         }
         case 'noRecentRuns':
@@ -185,7 +189,7 @@ const TEMPLATE_VARS = {
         const lastRun = ctx.recentRuns[0];
         if (!lastRun)
             return 0;
-        const lastRunTime = new Date(lastRun.finishedAt).getTime();
+        const lastRunTime = new Date(lastRun.finishedAt ?? lastRun.startedAt).getTime();
         const elapsed = ctx.nowMs - lastRunTime;
         const cooldown = 60 * 60 * 1000;
         return Math.max(0, Math.ceil((cooldown - elapsed) / 60_000));

package/dist/agent/execution-advisor.js

@@ -75,7 +75,7 @@ function computeLegacyAdvice(jobName, job) {
                 advice.skipReason = `${consecutiveErrors} consecutive errors — circuit breaker engaged`;
                 return advice;
             }
-            const lastRunTime = new Date(lastRun.finishedAt).getTime();
+            const lastRunTime = new Date(lastRun.finishedAt ?? lastRun.startedAt).getTime();
             const elapsed = Date.now() - lastRunTime;
             if (elapsed < CIRCUIT_BREAKER_COOLDOWN_MS) {
                 advice.shouldSkip = true;

package/dist/channels/discord-agent-bot.js

@@ -366,7 +366,7 @@ export class AgentBotClient {
                 if (recent.length > 0) {
                     const r = recent[0];
                     const icon = r.status === 'ok' ? '\u2705' : r.status === 'error' ? '\u274C' : '\u23ED';
-                    const ago = Math.round((Date.now() - new Date(r.finishedAt).getTime()) / 60000);
+                    const ago = Math.round((Date.now() - new Date(r.finishedAt ?? r.startedAt).getTime()) / 60000);
                     lastRunLines.push(`${icon} ${display} \u2014 ${formatAgentDuration(ago)} ago`);
                 }
                 else {

package/dist/cli/cron.js

@@ -48,7 +48,7 @@ export async function cmdCronList() {
         const status = job.enabled ? 'enabled' : 'disabled';
         const recent = runLog.readRecent(job.name, 1);
         const lastRun = recent.length > 0
-            ? `last run: ${recent[0].finishedAt.slice(0, 16).replace('T', ' ')} (${recent[0].status})`
+            ? `last run: ${(recent[0].finishedAt ?? recent[0].startedAt).slice(0, 16).replace('T', ' ')} (${recent[0].status})`
             : 'never run';
         const errors = runLog.consecutiveErrors(job.name);
         const errorTag = errors > 0 ? ` [${errors} consecutive error(s)]` : '';

package/dist/cli/dashboard.js

@@ -1350,20 +1350,13 @@ function getCronJobs() {
         }
         catch { /* ignore */ }
     }
-    // Attach recent run history
-    const runsDir = path.join(BASE_DIR, 'cron', 'runs');
+    // Attach recent run history. Single source of truth via CronRunLog.readRecent
+    // — same path the new /api/cron/runs cross-job endpoint uses, so per-card
+    // last-run and the Recent History zone never disagree.
+    const log = new CronRunLog();
     const enriched = jobs.map((job) => {
         const name = String(job.name ?? '');
-        const safe = name.replace(/[^a-zA-Z0-9_-]/g, '_');
-        const logPath = path.join(runsDir, `${safe}.jsonl`);
-        let recentRuns = [];
-        if (existsSync(logPath)) {
-            try {
-                const lines = readFileSync(logPath, 'utf-8').trim().split('\n').filter(Boolean);
-                recentRuns = lines.slice(-10).map((l) => JSON.parse(l)).reverse();
-            }
-            catch { /* ignore */ }
-        }
+        const recentRuns = log.readRecent(name, 10);
         return { ...job, recentRuns };
     });
     return { jobs: enriched };
@@ -4597,6 +4590,25 @@ export async function cmdDashboard(opts) {
     app.post('/api/cron/run/:job', (req, res) => {
         const jobName = req.params.job;
         try {
+            // Concurrency lock (1.18.76+): the CronScheduler persists a sidecar
+            // ~/.clementine/cron-running.json with one entry per in-flight job.
+            // If a manual Run Now collides with an already-running job (whether
+            // from another Run Now click or a scheduled tick), reject 409 so the
+            // user gets immediate feedback instead of two parallel runs racing
+            // each other into the same outputs.
+            try {
+                const runningFile = path.join(BASE_DIR, 'cron-running.json');
+                if (existsSync(runningFile)) {
+                    const entries = JSON.parse(readFileSync(runningFile, 'utf-8'));
+                    const collision = Array.isArray(entries) && entries.find(e => String(e.jobName ?? '').toLowerCase() === jobName.toLowerCase());
+                    if (collision) {
+                        const since = collision.startedAt ? ` (running since ${collision.startedAt})` : '';
+                        res.status(409).json({ ok: false, error: `Job "${jobName}" is already running${since}` });
+                        return;
+                    }
+                }
+            }
+            catch { /* if the sidecar is unreadable, fall through and let the runner handle dedup */ }
             const child = spawn('node', [DIST_ENTRY, 'cron', 'run', jobName], {
                 detached: true,
                 stdio: ['ignore', 'pipe', 'pipe'],
@@ -6356,7 +6368,9 @@ If the tool returns nothing or errors, return an empty array \`[]\`.`,
     // ── CRON CRUD routes (continued) ──────────────────────────────
     app.post('/api/cron', (req, res) => {
         try {
-            const { name, schedule, prompt, tier, enabled, work_dir, mode, max_hours, max_retries, after, agent, context, skills, allowedTools, allowedMcpServers, tags, category, predictable, } = req.body;
+            const { name, schedule, prompt, tier, enabled, work_dir, mode, max_hours, max_retries, after, agent, context, skills, allowedTools, allowedMcpServers, tags, category, predictable, 
+            // PRD Phase 1 fields (camelCase from API; written as snake_case YAML).
+            successCriteriaText, successSchema, addDirs, } = req.body;
             if (!name || !schedule || !prompt) {
                 res.status(400).json({ error: 'name, schedule, and prompt are required' });
                 return;
@@ -6415,6 +6429,16 @@ If the tool returns nothing or errors, return an empty array \`[]\`.`,
             // Predictable mode — default to true (contract execution) for new
             // tricks created via the dashboard. Mirror the MCP tool default.
             job.predictable = (predictable === false) ? false : true;
+            // PRD Phase 1: goal-orientation fields (camelCase from API → snake_case YAML).
+            if (typeof successCriteriaText === 'string' && successCriteriaText.trim()) {
+                job.success_criteria_text = successCriteriaText.trim();
+            }
+            if (successSchema && typeof successSchema === 'object' && !Array.isArray(successSchema) && Object.keys(successSchema).length > 0) {
+                job.success_schema = successSchema;
+            }
+            if (Array.isArray(addDirs) && addDirs.length) {
+                job.add_dirs = addDirs.map(String).map((s) => s.trim()).filter(Boolean);
+            }
             jobs.push(job);
             writeCronFileAt(cronFile, parsed, jobs);
             res.json({ ok: true, message: `Created cron job: ${name}` });
@@ -6556,6 +6580,36 @@ If the tool returns nothing or errors, return an empty array \`[]\`.`,
             if (updates.predictable !== undefined) {
                 jobs[idx].predictable = Boolean(updates.predictable);
             }
+            // PRD Phase 1 goal fields. set-when-non-empty / delete-when-cleared,
+            // matching the existing trick-capability pattern above. The deprecated
+            // success_criteria array is dropped on the first save through this path.
+            if (updates.successCriteriaText !== undefined) {
+                const v = typeof updates.successCriteriaText === 'string' ? updates.successCriteriaText.trim() : '';
+                if (v) {
+                    jobs[idx].success_criteria_text = v;
+                    delete jobs[idx].success_criteria; // sunset the deprecated alias on first save
+                }
+                else {
+                    delete jobs[idx].success_criteria_text;
+                }
+            }
+            if (updates.successSchema !== undefined) {
+                const s = updates.successSchema;
+                if (s && typeof s === 'object' && !Array.isArray(s) && Object.keys(s).length > 0) {
+                    jobs[idx].success_schema = s;
+                }
+                else {
+                    delete jobs[idx].success_schema;
+                }
+            }
+            if (updates.addDirs !== undefined) {
+                if (Array.isArray(updates.addDirs) && updates.addDirs.length) {
+                    jobs[idx].add_dirs = updates.addDirs.map(String).map((s) => s.trim()).filter(Boolean);
+                }
+                else {
+                    delete jobs[idx].add_dirs;
+                }
+            }
             if (updates.name !== undefined && updates.name !== bareJobName) {
                 // Rename — check for duplicates
                 const dup = jobs.find((j, i) => i !== idx && String(j.name ?? '').toLowerCase() === String(updates.name).toLowerCase());
@@ -6584,8 +6638,18 @@ If the tool returns nothing or errors, return an empty array \`[]\`.`,
             }
             jobs[idx].enabled = !jobs[idx].enabled;
             writeCronFileAt(cronFile, parsed, jobs);
-            const state = jobs[idx].enabled ? 'enabled' : 'disabled';
-            res.json({ ok: true, message: `${jobName} is now ${state}` });
+            const enabled = jobs[idx].enabled;
+            const state = enabled ? 'enabled' : 'disabled';
+            // Broadcast so other open dashboard tabs flip the toggle without polling.
+            try {
+                broadcastEvent({ type: 'cron_toggled', data: { job: bareJobName, enabled } });
+            }
+            catch { /* non-fatal */ }
+            // The CronScheduler's watchFile polls CRON.md every 2s and reloads
+            // automatically (cron-scheduler.ts:756 watchCronFile), so the toggle
+            // takes effect within 2 seconds without an explicit reload here. We
+            // surface a hint to the user about that latency in the response.
+            res.json({ ok: true, message: `${jobName} is now ${state}`, enabled });
         }
         catch (err) {
             res.status(500).json({ error: String(err) });
@@ -19976,6 +20040,32 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           </div>
         </div>
 
+        <!-- ── PRD Phase 1: Goal (success_criteria_text + success_schema) ──
+             The single most important new field set. Without one of these, a
+             run "finished"; with one, a run "accomplished what it was meant
+             to". Banner warns (does not block) when neither is set. -->
+        <div class="cron-section-card">
+          <h4>Goal <span style="color:var(--text-muted);font-weight:normal;font-size:12px">— how do you know this task succeeded?</span></h4>
+          <p class="cron-section-desc">Optional but strongly recommended. Use plain English (an evaluator agent grades the run) or a JSON Schema (validated against the agent's structured output).</p>
+          <div id="cron-goal-warning" style="display:none;margin-bottom:12px;padding:10px 12px;border-radius:6px;background:rgba(245,158,11,0.10);border:1px solid rgba(245,158,11,0.30);color:var(--yellow);font-size:12px">
+            ⚠ No goal set — runs will be marked "finished" but not "accomplished". Add a success criterion below or a JSON Schema.
+          </div>
+          <div class="form-group">
+            <label class="form-label">Success criterion <span style="color:var(--text-muted);font-weight:normal">(plain English)</span></label>
+            <textarea id="cron-success-criteria-text" rows="3" placeholder="e.g. 'A daily briefing email was sent to nathan@example.com containing the top 3 overnight items.'" oninput="updateGoalWarning()"></textarea>
+            <div class="form-hint">An evaluator sub-agent reads the run's output and this criterion, then emits pass/fail with reasoning.</div>
+          </div>
+          <div class="form-group" style="margin-bottom:0">
+            <details>
+              <summary style="cursor:pointer;font-size:12px;color:var(--text-secondary);font-weight:500;padding:6px 0">▾ Success schema (JSON Schema, advanced)</summary>
+              <div style="margin-top:8px">
+                <textarea id="cron-success-schema" rows="6" placeholder='{ "type": "object", "required": ["sent"], "properties": { "sent": { "type": "boolean" } } }' style="font-family:'JetBrains Mono',monospace;font-size:11px" oninput="updateGoalWarning()"></textarea>
+                <div class="form-hint">JSON Schema validated against the agent's <code>structured_output</code>. Mechanically successful = parses + validates.</div>
+              </div>
+            </details>
+          </div>
+        </div>
+
         <!-- Skills & tools: pinned skills + MCP + tools + tags -->
         <div class="cron-section-card">
           <h4>Skills &amp; tools</h4>
@@ -20052,6 +20142,14 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
                 <div class="form-hint">Run inside a project directory. Agent gets that project's CLAUDE.md.</div>
               </div>
             </div>
+            <!-- PRD Phase 1: read scope beyond cwd. One absolute path per line. -->
+            <div class="form-row">
+              <div class="form-group" style="flex:1">
+                <label class="form-label">Additional read directories <span style="color:var(--text-muted);font-weight:normal">(optional)</span></label>
+                <textarea id="cron-add-dirs" rows="2" placeholder="/Users/me/notes&#10;/Users/me/clients/acme" style="font-family:'JetBrains Mono',monospace;font-size:11px"></textarea>
+                <div class="form-hint">One absolute path per line. The agent gets read access to these in addition to the Project Context cwd.</div>
+              </div>
+            </div>
             <div class="form-row">
               <div class="form-group">
                 <label class="form-label">Mode</label>
@@ -23441,7 +23539,7 @@ async function openTraceViewer(jobName) {
     traceData = d.traces || [];
 
     if (traceData.length === 0) {
-      document.getElementById('trace-content').innerHTML = '<div style="padding:20px;color:var(--text-muted)">No traces recorded yet. Traces are captured on the next run.</div>';
+      document.getElementById('trace-content').innerHTML = '<div style="padding:24px;color:var(--text-muted);line-height:1.6"><div style="font-weight:500;color:var(--text-secondary);margin-bottom:8px">No traces recorded yet</div><div style="font-size:12px">Traces are captured per run and show every tool call the agent made.<br/>If a recent run errored before any tool was invoked, see the <strong>Recent history</strong> zone on the Tasks page for the error message.</div></div>';
       document.getElementById('trace-run-selector').innerHTML = '';
       return;
     }
@@ -23493,7 +23591,7 @@ function renderTrace(idx) {
       + '</div>';
   }
 
-  document.getElementById('trace-content').innerHTML = html || '<div style="padding:20px;color:var(--text-muted)">Empty trace</div>';
+  document.getElementById('trace-content').innerHTML = html || '<div style="padding:24px;color:var(--text-muted);line-height:1.6"><div style="font-weight:500;color:var(--text-secondary);margin-bottom:8px">This run produced no tool calls</div><div style="font-size:12px">The run started but didn\\x27t reach any tool — it likely errored before the agent acted, or was a no-op response.<br/>Check <strong>Recent history</strong> on the Tasks page for the error message or output preview from this run.</div></div>';
 }
 
 // ── Inline Trace (Execution tab) ──
@@ -24697,6 +24795,18 @@ function renderCronLegacyBanner(job) {
     + '</div>';
 }
 
+// PRD Phase 1: show a non-blocking warning under the Goal section header
+// when neither success_criteria_text nor success_schema is set. The PRD's
+// "the run accomplished what it was supposed to" promise depends on at
+// least one of the two being present.
+function updateGoalWarning() {
+  var sct = (document.getElementById('cron-success-criteria-text')?.value || '').trim();
+  var ssc = (document.getElementById('cron-success-schema')?.value || '').trim();
+  var warn = document.getElementById('cron-goal-warning');
+  if (!warn) return;
+  warn.style.display = (!sct && !ssc) ? '' : 'none';
+}
+
 // One-click migration: flip predictable=true AND save immediately so the
 // user doesn't have to remember to also click Save Changes.
 async function enablePredictableFromBanner() {
@@ -24738,6 +24848,11 @@ function openCreateCronModal(agentSlug) {
   toggleUnleashedOptions();
   document.getElementById('cron-prompt').value = '';
   document.getElementById('cron-context').value = '';
+  // PRD Phase 1 goal fields — empty by default. Warning banner will show.
+  var sct = document.getElementById('cron-success-criteria-text'); if (sct) sct.value = '';
+  var ssc = document.getElementById('cron-success-schema'); if (ssc) ssc.value = '';
+  var addDirsEl = document.getElementById('cron-add-dirs'); if (addDirsEl) addDirsEl.value = '';
+  if (typeof updateGoalWarning === 'function') updateGoalWarning();
   document.getElementById('cron-training-section').style.display = 'none';
   document.getElementById('cron-train-btn').style.display = '';
   resetCronTrainingChat();
@@ -24784,6 +24899,30 @@ function openEditCronModal(jobName) {
   toggleUnleashedOptions();
   document.getElementById('cron-prompt').value = job.prompt || '';
   document.getElementById('cron-context').value = job.context || '';
+  // PRD Phase 1: load goal fields. Accept either casing — old YAML may have
+  // success_criteria as a list (legacy); the parser already coalesces those
+  // into successCriteriaText on read, but defend here too in case the API
+  // shape differs from what the parser produces.
+  var sctE = document.getElementById('cron-success-criteria-text');
+  if (sctE) {
+    var sctVal = job.successCriteriaText || job.success_criteria_text || '';
+    if (!sctVal && Array.isArray(job.successCriteria || job.success_criteria)) {
+      sctVal = (job.successCriteria || job.success_criteria || []).join('\\n');
+    }
+    sctE.value = sctVal;
+  }
+  var sscE = document.getElementById('cron-success-schema');
+  if (sscE) {
+    var sscObj = job.successSchema || job.success_schema;
+    sscE.value = (sscObj && typeof sscObj === 'object') ? JSON.stringify(sscObj, null, 2) : '';
+  }
+  var addDirsE = document.getElementById('cron-add-dirs');
+  if (addDirsE) {
+    var addDirsArr = Array.isArray(job.addDirs) ? job.addDirs
+      : (Array.isArray(job.add_dirs) ? job.add_dirs : []);
+    addDirsE.value = addDirsArr.join('\\n');
+  }
+  if (typeof updateGoalWarning === 'function') updateGoalWarning();
   document.getElementById('cron-training-section').style.display = 'none';
   document.getElementById('cron-train-btn').style.display = '';
   resetCronTrainingChat();
@@ -24979,6 +25118,11 @@ function captureCronModalSnapshot() {
     v('cron-workdir'),
     v('cron-allowed-tools'),
     v('cron-category'),
+    // PRD Phase 1 goal fields — included in dirty check so leaving the
+    // modal with an unsaved success_schema or success_criteria_text prompts.
+    v('cron-success-criteria-text'),
+    v('cron-success-schema'),
+    v('cron-add-dirs'),
     (document.getElementById('cron-predictable') || {}).checked ? '1' : '0',
     JSON.stringify(_cronSelectedSkills || []),
     JSON.stringify(_cronSelectedMcp || []),
@@ -25003,6 +25147,9 @@ function isCronModalDirty() {
     v('cron-workdir'),
     v('cron-allowed-tools'),
     v('cron-category'),
+    v('cron-success-criteria-text'),
+    v('cron-success-schema'),
+    v('cron-add-dirs'),
     (document.getElementById('cron-predictable') || {}).checked ? '1' : '0',
     JSON.stringify(_cronSelectedSkills || []),
     JSON.stringify(_cronSelectedMcp || []),
@@ -25177,6 +25324,34 @@ async function saveCronJob() {
   }
   if (!prompt) { toast('Prompt is required — tell the agent what to do', 'error'); document.getElementById('cron-prompt').focus(); return; }
 
+  // PRD Phase 1 goal fields. successCriteriaText is freeform; successSchema
+  // is parsed JSON. Validate JSON early so the user gets a clean error before
+  // the round-trip. Empty schema is fine — we just send {} or undefined.
+  var successCriteriaText = (document.getElementById('cron-success-criteria-text')?.value || '').trim();
+  var successSchemaRaw = (document.getElementById('cron-success-schema')?.value || '').trim();
+  var successSchema;
+  if (successSchemaRaw) {
+    try {
+      successSchema = JSON.parse(successSchemaRaw);
+      if (!successSchema || typeof successSchema !== 'object' || Array.isArray(successSchema)) {
+        toast('Success schema must be a JSON object', 'error');
+        document.getElementById('cron-success-schema').focus();
+        return;
+      }
+    } catch (e) {
+      toast('Success schema is not valid JSON: ' + (e.message || String(e)), 'error');
+      document.getElementById('cron-success-schema').focus();
+      return;
+    }
+  }
+  // add_dirs: one absolute path per line. Trim, dedupe, drop blanks.
+  var addDirsRaw = (document.getElementById('cron-add-dirs')?.value || '').split(/\\r?\\n/);
+  var addDirs = addDirsRaw.map(function(s){ return s.trim(); }).filter(Boolean);
+  // Quick sanity — warn but don't block on relative paths.
+  if (addDirs.some(function(p){ return !p.startsWith('/') && !p.startsWith('~'); })) {
+    toast('Heads up: add_dirs entries should be absolute paths.', 'info');
+  }
+
   const body = {
     name, schedule, tier, prompt, enabled: true,
     work_dir: work_dir || undefined, mode, max_hours, max_retries, after, context,
@@ -25194,6 +25369,10 @@ async function saveCronJob() {
     tags: editingCronJob ? _cronTags : (_cronTags.length ? _cronTags : undefined),
     category: editingCronJob ? (category || '') : category,
     predictable,
+    // PRD Phase 1 goal-orientation. PUT delete-on-empty pattern below.
+    successCriteriaText: editingCronJob ? successCriteriaText : (successCriteriaText || undefined),
+    successSchema: editingCronJob ? (successSchema || null) : (successSchema || undefined),
+    addDirs: editingCronJob ? addDirs : (addDirs.length ? addDirs : undefined),
   };
 
   var wasEditing = !!editingCronJob;

package/dist/gateway/cron-scheduler.js

@@ -113,6 +113,24 @@ export function parseCronJobs() {
         const successCriteria = Array.isArray(job.success_criteria)
             ? job.success_criteria.map(c => String(c))
             : undefined;
+        // PRD Phase 1: prefer success_criteria_text (free-form). On read, fall
+        // back to joining the legacy success_criteria string[] so legacy YAML
+        // keeps rendering in the new editor surface. Writes go to the new field.
+        let successCriteriaText = typeof job.success_criteria_text === 'string'
+            ? String(job.success_criteria_text)
+            : (typeof job.successCriteriaText === 'string' ? String(job.successCriteriaText) : undefined);
+        if (!successCriteriaText && Array.isArray(successCriteria) && successCriteria.length > 0) {
+            successCriteriaText = successCriteria.join('\n');
+        }
+        // PRD Phase 1: JSON Schema validated against ResultMessage.structured_output.
+        // Accept either snake_case (success_schema) or camelCase from API. Stored
+        // as a plain object; ajv is loaded lazily at validation time.
+        const successSchemaRaw = job.success_schema ?? job.successSchema;
+        const successSchema = (successSchemaRaw && typeof successSchemaRaw === 'object' && !Array.isArray(successSchemaRaw))
+            ? successSchemaRaw
+            : undefined;
+        // PRD Phase 1: read scope beyond cwd. Accept either casing.
+        const addDirs = normalizeStringArray(job.add_dirs ?? job.addDirs);
         const alwaysDeliver = job.always_deliver === true ? true : undefined;
         const context = job.context != null ? String(job.context) : undefined;
         const preCheck = job.pre_check != null ? String(job.pre_check) : undefined;
@@ -140,7 +158,8 @@ export function parseCronJobs() {
         }
         jobs.push({
             name, schedule, prompt, enabled, tier, maxTurns, model, workDir, mode,
-            maxHours, maxRetries, after, successCriteria, alwaysDeliver, context, preCheck, agentSlug,
+            maxHours, maxRetries, after, successCriteria, successCriteriaText, successSchema, addDirs,
+            alwaysDeliver, context, preCheck, agentSlug,
             skills, allowedTools, allowedMcpServers, tags, category, predictable,
         });
     }
@@ -187,6 +206,18 @@ export function parseAgentCronJobs(agentsDir) {
                 const successCriteria = Array.isArray(job.success_criteria)
                     ? job.success_criteria.map(c => String(c))
                     : undefined;
+                // PRD Phase 1 fields — symmetric with global parser above.
+                let successCriteriaText = typeof job.success_criteria_text === 'string'
+                    ? String(job.success_criteria_text)
+                    : (typeof job.successCriteriaText === 'string' ? String(job.successCriteriaText) : undefined);
+                if (!successCriteriaText && Array.isArray(successCriteria) && successCriteria.length > 0) {
+                    successCriteriaText = successCriteria.join('\n');
+                }
+                const successSchemaRaw = job.success_schema ?? job.successSchema;
+                const successSchema = (successSchemaRaw && typeof successSchemaRaw === 'object' && !Array.isArray(successSchemaRaw))
+                    ? successSchemaRaw
+                    : undefined;
+                const addDirs = normalizeStringArray(job.add_dirs ?? job.addDirs);
                 const context = job.context != null ? String(job.context) : undefined;
                 const preCheck = job.pre_check != null ? String(job.pre_check) : undefined;
                 // ── Trick capabilities — symmetric with global parser ─────────
@@ -210,7 +241,9 @@ export function parseAgentCronJobs(agentsDir) {
                 allJobs.push({
                     name: `${slug}:${name}`,
                     schedule, prompt, enabled, tier, maxTurns, model, workDir,
-                    mode, maxHours, maxRetries, after, successCriteria, context, preCheck,
+                    mode, maxHours, maxRetries, after,
+                    successCriteria, successCriteriaText, successSchema, addDirs,
+                    context, preCheck,
                     agentSlug: slug,
                     skills, allowedTools, allowedMcpServers, tags, category, predictable,
                 });

package/dist/gateway/job-health.js

@@ -85,12 +85,20 @@ export function classifyRunHealth(entry) {
             requiresApproval: true,
         };
     }
-    if (entry.status === 'error' || entry.status === 'retried') {
+    // 'timeout' and 'lost' are explicit terminal failure states (1.18.76+).
+    // 'timeout' = the job exceeded max_hours. 'lost' = the daemon-boot sweep
+    // closed an orphaned 'running' entry that never got a finishedAt because
+    // the daemon crashed mid-run.
+    if (entry.status === 'error' || entry.status === 'retried' || entry.status === 'timeout' || entry.status === 'lost') {
         return {
             ...base,
             status: 'failed',
             evidence: compactEvidence(entry.error, entry.outputPreview, terminalReason ? `terminalReason=${terminalReason}` : undefined),
-            recommendedAction: 'Inspect the latest error and current job definition before applying a fix.',
+            recommendedAction: entry.status === 'lost'
+                ? 'The daemon crashed before this run completed. Check the daemon logs around the startedAt timestamp; if this repeats, lower max_hours or investigate the underlying job.'
+                : entry.status === 'timeout'
+                    ? 'The job exceeded its time budget. Lower max_hours, narrow the prompt, or split into smaller steps.'
+                    : 'Inspect the latest error and current job definition before applying a fix.',
             requiresApproval: true,
         };
     }

package/dist/types.d.ts

@@ -328,7 +328,22 @@ export interface CronJobDefinition {
     maxRetries?: number;
     after?: string;
     agentSlug?: string;
+    /** @deprecated Use successCriteriaText (free-text) or successSchema (JSON Schema)
+     *  per PRD Phase 1. successCriteria is kept readable for one release; on read,
+     *  parseCronJobs coalesces it into successCriteriaText. */
     successCriteria?: string[];
+    /** PRD Phase 1: free-text "this task is done when…". An evaluator sub-agent reads
+     *  the run's final state and the criterion and emits a pass/fail with reasoning.
+     *  Stored as RunEvaluation on the Run. Optional but recommended. */
+    successCriteriaText?: string;
+    /** PRD Phase 1: JSON Schema validated against ResultMessage.structured_output.
+     *  If it parses, the run is mechanically successful. The Task editor shows a
+     *  non-blocking "Goal not set" warning when neither this nor successCriteriaText
+     *  is present. */
+    successSchema?: Record<string, unknown>;
+    /** PRD Phase 1: read scope beyond the cwd (workDir). Surfaced as a chip list
+     *  in the editor's Scope tab. The runner passes these to the SDK as add_dirs. */
+    addDirs?: string[];
     alwaysDeliver?: boolean;
     context?: string;
     preCheck?: string;
@@ -392,8 +407,17 @@ export type TerminalReason = 'blocking_limit' | 'rapid_refill_breaker' | 'prompt
 export interface CronRunEntry {
     jobName: string;
     startedAt: string;
-    finishedAt: string;
-    status: 'ok' | 'error' | 'retried' | 'skipped';
+    /** Optional: in-progress runs are appended with status='running' before the
+     *  finishedAt is known. The runner replaces or supersedes the entry on
+     *  completion. The stale-running sweep emits a closing 'lost' entry for
+     *  any 'running' row whose startedAt has aged past the deadline. */
+    finishedAt?: string;
+    /** 'ok' | 'error' | 'retried' | 'skipped' are terminal. 'running' is in-flight.
+     *  'timeout' fires when max_hours is exceeded. 'lost' is appended by the
+     *  daemon-boot stale sweep when a 'running' entry has no companion close
+     *  (daemon likely crashed mid-run). */
+    status: 'ok' | 'error' | 'retried' | 'skipped' | 'running' | 'timeout' | 'lost';
+    /** 0 for in-progress 'running' rows; populated when terminal. */
     durationMs: number;
     error?: string;
     errorType?: 'transient' | 'permanent';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.18.75",
+  "version": "1.18.77",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",