@sqldoc/cli 0.0.1

package/package.json

@@ -0,0 +1,37 @@
+{
+  "type": "module",
+  "name": "@sqldoc/cli",
+  "version": "0.0.1",
+  "description": "CLI tool for sqldoc -- compile and validate SQL files with tag-driven generation",
+  "exports": {
+    ".": {
+      "types": "./src/index.ts",
+      "import": "./src/index.ts",
+      "default": "./src/index.ts"
+    }
+  },
+  "main": "./src/index.ts",
+  "types": "./src/index.ts",
+  "bin": {
+    "sqldoc": "./src/index.ts"
+  },
+  "files": [
+    "src",
+    "package.json"
+  ],
+  "dependencies": {
+    "commander": "^14.0.3",
+    "fast-glob": "^3.3.3",
+    "picocolors": "^1.1.1",
+    "@sqldoc/atlas": "0.0.1",
+    "@sqldoc/core": "0.0.1"
+  },
+  "devDependencies": {
+    "typescript": "^5.9.3",
+    "vitest": "^4.1.0"
+  },
+  "scripts": {
+    "test": "vitest run",
+    "typecheck": "tsc --noEmit"
+  }
+}

package/src/__tests__/binary-entry.test.ts

@@ -0,0 +1,19 @@
+import { describe, it, expect } from 'vitest'
+import { isBunRuntime, isCompiledBinary } from '../runtime'
+
+describe('runtime detection', () => {
+  it('isBunRuntime returns false in Node.js', () => {
+    expect(isBunRuntime()).toBe(false)
+  })
+
+  it('isCompiledBinary returns false in Node.js', () => {
+    expect(isCompiledBinary()).toBe(false)
+  })
+
+  it('isCompiledBinary depends on isBunRuntime', () => {
+    // If not running in Bun, compiled binary detection must also be false
+    if (!isBunRuntime()) {
+      expect(isCompiledBinary()).toBe(false)
+    }
+  })
+})

package/src/__tests__/codegen.test.ts

@@ -0,0 +1,103 @@
+import * as fs from 'node:fs'
+import * as os from 'node:os'
+import * as path from 'node:path'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+
+describe('codegenCommand', () => {
+  let tmpDir: string
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'sqldoc-cli-codegen-'))
+  })
+
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true })
+  })
+
+  it('does not write SQL to stdout', async () => {
+    const nsFile = path.join(tmpDir, 'ns-test.ts')
+    fs.writeFileSync(
+      nsFile,
+      `
+      export default {
+        name: 'test',
+        tags: {
+          marker: { description: 'test marker', targets: ['table'] },
+        },
+        apiVersion: 1,
+        generateSQL(ctx) {
+          return [{ sql: '-- generated by test namespace' }]
+        },
+      }
+    `,
+      'utf-8',
+    )
+
+    const sqlFile = path.join(tmpDir, 'schema.sql')
+    fs.writeFileSync(
+      sqlFile,
+      `-- @import './ns-test.ts'
+-- @test.marker
+CREATE TABLE users (
+  id bigserial PRIMARY KEY
+);
+`,
+      'utf-8',
+    )
+
+    const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true)
+    const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+
+    const { codegenCommand } = await import('../commands/codegen.ts')
+    await codegenCommand(sqlFile, {})
+
+    // codegen should NOT write SQL to stdout (no -o option exists)
+    const stdoutOutput = writeSpy.mock.calls.map((c) => String(c[0])).join('')
+    expect(stdoutOutput).not.toContain('CREATE TABLE users')
+
+    consoleErrorSpy.mockRestore()
+    writeSpy.mockRestore()
+  }, 30_000)
+
+  it('throws CliError when errors are encountered', async () => {
+    const nsFile = path.join(tmpDir, 'ns-error.ts')
+    fs.writeFileSync(
+      nsFile,
+      `
+      export default {
+        name: 'broken',
+        tags: {
+          fail: { description: 'always fails', targets: ['table'] },
+        },
+        apiVersion: 1,
+        generateSQL(ctx) {
+          throw new Error('Generator exploded')
+        },
+      }
+    `,
+      'utf-8',
+    )
+
+    const sqlFile = path.join(tmpDir, 'broken.sql')
+    fs.writeFileSync(
+      sqlFile,
+      `-- @import './ns-error.ts'
+-- @broken.fail
+CREATE TABLE bad (
+  id bigserial PRIMARY KEY
+);
+`,
+      'utf-8',
+    )
+
+    const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+    const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true)
+
+    const { codegenCommand } = await import('../commands/codegen.ts')
+    const { CliError } = await import('../errors.ts')
+    await expect(codegenCommand(sqlFile, {})).rejects.toThrow(CliError)
+
+    consoleErrorSpy.mockRestore()
+    writeSpy.mockRestore()
+  })
+})

