@eslint-config-snapshot/cli 0.3.0 → 0.4.0

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # @eslint-config-snapshot/cli
 
+## 0.4.0
+
+### Minor Changes
+
+- Minor release with improved CLI output consistency and faster workspace extraction using ESLint API fallback strategy.
+
+### Patch Changes
+
+- Updated dependencies
+  - @eslint-config-snapshot/api@0.4.0
+
+## 0.3.2
+
+### Patch Changes
+
+- Fix deterministic aggregation when same-severity ESLint rule options differ across sampled files, preventing update crashes.
+- Updated dependencies
+  - @eslint-config-snapshot/api@0.3.2
+
+## 0.3.1
+
+### Patch Changes
+
+- Release patch bump after init UX clarity improvements for default catch-all group messaging.
+- Updated dependencies
+  - @eslint-config-snapshot/api@0.3.1
+
 ## 0.3.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -124,7 +124,7 @@ function createProgram(cwd, onActionExit) {
 Examples:
   $ eslint-config-snapshot init
     Runs interactive select prompts for target/preset.
-    Recommended preset uses checkbox selection for non-default workspaces and group selection.
+    Recommended preset keeps a dynamic catch-all default group ("*") and asks only for static exception groups.
 
   $ eslint-config-snapshot init --yes --target package-json --preset recommended --show-effective
     Non-interactive recommended setup in package.json, with effective preview.
@@ -266,8 +266,13 @@ async function executeUpdate(cwd, printSummary) {
   await writeSnapshots(cwd, currentSnapshots);
   if (printSummary) {
     const summary = summarizeSnapshots(currentSnapshots);
-    process.stdout.write(`Baseline updated: ${summary.groups} groups, ${summary.rules} rules.
-`);
+    const color = createColorizer();
+    writeSectionTitle("Summary", color);
+    process.stdout.write(
+      `Baseline updated: ${summary.groups} groups, ${summary.rules} rules.
+Severity mix: ${summary.error} errors, ${summary.warn} warnings, ${summary.off} off.
+`
+    );
   }
   return 0;
 }
