[codex] fix fallow dead code

apps/docs/package.json

@@ -14,7 +14,6 @@
   "dependencies": {
     "@astrojs/starlight": "^0.37.6",
     "astro": "^5.6.1",
-    "sharp": "^0.34.2",
-    "zod": "^3.25.76"
+    "sharp": "^0.34.2"
   }
 }

packages/cli/package.json

@@ -39,11 +39,10 @@
     "clean": "rm -rf dist"
   },
   "dependencies": {
-    "@clickhouse/client": "^1.11.0",
+    "fast-glob": "^3.3.2",
     "@chkit/clickhouse": "workspace:*",
     "@chkit/codegen": "workspace:*",
     "@chkit/core": "workspace:*",
-    "@logtape/logtape": "^2.0.5",
-    "fast-glob": "^3.3.2"
+    "@logtape/logtape": "^2.0.5"
   }
 }

packages/cli/src/bin/command-dispatch.ts

@@ -10,7 +10,7 @@
 import { loadSchemaDefinitions } from './schema-loader.js'
 import { resolveTableScope, tableKeysFromDefinitions } from './table-scope.js'
 
-export function stripGlobalFlags(argv: string[]): {
+function stripGlobalFlags(argv: string[]): {
   jsonMode: boolean
   tableSelector: string | undefined
   rest: string[]
@@ -65,7 +65,7 @@
   }
 }
 
 async function runPluginCommand(input: {
   argv: string[]
   commandName: string
   resolved: RegisteredCommand
@@ -138,7 +138,7 @@
   if (exitCode !== 0) process.exitCode = exitCode
 }
 
 async function runCoreOrBuiltinCommand(input: {
   argv: string[]
   commandName: string
   resolved: RegisteredCommand

packages/cli/src/bin/commands/drift.ts

@@ -1,3 +1,3 @@
 import { join } from 'node:path'
 
 import { isUnknownDatabaseError, type ClickHouseExecutor, type SchemaObjectRef } from '@chkit/clickhouse'
@@ -16,7 +16,7 @@
 import { readSnapshot } from '../migration-store.js'
 import { resolveTableScope, tableKeysFromDefinitions, type TableScope } from '../table-scope.js'
 
-export interface DriftPayload {
+interface DriftPayload {
   scope?: TableScope
   snapshotFile: string
   expectedCount: number
@@ -38,7 +38,7 @@
   run: cmdDrift,
 }
 
 export async function buildDriftPayload(
   config: ChxConfig,
   metaDir: string,
   snapshot: Snapshot,
@@ -139,7 +139,7 @@
   }
 }
 
 async function cmdDrift(runCtx: CommandRunContext): Promise<void> {
   const { flags, config, dirs, ctx } = runCtx
   const f = typedFlags(flags, GLOBAL_FLAGS)
   const tableSelector = f['--table']

packages/cli/src/bin/internal-plugins/skill-hint.ts

@@ -9,7 +9,7 @@ export type AgentKind = 'claude' | 'cursor' | 'copilot' | 'windsurf' | 'roo' | '
 
 interface AgentInfo { name: string; skillsDir: string }
 
-export const AGENT_REGISTRY: Record<Exclude<AgentKind, 'unknown'>, AgentInfo> = {
+const AGENT_REGISTRY: Record<Exclude<AgentKind, 'unknown'>, AgentInfo> = {
   claude:   { name: 'Claude Code', skillsDir: '.claude/skills' },
   cursor:   { name: 'Cursor',      skillsDir: '.agents/skills' },
   copilot:  { name: 'Copilot',     skillsDir: '.agents/skills' },
@@ -33,7 +33,7 @@ const AGENT_MARKERS: { path: string; agent: AgentKind }[] = [
   { path: '.trae', agent: 'trae' },
 ]
 
-export interface AgentRootResult { root: string; agent: AgentKind }
+interface AgentRootResult { root: string; agent: AgentKind }
 
 /**
  * Walk up from `cwd` to find the best directory for installing agent skills.

packages/cli/src/bin/journal-store.ts

@@ -1,11 +1,10 @@
-import { createClickHouseExecutor, isUnknownDatabaseError, type ClickHouseExecutor } from '@chkit/clickhouse'
-import type { ChxConfig } from '@chkit/core'
+import { isUnknownDatabaseError, type ClickHouseExecutor } from '@chkit/clickhouse'
 
 import type { MigrationJournal, MigrationJournalEntry } from './migration-store.js'
 import { CLI_VERSION } from './version.js'
 import { debug } from './debug.js'
 
-export interface JournalStore {
+interface JournalStore {
   readJournal(): Promise<MigrationJournal>
   appendEntry(entry: MigrationJournalEntry): Promise<void>
   readonly databaseMissing: boolean
@@ -51,7 +50,7 @@
   let bootstrapped = false
   let _databaseMissing = false
 
   async function ensureTable(): Promise<void> {
     if (bootstrapped) return
     debug('journal', `probing journal table "${journalTable}"`)
     try {
@@ -152,10 +151,3 @@
     },
   }
 }
-
-export function createJournalStoreFromConfig(
-  clickhouseConfig: NonNullable<ChxConfig['clickhouse']>
-): { journalStore: JournalStore; db: ClickHouseExecutor } {
-  const db = createClickHouseExecutor(clickhouseConfig)
-  return { journalStore: createJournalStore(db), db }
-}

packages/cli/src/bin/json-output.ts

@@ -1,4 +1,4 @@
-export type Command =
+type Command =
   | 'generate'
   | 'migrate'
   | 'status'
@@ -18,7 +18,7 @@ export function printOutput(value: unknown, jsonMode: boolean): void {
   }
 }
 
-export function jsonPayload<T extends object>(command: Command, payload: T): T & {
+function jsonPayload<T extends object>(command: Command, payload: T): T & {
   command: Command
   schemaVersion: number
 } {

packages/cli/src/bin/lib.ts

@@ -1,36 +1,5 @@
 export { CLI_VERSION } from './version.js'
 
-export {
-  DEFAULT_CONFIG_FILE,
-  loadConfig,
-  resolveDirs,
-  writeIfMissing,
-} from './config.js'
-export {
-  createJournalStore,
-  createJournalStoreFromConfig,
-  type JournalStore,
-} from './journal-store.js'
-export {
-  checksumSQL,
-  findChecksumMismatches,
-  listMigrations,
-  parseJSONOrThrow,
-  readSnapshot,
-  summarizePlan,
-  type ChecksumMismatch,
-  type MigrationJournal,
-  type MigrationJournalEntry,
-} from './migration-store.js'
-export { emitJson, jsonPayload, printOutput, type Command } from './json-output.js'
-export {
-  collectDestructiveOperationMarkers,
-  extractMigrationOperationSummaries,
-  extractDestructiveOperationSummaries,
-  extractExecutableStatements,
-  migrationContainsDangerOperation,
-  parseOperationLine,
-  type MigrationOperationSummary,
-  type DestructiveOperationMarker,
-} from './safety-markers.js'
+export { readSnapshot, summarizePlan } from './migration-store.js'
+export { emitJson } from './json-output.js'
 export { loadSchemaDefinitions } from './schema-loader.js'

packages/cli/src/bin/migration-store.ts

@@ -18,13 +18,13 @@ export interface MigrationJournal {
   applied: MigrationJournalEntry[]
 }
 
-export interface ChecksumMismatch {
+interface ChecksumMismatch {
   name: string
   expected: string
   actual: string
 }
 
-export function parseJSONOrThrow<T>(raw: string, filePath: string, kind: string): T {
+function parseJSONOrThrow<T>(raw: string, filePath: string, kind: string): T {
   try {
     return JSON.parse(raw) as T
   } catch {

packages/cli/src/bin/safety-markers.ts

@@ -14,7 +14,7 @@ export interface DestructiveOperationMarker {
   summary: string
 }
 
-export interface MigrationOperationSummary {
+interface MigrationOperationSummary {
   type: string
   key: string
   risk: string
@@ -25,15 +25,15 @@ export function migrationContainsDangerOperation(sql: string): boolean {
   return extractDestructiveOperationSummaries(sql).length > 0
 }
 
-export function extractDestructiveOperationSummaries(sql: string): string[] {
+function extractDestructiveOperationSummaries(sql: string): string[] {
   return sql
     .split('\n')
     .map((line) => line.trim())
     .filter((line) => line.startsWith('-- operation:') && line.includes('risk=danger'))
     .map((line) => line.replace(/^-- operation:\s*/, ''))
 }
 
-export function parseOperationLine(summary: string): MigrationOperationSummary | null {
+function parseOperationLine(summary: string): MigrationOperationSummary | null {
   const match = summary.match(/^([a-z_]+)\s+key=([^\s]+)\s+risk=([a-z_]+)$/)
   if (!match) return null
   return {

packages/cli/src/bin/table-scope.ts

@@ -18,7 +18,7 @@
   matchCount: number
 }
 
-export interface TableScopeFilterResult {
+interface TableScopeFilterResult {
   plan: MigrationPlan
   omittedOperationCount: number
 }
@@ -30,7 +30,7 @@
   newName: string
 }
 
-export function tableKey(database: string, name: string): string {
+function tableKey(database: string, name: string): string {
   return `${database}.${name}`
 }
 
@@ -198,7 +198,7 @@
   }
 }
 
 export function buildScopedSnapshotDefinitions(input: {
   previousDefinitions: SchemaDefinition[]
   nextDefinitions: SchemaDefinition[]
   matchedTables: ReadonlySet<string>

packages/cli/src/drift.ts

@@ -8,7 +8,7 @@
 } from '@chkit/core'
 import { diffByName, diffNamedShapeMaps, diffSettings } from './drift-diff.js'
 
-export type TableDriftReasonCode =
+type TableDriftReasonCode =
   | 'missing_column'
   | 'extra_column'
   | 'changed_column'
@@ -22,10 +22,10 @@
   | 'unique_key_mismatch'
   | 'projection_mismatch'
 
-export type ObjectDriftReasonCode = 'missing_object' | 'extra_object' | 'kind_mismatch'
-export type DriftReasonCode = ObjectDriftReasonCode | TableDriftReasonCode
+type ObjectDriftReasonCode = 'missing_object' | 'extra_object' | 'kind_mismatch'
+type DriftReasonCode = ObjectDriftReasonCode | TableDriftReasonCode
 
-export interface SchemaObjectShape {
+interface SchemaObjectShape {
   kind: 'table' | 'view' | 'materialized_view'
   database: string
   name: string
@@ -38,7 +38,7 @@
   actualKind?: SchemaObjectShape['kind']
 }
 
-export interface ActualTableShape {
+interface ActualTableShape {
   columns: ColumnDefinition[]
   settings: Record<string, string>
   indexes: SkipIndexDefinition[]
@@ -68,7 +68,7 @@
   projectionDiffs: string[]
 }
 
-export interface DriftReasonSummary {
+interface DriftReasonSummary {
   counts: Partial<Record<DriftReasonCode, number>>
   total: number
   object: number
@@ -242,7 +242,7 @@
   return coreNormalizeEngine(normalizeSQLFragment(value)).toLowerCase()
 }
 
 export function compareTableShape(expected: TableDefinition, actual: ActualTableShape): TableDriftDetail | null {
   const columnDiff = diffByName(
     expected.columns,
     actual.columns,

packages/core/src/diff-primitives.ts

@@ -1,10 +1,10 @@
-export interface NamedDiffChange<T> {
+interface NamedDiffChange<T> {
   name: string
   oldItem: T
   newItem: T
 }
 
-export interface NamedDiffResult<T> {
+interface NamedDiffResult<T> {
   added: T[]
   removed: T[]
   changed: NamedDiffChange<T>[]
@@ -48,7 +48,7 @@ export function diffByName<T>(
   }
 }
 
-export interface SettingDiffResult {
+interface SettingDiffResult {
   changes: Array<
     | { kind: 'modify'; key: string; value: string | number | boolean }
     | { kind: 'reset'; key: string }
@@ -79,7 +79,7 @@ export function diffSettings(
   }
 }
 
-export interface ClauseComparison {
+interface ClauseComparison {
   oldValue: string
   newValue: string
 }

packages/core/src/sql.ts

@@ -1,3 +1,3 @@
 import type {
   ColumnDefinition,
   MaterializedViewDefinition,
@@ -50,7 +50,7 @@
   }
 }
 
 function renderTableSQL(def: TableDefinition): string {
   const columns = def.columns.map(renderColumn)
   const indexes = (def.indexes ?? []).map(
     (idx) =>
@@ -101,7 +101,7 @@
  *                [DEPENDS ON <list>] [SETTINGS <kv>] [APPEND]
  * Note: EMPTY belongs after TO in CREATE, so it's NOT included here.
  */
-export function renderRefreshClause(refresh: MaterializedViewRefresh): string {
+function renderRefreshClause(refresh: MaterializedViewRefresh): string {
   const parts: string[] = []
   if (refresh.every) parts.push(`REFRESH EVERY ${refresh.every}`)
   else if (refresh.after) parts.push(`REFRESH AFTER ${refresh.after}`)