seo-intel 1.5.39 → 1.5.40

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.5.40 (2026-05-28)
+
+### Setup wizard — daily notifications now self-install
+The notify loop is now opt-in via a single click. Setup-wizard's Done step gains a small "Daily problem notifications" card that toggles a managed `crontab` line via `lib/cron.js`. No more manual `crontab -e`.
+
+- **New CLI:** `seo-intel install-cron [--schedule "0 9 * * *"] [--open] [--remove]` — install / replace / remove a managed cron line. Idempotent. macOS + Linux. Windows returns a clear "use Task Scheduler" message.
+- **New library:** `lib/cron.js` exports `installNotifyCron`, `removeNotifyCron`, `getNotifyCronStatus`. Uses a `# managed-by-seo-intel` marker comment so we only touch our own entries — the user's other cron jobs are never modified.
+- **Setup wizard card** (Step 6, Done): live status pill, one-click Enable/Disable button, default schedule 9am daily, intel-blue accent matching the visual brief tokens. Polls `/api/setup/cron` on step entry; toggle hits `/api/setup/cron POST { action: 'install'|'remove' }`.
+- Cron line writes the absolute `process.execPath` so the scheduled job works regardless of the user's PATH inside cron's minimal environment (a classic foot-gun avoided).
+
+Full lifecycle verified: status read → install (`0 9 * * *` line written, crontab confirms) → status reports installed → remove → crontab clean → status reports clean. Smoke 10/10.
+
 ## 1.5.39 (2026-05-27)
 
 ### Dashboard — Problems card as the landing surface (Ahrefs-style "what's broken")

package/cli.js

@@ -1511,6 +1511,41 @@ program
     if (fired === 0) console.log(chalk.dim('  ✓ No projects need attention.'));
   });
 
+// ── INSTALL-CRON (one-shot cron entry for daily notify) ────────────────────
+program
+  .command('install-cron')
+  .description('Install / remove the daily seo-intel notify cron entry (macOS + Linux)')
+  .option('--remove', 'Remove the managed cron entry instead of installing')
+  .option('--schedule <cron>', 'Cron schedule, 5 fields (default: 9am daily)', '0 9 * * *')
+  .option('--open', 'Pass --open to the scheduled notify command (dashboard opens when fired)')
+  .action(async (opts) => {
+    const { installNotifyCron, removeNotifyCron, getNotifyCronStatus, DEFAULT_SCHEDULE } = await import('./lib/cron.js');
+    if (opts.remove) {
+      const r = removeNotifyCron();
+      if (!r.ok) { console.error(chalk.red(`Failed: ${r.error}`)); process.exit(1); }
+      console.log(chalk.green(r.removed ? '  ✓ Removed managed cron entry.' : '  (Nothing to remove.)'));
+      return;
+    }
+    const status = getNotifyCronStatus();
+    if (status.platform === 'win32') {
+      console.log(chalk.yellow('  ⚠ Windows: use Task Scheduler manually.'));
+      console.log(chalk.dim('  Daily task command:'));
+      console.log(chalk.dim(`    ${process.execPath} ${join(__dirname, 'cli.js')} notify`));
+      return;
+    }
+    if (status.installed) {
+      console.log(chalk.dim(`  Existing entry detected (schedule: ${status.schedule}). Replacing…`));
+    }
+    const result = installNotifyCron({ schedule: opts.schedule, openOnFire: !!opts.open });
+    if (!result.ok) { console.error(chalk.red(`Failed: ${result.error}`)); process.exit(1); }
+    console.log(chalk.green(`  ✓ Daily notification scheduled (cron: ${result.schedule})`));
+    console.log(chalk.dim(`  Cron line: ${result.line}`));
+    console.log(chalk.dim(`  Remove with: seo-intel install-cron --remove`));
+    if (process.platform === 'darwin') {
+      console.log(chalk.dim(`  Note: macOS may prompt to authorise cron access on first fire — approve in System Settings if so.`));
+    }
+  });
+
 // ── STATUS ─────────────────────────────────────────────────────────────────
 program
   .command('status')

