clementine-agent 1.18.121 → 1.18.123

package/dist/cli/dashboard.js

@@ -14419,6 +14419,9 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
   }
   .toast.success { border-left: 3px solid var(--green); }
   .toast.error { border-left: 3px solid var(--red); }
+  /* 1.18.122 — variants used by callsites that lacked CSS support */
+  .toast.warn { border-left: 3px solid var(--yellow); }
+  .toast.info { border-left: 3px solid var(--accent); }
   @keyframes toastIn {
     from { transform: translateX(40px); opacity: 0; }
     to { transform: translateX(0); opacity: 1; }
@@ -19466,11 +19469,11 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           clearInterval(ticker);
 
           if (errorMsg) {
-            manifestEl.innerHTML = '<div style="color:#e66">Error: ' + escapeHtml(errorMsg) + '</div>';
+            manifestEl.innerHTML = '<div style="color:var(--red)">Error: ' + escapeHtml(errorMsg) + '</div>';
             return;
           }
           if (!manifestData || !finalData) {
-            manifestEl.innerHTML = '<div style="color:#e66">Preview ended without a result. Check dashboard logs.</div>';
+            manifestEl.innerHTML = '<div style="color:var(--red)">Preview ended without a result. Check dashboard logs.</div>';
             return;
           }
 
@@ -19480,13 +19483,13 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           const manifestRows = Object.entries(manifest.formats || {})
             .map(([fmt, n]) => '<tr><td>' + escapeHtml(fmt) + '</td><td>' + n + '</td></tr>').join('');
           const warnBlock = errorsList.length
-            ? '<div style="margin-top:10px;padding:10px;background:#fff3cd;border:1px solid #f0c36d;border-radius:6px;color:#8a5a00;font-size:13px">' +
+            ? '<div style="margin-top:10px;padding:10px;background:rgba(245,158,11,0.10);border:1px solid rgba(245,158,11,0.32);border-radius:6px;color:var(--text-primary);font-size:13px">' +
               '<div style="font-weight:600;margin-bottom:4px">' + errorsList.length + ' file(s) could not be ingested</div>' +
               errorsList.map((e) => '<div style="font-family:monospace;font-size:12px">• ' + escapeHtml(e.error) + '</div>').join('') +
               '</div>'
             : '';
           const emptyNote = (preview.length === 0 && !errorsList.length)
-            ? '<div style="margin-top:10px;padding:10px;background:#fff3cd;border:1px solid #f0c36d;border-radius:6px;color:#8a5a00;font-size:13px">No records extracted. The file may be empty or in an unsupported format.</div>'
+            ? '<div style="margin-top:10px;padding:10px;background:rgba(245,158,11,0.10);border:1px solid rgba(245,158,11,0.32);border-radius:6px;color:var(--text-primary);font-size:13px">No records extracted. The file may be empty or in an unsupported format.</div>'
             : '';
           manifestEl.innerHTML =
             '<div class="card" style="padding:12px"><div style="font-weight:600;margin-bottom:8px">Manifest</div>' +
@@ -19501,7 +19504,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               '<div style="font-weight:600">#' + (i + 1) + ' ' + escapeHtml(p.title || '(untitled)') + '</div>' +
               '<div style="color:var(--muted);font-size:12px;margin:4px 0">' + escapeHtml(p.targetRelPath || '') + '</div>' +
               '<div style="font-size:13px">' + escapeHtml((p.body || '').slice(0, 400)) + '</div>' +
-              (p.tags && p.tags.length ? '<div style="margin-top:6px;font-size:12px;color:#888">tags: ' + p.tags.map(escapeHtml).join(', ') + '</div>' : '') +
+              (p.tags && p.tags.length ? '<div style="margin-top:6px;font-size:12px;color:var(--text-muted)">tags: ' + p.tags.map(escapeHtml).join(', ') + '</div>' : '') +
               '</div>').join('');
             previewEl.innerHTML =
               '<div style="font-weight:600;margin-bottom:8px">Preview (first ' + Math.min(preview.length, 10) + ' records, dry-run)</div>' + previewHtml;
@@ -19549,11 +19552,11 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           }
 
           if (errorMsg) {
-            progEl.innerHTML = '<div style="color:#e66">Error: ' + escapeHtml(errorMsg) + '</div>';
+            progEl.innerHTML = '<div style="color:var(--red)">Error: ' + escapeHtml(errorMsg) + '</div>';
             return;
           }
           if (!finalData) {
-            progEl.innerHTML = '<div style="color:#e66">Ingestion ended without a result. Check dashboard logs.</div>';
+            progEl.innerHTML = '<div style="color:var(--red)">Ingestion ended without a result. Check dashboard logs.</div>';
             return;
           }
           const elapsed = Math.floor((Date.now() - progress.startedAt) / 1000);
