@nerviq/cli 1.29.0 → 1.30.0

package/src/auto-suggest.js

@@ -56,13 +56,20 @@ function analyzeSuggestions(dir) {
   let bootstrap = { ready: true, state: 'ready', message: null, steps: [] };
 
   if (totalEvents === 0 && auditSnapshots.length === 0) {
+    // BUG-07 fix: when feedback was just recorded but pattern events are 0,
+    // the user already ran `nerviq feedback` and got "No local usage..."
+    // back. After the activity.js fix, feedback now bumps usage-patterns,
+    // so this state means feedback never ran AT ALL. Be explicit about
+    // what's missing so the user doesn't think the loop is broken.
     bootstrap = {
       ready: false,
       state: 'empty',
-      message: 'No local usage or snapshot history exists yet.',
+      message: 'No local usage or snapshot history exists yet. Need at least 2 snapshots and/or 2 recorded outcomes per check before suggestions surface.',
+      missingSignals: ['snapshots', 'recorded-outcomes'],
+      threshold: { minEventsPerCheck: MIN_EVENTS, minSnapshotsForPriority: 2 },
       steps: [
         'Run `nerviq audit --snapshot` to save the baseline.',
-        'Use `nerviq fix`, `nerviq fix --all-critical`, or `nerviq feedback` to record recommendation outcomes.',
+        'Use `nerviq fix`, `nerviq fix --all-critical`, or `nerviq feedback --key <K> --status accepted` to record recommendation outcomes.',
         'Run `nerviq audit --snapshot` again after a meaningful repo change.',
         'Re-run `nerviq suggest-rules`.',
       ],

package/src/behavioral-drift.js

@@ -685,12 +685,29 @@ function analyzeBehavioralDrift(dir, options = {}) {
   const structuralSignals = buildStructuralSignals(dir, options);
   const intentSignals = collectIntentSignals(dir);
   const { findings, labels } = deriveBehavioralFindings(structuralSignals, intentSignals);
-  const score = buildBehavioralScore(structuralSignals, findings);
+
+  // BUG-05 fix: a perfect 100 alignment score on a repo with no source files
+  // is misleading — the buildBehavioralScore math starts at 100 and subtracts
+  // penalties; with zero structural signal there's nothing to subtract, so
+  // empty repos look "perfectly aligned." Surface insufficient-signal status
+  // explicitly instead of returning the bogus 100. Threshold: <5 source files
+  // is treated as insufficient signal (calibrated from the user-lab fixture
+  // where 0 source files returned 100).
+  const SUFFICIENT_SIGNAL_FLOOR = 5;
+  const insufficientSignal = structuralSignals.sourceFiles < SUFFICIENT_SIGNAL_FLOOR;
+
+  const score = insufficientSignal
+    ? null
+    : buildBehavioralScore(structuralSignals, findings);
 
   return {
     mode: 'behavioral-drift',
     scoreType: 'behavioral-alignment-score',
     score,
+    status: insufficientSignal ? 'insufficient-signal' : 'ok',
+    insufficientSignalReason: insufficientSignal
+      ? `Need ≥${SUFFICIENT_SIGNAL_FLOOR} source files to compute a meaningful alignment score; found ${structuralSignals.sourceFiles}.`
+      : null,
     scope: SCOPE_CONTRACT,
     repoSummary: {
       project: path.basename(dir),
@@ -703,7 +720,13 @@ function analyzeBehavioralDrift(dir, options = {}) {
     intentSignals,
     driftLabels: labels,
     findings,
-    nextSteps: buildBehavioralNextSteps(findings),
+    nextSteps: insufficientSignal
+      ? [{
+          key: 'add-source-code',
+          title: `Add at least ${SUFFICIENT_SIGNAL_FLOOR} source files before re-running behavioral review.`,
+          severity: 'low',
+        }]
+      : buildBehavioralNextSteps(findings),
   };
 }
 
@@ -713,6 +736,18 @@ function writeBehavioralSnapshot(dir, report, meta = {}) {
 
 function formatBehavioralReport(report, options = {}) {
   const lines = [];
+  // BUG-05 fix: handle insufficient-signal status — the score is null and a
+  // human-friendly explanation replaces the colored gauge.
+  if (report.status === 'insufficient-signal') {
+    lines.push('');
+    lines.push(c('  nerviq behavioral drift review', 'bold'));
+    lines.push(c('  ═══════════════════════════════════════', 'dim'));
+    lines.push(c('  Alignment score: insufficient signal', 'yellow'));
+    lines.push(c(`  ${report.insufficientSignalReason || 'Not enough source files to compute a meaningful score.'}`, 'dim'));
+    lines.push(c('  No score returned to avoid a misleading 100 on empty repos.', 'dim'));
+    lines.push('');
+    return lines.join('\n');
+  }
   const scoreColor = report.score >= 75 ? 'green' : report.score >= 55 ? 'yellow' : 'red';
 
   lines.push('');