promptup-plugin 0.1.8 → 0.2.0

package/dist/evaluator.js

@@ -7,10 +7,14 @@
  * STANDALONE copy — no imports from @promptup/shared or session-watcher.
  */
 import { spawn } from 'node:child_process';
+import { writeFileSync, unlinkSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
 import { ulid } from 'ulid';
 import { BASE_DIMENSIONS, BASE_DIMENSION_KEYS, DOMAIN_DIMENSIONS, DOMAIN_DIMENSION_KEYS, WEIGHT_PROFILES, } from './shared/dimensions.js';
 import { computeCompositeScore, computeDomainComposite, computeTechComposite, computeOverallComposite, computeGrandComposite, computeRiskFlagsWithHistory, } from './shared/scoring.js';
 import { getLatestEvaluation, insertEvaluation, insertDecision, } from './db.js';
+import { detectDecisions } from './decision-detector.js';
 /**
  * Combined role + skill roadmaps catalog for tech detection.
  * Mirrors the full list from @promptup/shared/roadmaps without importing it.
@@ -182,12 +186,18 @@ Return ONLY valid JSON with no markdown formatting, no code fences, no extra tex
 }
 function runClaudeCode(prompt, timeoutMs = 180_000) {
     return new Promise((resolve, reject) => {
+        // Write prompt to temp file to avoid stdin piping conflicts.
+        // The MCP server uses stdio for its protocol — spawning claude -p
+        // with piped stdin from within an MCP server causes hangs because
+        // the child's stdin competes with the parent's MCP pipe.
+        const tmpFile = join(tmpdir(), `promptup-eval-${Date.now()}.txt`);
+        writeFileSync(tmpFile, prompt, 'utf-8');
         // Strip CLAUDECODE env var to allow spawning from within a Claude Code session
         const env = { ...process.env };
         delete env.CLAUDECODE;
         delete env.CLAUDE_CODE;
-        const proc = spawn('claude', ['-p', '--output-format', 'text', '--no-session-persistence'], {
-            stdio: ['pipe', 'pipe', 'pipe'],
+        const proc = spawn('bash', ['-c', `cat "${tmpFile}" | claude -p --output-format text --no-session-persistence`], {
+            stdio: ['ignore', 'pipe', 'pipe'],
             env,
         });
         let stdout = '';
@@ -196,10 +206,12 @@ function runClaudeCode(prompt, timeoutMs = 180_000) {
         proc.stderr.on('data', (chunk) => { stderr += chunk.toString(); });
         const timer = setTimeout(() => {
             proc.kill('SIGTERM');
+            try { unlinkSync(tmpFile); } catch {}
             reject(new Error(`[timeout] Claude Code timed out after ${timeoutMs}ms (prompt size: ${prompt.length} chars)`));
         }, timeoutMs);
         proc.on('close', (code) => {
             clearTimeout(timer);
+            try { unlinkSync(tmpFile); } catch {}
             if (code === 0) {
                 resolve(stdout.trim());
             }
@@ -209,17 +221,9 @@ function runClaudeCode(prompt, timeoutMs = 180_000) {
         });
         proc.on('error', (err) => {
             clearTimeout(timer);
+            try { unlinkSync(tmpFile); } catch {}
             reject(new Error(`[spawn] Could not start claude: ${err.message}`));
         });
-        // Write prompt to stdin with backpressure handling
-        const ok = proc.stdin.write(prompt);
-        if (!ok) {
-            // Buffer is full — wait for drain before closing
-            proc.stdin.once('drain', () => { proc.stdin.end(); });
-        }
-        else {
-            proc.stdin.end();
-        }
     });
 }
 function parseClaudeResponse(raw) {
@@ -362,6 +366,19 @@ export async function evaluateSession(sessionId, messages, triggerType, weightPr
         domainDimensionScores = heuristic.domainDimensionScores;
         techExpertise = heuristicTechDetect(messages);
         recommendations = heuristic.recommendations;
+        // Extract decisions via heuristic detector (Claude path does this via LLM)
+        try {
+            const heuristicDecisions = detectDecisions(messages, sessionId);
+            if (heuristicDecisions.length > 0) {
+                for (const d of heuristicDecisions) {
+                    insertDecision(d);
+                }
+                console.log(`[eval] Heuristic extracted ${heuristicDecisions.length} decisions`);
+            }
+        }
+        catch (decErr) {
+            console.warn(`[eval] Heuristic decision extraction failed: ${decErr}`);
+        }
         rawEvaluation = JSON.stringify({
             activity_log: heuristicActivityLog(messages),
             domain_dimensions: domainDimensionScores,

package/dist/pr-report-generator.js

@@ -130,22 +130,23 @@ function matchSessionsToBranch(branch, commits, projectPath) {
 }
 // ─── Decision gathering ───────────────────────────────────────────────────────
 function gatherDecisions(sessionIds) {
-    // First try plugin/daemon-captured decisions
+    // Get existing decisions from DB
     const existing = getDecisionsBySessions(sessionIds);
-    if (existing.length > 0)
-        return existing;
-    // Fall back to heuristic detection from messages
-    const heuristic = [];
+    // Also run heuristic detection on sessions that have no decisions yet
+    const sessionsWithDecisions = new Set(existing.map(d => d.session_id));
     for (const sid of sessionIds) {
+        if (sessionsWithDecisions.has(sid))
+            continue; // Already has decisions from Claude or previous heuristic
         const messages = getMessagesBySession(sid, 10000, 0);
+        if (messages.length < 3)
+            continue;
         const detected = detectDecisions(messages, sid);
-        // Persist detected decisions so they're available in future queries
         for (const d of detected) {
             insertDecision(d);
         }
-        heuristic.push(...detected);
+        existing.push(...detected);
     }
-    return heuristic;
+    return existing;
 }
 function getSignal(d) {
     return d.signal ?? 'low';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promptup-plugin",
-  "version": "0.1.8",
+  "version": "0.2.0",
   "description": "AI coding skill evaluator for Claude Code — 11-dimension scoring, decision intelligence, PR reports",
   "type": "module",
   "main": "./dist/index.js",