Convert errors into Actions-compatible logging with annotations

package.json

@@ -26,6 +26,7 @@
   "dependencies": {
     "@actions/core": "^1.10.1",
     "@actions/github": "^6.0.0",
+    "error-stack-parser": "^2.1.4",
     "espree": "^9.6.1"
   },
   "devDependencies": {

src/api-client.js

@@ -1,5 +1,6 @@
 const core = require('@actions/core')
 const github = require('@actions/github')
+const { convertErrorToAnnotationProperties } = require('./error-utils')
 
 async function enablePagesSite({ githubToken }) {
   const octokit = github.getOctokit(githubToken)
@@ -43,20 +44,25 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
   } catch (error) {
     if (!enablement) {
       core.error(
-        'Get Pages site failed. Please verify that the repository has Pages enabled and configured to build using GitHub Actions, or consider exploring the `enablement` parameter for this action.',
-        error
+        `Get Pages site failed. Please verify that the repository has Pages enabled and configured to build using GitHub Actions, or consider exploring the \`enablement\` parameter for this action. Error: ${error.message}`,
+        convertErrorToAnnotationProperties(error)
       )
       throw error
     }
-    core.warning('Get Pages site failed', error)
-  }
+    core.warning(
+      `Get Pages site failed. Error: ${error.message}`,
+      convertErrorToAnnotationProperties(error)
+    )
 
   if (!pageObject && enablement) {
     // Create a new Pages site if one doesn't exist
     try {
       pageObject = await enablePagesSite({ githubToken })
     } catch (error) {
-      core.error('Create Pages site failed', error)
+      core.error(
+        `Create Pages site failed. Error: ${error.message}`,
+        convertErrorToAnnotationProperties(error)
+      )
       throw error
     }
 
@@ -66,7 +72,10 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
       try {
         pageObject = await getPagesSite({ githubToken })
       } catch (error) {
-        core.error('Get Pages site still failed', error)
+        core.error(
+          `Get Pages site still failed. Error: ${error.message}`,
+          convertErrorToAnnotationProperties(error)
+        )
         throw error
       }
     }

src/error-utils.js

@@ -0,0 +1,24 @@
+const ErrorStackParser = require('error-stack-parser')
+
+// Convert an Error's stack into `@actions/core` toolkit AnnotationProperties:
+// https://github.com/actions/toolkit/blob/ef77c9d60bdb03700d7758b0d04b88446e72a896/packages/core/src/core.ts#L36-L71
+function convertErrorToAnnotationProperties(error, title = error.name) {
+  if (!(error instanceof Error)) {
+    throw new TypeError('error must be an instance of Error')
+  }
+
+  const stack = ErrorStackParser.parse(error)
+  const firstFrame = stack && stack.length > 0 ? stack[0] : null
+  if (!firstFrame) {
+    throw new Error('Error stack is empty or unparseable')
+  }
+
+  return {
+    title,
+    file: firstFrame.fileName,
+    startLine: firstFrame.lineNumber,
+    startColumn: firstFrame.columnNumber
+  }
+}
+
+module.exports = { convertErrorToAnnotationProperties }

src/error-utils.test.js

@@ -0,0 +1,38 @@
+const { convertErrorToAnnotationProperties } = require('./error-utils')
+
+describe('error-utils', () => {
+  describe('convertErrorToAnnotationProperties', () => {
+    it('throws a TypeError if the first argument is not an Error instance', () => {
+      expect(() => convertErrorToAnnotationProperties('not an Error')).toThrow(
+        TypeError,
+        'error must be an instance of Error'
+      )
+    })
+
+    it('throws an Error if the first argument is an Error instance without a parseable stack', () => {
+      const error = new Error('Test error')
+      error.stack = ''
+      expect(() => convertErrorToAnnotationProperties(error)).toThrow(Error, 'Error stack is empty or unparseable')
+    })
+
+    it('returns an AnnotationProperties-compatible object', () => {
+      const result = convertErrorToAnnotationProperties(new TypeError('Test error'))
+      expect(result).toEqual({
+        title: 'TypeError',
+        file: __filename,
+        startLine: expect.any(Number),
+        startColumn: expect.any(Number)
+      })
+    })
+
+    it('returns an AnnotationProperties-compatible object with a custom title', () => {
+      const result = convertErrorToAnnotationProperties(new TypeError('Test error'), 'custom title')
+      expect(result).toEqual({
+        title: 'custom title',
+        file: __filename,
+        startLine: expect.any(Number),
+        startColumn: expect.any(Number)
+      })
+    })
+  })
+})

src/set-pages-config.js

@@ -1,6 +1,7 @@
 const core = require('@actions/core')
 const { ConfigParser } = require('./config-parser')
 const removeTrailingSlash = require('./remove-trailing-slash')
+const { convertErrorToAnnotationProperties } = require('./error-utils')
 
 const SUPPORTED_FILE_EXTENSIONS = ['.js', '.cjs', '.mjs']
 
@@ -88,13 +89,13 @@ function setPagesConfig({ staticSiteGenerator, generatorConfigFile, siteUrl }) {
       core.warning(
         `Unsupported configuration file extension. Currently supported extensions: ${SUPPORTED_FILE_EXTENSIONS.map(
           ext => JSON.stringify(ext)
-        ).join(', ')}`,
-        error
+        ).join(', ')}. Error: ${error.message}`,
+        convertErrorToAnnotationProperties(error)
       )
     } else {
       core.warning(
-        `We were unable to determine how to inject the site metadata into your config. Generated URLs may be incorrect. The base URL for this site should be ${siteUrl}. Please ensure your framework is configured to generate relative links appropriately.`,
-        error
+        `We were unable to determine how to inject the site metadata into your config. Generated URLs may be incorrect. The base URL for this site should be ${siteUrl}. Please ensure your framework is configured to generate relative links appropriately. Error: ${error.message}`,
+        convertErrorToAnnotationProperties(error)
       )
     }
   }