@yemi33/minions 0.1.2347 → 0.1.2348

package/dashboard.js

@@ -9845,8 +9845,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const body = await readBody(req);
       const message = body && typeof body.message === 'string' ? body.message : '';
       if (!message.trim()) return jsonReply(res, 400, { error: 'message required' });
-      const watchId = body.watchId ? String(body.watchId) : '';
-      const label = watchId ? `watch-cc-triage:${watchId}` : 'watch-cc-triage';
+      // W-mrb1a3950003d9ea — flat, stable label (no per-watch-id interpolation).
+      // Every other _engine category (kb-sweep, consolidation, command-center, …)
+      // uses one flat label; a dynamic watchId suffix here used to mint a brand-new
+      // permanent metrics.json._engine key per watch firing (20+ confirmed live).
+      const label = 'watch-cc-triage';
 
       const model = CONFIG.engine?.ccModel || shared.ENGINE_DEFAULTS.ccModel;
       const maxTurns = CONFIG.engine?.ccMaxTurns || shared.ENGINE_DEFAULTS.ccMaxTurns;

package/engine/metrics-store.js

@@ -172,6 +172,8 @@ function readMetrics() {
   try { db = getDb(); }
   catch { return _readJsonObjectFallback(); }
 
+  migrateWatchCcTriageKeysOnce();
+
   _resyncIfJsonDiverged(db);
 
   const rows = _readAllRows(db);
@@ -285,9 +287,73 @@ function _resetMetricsTableForTest() {
   _lastMirrorHash = null;
 }
 
+// W-mrb1a3950003d9ea — one-time collapse of legacy `watch-cc-triage:<watchId>`
+// per-watch _engine keys (minted before dashboard.js's handleCommandCenterTriage
+// stopped interpolating watchId into the trackEngineUsage label) into a single
+// flat `watch-cc-triage` bucket, matching every other _engine category. Sums
+// calls/cost/tokens/duration/errorsByCode across all matching keys, then deletes
+// the per-id keys. Idempotent — a no-op (no SQL/JSON write) once no key matches
+// /^watch-cc-triage:/, so it's safe to call unconditionally on every process
+// startup via migrateWatchCcTriageKeysOnce().
+const WATCH_CC_TRIAGE_LEGACY_KEY_RE = /^watch-cc-triage:/;
+const WATCH_CC_TRIAGE_FLAT_KEY = 'watch-cc-triage';
+
+function migrateWatchCcTriageKeys() {
+  return applyMetricsMutation((metrics) => {
+    const engineMetrics = metrics._engine;
+    if (!engineMetrics || typeof engineMetrics !== 'object') return metrics;
+    const staleKeys = Object.keys(engineMetrics).filter((k) => WATCH_CC_TRIAGE_LEGACY_KEY_RE.test(k));
+    if (staleKeys.length === 0) return metrics;
+
+    if (!engineMetrics[WATCH_CC_TRIAGE_FLAT_KEY]) {
+      engineMetrics[WATCH_CC_TRIAGE_FLAT_KEY] = { calls: 0, costUsd: 0, inputTokens: 0, outputTokens: 0, cacheRead: 0, cacheCreation: 0 };
+    }
+    const flat = engineMetrics[WATCH_CC_TRIAGE_FLAT_KEY];
+    for (const key of staleKeys) {
+      const cat = engineMetrics[key] || {};
+      flat.calls = (flat.calls || 0) + (cat.calls || 0);
+      flat.costUsd = (flat.costUsd || 0) + (cat.costUsd || 0);
+      flat.inputTokens = (flat.inputTokens || 0) + (cat.inputTokens || 0);
+      flat.outputTokens = (flat.outputTokens || 0) + (cat.outputTokens || 0);
+      flat.cacheRead = (flat.cacheRead || 0) + (cat.cacheRead || 0);
+      flat.cacheCreation = (flat.cacheCreation || 0) + (cat.cacheCreation || 0);
+      if (cat.totalDurationMs) flat.totalDurationMs = (flat.totalDurationMs || 0) + cat.totalDurationMs;
+      if (cat.timedCalls) flat.timedCalls = (flat.timedCalls || 0) + cat.timedCalls;
+      if (cat.errorsByCode) {
+        if (!flat.errorsByCode) flat.errorsByCode = {};
+        for (const [code, count] of Object.entries(cat.errorsByCode)) {
+          flat.errorsByCode[code] = (flat.errorsByCode[code] || 0) + count;
+        }
+      }
+      delete engineMetrics[key];
+    }
+    return metrics;
+  });
+}
+
+let _watchCcTriageMigratedThisProcess = false;
+
+// Startup-safe wrapper: runs migrateWatchCcTriageKeys() at most once per
+// process (subsequent calls, e.g. on every readMetrics(), are a cheap
+// no-op flag check). Failures (e.g. SQLite unavailable) are swallowed and
+// retried on the next process start rather than surfaced to callers.
+function migrateWatchCcTriageKeysOnce() {
+  if (_watchCcTriageMigratedThisProcess) return;
+  _watchCcTriageMigratedThisProcess = true;
+  try { migrateWatchCcTriageKeys(); }
+  catch { /* best-effort — next process start retries */ }
+}
+
+function _resetWatchCcTriageMigrationFlagForTest() {
+  _watchCcTriageMigratedThisProcess = false;
+}
+
 module.exports = {
   readMetrics,
   applyMetricsMutation,
   _mirrorJsonFromSql,
   _resetMetricsTableForTest,
+  migrateWatchCcTriageKeys,
+  migrateWatchCcTriageKeysOnce,
+  _resetWatchCcTriageMigrationFlagForTest,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2347",
+  "version": "0.1.2348",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"