@strav/cli 0.4.0 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@strav/cli",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "type": "module",
   "description": "CLI framework and code generators for the Strav framework",
   "license": "MIT",
@@ -33,11 +33,11 @@
     "strav": "./src/cli/strav.ts"
   },
   "peerDependencies": {
-    "@strav/kernel": "0.4.0",
-    "@strav/http": "0.4.0",
-    "@strav/database": "0.4.0",
-    "@strav/queue": "0.4.0",
-    "@strav/signal": "0.4.0"
+    "@strav/kernel": "0.4.2",
+    "@strav/http": "0.4.2",
+    "@strav/database": "0.4.2",
+    "@strav/queue": "0.4.2",
+    "@strav/signal": "0.4.2"
   },
   "dependencies": {
     "chalk": "^5.6.2",

package/src/commands/generate_api.ts

@@ -17,7 +17,8 @@ export function register(program: Command): void {
     .description(
       'Generate services, controllers, policies, validators, events, and routes from schemas'
     )
-    .action(async () => {
+    .option('-f, --force', 'Overwrite existing generated files')
+    .action(async ({ force }: { force?: boolean }) => {
       try {
         console.log(chalk.cyan('Generating API layer from schemas...'))
 
@@ -35,7 +36,7 @@ export function register(program: Command): void {
         const config = await loadGeneratorConfig()
 
         const apiGen = new ApiGenerator(schemas, representation, config)
-        const apiFiles = await apiGen.writeAll()
+        const apiResult = await apiGen.writeAll(force)
 
         // Load API routing config from config/http.ts (if available)
         let apiConfig: Partial<ApiRoutingConfig> | undefined
@@ -47,24 +48,41 @@ export function register(program: Command): void {
         }
 
         const routeGen = new RouteGenerator(schemas, config, apiConfig)
-        const routeFiles = await routeGen.writeAll()
+        const routeResult = await routeGen.writeAll(force)
 
         const testGen = new TestGenerator(schemas, representation, config, apiConfig)
-        const testFiles = await testGen.writeAll()
+        const testResult = await testGen.writeAll(force)
 
         const docGen = new DocGenerator(schemas, representation, config, apiConfig)
-        const docFiles = await docGen.writeAll()
+        const docResult = await docGen.writeAll(force)
 
-        const files = [...apiFiles, ...routeFiles, ...testFiles, ...docFiles]
+        const written = [
+          ...apiResult.written,
+          ...routeResult.written,
+          ...testResult.written,
+          ...docResult.written,
+        ]
+        const skipped = [
+          ...apiResult.skipped,
+          ...routeResult.skipped,
+          ...testResult.skipped,
+          ...docResult.skipped,
+        ]
 
-        if (files.length === 0) {
+        if (written.length === 0 && skipped.length === 0) {
           console.log(chalk.yellow('No API files to generate.'))
           return
         }
 
-        console.log(chalk.green(`\nGenerated ${files.length} file(s):`))
-        for (const file of files) {
-          console.log(chalk.dim(`  ${file.path}`))
+        for (const file of written) {
+          console.log(chalk.green(`  CREATE  `) + chalk.dim(file.path))
+        }
+        for (const file of skipped) {
+          console.log(chalk.yellow(`  SKIP    `) + chalk.dim(file.path) + chalk.dim(' (already exists)'))
+        }
+
+        if (skipped.length > 0) {
+          console.log(chalk.dim(`\nSkipped ${skipped.length} existing file(s). Use --force to overwrite.`))
         }
       } catch (err) {
         console.error(chalk.red(`Error: ${err instanceof Error ? err.message : err}`))

package/src/commands/generate_models.ts

@@ -9,7 +9,8 @@ export function register(program: Command): void {
     .command('generate:models')
     .alias('g:models')
     .description('Generate model classes and enums from schema definitions')
-    .action(async () => {
+    .option('-f, --force', 'Overwrite existing generated files')
+    .action(async ({ force }: { force?: boolean }) => {
       try {
         const dbPaths = await getDatabasePaths()
         const config = await loadGeneratorConfig()
@@ -22,16 +23,22 @@ export function register(program: Command): void {
 
         const representation = registry.buildRepresentation()
         const generator = new ModelGenerator(registry.all(), representation, config)
-        const files = await generator.writeAll()
+        const { written, skipped } = await generator.writeAll(force)
 
-        if (files.length === 0) {
+        if (written.length === 0 && skipped.length === 0) {
           console.log(chalk.yellow('No models to generate.'))
           return
         }
 
-        console.log(chalk.green(`\nGenerated ${files.length} file(s):`))
-        for (const file of files) {
-          console.log(chalk.dim(`  ${file.path}`))
+        for (const file of written) {
+          console.log(chalk.green(`  CREATE  `) + chalk.dim(file.path))
+        }
+        for (const file of skipped) {
+          console.log(chalk.yellow(`  SKIP    `) + chalk.dim(file.path) + chalk.dim(' (already exists)'))
+        }
+
+        if (skipped.length > 0) {
+          console.log(chalk.dim(`\nSkipped ${skipped.length} existing file(s). Use --force to overwrite.`))
         }
       } catch (err) {
         console.error(chalk.red(`Error: ${err instanceof Error ? err.message : err}`))

package/src/generators/api_generator.ts

@@ -10,7 +10,7 @@ import type { FieldDefinition, FieldValidator } from '@strav/database/schema/fie
 import type { PostgreSQLCustomType } from '@strav/database/schema/postgres'
 import { toSnakeCase, toCamelCase, toPascalCase } from '@strav/kernel/helpers/strings'
 import type { GeneratedFile } from './model_generator.ts'
-import type { GeneratorConfig, GeneratorPaths } from './config.ts'
+import type { GeneratorConfig, GeneratorPaths, WriteResult } from './config.ts'
 import { resolvePaths, relativeImport, formatAndWrite } from './config.ts'
 
 // ---------------------------------------------------------------------------
@@ -208,10 +208,9 @@ export default class ApiGenerator {
   }
 
   /** Generate, format with Prettier, and write all files to disk. */
-  async writeAll(): Promise<GeneratedFile[]> {
+  async writeAll(force?: boolean): Promise<WriteResult> {
     const files = this.generate()
-    await formatAndWrite(files)
-    return files
+    return formatAndWrite(files, { force })
   }
 
   // ---------------------------------------------------------------------------

package/src/generators/config.ts

@@ -1,3 +1,4 @@
+import { existsSync } from 'node:fs'
 import { resolve, relative } from 'node:path'
 import type { GeneratedFile } from './model_generator.ts'
 
@@ -66,11 +67,22 @@ export function relativeImport(fromDir: string, toDir: string): string {
   return rel.startsWith('.') ? rel : './' + rel
 }
 
+export interface WriteResult {
+  written: GeneratedFile[]
+  skipped: GeneratedFile[]
+}
+
 /**
  * Format generated files with Prettier and write them to disk.
  * Falls back to writing unformatted content if Prettier is not installed.
+ *
+ * Skips files that already exist unless `force` is true. Returns the
+ * partition of written and skipped files so callers can report them.
  */
-export async function formatAndWrite(files: GeneratedFile[]): Promise<void> {
+export async function formatAndWrite(
+  files: GeneratedFile[],
+  options: { force?: boolean } = {}
+): Promise<WriteResult> {
   let prettier: typeof import('prettier') | null = null
   try {
     prettier = await import('prettier')
@@ -78,13 +90,24 @@ export async function formatAndWrite(files: GeneratedFile[]): Promise<void> {
     // Prettier not installed — write unformatted
   }
 
+  const written: GeneratedFile[] = []
+  const skipped: GeneratedFile[] = []
+
   for (const file of files) {
+    if (existsSync(file.path) && !options.force) {
+      skipped.push(file)
+      continue
+    }
+
     let content = file.content
     if (prettier) {
       const filePath = resolve(file.path)
-      const options = await prettier.resolveConfig(filePath)
-      content = await prettier.format(content, { ...options, filepath: filePath })
+      const prettierOpts = await prettier.resolveConfig(filePath)
+      content = await prettier.format(content, { ...prettierOpts, filepath: filePath })
     }
     await Bun.write(file.path, content)
+    written.push(file)
   }
+
+  return { written, skipped }
 }

package/src/generators/doc_generator.ts

@@ -14,8 +14,9 @@ import {
   toPascalCase,
   pluralize,
 } from '@strav/kernel/helpers/strings'
+import { existsSync } from 'node:fs'
 import type { GeneratedFile } from './model_generator.ts'
-import type { GeneratorConfig, GeneratorPaths } from './config.ts'
+import type { GeneratorConfig, GeneratorPaths, WriteResult } from './config.ts'
 import { resolvePaths } from './config.ts'
 import { ApiRouting, toRouteSegment, toChildSegment } from './route_generator.ts'
 import type { ApiRoutingConfig } from './route_generator.ts'
@@ -91,12 +92,21 @@ export default class DocGenerator {
     return [this.generateIndexPage()]
   }
 
-  async writeAll(): Promise<GeneratedFile[]> {
+  async writeAll(force?: boolean): Promise<WriteResult> {
     const files = this.generate()
+    const written: GeneratedFile[] = []
+    const skipped: GeneratedFile[] = []
+
     for (const file of files) {
+      if (existsSync(file.path) && !force) {
+        skipped.push(file)
+        continue
+      }
       await Bun.write(file.path, file.content)
+      written.push(file)
     }
-    return files
+
+    return { written, skipped }
   }
 
   // ---------------------------------------------------------------------------

package/src/generators/model_generator.ts

@@ -9,7 +9,7 @@ import type {
 } from '@strav/database/schema/database_representation'
 import type { PostgreSQLCustomType } from '@strav/database/schema/postgres'
 import { toSnakeCase, toCamelCase, toPascalCase } from '@strav/kernel/helpers/strings'
-import type { GeneratorConfig, GeneratorPaths } from './config.ts'
+import type { GeneratorConfig, GeneratorPaths, WriteResult } from './config.ts'
 import { resolvePaths, relativeImport, formatAndWrite } from './config.ts'
 
 export interface GeneratedFile {
@@ -54,10 +54,9 @@ export default class ModelGenerator {
   }
 
   /** Generate, format with Prettier, and write all files to disk. */
-  async writeAll(): Promise<GeneratedFile[]> {
+  async writeAll(force?: boolean): Promise<WriteResult> {
     const files = this.generate()
-    await formatAndWrite(files)
-    return files
+    return formatAndWrite(files, { force })
   }
 
   // ---------------------------------------------------------------------------

package/src/generators/route_generator.ts

@@ -3,7 +3,7 @@ import { Archetype } from '@strav/database/schema/types'
 import type { SchemaDefinition } from '@strav/database/schema/types'
 import { toSnakeCase, toPascalCase, pluralize } from '@strav/kernel/helpers/strings'
 import type { GeneratedFile } from './model_generator.ts'
-import type { GeneratorConfig, GeneratorPaths } from './config.ts'
+import type { GeneratorConfig, GeneratorPaths, WriteResult } from './config.ts'
 import { resolvePaths, relativeImport, formatAndWrite } from './config.ts'
 
 /** Archetypes that sit under a parent's /:parentId group. */
@@ -66,10 +66,9 @@ export default class RouteGenerator {
   }
 
   /** Generate, format with Prettier, and write the route file to disk. */
-  async writeAll(): Promise<GeneratedFile[]> {
+  async writeAll(force?: boolean): Promise<WriteResult> {
     const files = this.generate()
-    await formatAndWrite(files)
-    return files
+    return formatAndWrite(files, { force })
   }
 
   // ---------------------------------------------------------------------------

package/src/generators/test_generator.ts

@@ -13,7 +13,7 @@ import {
   pluralize,
 } from '@strav/kernel/helpers/strings'
 import type { GeneratedFile } from './model_generator.ts'
-import type { GeneratorConfig, GeneratorPaths } from './config.ts'
+import type { GeneratorConfig, GeneratorPaths, WriteResult } from './config.ts'
 import { resolvePaths, relativeImport, formatAndWrite } from './config.ts'
 import { ApiRouting, toRouteSegment, toChildSegment } from './route_generator.ts'
 import type { ApiRoutingConfig } from './route_generator.ts'
@@ -82,10 +82,9 @@ export default class TestGenerator {
     return files
   }
 
-  async writeAll(): Promise<GeneratedFile[]> {
+  async writeAll(force?: boolean): Promise<WriteResult> {
     const files = this.generate()
-    await formatAndWrite(files)
-    return files
+    return formatAndWrite(files, { force })
   }
 
   // ---------------------------------------------------------------------------