quietlight/skraak.app - Change ST5WLNWFS4AOVPOJHMREC4VGXQMIDRDLBZ2M4FCE5FPKLPQTUTRQC

api error handling refactor

Created by AEj8dahVWy718uSSFPe9VSRJ5qX5G8pC2zvFzJJ8yzBd on June 5, 2025

ST5WLNWFS4AOVPOJHMREC4VGXQMIDRDLBZ2M4FCE5FPKLPQTUTRQC

Dependencies

In channels

main

Change contents

File addition: errorHandler.ts (----------)

[5.1]

/**
 * Standardized error handling utilities for API routes
 */
export interface ErrorResponse {
  error: string;
  details: string;
}
/**
 * Extracts a user-friendly error message from various error types
 */
export function getErrorMessage(error: unknown): string {
  if (error instanceof Error) {
    return error.message;
  }
  if (typeof error === 'string') {
    return error;
  }
  return 'An unexpected error occurred';
}
/**
 * Creates a standardized error response with logging
 */
export function standardErrorResponse(error: unknown, context: string): ErrorResponse {
  console.error(`${context}:`, error);
  return {
    error: getErrorMessage(error),
    details: error instanceof Error ? error.message : String(error)
  };
}
/**
 * Handles database constraint violations (e.g., duplicate keys)
 */
export function handleDatabaseError(error: unknown, context: string): ErrorResponse {
  console.error(`Database error in ${context}:`, error);
  
  const errorMessage = getErrorMessage(error);
  
  // Handle common database constraint violations
  if (errorMessage.includes('duplicate key') || errorMessage.includes('unique constraint')) {
    return {
      error: 'A record with these details already exists',
      details: 'Duplicate entry'
    };
  }
  
  if (errorMessage.includes('foreign key constraint')) {
    return {
      error: 'Referenced record does not exist',
      details: 'Foreign key violation'
    };
  }
  
  if (errorMessage.includes('not null constraint')) {
    return {
      error: 'Required field is missing',
      details: 'Missing required data'
    };
  }
  
  // Default database error response
  return {
    error: 'Database operation failed',
    details: errorMessage
  };
}
/**
 * Handles authentication and authorization errors
 */
export function handleAuthError(context: string): ErrorResponse {
  console.error(`Authentication failed in ${context}`);
  return {
    error: 'Authentication required',
    details: 'Please log in to access this resource'
  };
}
/**
 * Handles validation errors
 */
export function handleValidationError(message: string, context: string): ErrorResponse {
  console.error(`Validation error in ${context}: ${message}`);
  return {
    error: 'Invalid request data',
    details: message
  };
}
/**
 * Handles permission errors
 */
export function handlePermissionError(action: string, context: string): ErrorResponse {
  console.error(`Permission denied for ${action} in ${context}`);
  return {
    error: 'Insufficient permissions',
    details: `You don't have permission to ${action}`
  };
}

Insertion in src/worker/routes/files.ts at line 19 [5.16112]

[6.21255]

[7.7432]

import { standardErrorResponse, handleAuthError } from "../utils/errorHandler";

Replacement in src/worker/routes/files.ts at line 368 [5.16112]

