spine-framework 0.3.124 → 0.3.126

package/.framework/cli/commands/validate-app.ts

@@ -1,5 +1,28 @@
-#!/usr/bin/env node
 
+
+
+
+
+
+
+
+
+
+// CIL[BEGIN chunk_id="validate-app_moduleheader_1784068794" type="module_header" identity="imports" parent="none"]
+// CIL[DOC-MACRO purpose="Import dependencies and define validation types"]
+// CIL[DOC-RESPONSIBILITY responsibility="Load required modules and define interfaces"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[\".framework/functions/_shared/schema-utils.ts\",\"custom/apps/portal/pages/ContentPage.tsx\",\"custom/apps/cortex/pages/operations/OperationsDashboard.tsx\",\".framework/src/pages/admin/IntegrationsPage.tsx\",\".framework/functions/_shared/principal.ts\",\"\\\"]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="loaded_modules" errors="none" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Imports file system, path utilities, Commander, and defines validation types" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-AUDIENCE audience="developer"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: .framework/functions/_shared/schema-utils.ts, custom/apps/portal/pages/ContentPage.tsx, custom/apps/cortex/pages/operations/OperationsDashboard.tsx, .framework/src/pages/admin/IntegrationsPage.tsx, .framework/functions/_shared/principal.ts, \."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 import { Command } from 'commander'
 import { readFileSync, existsSync, readdirSync, statSync } from 'fs'
 import { join, dirname } from 'path'
@@ -53,7 +76,21 @@ class ValidationError extends Error {
     this.name = 'ValidationError'
   }
 }
-
+// CIL[END chunk_id="validate-app_moduleheader_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validatemanifest_1784068794" type="function" identity="validateManifest" parent="none"]
+// CIL[DOC-MACRO purpose="Validate manifest.json structure and required fields"]
+// CIL[DOC-RESPONSIBILITY responsibility="Parse and validate manifest file"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[\"custom/apps/portal/pages/ContentPage.tsx\",\".framework/src/pages/admin/IntegrationsPage.tsx\",\"custom/apps/cortex/pages/operations/OperationsDashboard.tsx\",\".framework/functions/_shared/db.ts\",\".framework/functions/_shared/schema-utils.ts\",\"\\\"]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="manifest" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Validates JSON format, required fields, and app_type values" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: custom/apps/portal/pages/ContentPage.tsx, .framework/src/pages/admin/IntegrationsPage.tsx, custom/apps/cortex/pages/operations/OperationsDashboard.tsx, .framework/functions/_shared/db.ts, .framework/functions/_shared/schema-utils.ts, \."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateManifest(appPath: string): Manifest {
   const manifestPath = join(appPath, 'manifest.json')
   
@@ -68,7 +105,6 @@ function validateManifest(appPath: string): Manifest {
     throw new ValidationError('❌ manifest.json is not valid JSON')
   }
   