@@ -19564,7 +19567,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             ? 'Ingestion complete'
             : 'Ingestion finished, but nothing was written';
           const errBlock = errList.length
-            ? '<div style="margin-top:10px;padding:10px;background:#fff3cd;border:1px solid #f0c36d;border-radius:6px;color:#8a5a00;font-size:13px">' +
+            ? '<div style="margin-top:10px;padding:10px;background:rgba(245,158,11,0.10);border:1px solid rgba(245,158,11,0.32);border-radius:6px;color:var(--text-primary);font-size:13px">' +
               '<div style="font-weight:600;margin-bottom:4px">' + errList.length + ' error(s)</div>' +
               errList.map((e) => '<div style="font-family:monospace;font-size:12px">• ' + escapeHtml(e.error) + '</div>').join('') +
               '</div>'
@@ -19655,8 +19658,8 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             }
             el.innerHTML = data.integrations.map(function(i) {
               const ok = i.connected && i.hasFeedReadyTools;
-              const color = ok ? '#2f7d32' : '#8a5a00';
-              const bg = ok ? '#e8f5e9' : '#fff3cd';
+              const color = ok ? 'var(--green)' : 'var(--text-primary)';
+              const bg = ok ? 'rgba(34,197,94,0.10)' : 'rgba(245,158,11,0.10)';
               const dot = ok ? '✓' : '⚠';
               const source = i.kind === 'composio' ? 'Composio' : (i.kind === 'claude-desktop' ? 'Claude Desktop' : 'MCP');
               const label = ok ? i.label + ' · ' + source : i.label + ' (incomplete in ' + source + ')';
@@ -19685,7 +19688,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
                 : '<span style="color:var(--muted)">no fields</span>';
               return '<div class="card" style="padding:10px 12px;margin-bottom:8px;display:flex;align-items:center;gap:12px">' +
                 '<div style="flex:1">' +
-                '<div style="font-weight:600">' + escapeHtml(f.name) + (f.enabled ? '' : ' <span style="color:#e66;font-weight:normal">(disabled)</span>') + '</div>' +
+                '<div style="font-weight:600">' + escapeHtml(f.name) + (f.enabled ? '' : ' <span style="color:var(--red);font-weight:normal">(disabled)</span>') + '</div>' +
                 '<div style="font-size:12px;color:var(--muted)">' +
                 'Recipe: <code>' + escapeHtml(f.recipeId) + '</code> · Schedule: <code>' + escapeHtml(f.schedule) + '</code> · Target: <code>' + escapeHtml(f.targetFolder) + '</code>' +
                 '</div>' +
@@ -19732,10 +19735,10 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           if (!brainFeedWizardState) return;
           const s = brainFeedWizardState;
           if (s.step === 0) {
-            if (!s.pick) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Pick a connector.</span>'; return; }
+            if (!s.pick) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Pick a connector.</span>'; return; }
             s.step = 1;
           } else if (s.step === 1) {
-            if (!s.recipe) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Pick a recipe.</span>'; return; }
+            if (!s.recipe) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Pick a recipe.</span>'; return; }
             s.values = {};
             for (const f of (s.recipe.fields || [])) {
               if (f.defaultValue) s.values[f.key] = f.defaultValue;
@@ -19752,11 +19755,11 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             const inputs = document.querySelectorAll('#brain-feed-wizard-step [data-field]');
             inputs.forEach(function(inp) { s.values[inp.dataset.field] = inp.value; });
             const missing = (s.recipe.fields || []).filter(function(f) { return f.required && !(s.values[f.key] || '').trim(); });
-            if (missing.length) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Required: ' + missing.map(function(f) { return f.label; }).join(', ') + '</span>'; return; }
+            if (missing.length) { document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Required: ' + missing.map(function(f) { return f.label; }).join(', ') + '</span>'; return; }
             if (s.recipe && s.recipe.id === 'tool-backed-memory-seed') {
               const toolName = String(s.values.toolName || '').trim();
               if (!/^mcp__.+__.+$/.test(toolName)) {
-                document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Pick an exact tool before continuing.</span>';
+                document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Pick an exact tool before continuing.</span>';
                 return;
               }
               const rawVariables = String(s.values.variablesJson || '').trim();
@@ -19764,12 +19767,12 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
                 try {
                   const parsedVariables = JSON.parse(rawVariables);
                   if (!parsedVariables || typeof parsedVariables !== 'object' || Array.isArray(parsedVariables)) {
-                    document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Tool variables must be a JSON object, for example {}.</span>';
+                    document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Tool variables must be a JSON object, for example {}.</span>';
                     return;
                   }
                 } catch (err) {
                   void err;
-                  document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">Tool variables must be valid JSON, for example {}.</span>';
+                  document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">Tool variables must be valid JSON, for example {}.</span>';
                   return;
                 }
               }
@@ -19836,7 +19839,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               // can still set the value manually.
               container.innerHTML =
                 '<input type="text" value="' + escapeHtml(currentVal) + '" placeholder="(type a value)" style="width:100%" oninput="(document.querySelector(\\'input[type=hidden][data-field=' + field.key + ']\\')||{}).value=this.value">' +
-                '<div style="color:#8a5a00;font-size:11px;margin-top:4px">Nothing returned from the probe — type a value manually' + (data.rawPreview ? ' (probe output: ' + escapeHtml(data.rawPreview.slice(0, 120)) + '…)' : '') + '</div>';
+                '<div style="color:var(--text-primary);font-size:11px;margin-top:4px">Nothing returned from the probe — type a value manually' + (data.rawPreview ? ' (probe output: ' + escapeHtml(data.rawPreview.slice(0, 120)) + '…)' : '') + '</div>';
               return;
             }
 
@@ -19862,7 +19865,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           } catch (err) {
             container.innerHTML =
               '<input type="text" value="' + escapeHtml(values[field.key] || '') + '" placeholder="(probe failed — type manually)" style="width:100%" oninput="(document.querySelector(\\'input[type=hidden][data-field=' + field.key + ']\\')||{}).value=this.value">' +
-              '<div style="color:#e66;font-size:11px;margin-top:4px">Picker failed: ' + escapeHtml(String(err && err.message ? err.message : err)) + ' — type a value manually.</div>';
+              '<div style="color:var(--red);font-size:11px;margin-top:4px">Picker failed: ' + escapeHtml(String(err && err.message ? err.message : err)) + ' — type a value manually.</div>';
           }
         }
 
@@ -19888,7 +19891,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
               const rawHint = trimmed && !isEmptyArray
                 ? ' <span style="color:var(--muted)">Tool said: ' + escapeHtml(trimmed.slice(0, 140)) + '</span>'
                 : '';
-              resultsEl.innerHTML = '<div style="color:#8a5a00;font-size:12px;padding:6px">No matches for "' + escapeHtml(query) + '". The tool may be limited by macOS permissions or not support this query — use <a href="#" onclick="brainFieldPickerToggleCustom(\\'' + field.key + '\\', \\'\\');return false">type a value directly</a>.' + rawHint + '</div>';
+              resultsEl.innerHTML = '<div style="color:var(--text-primary);font-size:12px;padding:6px">No matches for "' + escapeHtml(query) + '". The tool may be limited by macOS permissions or not support this query — use <a href="#" onclick="brainFieldPickerToggleCustom(\\'' + field.key + '\\', \\'\\');return false">type a value directly</a>.' + rawHint + '</div>';
               return;
             }
             resultsEl.innerHTML = items.map(function(it) {
@@ -19897,7 +19900,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             }).join('') +
               '<div style="font-size:11px;color:var(--muted);margin-top:4px">' + items.length + ' result' + (items.length === 1 ? '' : 's') + (data.cached ? ' (cached)' : '') + '</div>';
           } catch (err) {
-            resultsEl.innerHTML = '<div style="color:#e66;font-size:12px;padding:6px">' + escapeHtml(String(err && err.message ? err.message : err)) + '</div>';
+            resultsEl.innerHTML = '<div style="color:var(--red);font-size:12px;padding:6px">' + escapeHtml(String(err && err.message ? err.message : err)) + '</div>';
           }
         }
 
