phewsh 0.15.25 → 0.15.27

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
@@ -491,15 +492,22 @@ async function main() {
     let installed = [];
     try { installed = listHarnesses().filter(h => h.installed).map(h => h.id); } catch { /* best-effort */ }
 
+    let bestKeeper = null;
+    try {
+      const br = learning.bestRoute(outcomeStats({ project: projectName }), { minSample: 4 });
+      if (br) bestKeeper = { route: br.route, label: continuity.labelFor(br.route), keptRate: br.keptRate, total: br.total };
+    } catch { /* best-effort */ }
+
     return {
       hasIntentDir: fs.existsSync(path.join(process.cwd(), '.intent')),
       intentFileCount: intentFiles.length,
       pendingOutcomes: pending,
       installedHarnesses: installed,
-      route,
+      route: route?.id || route,
       turnsThisSession: Math.floor(messages.length / 2),
       seqStale,
       ambientOn,
+      bestKeeper,
     };
   }
 
@@ -597,6 +605,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 +964,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 +998,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;
@@ -1072,10 +1101,23 @@ async function main() {
       if (n === 5) {
         console.log('');
         console.log(`  ${b(cream('Ask another model — switch the route'))}`);
+        let mStats = null, best = null;
+        try {
+          mStats = outcomeStats({ project: projectName });
+          best = learning.bestRoute(mStats, { minSample: 3 });
+        } catch { /* best-effort */ }
+        // Only celebrate a route that actually holds up (≥50% kept) — a star on
+        // a poor rate would be dishonest. The badge shows regardless.
+        const starRoute = best && best.keptRate >= 0.5 ? best.route : null;
         for (const h of harnesses) {
-          if (h.installed) console.log(`    ${teal('/use ' + h.id.padEnd(12))} ${sage(h.label)} ${slate('(' + h.auth + ')')}`);
+          if (!h.installed) continue;
+          const badge = mStats ? learning.keptBadge(mStats, h.id) : '';
+          const star = starRoute === h.id ? ` ${green('★ keeps best')}` : '';
+          const rec = badge ? ` ${slate('· ' + badge)}` : '';
+          console.log(`    ${teal('/use ' + h.id.padEnd(12))} ${sage(h.label)} ${slate('(' + h.auth + ')')}${rec}${star}`);
         }
         if (config?.apiKey) console.log(`    ${teal('/use api'.padEnd(17))} ${sage('Direct API')} ${slate('(your key)')}`);
+        if (starRoute) console.log(`  ${slate('★ = highest kept-rate in your record so far')}`);
         console.log('');
         rl.prompt();
         return;
@@ -2268,6 +2310,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/suggest.js

@@ -38,6 +38,7 @@ function suggestAll(s = {}) {
     turnsThisSession = 0,
     seqStale = false,
     ambientOn = false,
+    bestKeeper = null,   // { route, label, keptRate, total } from the record, or null
   } = s;
 
   const out = [];
@@ -88,6 +89,18 @@ function suggestAll(s = {}) {
     });
   }
 
+  // 4b. The record has a clear favorite that isn't your current route —
+  //     auto-weighting: lean on what actually holds for you.
+  if (bestKeeper && bestKeeper.route !== route && bestKeeper.total >= 4 && bestKeeper.keptRate >= 0.6) {
+    out.push({
+      id: 'prefer-best-route',
+      priority: 55,
+      message: `${bestKeeper.label || bestKeeper.route} keeps best in your record (${Math.round(bestKeeper.keptRate * 100)}%, ${bestKeeper.total} labeled).`,
+      command: `/use ${bestKeeper.route}`,
+      why: 'The record weights routing — lean on the tool whose work you actually keep.',
+    });
+  }
+
   // 5. Ambient off — continuity that costs nothing once turned on.
   if (hasIntentDir && !ambientOn && installedHarnesses.includes('claude-code')) {
     out.push({

package/package.json

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