@eslint-config-snapshot/cli 0.1.0 → 0.1.4

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

@@ -1,29 +1,29 @@
-import { spawnSync } from 'node:child_process'
+import { spawnSync } from 'node:child_process'
 import { cp, mkdir, mkdtemp, readFile, rm, writeFile } from 'node:fs/promises'
-import os from 'node:os'
-import path from 'node:path'
-import { afterEach, beforeEach, describe, expect, it } from 'vitest'
-
-const fixtureRoot = path.resolve('test/fixtures/repo')
-const cliDist = path.resolve('dist/index.js')
-
-let tmpDir = ''
-let repoRoot = ''
-
-function run(args: string[]): { status: number; stdout: string; stderr: string } {
-  const proc = spawnSync(process.execPath, [cliDist, ...args], {
-    cwd: repoRoot,
-    encoding: 'utf8',
-    env: { ...process.env }
-  })
-
-  return {
-    status: proc.status ?? 1,
-    stdout: proc.stdout ?? '',
-    stderr: proc.stderr ?? ''
-  }
-}
-
+import os from 'node:os'
+import path from 'node:path'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
+
+const fixtureRoot = path.resolve('test/fixtures/repo')
+const cliDist = path.resolve('dist/index.js')
+
+let tmpDir = ''
+let repoRoot = ''
+
+function run(args: string[]): { status: number; stdout: string; stderr: string } {
+  const proc = spawnSync(process.execPath, [cliDist, ...args], {
+    cwd: repoRoot,
+    encoding: 'utf8',
+    env: { ...process.env }
+  })
+
+  return {
+    status: proc.status ?? 1,
+    stdout: proc.stdout ?? '',
+    stderr: proc.stderr ?? ''
+  }
+}
+
 beforeEach(async () => {
   tmpDir = await mkdtemp(path.join(os.tmpdir(), 'snapshot-cli-terminal-'))
   repoRoot = path.join(tmpDir, 'repo')
@@ -49,53 +49,53 @@ beforeEach(async () => {
     JSON.stringify({ name: 'eslint', version: '9.0.0' }, null, 2)
   )
 })
-
-afterEach(async () => {
-  if (tmpDir) {
-    await rm(tmpDir, { recursive: true, force: true })
-    tmpDir = ''
-    repoRoot = ''
-  }
-})
-
-describe('cli terminal invocation', () => {
+
+afterEach(async () => {
+  if (tmpDir) {
+    await rm(tmpDir, { recursive: true, force: true })
+    tmpDir = ''
+    repoRoot = ''
+  }
+})
+
+describe('cli terminal invocation', () => {
   it('prints help text and exits 0', () => {
-    const result = run(['--help'])
-    expect(result.status).toBe(0)
-    expect(result.stdout).toContain('Usage: eslint-config-snapshot [options] [command]')
-    expect(result.stdout).toContain('check [options]')
-    expect(result.stdout).toContain('update|snapshot')
-    expect(result.stdout).toContain('print [options]')
-    expect(result.stdout).toContain('init')
-    expect(result.stderr).toBe('')
-  })
-
-  it('returns 1 for unknown command', () => {
-    const result = run(['unknown'])
-    expect(result.status).toBe(1)
-    expect(result.stdout).toBe('')
-    expect(result.stderr).toContain("error: unknown command 'unknown'")
-  })
-
+    const result = run(['--help'])
+    expect(result.status).toBe(0)
+    expect(result.stdout).toContain('Usage: eslint-config-snapshot [options] [command]')
+    expect(result.stdout).toContain('check [options]')
+    expect(result.stdout).toContain('update|snapshot')
+    expect(result.stdout).toContain('print [options]')
+    expect(result.stdout).toContain('init')
+    expect(result.stderr).toBe('')
+  })
+
+  it('returns 1 for unknown command', () => {
+    const result = run(['unknown'])
+    expect(result.status).toBe(1)
+    expect(result.stdout).toBe('')
+    expect(result.stderr).toContain("error: unknown command 'unknown'")
+  })
+
   it('snapshot succeeds and compare returns clean result', () => {
     const snapshot = run(['snapshot'])
     expect(snapshot.status).toBe(0)
     expect(snapshot.stdout).toContain('Baseline updated:')
-    expect(snapshot.stderr).toBe('')
-
+    expect(snapshot.stderr).toBe('')
+
     const compare = run(['compare'])
     expect(compare.status).toBe(0)
     expect(compare.stdout).toBe('Great news: no snapshot changes detected.\n')
     expect(compare.stderr).toBe('')
   })
-
-  it('default command prints clean summary when no drift', () => {
+
+  it('default command prints clean summary when no drift', () => {
     expect(run(['snapshot']).status).toBe(0)
     const result = run([])
     expect(result.status).toBe(0)
     expect(result.stdout).toContain('Great news: no snapshot drift detected.')
-  })
-
+  })
+
   it('default command reports missing local snapshots', () => {
     const result = run([])
     expect(result.status).toBe(1)
@@ -103,15 +103,15 @@ describe('cli terminal invocation', () => {
       'Current rule state: 1 groups, 3 rules (severity mix: 2 errors, 0 warnings, 1 off).\nYou are almost set: no baseline snapshot found yet.\nRun `eslint-config-snapshot --update` to create your first baseline.\n'
     )
   })
-
-  it('compare returns 1 and deterministic diff output when rules change', async () => {
-    expect(run(['snapshot']).status).toBe(0)
-
-    await writeFile(
-      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
-      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: 0 } }))\n"
-    )
-
+
+  it('compare returns 1 and deterministic diff output when rules change', async () => {
+    expect(run(['snapshot']).status).toBe(0)
+
+    await writeFile(
+      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
+      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: 0 } }))\n"
+    )
+
     const compare = run(['compare'])
     expect(compare.status).toBe(1)
     expect(compare.stdout).toBe(
@@ -119,15 +119,15 @@ describe('cli terminal invocation', () => {
     )
     expect(compare.stderr).toBe('')
   })