@@ -19963,7 +19966,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           let html = '';
           if (s.step === 0) {
             if (!s.connected.length) {
-              html = '<div style="color:#8a5a00">No connectors have feed-ready tools. Connect Composio or open Claude Desktop → Connectors and sign into Google Drive, Outlook, Gmail, or Slack first.</div>';
+              html = '<div style="color:var(--text-primary)">No connectors have feed-ready tools. Connect Composio or open Claude Desktop → Connectors and sign into Google Drive, Outlook, Gmail, or Slack first.</div>';
             } else {
               html = '<div style="display:grid;grid-template-columns:repeat(auto-fill,minmax(180px,1fr));gap:8px">' +
                 s.connected.map(function(i) {
@@ -20026,7 +20029,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
                   } else {
                     control = '<input type="text" data-field="' + f.key + '" value="' + escapeHtml(val) + '" placeholder="' + escapeHtml(f.placeholder || '') + '" style="width:100%">';
                   }
-                  return '<label style="font-weight:500;padding-top:6px">' + escapeHtml(f.label) + (f.required ? ' <span style="color:#e66">*</span>' : '') + '</label>' +
+                  return '<label style="font-weight:500;padding-top:6px">' + escapeHtml(f.label) + (f.required ? ' <span style="color:var(--red)">*</span>' : '') + '</label>' +
                     '<div>' + control + help + '</div>';
                 }).join('') + '</div>';
 
@@ -20091,13 +20094,13 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             });
             const data = await resp.json();
             if (!resp.ok) {
-              document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">' + escapeHtml(data.error || 'save failed') + '</span>';
+              document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">' + escapeHtml(data.error || 'save failed') + '</span>';
               return;
             }
             brainCloseFeedWizard();
             brainLoadFeeds();
           } catch (err) {
-            document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:#e66">' + escapeHtml(String(err)) + '</span>';
+            document.getElementById('brain-feed-wizard-status').innerHTML = '<span style="color:var(--red)">' + escapeHtml(String(err)) + '</span>';
           }
         }
 
