chore: remove dead code and fix minor issues (API + infra)

apps/api/src/routes/chat.ts

@@ -15,6 +15,7 @@ import { Hono } from 'hono';
 import * as schema from '../db/schema';
 import type { Env } from '../env';
 import { log } from '../lib/logger';
+import { requireRouteParam } from '../lib/route-helpers';
 import { getUserId, requireApproved,requireAuth } from '../middleware/auth';
 import { errors } from '../middleware/error';
 import { requireOwnedProject } from '../middleware/project-auth';
@@ -25,17 +26,6 @@ import { isTaskStatus } from '../services/task-status';
 
 const chatRoutes = new Hono<{ Bindings: Env }>();
 
-function requireRouteParam(
-  c: { req: { param: (name: string) => string | undefined } },
-  name: string
-): string {
-  const value = c.req.param(name);
-  if (!value) {
-    throw errors.badRequest(`${name} is required`);
-  }
-  return value;
-}
-
 chatRoutes.use('/*', requireAuth(), requireApproved());
 
 /**

apps/api/src/routes/nodes.ts

@@ -278,7 +278,7 @@ nodesRoutes.post('/:id/stop', async (c) => {
       .where(inArray(schema.agentSessions.workspaceId, workspaceIds));
   }
 
-  return c.json({ status: 'deleted' });
+  return c.json({ status: 'stopped' });
 });
 
 nodesRoutes.delete('/:id', async (c) => {

apps/api/src/routes/tasks/_helpers.ts

@@ -248,14 +248,16 @@ export async function setTaskStatus(
         errorMessage: toStatus === 'failed' ? (options.errorMessage?.trim() || 'Task failed') : null,
       })
       .where(eq(schema.triggerExecutions.id, updatedTask.triggerExecutionId))
-      .catch((err) => {
+      .catch((err: unknown) => {
         // Best-effort — don't fail the task status update if execution sync fails
         // eslint-disable-next-line no-console
-        console.error('trigger_execution_sync_failed', {
+        console.error(JSON.stringify({
+          level: 'error',
+          event: 'trigger_execution_sync_failed',
           taskId: task.id,
           triggerExecutionId: updatedTask.triggerExecutionId,
           error: String(err),
-        });
+        }));
       });
   }
 

apps/web/src/pages/CreateWorkspace.tsx

@@ -196,7 +196,7 @@ export function CreateWorkspace() {
         setBranchesError('Could not fetch branches, showing common defaults');
       }
     } catch (err) {
-      console.log('Could not fetch branches:', err);
+      console.error('Could not fetch branches:', err);
       // Provide common branch names as fallback
       setBranches([{ name: 'main' }, { name: 'master' }, { name: 'develop' }]);
       setBranchesError('Unable to fetch branches. Common branch names provided.');

packages/providers/src/types.ts

@@ -179,4 +179,15 @@ export class ProviderError extends Error {
   ) {
     super(message, options);
   }
+
+  /** Make Error properties visible to JSON.stringify */
+  toJSON(): Record<string, unknown> {
+    return {
+      name: this.name,
+      message: this.message,
+      provider: this.providerName,
+      statusCode: this.statusCode,
+      cause: this.cause instanceof Error ? this.cause.message : this.cause,
+    };
+  }
 }

scripts/deploy/types.ts

@@ -125,18 +125,6 @@ export interface MigrationStatus {
   lastRun?: string;
 }
 
-export interface DeploymentState {
-  version: string;
-  environment: string;
-  timestamp: string;
-  status: DeploymentStatus;
-  resources: ProvisionedResources;
-  dnsRecords: DnsRecord[];
-  steps: DeploymentStep[];
-  secretsConfigured: string[];
-  migrations: MigrationStatus;
-}
-
 // ============================================================================
 // Preflight Check Types
 // ============================================================================
@@ -314,8 +302,6 @@ export interface CloudflareWorker {
 // Constants
 // ============================================================================
 
-export const DEPLOYMENT_STATE_VERSION = '1.0.0';
-
 // Note: Resource naming is centralized in config.ts (DEPLOYMENT_CONFIG.resources)
 // Do NOT add resource naming constants here - use DEPLOYMENT_CONFIG instead.