-
-  it('compare shows options changed when severity does not change', async () => {
-    expect(run(['snapshot']).status).toBe(0)
-
-    await writeFile(
-      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
-      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: [2, 'smart'] } }))\n"
-    )
-
+
+  it('compare shows options changed when severity does not change', async () => {
+    expect(run(['snapshot']).status).toBe(0)
+
+    await writeFile(
+      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
+      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: [2, 'smart'] } }))\n"
+    )
+
     const compare = run(['compare'])
     expect(compare.status).toBe(1)
     expect(compare.stdout).toBe(
@@ -135,34 +135,34 @@ describe('cli terminal invocation', () => {
     )
     expect(compare.stderr).toBe('')
   })
-
-  it('compare prints removed rules as nested list', async () => {
-    expect(run(['snapshot']).status).toBe(0)
-
-    await writeFile(
-      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
-      "console.log(JSON.stringify({ rules: { 'no-console': 1 } }))\n"
-    )
-
-    const compare = run(['compare'])
-    expect(compare.status).toBe(1)
-    expect(compare.stdout).toContain('removed rules:\n  - eqeqeq\n')
-    expect(compare.stdout).not.toContain('options changed:\n  - eqeqeq')
-  })
-
-  it('status returns clean and changes variants', async () => {
-    expect(run(['snapshot']).status).toBe(0)
-
+
+  it('compare prints removed rules as nested list', async () => {
+    expect(run(['snapshot']).status).toBe(0)
+
+    await writeFile(
+      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
+      "console.log(JSON.stringify({ rules: { 'no-console': 1 } }))\n"
+    )
+
+    const compare = run(['compare'])
+    expect(compare.status).toBe(1)
+    expect(compare.stdout).toContain('removed rules:\n  - eqeqeq\n')
+    expect(compare.stdout).not.toContain('options changed:\n  - eqeqeq')
+  })
+
+  it('status returns clean and changes variants', async () => {
+    expect(run(['snapshot']).status).toBe(0)
+
     const clean = run(['status'])
     expect(clean.status).toBe(0)
     expect(clean.stdout).toBe('clean\n')
-    expect(clean.stderr).toBe('')
-
-    await writeFile(
-      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
-      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: 0 } }))\n"
-    )
-
+    expect(clean.stderr).toBe('')
+
+    await writeFile(
+      path.join(repoRoot, 'packages/ws-a/node_modules/eslint/bin/eslint.js'),
+      "console.log(JSON.stringify({ rules: { 'no-console': 1, eqeqeq: 0 } }))\n"
+    )
+
     const changed = run(['status'])
     expect(changed.status).toBe(1)
     expect(changed.stdout).toBe(
@@ -170,56 +170,56 @@ describe('cli terminal invocation', () => {
     )
     expect(changed.stderr).toBe('')
   })
