@nerviq/cli 1.26.0 → 1.27.1

package/src/formatters/markdown.js

@@ -1,135 +1,164 @@
-/**
- * Markdown Formatter
- *
- * Converts a nerviq audit result into GitHub-flavoured markdown suitable
- * for posting as a PR comment. Structure:
- *
- *   - Header with score badge and pass/fail/skip counts
- *   - Top 5 topNextActions as a GitHub task-list checklist
- *   - Collapsible <details> block with the full failed-checks table
- *   - Footer with Nerviq link, version, timestamp
- *
- * Output is plain GitHub-flavoured markdown. The only HTML used is
- * <details>/<summary>, which GitHub renders natively.
- */
-
-'use strict';
-
-const { version: nerviqVersion } = require('../../package.json');
-
-function escapeCell(value) {
-  if (value === null || value === undefined) return '';
-  return String(value)
-    .replace(/\r?\n/g, ' ')
-    .replace(/\|/g, '\\|');
-}
-
-function escapeInline(value) {
-  if (value === null || value === undefined) return '';
-  return String(value).replace(/\r?\n/g, ' ');
-}
-
-function scoreBadge(score) {
-  const s = Number.isFinite(score) ? Math.round(score) : 0;
-  let color;
-  if (s >= 80) color = 'brightgreen';
-  else if (s >= 60) color = 'yellow';
-  else color = 'red';
-  return `![score](https://img.shields.io/badge/nerviq_score-${s}%2F100-${color})`;
-}
-
-function severityFor(item) {
-  return item.severity || item.impact || 'medium';
-}
-
-function formatMarkdown(auditResult, options = {}) {
-  const platformLabel = auditResult.platformLabel || auditResult.platform || 'claude';
-  const score = auditResult.score ?? 0;
-  const passed = auditResult.passed ?? 0;
-  const failed = auditResult.failed ?? 0;
-  const skipped = auditResult.skipped ?? 0;
-  const version = auditResult.version || nerviqVersion;
-  const timestamp = auditResult.timestamp || new Date().toISOString();
-
-  const lines = [];
-
-  lines.push(`## Score: ${score}/100 ${scoreBadge(score)}`);
-  lines.push('');
-  lines.push(`**Platform:** ${platformLabel}  `);
-  lines.push(`**Checks:** ${passed} passed, ${failed} failed, ${skipped} skipped`);
-  lines.push('');
-
-  const top = Array.isArray(auditResult.topNextActions)
-    ? auditResult.topNextActions.slice(0, 5)
-    : [];
-
-  if (top.length > 0) {
-    lines.push('### Top next actions');
-    lines.push('');
-    for (const item of top) {
-      const sev = severityFor(item).toString().toUpperCase();
-      const title = escapeInline(item.name || item.title || item.key);
-      const key = escapeInline(item.key || item.id || '');
-      let loc = '';
-      if (item.file) {
-        loc = ` — \`${escapeInline(item.file)}${item.line ? ':' + item.line : ''}\``;
-      }
-      let delta = '';
-      if (Number.isFinite(item.projectedScoreDelta) && item.projectedScoreDelta > 0) {
-        delta = ` (+${item.projectedScoreDelta} pts → ${item.projectedScoreAfter}/100)`;
-      }
-      // CTO-08: include scope layer as a small suffix after the key so
-      // evaluators see which layer each next-action belongs to.
-      const layerSuffix = item.layer ? ` · _layer: ${escapeInline(item.layer)}_` : '';
-      lines.push(`- [ ] **[${sev}] ${title}** (\`${key}\`)${loc}${delta}${layerSuffix}`);
-      const hint = item.fix || item.hint || '';
-      if (hint) {
-        lines.push(`  - ${escapeInline(hint)}`);
-      }
-      if (item.snippet) {
-        lines.push('');
-        lines.push('  ```');
-        for (const snipLine of String(item.snippet).split(/\r?\n/)) {
-          lines.push(`  ${snipLine}`);
-        }
-        lines.push('  ```');
-      }
-    }
-    lines.push('');
-  }
-
-  const failedResults = Array.isArray(auditResult.results)
-    ? auditResult.results.filter((r) => r.passed === false)
-    : [];
-
-  if (failedResults.length > 0) {
-    lines.push('<details>');
-    lines.push(`<summary>All failed checks (${failedResults.length})</summary>`);
-    lines.push('');
-    // CTO-08: add layer column between category and rating.
-    lines.push('| key | name | category | layer | rating | file | line |');
-    lines.push('| --- | --- | --- | --- | --- | --- | --- |');
-    for (const r of failedResults) {
-      const row = [
-        escapeCell(r.key),
-        escapeCell(r.name),
-        escapeCell(r.category),
-        escapeCell(r.layer || ''),
-        escapeCell(r.rating ?? ''),
-        escapeCell(r.file || ''),
-        escapeCell(r.line || ''),
-      ];
-      lines.push(`| ${row.join(' | ')} |`);
-    }
-    lines.push('');
-    lines.push('</details>');
-    lines.push('');
-  }
-
-  lines.push(`---`);
-  lines.push(`Generated by [Nerviq](https://nerviq.net) v${version} · ${timestamp}`);
-
-  return lines.join('\n');
-}
-
-module.exports = { formatMarkdown };
+/**
+ * Markdown Formatter
+ *
+ * Converts a nerviq audit result into GitHub-flavoured markdown suitable
+ * for posting as a PR comment. Structure:
+ *
+ *   - Header with score badge and pass/fail/skip counts
+ *   - Top 5 topNextActions as a GitHub task-list checklist
+ *   - Collapsible <details> block with the full failed-checks table
+ *   - Footer with Nerviq link, version, timestamp
+ *
+ * Output is plain GitHub-flavoured markdown. The only HTML used is
+ * <details>/<summary>, which GitHub renders natively.
+ */
+
+'use strict';
+
+const { version: nerviqVersion } = require('../../package.json');
+const { SHALLOW_RISK_BANNER_LINES } = require('../shallow-risk');
+
+function escapeCell(value) {
+  if (value === null || value === undefined) return '';
+  return String(value)
+    .replace(/\r?\n/g, ' ')
+    .replace(/\|/g, '\\|');
+}
+
+function escapeInline(value) {
+  if (value === null || value === undefined) return '';
+  return String(value).replace(/\r?\n/g, ' ');
+}
+
+function scoreBadge(score) {
+  const s = Number.isFinite(score) ? Math.round(score) : 0;
+  let color;
+  if (s >= 80) color = 'brightgreen';
+  else if (s >= 60) color = 'yellow';
+  else color = 'red';
+  return `![score](https://img.shields.io/badge/nerviq_score-${s}%2F100-${color})`;
+}
+
+function severityFor(item) {
+  return item.severity || item.impact || 'medium';
+}
+
+function formatMarkdown(auditResult, options = {}) {
+  const platformLabel = auditResult.platformLabel || auditResult.platform || 'claude';
+  const score = auditResult.score ?? 0;
+  const passed = auditResult.passed ?? 0;
+  const failed = auditResult.failed ?? 0;
+  const skipped = auditResult.skipped ?? 0;
+  const version = auditResult.version || nerviqVersion;
+  const timestamp = auditResult.timestamp || new Date().toISOString();
+
+  const lines = [];
+
+  lines.push(`## Score: ${score}/100 ${scoreBadge(score)}`);
+  lines.push('');
+  lines.push(`**Platform:** ${platformLabel}  `);
+  lines.push(`**Checks:** ${passed} passed, ${failed} failed, ${skipped} skipped`);
+  lines.push('');
+
+  const top = Array.isArray(auditResult.topNextActions)
+    ? auditResult.topNextActions.slice(0, 5)
+    : [];
+
+  if (top.length > 0) {
+    lines.push('### Top next actions');
+    lines.push('');
+    for (const item of top) {
+      const sev = severityFor(item).toString().toUpperCase();
+      const title = escapeInline(item.name || item.title || item.key);
+      const key = escapeInline(item.key || item.id || '');
+      let loc = '';
+      if (item.file) {
+        loc = ` — \`${escapeInline(item.file)}${item.line ? ':' + item.line : ''}\``;
+      }
+      let delta = '';
+      if (Number.isFinite(item.projectedScoreDelta) && item.projectedScoreDelta > 0) {
+        delta = ` (+${item.projectedScoreDelta} pts → ${item.projectedScoreAfter}/100)`;
+      }
+      // CTO-08: include scope layer as a small suffix after the key so
+      // evaluators see which layer each next-action belongs to.
+      const layerSuffix = item.layer ? ` · _layer: ${escapeInline(item.layer)}_` : '';
+      lines.push(`- [ ] **[${sev}] ${title}** (\`${key}\`)${loc}${delta}${layerSuffix}`);
+      const hint = item.fix || item.hint || '';
+      if (hint) {
+        lines.push(`  - ${escapeInline(hint)}`);
+      }
+      if (item.snippet) {
+        lines.push('');
+        lines.push('  ```');
+        for (const snipLine of String(item.snippet).split(/\r?\n/)) {
+          lines.push(`  ${snipLine}`);
+        }
+        lines.push('  ```');
+      }
+    }
+    lines.push('');
+  }
+
+  if (Array.isArray(auditResult.shallowRiskHints)) {
+    lines.push('### Shallow Risk (experimental, opt-in)');
+    lines.push('');
+    for (const line of SHALLOW_RISK_BANNER_LINES) {
+      lines.push(`> ${line}`);
+    }
+    lines.push('');
+
+    if (auditResult.shallowRiskHints.length === 0) {
+      lines.push('_No shallow-risk hints found._');
+      lines.push('');
+    } else {
+      for (const item of auditResult.shallowRiskHints) {
+        const sev = severityFor(item).toString().toUpperCase();
+        const title = escapeInline(item.name || item.title || item.key);
+        let loc = '';
+        if (item.file) {
+          loc = ` — \`${escapeInline(item.file)}${item.line ? ':' + item.line : ''}\``;
+        }
+        lines.push(`- **[${sev}] ${title}** (\`${escapeInline(item.key || '')}\`)${loc}`);
+        if (item.fix) {
+          lines.push(`  - ${escapeInline(item.fix)}`);
+        }
+      }
+      lines.push('');
+    }
+  }
+
+  const failedResults = Array.isArray(auditResult.results)
+    ? auditResult.results.filter((r) => r.passed === false)
+    : [];
+
+  if (failedResults.length > 0) {
+    lines.push('<details>');
+    lines.push(`<summary>All failed checks (${failedResults.length})</summary>`);
+    lines.push('');
+    // CTO-08: add layer column between category and rating.
+    lines.push('| key | name | category | layer | rating | file | line |');
+    lines.push('| --- | --- | --- | --- | --- | --- | --- |');
+    for (const r of failedResults) {
+      const row = [
+        escapeCell(r.key),
+        escapeCell(r.name),
+        escapeCell(r.category),
+        escapeCell(r.layer || ''),
+        escapeCell(r.rating ?? ''),
+        escapeCell(r.file || ''),
+        escapeCell(r.line || ''),
+      ];
+      lines.push(`| ${row.join(' | ')} |`);
+    }
+    lines.push('');
+    lines.push('</details>');
+    lines.push('');
+  }
+
+  lines.push(`---`);
+  lines.push(`Generated by [Nerviq](https://nerviq.net) v${version} · ${timestamp}`);
+
+  return lines.join('\n');
+}
+
+module.exports = { formatMarkdown };