@@ -20210,8 +20213,8 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           const slug = document.getElementById('brain-webhook-slug').value.trim();
           const folder = document.getElementById('brain-webhook-folder').value.trim() || ('04-Ingest/' + slug);
           const statusEl = document.getElementById('brain-webhook-status');
-          if (!slug) { statusEl.innerHTML = '<span style="color:#e66">slug required</span>'; return; }
-          if (!/^[a-z][a-z0-9_-]*$/.test(slug)) { statusEl.innerHTML = '<span style="color:#e66">slug must be lowercase alphanumeric</span>'; return; }
+          if (!slug) { statusEl.innerHTML = '<span style="color:var(--red)">slug required</span>'; return; }
+          if (!/^[a-z][a-z0-9_-]*$/.test(slug)) { statusEl.innerHTML = '<span style="color:var(--red)">slug must be lowercase alphanumeric</span>'; return; }
 
           // 1) Generate a random 32-byte secret and save it under ref "webhook_<slug>"
           const bytes = new Uint8Array(32);
@@ -20226,7 +20229,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           });
           if (!credResp.ok) {
             const e = await credResp.json();
-            statusEl.innerHTML = '<span style="color:#e66">Secret save failed: ' + escapeHtml(e.error || 'unknown') + '</span>';
+            statusEl.innerHTML = '<span style="color:var(--red)">Secret save failed: ' + escapeHtml(e.error || 'unknown') + '</span>';
             return;
           }
 
@@ -20246,7 +20249,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           });
           if (!resp.ok) {
             const e = await resp.json();
-            statusEl.innerHTML = '<span style="color:#e66">' + escapeHtml(e.error || 'save failed') + '</span>';
+            statusEl.innerHTML = '<span style="color:var(--red)">' + escapeHtml(e.error || 'save failed') + '</span>';
             return;
           }
 
@@ -20271,7 +20274,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           const cronExpr = document.getElementById('brain-poll-cron').value.trim();
           const folder = document.getElementById('brain-poll-folder').value.trim();
           const statusEl = document.getElementById('brain-poll-status');
-          if (!slug || !url) { statusEl.innerHTML = '<span style="color:#e66">slug and URL required</span>'; return; }
+          if (!slug || !url) { statusEl.innerHTML = '<span style="color:var(--red)">slug and URL required</span>'; return; }
 
           const headers = brainCollectKv('headers');
           const params = brainCollectKv('params');
@@ -20293,7 +20296,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             }),
           });
           const data = await resp.json();
-          if (!resp.ok) { statusEl.innerHTML = '<span style="color:#e66">' + escapeHtml(data.error || 'save failed') + '</span>'; return; }
+          if (!resp.ok) { statusEl.innerHTML = '<span style="color:var(--red)">' + escapeHtml(data.error || 'save failed') + '</span>'; return; }
           statusEl.innerHTML = '<span style="color:#4ade80">✓ Saved</span>';
 
           if (runNow) {
@@ -20302,7 +20305,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             const runData = await runResp.json();
             statusEl.innerHTML = runResp.ok
               ? '<span style="color:#4ade80">✓ Saved + run: in=' + runData.recordsIn + ' written=' + runData.recordsWritten + '</span>'
-              : '<span style="color:#e66">Saved but run failed: ' + escapeHtml(runData.error || 'unknown') + '</span>';
+              : '<span style="color:var(--red)">Saved but run failed: ' + escapeHtml(runData.error || 'unknown') + '</span>';
           }
           brainLoadSources();
         }
@@ -20323,7 +20326,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           const ref = document.getElementById('brain-cred-ref').value.trim();
           const value = document.getElementById('brain-cred-val').value;
           const statusEl = document.getElementById('brain-cred-status');
