phewsh 0.15.25 → 0.15.26

package/commands/session.js

@@ -334,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
@@ -597,6 +598,7 @@ async function main() {
       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 */ }
   }
@@ -955,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;
 
@@ -979,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;
@@ -2268,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/package.json

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