Improve error handling

SmartsquareGmbH · Feb 26, 2024 · 3ed92c8 · 3ed92c8
1 parent f7844fb
commit 3ed92c8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -10,4 +10,10 @@ async function run() {
   await executeAction(input, decideQueryStrategy(input), decideDeleteStrategy(input))
 }
 
-run().catch((error: string | Error) => setFailed(error))
+function formatError(error: Error): string {
+  const cause = error.cause instanceof Error ? "\nCaused by: " + formatError(error.cause) : ""
+
+  return error.toString() + cause
+}
+
+run().catch((error: string | Error) => setFailed(error instanceof Error ? formatError(error) : error))
diff --git a/src/query/strategies/organization.query.strategy.ts b/src/query/strategies/organization.query.strategy.ts
@@ -26,7 +26,7 @@ export default class OrganizationQueryStrategy implements QueryStrategy {
 
       return await this.octokit.rest.packages.getAllPackageVersionsForPackageOwnedByOrg(params)
     } catch (error) {
-      throw new Error(`Failed to query package ${name}`, { cause: error })
+      throw new Error(`Failed to query package ${name} of type ${input.type}`, { cause: error })
     }
   }
 }
diff --git a/src/query/strategies/user.query.strategy.ts b/src/query/strategies/user.query.strategy.ts
@@ -26,7 +26,7 @@ export default class UserQueryStrategy implements QueryStrategy {
 
       return await this.octokit.rest.packages.getAllPackageVersionsForPackageOwnedByUser(params)
     } catch (error) {
-      throw new Error(`Failed to query package ${name}`, { cause: error })
+      throw new Error(`Failed to query package ${name} of type ${input.type}`, { cause: error })
     }
   }
 }