-          if (!ref || !value) { statusEl.innerHTML = '<span style="color:#e66">ref and value required</span>'; return; }
+          if (!ref || !value) { statusEl.innerHTML = '<span style="color:var(--red)">ref and value required</span>'; return; }
           const resp = await apiFetch('/api/brain/credentials', {
             method: 'POST', headers: { 'content-type': 'application/json' },
             body: JSON.stringify({ ref, value }),
@@ -20331,7 +20334,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
           const data = await resp.json();
           statusEl.innerHTML = resp.ok
             ? '<span style="color:#4ade80">✓ Saved ' + escapeHtml(ref) + '</span>'
-            : '<span style="color:#e66">' + escapeHtml(data.error || 'save failed') + '</span>';
+            : '<span style="color:var(--red)">' + escapeHtml(data.error || 'save failed') + '</span>';
           document.getElementById('brain-cred-ref').value = '';
           document.getElementById('brain-cred-val').value = '';
           brainShowCredsForm(); // refresh list
@@ -20425,7 +20428,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             });
             const data = await resp.json();
             if (!resp.ok) {
-              statusEl.innerHTML = '<span style="color:#e66">Upload failed: ' + escapeHtml(data.error || 'unknown') + '</span>';
+              statusEl.innerHTML = '<span style="color:var(--red)">Upload failed: ' + escapeHtml(data.error || 'unknown') + '</span>';
               return;
             }
             statusEl.innerHTML = '<span style="color:#4ade80">✓ Uploaded ' + data.count + ' file(s)</span>';
@@ -20433,7 +20436,7 @@ if('serviceWorker' in navigator){navigator.serviceWorker.getRegistrations().then
             // Kick off preview immediately
             await brainPreviewSeed();
           } catch (err) {
-            statusEl.innerHTML = '<span style="color:#e66">Upload error: ' + escapeHtml(String(err)) + '</span>';
+            statusEl.innerHTML = '<span style="color:var(--red)">Upload error: ' + escapeHtml(String(err)) + '</span>';
           }
         }
 

package/dist/gateway/cron-scheduler.js

@@ -79,6 +79,87 @@ function normalizeStringArray(v) {
     const out = Array.from(new Set(v.map(x => typeof x === 'string' ? x.trim() : '').filter(Boolean)));
     return out.length > 0 ? out : undefined;
 }