package/src/gemini/setup.js

@@ -6,6 +6,7 @@ const { STACKS } = require('../techniques');
 const { writeActivityArtifact, writeRollbackArtifact } = require('../activity');
 const { GeminiProjectContext } = require('./context');
 const { recommendGeminiMcpPacks, packToJson } = require('./mcp-packs');
+const { icon } = require('../output-icons');
 
 function detectScripts(ctx) {
   const pkg = ctx.jsonFile('package.json');
@@ -622,14 +623,14 @@ async function setupGemini(options) {
     const fullPath = path.join(options.dir, file.path);
     if (fs.existsSync(fullPath)) {
       preservedFiles.push(file.path);
-      log(`  \x1b[2m⏭️  Skipped ${file.path} (already exists — your version is kept)\x1b[0m`);
+      log(`  \x1b[2m${icon('skip')} Skipped ${file.path} (already exists - your version is kept)\x1b[0m`);
       continue;
     }
 
     fs.mkdirSync(path.dirname(fullPath), { recursive: true });
     fs.writeFileSync(fullPath, file.content, 'utf8');
     writtenFiles.push(file.path);
-    log(`  \x1b[32m✅\x1b[0m Created ${file.path}`);
+    log(`  \x1b[32m${icon('ok')}\x1b[0m Created ${file.path}`);
   }
 
   const skippedSet = new Set(preservedFiles);

package/src/init.js

@@ -6,6 +6,7 @@ const { audit } = require('./audit');
 const { setup } = require('./setup');
 const { ProjectContext } = require('./context');
 const { STACKS } = require('./techniques');
+const { icon } = require('./output-icons');
 
 const PLATFORM_LABELS = {
   claude: 'Claude Code',
@@ -136,10 +137,10 @@ async function runInit(dir, flags) {
   });
 
   for (const f of setupResult.writtenFiles) {
-    console.log(`  ${green}✅${reset} Created ${f}`);
+    console.log(`  ${green}${icon('ok')}${reset} Created ${f}`);
   }
   for (const f of setupResult.preservedFiles) {
-    console.log(`  ${dim}⏭️  Kept ${f} (already exists)${reset}`);
+    console.log(`  ${dim}${icon('skip')} Kept ${f} (already exists)${reset}`);
   }
 
   // --- Run additional platform setups ---
@@ -153,7 +154,7 @@ async function runInit(dir, flags) {
         mcpPacks: [],
       });
       for (const f of extraResult.writtenFiles) {
-        console.log(`  ${green}✅${reset} Created ${f}`);
+        console.log(`  ${green}${icon('ok')}${reset} Created ${f}`);
       }
     } catch {
       // Platform setup not available, skip

package/src/opencode/context.js

@@ -203,12 +203,51 @@ class OpenCodeProjectContext extends ProjectContext {
   }
 
   skillDirs() {
-    const skillsDir = path.join(this.dir, '.opencode', 'commands');
-    return listDirs(skillsDir).map(entry => entry.name);
+    const names = new Set();
+    const roots = [
+      path.join('.opencode', 'skills'),
+      path.join('.opencode', 'skill'),
+      path.join('.claude', 'skills'),
+      path.join('.agents', 'skills'),
+    ];
+
+    for (const root of roots) {
+      const fullRoot = path.join(this.dir, root);
+      for (const entry of listDirs(fullRoot)) {
+        if (this.fileContent(path.join(root, entry.name, 'SKILL.md'))) {
+          names.add(entry.name);
+        }
+      }
+    }
+
+    // Legacy NERVIQ compatibility: older generated fixtures placed skills
+    // under .opencode/commands/<name>/SKILL.md before OpenCode's native
+    // .opencode/skills/ path was verified.
+    const legacyCommandsRoot = path.join(this.dir, '.opencode', 'commands');
+    for (const entry of listDirs(legacyCommandsRoot)) {
+      if (this.fileContent(path.join('.opencode', 'commands', entry.name, 'SKILL.md'))) {
+        names.add(entry.name);
+      }
+    }
+
+    return [...names];
   }
 
   skillMetadata(name) {
-    return this.fileContent(path.join('.opencode', 'commands', name, 'SKILL.md'));
+    const candidates = [
+      path.join('.opencode', 'skills', name, 'SKILL.md'),
+      path.join('.opencode', 'skill', name, 'SKILL.md'),
+      path.join('.claude', 'skills', name, 'SKILL.md'),
+      path.join('.agents', 'skills', name, 'SKILL.md'),
+      path.join('.opencode', 'commands', name, 'SKILL.md'),
+    ];
+
+    for (const candidate of candidates) {
+      const content = this.fileContent(candidate);
+      if (content) return content;
+    }
+
+    return null;
   }
 
   themeFiles() {