@yawlabs/ctxlint 0.4.0 → 0.5.0

package/dist/{cli-VYWAONGX.js → cli-7IBRPDA6.js}

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 import {
   ALL_CHECKS,
+  ALL_MCP_CHECKS,
   VERSION,
   applyFixes,
   freeEncoder,
@@ -9,7 +10,7 @@ import {
   resetTokenThresholds,
   runAudit,
   setTokenThresholds
-} from "./chunk-WEYNMCAH.js";
+} from "./chunk-FHTSMC5D.js";
 
 // src/cli.ts
 import * as fs2 from "fs";
@@ -25,30 +26,61 @@ function formatText(result, verbose = false) {
   lines.push("");
   lines.push(`Scanning ${result.projectRoot}...`);
   lines.push("");
+  const isMcpFile = (f) => f.issues.some((i) => i.check.startsWith("mcp-"));
+  const contextFiles = result.files.filter((f) => !isMcpFile(f));
+  const mcpFiles = result.files.filter((f) => isMcpFile(f));
   const totalTokens = result.summary.totalTokens;
-  lines.push(
-    `Found ${result.files.length} context file${result.files.length !== 1 ? "s" : ""} (${totalTokens.toLocaleString()} tokens total)`
-  );
-  for (const file of result.files) {
-    let desc = `  ${file.path} (${file.tokens.toLocaleString()} tokens, ${file.lines} lines)`;
-    if (file.isSymlink && file.symlinkTarget) {
-      desc = `  ${file.path} ${chalk.dim(`\u2192 ${file.symlinkTarget} (symlink)`)}`;
+  if (contextFiles.length > 0) {
+    lines.push(
+      `Found ${contextFiles.length} context file${contextFiles.length !== 1 ? "s" : ""} (${totalTokens.toLocaleString()} tokens total)`
+    );
+    for (const file of contextFiles) {
+      let desc = `  ${file.path} (${file.tokens.toLocaleString()} tokens, ${file.lines} lines)`;
+      if (file.isSymlink && file.symlinkTarget) {
+        desc = `  ${file.path} ${chalk.dim(`\u2192 ${file.symlinkTarget} (symlink)`)}`;
+      }
+      lines.push(desc);
+    }
+  }
+  if (mcpFiles.length > 0) {
+    if (contextFiles.length > 0) lines.push("");
+    lines.push(`Found ${mcpFiles.length} MCP config${mcpFiles.length !== 1 ? "s" : ""}`);
+    for (const file of mcpFiles) {
+      lines.push(`  ${file.path}`);
     }
-    lines.push(desc);
+  }
+  if (contextFiles.length === 0 && mcpFiles.length === 0) {
+    lines.push(`Found ${result.files.length} file${result.files.length !== 1 ? "s" : ""}`);
   }
   lines.push("");
-  for (const file of result.files) {
-    const fileIssues = file.issues;
-    if (fileIssues.length === 0 && !verbose) continue;
-    lines.push(chalk.underline(file.path));
-    if (fileIssues.length === 0) {
-      lines.push(chalk.green("  \u2713 All checks passed"));
-    } else {
-      for (const issue of fileIssues) {
-        lines.push(formatIssue(issue));
+  const renderFileGroup = (files) => {
+    for (const file of files) {
+      const fileIssues = file.issues;
+      if (fileIssues.length === 0 && !verbose) continue;
+      lines.push(chalk.underline(file.path));
+      if (fileIssues.length === 0) {
+        lines.push(chalk.green("  \u2713 All checks passed"));
+      } else {
+        for (const issue of fileIssues) {
+          lines.push(formatIssue(issue));
+        }
       }
+      lines.push("");
     }
-    lines.push("");
+  };
+  if (contextFiles.length > 0 && mcpFiles.length > 0) {
+    const contextWithIssues = contextFiles.filter((f) => f.issues.length > 0 || verbose);
+    const mcpWithIssues = mcpFiles.filter((f) => f.issues.length > 0 || verbose);
+    if (contextWithIssues.length > 0) {
+      lines.push(chalk.bold("Context Files"));
+      renderFileGroup(contextFiles);
+    }
+    if (mcpWithIssues.length > 0) {
+      lines.push(chalk.bold("MCP Configs"));
+      renderFileGroup(mcpFiles);
+    }
+  } else {
+    renderFileGroup(result.files);
   }
   const { errors, warnings, info } = result.summary;
   const parts = [];
@@ -113,7 +145,7 @@ function formatSarif(result) {
   for (const file of result.files) {
     for (const issue of file.issues) {
       const sarifResult = {
-        ruleId: `ctxlint/${issue.check}`,
+        ruleId: issue.ruleId ? `ctxlint/${issue.check}/${issue.ruleId}` : `ctxlint/${issue.check}`,
         level: severityToLevel[issue.severity] || "note",
         message: {
           text: issue.message + (issue.suggestion ? ` (${issue.suggestion})` : "")
@@ -194,6 +226,46 @@ function buildRuleDescriptors() {
       id: "ctxlint/frontmatter",
       shortDescription: { text: "Invalid or missing frontmatter" },
       helpUri: "https://github.com/yawlabs/ctxlint#what-it-checks"
+    },
+    {
+      id: "ctxlint/mcp-schema",
+      shortDescription: { text: "MCP config structural validation error" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-security",
+      shortDescription: { text: "Hardcoded secret in MCP config" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-commands",
+      shortDescription: { text: "MCP stdio command validation issue" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-deprecated",
+      shortDescription: { text: "Deprecated MCP transport or pattern" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-env",
+      shortDescription: { text: "MCP environment variable syntax issue" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-urls",
+      shortDescription: { text: "MCP URL validation issue" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-consistency",
+      shortDescription: { text: "MCP config inconsistency across clients" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
+    },
+    {
+      id: "ctxlint/mcp-redundancy",
+      shortDescription: { text: "Redundant MCP config entry" },
+      helpUri: "https://github.com/yawlabs/ctxlint#mcp-config-linting"
     }
   ];
 }
@@ -235,13 +307,29 @@ async function runCli() {
   const program = new Command();
   program.name("ctxlint").description(
     "Lint your AI agent context files (CLAUDE.md, AGENTS.md, etc.) against your actual codebase"
-  ).version(VERSION).argument("[path]", "Project directory to scan", ".").option("--strict", "Exit code 1 on any warning or error (for CI)", false).option("--checks <checks>", "Comma-separated list of checks to run", "").option("--format <format>", "Output format: text, json, or sarif", "text").option("--tokens", "Show token breakdown per file", false).option("--verbose", "Show passing checks too", false).option("--fix", "Auto-fix broken paths using git history and fuzzy matching", false).option("--ignore <checks>", "Comma-separated list of checks to ignore", "").option("--quiet", "Suppress all output except errors (exit code only)", false).option("--config <path>", "Path to config file (default: .ctxlintrc in project root)").option("--depth <n>", "Max subdirectory depth to scan (default: 2)", "2").action(async (projectPath, opts) => {
+  ).version(VERSION).argument("[path]", "Project directory to scan", ".").option("--strict", "Exit code 1 on any warning or error (for CI)", false).option("--checks <checks>", "Comma-separated list of checks to run", "").option("--format <format>", "Output format: text, json, or sarif", "text").option("--tokens", "Show token breakdown per file", false).option("--verbose", "Show passing checks too", false).option("--fix", "Auto-fix broken paths using git history and fuzzy matching", false).option("--ignore <checks>", "Comma-separated list of checks to ignore", "").option("--quiet", "Suppress all output except errors (exit code only)", false).option("--config <path>", "Path to config file (default: .ctxlintrc in project root)").option("--depth <n>", "Max subdirectory depth to scan (default: 2)", "2").option("--mcp", "Enable MCP config linting alongside context file checks", false).option("--mcp-only", "Run only MCP config checks, skip context file checks", false).option("--mcp-global", "Also scan user/global MCP config files (implies --mcp)", false).action(async (projectPath, opts) => {
     const resolvedPath = path2.resolve(projectPath);
     const configPath = opts.config ? path2.resolve(opts.config) : void 0;
     const config = configPath ? loadConfigFromPath(configPath) : loadConfig(resolvedPath);
+    const mcpGlobal = opts.mcpGlobal || false;
+    const mcpOnly = opts.mcpOnly || false;
+    const mcpFlag = opts.mcp || mcpGlobal || mcpOnly || config?.mcp || false;
+    const explicitChecks = opts.checks ? opts.checks.split(",").map((c) => c.trim()) : null;
+    const hasMcpInChecks = explicitChecks?.some((c) => c.startsWith("mcp-")) || false;
+    const effectiveMcp = mcpFlag || hasMcpInChecks;
+    let checks;
+    if (explicitChecks) {
+      checks = explicitChecks;
+    } else if (mcpOnly) {
+      checks = ALL_MCP_CHECKS;
+    } else if (effectiveMcp) {
+      checks = [...config?.checks || ALL_CHECKS, ...ALL_MCP_CHECKS];
+    } else {
+      checks = config?.checks || ALL_CHECKS;
+    }
     const options = {
       projectPath: resolvedPath,
-      checks: opts.checks ? opts.checks.split(",").map((c) => c.trim()) : config?.checks || ALL_CHECKS,
+      checks,
       strict: opts.strict || config?.strict || false,
       format: opts.format,
       verbose: opts.verbose,
@@ -249,7 +337,10 @@ async function runCli() {
       ignore: opts.ignore ? opts.ignore.split(",").map((c) => c.trim()) : config?.ignore || [],
       tokensOnly: opts.tokens,
       quiet: opts.quiet,
-      depth: parseInt(opts.depth, 10) || 2
+      depth: parseInt(opts.depth, 10) || 2,
+      mcp: effectiveMcp,
+      mcpOnly,
+      mcpGlobal: mcpGlobal || config?.mcpGlobal || false
     };
     if (config?.tokenThresholds) {
       setTokenThresholds(config.tokenThresholds);
@@ -260,7 +351,10 @@ async function runCli() {
       if (spinner) spinner.text = "Running checks...";
       const result = await runAudit(resolvedPath, activeChecks, {
         depth: options.depth,
-        extraPatterns: config?.contextFiles
+        extraPatterns: config?.contextFiles,
+        mcp: options.mcp,
+        mcpGlobal: options.mcpGlobal,
+        mcpOnly: options.mcpOnly
       });
       spinner?.stop();
       if (result.files.length === 0) {

package/dist/index.js

@@ -2,8 +2,8 @@
 
 // src/index.ts
 if (process.argv.includes("--mcp")) {
-  await import("./server-7C2IQ7VV.js");
+  await import("./server-IFZ3VEK3.js");
 } else {
-  const { runCli } = await import("./cli-VYWAONGX.js");
+  const { runCli } = await import("./cli-7IBRPDA6.js");
   await runCli();
 }