clementine-agent 1.18.74 → 1.18.76

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 };
@@ -4267,6 +4260,10 @@ export async function cmdDashboard(opts) {
             const agentSlug = body.agent ? (String(body.agent).trim() || undefined) : undefined;
             // Parse the agent's draft if present; on any parse error fall back to the
             // single-step stub so the user lands in the editor with something usable.
+            // ── KEY FIX (1.18.75): forward EVERY step-kind body the agent produces.
+            // Previously this parser dropped mcp/cli/channel/transform/conditional/loop
+            // configs, leaving the user with prompt-only stubs and forcing them to
+            // rebuild every Slack-send / shell-cmd / branch by hand.
             let steps = null;
             if (body.draftYaml && typeof body.draftYaml === 'string') {
                 try {
@@ -4277,16 +4274,39 @@ export async function cmdDashboard(opts) {
                             const dependsOn = Array.isArray(r.dependsOn)
                                 ? r.dependsOn.map(String)
                                 : (typeof r.dependsOn === 'string' ? r.dependsOn.split(',').map(s => s.trim()).filter(Boolean) : []);
-                            return {
+                            const kind = typeof r.kind === 'string' ? r.kind : 'prompt';
+                            const step = {
                                 id: String(id),
-                                prompt: String(r.prompt ?? ''),
                                 dependsOn,
                                 tier: typeof r.tier === 'number' ? r.tier : 1,
                                 maxTurns: typeof r.maxTurns === 'number' ? r.maxTurns : 15,
-                                ...(typeof r.model === 'string' ? { model: r.model } : {}),
-                                ...(typeof r.workDir === 'string' ? { workDir: r.workDir } : {}),
-                                ...(typeof r.kind === 'string' && r.kind !== 'prompt' ? { kind: r.kind } : {}),
                             };
+                            if (kind !== 'prompt')
+                                step.kind = kind;
+                            if (typeof r.model === 'string')
+                                step.model = r.model;
+                            if (typeof r.workDir === 'string')
+                                step.workDir = r.workDir;
+                            // Prompt body — present on prompt steps and optionally on others.
+                            if (r.prompt != null)
+                                step.prompt = String(r.prompt);
+                            else if (kind === 'prompt')
+                                step.prompt = '';
+                            // Non-prompt step bodies. We forward them as-is when shaped like
+                            // an object; the validator on save will catch missing required
+                            // sub-fields and surface a structured error to the chat UI.
+                            const passThrough = (key) => {
+                                const v = r[key];
+                                if (v && typeof v === 'object' && !Array.isArray(v))
+                                    step[key] = v;
+                            };
+                            passThrough('mcp');
+                            passThrough('cli');
+                            passThrough('channel');
+                            passThrough('transform');
+                            passThrough('conditional');
+                            passThrough('loop');
+                            return step;
                         }).filter(s => s.id);
                         if (steps.length === 0)
                             steps = null;
@@ -4570,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'],
@@ -6557,8 +6596,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) });
@@ -6576,7 +6625,52 @@ If the tool returns nothing or errors, return an empty array \`[]\`.`,
             }
             jobs.splice(idx, 1);
             writeCronFileAt(cronFile, parsed, jobs);