package/src/__tests__/destructive.test.ts

@@ -0,0 +1,132 @@
+import { describe, expect, it } from 'vitest'
+import { detectDestructiveChanges } from '../utils/destructive.ts'
+
+describe('detectDestructiveChanges', () => {
+  it('returns empty array for non-destructive statements', () => {
+    const statements = [
+      'CREATE TABLE users (id INT)',
+      'ALTER TABLE users ADD COLUMN email TEXT',
+      'CREATE INDEX idx_users_email ON users (email)',
+    ]
+    const result = detectDestructiveChanges(statements)
+    expect(result).toEqual([])
+  })
+
+  it('detects DROP TABLE', () => {
+    const statements = ['DROP TABLE audit_log']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP TABLE audit_log')
+  })
+
+  it('detects DROP TABLE IF EXISTS', () => {
+    const statements = ['DROP TABLE IF EXISTS audit_log']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP TABLE audit_log')
+  })
+
+  it('detects DROP TABLE with quoted name', () => {
+    const statements = ['DROP TABLE "audit_log"']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP TABLE audit_log')
+  })
+
+  it('detects ALTER TABLE DROP COLUMN', () => {
+    const statements = ['ALTER TABLE users DROP COLUMN legacy_id']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('ALTER TABLE users DROP COLUMN legacy_id')
+  })
+
+  it('detects ALTER TABLE DROP COLUMN IF EXISTS', () => {
+    const statements = ['ALTER TABLE users DROP COLUMN IF EXISTS legacy_id']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('ALTER TABLE users DROP COLUMN legacy_id')
+  })
+
+  it('detects ALTER TABLE ONLY ... DROP COLUMN', () => {
+    const statements = ['ALTER TABLE ONLY users DROP COLUMN legacy_id']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('ALTER TABLE users DROP COLUMN legacy_id')
+  })
+
+  it('detects DROP INDEX', () => {
+    const statements = ['DROP INDEX idx_users_email']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP INDEX idx_users_email')
+  })
+
+  it('detects DROP INDEX CONCURRENTLY IF EXISTS', () => {
+    const statements = ['DROP INDEX CONCURRENTLY IF EXISTS idx_users_email']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP INDEX idx_users_email')
+  })
+
+  it('detects DROP VIEW', () => {
+    const statements = ['DROP VIEW active_users']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP VIEW active_users')
+  })
+
+  it('detects DROP FUNCTION', () => {
+    const statements = ['DROP FUNCTION calculate_total']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('DROP FUNCTION calculate_total')
+  })
+
+  it('detects TRUNCATE', () => {
+    const statements = ['TRUNCATE TABLE sessions']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('TRUNCATE sessions')
+  })
+
+  it('detects TRUNCATE without TABLE keyword', () => {
+    const statements = ['TRUNCATE sessions']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('TRUNCATE sessions')
+  })
+
+  it('detects multiple destructive changes', () => {
+    const statements = [
+      'CREATE TABLE new_users (id INT)',
+      'DROP TABLE audit_log',
+      'ALTER TABLE users DROP COLUMN legacy_id',
+      'CREATE INDEX idx ON new_users (id)',
+      'DROP INDEX idx_old',
+    ]
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(3)
+    expect(result[0].description).toBe('DROP TABLE audit_log')
+    expect(result[1].description).toBe('ALTER TABLE users DROP COLUMN legacy_id')
+    expect(result[2].description).toBe('DROP INDEX idx_old')
+  })
+
+  it('is case-insensitive', () => {
+    const statements = ['drop table USERS', 'alter table Orders drop column old_col']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(2)
+  })
+
+  it('handles multiline statements (whitespace normalization)', () => {
+    const statements = ['ALTER TABLE\n  users\n  DROP COLUMN\n  legacy_id']
+    const result = detectDestructiveChanges(statements)
+    expect(result).toHaveLength(1)
+    expect(result[0].description).toBe('ALTER TABLE users DROP COLUMN legacy_id')
+  })
+
+  it('preserves the original statement in the result', () => {
+    const stmt = 'DROP TABLE "my_table"'
+    const result = detectDestructiveChanges([stmt])
+    expect(result[0].statement).toBe(stmt)
+  })
+})