-  // Required fields
   const requiredFields = ['name', 'slug', 'version', 'app_type']
   for (const field of requiredFields) {
     if (!manifest[field]) {
@@ -76,7 +112,6 @@ function validateManifest(appPath: string): Manifest {
     }
   }
   
-  // Validate app_type
   const validAppTypes = ['full', 'ui', 'backend', 'data']
   if (!validAppTypes.includes(manifest.app_type)) {
     throw new ValidationError(`❌ manifest.json app_type must be one of: ${validAppTypes.join(', ')}`)
@@ -84,7 +119,21 @@ function validateManifest(appPath: string): Manifest {
   
   return manifest
 }
-
+// CIL[END chunk_id="validate-app_validatemanifest_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validatepackagejson_1784068794" type="function" identity="validatePackageJson" parent="none"]
+// CIL[DOC-MACRO purpose="Validate package.json structure and version consistency"]
+// CIL[DOC-RESPONSIBILITY responsibility="Parse and validate package file"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[\"custom/apps/portal/pages/ContentPage.tsx\",\".framework/src/pages/admin/IntegrationsPage.tsx\",\"custom/apps/cortex/pages/operations/OperationsDashboard.tsx\",\".framework/src/components/ui/dialog.tsx\",\".framework/src/pages/admin/RolesPage.tsx\",\"\\\"]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="package_json" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Validates JSON format and version match with manifest" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: custom/apps/portal/pages/ContentPage.tsx, .framework/src/pages/admin/IntegrationsPage.tsx, custom/apps/cortex/pages/operations/OperationsDashboard.tsx, .framework/src/components/ui/dialog.tsx, .framework/src/pages/admin/RolesPage.tsx, \."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validatePackageJson(appPath: string, manifest: Manifest): PackageJson {
   const packagePath = join(appPath, 'package.json')
   
@@ -99,39 +148,63 @@ function validatePackageJson(appPath: string, manifest: Manifest): PackageJson {
     throw new ValidationError('❌ package.json is not valid JSON')
   }
   
-  // Check version sync
   if (packageJson.version !== manifest.version) {
     throw new ValidationError(`❌ Version mismatch: package.json has ${packageJson.version} but manifest.json has ${manifest.version}`)
   }
   
   return packageJson
 }
-
+// CIL[END chunk_id="validate-app_validatepackagejson_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validaterequiredfiles_1784068794" type="function" identity="validateRequiredFiles" parent="none"]
+// CIL[DOC-MACRO purpose="Validate required files and directories for app type"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check existence of required app structure"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[\"custom/apps/portal/pages/ContentPage.tsx\",\"custom/apps/cortex/pages/operations/OperationsDashboard.tsx\",\".framework/src/pages/admin/TypeDetailPage.tsx\",\".framework/migrations/015_simplify_apps_table.sql\",\"\\\"]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Validates presence of index.tsx, pages/, and functions/ based on app_type" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: custom/apps/portal/pages/ContentPage.tsx, custom/apps/cortex/pages/operations/OperationsDashboard.tsx, .framework/src/pages/admin/TypeDetailPage.tsx, .framework/migrations/015_simplify_apps_table.sql, \."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateRequiredFiles(appPath: string, manifest: Manifest): void {
   const { app_type } = manifest
   
-  // index.tsx is required for full and ui apps
   if (['full', 'ui'].includes(app_type)) {
     if (!existsSync(join(appPath, 'index.tsx'))) {
       throw new ValidationError('❌ index.tsx is required for full and ui apps')
     }
   }
   
-  // pages/ is required for full and ui apps
   if (['full', 'ui'].includes(app_type)) {
     if (!existsSync(join(appPath, 'pages'))) {
       throw new ValidationError('❌ pages/ directory is required for full and ui apps')
     }
   }
   
-  // functions/ is required for full and backend apps
   if (['full', 'backend'].includes(app_type)) {
     if (!existsSync(join(appPath, 'functions'))) {
       throw new ValidationError('❌ functions/ directory is required for full and backend apps')
     }
   }
 }
