fix: improve logging and remove try/catch block that prevents error from bubbling up

Author: vmasek

dist/index.js

@@ -21,100 +21,100 @@ export async function run(
   getOctokit = github.getOctokit,
   git = simpleGit()
 ): Promise<void> {
-  try {
-    const inputs = parseInputs()
-    const options = createOptions(inputs)
+  const inputs = parseInputs()
+  const options = createOptions(inputs)
 
-    const [nodeMajorString] = (
-      process.version.startsWith('v')
-        ? process.version.slice(1)
-        : process.version
-    ).split('.')
-    const majorVersion = parseInt(nodeMajorString, 10)
-    const isUnsupportedVersion = majorVersion < 20
+  // https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/troubleshooting-workflows/enabling-debug-logging
+  if (
+    process.env['ACTIONS_RUNNER_DEBUG'] === 'true' ||
+    process.env['ACTIONS_STEP_DEBUG'] === 'true'
+  ) {
+    options.debug = true
+  }
 
-    if (isUnsupportedVersion) {
-      core.warning(
-        `${LOG_PREFIX} Internal runner is using unsupported NodeJS version ${process.version}`
-      )
-    } else if (options.debug) {
-      core.info(
-        `${LOG_PREFIX} Internal runner is using NodeJS version ${process.version}`
-      )
-    }
+  if (options.debug) {
+    core.info(
+      `${LOG_PREFIX} Debug in actions in enabled, setting CP_VERBOSE env variable to true`
+    )
+    process.env['CP_VERBOSE'] = 'true'
+  }
 
-    const refs = parseGitRefs()
-    const api = new GitHubApiClient(inputs.token, refs, artifact, getOctokit)
+  const [nodeMajorString] = (
+    process.version.startsWith('v') ? process.version.slice(1) : process.version
+  ).split('.')
+  const majorVersion = parseInt(nodeMajorString, 10)
+  const isUnsupportedVersion = majorVersion < 20
 
-    const result = await runInCI(refs, api, options, git)
+  if (isUnsupportedVersion) {
+    core.warning(
+      `${LOG_PREFIX} Internal runner is using unsupported NodeJS version ${process.version}`
+    )
+  } else if (options.debug) {
+    core.info(
+      `${LOG_PREFIX} Internal runner is using NodeJS version ${process.version}`
+    )
+  }
 
-    if (result.commentId != null) {
-      core.setOutput('comment-id', result.commentId)
-      core.info(`Commented on PR #${github.context.issue.number}`)
-    }
+  const refs = parseGitRefs()
+  const api = new GitHubApiClient(inputs.token, refs, artifact, getOctokit)
 
-    const diffFiles =
-      result.mode === 'standalone'
-        ? Object.values(result.files.diff ?? {})
-        : result.diffPath
-          ? [result.diffPath]
-          : []
-    if (diffFiles.length > 0) {
-      await uploadArtifact(
-        artifact,
-        createDiffArtifactName(),
-        diffFiles,
-        inputs
-      )
-    }
+  const result = await runInCI(refs, api, options, git)
 
-    if (result.mode === 'standalone') {
-      const id = await uploadArtifact(
+  if (result.commentId != null) {
+    core.setOutput('comment-id', result.commentId)
+    core.info(`Commented on PR #${github.context.issue.number}`)
+  }
+
+  const diffFiles =
+    result.mode === 'standalone'
+      ? Object.values(result.files.diff ?? {})
+      : result.diffPath
+        ? [result.diffPath]
+        : []
+
+  if (diffFiles.length > 0) {
+    await uploadArtifact(artifact, createDiffArtifactName(), diffFiles, inputs)
+  }
+
+  if (result.mode === 'standalone') {
+    const id = await uploadArtifact(
+      artifact,
+      createReportArtifactName(),
+      Object.values(result.files.report),
+      inputs
+    )
+    core.setOutput('artifact-id', id)
+  } else {
+    for (const project of result.projects) {
+      await uploadArtifact(
         artifact,
-        createReportArtifactName(),
-        Object.values(result.files.report),
+        createReportArtifactName(project.name),
+        Object.values(project.files.report),
         inputs
       )
-      core.setOutput('artifact-id', id)
-    } else {
-      for (const project of result.projects) {
+      if (project.files.diff) {
         await uploadArtifact(
           artifact,
-          createReportArtifactName(project.name),
-          Object.values(project.files.report),
+          createDiffArtifactName(project.name),
+          Object.values(project.files.diff),
           inputs
         )
-        if (project.files.diff) {
-          await uploadArtifact(
-            artifact,
-            createDiffArtifactName(project.name),
-            Object.values(project.files.diff),
-            inputs
-          )
-        }
       }
     }
+  }
 
-    if (inputs.annotations) {
-      const issues =
-        result.mode === 'standalone'
-          ? (result.newIssues ?? [])
-          : result.projects.flatMap(project => project.newIssues ?? [])
-      if (issues.length > 0) {
-        core.info(
-          `Found ${issues.length} new issues, creating GitHub annotations`
-        )
-        createAnnotationsFromIssues(issues)
-      } else {
-        core.info('No new issues found, skipping GitHub annotations')
-      }
+  if (inputs.annotations) {
+    const issues =
+      result.mode === 'standalone'
+        ? (result.newIssues ?? [])
+        : result.projects.flatMap(project => project.newIssues ?? [])
+    if (issues.length > 0) {
+      core.info(
+        `Found ${issues.length} new issues, creating GitHub annotations`
+      )
+      createAnnotationsFromIssues(issues)
+    } else {
+      core.info('No new issues found, skipping GitHub annotations')
     }
-  } catch (error) {
-    const errorMessage = error instanceof Error ? error.message : `${error}`
-    core.error(errorMessage)
-    core.setFailed(errorMessage)
-    return
   }
-
-  core.info(`${LOG_PREFIX} Finished running successfully`)
 }