skills-atlas-cli 0.8.4 → 0.8.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skills-atlas-cli",
-  "version": "0.8.4",
+  "version": "0.8.5",
   "description": "Search, install and learn AI agent skills from the terminal — powered by the Skills Atlas catalog.",
   "bin": {
     "skills-atlas": "bin/skills.js",

package/src/commands/suggest.js

@@ -20,8 +20,7 @@ const transcripts = require('../transcripts');
 const gapstate = require('../gapstate');
 
 const COOLDOWN = 3; // min prompts between suggestions
-const GAP_EVERY = 12;                 // earliest a gap nudge may fire (per session)
-const NUDGE_COOLDOWN_MS = 24 * 3600000; // and at most one gap nudge per day (across sessions)
+const GAP_EVERY = 12; // a gap nudge is only considered at every Nth prompt (per session)
 
 function stateFile(sessionId) {
   const base = process.env.XDG_CACHE_HOME || path.join(os.homedir(), '.cache');
@@ -58,12 +57,14 @@ module.exports = async function suggest() {
     state.count = (state.count || 0) + 1;
     const ap = registry.getAutopilot();
 
-    // --- Proactive gap nudge: periodic + throttled to once/day; Claude judges ---
+    // --- Proactive gap nudge: periodic, and refired when the recurring work shifts
+    // to something new (anti-spam floor + activity fingerprint, not a daily clock) ---
     if (ap.gapAlerts && state.count % GAP_EVERY === 0) {
-      const gs = gapstate.read();
-      if (Date.now() - (gs.lastNudge || 0) >= NUDGE_COOLDOWN_MS) {
-        const recent = transcripts.recentPrompts({ max: 20 });
-        if (recent.length >= 8) {
+      const recent = transcripts.recentPrompts({ max: 20 });
+      if (recent.length >= 8) {
+        const gs = gapstate.read();
+        const { fire, sig } = gapstate.shouldNudge(gs, recent, Date.now());
+        if (fire) {
           const dismissed = gs.dismissed || [];
           const lines = recent.map(r => `- ${r.text.replace(/\s+/g, ' ').slice(0, 100)}`).join('\n');
           const days = Math.max(1, Math.round((Date.now() - recent[recent.length - 1].ts) / 86400000));
@@ -73,7 +74,7 @@ module.exports = async function suggest() {
             `\`skills-atlas info <skill>\` and install with \`skills-atlas use <skill> --yes\`.` +
             (dismissed.length ? ` Already dismissed (skip): ${dismissed.join(', ')}.` : '') +
             ` If nothing clearly recurs or it doesn't fit right now, stay silent.`);
-          gapstate.touchNudge();
+          gapstate.touchNudge(sig);
           writeState(file, state);
           return;
         }

package/src/gapstate.js

@@ -1,10 +1,18 @@
 // The only thing capability-gaps persists: which suggestions the user dismissed,
-// and when we last proactively nudged. (Judgment is Claude's; no prompt text here.)
+// when we last proactively nudged, and a fingerprint of what the user was doing at
+// that nudge. (Judgment is Claude's; no prompt text is stored — only token stems.)
 'use strict';
 
 const fs = require('fs');
 const os = require('os');
 const path = require('path');
+const { tokenize } = require('./search-core');
+
+// Gap nudges are gated by activity, not a wall clock: a short anti-spam floor, then
+// a refire whenever the recurring work shifts to something new (so it can catch the
+// user on their NEXT task), plus a long fallback so a persistent gap can resurface.
+const MIN_INTERVAL_MS = 90 * 60 * 1000;          // ~90 min: never two nudges in a burst
+const REFRESH_INTERVAL_MS = 12 * 60 * 60 * 1000; // ~12 h: an unchanged gap may resurface
 
 function file() {
   const base = process.env.XDG_CACHE_HOME || path.join(os.homedir(), '.cache');
@@ -19,7 +27,45 @@ function write(s) {
 }
 function dismiss(x) { const s = read(); if (x && !s.dismissed.includes(x)) s.dismissed.push(x); write(s); return s; }
 function isDismissed(x) { return read().dismissed.includes(x); }
-function touchNudge() { const s = read(); s.lastNudge = Date.now(); write(s); }
-function clear() { write({ dismissed: [], lastNudge: 0 }); }
+function touchNudge(sig) { const s = read(); s.lastNudge = Date.now(); if (sig) s.lastSig = sig; write(s); }
+function clear() { write({ dismissed: [], lastNudge: 0, lastSig: [] }); }
+
+// A coarse fingerprint of recent work: the most frequent contentful tokens across
+// recent prompts. When this set shifts, the user has moved to a new kind of work.
+function activitySignature(prompts, topN = 8) {
+  const freq = new Map();
+  for (const p of prompts || []) {
+    const text = typeof p === 'string' ? p : (p && p.text) || '';
+    for (const t of tokenize(String(text).toLowerCase())) freq.set(t, (freq.get(t) || 0) + 1);
+  }
+  return [...freq.entries()]
+    .sort((a, b) => b[1] - a[1] || (a[0] < b[0] ? -1 : 1))
+    .slice(0, topN).map(e => e[0]).sort();
+}
+
+// Did the dominant activity shift by more than half (Jaccard < 0.5)? No prior → yes.
+function signatureShifted(sig, prev) {
+  if (!prev || !prev.length) return true;
+  const a = new Set(sig);
+  if (!a.size) return false;
+  const b = new Set(prev);
+  let inter = 0; for (const x of a) if (b.has(x)) inter++;
+  const union = new Set([...a, ...b]).size || 1;
+  return inter / union < 0.5;
+}
+
+// Smart-refire decision (pure). Fire when past the anti-spam floor AND either the
+// activity shifted to something new, or the long fallback has elapsed.
+function shouldNudge(state, recent, now) {
+  const sig = activitySignature(recent);
+  const since = now - ((state && state.lastNudge) || 0);
+  if (since < MIN_INTERVAL_MS) return { fire: false, sig };
+  const fire = signatureShifted(sig, state && state.lastSig) || since >= REFRESH_INTERVAL_MS;
+  return { fire, sig };
+}
 
-module.exports = { file, read, write, dismiss, isDismissed, touchNudge, clear };
+module.exports = {
+  file, read, write, dismiss, isDismissed, touchNudge, clear,
+  activitySignature, signatureShifted, shouldNudge,
+  MIN_INTERVAL_MS, REFRESH_INTERVAL_MS,
+};