-
+// CIL[END chunk_id="validate-app_validaterequiredfiles_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validatealloweddirectories_1784068794" type="function" identity="validateAllowedDirectories" parent="none"]
+// CIL[DOC-MACRO purpose="Validate only allowed directories exist in app"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check for unauthorized directory names"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[\".framework/migrations/015_simplify_apps_table.sql\",\"custom/apps/cortex/functions/custom_cil-embeddings.ts\",\".framework/functions/_shared/principal.ts\",\".framework/src/pages/admin/TypeDetailPage.tsx\",\"\\\"]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Ensures only standard directories plus optional .devin/ exist" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: .framework/migrations/015_simplify_apps_table.sql, custom/apps/cortex/functions/custom_cil-embeddings.ts, .framework/functions/_shared/principal.ts, .framework/src/pages/admin/TypeDetailPage.tsx, \."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateAllowedDirectories(appPath: string): void {
   const allowedDirs = [
     'pages', 'components', 'hooks', 'utils', 'functions', 
@@ -142,14 +215,27 @@ function validateAllowedDirectories(appPath: string): void {
     .filter(dirent => dirent.isDirectory())
     .map(dirent => dirent.name)
   
-  // Filter out .devin/ since it's optional and handled separately
   const unknownDirs = entries.filter(dir => !allowedDirs.includes(dir) && dir !== '.devin')
   
   if (unknownDirs.length > 0) {
     throw new ValidationError(`❌ Unknown directories found: ${unknownDirs.join(', ')}. Allowed directories: ${allowedDirs.join(', ')} (plus optional .devin/)`)
   }
 }
-
+// CIL[END chunk_id="validate-app_validatealloweddirectories_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validatedevindirectory_1784068794" type="function" identity="validateDevinDirectory" parent="none"]
+// CIL[DOC-MACRO purpose="Validate .devin directory structure if present"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check AGENTS.md and skills structure"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="console_output"]
+// CIL[DOC-MICRO logic_summary="Validates AGENTS.md presence and skills directory structure" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateDevinDirectory(appPath: string): void {
   const devinPath = join(appPath, '.devin')
   
@@ -161,7 +247,6 @@ function validateDevinDirectory(appPath: string): void {
       throw new ValidationError('❌ .devin/ directory exists but AGENTS.md is missing. If .devin/ exists, it must contain AGENTS.md with agentic guidance.')
     }
     
-    // Validate skills if they exist
     const skillsPath = join(devinPath, 'skills')
     if (existsSync(skillsPath)) {
       const skillDirs = readdirSync(skillsPath, { withFileTypes: true })
@@ -180,7 +265,21 @@ function validateDevinDirectory(appPath: string): void {
     console.log(`✅ .devin/ directory not present (optional)`)
   }
 }
-
+// CIL[END chunk_id="validate-app_validatedevindirectory_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validateroutes_1784068794" type="function" identity="validateRoutes" parent="none"]
+// CIL[DOC-MACRO purpose="Validate route and navigation item paths"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check route prefix and path formatting"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="console_output"]
+// CIL[DOC-MICRO logic_summary="Validates routes and nav_items are root-relative and properly formatted" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="complex_dependencies"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateRoutes(appPath: string, manifest: Manifest): void {
   if (!manifest.route_prefix) {
     console.warn('⚠️  manifest.json is missing route_prefix — install-app will default to /{slug}')
@@ -204,7 +303,6 @@ function validateRoutes(appPath: string, manifest: Manifest): void {
     }
   }
 
-  // nav_items paths must also be root-relative
   if (Array.isArray((manifest as any).nav_items)) {
     const checkNavItem = (item: any) => {
       if (item.path && prefix && item.path.startsWith(prefix)) {
@@ -219,9 +317,22 @@ function validateRoutes(appPath: string, manifest: Manifest): void {
     for (const item of (manifest as any).nav_items) checkNavItem(item)
   }
 }
-
+// CIL[END chunk_id="validate-app_validateroutes_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validatemanifestcomponents_1784068794" type="function" identity="validateManifestComponents" parent="none"]
+// CIL[DOC-MACRO purpose="Validate manifest component references exist"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check entry_point and sidebar_component files"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Validates referenced files exist and registration config is consistent" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateManifestComponents(appPath: string, manifest: Manifest): void {
-  // entry_point — if declared, the file must exist
   if (manifest.entry_point) {
     const ep = manifest.entry_point.replace(/^\.\//, '')
     if (!existsSync(join(appPath, ep))) {
@@ -229,7 +340,6 @@ function validateManifestComponents(appPath: string, manifest: Manifest): void {
     }
   }
 
-  // sidebar_component — if declared, the file must exist
   if (manifest.sidebar_component) {
     const sc = manifest.sidebar_component.replace(/^\.\//, '')
     if (!existsSync(join(appPath, sc))) {
@@ -237,14 +347,27 @@ function validateManifestComponents(appPath: string, manifest: Manifest): void {
     }
   }
 
-  // registration — account_strategy "existing" requires target_account
   if (manifest.registration?.account_strategy === 'existing' && !manifest.registration?.target_account) {
     throw new ValidationError(`❌ manifest.json registration.account_strategy is "existing" but target_account is not set`)
   }
 }
-
+// CIL[END chunk_id="validate-app_validatemanifestcomponents_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validateseedblock_1784068794" type="function" identity="validateSeedBlock" parent="none"]
+// CIL[DOC-MACRO purpose="Validate seed data configuration"]
+// CIL[DOC-RESPONSIBILITY responsibility="Check seed files and configuration"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="none"]
+// CIL[DOC-MICRO logic_summary="Validates seed array structure and SQL file existence" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
 function validateSeedBlock(appPath: string, manifest: Manifest): void {
-  if (!manifest.seed) return // optional — legacy apps without seed block are fine
+  if (!manifest.seed) return
 
   if (!Array.isArray(manifest.seed)) {
     throw new ValidationError('❌ manifest.json seed must be an array')
@@ -252,109 +375,80 @@ function validateSeedBlock(appPath: string, manifest: Manifest): void {
 
   for (const entry of manifest.seed) {
     if (!entry.file || !entry.table || !entry.conflict) {
-      throw new ValidationError(
-        `❌ manifest.json seed entry missing required field(s) (file, table, conflict): ${JSON.stringify(entry)}`
-      )
-    }
-    const filePath = join(appPath, entry.file)
-    if (!existsSync(filePath)) {
-      throw new ValidationError(`❌ manifest.json seed entry file "${entry.file}" does not exist`)
-    }
-    // Verify the file is valid JSON and an array
-    try {
-      const contents = JSON.parse(readFileSync(filePath, 'utf8'))
-      if (!Array.isArray(contents)) {
-        throw new ValidationError(`❌ Seed file "${entry.file}" must export a JSON array`)
-      }
-    } catch (err: any) {
-      if (err instanceof ValidationError) throw err
-      throw new ValidationError(`❌ Seed file "${entry.file}" is not valid JSON: ${err.message}`)
+      throw new ValidationError('❌ manifest.json seed entries must have file, table, and conflict fields')
     }
-  }
-}
 
-function validatePackagePublishability(packageJson: PackageJson): void {
-  if (packageJson.private === true) {
-    throw new ValidationError('❌ package.json has "private": true — app cannot be published to npm')
-  }
-  if (packageJson.private === undefined) {
-    console.warn('⚠️  package.json is missing "private": false — add it to confirm the package is publishable')
-  }
-}
-
-function validatePackageFiles(appPath: string, packageJson: PackageJson): void {
-  const { files = [] } = packageJson
-  
-  // Check that important directories are included in files array
-  const importantFiles = ['index.tsx', 'manifest.json']
-  const importantDirs = ['pages', 'components', 'hooks', 'utils', 'functions', 'seed', 'public']
-  
-  for (const file of importantFiles) {
-    if (existsSync(join(appPath, file)) && !files.includes(file)) {
-      console.warn(`⚠️  ${file} exists but is not in package.json files array`)
-    }
-  }
-  
-  for (const dir of importantDirs) {
-    if (existsSync(join(appPath, dir)) && !files.includes(`${dir}/`)) {
-      console.warn(`⚠️  ${dir}/ exists but is not in package.json files array`)
+    const seedFile = join(appPath, 'seed', entry.file)
+    if (!existsSync(seedFile)) {
+      throw new ValidationError(`❌ Seed file not found: seed/${entry.file}`)
     }
-  }
-}
-
-function validateApp(appPath: string): void {
-  try {
-    console.log(`🔍 Validating app at: ${appPath}`)
-    
-    const manifest = validateManifest(appPath)
-    console.log(`✅ manifest.json valid for ${manifest.name} v${manifest.version}`)
-    
-    const packageJson = validatePackageJson(appPath, manifest)
-    console.log(`✅ package.json valid and versions match`)
-    
-    validateRequiredFiles(appPath, manifest)
-    console.log(`✅ Required files present for app_type: ${manifest.app_type}`)
-
-    validateRoutes(appPath, manifest)
-    console.log(`✅ manifest.json routes and nav_items are root-relative`)
-
-    validateManifestComponents(appPath, manifest)
-    console.log(`✅ manifest.json entry_point, sidebar_component, and registration valid`)
-
-    validateSeedBlock(appPath, manifest)
-    console.log(`✅ manifest.json seed block valid`)
 
-    validateAllowedDirectories(appPath)
-    console.log(`✅ No unknown directories found`)
-
-    validateDevinDirectory(appPath)
-    console.log(`✅ .devin/ directory valid (if present)`)
-
-    validatePackagePublishability(packageJson)
-    console.log(`✅ package.json publishability valid`)
-
-    validatePackageFiles(appPath, packageJson)
-    console.log(`✅ package.json files array checked`)
-    
-    console.log(`\n🎉 App validation passed!`)
-    
-  } catch (err) {
-    if (err instanceof ValidationError) {
-      console.error(err.message)
-      process.exit(1)
-    } else {
-      console.error(`❌ Unexpected error: ${err.message}`)
-      process.exit(1)
+    if (!seedFile.endsWith('.sql')) {
+      throw new ValidationError(`❌ Seed file must be .sql: seed/${entry.file}`)
     }
   }
 }
-
-const program = new Command()
-
-program
-  .name('validate-app')
-  .description('Validate a Spine app structure and configuration')
-  .argument('[path]', 'Path to the app directory', '.')
-  .action(validateApp)
-
-program.parse()
+// CIL[END chunk_id="validate-app_validateseedblock_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_validateappstructure_1784068794" type="function" identity="validateAppStructure" parent="none"]
+// CIL[DOC-MACRO purpose="Run complete app structure validation"]
+// CIL[DOC-RESPONSIBILITY responsibility="Execute all validation checks"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="none" errors="validation_errors" side_effects="console_output"]
+// CIL[DOC-MICRO logic_summary="Runs all validation functions in sequence and reports success" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
+function validateAppStructure(appPath: string): void {
+  const manifest = validateManifest(appPath)
+  validatePackageJson(appPath, manifest)
+  validateRequiredFiles(appPath, manifest)
+  validateAllowedDirectories(appPath)
+  validateDevinDirectory(appPath)
+  validateRoutes(appPath, manifest)
+  validateManifestComponents(appPath, manifest)
+  validateSeedBlock(appPath, manifest)
+  console.log(`✅ App structure is valid`)
+}
+// CIL[END chunk_id="validate-app_validateappstructure_1784068794"]
+// CIL[BEGIN chunk_id="validate-app_registervalidateappcommands_1784068794" type="function" identity="registerValidateAppCommands" parent="none"]
+// CIL[DOC-MACRO purpose="Register validate-app command with CLI program"]
+// CIL[DOC-RESPONSIBILITY responsibility="Configure app validation command interface"]
+// CIL[DOC-CONTEXT module="cli" layer="commands" related_files="[]"]
+// CIL[DOC-CONTRACT inputs="none" outputs="validate_app_command" errors="command_errors" side_effects="process_exit"]
+// CIL[DOC-MICRO logic_summary="Registers validate-app command with path option and error handling" notable_decisions="[]"]
+// CIL[DOC-CONFIDENCE pass="second" score="0.95" flags="none"]
+// CIL[DOC-STABILITY stability="stable"]
+// CIL[DOC-VECTOR type="context" content="Layer: commands. Direct callers: 0. Direct calls: 0. Related files: none."]
+// CIL[DOC-VECTOR type="blast_radius" content="Direct callers: 0. Total affected: 0."]
+// CIL[DOC-VECTOR type="configuration" content="No configuration dependencies."]
+// CIL[DOC-VECTOR type="constraint" content="No test paths identified."]
+// CIL[DOC-VECTOR type="implementation_pattern" content="" status="pending_human_input"]
+// CIL[DOC-VECTOR type="user_journey" content="" status="pending_human_input"]
+export function registerValidateAppCommands(program: Command) {
+  program
+    .command('validate-app [path]')
+    .description('Validate app structure and manifest')
+    .option('--fix', 'Attempt to fix common issues (not implemented)')
+    .action(async (path = '.') => {
+      try {
+        const appPath = join(process.cwd(), path)
+        console.log(`Validating app at: ${appPath}`)
+        validateAppStructure(appPath)
+        console.log(`✅ App validation passed`)
+      } catch (err: any) {
+        if (err instanceof ValidationError) {
+          console.error(err.message)
+          process.exit(1)
+        } else {
+          console.error('❌ Unexpected error:', err.message)
+          process.exit(1)
+        }
+      }
+    })
+}
+// CIL[END chunk_id="validate-app_registervalidateappcommands_1784068794"]