coder-config 0.50.4-beta → 0.50.6-beta

package/lib/constants.js

@@ -2,7 +2,7 @@
  * Constants and tool path configurations
  */
 
-const VERSION = '0.50.4-beta';
+const VERSION = '0.50.6-beta';
 
 // Tool-specific path configurations
 const TOOL_PATHS = {

package/lib/heartbeat.js

@@ -4,7 +4,7 @@
 
 const fs = require('fs');
 const path = require('path');
-const { loadLoops, loadLoopState } = require('./loops.js');
+const { loadLoops, loadLoopState, getLoopsPath } = require('./loops.js');
 
 /**
  * Get default heartbeat configuration
@@ -21,6 +21,122 @@ function getDefaultHeartbeatConfig() {
   };
 }
 
+/**
+ * Load heartbeat config from loops.json, merged with defaults
+ * @param {string} installDir - path to coder-config install dir
+ * @returns {object} merged heartbeat config
+ */
+function loadHeartbeatConfig(installDir) {
+  const data = loadLoops(installDir);
+  return { ...getDefaultHeartbeatConfig(), ...data.config?.heartbeat };
+}
+
+/**
+ * Evaluate a single loop state and return any alerts
+ * @param {object} state - loop state object
+ * @param {object} globalConfig - heartbeat config (merged defaults)
+ * @param {Date} now - current date/time
+ * @returns {Array} array of alert objects for this loop
+ */
+function evaluateLoop(state, globalConfig, now) {
+  const alerts = [];
+  const status = state.status || 'unknown';
+  const name = state.name || state.id;
+
+  // Check: failed
+  if (status === 'failed') {
+    alerts.push({
+      loopId: state.id,
+      name,
+      type: 'failed',
+      message: `Loop "${name}" has failed`,
+      severity: 'critical'
+    });
+    return alerts;
+  }
+
+  // Check: stale
+  const perLoopThreshold = state.heartbeat && state.heartbeat.staleThresholdMinutes != null
+    ? state.heartbeat.staleThresholdMinutes
+    : globalConfig.staleThresholdMinutes;
+
+  if (state.updatedAt) {
+    const updatedAt = new Date(state.updatedAt);
+    const ageMinutes = ((now || new Date()).getTime() - updatedAt.getTime()) / 60000;
+    if (ageMinutes >= perLoopThreshold) {
+      alerts.push({
+        loopId: state.id,
+        name,
+        type: 'stale',
+        message: `Loop "${name}" has not updated in ${Math.floor(ageMinutes)} minutes`,
+        severity: 'warning'
+      });
+    }
+  }
+
+  // Check: iteration_limit
+  const iterations = state.iterations || {};
+  const current = iterations.current != null ? iterations.current : 0;
+  const max = iterations.max != null ? iterations.max : 0;
+  if (max > 0) {
+    const pct = (current / max) * 100;
+    if (pct >= globalConfig.iterationLimitPercent) {
+      alerts.push({
+        loopId: state.id,
+        name,
+        type: 'iteration_limit',
+        message: `Loop "${name}" is at ${Math.round(pct)}% of iteration limit (${current}/${max})`,
+        severity: 'warning'
+      });
+    }
+  }
+
+  // Check: phase_gate (plan phase + running)
+  if (state.phase === 'plan' && status === 'running') {
+    alerts.push({
+      loopId: state.id,
+      name,
+      type: 'phase_gate',
+      message: `Loop "${name}" is waiting at plan phase gate`,
+      severity: 'info'
+    });
+  }
+
+  // Check: blocked (paused with pauseReason)
+  if (status === 'paused' && state.pauseReason) {
+    alerts.push({
+      loopId: state.id,
+      name,
+      type: 'blocked',
+      message: `Loop "${name}" is blocked: ${state.pauseReason}`,
+      severity: 'info'
+    });
+  }
+
+  return alerts;
+}
+
+/**
+ * Build a summary string from heartbeat results
+ * @param {number} activeLoops - count of active (non-terminal) loops
+ * @param {Array} alerts - array of alert objects
+ * @param {Array} healthy - array of healthy loop objects
+ * @returns {string} summary string
+ */
+function buildSummary(activeLoops, alerts, healthy) {
+  if (alerts.length === 0) {
+    return `${activeLoops} active loop${activeLoops !== 1 ? 's' : ''}, all healthy`;
+  }
+  const criticalCount = alerts.filter(a => a.severity === 'critical').length;
+  const warningCount = alerts.filter(a => a.severity === 'warning').length;
+  const infoCount = alerts.filter(a => a.severity === 'info').length;
+  const parts = [];
+  if (criticalCount > 0) parts.push(`${criticalCount} critical`);
+  if (warningCount > 0) parts.push(`${warningCount} warning${warningCount !== 1 ? 's' : ''}`);
+  if (infoCount > 0) parts.push(`${infoCount} info`);
+  return `${activeLoops} active loop${activeLoops !== 1 ? 's' : ''}, ${parts.join(', ')}`;
+}
+
 /**
  * Evaluate heartbeat for all active loops
  * @param {string} installDir - path to coder-config install dir
@@ -36,6 +152,7 @@ function heartbeat(installDir, config) {
   const alerts = [];
   const healthy = [];
   const terminalStates = new Set(['completed', 'cancelled']);
+  const now = new Date();
 
   for (const entry of loops) {
     const state = loadLoopState(installDir, entry.id);
@@ -45,84 +162,16 @@ function heartbeat(installDir, config) {
     if (terminalStates.has(status)) continue;
 
     const name = state.name || entry.name || entry.id;
+    // Ensure state has id and name for evaluateLoop
+    const stateWithMeta = { ...state, id: entry.id, name };
 
-    // Check: failed
-    if (status === 'failed') {
-      alerts.push({
-        loopId: entry.id,
-        name,
-        type: 'failed',
-        message: `Loop "${name}" has failed`,
-        severity: 'critical'
-      });
-      continue;
-    }
+    const loopAlerts = evaluateLoop(stateWithMeta, hbConfig, now);
 
-    let hasAlert = false;
-
-    // Check: stale
-    const perLoopThreshold = state.heartbeat && state.heartbeat.staleThresholdMinutes != null
-      ? state.heartbeat.staleThresholdMinutes
-      : hbConfig.staleThresholdMinutes;
-
-    if (state.updatedAt) {
-      const updatedAt = new Date(state.updatedAt);
-      const ageMinutes = (Date.now() - updatedAt.getTime()) / 60000;
-      if (ageMinutes >= perLoopThreshold) {
-        alerts.push({
-          loopId: entry.id,
-          name,
-          type: 'stale',
-          message: `Loop "${name}" has not updated in ${Math.floor(ageMinutes)} minutes`,
-          severity: 'warning'
-        });
-        hasAlert = true;
-      }
-    }
-
-    // Check: iteration_limit
-    const iterations = state.iterations || {};
-    const current = iterations.current != null ? iterations.current : 0;
-    const max = iterations.max != null ? iterations.max : 0;
-    if (max > 0) {
-      const pct = (current / max) * 100;
-      if (pct >= hbConfig.iterationLimitPercent) {
-        alerts.push({
-          loopId: entry.id,
-          name,
-          type: 'iteration_limit',
-          message: `Loop "${name}" is at ${Math.round(pct)}% of iteration limit (${current}/${max})`,
-          severity: 'warning'
-        });
-        hasAlert = true;
-      }
-    }
-
-    // Check: phase_gate (plan phase + running)
-    if (state.phase === 'plan' && status === 'running') {
-      alerts.push({
-        loopId: entry.id,
-        name,
-        type: 'phase_gate',
-        message: `Loop "${name}" is waiting at plan phase gate`,
-        severity: 'info'
-      });
-      hasAlert = true;
-    }
-
-    // Check: blocked (paused with pauseReason)
-    if (status === 'paused' && state.pauseReason) {
-      alerts.push({
-        loopId: entry.id,
-        name,
-        type: 'blocked',
-        message: `Loop "${name}" is blocked: ${state.pauseReason}`,
-        severity: 'info'
-      });
-      hasAlert = true;
-    }
-
-    if (!hasAlert) {
+    if (loopAlerts.length > 0) {
+      alerts.push(...loopAlerts);
+    } else {
+      const iterations = state.iterations || {};
+      const current = iterations.current != null ? iterations.current : 0;
       healthy.push({
         loopId: entry.id,
         name,
@@ -139,23 +188,10 @@ function heartbeat(installDir, config) {
     return !terminalStates.has(status);
   }).length;
 
-  const criticalCount = alerts.filter(a => a.severity === 'critical').length;
-  const warningCount = alerts.filter(a => a.severity === 'warning').length;
-
-  let summary;
-  if (alerts.length === 0) {
-    summary = `${activeLoops} active loop${activeLoops !== 1 ? 's' : ''}, all healthy`;
-  } else {
-    const parts = [];
-    if (criticalCount > 0) parts.push(`${criticalCount} critical`);
-    if (warningCount > 0) parts.push(`${warningCount} warning${warningCount !== 1 ? 's' : ''}`);
-    const infoCount = alerts.filter(a => a.severity === 'info').length;
-    if (infoCount > 0) parts.push(`${infoCount} info`);
-    summary = `${activeLoops} active loop${activeLoops !== 1 ? 's' : ''}, ${parts.join(', ')}`;
-  }
+  const summary = buildSummary(activeLoops, alerts, healthy);
 
   return {
-    timestamp: new Date().toISOString(),
+    timestamp: now.toISOString(),
     activeLoops,
     alerts,
     healthy,
@@ -163,7 +199,93 @@ function heartbeat(installDir, config) {
   };
 }
 
+/**
+ * Save the last heartbeat report to disk for deduplication
+ * @param {string} installDir - path to coder-config install dir
+ * @param {object} report - heartbeat report with alerts array
+ */
+function saveLastHeartbeat(installDir, report) {
+  const loopsDir = path.join(installDir, 'loops');
+  fs.mkdirSync(loopsDir, { recursive: true });
+  const alertHashes = {};
+  const now = Date.now();
+  for (const alert of (report.alerts || [])) {
+    const key = `${alert.loopId}:${alert.type}`;
+    alertHashes[key] = now;
+  }
+  const data = { alertHashes, timestamp: now };
+  fs.writeFileSync(path.join(loopsDir, 'last-heartbeat.json'), JSON.stringify(data, null, 2));
+}
+
+/**
+ * Load the last saved heartbeat from disk
+ * @param {string} installDir - path to coder-config install dir
+ * @returns {object|null} saved heartbeat data or null if not found
+ */
+function loadLastHeartbeat(installDir) {
+  const filePath = path.join(installDir, 'loops', 'last-heartbeat.json');
+  try {
+    const raw = fs.readFileSync(filePath, 'utf8');
+    return JSON.parse(raw);
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Determine whether an alert should fire (not within cooldown)
+ * @param {string} installDir - path to coder-config install dir
+ * @param {object} alert - alert object with loopId and type
+ * @param {number} cooldownMinutes - cooldown in minutes
+ * @returns {boolean} true if should notify, false if within cooldown
+ */
+function shouldNotify(installDir, alert, cooldownMinutes) {
+  const last = loadLastHeartbeat(installDir);
+  if (!last || !last.alertHashes) return true;
+  const key = `${alert.loopId}:${alert.type}`;
+  const lastTime = last.alertHashes[key];
+  if (lastTime == null) return true;
+  const ageMinutes = (Date.now() - lastTime) / 60000;
+  return ageMinutes >= cooldownMinutes;
+}
+
+/**
+ * Build a macOS osascript notification command for a heartbeat report
+ * @param {object} report - heartbeat report
+ * @returns {string} shell command string
+ */
+function buildMacosNotification(report) {
+  const alerts = report.alerts || [];
+  const criticalCount = alerts.filter(a => a.severity === 'critical').length;
+  const warningCount = alerts.filter(a => a.severity === 'warning').length;
+  const parts = [];
+  if (criticalCount > 0) parts.push(`${criticalCount} critical`);
+  if (warningCount > 0) parts.push(`${warningCount} warning${warningCount !== 1 ? 's' : ''}`);
+  const subtitle = parts.length > 0 ? parts.join(', ') : 'all healthy';
+  const body = (report.summary || subtitle).replace(/'/g, "\\'");
+  return `osascript -e 'display notification "${body}" with title "Ralph Heartbeat" subtitle "${subtitle}"'`;
+}
+
+/**
+ * Get exit code for a heartbeat report
+ * @param {object} report - heartbeat report with alerts array
+ * @returns {number} 0 if healthy or only info, 1 if any warning or critical
+ */
+function getExitCode(report) {
+  const alerts = report.alerts || [];
+  const hasActionable = alerts.some(a => a.severity === 'critical' || a.severity === 'warning');
+  return hasActionable ? 1 : 0;
+}
+
 module.exports = {
+  heartbeat,
   getDefaultHeartbeatConfig,
-  heartbeat
+  loadHeartbeatConfig,
+  evaluateLoop,
+  buildSummary,
+  saveLastHeartbeat,
+  loadLastHeartbeat,
+  shouldNotify,
+  buildMacosNotification,
+  getExitCode
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coder-config",
-  "version": "0.50.4-beta",
+  "version": "0.50.6-beta",
   "description": "Configuration manager for AI coding tools - Claude Code, Gemini CLI, Codex CLI, Antigravity. Manage MCPs, rules, permissions, memory, and workstreams.",
   "author": "regression.io",
   "main": "config-loader.js",