skillmds 1.0.7 → 1.0.9

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to `skillmds` are documented here. Versions follow [semver](https://semver.org).
 
+## 1.0.9
+
+- Compact lint output: one line per skill (status dot, score, name, rule-id chips) with a single summary line, instead of a tall multi-row block per skill. Linting a single skill still shows full diagnostic messages.
+
+## 1.0.8
+
+- `skillmds` with no command in a terminal now opens the interactive CLI menu; it only starts the MCP server when launched over piped stdio (how MCP clients run it). Previously a bare `npx skillmds` in a terminal appeared to hang on the silent MCP server.
+
 ## 1.0.7
 
 - Nicer CLI output: colored PASS/FAIL, quality-score bars, `ERROR`/`WARN`/`INFO` badges, and a summary with average score.

package/bin/skillmd-mcp.mjs

@@ -211,16 +211,21 @@ async function startMcpServer() {
   console.error("skillmd MCP server running (stdio)");
 }
 
-// This binary is invoked two ways:
-//   - `skillmds`                  → start the MCP server (stdio; how MCP clients launch it)
+// This binary is invoked three ways:
 //   - `skillmds lint .` etc.      → run the CLI (so `npx skillmds <cmd>` works without -p)
+//   - `skillmds` in a terminal    → run the CLI (opens the interactive menu)
+//   - `skillmds` over piped stdio → start the MCP server (how MCP clients launch it)
+// MCP clients always connect stdin as a pipe (never a TTY), so an interactive
+// stdin is a reliable signal that a human is present.
 const CLI_COMMANDS = new Set([
   "lint", "check", "scan", "rules", "init", "search", "add", "list", "ls",
   "remove", "rm", "update", "publish", "submit", "login", "logout", "help",
   "--help", "-h", "--version", "-V",
 ]);
 const first = process.argv[2];
-if (first && CLI_COMMANDS.has(first)) {
+const isCliCommand = first && CLI_COMMANDS.has(first);
+const humanInTerminal = !first && process.stdin.isTTY;
+if (isCliCommand || humanInTerminal) {
   await import("../dist/cli.js"); // the CLI parses process.argv itself
 } else {
   await startMcpServer();

package/dist/cli.js

@@ -370,40 +370,63 @@ var decorate = (text3, kind) => {
       return pc.dim(text3);
   }
 };
-var badge = {
-  error: pc.bgRed(pc.white(" ERROR ")),
-  warn: pc.bgYellow(pc.black(" WARN  ")),
-  info: pc.bgCyan(pc.black(" INFO  "))
+var GLYPH = {
+  error: pc.red("\u2716"),
+  warn: pc.yellow("\u26A0"),
+  info: pc.cyan("\xB7")
 };
-function scoreBar(score) {
-  const filled = Math.max(0, Math.min(10, Math.round(score / 10)));
-  const bar = "\u2588".repeat(filled) + "\u2591".repeat(10 - filled);
-  const color = score >= 80 ? pc.green : score >= 50 ? pc.yellow : pc.red;
-  return `${color(bar)} ${pc.bold(String(score))}${pc.dim("/100")}`;
+var scoreColor = (s) => s >= 80 ? pc.green : s >= 50 ? pc.yellow : pc.red;
+function label(file) {
+  const parts = file.split(/[\\/]/).filter(Boolean);
+  const i = parts.lastIndexOf("SKILL.md");
+  return i > 0 ? parts[i - 1] : parts[parts.length - 1] || file;
 }
-function renderReport(results) {
-  const out = [];
-  for (const { file, result } of results) {
-    const status = result.ok ? pc.green("\u25CF") + pc.bold(" PASS") : pc.red("\u25CF") + pc.bold(" FAIL");
-    out.push("");
-    out.push(`${status}  ${pc.bold(file)}`);
-    out.push(`        ${scoreBar(result.score)}`);
-    for (const d of result.diagnostics) {
-      const loc = d.line !== void 0 ? pc.dim(`:${d.line}`) : "";
-      out.push(`   ${badge[d.severity]} ${pc.dim(d.id)}${loc}  ${d.message}`);
-    }
-    const flags = result.security.flags.filter((f) => f !== "docs_only");
-    if (flags.length) out.push(`   ${pc.magenta("\u2691 security")}  ${pc.dim(flags.join(", "))}`);
+function statusDot(result) {
+  if (!result.ok) return pc.red("\u25CF");
+  return result.diagnostics.some((d) => d.severity === "warn") ? pc.yellow("\u25CF") : pc.green("\u25CF");
+}
+function securityFlags(result) {
+  return result.security.flags.filter((f) => f !== "docs_only");
+}
+function renderOne(file, result) {
+  const status = result.ok ? pc.green("\u25CF PASS") : pc.red("\u25CF FAIL");
+  const out = [`${status}  ${pc.bold(label(file))}  ${scoreColor(result.score)(`${result.score}/100`)}`];
+  for (const d of result.diagnostics) {
+    const loc = d.line !== void 0 ? pc.dim(`:${d.line}`) : "";
+    out.push(`  ${GLYPH[d.severity]} ${pc.dim(d.id)}${loc}  ${d.message}`);
   }
+  const flags = securityFlags(result);
+  if (flags.length) out.push(`  ${pc.magenta(`\u2691 ${flags.join(", ")}`)}`);
+  return out.join("\n");
+}
+function renderMany(results) {
+  const names = results.map((r) => label(r.file));
+  const w = Math.min(34, Math.max(4, ...names.map((n) => n.length)));
+  const out = [];
+  results.forEach(({ result }, i) => {
+    const score = scoreColor(result.score)(String(result.score).padStart(3));
+    const name = names[i].length > w ? names[i].slice(0, w - 1) + "\u2026" : names[i].padEnd(w);
+    const issues = result.diagnostics.map((d) => `${GLYPH[d.severity]}${pc.dim(d.id)}`).join(" ");
+    const flags = securityFlags(result);
+    const sec = flags.length ? `  ${pc.magenta(`\u2691 ${flags.join(",")}`)}` : "";
+    out.push(`${statusDot(result)} ${score}  ${name}  ${issues}${sec}`.trimEnd());
+  });
   const failed = results.filter((r) => !r.result.ok).length;
-  const passed = results.length - failed;
-  const avg = results.length ? Math.round(results.reduce((s, r) => s + r.result.score, 0) / results.length) : 0;
+  const warned = results.filter((r) => r.result.ok && r.result.diagnostics.some((d) => d.severity === "warn")).length;
+  const clean = results.length - failed - warned;
+  const avg = Math.round(results.reduce((s, r) => s + r.result.score, 0) / results.length);
   out.push("");
-  out.push(pc.dim("\u2500".repeat(44)));
-  const tally = failed === 0 ? pc.green(`\u2713 ${passed} passed`) : `${pc.green(`${passed} passed`)}  ${pc.red(`\u2717 ${failed} failed`)}`;
-  out.push(`${tally}    ${pc.dim("avg")} ${scoreBar(avg)}`);
+  const parts = [pc.green(`${clean} clean`)];
+  if (warned) parts.push(pc.yellow(`${warned} warn`));
+  if (failed) parts.push(pc.red(`${failed} fail`));
+  out.push(`${parts.join(pc.dim(" \xB7 "))}   ${pc.dim("avg")} ${scoreColor(avg)(`${avg}/100`)}`);
   return out.join("\n");
 }
+function renderReport(results) {
+  if (results.length === 0) return pc.dim("No skills found.");
+  if (results.length === 1) return renderOne(results[0].file, results[0].result);
+  return renderMany(results);
+}
 function exitCodeFor(results, policy = {}) {
   for (const { result } of results) {
     const hasError = result.diagnostics.some((d) => d.severity === "error");
@@ -540,9 +563,9 @@ function rulesCommand() {
       console.log(JSON.stringify(list.map((r) => ({ id: r.id, severity: r.severity, description: r.description })), null, 2));
       return;
     }
-    const badge2 = (sev) => sev === "error" ? pc4.bgRed(pc4.white(" error ")) : sev === "warn" ? pc4.bgYellow(pc4.black(" warn  ")) : pc4.bgCyan(pc4.black(" info  "));
+    const badge = (sev) => sev === "error" ? pc4.bgRed(pc4.white(" error ")) : sev === "warn" ? pc4.bgYellow(pc4.black(" warn  ")) : pc4.bgCyan(pc4.black(" info  "));
     console.log(pc4.bold("\nLint rules\n"));
-    for (const r of list) console.log(`  ${pc4.bold(r.id)}  ${badge2(r.severity)}  ${r.description}`);
+    for (const r of list) console.log(`  ${pc4.bold(r.id)}  ${badge(r.severity)}  ${r.description}`);
     console.log("");
   });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillmds",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "SkillMD for agents: the `skillmds` MCP server + the `skillmd` CLI (lint, scan, search, install, publish Agent Skills) from skillmd.com.",
   "type": "module",
   "bin": {