+/**
+ * Single-source-of-truth YAML → CronJobDefinition parser.
+ *
+ * Used by parseCronJobs (global CRON.md) AND parseAgentCronJobs (per-agent).
+ * Caller is responsible for handling the name prefix (agent jobs are
+ * prefixed with `<slug>:`) and for tagging agentSlug.
+ *
+ * Returns null when the row is malformed (missing name/schedule/prompt) —
+ * the caller logs + skips. Accepts both snake_case and camelCase YAML keys
+ * since users hand-edit CRON.md and we want to be forgiving.
+ *
+ * Centralizing here closes the drift the previous audit flagged: the agent
+ * variant was missing alwaysDeliver/attachments/requiresConfirmation/
+ * confirmationTimeoutMin and the description field. After this refactor
+ * both surfaces have the same field set.
+ */
+function parseJobYaml(job) {
+    const name = String(job.name ?? '');
+    const schedule = String(job.schedule ?? '');
+    const prompt = String(job.prompt ?? '');
+    if (!name || !schedule || !prompt)
+        return null;
+    const enabled = job.enabled !== false;
+    const tier = Number(job.tier ?? 1);
+    const maxTurns = job.max_turns != null ? Number(job.max_turns) : undefined;
+    const model = job.model != null ? String(job.model) : undefined;
+    const workDir = job.work_dir != null ? String(job.work_dir) : undefined;
+    const mode = job.mode === 'unleashed' ? 'unleashed' : 'standard';
+    const maxHours = job.max_hours != null ? Number(job.max_hours) : undefined;
+    const maxRetries = job.max_retries != null ? Number(job.max_retries) : undefined;
+    const after = job.after != null ? String(job.after) : undefined;
+    const successCriteria = Array.isArray(job.success_criteria)
+        ? job.success_criteria.map(c => String(c))
+        : undefined;
+    // Prefer free-form successCriteriaText; fall back to joining the legacy
+    // string[] so legacy YAML keeps rendering. 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');
+    }
+    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 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;
+    const attachments = normalizeStringArray(job.attachments);
+    const requiresConfirmation = job.requires_confirmation === true || job.requiresConfirmation === true ? true : undefined;
+    const confirmationTimeoutMin = job.confirmation_timeout_min != null
+        ? Number(job.confirmation_timeout_min)
+        : (job.confirmationTimeoutMin != null ? Number(job.confirmationTimeoutMin) : undefined);
+    // Per-job agent scoping (a global cron can be scoped to a specific
+    // agent's profile). Accept both casings.
+    const agentSlugRaw = job.agentSlug ?? job.agent_slug;
+    const agentSlug = typeof agentSlugRaw === 'string' && /^[a-z0-9-]+$/i.test(agentSlugRaw)
+        ? agentSlugRaw
+        : undefined;
+    const skills = normalizeStringArray(job.skills);
+    const allowedTools = normalizeStringArray(job.allowed_tools ?? job.allowedTools);
+    const allowedMcpServers = normalizeStringArray(job.allowed_mcp_servers ?? job.allowedMcpServers);
+    const tags = normalizeStringArray(job.tags);
+    const categoryRaw = job.category;
+    const category = typeof categoryRaw === 'string' && categoryRaw.trim()
+        ? categoryRaw.trim().slice(0, 64)
+        : undefined;
+    const predictable = typeof job.predictable === 'boolean' ? job.predictable : undefined;
+    const description = typeof job.description === 'string' && job.description.trim()
+        ? job.description.trim().slice(0, 500)
+        : undefined;
+    return {
+        name, schedule, prompt, enabled, tier, description, maxTurns, model, workDir, mode,
+        maxHours, maxRetries, after, successCriteria, successCriteriaText, successSchema, addDirs,
+        alwaysDeliver, context, preCheck, attachments, requiresConfirmation, confirmationTimeoutMin,
+        agentSlug,
+        skills, allowedTools, allowedMcpServers, tags, category, predictable,
+    };
+}
 /**
  * Parse cron job definitions from vault/00-System/CRON.md frontmatter.
  * Used by both the in-process CronScheduler and the standalone CLI runner.
@@ -86,10 +167,9 @@ function normalizeStringArray(v) {
 export function parseCronJobs() {
     if (!existsSync(CRON_FILE))
         return [];
-    const raw = readFileSync(CRON_FILE, 'utf-8');
     let parsed;
     try {
-        parsed = matter(raw);
+        parsed = matter(readFileSync(CRON_FILE, 'utf-8'));
     }
     catch (err) {
         logger.error({ err }, 'CRON.md YAML parse error — keeping previous jobs. Fix the file manually.');
@@ -98,76 +178,11 @@ export function parseCronJobs() {
     const jobDefs = (parsed.data.jobs ?? []);
     const jobs = [];
     for (const job of jobDefs) {
-        const name = String(job.name ?? '');
-        const schedule = String(job.schedule ?? '');
-        const prompt = String(job.prompt ?? '');
-        const enabled = job.enabled !== false;
-        const tier = Number(job.tier ?? 1);
-        const maxTurns = job.max_turns != null ? Number(job.max_turns) : undefined;
-        const model = job.model != null ? String(job.model) : undefined;
-        const workDir = job.work_dir != null ? String(job.work_dir) : undefined;
-        const mode = job.mode === 'unleashed' ? 'unleashed' : 'standard';
-        const maxHours = job.max_hours != null ? Number(job.max_hours) : undefined;
-        const maxRetries = job.max_retries != null ? Number(job.max_retries) : undefined;
-        const after = job.after != null ? String(job.after) : undefined;
-        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;
-        // Optional: scope a global job to a specific agent's profile (loads
-        // the agent's allowedTools whitelist, system prompt, etc.). Accept
-        // both camelCase and snake_case to be forgiving of user-written YAML.
-        const agentSlugRaw = job.agentSlug ?? job.agent_slug;
-        const agentSlug = typeof agentSlugRaw === 'string' && /^[a-z0-9-]+$/i.test(agentSlugRaw)
-            ? agentSlugRaw
-            : undefined;
-        // ── Trick capabilities — accept both camelCase and snake_case ─────
-        const skills = normalizeStringArray(job.skills);
-        const allowedTools = normalizeStringArray(job.allowed_tools ?? job.allowedTools);
-        const allowedMcpServers = normalizeStringArray(job.allowed_mcp_servers ?? job.allowedMcpServers);
-        const tags = normalizeStringArray(job.tags);
-        const categoryRaw = job.category;
-        const category = typeof categoryRaw === 'string' && categoryRaw.trim()
-            ? categoryRaw.trim().slice(0, 64)
-            : undefined;
-        // Predictable (contract) mode — undefined means legacy behavior.
-        const predictable = typeof job.predictable === 'boolean' ? job.predictable : undefined;
-        // 1.18.119 — human-readable description (used by the task card preview
-        // and by the cron-clean migrator to surface what each job does without
-        // showing raw prompt boilerplate).
-        const description = typeof job.description === 'string' && job.description.trim()
-            ? job.description.trim().slice(0, 500)
-            : undefined;
-        if (!name || !schedule || !prompt) {
+        const def = parseJobYaml(job);
+        if (def)
+            jobs.push(def);
+        else
             logger.warn({ job }, 'Skipping malformed cron job');
-            continue;
-        }
-        jobs.push({
-            name, schedule, prompt, enabled, tier, description, maxTurns, model, workDir, mode,
-            maxHours, maxRetries, after, successCriteria, successCriteriaText, successSchema, addDirs,
-            alwaysDeliver, context, preCheck, agentSlug,
-            skills, allowedTools, allowedMcpServers, tags, category, predictable,
-        });
     }
     return jobs;
 }
@@ -193,72 +208,20 @@ export function parseAgentCronJobs(agentsDir) {
         if (!existsSync(cronFile))
             continue;
         try {
-            const raw = readFileSync(cronFile, 'utf-8');
-            const parsed = matter(raw);
+            const parsed = matter(readFileSync(cronFile, 'utf-8'));
             const jobDefs = (parsed.data.jobs ?? []);
             for (const job of jobDefs) {
-                const name = String(job.name ?? '');
-                const schedule = String(job.schedule ?? '');
-                const prompt = String(job.prompt ?? '');
-                const enabled = job.enabled !== false;
-                const tier = Number(job.tier ?? 1);
-                const maxTurns = job.max_turns != null ? Number(job.max_turns) : undefined;
-                const model = job.model != null ? String(job.model) : undefined;
-                const workDir = job.work_dir != null ? String(job.work_dir) : undefined;
-                const mode = job.mode === 'unleashed' ? 'unleashed' : 'standard';
-                const maxHours = job.max_hours != null ? Number(job.max_hours) : undefined;
-                const maxRetries = job.max_retries != null ? Number(job.max_retries) : undefined;
-                const after = job.after != null ? String(job.after) : undefined;
-                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 ─────────
-                // (NB: this parser still lacks alwaysDeliver/attachments/
-                // requiresConfirmation/confirmationTimeoutMin from the global
-                // parser — pre-existing drift, fix in a separate change.)
-                const skills = normalizeStringArray(job.skills);
-                const allowedTools = normalizeStringArray(job.allowed_tools ?? job.allowedTools);
-                const allowedMcpServers = normalizeStringArray(job.allowed_mcp_servers ?? job.allowedMcpServers);
-                const tags = normalizeStringArray(job.tags);
-                const categoryRaw = job.category;
-                const category = typeof categoryRaw === 'string' && categoryRaw.trim()
-                    ? categoryRaw.trim().slice(0, 64)
-                    : undefined;
-                const predictable = typeof job.predictable === 'boolean' ? job.predictable : undefined;
-                // 1.18.119 — symmetric with the global parseCronJobs description
-                // field. Without this, agent jobs always look "missing description"
-                // to the cron-clean migrator and stay flagged as eligible forever.
-                const description = typeof job.description === 'string' && job.description.trim()
-                    ? job.description.trim().slice(0, 500)
-                    : undefined;
-                if (!name || !schedule || !prompt) {
+                const def = parseJobYaml(job);
+                if (!def) {
                     logger.warn({ job, agent: slug }, 'Skipping malformed agent cron job');
                     continue;
                 }
-                // Prefix name with agent slug and tag with agentSlug
-                allJobs.push({
-                    name: `${slug}:${name}`,
-                    schedule, prompt, enabled, tier, description, maxTurns, model, workDir,
-                    mode, maxHours, maxRetries, after,
-                    successCriteria, successCriteriaText, successSchema, addDirs,
-                    context, preCheck,
-                    agentSlug: slug,
-                    skills, allowedTools, allowedMcpServers, tags, category, predictable,
-                });
+                // Agent CRON.md stores BARE job names; we prefix with the slug at
+                // read time so the runtime can route by `<slug>:<job>` and the
+                // dashboard can disambiguate same-named jobs across agents.
+                // agentSlug is stamped from the folder location, overriding any
+                // value in the YAML — single source of truth.
+                allJobs.push({ ...def, name: `${slug}:${def.name}`, agentSlug: slug });
             }
         }
         catch (err) {

package/dist/types.d.ts

@@ -767,23 +767,21 @@ export interface Feedback {
     comment?: string;
     createdAt?: string;
 }
-export interface BehavioralCorrection {
-    correction: string;
-    category: 'verbosity' | 'tone' | 'workflow' | 'format' | 'accuracy' | 'proactivity' | 'scope';
-    strength: 'explicit' | 'implicit';
-}
-export interface PreferenceLearned {
-    preference: string;
-    confidence: 'high' | 'medium' | 'low';
-}
 export interface SessionReflection {
     id?: number;
     sessionKey: string;
     exchangeCount: number;
     frictionSignals: string[];
     qualityScore: number;
-    behavioralCorrections: BehavioralCorrection[];
-    preferencesLearned: PreferenceLearned[];
+    behavioralCorrections: Array<{
+        correction: string;
+        category: 'verbosity' | 'tone' | 'workflow' | 'format' | 'accuracy' | 'proactivity' | 'scope';
+        strength: 'explicit' | 'implicit';
+    }>;
+    preferencesLearned: Array<{
+        preference: string;
+        confidence: 'high' | 'medium' | 'low';
+    }>;
     agentSlug?: string;
     createdAt?: string;
 }
@@ -835,13 +833,6 @@ export interface ExecutionPlan {
     steps: PlanStep[];
     synthesisPrompt: string;
 }
-export interface PlanProgressUpdate {
-    stepId: string;
-    status: 'waiting' | 'running' | 'done' | 'failed';
-    description: string;
-    durationMs?: number;
-    resultPreview?: string;
-}
 export interface WorkflowInput {
     type: 'string' | 'number';
     default?: string;
@@ -853,13 +844,6 @@ export interface WorkflowStepMcpConfig {
     tool: string;
     inputs?: Record<string, unknown>;
 }
-export interface WorkflowStepCliConfig {
-    cmd: string;
-    args?: string[];
-    workDir?: string;
-    timeoutMs?: number;
-    captureStderr?: boolean;
-}
 export interface WorkflowStepChannelConfig {
     channel: 'discord' | 'slack' | 'telegram' | 'whatsapp' | 'email' | 'webhook';
     target: string;
@@ -891,7 +875,16 @@ export interface WorkflowStep {
     workDir?: string;
     kind?: WorkflowStepKind;
     mcp?: WorkflowStepMcpConfig;
-    cli?: WorkflowStepCliConfig;
+    /** CLI step config — inline shape (was the standalone WorkflowStepCliConfig
+     *  type that had zero external references; the field stays but the
+     *  named alias was dropped in 1.18.122). */
+    cli?: {
+        cmd: string;
+        args?: string[];
+        workDir?: string;
+        timeoutMs?: number;
+        captureStderr?: boolean;
+    };
     channel?: WorkflowStepChannelConfig;
     transform?: WorkflowStepTransformConfig;
     conditional?: WorkflowStepConditionalConfig;
