@stacksjs/error-handling 0.59.11 → 0.61.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stacksjs/error-handling",
   "type": "module",
-  "version": "0.59.11",
+  "version": "0.61.1",
   "description": "Type safe error handling.",
   "author": "Chris Breuer",
   "license": "MIT",
@@ -55,7 +55,7 @@
     "@stacksjs/cli": "latest",
     "@stacksjs/path": "latest",
     "@stacksjs/types": "latest",
-    "neverthrow": "^6.1.0"
+    "neverthrow": "^6.2.1"
   },
   "devDependencies": {
     "@stacksjs/development": "latest"

package/src/handler.ts

@@ -1,4 +1,5 @@
-import { dirname, logsPath } from '@stacksjs/path'
+import * as path from '@stacksjs/path'
+import { ExitCode } from '@stacksjs/types'
 import fs from 'fs-extra'
 
 interface ErrorOptions {
@@ -8,50 +9,57 @@ interface ErrorOptions {
 export const StacksError = Error
 
 export class ErrorHandler {
-  static logFile = logsPath('errors.log')
+  // static logFile = path.logsPath('errors.log')
 
-  static handle(err: ErrorDescription | Error, options?: ErrorOptions | Error) {
+  static handle(err: ErrorDescription | Error | unknown, options?: ErrorOptions): Error {
     // let's only write to the console if we are not in silent mode
-    if (!(options instanceof Error) && options?.silent !== false)
-      this.writeErrorToConsole(err, options)
+    if (options?.silent !== false) this.writeErrorToConsole(err)
 
-    if (typeof err === 'string')
-      err = new StacksError(err)
+    if (typeof err === 'string') err = new StacksError(err)
 
-    this.writeErrorToFile(err).catch(e => console.error(e))
+    this.writeErrorToFile(err).catch((e) => console.error(e))
 
-    return err
+    return err as Error // TODO: improve this type
   }
 
-  static handleError(err: Error, options?: ErrorOptions) {
+  static handleError(err: Error, options?: ErrorOptions): Error {
     this.handle(err, options)
     return err
   }
 
-  static async writeErrorToFile(err: Error) {
+  static async writeErrorToFile(err: Error | unknown) {
+    if (!(err instanceof Error)) {
+      console.error('Error is not an instance of Error:', err)
+      return
+    }
+
     const formattedError = `[${new Date().toISOString()}] ${err.name}: ${err.message}\n`
-    const errorsLogFilePath = logsPath('errors.log')
+    const errorsLogFilePath = path.logsPath('errors.log')
 
     try {
       // Ensure the directory exists
-      await fs.mkdir(dirname(errorsLogFilePath), { recursive: true })
+      await fs.mkdir(path.dirname(errorsLogFilePath), { recursive: true })
       // Append the message to the log file
       await fs.appendFile(errorsLogFilePath, formattedError)
-    }
-    catch (error) {
+    } catch (error) {
       console.error('Failed to write to error file:', error)
     }
   }
 
-  static writeErrorToConsole(err: string | Error, options?: ErrorOptions) {
-    if (options)
-      console.error(err, options)
-    else
+  static writeErrorToConsole(err: string | Error | unknown): void {
+    if (
+      err === 'Failed to execute command: bunx biome check --apply .' ||
+      err === 'Failed to execute command: bun --bun storage/framework/core/actions/src/lint/fix.ts'
+    )
+      // To trigger this, run `buddy release` with a lint error in your codebase
       console.error(err)
+    process.exit(ExitCode.FatalError) // TODO: abstract this by differently catching the error somewhere
+    console.error(err)
   }
 }
 
 type ErrorDescription = string
-export function handleError(err: ErrorDescription | Error, options?: ErrorOptions | Error): Error {
+
+export function handleError(err: ErrorDescription | Error | unknown, options?: ErrorOptions): Error {
   return ErrorHandler.handle(err, options)
 }

package/src/index.ts

@@ -1,2 +1,14 @@
 export * from './handler'
-export { err, errAsync, fromPromise, fromSafePromise, fromThrowable, ok, okAsync, Err, Ok, Result, ResultAsync } from 'neverthrow'
+export {
+  err,
+  errAsync,
+  fromPromise,
+  fromSafePromise,
+  fromThrowable,
+  ok,
+  okAsync,
+  Err,
+  Ok,
+  Result,
+  ResultAsync,
+} from 'neverthrow'

package/dist/handler.d.ts

@@ -1,14 +0,0 @@
-interface ErrorOptions {
-    silent?: boolean;
-}
-export declare const StacksError: ErrorConstructor;
-export declare class ErrorHandler {
-    static logFile: any;
-    static handle(err: ErrorDescription | Error, options?: ErrorOptions | Error): Error;
-    static handleError(err: Error, options?: ErrorOptions): Error;
-    static writeErrorToFile(err: Error): Promise<void>;
-    static writeErrorToConsole(err: string | Error, options?: ErrorOptions): void;
-}
-type ErrorDescription = string;
-export declare function handleError(err: ErrorDescription | Error, options?: ErrorOptions | Error): Error;
-export {};

package/dist/index.d.ts

@@ -1,2 +0,0 @@
-export * from './handler';
-export { err, errAsync, fromPromise, fromSafePromise, fromThrowable, ok, okAsync, Err, Ok, Result, ResultAsync } from 'neverthrow';