-
-  it('print returns deterministic json output', () => {
-    const result = run(['print'])
-    expect(result.status).toBe(0)
-    expect(result.stdout).toBe(
-      `[
-  {
-    "groupId": "default",
-    "rules": {
-      "eqeqeq": [
-        "error",
-        "always"
-      ],
-      "no-console": [
-        "error"
-      ],
-      "no-debugger": [
-        "off"
-      ]
-    }
-  }
-]
-`
-    )
-    expect(result.stderr).toBe('')
-  })
-
-  it('print --short returns compact human-readable output', () => {
-    const result = run(['print', '--short'])
-    expect(result.status).toBe(0)
-    expect(result.stdout).toBe(
-      `group: default
-workspaces (2): packages/ws-a, packages/ws-b
-rules (3): error 2, warn 0, off 1
-eqeqeq: error "always"
-no-console: error
-no-debugger: off
-`
-    )
-    expect(result.stderr).toBe('')
-  })
-
+
+  it('print returns deterministic json output', () => {
+    const result = run(['print'])
+    expect(result.status).toBe(0)
+    expect(result.stdout).toBe(
+      `[
+  {
+    "groupId": "default",
+    "rules": {
+      "eqeqeq": [
+        "error",
+        "always"
+      ],
+      "no-console": [
+        "error"
+      ],
+      "no-debugger": [
+        "off"
+      ]
+    }
+  }
+]
+`
+    )
+    expect(result.stderr).toBe('')
+  })
+
+  it('print --short returns compact human-readable output', () => {
+    const result = run(['print', '--short'])
+    expect(result.status).toBe(0)
+    expect(result.stdout).toBe(
+      `group: default
+workspaces (2): packages/ws-a, packages/ws-b
+rules (3): error 2, warn 0, off 1
+eqeqeq: error "always"
+no-console: error
+no-debugger: off
+`
+    )
+    expect(result.stderr).toBe('')
+  })
+
   it('init handles success and existing-file error paths', async () => {
-    const initRoot = path.join(tmpDir, 'init-case')
-    await rm(initRoot, { recursive: true, force: true })
-    await cp(fixtureRoot, initRoot, { recursive: true })
-    repoRoot = initRoot
-
-    await rm(path.join(repoRoot, 'eslint-config-snapshot.config.mjs'), { force: true })
-
+    const initRoot = path.join(tmpDir, 'init-case')
+    await rm(initRoot, { recursive: true, force: true })
+    await cp(fixtureRoot, initRoot, { recursive: true })
+    repoRoot = initRoot
+
+    await rm(path.join(repoRoot, 'eslint-config-snapshot.config.mjs'), { force: true })
+
     const created = run(['init', '--yes', '--target', 'file', '--preset', 'minimal'])
     expect(created.status).toBe(0)
     expect(created.stdout).toBe('Created eslint-config-snapshot.config.mjs\n')
@@ -282,52 +282,52 @@ no-debugger: off
     expect(forced.stdout).toBe('Created eslint-config-snapshot.config.mjs\n')
     expect(forced.stderr).toBe('')
   })
-
-  it('surfaces runtime errors with exit code 1', async () => {
-    await writeFile(
-      path.join(repoRoot, 'eslint-config-snapshot.config.mjs'),
-      `export default {
-  workspaceInput: { mode: 'manual', workspaces: ['packages/ws-a'] },
-  grouping: { mode: 'match', allowEmptyGroups: false, groups: [{ name: 'never', match: ['ops/**'] }] },
-  sampling: { maxFilesPerWorkspace: 8, includeGlobs: ['**/*.ts'], excludeGlobs: ['**/node_modules/**'], hintGlobs: [] }
-}
-`
-    )
-
-    const result = run(['snapshot'])
-    expect(result.status).toBe(1)
-    expect(result.stdout).toBe('')
-    expect(result.stderr).toBe('Unmatched workspaces: packages/ws-a\n')
-  })
-
-  it('loads config from package.json through cosmiconfig', async () => {
-    await rm(path.join(repoRoot, 'eslint-config-snapshot.config.mjs'), { force: true })
-    await writeFile(
-      path.join(repoRoot, 'package.json'),
-      JSON.stringify(
-        {
-          name: 'fixture-repo',
-          private: true,
-          workspaces: ['packages/*'],
-          'eslint-config-snapshot': {
-            workspaceInput: { mode: 'manual', workspaces: ['packages/ws-a'] },
-            grouping: { mode: 'match', groups: [{ name: 'default', match: ['**/*'] }] },
-            sampling: { maxFilesPerWorkspace: 8, includeGlobs: ['**/*.ts'], excludeGlobs: ['**/node_modules/**'], hintGlobs: [] }
-          }
-        },
-        null,
-        2
-      )
-    )
-
-    const snapshot = run(['snapshot'])
-    expect(snapshot.status).toBe(0)
-
-    const raw = await readFile(path.join(repoRoot, '.eslint-config-snapshot/default.json'), 'utf8')
-    const parsed = JSON.parse(raw) as { workspaces: string[] }
-    expect(parsed.workspaces).toEqual(['packages/ws-a'])
-  })
-
+
+  it('surfaces runtime errors with exit code 1', async () => {
+    await writeFile(
+      path.join(repoRoot, 'eslint-config-snapshot.config.mjs'),
+      `export default {
+  workspaceInput: { mode: 'manual', workspaces: ['packages/ws-a'] },
+  grouping: { mode: 'match', allowEmptyGroups: false, groups: [{ name: 'never', match: ['ops/**'] }] },
+  sampling: { maxFilesPerWorkspace: 8, includeGlobs: ['**/*.ts'], excludeGlobs: ['**/node_modules/**'], hintGlobs: [] }
+}
+`
+    )
+
+    const result = run(['snapshot'])
+    expect(result.status).toBe(1)
+    expect(result.stdout).toBe('')
+    expect(result.stderr).toBe('Unmatched workspaces: packages/ws-a\n')
+  })
+
+  it('loads config from package.json through cosmiconfig', async () => {
+    await rm(path.join(repoRoot, 'eslint-config-snapshot.config.mjs'), { force: true })
+    await writeFile(
+      path.join(repoRoot, 'package.json'),
+      JSON.stringify(
+        {
+          name: 'fixture-repo',
+          private: true,
+          workspaces: ['packages/*'],
+          'eslint-config-snapshot': {
+            workspaceInput: { mode: 'manual', workspaces: ['packages/ws-a'] },
+            grouping: { mode: 'match', groups: [{ name: 'default', match: ['**/*'] }] },
+            sampling: { maxFilesPerWorkspace: 8, includeGlobs: ['**/*.ts'], excludeGlobs: ['**/node_modules/**'], hintGlobs: [] }
+          }
+        },
+        null,
+        2
+      )
+    )
+
+    const snapshot = run(['snapshot'])
+    expect(snapshot.status).toBe(0)
+
+    const raw = await readFile(path.join(repoRoot, '.eslint-config-snapshot/default.json'), 'utf8')
+    const parsed = JSON.parse(raw) as { workspaces: string[] }
+    expect(parsed.workspaces).toEqual(['packages/ws-a'])
+  })
+
   it('updates snapshots with --update without command', () => {
     const result = run(['--update'])
     expect(result.status).toBe(0)
@@ -346,7 +346,7 @@ no-debugger: off
     expect(result.stdout).toContain('--yes --force --target file --preset full')
     expect(result.stderr).toBe('')
   })
