analyzthis_design 2.4.0 → 2.5.0

package/dist/bin/cli.js

@@ -60,6 +60,10 @@ Usage:
   session accept  Mark a persona's last output accepted, for training export
                   (--persona <id>, optional --reject to unmark)
                   Optional feedback: --comment, --correction, --rating 1-5, --tags a,b
+  accept          Designer keep / skip (same as /accept). --keep or --fix
+                  Optional: --persona, --because "one sentence", --liked (keep, not shipped)
+  share           Designer send-to-package (same as /share). Preview, then --send
+                  Optional: --because "one sentence", --persona
 
 ── Persona feedback commands ─────────────────────────────────
   feedback record   Record unhappiness or a correction for a persona output
@@ -220,6 +224,10 @@ Usage:
   npx analyzthis_design devi respond --run ~/.analyzthis_design/runs/... --step 001-arjun --file out.md
   npx analyzthis_design run-unchunked --task "Review this screen" --provider anthropic
 
+  npx analyzthis_design accept --keep --persona zara
+  npx analyzthis_design accept --fix --persona zara --because "too much motion on the table"
+  npx analyzthis_design share
+  npx analyzthis_design share --send
   npx analyzthis_design session accept --persona arjun
   npx analyzthis_design session accept --persona arjun --reject \\
     --comment "Invented tokens not in our DS" \\
@@ -322,6 +330,11 @@ const budgetVal    = getFlag('budget');
 const sequentialFlag = flags.includes('--sequential');
 const maxChunksVal = getFlag('max-chunks');
 const unchunkedFlag = flags.includes('--unchunked');
+const keepFlag = flags.includes('--keep');
+const fixFlag = flags.includes('--fix') || flags.includes('--no');
+const likedFlag = flags.includes('--liked');
+const becauseVal = getFlag('because');
+const sendFlag = flags.includes('--send');
 
 // ─── Commands ────────────────────────────────────────────────────────────────
 
@@ -987,12 +1000,183 @@ switch (cmd) {
     break;
   }
 
+  case 'scores': {
+    const em = require('../lib/evolution-metrics');
+    if (flags.includes('--submit') || flags.includes('--submit-dry-run')) {
+      const submitter = require('../lib/feedback-submit');
+      const dry = flags.includes('--submit-dry-run');
+      submitter.submitScores({ dryRun: dry })
+        .then((r) => {
+          if (r.skipped) console.log(`\n  Not sent: ${r.skipped}\n`);
+          else if (r.dryRun) console.log(`\n  Would send ${r.rows.length} persona score row(s). Counts only — no note text.\n`);
+          else console.log(`\n  Sent ${r.sent} persona score row(s).\n`);
+        })
+        .catch((e) => console.log(`\n  Submit failed: ${e.message}\n`));
+      break;
+    }
+    const metrics = em.computeEvolutionMetrics(projectVal ? { project: projectVal } : undefined);
+    if (personaVal) {
+      const m = metrics.personas[personaVal];
+      if (!m) {
+        console.log(`\n  Unknown persona "${personaVal}". Use: ${em.PERSONAS.join(', ')}\n`);
+        break;
+      }
+      console.log(`\n  ${personaVal}: ${m.score}/100 ${m.level}` +
+        (m.sufficient_evidence ? '' : ' (insufficient evidence)'));
+      console.log(`  ${m.evidence_count} signal(s) · shipped ${m.outcome_breakdown.shipped}` +
+        ` · revised ${m.outcome_breakdown.revised} · missed ${m.outcome_breakdown.missed}` +
+        (m.avg_rating != null ? ` · avg rating ${m.avg_rating}` : ''));
+      console.log(`  contributions: ${JSON.stringify(m.contributions)}\n`);
+      break;
+    }
+    console.log('\n' + em.formatEvolutionSummary(metrics));
+    break;
+  }
+
+  case 'system-prompt': {
+    // The only path into the personas for browser-only tools (Lovable, v0, Bolt,
+    // Replit, ChatGPT). They cannot reach the MCP server — it speaks stdio, and a
+    // cloud-hosted tool cannot spawn a process on the user's machine.
+    const builder = require('../lib/system-prompt');
+    const mode = getFlag('mode') || 'both';
+    if (!['create', 'critique', 'both'].includes(mode)) {
+      console.log(`\n  Unknown mode "${mode}". Use: create, critique, or both.\n`);
+      break;
+    }
+    const prompt = builder.buildPrompt({ mode });
+    const out = getFlag('out');
+    if (out) {
+      require('fs').writeFileSync(out, prompt);
+      console.log(`\n  Wrote ${prompt.length} chars to ${out}\n  Paste it into your tool's system instructions.\n`);
+    } else {
+      console.log(prompt);
+    }
+    break;
+  }
+
   case 'receipt': {
     const receipt = require('../lib/receipt');
     console.log('\n' + receipt.report({ project: projectVal }) + '\n');
     break;
   }
 