-            res.json({ ok: true, message: `Deleted cron job: ${jobName}` });
+            // Cascade cleanup unless explicitly opted out (?purge=false). Storing the
+            // job name's safe filename form once — same sanitization the run-log and
+            // trace writers use, so we hit the right files.
+            const purge = String(req.query.purge ?? 'true') !== 'false';
+            const purged = [];
+            if (purge) {
+                const safe = bareJobName.replace(/[^a-zA-Z0-9_-]/g, '_');
+                const runLog = path.join(BASE_DIR, 'cron', 'runs', `${safe}.jsonl`);
+                try {
+                    if (existsSync(runLog)) {
+                        unlinkSync(runLog);
+                        purged.push('runs.jsonl');
+                    }
+                }
+                catch { /* non-fatal */ }
+                try {
+                    const traceDir = path.join(BASE_DIR, 'cron', 'traces');
+                    if (existsSync(traceDir)) {
+                        const traceFiles = readdirSync(traceDir).filter(f => f.startsWith(`${safe}_`) && f.endsWith('.json'));
+                        for (const f of traceFiles) {
+                            try {
+                                unlinkSync(path.join(traceDir, f));
+                            }
+                            catch { /* skip */ }
+                        }
+                        if (traceFiles.length > 0)
+                            purged.push(`${traceFiles.length} trace${traceFiles.length === 1 ? '' : 's'}`);
+                    }
+                }
+                catch { /* non-fatal */ }
+                try {
+                    const uploadsDir = path.join(BASE_DIR, 'uploads', `cron-${safe}`);
+                    if (existsSync(uploadsDir)) {
+                        rmSync(uploadsDir, { recursive: true, force: true });
+                        purged.push('attachments');
+                    }
+                }
+                catch { /* non-fatal */ }
+            }
+            // Notify other dashboard tabs so they drop the card without polling.
+            try {
+                broadcastEvent({ type: 'cron_deleted', data: { job: bareJobName, purged } });
+            }
+            catch { /* non-fatal */ }
+            const purgeNote = purged.length > 0 ? ` (purged: ${purged.join(', ')})` : '';
+            res.json({ ok: true, message: `Deleted cron job: ${jobName}${purgeNote}`, purged });
         }
         catch (err) {
             res.status(500).json({ error: String(err) });
@@ -16422,7 +16516,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             apiFetch('/api/routines/' + encodeURIComponent(id) + '/toggle', { method: 'POST' })
               .then(function(r){ return r.json(); })
               .then(function(){ R.refreshList(); })
-              .catch(function(err){ alert('Toggle failed: ' + err); });
+              .catch(function(err){ toast('Toggle failed: ' + err, 'error'); });
           },
           run: function(id, approvedSideEffects) {
             apiFetch('/api/routines/' + encodeURIComponent(id) + '/run', {
@@ -16438,19 +16532,19 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               }
               return r.json().then(function(j){
                 if (j.ok) R.flash('Triggered.');
-                else alert('Run failed: ' + (j.error || 'unknown'));
+                else toast('Run failed: ' + (j.error || 'unknown'), 'error');
               });
-            }).catch(function(err){ alert('Run failed: ' + err); });
+            }).catch(function(err){ toast('Run failed: ' + err, 'error'); });
           },
           // ── editor ──────────────────────────────────────────────────
           openEditor: function(id) {
             apiFetch('/api/routines/' + encodeURIComponent(id))
               .then(function(r){ return r.json(); })
               .then(function(data){
-                if (!data || !data.routine) { alert('Failed to load workflow'); return; }
+                if (!data || !data.routine) { toast('Failed to load workflow', 'error'); return; }
                 R.state.editing = { id: data.id, routine: data.routine, dirty: false, validation: data.validation };
                 R.showEditor();
-              }).catch(function(err){ alert('Open failed: ' + err); });
+              }).catch(function(err){ toast('Open failed: ' + err, 'error'); });
           },
           showEditor: function() {
             document.getElementById('routines-list-pane').style.display = 'none';
@@ -16702,7 +16796,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           },
           removeStep: function(idx) {
             if (!R.state.editing) return;
-            if (R.state.editing.routine.steps.length <= 1) { alert('A workflow must have at least one step.'); return; }
+            if (R.state.editing.routine.steps.length <= 1) { toast('A workflow must have at least one step.', 'error'); return; }
             if (!confirm('Remove this step?')) return;
             var removed = R.state.editing.routine.steps.splice(idx, 1)[0];
             // Strip lingering dependsOn references.
@@ -16803,7 +16897,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
                 (d.steps || []).forEach(function(s){ lines.push('• ' + s.description + (s.warnings.length ? '\\n   ⚠ ' + s.warnings.join('; ') : '')); });
                 if (d.notes && d.notes.length) lines.push('\\n' + d.notes.join('\\n'));
                 alert(lines.join('\\n'));
-              }).catch(function(err){ alert('Dry-run failed: ' + err); });
+              }).catch(function(err){ toast('Dry-run failed: ' + err, 'error'); });
           },
           testCurrent: function() {
             if (!R.state.editing) return;
@@ -16822,9 +16916,9 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             apiFetch('/api/routines/' + encodeURIComponent(R.state.editing.id), { method: 'DELETE' })
               .then(function(r){ return r.json(); })
               .then(function(j){
-                if (j.ok) { R.state.editing = null; R.closeEditor(); R.refreshList(); }
-                else alert('Delete failed: ' + (j.error || 'unknown'));
-              }).catch(function(err){ alert('Delete error: ' + err); });
+                if (j.ok) { R.state.editing = null; R.closeEditor(); R.refreshList(); toast('Deleted.', 'success'); }
+                else toast('Delete failed: ' + (j.error || 'unknown'), 'error');
+              }).catch(function(err){ toast('Delete error: ' + err, 'error'); });
           },
           setStatus: function(msg) {
             var el = document.getElementById('re-status');
@@ -16879,7 +16973,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           },
           submitCreate: function() {
             var name = document.getElementById('routines-create-name').value.trim();
-            if (!name) { alert('Name is required'); return; }
+            if (!name) { toast('Name is required', 'error'); document.getElementById('routines-create-name').focus(); return; }
             var body = {
               name: name,
               description: document.getElementById('routines-create-description').value.trim(),
@@ -16892,11 +16986,12 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               body: JSON.stringify(body)
             }).then(function(r){ return r.json().then(function(j){ return { ok: r.ok, body: j }; }); })
               .then(function(res){
-                if (!res.ok) { alert('Create failed: ' + (res.body.error || 'unknown')); return; }
+                if (!res.ok) { toast('Create failed: ' + (res.body.error || 'unknown'), 'error'); return; }
                 R.closeCreate();
                 R.refreshList();
                 R.openEditor(res.body.id);
-              }).catch(function(err){ alert('Create error: ' + err); });
+                toast('Workflow created.', 'success');
+              }).catch(function(err){ toast('Create error: ' + err, 'error'); });
           },
           // ── chat-first builder ──────────────────────────────────────
           // Multi-turn conversation that asks clarifying questions and