-
+
   it('supports canonical check and update commands', () => {
     const update = run(['update'])
     expect(update.status).toBe(0)

package/test/fixtures/npm-isolated-template/package.json

@@ -1,7 +1,7 @@
-{
-  "name": "npm-isolated-fixture",
-  "private": true,
-  "workspaces": [
-    "packages/*"
-  ]
-}
+{
+  "name": "npm-isolated-fixture",
+  "private": true,
+  "workspaces": [
+    "packages/*"
+  ]
+}

package/test/fixtures/npm-isolated-template/packages/ws-a/.eslintrc.cjs

@@ -4,4 +4,4 @@ module.exports = {
     'no-console': 'warn',
     eqeqeq: ['error', 'always']
   }
-}
+}

package/test/fixtures/npm-isolated-template/packages/ws-a/src/index.ts

@@ -1 +1 @@
-console.log('a')
+console.log('a')

package/test/fixtures/npm-isolated-template/packages/ws-b/.eslintrc.cjs

@@ -4,4 +4,4 @@ module.exports = {
     'no-console': 'error',
     'no-debugger': 'off'
   }
-}
+}

package/test/fixtures/npm-isolated-template/packages/ws-b/src/index.ts

@@ -1 +1 @@
-console.log('b')
+console.log('b')

package/test/fixtures/repo/eslint-config-snapshot.config.mjs

@@ -13,4 +13,4 @@ export default {
     excludeGlobs: ['**/node_modules/**'],
     hintGlobs: []
   }
-}
+}

package/test/fixtures/repo/package.json

@@ -4,4 +4,4 @@
   "workspaces": [
     "packages/*"
   ]
-}
+}

package/test/fixtures/repo/packages/ws-a/package.json

@@ -1,4 +1,4 @@
 {
   "name": "ws-a",
   "private": true
-}
+}

package/test/fixtures/repo/packages/ws-a/src/index.ts

@@ -1 +1 @@
-export const a = 1
+export const a = 1

package/test/fixtures/repo/packages/ws-b/package.json

@@ -1,4 +1,4 @@
 {
   "name": "ws-b",
   "private": true
-}
+}

package/test/fixtures/repo/packages/ws-b/src/index.ts

@@ -1 +1 @@
-export const b = 1
+export const b = 1

package/tsconfig.json

@@ -1,12 +1,12 @@
-{
-    "extends":  "../../tsconfig.base.json",
-    "compilerOptions":  {
-
-                        },
-    "include":  [
-                    "src/**/*.ts"
-                ],
-    "references":  [
-
-                   ]
-}
+{
+    "extends":  "../../tsconfig.base.json",
+    "compilerOptions":  {
+
+                        },
+    "include":  [
+                    "src/**/*.ts"
+                ],
+    "references":  [
+
+                   ]
+}