@@ -1180,14 +1173,6 @@ export interface SessionRecord {
     lastUsedAt: number;
     userAgent?: string;
 }
-export interface ConfigRevision {
-    id?: number;
-    agentSlug: string;
-    fileName: string;
-    content: string;
-    changedBy?: string;
-    createdAt?: string;
-}
 export interface Lead {
     id?: number;
     agentSlug: string;
@@ -1202,16 +1187,6 @@ export interface Lead {
     createdAt?: string;
     updatedAt?: string;
 }
-export interface SequenceEnrollment {
-    id?: number;
-    leadId: number;
-    sequenceName: string;
-    currentStep: number;
-    status: 'active' | 'paused' | 'replied' | 'completed' | 'opted_out';
-    nextStepDueAt?: string;
-    startedAt?: string;
-    updatedAt?: string;
-}
 export interface Activity {
     id?: number;
     leadId?: number;
@@ -1222,40 +1197,6 @@ export interface Activity {
     templateUsed?: string;
     performedAt?: string;
 }
-export interface SuppressionEntry {
-    id?: number;
-    email: string;
-    reason: 'unsubscribe' | 'bounce' | 'manual' | 'complaint';
-    addedAt?: string;
-    addedBy?: string;
-}
-export interface ApprovalRequest {
-    id?: number;
-    agentSlug: string;
-    actionType: 'email_send' | 'sequence_start' | 'escalation';
-    summary: string;
-    detail?: Record<string, unknown>;
-    status: 'pending' | 'approved' | 'rejected';
-    requestedAt?: string;
-    resolvedAt?: string;
-    resolvedBy?: string;
-}
-export interface SfSyncRecord {
-    id?: number;
-    localTable: 'leads' | 'activities';
-    localId: number;
-    sfObjectType: 'Lead' | 'Contact' | 'Opportunity' | 'Task' | 'Event';
-    sfId: string;
-    syncDirection: 'push' | 'pull';
-    syncedAt?: string;
-    syncStatus: 'success' | 'error' | 'conflict';
-    errorMessage?: string;
-}
-export interface SfFieldMapping {
-    localField: string;
-    sfField: string;
-    direction: 'bidirectional' | 'push-only' | 'pull-only';
-}
 /** Supported v1 ingest formats detected by format-detector. */
 export type DetectedFormat = 'csv' | 'json' | 'jsonl' | 'markdown' | 'pdf' | 'email' | 'docx' | 'unknown';
 /** Operational mode for a source. */

package/package.json

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