@@ -17153,14 +17248,15 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               .then(function(res){
                 if (btn) { btn.textContent = 'Save workflow'; btn.disabled = false; }
                 if (!res.ok) {
-                  alert('Save failed: ' + (res.body && res.body.error || 'unknown'));
+                  toast('Save failed: ' + (res.body && res.body.error || 'unknown'), 'error');
                   return;
                 }
+                toast('Workflow saved.', 'success');
                 R.closeChat();
                 if (res.body && res.body.id) R.openEditor(res.body.id);
               }).catch(function(err){
                 if (btn) { btn.textContent = 'Save workflow'; btn.disabled = false; }
-                alert('Save failed: ' + err);
+                toast('Save failed: ' + err, 'error');
               });
           },
           // ── helpers ─────────────────────────────────────────────────
@@ -19950,7 +20046,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           <div class="cap-section">
             <label class="cap-section-label">Tags</label>
             <div class="cap-picker-chips" id="cron-tags-chips"></div>
-            <input type="text" class="cap-tag-input" id="cron-tags-input" placeholder="Type a tag and press Enter (e.g. morning, ops)" onkeydown="handleTagInputKeydown(event)">
+            <input type="text" class="cap-tag-input" id="cron-tags-input" placeholder="Type a tag and press Enter, Tab, or comma (e.g. morning, ops)" onkeydown="handleTagInputKeydown(event)" onblur="handleTagInputBlur()">
           </div>
         </div>
 
@@ -23367,7 +23463,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;
     }