@@ -317,19 +322,20 @@ async function computeCurrentSnapshots(cwd) {
       const sampled = await (0, import_api.sampleWorkspaceFiles)(workspaceAbs, config.sampling);
       let extractedCount = 0;
       let lastExtractionError;
-      for (const sampledRel of sampled) {
-        const sampledAbs = import_node_path.default.resolve(workspaceAbs, sampledRel);
-        try {
-          extractedForGroup.push((0, import_api.extractRulesFromPrintConfig)(workspaceAbs, sampledAbs));
+      const sampledAbs = sampled.map((sampledRel) => import_node_path.default.resolve(workspaceAbs, sampledRel));
+      const results = await (0, import_api.extractRulesForWorkspaceSamples)(workspaceAbs, sampledAbs);
+      for (const result of results) {
+        if (result.rules) {
+          extractedForGroup.push(result.rules);
           extractedCount += 1;
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          if (message.startsWith("Invalid JSON from eslint --print-config") || message.startsWith("Empty ESLint print-config output")) {
-            lastExtractionError = message;
-            continue;
-          }
-          throw error;
+          continue;
         }
+        const message = result.error instanceof Error ? result.error.message : String(result.error);
+        if (isRecoverableExtractionError(message)) {
+          lastExtractionError = message;
+          continue;
+        }
+        throw result.error ?? new Error(message);
       }
       if (extractedCount === 0) {
         const context = lastExtractionError ? ` Last error: ${lastExtractionError}` : "";
@@ -343,6 +349,9 @@ async function computeCurrentSnapshots(cwd) {
   }
   return snapshots;
 }
+function isRecoverableExtractionError(message) {
+  return message.startsWith("Invalid JSON from eslint --print-config") || message.startsWith("Empty ESLint print-config output") || message.includes("File ignored because of a matching ignore pattern") || message.includes("File ignored by default");
+}
 async function resolveWorkspaceAssignments(cwd, config) {
   const discovery = await (0, import_api.discoverWorkspaces)({ cwd, workspaceInput: config.workspaceInput });
   const assignments = config.grouping.mode === "standalone" ? discovery.workspacesRel.map((workspace) => ({ name: workspace, workspaces: [workspace] })) : (0, import_api.assignGroupsByMatch)(discovery.workspacesRel, config.grouping.groups ?? [{ name: "default", match: ["**/*"] }]);
@@ -484,7 +493,7 @@ async function askInitPreset(selectPrompt) {
   return selectPrompt({
     message: "Select preset",
     choices: [
-      { name: 'recommended (default group "*" + static overrides)', value: "recommended" },
+      { name: 'recommended (dynamic catch-all "*" + optional static exceptions)', value: "recommended" },
       { name: "minimal", value: "minimal" },
       { name: "full", value: "full" }
     ]
@@ -634,9 +643,12 @@ function trimTrailingSlashes(value) {
 }
 async function askRecommendedGroupAssignments(workspaces) {
   const { checkbox, select } = await import("@inquirer/prompts");
-  process.stdout.write('Recommended setup: select only workspaces that should leave default group "*".\n');
+  process.stdout.write(
+    'Recommended setup: default group "*" is a dynamic catch-all for every discovered workspace.\n'
+  );
+  process.stdout.write("Select only workspaces that should move to explicit static groups.\n");
   const overrides = await checkbox({
-    message: "Workspaces outside default group:",
+    message: 'Choose exception workspaces (leave empty to keep all in default "*"):',
     choices: workspaces.map((workspace) => ({ name: workspace, value: workspace })),
     pageSize: Math.min(12, Math.max(4, workspaces.length))
   });
@@ -702,22 +714,29 @@ function printWhatChanged(changes, currentSnapshots) {
   const changeSummary = summarizeChanges(changes);
   if (changes.length === 0) {
     process.stdout.write(color.green("Great news: no snapshot drift detected.\n"));
+    writeSectionTitle("Summary", color);
     process.stdout.write(
-      `Baseline status: ${currentSummary.groups} groups, ${currentSummary.rules} rules (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off).
+      `- baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules
+- severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off
 `
     );
     return 0;
   }
   process.stdout.write(color.red("Heads up: snapshot drift detected.\n"));
+  writeSectionTitle("Summary", color);
   process.stdout.write(
-    `Changed groups: ${changes.length} | introduced: ${changeSummary.introduced} | removed: ${changeSummary.removed} | severity: ${changeSummary.severity} | options: ${changeSummary.options} | workspace membership: ${changeSummary.workspace}
-`
-  );
-  process.stdout.write(
-    `Current rules: ${currentSummary.rules} (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off)
+    `- changed groups: ${changes.length}
+- introduced rules: ${changeSummary.introduced}
+- removed rules: ${changeSummary.removed}
+- severity changes: ${changeSummary.severity}
+- options changes: ${changeSummary.options}
+- workspace membership changes: ${changeSummary.workspace}
+- current baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules
+- current severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off
 
 `
   );
+  writeSectionTitle("Changes", color);
   for (const change of changes) {
     process.stdout.write(color.bold(`group ${change.groupId}
 `));
@@ -732,6 +751,10 @@ function printWhatChanged(changes, currentSnapshots) {
   writeSubtleInfo(UPDATE_HINT);
   return 1;
 }
+function writeSectionTitle(title, color) {
+  process.stdout.write(`${color.bold(title)}
+`);
+}
 function summarizeChanges(changes) {
   let introduced = 0;
   let removed = 0;

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   buildSnapshot,
   diffSnapshots,
   discoverWorkspaces,
-  extractRulesFromPrintConfig,
+  extractRulesForWorkspaceSamples,
   findConfigPath,
   getConfigScaffold,
   hasDiff,
@@ -104,7 +104,7 @@ function createProgram(cwd, onActionExit) {
 Examples:
   $ eslint-config-snapshot init
     Runs interactive select prompts for target/preset.
-    Recommended preset uses checkbox selection for non-default workspaces and group selection.
+    Recommended preset keeps a dynamic catch-all default group ("*") and asks only for static exception groups.
 
   $ eslint-config-snapshot init --yes --target package-json --preset recommended --show-effective
     Non-interactive recommended setup in package.json, with effective preview.
@@ -246,8 +246,13 @@ async function executeUpdate(cwd, printSummary) {
   await writeSnapshots(cwd, currentSnapshots);
   if (printSummary) {
     const summary = summarizeSnapshots(currentSnapshots);
-    process.stdout.write(`Baseline updated: ${summary.groups} groups, ${summary.rules} rules.
-`);
+    const color = createColorizer();
+    writeSectionTitle("Summary", color);
+    process.stdout.write(
+      `Baseline updated: ${summary.groups} groups, ${summary.rules} rules.
+Severity mix: ${summary.error} errors, ${summary.warn} warnings, ${summary.off} off.
+`
+    );
   }
   return 0;
 }
@@ -297,19 +302,20 @@ async function computeCurrentSnapshots(cwd) {
       const sampled = await sampleWorkspaceFiles(workspaceAbs, config.sampling);
       let extractedCount = 0;
       let lastExtractionError;
-      for (const sampledRel of sampled) {
-        const sampledAbs = path.resolve(workspaceAbs, sampledRel);
-        try {
-          extractedForGroup.push(extractRulesFromPrintConfig(workspaceAbs, sampledAbs));
+      const sampledAbs = sampled.map((sampledRel) => path.resolve(workspaceAbs, sampledRel));
+      const results = await extractRulesForWorkspaceSamples(workspaceAbs, sampledAbs);
+      for (const result of results) {
+        if (result.rules) {
+          extractedForGroup.push(result.rules);
           extractedCount += 1;
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          if (message.startsWith("Invalid JSON from eslint --print-config") || message.startsWith("Empty ESLint print-config output")) {
-            lastExtractionError = message;
-            continue;
-          }
-          throw error;
+          continue;
         }
+        const message = result.error instanceof Error ? result.error.message : String(result.error);
+        if (isRecoverableExtractionError(message)) {
+          lastExtractionError = message;
+          continue;
+        }
+        throw result.error ?? new Error(message);
       }
       if (extractedCount === 0) {
         const context = lastExtractionError ? ` Last error: ${lastExtractionError}` : "";
@@ -323,6 +329,9 @@ async function computeCurrentSnapshots(cwd) {
   }
   return snapshots;
 }
+function isRecoverableExtractionError(message) {
+  return message.startsWith("Invalid JSON from eslint --print-config") || message.startsWith("Empty ESLint print-config output") || message.includes("File ignored because of a matching ignore pattern") || message.includes("File ignored by default");
+}
 async function resolveWorkspaceAssignments(cwd, config) {
   const discovery = await discoverWorkspaces({ cwd, workspaceInput: config.workspaceInput });
   const assignments = config.grouping.mode === "standalone" ? discovery.workspacesRel.map((workspace) => ({ name: workspace, workspaces: [workspace] })) : assignGroupsByMatch(discovery.workspacesRel, config.grouping.groups ?? [{ name: "default", match: ["**/*"] }]);
@@ -464,7 +473,7 @@ async function askInitPreset(selectPrompt) {
   return selectPrompt({
     message: "Select preset",
     choices: [
-      { name: 'recommended (default group "*" + static overrides)', value: "recommended" },
+      { name: 'recommended (dynamic catch-all "*" + optional static exceptions)', value: "recommended" },
       { name: "minimal", value: "minimal" },
       { name: "full", value: "full" }
     ]
@@ -614,9 +623,12 @@ function trimTrailingSlashes(value) {
 }
 async function askRecommendedGroupAssignments(workspaces) {
   const { checkbox, select } = await import("@inquirer/prompts");
-  process.stdout.write('Recommended setup: select only workspaces that should leave default group "*".\n');
+  process.stdout.write(
+    'Recommended setup: default group "*" is a dynamic catch-all for every discovered workspace.\n'
+  );
+  process.stdout.write("Select only workspaces that should move to explicit static groups.\n");
   const overrides = await checkbox({
-    message: "Workspaces outside default group:",
+    message: 'Choose exception workspaces (leave empty to keep all in default "*"):',
     choices: workspaces.map((workspace) => ({ name: workspace, value: workspace })),
     pageSize: Math.min(12, Math.max(4, workspaces.length))
   });
@@ -682,22 +694,29 @@ function printWhatChanged(changes, currentSnapshots) {
   const changeSummary = summarizeChanges(changes);
   if (changes.length === 0) {
     process.stdout.write(color.green("Great news: no snapshot drift detected.\n"));
+    writeSectionTitle("Summary", color);
     process.stdout.write(
-      `Baseline status: ${currentSummary.groups} groups, ${currentSummary.rules} rules (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off).
+      `- baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules
+- severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off
 `
     );
     return 0;
   }
   process.stdout.write(color.red("Heads up: snapshot drift detected.\n"));
+  writeSectionTitle("Summary", color);
   process.stdout.write(
-    `Changed groups: ${changes.length} | introduced: ${changeSummary.introduced} | removed: ${changeSummary.removed} | severity: ${changeSummary.severity} | options: ${changeSummary.options} | workspace membership: ${changeSummary.workspace}
-`
-  );
-  process.stdout.write(
-    `Current rules: ${currentSummary.rules} (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off)
+    `- changed groups: ${changes.length}
+- introduced rules: ${changeSummary.introduced}
+- removed rules: ${changeSummary.removed}
+- severity changes: ${changeSummary.severity}
+- options changes: ${changeSummary.options}
+- workspace membership changes: ${changeSummary.workspace}
+- current baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules
+- current severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off
 
 `
   );
+  writeSectionTitle("Changes", color);
   for (const change of changes) {
     process.stdout.write(color.bold(`group ${change.groupId}
 `));
@@ -712,6 +731,10 @@ function printWhatChanged(changes, currentSnapshots) {
   writeSubtleInfo(UPDATE_HINT);
   return 1;
 }
+function writeSectionTitle(title, color) {
+  process.stdout.write(`${color.bold(title)}
+`);
+}
 function summarizeChanges(changes) {
   let introduced = 0;
   let removed = 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-config-snapshot/cli",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "type": "module",
   "repository": {
     "type": "git",
@@ -30,6 +30,6 @@
     "@inquirer/prompts": "^8.2.0",
     "commander": "^14.0.3",
     "fast-glob": "^3.3.3",
-    "@eslint-config-snapshot/api": "0.3.0"
+    "@eslint-config-snapshot/api": "0.4.0"
   }
 }

package/src/index.ts

@@ -5,7 +5,7 @@ import {
   buildSnapshot,
   diffSnapshots,
   discoverWorkspaces,
-  extractRulesFromPrintConfig,
+  extractRulesForWorkspaceSamples,
   findConfigPath,
   getConfigScaffold,
   hasDiff,
@@ -164,7 +164,7 @@ function createProgram(cwd: string, onActionExit: (code: number) => void): Comma
 Examples:
   $ eslint-config-snapshot init
     Runs interactive select prompts for target/preset.
-    Recommended preset uses checkbox selection for non-default workspaces and group selection.
+    Recommended preset keeps a dynamic catch-all default group ("*") and asks only for static exception groups.
 
   $ eslint-config-snapshot init --yes --target package-json --preset recommended --show-effective
     Non-interactive recommended setup in package.json, with effective preview.
@@ -340,7 +340,11 @@ async function executeUpdate(cwd: string, printSummary: boolean): Promise<number
 
   if (printSummary) {
     const summary = summarizeSnapshots(currentSnapshots)
-    process.stdout.write(`Baseline updated: ${summary.groups} groups, ${summary.rules} rules.\n`)
+    const color = createColorizer()
+    writeSectionTitle('Summary', color)
+    process.stdout.write(
+      `Baseline updated: ${summary.groups} groups, ${summary.rules} rules.\nSeverity mix: ${summary.error} errors, ${summary.warn} warnings, ${summary.off} off.\n`
+    )
   }
 
   return 0
@@ -400,23 +404,23 @@ async function computeCurrentSnapshots(cwd: string): Promise<Map<string, BuiltSn
       let extractedCount = 0
       let lastExtractionError: string | undefined
 
-      for (const sampledRel of sampled) {
-        const sampledAbs = path.resolve(workspaceAbs, sampledRel)
-        try {
-          extractedForGroup.push(extractRulesFromPrintConfig(workspaceAbs, sampledAbs))
+      const sampledAbs = sampled.map((sampledRel) => path.resolve(workspaceAbs, sampledRel))
+      const results = await extractRulesForWorkspaceSamples(workspaceAbs, sampledAbs)
+
+      for (const result of results) {
+        if (result.rules) {
+          extractedForGroup.push(result.rules)
           extractedCount += 1
-        } catch (error: unknown) {
-          const message = error instanceof Error ? error.message : String(error)
-          if (
-            message.startsWith('Invalid JSON from eslint --print-config') ||
-            message.startsWith('Empty ESLint print-config output')
-          ) {
-            lastExtractionError = message
-            continue
-          }
-
-          throw error
+          continue
+        }
+
+        const message = result.error instanceof Error ? result.error.message : String(result.error)
+        if (isRecoverableExtractionError(message)) {
+          lastExtractionError = message
+          continue
         }
+
+        throw result.error ?? new Error(message)
       }
 
       if (extractedCount === 0) {
@@ -434,6 +438,15 @@ async function computeCurrentSnapshots(cwd: string): Promise<Map<string, BuiltSn
   return snapshots
 }
 
+function isRecoverableExtractionError(message: string): boolean {
+  return (
+    message.startsWith('Invalid JSON from eslint --print-config') ||
+    message.startsWith('Empty ESLint print-config output') ||
+    message.includes('File ignored because of a matching ignore pattern') ||
+    message.includes('File ignored by default')
+  )
+}
+
 async function resolveWorkspaceAssignments(cwd: string, config: Awaited<ReturnType<typeof loadConfig>>) {
   const discovery = await discoverWorkspaces({ cwd, workspaceInput: config.workspaceInput })
 
@@ -617,7 +630,7 @@ async function askInitPreset(
   return selectPrompt({
     message: 'Select preset',
     choices: [
-      { name: 'recommended (default group "*" + static overrides)', value: 'recommended' },
+      { name: 'recommended (dynamic catch-all "*" + optional static exceptions)', value: 'recommended' },
       { name: 'minimal', value: 'minimal' },
       { name: 'full', value: 'full' }
     ]
@@ -798,9 +811,12 @@ function trimTrailingSlashes(value: string): string {
 
 async function askRecommendedGroupAssignments(workspaces: string[]): Promise<Map<string, number>> {
   const { checkbox, select } = await import('@inquirer/prompts')
-  process.stdout.write('Recommended setup: select only workspaces that should leave default group "*".\n')
+  process.stdout.write(
+    'Recommended setup: default group "*" is a dynamic catch-all for every discovered workspace.\n'
+  )
+  process.stdout.write('Select only workspaces that should move to explicit static groups.\n')
   const overrides = await checkbox<string>({
-    message: 'Workspaces outside default group:',
+    message: 'Choose exception workspaces (leave empty to keep all in default "*"):',
     choices: workspaces.map((workspace) => ({ name: workspace, value: workspace })),
     pageSize: Math.min(12, Math.max(4, workspaces.length))
   })
@@ -877,20 +893,20 @@ function printWhatChanged(changes: Array<{ groupId: string; diff: SnapshotDiff }
 
   if (changes.length === 0) {
     process.stdout.write(color.green('Great news: no snapshot drift detected.\n'))
+    writeSectionTitle('Summary', color)
     process.stdout.write(
-      `Baseline status: ${currentSummary.groups} groups, ${currentSummary.rules} rules (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off).\n`
+      `- baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules\n- severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off\n`
     )
     return 0
   }
 
   process.stdout.write(color.red('Heads up: snapshot drift detected.\n'))
+  writeSectionTitle('Summary', color)
   process.stdout.write(
-    `Changed groups: ${changes.length} | introduced: ${changeSummary.introduced} | removed: ${changeSummary.removed} | severity: ${changeSummary.severity} | options: ${changeSummary.options} | workspace membership: ${changeSummary.workspace}\n`
-  )
-  process.stdout.write(
-    `Current rules: ${currentSummary.rules} (severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off)\n\n`
+    `- changed groups: ${changes.length}\n- introduced rules: ${changeSummary.introduced}\n- removed rules: ${changeSummary.removed}\n- severity changes: ${changeSummary.severity}\n- options changes: ${changeSummary.options}\n- workspace membership changes: ${changeSummary.workspace}\n- current baseline: ${currentSummary.groups} groups, ${currentSummary.rules} rules\n- current severity mix: ${currentSummary.error} errors, ${currentSummary.warn} warnings, ${currentSummary.off} off\n\n`
   )
 
+  writeSectionTitle('Changes', color)
   for (const change of changes) {
     process.stdout.write(color.bold(`group ${change.groupId}\n`))
     const lines = formatDiff(change.groupId, change.diff).split('\n').slice(1)
@@ -905,6 +921,10 @@ function printWhatChanged(changes: Array<{ groupId: string; diff: SnapshotDiff }
   return 1
 }
 
+function writeSectionTitle(title: string, color: ReturnType<typeof createColorizer>): void {
+  process.stdout.write(`${color.bold(title)}\n`)
+}
+
 function summarizeChanges(changes: Array<{ groupId: string; diff: SnapshotDiff }>) {
   let introduced = 0
   let removed = 0

package/test/cli.terminal.integration.test.ts

@@ -389,7 +389,7 @@ no-debugger: off
     expect(result.stdout).toContain('Initialize config (file or package.json)')
     expect(result.stdout).toContain('-f, --force')
     expect(result.stdout).toContain('Runs interactive select prompts for target/preset.')
-    expect(result.stdout).toContain('Recommended preset uses checkbox selection')
+    expect(result.stdout).toContain('Recommended preset keeps a dynamic catch-all default group ("*")')
     expect(result.stdout).toContain('--show-effective')
     expect(result.stdout).toContain('--yes --force --target file --preset full')
     expect(result.stderr).toBe('')