prjct-cli 1.6.5 → 1.6.7

package/CHANGELOG.md

@@ -1,10 +1,46 @@
 # Changelog
 
-## [1.6.5] - 2026-02-07
+## [1.6.7] - 2026-02-07
+
+### Bug Fixes
+
+- add context to silent catch blocks in sync-service.ts (PRJ-80) (#120)
+
+### Refactoring
+
+- replace `any` types in routes-extended.ts and server.ts (PRJ-77) (#130)
+
+
+## [1.6.7] - 2026-02-07
+
+### Refactoring
+- **Replace `any` types in routes-extended.ts and server.ts (PRJ-77)**: Added `ProjectJson`, `StateJson`, `StateTask`, `QueueJson`, `QueueTask`, `RoadmapJson` interfaces to `core/types/storage.ts`. Replaced all 33 `any` types in `core/server/routes-extended.ts` with proper typed generics. Fixed `handleConnection: (c: any)` in `core/types/server.ts` with Hono `Context` type. Disabled redundant task component in statusline default config.
+
+### Test Plan
+
+#### For QA
+1. Build the project (`bun run build`) — should succeed
+2. Typecheck (`npx tsc -p core/tsconfig.json --noEmit`) — should pass clean
+3. Verify zero `any` types in `routes-extended.ts` and `server.ts`
+4. Status bar should no longer show task description segment
+
+#### For Users
+**What changed:** Internal type safety improvement. Cleaner status bar.
+**Breaking changes:** None
+
+
+## [1.6.6] - 2026-02-07
 
 ### Refactoring
 
-- extract common agent-base.md template (PRJ-95) (#128)
+- extract hardcoded values to constants (PRJ-71) (#129)
+
+
+## [1.6.8] - 2026-02-07
+
+### Refactor
+
+- **Extract hardcoded values to constants (PRJ-71)**: Added `OUTPUT_LIMITS`, `STORAGE_LIMITS`, `EVENT_LIMITS` to `core/constants/index.ts`. Replaced 16 magic numbers across `output.ts` (11 truncation lengths), `bus.ts` (history limit), and `jsonl-helper.ts` (max lines, rotation size, warning threshold). All limits now configurable from one place with `as const` typing.
 
 
 ## [1.6.7] - 2026-02-07

package/assets/statusline/default-config.json

@@ -13,7 +13,7 @@
       "position": 0
     },
     "task": {
-      "enabled": true,
+      "enabled": false,
       "position": 1,
       "maxLength": 25
     },

package/core/bus/bus.ts

@@ -9,6 +9,7 @@
 
 import fs from 'node:fs/promises'
 import path from 'node:path'
+import { EVENT_LIMITS } from '../constants'
 import { getErrorMessage } from '../errors'
 import pathManager from '../infrastructure/path-manager'
 import {
@@ -45,7 +46,7 @@ class EventBus {
     this.listeners = new Map()
     this.onceListeners = new Map()
     this.history = []
-    this.historyLimit = 100
+    this.historyLimit = EVENT_LIMITS.HISTORY_MAX
     this.projectId = null
   }
 

package/core/constants/index.ts

@@ -264,6 +264,65 @@ export function getTimeout(key: TimeoutKey): number {
   return TIMEOUTS[key]
 }
 
+// =============================================================================
+// Output Limits (PRJ-71)
+// =============================================================================
+
+/**
+ * Truncation lengths for CLI output messages.
+ * Centralizes magic numbers from output.ts.
+ */
+export const OUTPUT_LIMITS = {
+  /** Spinner message truncation */
+  SPINNER_MSG: 45,
+  /** Done/success message truncation */
+  DONE_MSG: 50,
+  /** Fail message truncation */
+  FAIL_MSG: 65,
+  /** Warn message truncation */
+  WARN_MSG: 65,
+  /** Step counter message truncation */
+  STEP_MSG: 35,
+  /** Progress bar text truncation */
+  PROGRESS_TEXT: 25,
+  /** Issue title truncation in lists */
+  ISSUE_TITLE: 50,
+  /** Fallback truncation when tier config is 0 */
+  FALLBACK_TRUNCATE: 50,
+  /** Terminal clear width */
+  CLEAR_WIDTH: 80,
+} as const
+
+// =============================================================================
+// Storage Limits (PRJ-71)
+// =============================================================================
+
+/**
+ * File size and line limits for JSONL/storage operations.
+ * Centralizes magic numbers from jsonl-helper.ts.
+ */
+export const STORAGE_LIMITS = {
+  /** Default max lines for streaming JSONL reads */
+  JSONL_MAX_LINES: 1000,
+  /** File rotation threshold in MB */
+  ROTATION_SIZE_MB: 10,
+  /** Warning threshold for large files in MB */
+  LARGE_FILE_WARN_MB: 50,
+} as const
+
+// =============================================================================
+// Event Bus Limits (PRJ-71)
+// =============================================================================
+
+/**
+ * Event bus configuration limits.
+ * Centralizes magic numbers from bus.ts.
+ */
+export const EVENT_LIMITS = {
+  /** Max events kept in history */
+  HISTORY_MAX: 100,
+} as const
+
 // =============================================================================
 // Combined Exports
 // =============================================================================

package/core/server/routes-extended.ts

@@ -13,6 +13,16 @@ import { Hono } from 'hono'
 import * as jsonc from 'jsonc-parser'
 import pathManager from '../infrastructure/path-manager'
 import { isNotFoundError } from '../types/fs'
+import type {
+  IdeasJson,
+  ProjectJson,
+  QueueJson,
+  QueueTask,
+  RoadmapJson,
+  ShippedJson,
+  StateJson,
+  StateTask,
+} from '../types/storage'
 
 // =============================================================================
 // HELPERS
@@ -52,9 +62,9 @@ function getProjectPath(projectId: string): string {
   return path.join(PROJECTS_DIR, projectId)
 }
 
-async function getProjectConfig(projectId: string): Promise<any> {
+async function getProjectConfig(projectId: string): Promise<ProjectJson | null> {
   const configPath = path.join(getProjectPath(projectId), 'project.json')
-  return await readJsonFile(configPath)
+  return await readJsonFile<ProjectJson>(configPath)
 }
 
 async function calculateDuration(startedAt: string | undefined): Promise<string> {
@@ -97,16 +107,18 @@ export function createExtendedRoutes(): Hono {
           const config = await getProjectConfig(id)
 
           // Read state
-          const state = await readJsonFile<any>(path.join(projectPath, 'storage/state.json'))
+          const state = await readJsonFile<StateJson>(path.join(projectPath, 'storage/state.json'))
 
           // Read queue for count
-          const queue = await readJsonFile<any>(path.join(projectPath, 'storage/queue.json'))
+          const queue = await readJsonFile<QueueJson>(path.join(projectPath, 'storage/queue.json'))
 
           // Read ideas for count
-          const ideas = await readJsonFile<any>(path.join(projectPath, 'storage/ideas.json'))
+          const ideas = await readJsonFile<IdeasJson>(path.join(projectPath, 'storage/ideas.json'))
 
           // Read shipped for count
-          const shipped = await readJsonFile<any>(path.join(projectPath, 'storage/shipped.json'))
+          const shipped = await readJsonFile<ShippedJson>(
+            path.join(projectPath, 'storage/shipped.json')
+          )
 
           const currentTask = state?.currentTask
           const duration = await calculateDuration(currentTask?.startedAt)
@@ -123,8 +135,8 @@ export function createExtendedRoutes(): Hono {
               : null,
             pausedTask: state?.previousTask || null,
             stats: {
-              queueCount: queue?.tasks?.filter((t: any) => !t.completed)?.length || 0,
-              ideasCount: ideas?.ideas?.filter((i: any) => i.status === 'pending')?.length || 0,
+              queueCount: queue?.tasks?.filter((t: QueueTask) => !t.completed)?.length || 0,
+              ideasCount: ideas?.ideas?.filter((i) => i.status === 'pending')?.length || 0,
               shippedCount: shipped?.shipped?.length || 0,
             },
           }
@@ -154,11 +166,11 @@ export function createExtendedRoutes(): Hono {
     try {
       const [config, state, queue, ideas, shipped, roadmap] = await Promise.all([
         getProjectConfig(projectId),
-        readJsonFile<any>(path.join(projectPath, 'storage/state.json')),
-        readJsonFile<any>(path.join(projectPath, 'storage/queue.json')),
-        readJsonFile<any>(path.join(projectPath, 'storage/ideas.json')),
-        readJsonFile<any>(path.join(projectPath, 'storage/shipped.json')),
-        readJsonFile<any>(path.join(projectPath, 'planning/roadmap.json')),
+        readJsonFile<StateJson>(path.join(projectPath, 'storage/state.json')),
+        readJsonFile<QueueJson>(path.join(projectPath, 'storage/queue.json')),
+        readJsonFile<IdeasJson>(path.join(projectPath, 'storage/ideas.json')),
+        readJsonFile<ShippedJson>(path.join(projectPath, 'storage/shipped.json')),
+        readJsonFile<RoadmapJson>(path.join(projectPath, 'planning/roadmap.json')),
       ])
 
       // Calculate current task duration
@@ -173,13 +185,13 @@ export function createExtendedRoutes(): Hono {
       weekStart.setDate(weekStart.getDate() - weekStart.getDay())
 
       const completedToday =
-        queue?.tasks?.filter((t: any) => {
+        queue?.tasks?.filter((t: QueueTask) => {
           if (!t.completed || !t.completedAt) return false
           return new Date(t.completedAt) >= todayStart
         })?.length || 0
 
       const completedThisWeek =
-        queue?.tasks?.filter((t: any) => {
+        queue?.tasks?.filter((t: QueueTask) => {
           if (!t.completed || !t.completedAt) return false
           return new Date(t.completedAt) >= weekStart
         })?.length || 0
@@ -196,8 +208,8 @@ export function createExtendedRoutes(): Hono {
         stats: {
           tasksToday: completedToday,
           tasksThisWeek: completedThisWeek,
-          queueCount: queue?.tasks?.filter((t: any) => !t.completed)?.length || 0,
-          ideasCount: ideas?.ideas?.filter((i: any) => i.status === 'pending')?.length || 0,
+          queueCount: queue?.tasks?.filter((t: QueueTask) => !t.completed)?.length || 0,
+          ideasCount: ideas?.ideas?.filter((i) => i.status === 'pending')?.length || 0,
           shippedCount: shipped?.shipped?.length || 0,
         },
         timestamp: new Date().toISOString(),
@@ -216,7 +228,7 @@ export function createExtendedRoutes(): Hono {
     const statePath = path.join(projectPath, 'storage/state.json')
 
     try {
-      const state = await readJsonFile<any>(statePath)
+      const state = await readJsonFile<StateJson>(statePath)
 
       if (!state?.currentTask) {
         return c.json({ success: false, error: 'No active task' }, 400)
@@ -225,7 +237,7 @@ export function createExtendedRoutes(): Hono {
       const completedTask = state.currentTask
 
       // Update state
-      const newState = {
+      const newState: StateJson = {
         currentTask: null,
         previousTask: null,
         lastUpdated: new Date().toISOString(),
@@ -255,13 +267,13 @@ export function createExtendedRoutes(): Hono {
       const body = await c.req.json().catch(() => ({}))
       const reason = body.reason
 
-      const state = await readJsonFile<any>(statePath)
+      const state = await readJsonFile<StateJson>(statePath)
 
       if (!state?.currentTask) {
         return c.json({ success: false, error: 'No active task' }, 400)
       }
 
-      const pausedTask = {
+      const pausedTask: StateTask = {
         id: state.currentTask.id,
         description: state.currentTask.description,
         status: 'paused',
@@ -270,7 +282,7 @@ export function createExtendedRoutes(): Hono {
         pauseReason: reason,
       }
 
-      const newState = {
+      const newState: StateJson = {
         currentTask: null,
         previousTask: pausedTask,
         lastUpdated: new Date().toISOString(),
@@ -297,20 +309,21 @@ export function createExtendedRoutes(): Hono {
     const statePath = path.join(projectPath, 'storage/state.json')
 
     try {
-      const state = await readJsonFile<any>(statePath)
+      const state = await readJsonFile<StateJson>(statePath)
 
       if (!state?.previousTask) {
         return c.json({ success: false, error: 'No paused task' }, 400)
       }
 
-      const resumedTask = {
+      const resumedTask: StateTask = {
         id: state.previousTask.id,
         description: state.previousTask.description,
+        status: 'active',
         startedAt: new Date().toISOString(),
         sessionId: `sess_${Date.now().toString(36)}`,
       }
 
-      const newState = {
+      const newState: StateJson = {
         currentTask: resumedTask,
         previousTask: null,
         lastUpdated: new Date().toISOString(),
@@ -345,8 +358,8 @@ export function createExtendedRoutes(): Hono {
         return c.json({ success: false, error: 'taskId required' }, 400)
       }
 
-      const state = await readJsonFile<any>(statePath)
-      const queue = await readJsonFile<any>(queuePath)
+      const state = await readJsonFile<StateJson>(statePath)
+      const queue = await readJsonFile<QueueJson>(queuePath)
 
       // Check if there's already an active task
       if (state?.currentTask) {
@@ -354,22 +367,23 @@ export function createExtendedRoutes(): Hono {
       }
 
       // Find task in queue
-      const task = queue?.tasks?.find((t: any) => t.id === taskId)
+      const task = queue?.tasks?.find((t: QueueTask) => t.id === taskId)
       if (!task) {
         return c.json({ success: false, error: 'Task not found in queue' }, 404)
       }
 
       // Create new current task
-      const newTask = {
+      const newTask: StateTask = {
         id: task.id,
         description: task.description,
+        status: 'active',
         startedAt: new Date().toISOString(),
         sessionId: `sess_${Date.now().toString(36)}`,
         featureId: task.featureId,
       }
 
       // Update state
-      const newState = {
+      const newState: StateJson = {
         currentTask: newTask,
         previousTask: null,
         lastUpdated: new Date().toISOString(),
@@ -403,14 +417,17 @@ export function createExtendedRoutes(): Hono {
         return c.json({ success: false, error: 'text required' }, 400)
       }
 
-      const ideas = (await readJsonFile<any>(ideasPath)) || { ideas: [], lastUpdated: '' }
+      const ideas: IdeasJson = (await readJsonFile<IdeasJson>(ideasPath)) || {
+        ideas: [],
+        lastUpdated: '',
+      }
 
       const newIdea = {
         id: `idea_${Date.now().toString(36)}`,
         text,
-        status: 'pending',
-        priority,
-        tags,
+        status: 'pending' as const,
+        priority: (priority || 'medium') as 'low' | 'medium' | 'high',
+        tags: tags as string[],
         addedAt: new Date().toISOString(),
       }
 
@@ -446,15 +463,17 @@ export function createExtendedRoutes(): Hono {
       for (const id of projectIds) {
         const projectPath = getProjectPath(id)
 
-        const state = await readJsonFile<any>(path.join(projectPath, 'storage/state.json'))
-        const queue = await readJsonFile<any>(path.join(projectPath, 'storage/queue.json'))
-        const ideas = await readJsonFile<any>(path.join(projectPath, 'storage/ideas.json'))
-        const shipped = await readJsonFile<any>(path.join(projectPath, 'storage/shipped.json'))
+        const state = await readJsonFile<StateJson>(path.join(projectPath, 'storage/state.json'))
+        const queue = await readJsonFile<QueueJson>(path.join(projectPath, 'storage/queue.json'))
+        const ideas = await readJsonFile<IdeasJson>(path.join(projectPath, 'storage/ideas.json'))
+        const shipped = await readJsonFile<ShippedJson>(
+          path.join(projectPath, 'storage/shipped.json')
+        )
 
         if (state?.currentTask) activeProjects++
 
-        totalTasks += queue?.tasks?.filter((t: any) => !t.completed)?.length || 0
-        totalIdeas += ideas?.ideas?.filter((i: any) => i.status === 'pending')?.length || 0
+        totalTasks += queue?.tasks?.filter((t: QueueTask) => !t.completed)?.length || 0
+        totalIdeas += ideas?.ideas?.filter((i) => i.status === 'pending')?.length || 0
         totalShipped += shipped?.shipped?.length || 0
       }
 
@@ -499,16 +518,16 @@ export function createExtendedRoutes(): Hono {
       }
 
       // Find active/paused task
-      let activeProject: any = null
-      let activeTask: any = null
-      let pausedTask: any = null
+      let activeProject: { id: string; name: string; path: string | undefined } | null = null
+      let activeTask: (StateTask & { duration?: string }) | null = null
+      let pausedTask: StateTask | null = null
 
       // If we have a target project, only check that one
       const idsToCheck = targetProjectId ? [targetProjectId] : projectIds
 
       for (const id of idsToCheck) {
         const projectPath = getProjectPath(id)
-        const state = await readJsonFile<any>(path.join(projectPath, 'storage/state.json'))
+        const state = await readJsonFile<StateJson>(path.join(projectPath, 'storage/state.json'))
         const config = await getProjectConfig(id)
 
         if (state?.currentTask) {

package/core/services/sync-service.ts

@@ -267,8 +267,8 @@ class SyncService {
           this.globalPath,
           localConfig?.verification
         )
-      } catch {
-        // Verification is non-critical — don't fail sync
+      } catch (error) {
+        log.debug('Verification failed (non-critical)', { error: getErrorMessage(error) })
       }
 
       return {
@@ -395,8 +395,8 @@ class SyncService {
         cwd: this.projectPath,
       })
       data.weeklyCommits = parseInt(weekly.trim(), 10) || 0
-    } catch {
-      // Not a git repo - use defaults
+    } catch (error) {
+      log.debug('Git analysis failed (not a git repo?)', { error: getErrorMessage(error) })
     }
 
     return data
@@ -424,7 +424,8 @@ class SyncService {
         { cwd: this.projectPath }
       )
       stats.fileCount = parseInt(stdout.trim(), 10) || 0
-    } catch {
+    } catch (error) {
+      log.debug('File count failed', { path: this.projectPath, error: getErrorMessage(error) })
       stats.fileCount = 0
     }
 
@@ -453,8 +454,8 @@ class SyncService {
       } else {
         stats.languages.push('JavaScript')
       }
-    } catch {
-      // No package.json
+    } catch (error) {
+      log.debug('No package.json found', { path: this.projectPath, error: getErrorMessage(error) })
     }
 
     // Check other ecosystems
@@ -620,8 +621,8 @@ class SyncService {
           await fs.unlink(path.join(agentsPath, file))
         }
       }
-    } catch {
-      // Directory might not exist yet
+    } catch (error) {
+      log.debug('Failed to purge old agents', { path: agentsPath, error: getErrorMessage(error) })
     }
 
     // Workflow agents (always generated) - IN PARALLEL
@@ -714,8 +715,11 @@ class SyncService {
       )
       content = await fs.readFile(templatePath, 'utf-8')
       content = await this.resolveTemplateIncludes(content)
-    } catch {
-      // Generate minimal agent
+    } catch (error) {
+      log.debug('Workflow agent template not found, generating minimal', {
+        name,
+        error: getErrorMessage(error),
+      })
       content = this.generateMinimalWorkflowAgent(name)
     }
 
@@ -749,8 +753,11 @@ class SyncService {
       content = content.replace('{projectName}', stats.name)
       content = content.replace('{frameworks}', stack.frameworks.join(', ') || 'None detected')
       content = content.replace('{ecosystem}', stats.ecosystem)
-    } catch {
-      // Generate minimal agent
+    } catch (error) {
+      log.debug('Domain agent template not found, generating minimal', {
+        name,
+        error: getErrorMessage(error),
+      })
       content = this.generateMinimalDomainAgent(name, stats, stack)
     }
 
@@ -839,8 +846,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
       path.join(this.globalPath, 'config', 'skills.json'),
       JSON.stringify(skillsConfig, null, 2),
       'utf-8'
-    ).catch(() => {
-      // Best effort
+    ).catch((error) => {
+      log.debug('Failed to write skills.json', { error: getErrorMessage(error) })
     })
 
     return skills
@@ -986,8 +993,11 @@ You are the ${name} expert for this project. Apply best practices for the detect
     let existing: Record<string, unknown> = {}
     try {
       existing = JSON.parse(await fs.readFile(projectJsonPath, 'utf-8'))
-    } catch {
-      // No existing file
+    } catch (error) {
+      log.debug('No existing project.json', {
+        path: projectJsonPath,
+        error: getErrorMessage(error),
+      })
     }
 
     const updated = {
@@ -1024,8 +1034,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
     let state: Record<string, unknown> = {}
     try {
       state = JSON.parse(await fs.readFile(statePath, 'utf-8'))
-    } catch {
-      // No existing file
+    } catch (error) {
+      log.debug('No existing state.json', { path: statePath, error: getErrorMessage(error) })
     }
 
     // Update with enterprise fields
@@ -1062,8 +1072,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
         this.projectPath,
         state as import('../schemas/state').StateJson
       )
-    } catch {
-      // Silently fail - local state is optional
+    } catch (error) {
+      log.debug('Local state generation failed (optional)', { error: getErrorMessage(error) })
     }
   }
 
@@ -1114,8 +1124,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
         const filePath = path.join(this.globalPath, file)
         const content = await fs.readFile(filePath, 'utf-8')
         filteredChars += content.length
-      } catch {
-        // File might not exist, skip
+      } catch (error) {
+        log.debug('Context file not found for metrics', { file, error: getErrorMessage(error) })
       }
     }
 
@@ -1125,8 +1135,11 @@ You are the ${name} expert for this project. Apply best practices for the detect
         const agentPath = path.join(this.globalPath, 'agents', `${agent.name}.md`)
         const content = await fs.readFile(agentPath, 'utf-8')
         filteredChars += content.length
-      } catch {
-        // Skip if not found
+      } catch (error) {
+        log.debug('Agent file not found for metrics', {
+          agent: agent.name,
+          error: getErrorMessage(error),
+        })
       }
     }
 
@@ -1152,8 +1165,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
         agents: agents.filter((a) => a.type === 'domain').map((a) => a.name),
       })
     } catch (error) {
-      // Non-blocking - metrics are nice to have
-      console.error('Warning: Failed to record metrics:', getErrorMessage(error))
+      log.debug('Failed to record sync metrics', { error: getErrorMessage(error) })
     }
 
     return {
@@ -1172,7 +1184,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
     try {
       await fs.access(path.join(this.projectPath, filename))
       return true
-    } catch {
+    } catch (error) {
+      log.debug('File not found', { filename, error: getErrorMessage(error) })
       return false
     }
   }
@@ -1183,7 +1196,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
       const pkgPath = path.join(__dirname, '..', '..', 'package.json')
       const pkg = JSON.parse(await fs.readFile(pkgPath, 'utf-8'))
       return pkg.version || '0.0.0'
-    } catch {
+    } catch (error) {
+      log.debug('Failed to read CLI version', { error: getErrorMessage(error) })
       return '0.0.0'
     }
   }

package/core/types/server.ts

@@ -3,7 +3,7 @@
  * Types for HTTP server and SSE modules.
  */
 
-import type { Hono } from 'hono'
+import type { Context, Hono } from 'hono'
 
 // =============================================================================
 // Server Types
@@ -36,8 +36,7 @@ export interface SSEClient {
 }
 
 export interface SSEManager {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  handleConnection: (c: any) => Response
+  handleConnection: (c: Context) => Response
   broadcast: (event: string, data: unknown) => void
   getClientCount: () => number
 }