phewsh 0.15.24 → 0.15.26

package/commands/session.js

@@ -22,6 +22,7 @@ const { recordDecision, labelOutcome, pendingDecisions, recentDecisions, outcome
 const { suggest, suggestAll } = require('../lib/suggest');
 const continuity = require('../lib/continuity');
 const learning = require('../lib/learning');
+const recall = require('../lib/recall');
 const { closest } = require('../lib/closest');
 const cmdHistory = require('../lib/history');
 const { recordSessionEvent } = require('../lib/receipts-data');
@@ -333,6 +334,7 @@ async function main() {
   let route = resolveRoute(config, harnesses);
   let sessionMode = null;          // INTENT_MODES id once picked
   let awaitingOutcome = null;      // decision id eligible for 1-4 labeling
+  let awaitingWhy = null;          // { id, outcome } — next line is the reason
   let awaitingFallback = null;     // { input, fullSystem, options } after a route failure
   let bootstrapChoices = null;     // root-bootstrap menu entries when no project here
   let nextChoices = null;          // ranked /next suggestions awaiting a numeric pick
@@ -583,7 +585,26 @@ async function main() {
   const failureTracker = createFailureTracker();
   let lastTurnFailure = null;
 
+  // The gate looking backward: if this is close to something you already
+  // reverted or failed, say so once — quietly, before the turn runs.
+  let lastRecallId = null;
+  function recallHeadsUp(input) {
+    try {
+      const past = recentDecisions(300, { project: projectName });
+      const hit = recall.closestRegret(past, input, { project: projectName, minSimilarity: 0.5 });
+      if (!hit || hit.id === lastRecallId) return;
+      lastRecallId = hit.id;
+      let s = (hit.summary || '').replace(/\s+/g, ' ');
+      if (s.length > 50) s = s.slice(0, 49).trimEnd() + '…';
+      const verb = hit.outcome === 'failed' ? 'failed' : 'reverted';
+      console.log(`  ${peach('↩')} ${sage(`You ${verb} something close before:`)} ${slate('“' + s + '” · via ' + continuity.labelFor(hit.route) + ' · ' + continuity.agoText(hit.ts))}`);
+      if (hit.why) console.log(`  ${slate('  why it didn\'t hold: ')}${sage(hit.why)}`);
+      console.log(`  ${slate('  not a block — just so the record doesn\'t let you repeat it blind.')}`);
+    } catch { /* recall is advisory, never blocks a turn */ }
+  }
+
   async function runHarnessTurn(input, harnessId, fullSystem) {
+    recallHeadsUp(input);
     const decisionId = recordDecision({
       project: projectName, route: harnessId, mode: sessionMode, summary: input,
     });
@@ -623,6 +644,7 @@ async function main() {
   }
 
   async function runApiTurn(input, fullSystem) {
+    recallHeadsUp(input);
     const decisionId = recordDecision({
       project: projectName, route: 'api', mode: sessionMode, summary: input,
     });
@@ -935,6 +957,16 @@ async function main() {
   async function handleInput(input) {
     input = expandPastes(input);
 
+    // Answering the "why?" prompt — the whole line is the reason, not a command.
+    if (awaitingWhy) {
+      const { id, outcome } = awaitingWhy;
+      awaitingWhy = null;
+      try { labelOutcome(id, outcome, input); console.log(`  ${slate('noted — the record will remember why.')}`); } catch { /* gone */ }
+      console.log('');
+      rl.prompt();
+      return;
+    }
+
     // Any typed input supersedes a pending "did you mean" offer.
     if (pendingDidYouMean) pendingDidYouMean = null;
 
@@ -959,12 +991,22 @@ async function main() {
     // A bare 1-4 right after a routed action labels its outcome
     if (awaitingOutcome && /^[1-4]$/.test(input)) {
       const outcome = OUTCOMES[parseInt(input, 10) - 1];
+      let labeled = null;
       try {
-        labelOutcome(awaitingOutcome, outcome);
+        labeled = labelOutcome(awaitingOutcome, outcome);
         const color = outcome === 'kept' ? green : outcome === 'superseded' ? peach : ember;
         console.log(`  ${teal('●')} ${sage('outcome:')} ${color(outcome)}`);
       } catch { /* decision vanished — nothing to do */ }
+      const id = awaitingOutcome;
       awaitingOutcome = null;
+      // For a regret, the reason is the gold — capture one line so /recall can
+      // tell you *why* next time. Kept/superseded stay frictionless.
+      if (labeled && (outcome === 'reverted' || outcome === 'failed')) {
+        awaitingWhy = { id, outcome };
+        console.log(`  ${slate('why? one line — feeds /recall next time · enter to skip')}`);
+        rl.prompt();
+        return;
+      }
       console.log('');
       rl.prompt();
       return;
@@ -2248,6 +2290,8 @@ async function main() {
   const lineDispatcher = createLineDispatcher(handleInput, {
     onBatch: ({ input, lines }) => { collapsePastedEcho(lines, input); cmdHistory.append(input); },
     onNoop: () => {
+      // Bare Enter skips the "why?" prompt — the label still stands.
+      if (awaitingWhy) { awaitingWhy = null; console.log(''); rl.prompt(); return; }
       // Bare Enter accepts a pending "did you mean" suggestion.
       if (pendingDidYouMean) {
         const cmd = pendingDidYouMean;

package/lib/outcomes.js

@@ -51,7 +51,7 @@ function recordDecision({ project, route, mode, summary }) {
 }
 
 /** Label a decision by id (or unambiguous id prefix). Returns the decision or null. */
-function labelOutcome(idOrPrefix, outcome) {
+function labelOutcome(idOrPrefix, outcome, why = null) {
   if (!OUTCOMES.includes(outcome)) {
     throw new Error(`Outcome must be one of: ${OUTCOMES.join(', ')}`);
   }
@@ -60,6 +60,9 @@ function labelOutcome(idOrPrefix, outcome) {
   if (matches.length !== 1) return null;
   matches[0].outcome = outcome;
   matches[0].labeledAt = new Date().toISOString();
+  // One-line reason — the most valuable thing about a reverted/failed call.
+  // Feeds /recall ("you reverted this before — because …") and /learn.
+  if (why && String(why).trim()) matches[0].why = String(why).trim().slice(0, 200);
   save(decisions);
   return matches[0];
 }

package/lib/recall.js

@@ -0,0 +1,59 @@
+// Recall — the record warning you before you repeat a mistake.
+//
+// Every decision is labeled. When you're about to do something close to what
+// you already tried and *reverted* or *failed*, phewsh should say so — once,
+// quietly, before you spend the turn. This is the decision gate looking
+// backward: "you've been here; it didn't hold."
+//
+// Pure: feed it past decisions + the new text, get back the closest prior
+// regret, or null. Similarity is token-overlap (Jaccard) so it matches intent
+// ("add dark mode toggle" ≈ "build the dark-mode switch"), not exact strings.
+
+const STOP = new Set([
+  'the', 'a', 'an', 'to', 'of', 'and', 'or', 'for', 'in', 'on', 'at', 'with',
+  'is', 'it', 'this', 'that', 'i', 'we', 'my', 'our', 'me', 'be', 'do', 'can',
+  'will', 'would', 'should', 'let', 'lets', 'please', 'just', 'make', 'add',
+  'use', 'using', 'get', 'set', 'so', 'as', 'by', 'from', 'into', 'up',
+]);
+
+const REGRET = new Set(['reverted', 'failed']);
+
+function tokens(s) {
+  return new Set(
+    String(s || '')
+      .toLowerCase()
+      .split(/[^a-z0-9]+/)
+      .filter((w) => w.length > 2 && !STOP.has(w))
+  );
+}
+
+/** Jaccard overlap of two strings' meaningful tokens, 0..1. */
+function similarity(a, b) {
+  const A = tokens(a), B = tokens(b);
+  if (A.size === 0 || B.size === 0) return 0;
+  let inter = 0;
+  for (const t of A) if (B.has(t)) inter++;
+  return inter / (A.size + B.size - inter);
+}
+
+/**
+ * Prior reverted/failed decisions similar to `text`, most-similar first.
+ * @param {object[]} decisions
+ * @param {string} text
+ * @param {object} [opts] { project, minSimilarity=0.5 }
+ */
+function recallSimilar(decisions, text, { project = null, minSimilarity = 0.5 } = {}) {
+  return (decisions || [])
+    .filter((d) => d && REGRET.has(d.outcome) && (!project || d.project === project))
+    .map((d) => ({ ...d, similarity: similarity(text, d.summary) }))
+    .filter((d) => d.similarity >= minSimilarity)
+    .sort((a, b) => b.similarity - a.similarity);
+}
+
+/** The single closest prior regret, or null. */
+function closestRegret(decisions, text, opts = {}) {
+  const hits = recallSimilar(decisions, text, opts);
+  return hits.length ? hits[0] : null;
+}
+
+module.exports = { similarity, recallSimilar, closestRegret, tokens };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "phewsh",
-  "version": "0.15.24",
+  "version": "0.15.26",
   "description": "Turn intent into action. Structure your thinking, execute your next step.",
   "bin": {
     "phewsh": "bin/phewsh.js"