+  case 'accept': {
+    const accept = require('../lib/accept');
+    const wantKeep = keepFlag || yesFlag;
+    const wantFix = fixFlag;
+    try {
+      if (wantKeep && wantFix) {
+        console.error('\n  ✗  Use --keep or --fix, not both.\n');
+        process.exit(1);
+      }
+      if (!wantKeep && !wantFix) {
+        const peek = accept.preview({ project: projectVal });
+        console.log('\n  Keep the last note?');
+        console.log('    Yes:  npx analyzthis_design accept --keep' + (peek.persona ? ` --persona ${peek.persona}` : ''));
+        console.log('    No:   npx analyzthis_design accept --fix --because "one sentence"');
+        if (peek.persona) console.log(`  Last persona: ${peek.persona}`);
+        else console.log('  Last persona: not sure — add --persona zara (or arjun, meera, …)');
+        console.log('');
+        break;
+      }
+      if (wantKeep) {
+        const result = accept.keep({
+          project: projectVal,
+          persona: personaVal,
+          comment: commentVal || becauseVal || '',
+          shipped: !likedFlag,
+          note: commentVal || '',
+        });
+        console.log(`\n✅ Kept ${result.persona}'s note.` + (result.shipped ? ' Marked as shipped for evolution.' : ' Saved as liked (not shipped).') + '\n');
+        break;
+      }
+      const result = accept.fix({
+        project: projectVal,
+        persona: personaVal,
+        because: becauseVal || commentVal || '',
+        correction: correctionVal || '',
+        rating: ratingVal,
+        tags: parseFeedbackTags(tagsVal),
+      });
+      console.log(`\n✅ Logged your fix for ${result.persona}. The team can learn from it.\n`);
+    } catch (err) {
+      if (err.code === 'NEED_PERSONA') {
+        console.error('\n  ✗  Which note? Add --persona zara (or arjun, meera, priya, noor, anuj, raj).\n');
+        process.exit(1);
+      }
+      if (err.code === 'NEED_BECAUSE') {
+        console.error('\n  ✗  Add one sentence: --because "what was wrong or what you did instead"\n');
+        process.exit(1);
+      }
+      console.error(`\n  ✗  ${err.message}\n`);
+      process.exit(1);
+    }
+
+    break;
+  }
+
+  case 'share': {
+    const share = require('../lib/share');
+    const acceptMod = require('../lib/accept');
+    const wantSend = sendFlag || yesFlag;
+    (async () => {
+      try {
+        if (!wantSend) {
+          if (becauseVal || commentVal) {
+            acceptMod.fix({
+              project: projectVal,
+              persona: personaVal,
+              because: becauseVal || commentVal,
+              correction: correctionVal || becauseVal || commentVal,
+            });
+          }
+          const peek = share.preview({ project: projectVal, persona: personaVal, all: allFlag });
+          if (!peek.count) {
+            console.log('\n  Nothing to send yet. After a critique: /accept no plus one sentence, then /share.\n');
+            return;
+          }
+          console.log(`\n  Ready to send ${peek.count} note(s) to the package (redacted — no folders, emails, or keys).`);
+          peek.rows.slice(0, 5).forEach((r) => {
+            console.log(`    • ${r.persona}: ${(r.comment || r.correction || '').slice(0, 100)}`);
+          });
+          if (peek.rows.length > 5) console.log(`    … and ${peek.rows.length - 5} more`);
+          console.log(peek.hasHttp ? '  Path: configured HTTP endpoint.' : '  Path: GitHub issue on joshirishi/analyzthis_design.');
+          console.log('  Send?  Run: npx analyzthis_design share --send\n');
+          return;
+        }
+        const result = await share.send({
+          project: projectVal,
+          persona: personaVal,
+          because: becauseVal || commentVal || '',
+          correction: correctionVal || '',
+          all: allFlag,
+        });
+        if (result.via === 'github-link') {
+          console.log(`\n  ${result.message}`);
+          console.log(`  ${result.url}\n`);
+          return;
+        }
+        console.log(`\n✅ ${result.message}` + (result.url ? `\n   ${result.url}` : '') + '\n');
+      } catch (err) {
+        if (err.code === 'NOTHING') {
+          console.error('\n  ✗  Nothing to send yet. /accept no plus one sentence, then /share.\n');
+          process.exit(1);
+        }
+        if (err.code === 'NEED_PERSONA') {
+          console.error('\n  ✗  Which note? Add --persona zara (or arjun, meera, priya, noor, anuj, raj).\n');
+          process.exit(1);
+        }
+        if (err.code === 'NEED_BECAUSE') {
+          console.error('\n  ✗  Add one sentence: --because "what was wrong or what you did instead"\n');
+          process.exit(1);
+        }
+        console.error(`\n  ✗  ${err.message}\n`);
+        process.exit(1);
+      }
+    })();
+    break;
+  }
+
   // ── Persona feedback (v1.16) ────────────────────────────────────────────
 
   case 'feedback': {

package/dist/lib/accept.js

@@ -0,0 +1,108 @@
+'use strict';
+
+/**
+ * Designer keep / skip for evolution.
+ * Slash /accept talks in yes/no. This module writes the session flags.
+ */
+
+var session = require('./session');
+var feedback = require('./feedback');
+var outcome = require('./outcome');
+var lessons = require('./lessons');
+
+var PERSONA_IDS = session.PERSONA_IDS;
+
+function resolvePersona(opts) {
+  var raw = String(opts.persona || '').toLowerCase().trim();
+  if (raw && PERSONA_IDS.indexOf(raw) === -1) {
+    var err = new Error('Unknown persona. Use: ' + PERSONA_IDS.join(', '));
+    err.code = 'BAD_PERSONA';
+    throw err;
+  }
+  var id = raw || session.inferLastPersona({ project: opts.project });
+  if (!id) {
+    var need = new Error('NEED_PERSONA');
+    need.code = 'NEED_PERSONA';
+    throw need;
+  }
+  return id;
+}
+
+// Lesson extraction must never break a designer's keep/skip.
+function safeExtract(project, persona) {
+  try {
+    var r = lessons.extractLessons({ project: project, persona: persona });
+    return (r && r.extracted) || 0;
+  } catch (e) {
+    return 0;
+  }
+}
+
+function keep(opts) {
+  opts = opts || {};
+  var persona = resolvePersona(opts);
+  session.ensurePersonaOutput({
+    project: opts.project,
+    persona: persona,
+    text: opts.note,
+  });
+  session.markAccepted({ project: opts.project, persona: persona, accepted: true });
+  // Always record — a bare "/accept yes" is still a rating, and scoring reads it.
+  feedback.recordFeedback({
+    project: opts.project,
+    persona: persona,
+    satisfied: true,
+    rating: opts.rating != null ? Number(opts.rating) : 5,
+    comment: opts.comment || '',
+    correction: '',
+    tags: ['positive'],
+    markRejected: false,
+  });
+  var shipped = opts.shipped !== false;
+  if (shipped) {
+    outcome.confirmOutcome({ project: opts.project, persona: persona, outcome: 'shipped' });
+  }
+  var learned = safeExtract(opts.project, persona);
+  return { persona: persona, kept: true, shipped: shipped, lessons: learned };
+}
+
+function fix(opts) {
+  opts = opts || {};
+  var persona = resolvePersona(opts);
+  var because = String(opts.because || opts.comment || '').trim();
+  var correction = String(opts.correction || because).trim();
+  if (!because) {
+    var need = new Error('NEED_BECAUSE');
+    need.code = 'NEED_BECAUSE';
+    throw need;
+  }
+  session.ensurePersonaOutput({
+    project: opts.project,
+    persona: persona,
+    text: opts.note,
+  });
+  feedback.recordFeedback({
+    project: opts.project,
+    persona: persona,
+    satisfied: false,
+    rating: opts.rating != null ? Number(opts.rating) : 2,
+    comment: because,
+    correction: correction,
+    tags: opts.tags || ['other'],
+    markRejected: true,
+  });
+  outcome.confirmOutcome({ project: opts.project, persona: persona, outcome: 'revised' });
+  var learnedFix = safeExtract(opts.project, persona);
+  return { persona: persona, kept: false, because: because, lessons: learnedFix };
+}
+
+function preview(opts) {
+  opts = opts || {};
+  var state = session.show({ project: opts.project });
+  return {
+    persona: session.inferLastPersona({ project: opts.project }),
+    hasSession: !!state,
+  };
+}
+
+module.exports = { keep, fix, preview, PERSONA_IDS };