Fix superfluous HttpError: Not Found by throwing user-friendly errors

Instead of logging a detailed error with core.error() and then re-throwing
the raw HTTP error (which gets logged again by core.setFailed() in index.js),
throw a new Error with the user-friendly message directly. This eliminates
the duplicate/superfluous raw error output like "HttpError: Not Found".

Fixes #125

Agent-Logs-Url: https://github.com/actions/configure-pages/sessions/ee6b7f39-cfc5-415b-b70c-826c2b8cfcfe

Co-authored-by: yoannchaudet <[email protected]>

.github/workflows/draft-release.yml

@@ -12,6 +12,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: release-drafter/release-drafter@v6
+      - uses: release-drafter/release-drafter@3f0f87098bd6b5c5b9a36d49c41d998ea58f9348
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/publish-immutable-actions.yml

@@ -0,0 +1,20 @@
+name: 'Publish Immutable Action Version'
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
+      packages: write
+
+    steps:
+      - name: Checking out
+        uses: actions/checkout@v4
+      - name: Publish
+        id: publish
+        uses: actions/[email protected]

.node-version

@@ -1 +1 @@
-20.10.0
+24.4.0

action.yml

@@ -2,7 +2,7 @@ name: 'Configure GitHub Pages'
 description: 'A GitHub Action to enable Pages, extract various metadata about a site, and configure some supported static site generators.'
 author: 'GitHub'
 runs:
-  using: 'node20'
+  using: 'node24'
   main: 'dist/index.js'
 inputs:
   static_site_generator:

dist/index.js

@@ -36419,11 +36419,9 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
     pageObject = await getPagesSite({ githubToken })
   } 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: ${error.message}`,
-        convertErrorToAnnotationProperties(error)
+      throw new 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.`
       )
-      throw error
     }
     core.warning(`Get Pages site failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
   }
@@ -36433,8 +36431,7 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
     try {
       pageObject = await enablePagesSite({ githubToken })
     } catch (error) {
-      core.error(`Create Pages site failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
-      throw error
+      throw new Error(`Create Pages site failed. Error: ${error.message}`)
     }
 
     // This somehow implies that the Pages site was already created but initially failed to be retrieved.
@@ -36443,8 +36440,7 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
       try {
         pageObject = await getPagesSite({ githubToken })
       } catch (error) {
-        core.error(`Get Pages site still failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
-        throw error
+        throw new Error(`Get Pages site still failed. Error: ${error.message}`)
       }
     }
   }

src/api-client.js

@@ -43,11 +43,9 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
     pageObject = await getPagesSite({ githubToken })
   } 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: ${error.message}`,
-        convertErrorToAnnotationProperties(error)
+      throw new 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.`
       )
-      throw error
     }
     core.warning(`Get Pages site failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
   }
@@ -57,8 +55,7 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
     try {
       pageObject = await enablePagesSite({ githubToken })
     } catch (error) {
-      core.error(`Create Pages site failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
-      throw error
+      throw new Error(`Create Pages site failed. Error: ${error.message}`)
     }
 
     // This somehow implies that the Pages site was already created but initially failed to be retrieved.
@@ -67,8 +64,7 @@ async function findOrCreatePagesSite({ githubToken, enablement = true }) {
       try {
         pageObject = await getPagesSite({ githubToken })
       } catch (error) {
-        core.error(`Get Pages site still failed. Error: ${error.message}`, convertErrorToAnnotationProperties(error))
-        throw error
+        throw new Error(`Get Pages site still failed. Error: ${error.message}`)
       }
     }
   }

src/api-client.test.js

@@ -169,8 +169,7 @@ describe('apiClient', () => {
         })
       } catch (error) {
         erred = true
-        // re-raised error
-        expect(error.response.status).toEqual(404)
+        expect(error.message).toContain('Get Pages site failed')
       }
       expect(erred).toBe(true)
       expect(mockGetPages).toHaveBeenCalledTimes(1)
@@ -188,8 +187,7 @@ describe('apiClient', () => {
         })
       } catch (error) {
         erred = true
-        // re-raised error
-        expect(error.response.status).toEqual(500)
+        expect(error.message).toContain('Create Pages site failed')
       }
       expect(erred).toBe(true)
       expect(mockGetPages).toHaveBeenCalledTimes(1)