B:BD[6.32563] → [6.32563:32780]

    console.error("Error fetching files:", error);
    return c.json(
      {
        error: "Failed to fetch files",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[6.32563]

[6.32780]

    return c.json(standardErrorResponse(error, "fetching files"), 500);

Deletion in src/worker/routes/files.ts at line 484 [5.16112]
B:BD[7.10486] → [7.10486:10544]
```
    console.error("Error downloading file:", error);
    
```

Replacement in src/worker/routes/files.ts at line 485 [5.16112]

B:BD[7.10579] → [7.10579:10995]

    if (error instanceof Error) {
      if (error.message.includes('File not found')) {
        return c.json({
          error: "File not found in storage",
          details: error.message
        }, 404);
      }
      
      if (error.message.includes('authentication failed')) {
        return c.json({
          error: "Storage authentication failed",
          details: error.message
        }, 500);
      }

[7.10579]

[7.10995]

    const errorMessage = error instanceof Error ? error.message : String(error);
    
    if (errorMessage.includes('File not found')) {
      return c.json(standardErrorResponse(error, "file not found in storage"), 404);
    }
    
    if (errorMessage.includes('authentication failed')) {
      return c.json(handleAuthError("storage authentication"), 500);

Replacement in src/worker/routes/files.ts at line 495 [5.16112]

B:BD[7.11006] → [7.11006:11150]

    return c.json({
      error: "Failed to download file",
      details: error instanceof Error ? error.message : String(error),
    }, 500);

[7.11006]

[7.11150]

    return c.json(standardErrorResponse(error, "downloading file"), 500);

Replacement in src/worker/routes/files.ts at line 677 [5.16112]

B:BD[8.4983] → [8.4983:5196]

    console.error("Error fetching file selections:", error);
    return c.json({
      error: "Failed to fetch file selections",
      details: error instanceof Error ? error.message : String(error),
    }, 500);

[8.4983]

[8.5196]

    return c.json(standardErrorResponse(error, "fetching file selections"), 500);

Replacement in src/worker/routes/files.ts at line 755 [5.16112]

B:BD[2.2140] → [2.2140:2347]

    console.error("Error fetching file context:", error);
    return c.json({
      error: "Failed to fetch file context",
      details: error instanceof Error ? error.message : String(error),
    }, 500);

[2.2140]

[2.2347]

    return c.json(standardErrorResponse(error, "fetching file context"), 500);

Insertion in src/worker/routes/datasets.ts at line 10 [5.16845]

[5.17128]

import { standardErrorResponse, handleDatabaseError } from "../utils/errorHandler";

Replacement in src/worker/routes/datasets.ts at line 180 [5.16845]

B:BD[5.21240] → [5.21240:21463]

    console.error("Error fetching datasets:", error);
    return c.json(
      {
        error: "Failed to fetch datasets",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[5.21240]

[5.21463]

    return c.json(standardErrorResponse(error, "fetching datasets"), 500);

Replacement in src/worker/routes/datasets.ts at line 304 [5.16845]

B:BD[5.25124] → [5.25124:25394]

    console.error("Error creating dataset:", error);
    
    // Handle unique constraint violations
    if (error instanceof Error && error.message.includes('duplicate key')) {
      return c.json({
        error: "A dataset with this ID already exists"
      }, 400);

[5.25124]

[5.25394]

    // Handle database constraint violations
    const errorMessage = error instanceof Error ? error.message : String(error);
    if (errorMessage.includes('duplicate key')) {
      return c.json(handleDatabaseError(error, "creating dataset"), 400);

Replacement in src/worker/routes/datasets.ts at line 309 [5.16845]

B:BD[5.25400] → [5.25400:25570]


    return c.json(
      {
        error: "Failed to create dataset",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[5.25400]

[5.25570]

    
    return c.json(standardErrorResponse(error, "creating dataset"), 500);

Replacement in src/worker/routes/datasets.ts at line 467 [5.16845]

B:BD[5.29886] → [5.29886:30113]

    console.error("Error updating dataset:", error);
    
    return c.json(
      {
        error: "Failed to update dataset",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[5.29886]

[5.30113]

    return c.json(standardErrorResponse(error, "updating dataset"), 500);

Replacement in src/worker/routes/datasets.ts at line 572 [5.16845]

B:BD[9.25272] → [9.25272:25493]

    console.error("Error fetching dataset:", error);
    return c.json(
      {
        error: "Failed to fetch dataset",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[9.25272]

[9.25493]

    return c.json(standardErrorResponse(error, "fetching dataset"), 500);

Insertion in src/worker/routes/clusters.ts at line 11 [5.30184]

[5.30428]

[3.4298]

import { standardErrorResponse, handleDatabaseError } from "../utils/errorHandler";

Replacement in src/worker/routes/clusters.ts at line 107 [5.30184]

B:BD[5.32425] → [5.32425:32648]

    console.error("Error fetching clusters:", error);
    return c.json(
      {
        error: "Failed to fetch clusters",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[5.32425]

[5.32648]

    return c.json(standardErrorResponse(error, "fetching clusters"), 500);

Replacement in src/worker/routes/clusters.ts at line 297 [5.30184]

B:BD[4.12184] → [4.12184:12454]

    console.error("Error creating cluster:", error);
    
    // Handle unique constraint violations
    if (error instanceof Error && error.message.includes('duplicate key')) {
      return c.json({
        error: "A cluster with this ID already exists"
      }, 400);

[4.12184]

[4.12454]

    // Handle database constraint violations
    const errorMessage = error instanceof Error ? error.message : String(error);
    if (errorMessage.includes('duplicate key')) {
      return c.json(handleDatabaseError(error, "creating cluster"), 400);

Replacement in src/worker/routes/clusters.ts at line 302 [5.30184]

B:BD[4.12460] → [4.12460:12630]


    return c.json(
      {
        error: "Failed to create cluster",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[4.12460]

[4.12630]

    
    return c.json(standardErrorResponse(error, "creating cluster"), 500);

Replacement in src/worker/routes/clusters.ts at line 574 [5.30184]

B:BD[4.21167] → [4.21167:21394]

    console.error("Error updating cluster:", error);
    
    return c.json(
      {
        error: "Failed to update cluster",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[4.21167]

[4.21394]

    return c.json(standardErrorResponse(error, "updating cluster"), 500);

Replacement in src/worker/routes/clusters.ts at line 655 [5.30184]

B:BD[4.23727] → [4.23727:23954]

    console.error("Error deleting cluster:", error);
    
    return c.json(
      {
        error: "Failed to delete cluster",
        details: error instanceof Error ? error.message : String(error),
      },
      500
    );

[4.23727]

[4.23954]

    return c.json(standardErrorResponse(error, "deleting cluster"), 500);