@@ -23419,7 +23515,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) ──
@@ -24472,8 +24568,24 @@ function toggleAllowedToolsPanel() {
 }
 
 function handleTagInputKeydown(event) {
-  if (event.key !== 'Enter' && event.key !== ',') return;
+  // Accept Enter, Tab, and comma as commit keys. Tab still moves focus
+  // afterward when the field is empty (no preventDefault). On-blur
+  // commit is handled by handleTagInputBlur below.
+  if (event.key !== 'Enter' && event.key !== ',' && event.key !== 'Tab') return;
+  var inp = document.getElementById('cron-tags-input');
+  if (!inp) return;
+  var val = inp.value.trim().replace(/^#+/, '');
+  if (!val) return;  // Tab on empty input → let it move focus naturally
   event.preventDefault();
+  if (_cronTags.indexOf(val) === -1) {
+    _cronTags.push(val);
+    renderTagsPickerChips();
+  }
+  inp.value = '';
+}
+
+function handleTagInputBlur() {
+  // Catch tags the user typed but forgot to commit before clicking elsewhere.
   var inp = document.getElementById('cron-tags-input');
   if (!inp) return;
   var val = inp.value.trim().replace(/^#+/, '');
@@ -24669,6 +24781,9 @@ function openCreateCronModal(agentSlug) {
   switchCronTab('configure');
   onPredictableChange();
   document.getElementById('cron-modal').classList.add('show');
+  // Snapshot AFTER all defaults are populated so an immediate close with no
+  // edits doesn't trigger the "discard changes?" prompt.
+  setTimeout(captureCronModalSnapshot, 0);
 }
 
 function openEditCronModal(jobName) {
@@ -24730,6 +24845,7 @@ function openEditCronModal(jobName) {
   if (previewBtn) previewBtn.removeAttribute('disabled');
   switchCronTab('configure');
   document.getElementById('cron-modal').classList.add('show');
+  setTimeout(captureCronModalSnapshot, 0);
 }
 
 /**
@@ -24864,10 +24980,68 @@ function renderCronPreview(d) {
   return html;
 }
 
-function closeCronModal() {
+// Snapshot of the form values at the moment the modal opened. closeCronModal
+// compares against this and prompts the user before discarding edits.
+var _cronModalSnapshot = null;
+
+function captureCronModalSnapshot() {
+  // String concatenation is enough for a dirty check; we don't need the
+  // structured object back. Order must match restoreCheck below.
+  function v(id) { var el = document.getElementById(id); return el ? (el.value || '') : ''; }
+  _cronModalSnapshot = [
+    v('cron-name'),
+    v('cron-schedule'),
+    v('cron-prompt'),
+    v('cron-context'),
+    v('cron-tier'),
+    v('cron-mode'),
+    v('cron-maxhours'),
+    v('cron-max-retries'),
+    v('cron-after'),
+    v('cron-workdir'),
+    v('cron-allowed-tools'),
+    v('cron-category'),
+    (document.getElementById('cron-predictable') || {}).checked ? '1' : '0',
+    JSON.stringify(_cronSelectedSkills || []),
+    JSON.stringify(_cronSelectedMcp || []),
+    JSON.stringify(_cronTags || []),
+    String((_pendingAttachments || []).length),
+  ].join('\\u0001');
+}
+
+function isCronModalDirty() {
+  if (_cronModalSnapshot === null) return false;
+  function v(id) { var el = document.getElementById(id); return el ? (el.value || '') : ''; }
+  var current = [
+    v('cron-name'),
+    v('cron-schedule'),
+    v('cron-prompt'),
+    v('cron-context'),
+    v('cron-tier'),
+    v('cron-mode'),
+    v('cron-maxhours'),
+    v('cron-max-retries'),
+    v('cron-after'),
+    v('cron-workdir'),
+    v('cron-allowed-tools'),
+    v('cron-category'),
+    (document.getElementById('cron-predictable') || {}).checked ? '1' : '0',
+    JSON.stringify(_cronSelectedSkills || []),
+    JSON.stringify(_cronSelectedMcp || []),
+    JSON.stringify(_cronTags || []),
+    String((_pendingAttachments || []).length),
+  ].join('\\u0001');
+  return current !== _cronModalSnapshot;
+}
+
+function closeCronModal(force) {
+  if (force !== true && isCronModalDirty()) {
+    if (!confirm('You have unsaved changes. Discard them?')) return;
+  }
   document.getElementById('cron-modal').classList.remove('show');
   editingCronJob = null;
   _cronPreviewLoadedFor = null;
+  _cronModalSnapshot = null;
   var attachList = document.getElementById('cron-attachments-list');
   if (attachList) attachList.innerHTML = '';
   var bannerHost = document.getElementById('cron-legacy-banner-host');
@@ -25001,10 +25175,29 @@ async function saveCronJob() {
   const allowedTools = parseAllowedToolsRaw();
   const predictable = !!document.getElementById('cron-predictable')?.checked;
 
-  if (!name || !schedule || !prompt) {
-    toast('Please fill in all fields', 'error');
+  // Field-specific validation. Toasting one message per problem is more
+  // actionable than the old "Please fill in all fields" — and we bail before
+  // hitting the API so the user sees the issue without a round-trip.
+  if (!name) { toast('Task name is required', 'error'); document.getElementById('cron-name').focus(); return; }
+  if (!/^[a-z][a-z0-9-]{0,63}$/.test(name)) {
+    toast('Task name must start with a lowercase letter and contain only a–z, 0–9, and hyphens (max 64 chars)', 'error');
+    document.getElementById('cron-name').focus();
+    return;
+  }
+  if (!editingCronJob && Array.isArray(cronJobsData)) {
+    var dup = cronJobsData.find(function(j) { return String(j.name || '').toLowerCase() === name.toLowerCase(); });
+    if (dup) { toast('A task named "' + name + '" already exists', 'error'); document.getElementById('cron-name').focus(); return; }
+  }
+  if (!schedule) { toast('Schedule is required', 'error'); return; }
+  // Light client-side cron sanity check — server uses real cron-parser.
+  // Catches obvious garbage like "foo bar" without making a round-trip.
+  // Note: the hyphen sits at the END of the character class to avoid being
+  // interpreted as a range. \\\\s in source → \\s in served JS → \s in regex.
+  if (!/^([0-9*/, -]+|@(yearly|annually|monthly|weekly|daily|hourly|reboot))$/i.test(schedule) && schedule.split(/\\s+/).length < 5) {
+    toast('Schedule does not look like a valid cron expression', 'error');
     return;
   }
+  if (!prompt) { toast('Prompt is required — tell the agent what to do', 'error'); document.getElementById('cron-prompt').focus(); return; }
 
   const body = {
     name, schedule, tier, prompt, enabled: true,
@@ -25026,12 +25219,18 @@ async function saveCronJob() {
   };
 
   var wasEditing = !!editingCronJob;
+  // Don't celebrate before the round-trip lands. apiJson toasts on its own;
+  // we just check the {ok} flag and bail so the modal doesn't close (and
+  // erase the user's input) on a 400/409/500.
+  var resp;
   if (editingCronJob) {
-    await apiJson('PUT', '/api/cron/' + encodeURIComponent(editingCronJob), body);
+    resp = await apiJson('PUT', '/api/cron/' + encodeURIComponent(editingCronJob), body);
+    if (!resp || resp.ok !== true) return;
     if (_pendingAttachments.length > 0) await uploadPendingAttachments(editingCronJob);
   } else {
     var createBody = _cronAgentContext ? Object.assign({}, body, { agent: _cronAgentContext }) : body;
-    await apiJson('POST', '/api/cron', createBody);
+    resp = await apiJson('POST', '/api/cron', createBody);
+    if (!resp || resp.ok !== true) return;
     var attachJobName = _cronAgentContext ? (_cronAgentContext + ':' + name) : name;
     if (_pendingAttachments.length > 0) await uploadPendingAttachments(attachJobName);
   }
@@ -25041,11 +25240,14 @@ async function saveCronJob() {
   // confirm what they just saved actually runs the way they intended.
   // This is the close-the-loop UX move that makes Predictable Mode visible.
   markCronPreviewDirty();
+  // Re-snapshot now that the form matches what's on disk — prevents the
+  // dirty-guard from firing if the user closes the modal without further edits.
+  captureCronModalSnapshot();
   if (wasEditing) {
     toast('Saved. Showing what will run…', 'success');
     switchCronTab('preview');
   } else {
-    closeCronModal();
+    closeCronModal(true);  // force; we just saved, no dirty prompt
   }
 }
 
@@ -34500,6 +34702,12 @@ try {
         if (currentPage === 'build') refreshCron();
         refreshTeamNav();
       }
+      // A delete on one tab should drop the card from every open dashboard
+      // without waiting for the next poll. cron_toggled is similar but lighter.
+      if (evt.type === 'cron_deleted' || evt.type === 'cron_toggled') {
+        if (currentPage === 'build') refreshCron();
+        refreshTeamNav();
+      }
       if (evt.type === 'agent_created' || evt.type === 'agent_updated' || evt.type === 'agent_deleted' || evt.type === 'agent_status') {
         refreshTeamNav();
         refreshActivity();

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

@@ -392,8 +392,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.74",
+  "version": "1.18.76",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",