claude-brain 0.30.2 → 0.30.3

package/src/routing/types.ts

@@ -1,94 +1,104 @@
-/**
- * Routing Types
- * Phase 19: Normalized result types for the brain() router
- *
- * Eliminates scattered `r.decision?.decision || r.content?.slice(0, 300)`
- * by normalizing all search results to a flat structure.
- */
-
-export interface NormalizedResult {
-  id: string
-  content: string
-  score: number
-  source: 'decision' | 'pattern' | 'correction' | 'memory' | 'graph' | 'episode' | 'cross-project'
-  project?: string
-  date?: string
-  metadata?: Record<string, unknown>
-}
-
-/**
- * Normalize raw searchRaw() results into flat NormalizedResult[]
- * Handles the nested { memory: { id }, decision: { id } } structure
- */
-/**
- * Ensure a value is a string — prevents [object Object] in template literals
- */
-function ensureString(val: unknown): string {
-  if (typeof val === 'string') return val
-  if (val === null || val === undefined) return ''
-  if (typeof val === 'object') return JSON.stringify(val)
-  return String(val)
-}
-
-export function normalizeSearchResults(rawResults: any[]): NormalizedResult[] {
-  return rawResults.map(r => {
-    const id = r.memory?.id || r.decision?.id || r.id || ''
-    const project = r.memory?.project || r.decision?.project || r.metadata?.project || ''
-    const date = r.memory?.createdAt || r.decision?.createdAt || r.metadata?.created_at || ''
-
-    // Prefer decision content, fall back to memory content
-    let content: string
-    if (r.decision?.decision) {
-      content = ensureString(r.decision.decision)
-      if (r.decision.reasoning) {
-        content = `**${content}**\n${ensureString(r.decision.reasoning)}`
-      }
-    } else if (r.memory?.content) {
-      content = ensureString(r.memory.content)
-    } else if (r.content) {
-      content = ensureString(r.content)
-    } else {
-      content = ''
-    }
-
-    return {
-      id,
-      content,
-      score: r.similarity || r.score || 0,
-      source: r.decision ? 'decision' as const : 'memory' as const,
-      project: typeof project === 'string' ? project : String(project),
-      date: date instanceof Date ? date.toISOString() : typeof date === 'string' ? date : '',
-      metadata: r.metadata || (r.decision ? { ...r.decision } : r.memory?.metadata) || {}
-    }
-  })
-}
-
-/**
- * Normalize pattern search results
- */
-export function normalizePatternResults(results: any[]): NormalizedResult[] {
-  return results.map(p => ({
-    id: p.id || '',
-    content: ensureString(p.metadata?.description || p.description || p.content || ''),
-    score: p.similarity || p.score || 0,
-    source: 'pattern' as const,
-    project: ensureString(p.metadata?.project || p.project || ''),
-    date: ensureString(p.metadata?.created_at || ''),
-    metadata: p.metadata || {}
-  }))
-}
-
-/**
- * Normalize correction search results
- */
-export function normalizeCorrectionResults(results: any[]): NormalizedResult[] {
-  return results.map(c => ({
-    id: c.id || '',
-    content: `Original: ${ensureString(c.metadata?.original || c.original || '')}\nFix: ${ensureString(c.metadata?.correction || c.correction || c.content || '')}`,
-    score: c.similarity || c.score || 0,
-    source: 'correction' as const,
-    project: ensureString(c.metadata?.project || c.project || ''),
-    date: ensureString(c.metadata?.created_at || ''),
-    metadata: c.metadata || {}
-  }))
-}
+/**
+ * Routing Types
+ * Phase 19: Normalized result types for the brain() router
+ *
+ * Eliminates scattered `r.decision?.decision || r.content?.slice(0, 300)`
+ * by normalizing all search results to a flat structure.
+ */
+
+export interface NormalizedResult {
+  id: string
+  content: string
+  score: number
+  source: 'decision' | 'pattern' | 'correction' | 'memory' | 'graph' | 'episode' | 'cross-project'
+  project?: string
+  date?: string
+  metadata?: Record<string, unknown>
+}
+
+/**
+ * Normalize raw searchRaw() results into flat NormalizedResult[]
+ * Handles the nested { memory: { id }, decision: { id } } structure
+ */
+/**
+ * Ensure a value is a string — prevents [object Object] in template literals
+ */
+function ensureString(val: unknown): string {
+  if (typeof val === 'string') return val
+  if (val === null || val === undefined) return ''
+  if (typeof val === 'object') return JSON.stringify(val)
+  return String(val)
+}
+
+export function normalizeSearchResults(rawResults: Record<string, unknown>[]): NormalizedResult[] {
+  return rawResults.map(r => {
+    const memory = r.memory as Record<string, unknown> | undefined
+    const decision = r.decision as Record<string, unknown> | undefined
+    const meta = r.metadata as Record<string, unknown> | undefined
+
+    const id = String(memory?.id || decision?.id || r.id || '')
+    const project = memory?.project || decision?.project || meta?.project || ''
+    const date = memory?.createdAt || decision?.createdAt || meta?.created_at || ''
+
+    // Prefer decision content, fall back to memory content
+    let content: string
+    if (decision?.decision) {
+      content = ensureString(decision.decision)
+      if (decision.reasoning) {
+        content = `**${content}**\n${ensureString(decision.reasoning)}`
+      }
+    } else if (memory?.content) {
+      content = ensureString(memory.content)
+    } else if (r.content) {
+      content = ensureString(r.content)
+    } else {
+      content = ''
+    }
+
+    return {
+      id,
+      content,
+      score: (r.similarity as number) || (r.score as number) || 0,
+      source: decision ? 'decision' as const : 'memory' as const,
+      project: typeof project === 'string' ? project : String(project),
+      date: date instanceof Date ? date.toISOString() : typeof date === 'string' ? date : '',
+      metadata: (meta || (decision ? { ...decision } : (memory?.metadata as Record<string, unknown>)) || {}) as Record<string, unknown>
+    }
+  })
+}
+
+/**
+ * Normalize pattern search results
+ */
+export function normalizePatternResults(results: Record<string, unknown>[]): NormalizedResult[] {
+  return results.map(p => {
+    const meta = (p.metadata || {}) as Record<string, unknown>
+    return {
+      id: String(p.id || ''),
+      content: ensureString(meta.description || p.description || p.content || ''),
+      score: (p.similarity as number) || (p.score as number) || 0,
+      source: 'pattern' as const,
+      project: ensureString(meta.project || p.project || ''),
+      date: ensureString(meta.created_at || ''),
+      metadata: meta
+    }
+  })
+}
+
+/**
+ * Normalize correction search results
+ */
+export function normalizeCorrectionResults(results: Record<string, unknown>[]): NormalizedResult[] {
+  return results.map(c => {
+    const meta = (c.metadata || {}) as Record<string, unknown>
+    return {
+      id: String(c.id || ''),
+      content: `Original: ${ensureString(meta.original || c.original || '')}\nFix: ${ensureString(meta.correction || c.correction || c.content || '')}`,
+      score: (c.similarity as number) || (c.score as number) || 0,
+      source: 'correction' as const,
+      project: ensureString(meta.project || c.project || ''),
+      date: ensureString(meta.created_at || ''),
+      metadata: meta
+    }
+  })
+}