package/lib/cron.js

@@ -0,0 +1,108 @@
+/**
+ * lib/cron.js — Install / remove the daily `seo-intel notify` cron entry.
+ *
+ * The "user forgets to check SEO" defense from v1.5.34's delivery brainstorm.
+ * Adds a single managed crontab line tagged with a marker comment so we can
+ * find and replace/remove our own entry without touching the user's other
+ * cron jobs.
+ *
+ * macOS + Linux: uses crontab(1). On macOS the first install will prompt the
+ * user to approve calendar/automation access via the system permission dialog
+ * — that's normal, nothing we can do about it.
+ *
+ * Windows: returns ok:false with a hint pointing at Task Scheduler. Out of
+ * scope for v1.5.40.
+ */
+
+import { spawnSync } from 'child_process';
+import { fileURLToPath } from 'url';
+import { dirname, join } from 'path';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const ROOT = join(__dirname, '..');
+const NODE_BIN = process.execPath;
+const MARKER = '# managed-by-seo-intel';
+
+export const DEFAULT_SCHEDULE = '0 9 * * *'; // 9am every day
+
+function readCrontab() {
+  const r = spawnSync('crontab', ['-l'], { encoding: 'utf8' });
+  if (r.status === 0) return r.stdout || '';
+  // status !== 0 typically means "no crontab for user yet" — return empty
+  return '';
+}
+
+function writeCrontab(content) {
+  const text = (content || '').replace(/\n*$/, '\n'); // ensure single trailing newline
+  const r = spawnSync('crontab', ['-'], { input: text, encoding: 'utf8' });
+  if (r.status !== 0) {
+    throw new Error(`crontab write failed: ${r.stderr || 'unknown error'}`);
+  }
+}
+
+function isWindows() { return process.platform === 'win32'; }
+
+/**
+ * @returns {{ installed: boolean, line: string|null, schedule: string|null, platform: string }}
+ */
+export function getNotifyCronStatus() {
+  if (isWindows()) return { installed: false, line: null, schedule: null, platform: 'win32' };
+  const lines = readCrontab().split('\n').filter(l => l.includes(MARKER));
+  if (!lines.length) return { installed: false, line: null, schedule: null, platform: process.platform };
+  const line = lines[0];
+  // Schedule is the first 5 space-separated fields
+  const parts = line.trim().split(/\s+/);
+  const schedule = parts.slice(0, 5).join(' ');
+  return { installed: true, line, schedule, platform: process.platform };
+}
+
+/**
+ * Install (or replace) the managed cron line.
+ *
+ * @param {object} [opts]
+ * @param {string} [opts.schedule]  Cron schedule, default DEFAULT_SCHEDULE (9am daily)
+ * @param {boolean} [opts.openOnFire]  Append `--open` flag so the dashboard opens when fired
+ * @returns {{ ok: boolean, line?: string, schedule?: string, error?: string, hint?: string }}
+ */
+export function installNotifyCron({ schedule = DEFAULT_SCHEDULE, openOnFire = false } = {}) {
+  if (isWindows()) {
+    return {
+      ok: false,
+      error: 'Windows not supported — use Task Scheduler manually',
+      hint: `Create a daily task running:  ${NODE_BIN} "${join(ROOT, 'cli.js')}" notify`,
+    };
+  }
+  // Sanity-check schedule (5 fields, no shell metachars)
+  if (!/^[-*\/0-9, ]+$/.test(schedule) || schedule.trim().split(/\s+/).length !== 5) {
+    return { ok: false, error: `Invalid cron schedule "${schedule}". Expected 5 fields, e.g. "0 9 * * *".` };
+  }
+  const cmd = `cd ${ROOT} && ${NODE_BIN} cli.js notify${openOnFire ? ' --open' : ''}`;
+  const newLine = `${schedule} ${cmd}  ${MARKER}`;
+  const current = readCrontab();
+  const kept = current.split('\n').filter(l => l && !l.includes(MARKER));
+  kept.push(newLine);
+  try {
+    writeCrontab(kept.join('\n'));
+    return { ok: true, line: newLine, schedule };
+  } catch (e) {
+    return { ok: false, error: e.message };
+  }
+}
+
+/**
+ * Remove the managed cron line (if any). Idempotent.
+ * @returns {{ ok: boolean, removed: boolean, error?: string }}
+ */
+export function removeNotifyCron() {
+  if (isWindows()) return { ok: true, removed: false }; // nothing to remove
+  const current = readCrontab();
+  const before = current.split('\n').filter(Boolean).length;
+  const kept = current.split('\n').filter(l => l && !l.includes(MARKER));
+  if (kept.length === before) return { ok: true, removed: false };
+  try {
+    writeCrontab(kept.join('\n'));
+    return { ok: true, removed: true };
+  } catch (e) {
+    return { ok: false, removed: false, error: e.message };
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "seo-intel",
-  "version": "1.5.39",
+  "version": "1.5.40",
   "description": "Local Ahrefs-style SEO competitor intelligence. Crawl → SQLite → cloud analysis.",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE",

package/setup/web-routes.js

@@ -128,6 +128,17 @@ export function handleSetupRequest(req, res, url) {
     return true;
   }
 
+  // GET /api/setup/cron — read notify-cron install state
+  if (path === '/api/setup/cron' && method === 'GET') {
+    handleCronStatus(req, res);
+    return true;
+  }
+  // POST /api/setup/cron — install / remove
+  if (path === '/api/setup/cron' && method === 'POST') {
+    handleCronAction(req, res);
+    return true;
+  }
+
   // POST /api/setup/save-env — save a key to .env
   if (path === '/api/setup/save-env' && method === 'POST') {
     handleSaveEnv(req, res);
@@ -341,6 +352,34 @@ async function handlePingOllama(req, res) {
   }
 }
 
+async function handleCronStatus(req, res) {
+  try {
+    const { getNotifyCronStatus } = await import('../lib/cron.js');
+    jsonResponse(res, getNotifyCronStatus());
+  } catch (err) {
+    jsonResponse(res, { error: err.message }, 500);
+  }
+}
+
+async function handleCronAction(req, res) {
+  try {
+    const body = await readBody(req);
+    const { action, schedule, openOnFire } = body || {};
+    const { installNotifyCron, removeNotifyCron, getNotifyCronStatus } = await import('../lib/cron.js');
+    if (action === 'install') {
+      const result = installNotifyCron({ schedule, openOnFire: !!openOnFire });
+      jsonResponse(res, { ...result, status: getNotifyCronStatus() });
+    } else if (action === 'remove') {
+      const result = removeNotifyCron();
+      jsonResponse(res, { ...result, status: getNotifyCronStatus() });
+    } else {
+      jsonResponse(res, { error: 'action must be "install" or "remove"' }, 400);
+    }
+  } catch (err) {
+    jsonResponse(res, { error: err.message }, 500);
+  }
+}
+
 async function handleSaveEnv(req, res) {
   try {
     const body = await readBody(req);

package/setup/wizard.html

@@ -1862,6 +1862,21 @@ input::placeholder {
       <!-- Populated by JS -->
     </table>
 
+    <!-- ── Daily notification scheduler (v1.5.40) ────────────────────────── -->
+    <div id="notifyCronCard" style="margin-top:20px; padding:14px 16px; background:var(--bg-elevated); border:1px solid rgba(59,130,246,0.18); border-radius:var(--radius); display:flex; gap:14px; align-items:center; flex-wrap:wrap;">
+      <i class="fa-solid fa-bell" style="font-size:1rem; color:#3b82f6;"></i>
+      <div style="flex:1; min-width:200px;">
+        <div style="font-size:0.8rem; font-weight:600; color:var(--text-primary); margin-bottom:2px;">Daily problem notifications</div>
+        <div id="notifyCronHint" style="font-size:0.66rem; color:var(--text-muted); line-height:1.4;">
+          Fires a native macOS / Linux notification each morning for projects with pending criticals or warns.
+        </div>
+      </div>
+      <div id="notifyCronStatus" style="font-size:0.66rem; color:var(--text-muted);">…</div>
+      <button id="notifyCronToggleBtn" class="btn btn-sm" onclick="toggleNotifyCron()" style="padding:5px 12px; font-size:0.7rem; min-width:90px;">
+        <i class="fa-solid fa-spinner fa-spin"></i>
+      </button>
+    </div>
+
     <h3 style="font-family:var(--font-display); font-size:0.85rem; color:var(--text-primary); margin: 20px 0 12px; font-weight:600;">
       <i class="fa-solid fa-rocket" style="color:var(--accent-gold); margin-right:6px;"></i> What's Next
     </h3>
@@ -3409,6 +3424,64 @@ input::placeholder {
     }
 
     goToStep(6);
+    loadNotifyCronStatus();
+  };
+
+  // ── Daily notification cron (v1.5.40) ──────────────────────────────────
+  let _cronState = null;
+  async function loadNotifyCronStatus() {
+    const statusEl = document.getElementById('notifyCronStatus');
+    const btn = document.getElementById('notifyCronToggleBtn');
+    const hint = document.getElementById('notifyCronHint');
+    if (!statusEl || !btn) return;
+    try {
+      const s = await API.get('/api/setup/cron');
+      _cronState = s;
+      if (s.platform === 'win32') {
+        statusEl.textContent = 'Windows: use Task Scheduler';
+        btn.disabled = true;
+        btn.innerHTML = 'N/A';
+        hint.textContent = 'Cron auto-install is macOS / Linux only — set up a daily Task Scheduler entry manually.';
+        return;
+      }
+      if (s.installed) {
+        statusEl.innerHTML = `<span style="color:var(--color-success);"><i class="fa-solid fa-check"></i> Active · ${s.schedule}</span>`;
+        btn.innerHTML = '<i class="fa-solid fa-bell-slash"></i> Disable';
+        btn.style.borderColor = 'rgba(244,123,93,0.4)';
+        btn.style.color = '#f47b5d';
+      } else {
+        statusEl.innerHTML = `<span style="color:var(--text-muted);">Not scheduled</span>`;
+        btn.innerHTML = '<i class="fa-solid fa-bell"></i> Enable';
+        btn.style.borderColor = 'rgba(59,130,246,0.4)';
+        btn.style.color = '#3b82f6';
+      }
+      btn.disabled = false;
+    } catch (err) {
+      statusEl.textContent = 'Status check failed';
+      btn.disabled = true;
+    }
+  }
+
+  window.toggleNotifyCron = async function() {
+    const btn = document.getElementById('notifyCronToggleBtn');
+    if (!btn || btn.disabled) return;
+    btn.disabled = true;
+    btn.innerHTML = '<i class="fa-solid fa-spinner fa-spin"></i>';
+    const action = _cronState?.installed ? 'remove' : 'install';
+    try {
+      const r = await API.post('/api/setup/cron', { action, schedule: '0 9 * * *' });
+      if (r.ok === false && r.error) {
+        document.getElementById('notifyCronStatus').textContent = 'Error: ' + r.error;
+        btn.disabled = false;
+        btn.innerHTML = '<i class="fa-solid fa-triangle-exclamation"></i> Retry';
+        return;
+      }
+      await loadNotifyCronStatus();
+    } catch (err) {
+      document.getElementById('notifyCronStatus').textContent = 'Error: ' + err.message;
+      btn.disabled = false;
+      btn.innerHTML = '<i class="fa-solid fa-triangle-exclamation"></i> Retry';
+    }
   };
 
   window.copyCli = function() {