package/src/scripts/health-check.ts

@@ -1,118 +1,118 @@
-#!/usr/bin/env bun
-
-import { existsSync, statSync, accessSync, constants } from 'node:fs'
-import { resolve } from 'node:path'
-import { loadConfig } from '@/config'
-
-interface HealthCheck {
-  name: string
-  status: 'pass' | 'fail' | 'warn'
-  message: string
-}
-
-const checks: HealthCheck[] = []
-
-function addCheck(name: string, status: HealthCheck['status'], message: string) {
-  checks.push({ name, status, message })
-}
-
-async function checkConfig(): Promise<void> {
-  try {
-    await loadConfig()
-    addCheck('Configuration', 'pass', 'Loaded and validated')
-  } catch (error) {
-    addCheck('Configuration', 'fail', error instanceof Error ? error.message : 'Failed to load')
-  }
-}
-
-function checkDirectory(name: string, path: string): void {
-  const dirPath = resolve(process.cwd(), path)
-  if (existsSync(dirPath)) {
-    try {
-      accessSync(dirPath, constants.R_OK | constants.W_OK)
-      addCheck(name, 'pass', `${path}/ exists and is writable`)
-    } catch {
-      addCheck(name, 'fail', `${path}/ exists but is not writable`)
-    }
-  } else {
-    addCheck(name, 'warn', `${path}/ does not exist`)
-  }
-}
-
-async function checkVaultPath(): Promise<void> {
-  try {
-    const config = await loadConfig()
-    if (!config.vaultPath || config.vaultPath === '/path/to/your/obsidian/vault') {
-      addCheck('Vault Path', 'warn', 'Not configured')
-      return
-    }
-
-    if (existsSync(config.vaultPath)) {
-      const stats = statSync(config.vaultPath)
-      if (stats.isDirectory()) {
-        addCheck('Vault Path', 'pass', `Exists: ${config.vaultPath}`)
-      } else {
-        addCheck('Vault Path', 'fail', 'Path exists but is not a directory')
-      }
-    } else {
-      addCheck('Vault Path', 'fail', `Does not exist: ${config.vaultPath}`)
-    }
-  } catch {
-    addCheck('Vault Path', 'warn', 'Could not verify (config error)')
-  }
-}
-
-function checkDependencies(): void {
-  const pkgPath = resolve(process.cwd(), 'package.json')
-  if (!existsSync(pkgPath)) {
-    addCheck('Dependencies', 'fail', 'package.json not found')
-    return
-  }
-
-  const nodeModulesPath = resolve(process.cwd(), 'node_modules')
-  if (!existsSync(nodeModulesPath)) {
-    addCheck('Dependencies', 'fail', 'node_modules not found - run bun install')
-    return
-  }
-
-  const requiredDeps = ['hono', '@modelcontextprotocol/sdk', 'zod', 'pino']
-  const missingDeps = requiredDeps.filter(dep =>
-    !existsSync(resolve(nodeModulesPath, dep))
-  )
-
-  if (missingDeps.length > 0) {
-    addCheck('Dependencies', 'fail', `Missing: ${missingDeps.join(', ')}`)
-  } else {
-    addCheck('Dependencies', 'pass', 'All required dependencies installed')
-  }
-}
-
-async function main() {
-  console.log('\n🏥 Claude Brain Health Check\n')
-  console.log('═'.repeat(50))
-
-  await checkConfig()
-  checkDirectory('Data Directory', 'data')
-  checkDirectory('Logs Directory', 'logs')
-  await checkVaultPath()
-  checkDependencies()
-
-  console.log('')
-  for (const check of checks) {
-    const symbol = check.status === 'pass' ? '✓' : check.status === 'warn' ? '⚠' : '✗'
-    const color = check.status === 'pass' ? '\x1b[32m' : check.status === 'warn' ? '\x1b[33m' : '\x1b[31m'
-    console.log(`${color}${symbol}\x1b[0m ${check.name}: ${check.message}`)
-  }
-
-  console.log('\n' + '═'.repeat(50))
-
-  const failures = checks.filter(c => c.status === 'fail').length
-  const warnings = checks.filter(c => c.status === 'warn').length
-  const passes = checks.filter(c => c.status === 'pass').length
-
-  console.log(`\nResults: ${passes} passed, ${warnings} warnings, ${failures} failed\n`)
-
-  process.exit(failures > 0 ? 1 : 0)
-}
-
-main()
+#!/usr/bin/env bun
+
+import { existsSync, statSync, accessSync, constants } from 'node:fs'
+import { resolve } from 'node:path'
+import { loadConfig } from '@/config'
+
+interface HealthCheck {
+  name: string
+  status: 'pass' | 'fail' | 'warn'
+  message: string
+}
+
+const checks: HealthCheck[] = []
+
+function addCheck(name: string, status: HealthCheck['status'], message: string) {
+  checks.push({ name, status, message })
+}
+
+async function checkConfig(): Promise<void> {
+  try {
+    await loadConfig()
+    addCheck('Configuration', 'pass', 'Loaded and validated')
+  } catch (error) {
+    addCheck('Configuration', 'fail', error instanceof Error ? error.message : 'Failed to load')
+  }
+}
+
+function checkDirectory(name: string, path: string): void {
+  const dirPath = resolve(process.cwd(), path)
+  if (existsSync(dirPath)) {
+    try {
+      accessSync(dirPath, constants.R_OK | constants.W_OK)
+      addCheck(name, 'pass', `${path}/ exists and is writable`)
+    } catch {
+      addCheck(name, 'fail', `${path}/ exists but is not writable`)
+    }
+  } else {
+    addCheck(name, 'warn', `${path}/ does not exist`)
+  }
+}
+
+async function checkVaultPath(): Promise<void> {
+  try {
+    const config = await loadConfig()
+    if (!config.vaultPath || config.vaultPath === '/path/to/your/obsidian/vault') {
+      addCheck('Vault Path', 'warn', 'Not configured')
+      return
+    }
+
+    if (existsSync(config.vaultPath)) {
+      const stats = statSync(config.vaultPath)
+      if (stats.isDirectory()) {
+        addCheck('Vault Path', 'pass', `Exists: ${config.vaultPath}`)
+      } else {
+        addCheck('Vault Path', 'fail', 'Path exists but is not a directory')
+      }
+    } else {
+      addCheck('Vault Path', 'fail', `Does not exist: ${config.vaultPath}`)
+    }
+  } catch {
+    addCheck('Vault Path', 'warn', 'Could not verify (config error)')
+  }
+}
+
+function checkDependencies(): void {
+  const pkgPath = resolve(process.cwd(), 'package.json')
+  if (!existsSync(pkgPath)) {
+    addCheck('Dependencies', 'fail', 'package.json not found')
+    return
+  }
+
+  const nodeModulesPath = resolve(process.cwd(), 'node_modules')
+  if (!existsSync(nodeModulesPath)) {
+    addCheck('Dependencies', 'fail', 'node_modules not found - run bun install')
+    return
+  }
+
+  const requiredDeps = ['hono', '@modelcontextprotocol/sdk', 'zod', 'pino']
+  const missingDeps = requiredDeps.filter(dep =>
+    !existsSync(resolve(nodeModulesPath, dep))
+  )
+
+  if (missingDeps.length > 0) {
+    addCheck('Dependencies', 'fail', `Missing: ${missingDeps.join(', ')}`)
+  } else {
+    addCheck('Dependencies', 'pass', 'All required dependencies installed')
+  }
+}
+
+async function main() {
+  console.log('\n🏥 Claude Brain Health Check\n')
+  console.log('═'.repeat(50))
+
+  await checkConfig()
+  checkDirectory('Data Directory', 'data')
+  checkDirectory('Logs Directory', 'logs')
+  await checkVaultPath()
+  checkDependencies()
+
+  console.log('')
+  for (const check of checks) {
+    const symbol = check.status === 'pass' ? '✓' : check.status === 'warn' ? '⚠' : '✗'
+    const color = check.status === 'pass' ? '\x1b[32m' : check.status === 'warn' ? '\x1b[33m' : '\x1b[31m'
+    console.log(`${color}${symbol}\x1b[0m ${check.name}: ${check.message}`)
+  }
+
+  console.log('\n' + '═'.repeat(50))
+
+  const failures = checks.filter(c => c.status === 'fail').length
+  const warnings = checks.filter(c => c.status === 'warn').length
+  const passes = checks.filter(c => c.status === 'pass').length
+
+  console.log(`\nResults: ${passes} passed, ${warnings} warnings, ${failures} failed\n`)
+
+  process.exit(failures > 0 ? 1 : 0)
+}
+
+main()