Merge pull request #48 from actions/do-not-enable-by-default

Update default behavior to NOT attempt to create/enable the Pages site

.eslintrc.json

@@ -0,0 +1,17 @@
+{
+  "env": {
+    "commonjs": true,
+    "es2021": true,
+    "node": true,
+    "jest": true
+  },
+  "plugins": ["github"],
+  "extends": ["eslint:recommended", "prettier", "plugin:github/internal"],
+  "parserOptions": {
+    "ecmaVersion": 12
+  },
+  "rules": {
+    "semi": ["error", "never"]
+  },
+  "ignorePatterns": ["/dist/", "/src/fixtures/", "/src/blank-configurations/"]
+}

.github/dependabot.yml

@@ -0,0 +1,6 @@
+version: 2
+updates:
+  - package-ecosystem: 'github-actions'
+    directory: '/'
+    schedule:
+      interval: 'weekly'

.github/release-drafter.yml

@@ -1,6 +1,6 @@
 ---
-name-template: "v$RESOLVED_VERSION"
-tag-template: "v$RESOLVED_VERSION"
+name-template: 'v$RESOLVED_VERSION'
+tag-template: 'v$RESOLVED_VERSION'
 template: |
   # Changelog
 
@@ -8,33 +8,33 @@ template: |
 
   See details of [all code changes](https://github.com/$OWNER/$REPOSITORY/compare/$PREVIOUS_TAG...v$RESOLVED_VERSION) since previous release.
 categories:
-  - title: "🚀 Features"
+  - title: '🚀 Features'
     labels:
-      - "feature"
-      - "enhancement"
-  - title: "🐛 Bug Fixes"
+      - 'feature'
+      - 'enhancement'
+  - title: '🐛 Bug Fixes'
     labels:
-      - "fix"
-      - "bugfix"
-      - "bug"
-  - title: "🧰 Maintenance"
+      - 'fix'
+      - 'bugfix'
+      - 'bug'
+  - title: '🧰 Maintenance'
     labels:
-      - "infrastructure"
-      - "automation"
-      - "documentation"
-  - title: "🏎 Performance"
-    label: "performance"
-change-template: "- $TITLE @$AUTHOR (#$NUMBER)"
+      - 'infrastructure'
+      - 'automation'
+      - 'documentation'
+  - title: '🏎 Performance'
+    label: 'performance'
+change-template: '- $TITLE @$AUTHOR (#$NUMBER)'
 version-resolver:
   major:
     labels:
-      - "type: breaking"
+      - 'type: breaking'
   minor:
     labels:
-      - "type: enhancement"
+      - 'type: enhancement'
   patch:
     labels:
-      - "type: bug"
-      - "type: maintenance"
-      - "type: documentation"
+      - 'type: bug'
+      - 'type: maintenance'
+      - 'type: documentation'
   default: patch

.github/workflows/check-dist.yml

@@ -0,0 +1,52 @@
+# When you reference this Action with `uses:` in a workflow,
+# `dist/index.js` is the code that will run.
+# For our project, we generate this file using `ncc`.
+# We need to make sure the checked-in `dist/index.js` actually matches what we expect it to be.
+name: Check dist/
+
+on:
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - '**.md'
+  pull_request:
+    paths-ignore:
+      - '**.md'
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
+jobs:
+  check-dist:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Rebuild the dist/ directory
+        run: npm run prepare
+
+      - name: Compare the expected and actual dist/ directories
+        run: |
+          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+            echo "Detected uncommitted changes after build in dist folder. See status below:"
+            git diff
+            exit 1
+          fi

.github/workflows/check-formatting.yml

@@ -0,0 +1,35 @@
+name: Check formatting
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+permissions:
+  contents: read
+
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    timeout-minutes: 2
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Verify formatting
+        run: npm run format:check

.github/workflows/draft-release.yml

@@ -4,11 +4,14 @@ on:
     branches:
       - main
 
+permissions:
+  contents: write
+
 jobs:
   draft-release:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - uses: release-drafter/release-drafter@v5
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/lint.yml

@@ -0,0 +1,35 @@
+name: Lint code
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+permissions:
+  contents: read
+
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    timeout-minutes: 2
+    steps:
+      - name: Check out repo
+        uses: actions/checkout@v3
+
+      - name: Setup Node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run linter
+        run: npm run lint:check

.github/workflows/release.yml

@@ -1,11 +1,11 @@
 name: Release
 on:
   release:
-    types: [published]
+    types: [released]
   workflow_dispatch:
     inputs:
       TAG_NAME:
-        description: "Tag name that the major tag will point to"
+        description: 'Tag name that the major tag will point to'
         required: true
 
 env:
@@ -24,7 +24,7 @@ jobs:
     steps:
       - name: Update the ${{ env.TAG_NAME }} tag
         id: update-major-tag
-        uses: actions/publish-action@v0.1.0
+        uses: actions/publish-action@v0.2.2
         with:
           source-tag: ${{ env.TAG_NAME }}
           slack-webhook: ${{ secrets.SLACK_WEBHOOK }}

.github/workflows/test.yml

@@ -6,20 +6,29 @@ on:
       - main
   pull_request:
 
+permissions:
+  contents: read
+
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
 jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-    - name: Checkout
-      uses: actions/checkout@v3
+      - name: Checkout
+        uses: actions/checkout@v3
 
-    - name: Set Node.JS
-      uses: actions/setup-node@v3
-      with:
-        node-version: 16.x
+      - name: Setup Node.JS
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: 'npm'
 
-    - name: Install dependencies
-      run: npm ci
-    
-    - name: Run tests
-      run: npm test
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run tests
+        run: npm test

.prettierignore

@@ -0,0 +1,5 @@
+# Ignore build artifacts
+/dist/
+
+# Ignore all Markdown files
+*.md

.prettierrc.yml

@@ -1,10 +1,10 @@
 # Prettier (formatter) configuration
 ---
-printWidth: 80
+printWidth: 120
 tabWidth: 2
 useTabs: false
 semi: false
 singleQuote: true
 trailingComma: none
-bracketSpacing: false
+bracketSpacing: true
 arrowParens: avoid

README.md

@@ -1,8 +1,8 @@
-# Configure-Pages
+# configure-pages
 
-An action to enable Pages and extract various metadata about a site. It can also be used to configure various static site generators we support as [starter workflows][starter-workflows].
+A GitHub Action to enable Pages and extract various metadata about a site. It can also be used to configure various static site generators we support as [starter workflows][starter-workflows].
 
-See [`set-pages-path.js`](./src/set-pages-path.js) for more details on how we configure static site generators to work "out of the box" with GitHub Pages.
+See [`set-pages-config.js`](src/set-pages-config.js) for more details on how we configure static site generators to work "out of the box" with GitHub Pages.
 
 # Usage
 
@@ -12,18 +12,13 @@ See [action.yml](action.yml) and the [Pages starter workflows][starter-workflows
 
 In order to release a new version of this Action:
 
-1. Locate the semantic version of the upcoming release (a draft is maintained by the [`draft-release` workflow][draft-release])
+1. Locate the semantic version of the [upcoming release][release-list] (a draft is maintained by the [`draft-release` workflow][draft-release]).
 
-2. Push a matching tag, for instance for `v0.1.0`:
+2. Publish the draft release from the `main` branch with semantic version as the tag name, _with_ the checkbox to publish to the GitHub Marketplace checked. :ballot_box_with_check:
 
-   ```bash
-   git tag v0.1.0
-   git push origin v0.1.0
-   ```
+3. After publishing the release, the [`release` workflow][release] will automatically run to create/update the corresponding the major version tag such as `v0`.
 
-3. Publish the draft release (the major tag such as `v0` will be created/updated by the [`release` workflow][release])
-
-   ⚠️ Environment approval is required.
+   ⚠️ Environment approval is required. Check the [Release workflow run list][release-workflow-runs].
 
 # License
 
@@ -31,5 +26,7 @@ The scripts and documentation in this project are released under the [MIT Licens
 
 <!-- references -->
 [starter-workflows]: https://github.com/actions/starter-workflows/tree/main/pages
+[release-list]: https://github.com/actions/configure-pages/releases
 [draft-release]: .github/workflows/draft-release.yml
-[release]: .github/workflows/release.yml
+[release]: .github/workflows/release.yml
+[release-workflow-runs]: https://github.com/actions/configure-pages/actions/workflows/release.yml

action.yml

@@ -1,22 +1,30 @@
-name: "Configure Pages"
-description: "An action to enable Pages and extract various metadata about a site. It can also be used to configure various static site generators we support as starter workflows."
+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: "node16"
-  main: "dist/index.js"
+  using: 'node16'
+  main: 'dist/index.js'
 inputs:
   static_site_generator:
-    description: "Optional static site generator to attempt to configure (nuxt, next or gatsby)"
+    description: 'Optional static site generator to attempt to configure: "nuxt", "next", "gatsby", or "sveltekit"'
+    required: false
+  generator_config_file:
+    description: 'Optional file path to static site generator configuration file'
     required: false
   token:
-    description: "GitHub token"
+    description: 'GitHub token'
     default: ${{ github.token }}
     required: true
+  enablement:
+    description: 'Should a Pages site be enabled for the repository if not so already? This will only work with user-to-server tokens.'
+    default: 'false'
+    required: false
 outputs:
   base_url:
-    description: 'GitHub Pages site full base URL. Examples: "https://octocat.github.io/my-repo/", "https://octocat.github.io/", "https://www.example.com/"'
+    description: 'GitHub Pages site full base URL. Examples: "https://octocat.github.io/my-repo", "https://octocat.github.io", "https://www.example.com"'
   origin:
     description: 'GitHub Pages site origin. Examples: "https://octocat.github.io", "https://www.example.com"'
   host:
     description: 'GitHub Pages site host. Examples: "octocat.github.io", "www.example.com"'
   base_path:
-    description: 'GitHub Pages site full base path. Examples: "/my-repo/" or "/"'
+    description: 'GitHub Pages site full base path. Examples: "/my-repo" or ""'

dist/licenses.txt

@@ -608,3 +608,16 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
+
+
+uuid
+MIT
+The MIT License (MIT)
+
+Copyright (c) 2010-2020 Robert Kieffer and other contributors
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

dist/sveltekit.js

@@ -0,0 +1,8 @@
+// Default Pages configuration for SvelteKit
+import adapter from '@sveltejs/adapter-auto'
+
+export default {
+  kit: {
+    adapter: adapter()
+  }
+}

package.json

@@ -1,31 +1,38 @@
 {
+  "private": true,
   "name": "configure-pages",
   "version": "1.0.0",
-  "description": "An action to enable Pages and extract various metadata about a site. It can also be used to configure various static site generators we support as starter workflows.",
-  "main": "src/index.js",
+  "description": "A GitHub Action to enable Pages and extract various metadata about a site. It can also be used to configure various static site generators we support as starter workflows.",
+  "main": "./dist/index.js",
   "scripts": {
+    "all": "npm run format && npm run lint && npm run prepare && npm run test",
+    "format": "prettier --write .",
+    "format:check": "prettier --check .",
+    "lint": "DEBUG=eslint:cli-engine eslint --fix .",
+    "lint:check": "DEBUG=eslint:cli-engine eslint .",
     "prepare": "ncc build src/index.js -o dist --source-map --license licenses.txt",
     "test": "jest"
   },
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/paper-spa/configure-pages.git"
+    "url": "git+https://github.com/actions/configure-pages.git"
   },
   "author": "GitHub",
   "license": "MIT",
   "bugs": {
-    "url": "https://github.com/paper-spa/configure-pages/issues"
+    "url": "https://github.com/actions/configure-pages/issues"
   },
-  "homepage": "https://github.com/paper-spa/configure-pages#readme",
+  "homepage": "https://github.com/actions/configure-pages#readme",
   "dependencies": {
-    "@actions/core": "^1.8.2",
+    "@actions/core": "^1.10.0",
     "axios": "^0.27.2",
-    "axios-retry": "^3.2.5",
-    "espree": "^9.3.2",
-    "string-format": "^1.0.0"
+    "espree": "^9.3.2"
   },
   "devDependencies": {
     "@vercel/ncc": "^0.34.0",
+    "eslint": "^8.23.1",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-plugin-github": "^4.3.7",
     "jest": "^28.1.1",
     "prettier": "^2.7.1"
   }

src/api-client.js

@@ -0,0 +1,87 @@
+const axios = require('axios')
+const core = require('@actions/core')
+
+function getApiBaseUrl() {
+  return process.env.GITHUB_API_URL || 'https://api.github.com'
+}
+
+async function enablePagesSite({ repositoryNwo, githubToken }) {
+  const pagesEndpoint = `${getApiBaseUrl()}/repos/${repositoryNwo}/pages`
+
+  try {
+    const response = await axios.post(
+      pagesEndpoint,
+      { build_type: 'workflow' },
+      {
+        headers: {
+          Accept: 'application/vnd.github.v3+json',
+          Authorization: `Bearer ${githubToken}`,
+          'Content-type': 'application/json'
+        }
+      }
+    )
+
+    const pageObject = response.data
+    return pageObject
+  } catch (error) {
+    if (error.response && error.response.status === 409) {
+      return null
+    }
+
+    throw error
+  }
+}
+
+async function getPagesSite({ repositoryNwo, githubToken }) {
+  const pagesEndpoint = `${getApiBaseUrl()}/repos/${repositoryNwo}/pages`
+
+  const response = await axios.get(pagesEndpoint, {
+    headers: {
+      Accept: 'application/vnd.github.v3+json',
+      Authorization: `Bearer ${githubToken}`
+    }
+  })
+
+  const pageObject = response.data
+  return pageObject
+}
+
+async function findOrCreatePagesSite({ repositoryNwo, githubToken, enablement = true }) {
+  let pageObject
+
+  // Try to find an existing Pages site first
+  try {
+    pageObject = await getPagesSite({ repositoryNwo, githubToken })
+  } catch (error) {
+    if (!enablement) {
+      core.error('Get Pages site failed', error)
+      throw error
+    }
+    core.warning('Get Pages site failed', error)
+  }
+
+  if (!pageObject && enablement) {
+    // Create a new Pages site if one doesn't exist
+    try {
+      pageObject = await enablePagesSite({ repositoryNwo, githubToken })
+    } catch (error) {
+      core.error('Create Pages site failed', error)
+      throw error
+    }
+
+    // This somehow implies that the Pages site was already created but initially failed to be retrieved.
+    // Try one more time for this extreme edge case!
+    if (pageObject == null) {
+      try {
+        pageObject = await getPagesSite({ repositoryNwo, githubToken })
+      } catch (error) {
+        core.error('Get Pages site still failed', error)
+        throw error
+      }
+    }
+  }
+
+  return pageObject
+}
+
+module.exports = { findOrCreatePagesSite, enablePagesSite, getPagesSite, getApiBaseUrl }

src/api-client.test.js

@@ -0,0 +1,215 @@
+const core = require('@actions/core')
+const axios = require('axios')
+
+const apiClient = require('./api-client')
+
+describe('apiClient', () => {
+  const GITHUB_REPOSITORY = 'actions/is-awesome'
+  const GITHUB_TOKEN = 'gha-token'
+  const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+
+  beforeEach(() => {
+    jest.restoreAllMocks()
+
+    // Mock error/warning/info/debug
+    jest.spyOn(core, 'error').mockImplementation(jest.fn())
+    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
+    jest.spyOn(core, 'info').mockImplementation(jest.fn())
+    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
+  })
+
+  describe('getApiBaseUrl', () => {
+    it('returns GITHUB_API_URL environment variable when set', async () => {
+      const expectedBaseUrl = 'https://api.ghe.com'
+      process.env.GITHUB_API_URL = expectedBaseUrl
+      const result = apiClient.getApiBaseUrl()
+      delete process.env.GITHUB_API_URL
+      expect(result).toEqual(expectedBaseUrl)
+    })
+
+    it('defaults to GitHub API if GITHUB_API_URL environment variable is empty', async () => {
+      process.env.GITHUB_API_URL = ''
+      const result = apiClient.getApiBaseUrl()
+      delete process.env.GITHUB_API_URL
+      expect(result).toEqual('https://api.github.com')
+    })
+
+    it('defaults to GitHub API if GITHUB_API_URL environment variable is not set', async () => {
+      delete process.env.GITHUB_API_URL
+      const result = apiClient.getApiBaseUrl()
+      expect(result).toEqual('https://api.github.com')
+    })
+  })
+
+  describe('enablePagesSite', () => {
+    it('makes a request to create a page', async () => {
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.resolve({ status: 201, data: PAGE_OBJECT }))
+
+      const result = await apiClient.enablePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+    })
+
+    it('handles a 409 response when the page already exists', async () => {
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 409 } }))
+
+      // Simply assert that no error is raised
+      const result = await apiClient.enablePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+
+      expect(result).toBe(null)
+    })
+
+    it('re-raises errors on failure status codes', async () => {
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+
+      let erred = false
+      try {
+        await apiClient.enablePagesSite({
+          repositoryNwo: GITHUB_REPOSITORY,
+          githubToken: GITHUB_TOKEN
+        })
+      } catch (error) {
+        erred = true
+        expect(error.response.status).toEqual(404)
+      }
+      expect(erred).toBe(true)
+    })
+  })
+
+  describe('getPagesSite', () => {
+    it('makes a request to get a page', async () => {
+      const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.resolve({ status: 200, data: PAGE_OBJECT }))
+
+      const result = await apiClient.getPagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+    })
+
+    it('re-raises errors on failure status codes', async () => {
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+
+      let erred = false
+      try {
+        await apiClient.getPagesSite({
+          repositoryNwo: GITHUB_REPOSITORY,
+          githubToken: GITHUB_TOKEN
+        })
+      } catch (error) {
+        erred = true
+        expect(error.response.status).toEqual(404)
+      }
+      expect(erred).toBe(true)
+    })
+  })
+
+  describe('findOrCreatePagesSite', () => {
+    it('does not make a request to create a page if it already exists', async () => {
+      const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.resolve({ status: 200, data: PAGE_OBJECT }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+
+      const result = await apiClient.findOrCreatePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+      expect(axios.get).toHaveBeenCalledTimes(1)
+      expect(axios.post).toHaveBeenCalledTimes(0)
+    })
+
+    it('makes request to create a page by default if it does not exist', async () => {
+      const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.resolve({ status: 201, data: PAGE_OBJECT }))
+
+      const result = await apiClient.findOrCreatePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+      expect(axios.get).toHaveBeenCalledTimes(1)
+      expect(axios.post).toHaveBeenCalledTimes(1)
+    })
+
+    it('makes a request to create a page when explicitly enabled if it does not exist', async () => {
+      const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.resolve({ status: 201, data: PAGE_OBJECT }))
+
+      const result = await apiClient.findOrCreatePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN,
+        enablement: true
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+      expect(axios.get).toHaveBeenCalledTimes(1)
+      expect(axios.post).toHaveBeenCalledTimes(1)
+    })
+
+    it('does not make a request to create a page when explicitly disabled even if it does not exist', async () => {
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 500 } })) // just so they both aren't 404
+
+      let erred = false
+      try {
+        await apiClient.findOrCreatePagesSite({
+          repositoryNwo: GITHUB_REPOSITORY,
+          githubToken: GITHUB_TOKEN,
+          enablement: false
+        })
+      } catch (error) {
+        erred = true
+        // re-raised error
+        expect(error.response.status).toEqual(404)
+      }
+      expect(erred).toBe(true)
+      expect(axios.get).toHaveBeenCalledTimes(1)
+      expect(axios.post).toHaveBeenCalledTimes(0)
+    })
+
+    it('does not make a second request to get page if create fails for reason other than existence', async () => {
+      jest.spyOn(axios, 'get').mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 500 } })) // just so they both aren't 404
+
+      let erred = false
+      try {
+        await apiClient.findOrCreatePagesSite({
+          repositoryNwo: GITHUB_REPOSITORY,
+          githubToken: GITHUB_TOKEN
+        })
+      } catch (error) {
+        erred = true
+        // re-raised error
+        expect(error.response.status).toEqual(500)
+      }
+      expect(erred).toBe(true)
+      expect(axios.get).toHaveBeenCalledTimes(1)
+      expect(axios.post).toHaveBeenCalledTimes(1)
+    })
+
+    it('makes second request to get page if create fails because of existence', async () => {
+      const PAGE_OBJECT = { html_url: 'https://actions.github.io/is-awesome/' }
+      jest
+        .spyOn(axios, 'get')
+        .mockImplementationOnce(() => Promise.reject({ response: { status: 404 } }))
+        .mockImplementationOnce(() => Promise.resolve({ status: 200, data: PAGE_OBJECT }))
+      jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.reject({ response: { status: 409 } }))
+
+      const result = await apiClient.findOrCreatePagesSite({
+        repositoryNwo: GITHUB_REPOSITORY,
+        githubToken: GITHUB_TOKEN
+      })
+      expect(result).toEqual(PAGE_OBJECT)
+      expect(axios.get).toHaveBeenCalledTimes(2)
+      expect(axios.post).toHaveBeenCalledTimes(1)
+    })
+  })
+})

src/blank-configurations/sveltekit.js

@@ -0,0 +1,8 @@
+// Default Pages configuration for SvelteKit
+import adapter from '@sveltejs/adapter-auto'
+
+export default {
+  kit: {
+    adapter: adapter()
+  }
+}

src/config-parser.js

@@ -9,7 +9,7 @@ Pages's path based routing (and work).
 
 Supported configuration initializations:
 
-(1) Default export:
+(1) Direct default export:
 
   export default {
     // configuration object here
@@ -21,19 +21,69 @@ Supported configuration initializations:
     // configuration object here
   }
 
-(3) Indirect module export:
+(3) Indirect default export:
 
-  const config = // configuration object here
+  const config = {
+    // configuration object here
+  }
+  export default config
+
+(4) Indirect module export:
+
+  const config = {
+    // configuration object here
+  }
   module.exports = config
+
+(5) Direct default export with wrapping call:
+
+  export default defineConfig({
+    // configuration object here
+  })
+
+(6) Direct module export with wrapping call:
+
+  module.exports = defineConfig({
+    // configuration object here
+  })
+
+(7) Indirect default export with wrapping call at the definition:
+
+  const config = defineConfig({
+    // configuration object here
+  })
+  export default config
+
+(8) Indirect default export with wrapping call at the export:
+
+  const config = {
+    // configuration object here
+  }
+  export default defineConfig(config)
+
+(9) Indirect module export with wrapping call at the definition:
+
+  const config = defineConfig({
+    // configuration object here
+  })
+  module.exports = config
+
+(10) Indirect module export with wrapping call at the export:
+
+  const config = {
+    // configuration object here
+  }
+  module.exports = defineConfig(config)
 */
 
 class ConfigParser {
   // Ctor
   // - configurationFile: path to the configuration file
   // - blankConfigurationFile: a blank configuration file to use if non was previously found
-  constructor({configurationFile, blankConfigurationFile, properties}) {
+  constructor({ configurationFile, blankConfigurationFile, allowWrappingCall = false, properties }) {
     // Save field
     this.configurationFile = configurationFile
+    this.allowWrappingCall = allowWrappingCall === true
     this.properties = properties
 
     // If the configuration file does not exist, initialize it with the blank configuration file
@@ -49,23 +99,110 @@ class ConfigParser {
     this.configuration = fs.readFileSync(this.configurationFile, 'utf8')
   }
 
+  findTopLevelVariableDeclarator(ast, identifierName) {
+    let targetDeclarator
+    ast.body.find(
+      node =>
+        node.type === 'VariableDeclaration' &&
+        node.declarations &&
+        node.declarations.length > 0 &&
+        node.declarations.find(declarator => {
+          if (
+            declarator.type === 'VariableDeclarator' &&
+            declarator.id &&
+            declarator.id.type === 'Identifier' &&
+            declarator.id.name === identifierName
+          ) {
+            targetDeclarator = declarator
+            return true
+          }
+        })
+    )
+    return targetDeclarator
+  }
+
   // Find the configuration object in an AST.
-  // Look for a default export, a direct module export or an indirect module
-  // export (in that order).
+  // Look for, in order:
+  //  - a direct default export
+  //  - a direct default export with a wrapping call
+  //  - an indirect default export
+  //  - an indirect default export with a wrapping call at the definition
+  //  - an indirect default export with a wrapping call at the export
+  //  - a direct module export
+  //  - a direct module export with a wrapping call
+  //  - an indirect module export
+  //  - an indirect module export with a wrapping call at the definition
+  //  - an indirect module export with a wrapping call at the export
   //
   // Return the configuration object or null.
-  findConfigurationObject(ast) {
+  findConfigurationObject(ast, allowWrappingCall = false) {
     // Try to find a default export
-    var defaultExport = ast.body.find(
-      node =>
-        node.type === 'ExportDefaultDeclaration' &&
-        node.declaration.type === 'ObjectExpression'
-    )
-    if (defaultExport) {
-      core.info('Found configuration object in default export declaration')
+    var defaultExport = ast.body.find(node => node.type === 'ExportDefaultDeclaration')
+
+    // Direct default export
+    if (defaultExport && defaultExport.declaration.type === 'ObjectExpression') {
+      core.info('Found configuration object in direct default export declaration')
       return defaultExport.declaration
     }
 
+    // Direct default export with a wrapping call
+    else if (
+      allowWrappingCall &&
+      defaultExport &&
+      defaultExport.declaration.type === 'CallExpression' &&
+      defaultExport.declaration.arguments.length > 0 &&
+      defaultExport.declaration.arguments[0] &&
+      defaultExport.declaration.arguments[0].type === 'ObjectExpression'
+    ) {
+      core.info('Found configuration object in direct default export declaration with a wrapping call')
+      return defaultExport.declaration.arguments[0]
+    }
+
+    // Indirect default export
+    else if (defaultExport && defaultExport.declaration.type === 'Identifier') {
+      const identifierName = defaultExport.declaration.name
+      const identifierDeclarator = this.findTopLevelVariableDeclarator(ast, identifierName)
+      const identifierInitialization = identifierDeclarator && identifierDeclarator.init
+      if (identifierInitialization && identifierInitialization.type === 'ObjectExpression') {
+        core.info('Found configuration object in indirect default export declaration')
+        return identifierInitialization
+      }
+      // Indirect default export with a wrapping call at the definition
+      else if (
+        allowWrappingCall &&
+        identifierInitialization &&
+        identifierInitialization.type === 'CallExpression' &&
+        identifierInitialization.arguments.length > 0 &&
+        identifierInitialization.arguments[0] &&
+        identifierInitialization.arguments[0].type === 'ObjectExpression'
+      ) {
+        core.info(
+          'Found configuration object in indirect default export declaration with a wrapping call at the definition'
+        )
+        return identifierInitialization.arguments[0]
+      }
+    }
+
+    // Indirect default export with a wrapping call at the export
+    else if (
+      allowWrappingCall &&
+      defaultExport &&
+      defaultExport.declaration.type === 'CallExpression' &&
+      defaultExport.declaration.arguments.length > 0 &&
+      defaultExport.declaration.arguments[0] &&
+      defaultExport.declaration.arguments[0].type === 'Identifier'
+    ) {
+      const identifierName = defaultExport.declaration.arguments[0].name
+      const identifierDeclarator = this.findTopLevelVariableDeclarator(ast, identifierName)
+      const identifierInitialization = identifierDeclarator && identifierDeclarator.init
+      if (identifierInitialization && identifierInitialization.type === 'ObjectExpression') {
+        core.info(
+          'Found configuration object in indirect default export declaration with a wrapping call at the export'
+        )
+        return identifierInitialization
+      }
+    }
+
     // Try to find a module export
     var moduleExport = ast.body.find(
       node =>
@@ -80,32 +217,62 @@ class ConfigParser {
     )
 
     // Direct module export
-    if (
-      moduleExport &&
-      moduleExport.expression.right.type === 'ObjectExpression'
-    ) {
+    if (moduleExport && moduleExport.expression.right.type === 'ObjectExpression') {
       core.info('Found configuration object in direct module export')
       return moduleExport.expression.right
     }
 
-    // Indirect module export
+    // Direct default export with a wrapping call
     else if (
+      allowWrappingCall &&
       moduleExport &&
-      moduleExport.expression.right.type === 'Identifier'
+      moduleExport.expression.right.type === 'CallExpression' &&
+      moduleExport.expression.right.arguments.length > 0 &&
+      moduleExport.expression.right.arguments[0] &&
+      moduleExport.expression.right.arguments[0].type === 'ObjectExpression'
     ) {
+      core.info('Found configuration object in direct module export with a wrapping call')
+      return moduleExport.expression.right.arguments[0]
+    }
+
+    // Indirect module export
+    else if (moduleExport && moduleExport.expression.right.type === 'Identifier') {
       const identifierName = moduleExport && moduleExport.expression.right.name
-      const identifierDefinition = ast.body.find(
-        node =>
-          node.type === 'VariableDeclaration' &&
-          node.declarations.length == 1 &&
-          node.declarations[0].type === 'VariableDeclarator' &&
-          node.declarations[0].id.type === 'Identifier' &&
-          node.declarations[0].id.name === identifierName &&
-          node.declarations[0].init.type === 'ObjectExpression'
-      )
-      if (identifierDefinition) {
+      const identifierDeclarator = this.findTopLevelVariableDeclarator(ast, identifierName)
+      const identifierInitialization = identifierDeclarator && identifierDeclarator.init
+      if (identifierInitialization && identifierInitialization.type === 'ObjectExpression') {
         core.info('Found configuration object in indirect module export')
-        return identifierDefinition.declarations[0].init
+        return identifierInitialization
+      }
+      // Indirect module export with a wrapping call at the definition
+      else if (
+        allowWrappingCall &&
+        identifierInitialization &&
+        identifierInitialization.type === 'CallExpression' &&
+        identifierInitialization.arguments.length > 0 &&
+        identifierInitialization.arguments[0] &&
+        identifierInitialization.arguments[0].type === 'ObjectExpression'
+      ) {
+        core.info('Found configuration object in indirect module export with a wrapping call at the definition')
+        return identifierInitialization.arguments[0]
+      }
+    }
+
+    // Indirect module export with a wrapping call at the export
+    else if (
+      allowWrappingCall &&
+      moduleExport &&
+      moduleExport.expression.right.type === 'CallExpression' &&
+      moduleExport.expression.right.arguments.length > 0 &&
+      moduleExport.expression.right.arguments[0] &&
+      moduleExport.expression.right.arguments[0].type === 'Identifier'
+    ) {
+      const identifierName = moduleExport.expression.right.arguments[0].name
+      const identifierDeclarator = this.findTopLevelVariableDeclarator(ast, identifierName)
+      const identifierInitialization = identifierDeclarator && identifierDeclarator.init
+      if (identifierInitialization && identifierInitialization.type === 'ObjectExpression') {
+        core.info('Found configuration object in indirect module export declaration with a wrapping call at the export')
+        return identifierInitialization
       }
     }
 
@@ -120,9 +287,7 @@ class ConfigParser {
     // Try to find a property matching a given name
     const property =
       object.type === 'ObjectExpression' &&
-      object.properties.find(
-        node => node.key.type === 'Identifier' && node.key.name === name
-      )
+      object.properties.find(node => node.key.type === 'Identifier' && node.key.name === name)
 
     // Return the property's value (if found) or null
     if (property) {
@@ -142,9 +307,7 @@ class ConfigParser {
       return `${properties[startIndex]}: ${JSON.stringify(propertyValue)}`
     } else {
       return (
-        `${properties[startIndex]}: {` +
-        this.getPropertyDeclaration(properties, startIndex + 1, propertyValue) +
-        '}'
+        `${properties[startIndex]}: {` + this.getPropertyDeclaration(properties, startIndex + 1, propertyValue) + '}'
       )
     }
   }
@@ -173,17 +336,18 @@ class ConfigParser {
     const ast = espree.parse(this.configuration, espreeOptions)
 
     // Find the configuration object
-    var configurationObject = this.findConfigurationObject(ast)
+    var configurationObject = this.findConfigurationObject(ast, this.allowWrappingCall)
     if (!configurationObject) {
       throw 'Could not find a configuration object in the configuration file'
     }
 
-    // A property may be nested in the configuration file. Split the property name with `.`
+    // A property may be nested in the configuration file. Split the property name with '.'
     // then walk the configuration object one property at a time.
     var depth = 0
     const properties = propertyName.split('.')
     var lastNode = configurationObject
-    while (1) {
+    // eslint-disable-next-line no-constant-condition
+    while (true) {
       // Find the node for the current property
       var propertyNode = this.findProperty(lastNode, properties[depth])
 
@@ -222,11 +386,7 @@ class ConfigParser {
     // Create nested properties in the configuration file
     else {
       // Build the declaration to inject
-      const declaration = this.getPropertyDeclaration(
-        properties,
-        depth,
-        propertyValue
-      )
+      const declaration = this.getPropertyDeclaration(properties, depth, propertyValue)
 
       // The last node identified is an object expression, so do the assignment
       if (lastNode.type === 'ObjectExpression') {
@@ -263,7 +423,7 @@ class ConfigParser {
     }
 
     // Logging
-    core.info(`Injection successful, new configuration:`)
+    core.info('Injection successful, new configuration:')
     core.info(this.configuration)
 
     // Finally write the new configuration in the file
@@ -273,4 +433,4 @@ class ConfigParser {
   }
 }
 
-module.exports = {ConfigParser}
+module.exports = { ConfigParser }

src/config-parser.test.js

@@ -1,7 +1,8 @@
 const fs = require('fs')
+const core = require('@actions/core')
 
-const {ConfigParser} = require('./config-parser')
-const {getTempFolder, compareFiles} = require('./test-helpers')
+const { ConfigParser } = require('./config-parser')
+const { getTempFolder, compareFiles } = require('./test-helpers')
 
 // Get the temp folder
 const tempFolder = getTempFolder()
@@ -9,84 +10,109 @@ const tempFolder = getTempFolder()
 // Cases to test
 const cases = [
   //
-  // Default export
+  // Direct default export
   //
-
   {
     property: 'property',
-    source: `export default {}`,
-    expected: `export default { property: "value" }`
+    source: 'export default {}',
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: 0 }`, // property exists and is a number
-    expected: `export default { property: "value" }`
+    source: 'export default { property: 0 }', // property exists and is a number
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: false }`, // property exists and is a boolean
-    expected: `export default { property: "value" }`
+    source: 'export default { property: false }', // property exists and is a boolean
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: "test" }`, // property exists and is a string
-    expected: `export default { property: "value" }`
+    source: 'export default { property: "test" }', // property exists and is a string
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: [1,2] }`, // property exists and is an array
-    expected: `export default { property: "value" }`
+    source: 'export default { property: [1,2] }', // property exists and is an array
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: null }`, // property exists and is null
-    expected: `export default { property: "value" }`
+    source: 'export default { property: null }', // property exists and is null
+    expected: 'export default { property: "value" }'
   },
   {
     property: 'property',
-    source: `export default { property: {}}`, // property exists and is an object
-    expected: `export default { property: "value" }`
+    source: 'export default { property: { } }', // property exists and is an object
+    expected: 'export default { property: "value" }'
   },
 
   // Deep properties (injection 1)
   {
     property: 'property.b.c',
-    source: `export default {}`,
-    expected: `export default { property: { b: { c: "value" }}}`
+    source: 'export default {}',
+    expected: 'export default { property: { b: { c: "value"  } } }'
   },
   {
     property: 'property.b.c',
-    source: `export default { property: 0 }`, // property exists and is a number
-    expected: `export default { property: { b: { c: "value" }}}`
+    source: 'export default { property: 0 }', // property exists and is a number
+    expected: 'export default { property: { b: { c: "value"  } } }'
   },
   {
     property: 'property.b.c',
-    source: `export default { property: {}}`, // property exists and is an object
-    expected: `export default { property: { b: { c: "value" }}}`
+    source: 'export default { property: { } }', // property exists and is an object
+    expected: 'export default { property: { b: { c: "value"  } } }'
   },
 
   // Deep properties (injection 2)
   {
     property: 'property.b.c',
-    source: `export default { property: { b: 0 }}`, // property exists and is a number
-    expected: `export default { property: { b: { c: "value" }}}`
+    source: 'export default { property: { b: 0  } }', // property exists and is a number
+    expected: 'export default { property: { b: { c: "value"  } } }'
   },
   {
     property: 'property.b.c',
-    source: `export default { property: { b: {}}}`, // property exists and is an object
-    expected: `export default { property: { b: { c: "value" }}}`
+    source: 'export default { property: { b: { } } }', // property exists and is an object
+    expected: 'export default { property: { b: { c: "value"  } } }'
   },
   {
     property: 'property.b.c',
-    source: `export default { property: { b: { hello: 123}}}`, // property exists and is a non-empty object
-    expected: `export default { property: { b: { c: "value", hello: 123 }}}`
+    source: 'export default { property: { b: { hello: 123 } } }', // property exists and is a non-empty object
+    expected: 'export default { property: { b: { c: "value", hello: 123  } } }'
   },
 
   // Deep properties (existing properties)
   {
     property: 'a1.a2',
-    source: `export default { a2: false, a1: { a3: [12]}}`, // property exists and is a non-empty object
-    expected: `export default { a2: false, a1: { a2: "value", a3: [12]}}`
+    source: 'export default { a2: false, a1: { a3: [12] } }', // property exists and is a non-empty object
+    expected: 'export default { a2: false, a1: { a2: "value", a3: [12] } }'
+  },
+
+  //
+  // Indirect default export
+  //
+  {
+    property: 'property',
+    source: 'const config = {}; export default config',
+    expected: 'const config = { property: "value"}; export default config'
+  },
+  // with more than 1 declaration chained together
+  {
+    property: 'property',
+    source: 'var temp = {}, config = {}; export default config',
+    expected: 'var temp = {}, config = { property: "value"}; export default config'
+  },
+  // deeper
+  {
+    property: 'a.b.c',
+    source: 'var config = {}; export default config',
+    expected: 'var config = { a: { b: { c: "value" } } }; export default config'
+  },
+  {
+    property: 'a.b.c',
+    source: 'var config = { a: { b: [], c: "hello" } }; export default config',
+    expected: 'var config = { a: { b: { c: "value"}, c: "hello" } }; export default config'
   },
 
   //
@@ -94,18 +120,18 @@ const cases = [
   //
   {
     property: 'property',
-    source: `module.exports = {}`,
-    expected: `module.exports = { property: "value"}`
+    source: 'module.exports = {}',
+    expected: 'module.exports = { property: "value"}'
   },
   {
     property: 'property',
-    source: `module.exports = { p1: 0}`,
-    expected: `module.exports = { property: "value", p1: 0}`
+    source: 'module.exports = { p1: 0}',
+    expected: 'module.exports = { property: "value", p1: 0}'
   },
   {
     property: 'a.b.c',
-    source: `module.exports = { p1: 0}`,
-    expected: `module.exports = { a: { b: { c: "value" }}, p1: 0}`
+    source: 'module.exports = { p1: 0}',
+    expected: 'module.exports = { a: { b: { c: "value"  } }, p1: 0}'
   },
 
   //
@@ -113,40 +139,156 @@ const cases = [
   //
   {
     property: 'property',
-    source: `const config = {}; module.exports = config`,
-    expected: `const config = { property: "value"}; module.exports = config`
+    source: 'const config = {}; module.exports = config',
+    expected: 'const config = { property: "value"}; module.exports = config'
   },
+  // with more than 1 declaration chained together
   {
     property: 'property',
-    source: `var config = {}; module.exports = config`,
-    expected: `var config = { property: "value"}; module.exports = config`
+    source: 'var temp = {}, config = {}; module.exports = config',
+    expected: 'var temp = {}, config = { property: "value"}; module.exports = config'
+  },
+  // deeper
+  {
+    property: 'a.b.c',
+    source: 'var config = {}; module.exports = config',
+    expected: 'var config = { a: { b: { c: "value" } } }; module.exports = config'
   },
   {
     property: 'a.b.c',
-    source: `var config = {}; module.exports = config`,
-    expected: `var config = { a: { b: { c: "value"}}}; module.exports = config`
+    source: 'var config = { a: { b: [], c: "hello" } }; module.exports = config',
+    expected: 'var config = { a: { b: { c: "value"}, c: "hello" } }; module.exports = config'
   },
+
+  //
+  // Direct default export with wrapping call
+  //
   {
-    property: 'a.b.c',
-    source: `var config = { a: { b: [], c: "hello"}}; module.exports = config`,
-    expected: `var config = { a: { b: { c: "value"}, c: "hello"}}; module.exports = config`
+    property: 'property',
+    source: 'import { defineConfig } from "astro/config"; export default defineConfig({ p1: 0 })',
+    expected: 'import { defineConfig } from "astro/config"; export default defineConfig({ property: "value", p1: 0 })',
+    allowWrappingCall: true
+  },
+
+  //
+  // Direct module exports with wrapping call
+  //
+  {
+    property: 'property',
+    source: 'const { defineConfig } = require("astro/config"); module.exports = defineConfig({ p1: 0 })',
+    expected:
+      'const { defineConfig } = require("astro/config"); module.exports = defineConfig({ property: "value", p1: 0 })',
+    allowWrappingCall: true
+  },
+
+  //
+  // Indirect default export with wrapping call at the definition
+  //
+  {
+    property: 'property',
+    source: 'import { defineConfig } from "astro/config"; const config = defineConfig({}); export default config',
+    expected:
+      'import { defineConfig } from "astro/config"; const config = defineConfig({ property: "value" }); export default config',
+    allowWrappingCall: true
+  },
+  // with more than 1 declaration chained together
+  {
+    property: 'property',
+    source:
+      'import { defineConfig } from "astro/config"; const temp = {}, config = defineConfig({}); export default config',
+    expected:
+      'import { defineConfig } from "astro/config"; const temp = {}, config = defineConfig({ property: "value" }); export default config',
+    allowWrappingCall: true
+  },
+
+  //
+  // Indirect default export with wrapping call at the export
+  //
+  {
+    property: 'property',
+    source: 'import { defineConfig } from "astro/config"; const config = {}; export default defineConfig(config)',
+    expected:
+      'import { defineConfig } from "astro/config"; const config = { property: "value" }; export default defineConfig(config)',
+    allowWrappingCall: true
+  },
+  // with more than 1 declaration chained together
+  {
+    property: 'property',
+    source:
+      'import { defineConfig } from "astro/config"; const temp = {}, config = {}; export default defineConfig(config)',
+    expected:
+      'import { defineConfig } from "astro/config"; const temp = {}, config = { property: "value" }; export default defineConfig(config)',
+    allowWrappingCall: true
+  },
+
+  //
+  // Indirect module exports with wrapping call at the definition
+  //
+  {
+    property: 'property',
+    source:
+      'const { defineConfig } = require("astro/config"); const config = defineConfig({}); module.exports = config',
+    expected:
+      'const { defineConfig } = require("astro/config"); const config = defineConfig({ property: "value"}); module.exports = config',
+    allowWrappingCall: true
+  },
+  // with more than 1 declaration chained together
+  {
+    property: 'property',
+    source:
+      'const { defineConfig } = require("astro/config"); const temp = {}, config = defineConfig({}); module.exports = config',
+    expected:
+      'const { defineConfig } = require("astro/config"); const temp = {}, config = defineConfig({ property: "value"}); module.exports = config',
+    allowWrappingCall: true
+  },
+
+  //
+  // Indirect module exports with wrapping call at the export
+  //
+  {
+    property: 'property',
+    source:
+      'const { defineConfig } = require("astro/config"); const config = {}; module.exports = defineConfig(config)',
+    expected:
+      'const { defineConfig } = require("astro/config"); const config = { property: "value"}; module.exports = defineConfig(config)',
+    allowWrappingCall: true
+  },
+  // with more than 1 declaration chained together
+  {
+    property: 'property',
+    source:
+      'const { defineConfig } = require("astro/config"); const temp = {}, config = {}; module.exports = defineConfig(config)',
+    expected:
+      'const { defineConfig } = require("astro/config"); const temp = {}, config = { property: "value"}; module.exports = defineConfig(config)',
+    allowWrappingCall: true
   }
 ]
 
 describe('config-parser', () => {
-  cases.forEach(({property, source, expected}, index) => {
-    it(`Inject path properly for case #${index}`, () => {
+  beforeEach(() => {
+    jest.restoreAllMocks()
+
+    // Mock error/warning/info/debug to silence their output
+    jest.spyOn(core, 'error').mockImplementation(jest.fn())
+    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
+    jest.spyOn(core, 'info').mockImplementation(jest.fn())
+    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
+  })
+
+  cases.forEach(({ property, source, expected, allowWrappingCall = false }, index) => {
+    it(`injects path properly for case #${index}`, () => {
       // Write the source file
       const sourceFile = `${tempFolder}/source.js`
-      fs.writeFileSync(sourceFile, source, {encoding: 'utf8'})
+      fs.writeFileSync(sourceFile, source, { encoding: 'utf8' })
 
       // Write the expected file
       const expectedFile = `${tempFolder}/expected.js`
-      fs.writeFileSync(expectedFile, expected, {encoding: 'utf8'})
+      fs.writeFileSync(expectedFile, expected, { encoding: 'utf8' })
 
       // Update the settings and do the injection
       new ConfigParser({
-        configurationFile: sourceFile
+        configurationFile: sourceFile,
+        allowWrappingCall
       }).inject(property, 'value')
 
       // Compare the files

src/context.js

@@ -5,7 +5,9 @@ function getRequiredVars() {
   return {
     repositoryNwo: process.env.GITHUB_REPOSITORY,
     githubToken: core.getInput('token'),
-    staticSiteGenerator: core.getInput('static_site_generator')
+    staticSiteGenerator: core.getInput('static_site_generator'),
+    generatorConfigFile: core.getInput('generator_config_file'),
+    enablement: core.getInput('enablement') !== 'true'
   }
 }
 
@@ -21,4 +23,4 @@ function getContext() {
   return requiredVars
 }
 
-module.exports = {getContext}
+module.exports = { getContext }

src/enable-pages.js

@@ -1,31 +0,0 @@
-const core = require('@actions/core')
-const axios = require('axios')
-
-async function enablePages({repositoryNwo, githubToken}) {
-  const pagesEndpoint = `https://api.github.com/repos/${repositoryNwo}/pages`
-
-  try {
-    const response = await axios.post(
-      pagesEndpoint,
-      {build_type: 'workflow'},
-      {
-        headers: {
-          Accept: 'application/vnd.github.v3+json',
-          Authorization: `Bearer ${githubToken}`,
-          'Content-type': 'application/json'
-        }
-      }
-    )
-    core.info('Created pages site')
-  } catch (error) {
-    if (error.response && error.response.status === 409) {
-      core.info('Pages site exists')
-      return
-    }
-
-    core.error("Couldn't create pages site", error)
-    throw error
-  }
-}
-
-module.exports = enablePages

src/enable-pages.test.js

@@ -1,55 +0,0 @@
-const core = require('@actions/core')
-const axios = require('axios')
-
-const enablePages = require('./enable-pages')
-
-describe('enablePages', () => {
-  const GITHUB_REPOSITORY = 'paper-spa/is-awesome'
-  const GITHUB_TOKEN = 'gha-token'
-
-  beforeEach(() => {
-    jest.restoreAllMocks()
-
-    // Mock error/warning/info/debug
-    jest.spyOn(core, 'error').mockImplementation(jest.fn())
-    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
-    jest.spyOn(core, 'info').mockImplementation(jest.fn())
-    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
-  })
-
-  it('makes a request to create a page', async () => {
-    jest.spyOn(axios, 'post').mockImplementationOnce(() => Promise.resolve({}))
-
-    await enablePages({
-      repositoryNwo: GITHUB_REPOSITORY,
-      githubToken: GITHUB_TOKEN
-    })
-  })
-
-  it('handles a 409 response when the page already exists', async () => {
-    jest
-      .spyOn(axios, 'post')
-      .mockImplementationOnce(() => Promise.reject({response: {status: 409}}))
-
-    // Simply assert that no error is raised
-    await enablePages({
-      repositoryNwo: GITHUB_REPOSITORY,
-      githubToken: GITHUB_TOKEN
-    })
-  })
-
-  it('re-raises errors on failure status codes', async () => {
-    jest
-      .spyOn(axios, 'post')
-      .mockImplementationOnce(() => Promise.reject({response: {status: 404}}))
-
-    try {
-      await enablePages({
-        repositoryNwo: GITHUB_REPOSITORY,
-        githubToken: GITHUB_TOKEN
-      })
-    } catch (error) {
-      expect(error.response.status).toEqual(404)
-    }
-  })
-})

src/fixtures/gatsby/blank.expected.js

@@ -1,2 +1,2 @@
 // Default Pages configuration for Gatsby
-module.exports = { pathPrefix: "/docs/" }
+module.exports = { siteMetadata: { siteUrl: 'https://configure-pages.github.io' }, pathPrefix: '/docs/' }

src/fixtures/gatsby/blank.js

@@ -1 +1 @@
-// This file is not read by the test suite
+// This file is not read by the test suite

src/fixtures/gatsby/default.cjs

@@ -0,0 +1,6 @@
+module.exports = {
+  siteMetadata: {
+    title: 'My Gatsby Site'
+  },
+  plugins: []
+}

src/fixtures/gatsby/default.expected.cjs

@@ -0,0 +1,8 @@
+module.exports = {
+  pathPrefix: '/docs/',
+  siteMetadata: {
+    siteUrl: 'https://configure-pages.github.io',
+    title: 'My Gatsby Site'
+  },
+  plugins: []
+}

src/fixtures/gatsby/default.expected.js

@@ -1,8 +1,8 @@
 module.exports = {
-  pathPrefix: "/docs/",
+  pathPrefix: '/docs/',
   siteMetadata: {
-    title: `My Gatsby Site`,
-    siteUrl: `https://www.yourdomain.tld`,
+    siteUrl: 'https://configure-pages.github.io',
+    title: 'My Gatsby Site'
   },
-  plugins: [],
-}
+  plugins: []
+}

src/fixtures/gatsby/default.expected.mjs

@@ -0,0 +1,8 @@
+export default {
+  pathPrefix: '/docs/',
+  siteMetadata: {
+    siteUrl: 'https://configure-pages.github.io',
+    title: 'My Gatsby Site'
+  },
+  plugins: []
+}

src/fixtures/gatsby/default.js

@@ -1,7 +1,6 @@
 module.exports = {
   siteMetadata: {
-    title: `My Gatsby Site`,
-    siteUrl: `https://www.yourdomain.tld`,
+    title: 'My Gatsby Site'
   },
-  plugins: [],
-}
+  plugins: []
+}

src/fixtures/gatsby/default.mjs

@@ -0,0 +1,6 @@
+export default {
+  siteMetadata: {
+    title: 'My Gatsby Site'
+  },
+  plugins: []
+}

src/fixtures/next/blank.expected.js

@@ -1,6 +1,3 @@
 // Default Pages configuration for Next
-const nextConfig = {
-  experimental: {images: {unoptimized: true}},
-  basePath: '/docs'
-}
+const nextConfig = { images: { unoptimized: true }, experimental: { images: { unoptimized: true } }, basePath: '/docs' }
 module.exports = nextConfig

src/fixtures/next/blank.js

@@ -1 +1 @@
-// This file is not read by the test suite
+// This file is not read by the test suite

src/fixtures/next/default.cjs

@@ -0,0 +1,7 @@
+/** @type {import('next').NextConfig} */
+const nextConfig = {
+  reactStrictMode: true,
+  swcMinify: true
+}
+
+module.exports = nextConfig

src/fixtures/next/default.expected.cjs

@@ -0,0 +1,10 @@
+/** @type {import('next').NextConfig} */
+const nextConfig = {
+  images: { unoptimized: true },
+  experimental: { images: { unoptimized: true } },
+  basePath: '/docs',
+  reactStrictMode: true,
+  swcMinify: true
+}
+
+module.exports = nextConfig

src/fixtures/next/default.expected.js

@@ -1,6 +1,7 @@
 /** @type {import('next').NextConfig} */
 const nextConfig = {
-  experimental: {images: {unoptimized: true}},
+  images: { unoptimized: true },
+  experimental: { images: { unoptimized: true } },
   basePath: '/docs',
   reactStrictMode: true,
   swcMinify: true

src/fixtures/next/default.expected.mjs

@@ -0,0 +1,10 @@
+/** @type {import('next').NextConfig} */
+const nextConfig = {
+  images: { unoptimized: true },
+  experimental: { images: { unoptimized: true } },
+  basePath: '/docs',
+  reactStrictMode: true,
+  swcMinify: true
+}
+
+export default nextConfig

src/fixtures/next/default.js

@@ -1,7 +1,7 @@
 /** @type {import('next').NextConfig} */
 const nextConfig = {
   reactStrictMode: true,
-  swcMinify: true,
+  swcMinify: true
 }
 
-module.exports = nextConfig
+module.exports = nextConfig

src/fixtures/next/default.mjs

@@ -0,0 +1,7 @@
+/** @type {import('next').NextConfig} */
+const nextConfig = {
+  reactStrictMode: true,
+  swcMinify: true
+}
+
+export default nextConfig

src/fixtures/nuxt/async.cjs

@@ -0,0 +1,15 @@
+const getAllDynamicRoute = async function () {
+  const routes = await (async () => {
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
+
+module.exports = {
+  mode: 'universal',
+  generate: {
+    async routes() {
+      return getAllDynamicRoute()
+    }
+  }
+}

src/fixtures/nuxt/async.expected.cjs

@@ -0,0 +1,17 @@
+const getAllDynamicRoute = async function () {
+  const routes = await (async () => {
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
+
+module.exports = {
+  target: 'static',
+  router: { base: '/docs/' },
+  mode: 'universal',
+  generate: {
+    async routes() {
+      return getAllDynamicRoute()
+    }
+  }
+}

src/fixtures/nuxt/async.expected.js

@@ -1,17 +1,17 @@
-const getAllDynamicRoute = async function() {
+const getAllDynamicRoute = async function () {
   const routes = await (async () => {
-    return ['/posts/hello-world', '/posts/hello-again'];
-  })();
-  return routes;
-};
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
 
 export default {
   target: 'static',
-  router: {base: '/docs/'},
+  router: { base: '/docs/' },
   mode: 'universal',
   generate: {
-    async routes () {
-      return getAllDynamicRoute();
+    async routes() {
+      return getAllDynamicRoute()
     }
   }
-};
+}

src/fixtures/nuxt/async.expected.mjs

@@ -0,0 +1,17 @@
+const getAllDynamicRoute = async function () {
+  const routes = await (async () => {
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
+
+export default {
+  target: 'static',
+  router: { base: '/docs/' },
+  mode: 'universal',
+  generate: {
+    async routes() {
+      return getAllDynamicRoute()
+    }
+  }
+}

src/fixtures/nuxt/async.js

@@ -1,15 +1,15 @@
-const getAllDynamicRoute = async function() {
+const getAllDynamicRoute = async function () {
   const routes = await (async () => {
-    return ['/posts/hello-world', '/posts/hello-again'];
-  })();
-  return routes;
-};
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
 
 export default {
   mode: 'universal',
   generate: {
-    async routes () {
-      return getAllDynamicRoute();
+    async routes() {
+      return getAllDynamicRoute()
     }
   }
-};
+}

src/fixtures/nuxt/async.mjs

@@ -0,0 +1,15 @@
+const getAllDynamicRoute = async function () {
+  const routes = await (async () => {
+    return ['/posts/hello-world', '/posts/hello-again']
+  })()
+  return routes
+}
+
+export default {
+  mode: 'universal',
+  generate: {
+    async routes() {
+      return getAllDynamicRoute()
+    }
+  }
+}

src/fixtures/nuxt/blank.expected.js

@@ -1,2 +1,2 @@
 // Default Pages configuration for Nuxt
-export default {target: 'static', router: {base: '/docs/'}}
+export default { target: 'static', router: { base: '/docs/' } }

src/fixtures/nuxt/blank.js

@@ -1 +1 @@
-// This file is not read by the test suite
+// This file is not read by the test suite

src/fixtures/nuxt/default.cjs

@@ -0,0 +1,37 @@
+module.exports = {
+  // Disable server-side rendering: https://go.nuxtjs.dev/ssr-mode
+  ssr: false,
+
+  // Global page headers: https://go.nuxtjs.dev/config-head
+  head: {
+    title: 'nuxt',
+    htmlAttrs: {
+      lang: 'en'
+    },
+    meta: [
+      { charset: 'utf-8' },
+      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
+      { hid: 'description', name: 'description', content: '' },
+      { name: 'format-detection', content: 'telephone=no' }
+    ],
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
+  },
+
+  // Global CSS: https://go.nuxtjs.dev/config-css
+  css: [],
+
+  // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
+  plugins: [],
+
+  // Auto import components: https://go.nuxtjs.dev/config-components
+  components: true,
+
+  // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
+  buildModules: [],
+
+  // Modules: https://go.nuxtjs.dev/config-modules
+  modules: [],
+
+  // Build Configuration: https://go.nuxtjs.dev/config-build
+  build: {}
+}

src/fixtures/nuxt/default.expected.cjs

@@ -0,0 +1,39 @@
+module.exports = {
+  // Disable server-side rendering: https://go.nuxtjs.dev/ssr-mode
+  target: 'static',
+  router: { base: '/docs/' },
+  ssr: false,
+
+  // Global page headers: https://go.nuxtjs.dev/config-head
+  head: {
+    title: 'nuxt',
+    htmlAttrs: {
+      lang: 'en'
+    },
+    meta: [
+      { charset: 'utf-8' },
+      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
+      { hid: 'description', name: 'description', content: '' },
+      { name: 'format-detection', content: 'telephone=no' }
+    ],
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
+  },
+
+  // Global CSS: https://go.nuxtjs.dev/config-css
+  css: [],
+
+  // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
+  plugins: [],
+
+  // Auto import components: https://go.nuxtjs.dev/config-components
+  components: true,
+
+  // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
+  buildModules: [],
+
+  // Modules: https://go.nuxtjs.dev/config-modules
+  modules: [],
+
+  // Build Configuration: https://go.nuxtjs.dev/config-build
+  build: {}
+}

src/fixtures/nuxt/default.expected.js

@@ -1,7 +1,7 @@
 export default {
   // Disable server-side rendering: https://go.nuxtjs.dev/ssr-mode
   target: 'static',
-  router: { base: "/docs/" },
+  router: { base: '/docs/' },
   ssr: false,
 
   // Global page headers: https://go.nuxtjs.dev/config-head
@@ -16,31 +16,24 @@ export default {
       { hid: 'description', name: 'description', content: '' },
       { name: 'format-detection', content: 'telephone=no' }
     ],
-    link: [
-      { rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }
-    ]
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
   },
 
   // Global CSS: https://go.nuxtjs.dev/config-css
-  css: [
-  ],
+  css: [],
 
   // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
-  plugins: [
-  ],
+  plugins: [],
 
   // Auto import components: https://go.nuxtjs.dev/config-components
   components: true,
 
   // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
-  buildModules: [
-  ],
+  buildModules: [],
 
   // Modules: https://go.nuxtjs.dev/config-modules
-  modules: [
-  ],
+  modules: [],
 
   // Build Configuration: https://go.nuxtjs.dev/config-build
-  build: {
-  }
-}
+  build: {}
+}

src/fixtures/nuxt/default.expected.mjs

@@ -0,0 +1,39 @@
+export default {
+  // Disable server-side rendering: https://go.nuxtjs.dev/ssr-mode
+  target: 'static',
+  router: { base: '/docs/' },
+  ssr: false,
+
+  // Global page headers: https://go.nuxtjs.dev/config-head
+  head: {
+    title: 'nuxt',
+    htmlAttrs: {
+      lang: 'en'
+    },
+    meta: [
+      { charset: 'utf-8' },
+      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
+      { hid: 'description', name: 'description', content: '' },
+      { name: 'format-detection', content: 'telephone=no' }
+    ],
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
+  },
+
+  // Global CSS: https://go.nuxtjs.dev/config-css
+  css: [],
+
+  // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
+  plugins: [],
+
+  // Auto import components: https://go.nuxtjs.dev/config-components
+  components: true,
+
+  // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
+  buildModules: [],
+
+  // Modules: https://go.nuxtjs.dev/config-modules
+  modules: [],
+
+  // Build Configuration: https://go.nuxtjs.dev/config-build
+  build: {}
+}

src/fixtures/nuxt/default.js

@@ -14,31 +14,24 @@ export default {
       { hid: 'description', name: 'description', content: '' },
       { name: 'format-detection', content: 'telephone=no' }
     ],
-    link: [
-      { rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }
-    ]
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
   },
 
   // Global CSS: https://go.nuxtjs.dev/config-css
-  css: [
-  ],
+  css: [],
 
   // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
-  plugins: [
-  ],
+  plugins: [],
 
   // Auto import components: https://go.nuxtjs.dev/config-components
   components: true,
 
   // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
-  buildModules: [
-  ],
+  buildModules: [],
 
   // Modules: https://go.nuxtjs.dev/config-modules
-  modules: [
-  ],
+  modules: [],
 
   // Build Configuration: https://go.nuxtjs.dev/config-build
-  build: {
-  }
-}
+  build: {}
+}

src/fixtures/nuxt/default.mjs

@@ -0,0 +1,37 @@
+export default {
+  // Disable server-side rendering: https://go.nuxtjs.dev/ssr-mode
+  ssr: false,
+
+  // Global page headers: https://go.nuxtjs.dev/config-head
+  head: {
+    title: 'nuxt',
+    htmlAttrs: {
+      lang: 'en'
+    },
+    meta: [
+      { charset: 'utf-8' },
+      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
+      { hid: 'description', name: 'description', content: '' },
+      { name: 'format-detection', content: 'telephone=no' }
+    ],
+    link: [{ rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }]
+  },
+
+  // Global CSS: https://go.nuxtjs.dev/config-css
+  css: [],
+
+  // Plugins to run before rendering page: https://go.nuxtjs.dev/config-plugins
+  plugins: [],
+
+  // Auto import components: https://go.nuxtjs.dev/config-components
+  components: true,
+
+  // Modules for dev and build (recommended): https://go.nuxtjs.dev/config-modules
+  buildModules: [],
+
+  // Modules: https://go.nuxtjs.dev/config-modules
+  modules: [],
+
+  // Build Configuration: https://go.nuxtjs.dev/config-build
+  build: {}
+}

src/fixtures/sveltekit/blank.expected.js

@@ -0,0 +1,10 @@
+// Default Pages configuration for SvelteKit
+import adapter from '@sveltejs/adapter-auto'
+
+export default {
+  kit: {
+    prerender: { origin: 'https://configure-pages.github.io' },
+    paths: { base: '/docs' },
+    adapter: adapter()
+  }
+}

src/fixtures/sveltekit/blank.js

@@ -0,0 +1 @@
+// This file is not read by the test suite

src/fixtures/sveltekit/default.expected.js

@@ -0,0 +1,12 @@
+import adapter from '@sveltejs/adapter-auto'
+
+/** @type {import('@sveltejs/kit').Config} */
+const config = {
+  kit: {
+    prerender: { origin: 'https://configure-pages.github.io' },
+    paths: { base: '/docs' },
+    adapter: adapter()
+  }
+}
+
+export default config

src/fixtures/sveltekit/default.js

@@ -0,0 +1,10 @@
+import adapter from '@sveltejs/adapter-auto'
+
+/** @type {import('@sveltejs/kit').Config} */
+const config = {
+  kit: {
+    adapter: adapter()
+  }
+}
+
+export default config

src/get-pages-base-url.js

@@ -1,38 +0,0 @@
-const core = require('@actions/core')
-const axios = require('axios')
-const {setPagesPath} = require('./set-pages-path')
-
-async function getPagesBaseUrl({
-  repositoryNwo,
-  githubToken,
-  staticSiteGenerator
-}) {
-  try {
-    const pagesEndpoint = `https://api.github.com/repos/${repositoryNwo}/pages`
-
-    core.info(`Get the Base URL to the page with endpoint ${pagesEndpoint}`)
-    const response = await axios.get(pagesEndpoint, {
-      headers: {
-        Accept: 'application/vnd.github.v3+json',
-        Authorization: `Bearer ${githubToken}`
-      }
-    })
-
-    pageObject = response.data
-    core.info(JSON.stringify(pageObject))
-
-    const siteUrl = new URL(pageObject.html_url)
-    if (staticSiteGenerator) {
-      setPagesPath({staticSiteGenerator, path: siteUrl.pathname})
-    }
-    core.setOutput('base_url', siteUrl.href)
-    core.setOutput('origin', siteUrl.origin)
-    core.setOutput('host', siteUrl.host)
-    core.setOutput('base_path', siteUrl.pathname)
-  } catch (error) {
-    core.error('Get on the Page failed', error)
-    throw error
-  }
-}
-
-module.exports = getPagesBaseUrl

src/get-pages-base-url.test.js

@@ -1,93 +0,0 @@
-const core = require('@actions/core')
-const axios = require('axios')
-
-const getPagesBaseUrl = require('./get-pages-base-url')
-
-describe('getPagesBaseUrl', () => {
-  const GITHUB_REPOSITORY = 'paper-spa/is-awesome'
-  const GITHUB_TOKEN = 'gha-token'
-
-  beforeEach(() => {
-    jest.restoreAllMocks()
-
-    jest.spyOn(core, 'setOutput').mockImplementation((key, value) => {
-      key, value
-    })
-    jest.spyOn(core, 'setFailed').mockImplementation(param => param)
-
-    // Mock error/warning/info/debug
-    jest.spyOn(core, 'error').mockImplementation(jest.fn())
-    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
-    jest.spyOn(core, 'info').mockImplementation(jest.fn())
-    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
-  })
-
-  it('gets expected outputs for profile site', async () => {
-    const baseUrl = 'https://octocat.github.io/'
-
-    jest
-      .spyOn(axios, 'get')
-      .mockImplementationOnce(() =>
-        Promise.resolve({data: {html_url: baseUrl}})
-      )
-
-    await getPagesBaseUrl({
-      repositoryNwo: GITHUB_REPOSITORY,
-      githubToken: GITHUB_TOKEN
-    })
-
-    expect(core.setOutput).toHaveBeenCalledWith('base_url', baseUrl)
-    expect(core.setOutput).toHaveBeenCalledWith(
-      'origin',
-      'https://octocat.github.io'
-    )
-    expect(core.setOutput).toHaveBeenCalledWith('host', 'octocat.github.io')
-    expect(core.setOutput).toHaveBeenCalledWith('base_path', '/')
-  })
-
-  it('gets expected outputs for project site', async () => {
-    const baseUrl = 'https://octocat.github.io/my-repo/'
-
-    jest
-      .spyOn(axios, 'get')
-      .mockImplementationOnce(() =>
-        Promise.resolve({data: {html_url: baseUrl}})
-      )
-
-    await getPagesBaseUrl({
-      repositoryNwo: GITHUB_REPOSITORY,
-      githubToken: GITHUB_TOKEN
-    })
-
-    expect(core.setOutput).toHaveBeenCalledWith('base_url', baseUrl)
-    expect(core.setOutput).toHaveBeenCalledWith(
-      'origin',
-      'https://octocat.github.io'
-    )
-    expect(core.setOutput).toHaveBeenCalledWith('host', 'octocat.github.io')
-    expect(core.setOutput).toHaveBeenCalledWith('base_path', '/my-repo/')
-  })
-
-  it('gets expected outputs for site with custom domain name', async () => {
-    const baseUrl = 'https://www.example.com/'
-
-    jest
-      .spyOn(axios, 'get')
-      .mockImplementationOnce(() =>
-        Promise.resolve({data: {html_url: baseUrl}})
-      )
-
-    await getPagesBaseUrl({
-      repositoryNwo: GITHUB_REPOSITORY,
-      githubToken: GITHUB_TOKEN
-    })
-
-    expect(core.setOutput).toHaveBeenCalledWith('base_url', baseUrl)
-    expect(core.setOutput).toHaveBeenCalledWith(
-      'origin',
-      'https://www.example.com'
-    )
-    expect(core.setOutput).toHaveBeenCalledWith('host', 'www.example.com')
-    expect(core.setOutput).toHaveBeenCalledWith('base_path', '/')
-  })
-})

src/index.js

@@ -1,16 +1,24 @@
 const core = require('@actions/core')
 
-const enablePages = require('./enable-pages')
-const getPagesBaseUrl = require('./get-pages-base-url')
-
 // All variables we need from the runtime are loaded here
-const {getContext} = require('./context')
+const { getContext } = require('./context')
+
+const { findOrCreatePagesSite } = require('./api-client')
+const { setPagesConfig } = require('./set-pages-config')
+const outputPagesBaseUrl = require('./output-pages-base-url')
 
 async function main() {
   try {
-    const context = getContext()
-    await enablePages(context)
-    await getPagesBaseUrl(context)
+    const { repositoryNwo, githubToken, enablement, staticSiteGenerator, generatorConfigFile } = getContext()
+
+    const pageObject = await findOrCreatePagesSite({ repositoryNwo, githubToken, enablement })
+    const siteUrl = new URL(pageObject.html_url)
+
+    if (staticSiteGenerator) {
+      setPagesConfig({ staticSiteGenerator, generatorConfigFile, siteUrl })
+    }
+    outputPagesBaseUrl(siteUrl)
+    core.exportVariable('GITHUB_PAGES', 'true')
   } catch (error) {
     core.setFailed(error)
     process.exit(1)

src/output-pages-base-url.js

@@ -0,0 +1,15 @@
+const core = require('@actions/core')
+const removeTrailingSlash = require('./remove-trailing-slash')
+
+function outputPagesBaseUrl(siteUrl) {
+  // Many static site generators do not want the trailing slash, and it is much easier to add than remove in a workflow
+  const baseUrl = removeTrailingSlash(siteUrl.href)
+  const basePath = removeTrailingSlash(siteUrl.pathname)
+
+  core.setOutput('base_url', baseUrl)
+  core.setOutput('origin', siteUrl.origin)
+  core.setOutput('host', siteUrl.host)
+  core.setOutput('base_path', basePath)
+}
+
+module.exports = outputPagesBaseUrl

src/output-pages-base-url.test.js

@@ -0,0 +1,53 @@
+const core = require('@actions/core')
+
+const outputPagesBaseUrl = require('./output-pages-base-url')
+
+describe('outputPagesBaseUrl', () => {
+  beforeEach(() => {
+    jest.restoreAllMocks()
+
+    jest.spyOn(core, 'setOutput').mockImplementation((key, value) => {
+      key, value
+    })
+    jest.spyOn(core, 'setFailed').mockImplementation(param => param)
+
+    // Mock error/warning/info/debug
+    jest.spyOn(core, 'error').mockImplementation(jest.fn())
+    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
+    jest.spyOn(core, 'info').mockImplementation(jest.fn())
+    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
+  })
+
+  it('gets expected outputs for profile site', async () => {
+    const baseUrl = 'https://octocat.github.io/'
+
+    outputPagesBaseUrl(new URL(baseUrl))
+
+    expect(core.setOutput).toHaveBeenCalledWith('base_url', 'https://octocat.github.io')
+    expect(core.setOutput).toHaveBeenCalledWith('origin', 'https://octocat.github.io')
+    expect(core.setOutput).toHaveBeenCalledWith('host', 'octocat.github.io')
+    expect(core.setOutput).toHaveBeenCalledWith('base_path', '')
+  })
+
+  it('gets expected outputs for project site', async () => {
+    const baseUrl = 'https://octocat.github.io/my-repo/'
+
+    outputPagesBaseUrl(new URL(baseUrl))
+
+    expect(core.setOutput).toHaveBeenCalledWith('base_url', 'https://octocat.github.io/my-repo')
+    expect(core.setOutput).toHaveBeenCalledWith('origin', 'https://octocat.github.io')
+    expect(core.setOutput).toHaveBeenCalledWith('host', 'octocat.github.io')
+    expect(core.setOutput).toHaveBeenCalledWith('base_path', '/my-repo')
+  })
+
+  it('gets expected outputs for site with custom domain name', async () => {
+    const baseUrl = 'https://www.example.com/'
+
+    outputPagesBaseUrl(new URL(baseUrl))
+
+    expect(core.setOutput).toHaveBeenCalledWith('base_url', 'https://www.example.com')
+    expect(core.setOutput).toHaveBeenCalledWith('origin', 'https://www.example.com')
+    expect(core.setOutput).toHaveBeenCalledWith('host', 'www.example.com')
+    expect(core.setOutput).toHaveBeenCalledWith('base_path', '')
+  })
+})

src/remove-trailing-slash.js

@@ -0,0 +1,3 @@
+module.exports = function removeTrailingSlash(str) {
+  return str.endsWith('/') ? str.slice(0, -1) : str
+}

src/set-pages-config.js

@@ -0,0 +1,100 @@
+const core = require('@actions/core')
+const { ConfigParser } = require('./config-parser')
+const removeTrailingSlash = require('./remove-trailing-slash')
+
+const SUPPORTED_FILE_EXTENSIONS = ['.js', '.cjs', '.mjs']
+
+// Return the settings to be passed to a {ConfigParser} for a given static site generator,
+// optional configuration file path, and a Pages siteUrl value to inject
+function getConfigParserSettings({ staticSiteGenerator, generatorConfigFile, siteUrl }) {
+  let { pathname: path, origin } = siteUrl
+
+  switch (staticSiteGenerator) {
+    case 'nuxt':
+      return {
+        configurationFile: generatorConfigFile || './nuxt.config.js',
+        blankConfigurationFile: `${__dirname}/blank-configurations/nuxt.js`,
+        properties: {
+          // Configure a base path on the router
+          'router.base': path,
+
+          // Set the target to static too
+          // https://nuxtjs.org/docs/configuration-glossary/configuration-target/
+          target: 'static'
+        }
+      }
+    case 'next':
+      // Next does not want a trailing slash
+      path = removeTrailingSlash(path)
+
+      return {
+        configurationFile: generatorConfigFile || './next.config.js',
+        blankConfigurationFile: `${__dirname}/blank-configurations/next.js`,
+        properties: {
+          // Configure a base path
+          basePath: path,
+
+          // Disable server side image optimization too
+          // https://nextjs.org/docs/api-reference/next/image#unoptimized
+          'experimental.images.unoptimized': true,
+          // No longer experimental as of Next.js v12.3.0
+          'images.unoptimized': true
+        }
+      }
+    case 'gatsby':
+      return {
+        configurationFile: generatorConfigFile || './gatsby-config.js',
+        blankConfigurationFile: `${__dirname}/blank-configurations/gatsby.js`,
+        properties: {
+          // Configure a path prefix
+          pathPrefix: path,
+          // Configure a site url
+          'siteMetadata.siteUrl': origin
+        }
+      }
+    case 'sveltekit':
+      // SvelteKit does not want a trailing slash
+      path = removeTrailingSlash(path)
+
+      return {
+        configurationFile: generatorConfigFile || './svelte.config.js',
+        blankConfigurationFile: `${__dirname}/blank-configurations/sveltekit.js`,
+        properties: {
+          // Configure a base path
+          'kit.paths.base': path,
+          // Configure a prerender origin
+          'kit.prerender.origin': origin
+        }
+      }
+    default:
+      throw `Unsupported static site generator: ${staticSiteGenerator}`
+  }
+}
+
+// Inject Pages configuration in a given static site generator's configuration file
+function setPagesConfig({ staticSiteGenerator, generatorConfigFile, siteUrl }) {
+  try {
+    // Parse the configuration file and try to inject the Pages configuration in it
+    const settings = getConfigParserSettings({ staticSiteGenerator, generatorConfigFile, siteUrl })
+    new ConfigParser(settings).injectAll()
+  } catch (error) {
+    const isSupportedFileExtension = SUPPORTED_FILE_EXTENSIONS.some(ext => generatorConfigFile.endsWith(ext))
+
+    // Logging
+    if (!isSupportedFileExtension) {
+      core.warning(
+        `Unsupported configuration file extension. Currently supported extensions: ${SUPPORTED_FILE_EXTENSIONS.map(
+          ext => JSON.stringify(ext)
+        ).join(', ')}`,
+        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
+      )
+    }
+  }
+}
+
+module.exports = { getConfigParserSettings, setPagesConfig }

src/set-pages-config.test.js

@@ -0,0 +1,115 @@
+const fs = require('fs')
+const path = require('path')
+const core = require('@actions/core')
+
+const { getConfigParserSettings } = require('./set-pages-config')
+const { ConfigParser } = require('./config-parser')
+const { getTempFolder, compareFiles } = require('./test-helpers')
+
+// Get the temp folder
+const tempFolder = getTempFolder()
+
+const SUPPORTED_GENERATORS = ['next', 'nuxt', 'gatsby', 'sveltekit']
+const SUPPORTED_FILE_EXTENSIONS = ['.js', '.cjs', '.mjs']
+const IS_BLANK_CONFIG_FILE_REGEX = new RegExp(
+  '^blank\\.(' + SUPPORTED_FILE_EXTENSIONS.map(ext => ext.slice(1)).join('|') + ')$'
+)
+
+function isBlankConfigFileName(fileName) {
+  return IS_BLANK_CONFIG_FILE_REGEX.test(fileName)
+}
+
+// Test suite
+describe('configParser', () => {
+  beforeEach(() => {
+    jest.restoreAllMocks()
+
+    // Mock error/warning/info/debug to silence their output
+    jest.spyOn(core, 'error').mockImplementation(jest.fn())
+    jest.spyOn(core, 'warning').mockImplementation(jest.fn())
+    jest.spyOn(core, 'info').mockImplementation(jest.fn())
+    jest.spyOn(core, 'debug').mockImplementation(jest.fn())
+  })
+
+  // Iterate over the static site generators
+  SUPPORTED_GENERATORS.forEach(staticSiteGenerator => {
+    // Folder containing the fixtures for a given static site generator
+    const fixtureFolder = `${__dirname}/fixtures/${staticSiteGenerator}`
+
+    // Get fixture files, excluding expected results
+    const configurationFiles = fs.readdirSync(fixtureFolder).filter(filename => !filename.includes('.expected.'))
+
+    // Create test siteUrl
+    const siteUrl = new URL('https://configure-pages.github.io/docs/')
+
+    // Iterate over the fixtures, outputting to default configuration file path
+    const defaultFileExtension = '.js'
+    configurationFiles
+      .filter(filename => filename.endsWith(defaultFileExtension))
+      .forEach(configurationFile => {
+        it(`injects path properly for ${staticSiteGenerator} in ${configurationFile} to default configuration file`, async () => {
+          // Copy the source fixture to a temp file
+          const fixtureSourceFile = `${fixtureFolder}/${configurationFile}`
+          const fixtureTargetFile = `${tempFolder}/${configurationFile}`
+          if (!isBlankConfigFileName(configurationFile)) {
+            fs.copyFileSync(fixtureSourceFile, fixtureTargetFile)
+          } else if (fs.existsSync(fixtureTargetFile)) {
+            fs.rmSync(fixtureTargetFile)
+          }
+
+          // Get settings for the static site generator
+          const settings = getConfigParserSettings({ staticSiteGenerator, siteUrl })
+          // Update the settings
+          settings.configurationFile = fixtureTargetFile
+          // Do the injection
+          new ConfigParser(settings).injectAll()
+
+          // Read the expected file
+          const expectedFile = `${fixtureFolder}/${path.basename(
+            configurationFile,
+            defaultFileExtension
+          )}.expected${defaultFileExtension}`
+
+          // Compare the actual and expected files
+          compareFiles(settings.configurationFile, expectedFile)
+        })
+      })
+
+    SUPPORTED_FILE_EXTENSIONS.forEach(fileExtension => {
+      // Iterate over the fixtures, outputting to specified configuration file path
+      configurationFiles
+        .filter(filename => filename.endsWith(fileExtension))
+        .forEach(configurationFile => {
+          it(`injects path properly for ${staticSiteGenerator} in ${configurationFile} to specified *${fileExtension} configuration file`, async () => {
+            // Copy the source fixture to a temp file
+            const fixtureSourceFile = `${fixtureFolder}/${configurationFile}`
+            const fixtureTargetFile = `${tempFolder}/${configurationFile}`
+            if (!isBlankConfigFileName(configurationFile)) {
+              fs.copyFileSync(fixtureSourceFile, fixtureTargetFile)
+            } else if (fs.existsSync(fixtureTargetFile)) {
+              fs.rmSync(fixtureTargetFile)
+            }
+
+            // Get settings for the static site generator
+            const settings = getConfigParserSettings({
+              staticSiteGenerator,
+              generatorConfigFile: fixtureTargetFile,
+              siteUrl
+            })
+
+            // Do the injection
+            new ConfigParser(settings).injectAll()
+
+            // Read the expected file
+            const expectedFile = `${fixtureFolder}/${path.basename(
+              configurationFile,
+              fileExtension
+            )}.expected${fileExtension}`
+
+            // Compare the actual and expected files
+            compareFiles(settings.configurationFile, expectedFile)
+          })
+        })
+    })
+  })
+})

src/set-pages-path.js

@@ -1,68 +0,0 @@
-const core = require('@actions/core')
-const {ConfigParser} = require('./config-parser')
-
-// Return the settings to be passed to a {ConfigParser} for a given
-// static site generator and a Pages path value to inject
-function getConfigParserSettings(staticSiteGenerator, path) {
-  switch (staticSiteGenerator) {
-    case 'nuxt':
-      return {
-        configurationFile: './nuxt.config.js',
-        blankConfigurationFile: `${__dirname}/blank-configurations/nuxt.js`,
-        properties: {
-          // Configure a base path on the router
-          'router.base': path,
-
-          // Set the target to static too
-          // https://nuxtjs.org/docs/configuration-glossary/configuration-target/
-          target: 'static'
-        }
-      }
-    case 'next':
-      // Next does not want a trailing slash
-      if (path.endsWith('/')) {
-        path = path.slice(0, -1)
-      }
-
-      return {
-        configurationFile: './next.config.js',
-        blankConfigurationFile: `${__dirname}/blank-configurations/next.js`,
-        properties: {
-          // Configure a base path
-          basePath: path,
-
-          // Disable server side image optimization too
-          // https://nextjs.org/docs/api-reference/next/image#unoptimized
-          'experimental.images.unoptimized': true
-        }
-      }
-    case 'gatsby':
-      return {
-        configurationFile: './gatsby-config.js',
-        blankConfigurationFile: `${__dirname}/blank-configurations/gatsby.js`,
-        properties: {
-          // Configure a path prefix
-          pathPrefix: path
-        }
-      }
-    default:
-      throw `Unsupported static site generator: ${staticSiteGenerator}`
-  }
-}
-
-// Inject Pages configuration in a given static site generator's configuration file
-function setPagesPath({staticSiteGenerator, path}) {
-  try {
-    // Parse the configuration file and try to inject the Pages configuration in it
-    const settings = getConfigParserSettings(staticSiteGenerator, path)
-    new ConfigParser(settings).injectAll()
-  } catch (error) {
-    // Logging
-    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 ${path}. Please ensure your framework is configured to generate relative links appropriately.`,
-      error
-    )
-  }
-}
-
-module.exports = {getConfigParserSettings, setPagesPath}

src/set-pages-path.test.js

@@ -1,53 +0,0 @@
-const fs = require('fs')
-const path = require('path')
-
-const {getConfigParserSettings} = require('./set-pages-path')
-const {ConfigParser} = require('./config-parser')
-const {getTempFolder, compareFiles} = require('./test-helpers')
-
-// Get the temp folder
-const tempFolder = getTempFolder()
-
-// Test suite
-describe('configParser', () => {
-  // Iterate over the static site generators
-  ;['next', 'nuxt', 'gatsby'].forEach(staticSiteGenerator => {
-    // Folder containing the fixtures for a given static site generator
-    const fixtureFolder = `${__dirname}/fixtures/${staticSiteGenerator}`
-
-    // Iterate over the fixtures
-    fs.readdirSync(fixtureFolder).forEach(configurationFile => {
-      // Ignore expectation
-      if (configurationFile.endsWith('.expected.js')) {
-        return
-      }
-
-      it(`Inject path properly for ${staticSiteGenerator} in ${configurationFile}`, async () => {
-        // Get settings for the static site generator
-        const settings = getConfigParserSettings(staticSiteGenerator, '/docs/')
-
-        // Copy the source fixture to a temp file
-        const fixtureSourceFile = `${fixtureFolder}/${configurationFile}`
-        const fixtureTargetFile = `${tempFolder}/${configurationFile}`
-        if (configurationFile != 'blank.js') {
-          fs.copyFileSync(fixtureSourceFile, fixtureTargetFile)
-        } else if (fs.existsSync(fixtureTargetFile)) {
-          fs.rmSync(fixtureTargetFile)
-        }
-
-        // Update the settings and do the injection
-        settings.configurationFile = fixtureTargetFile
-        new ConfigParser(settings).injectAll()
-
-        // Read the expected file
-        const expectedFile = `${fixtureFolder}/${path.basename(
-          configurationFile,
-          '.js'
-        )}.expected.js`
-
-        // Compare the actual and expected files
-        compareFiles(settings.configurationFile, expectedFile)
-      })
-    })
-  })
-})

src/test-helpers.js

@@ -6,7 +6,7 @@ const assert = require('assert')
 function getTempFolder() {
   const tmpFolder = `${__dirname}/fixtures/tmp`
   if (!fs.existsSync(tmpFolder)) {
-    fs.mkdirSync(tmpFolder, {recursive: true})
+    fs.mkdirSync(tmpFolder, { recursive: true })
   }
   return tmpFolder
 }
@@ -14,19 +14,21 @@ function getTempFolder() {
 // Read a JavaScript file and return it formatted
 function formatFile(file) {
   const fileContent = fs.readFileSync(file, 'utf8')
-  return prettier.format(fileContent, {
-    parser: 'espree',
+  return prettier
+    .format(fileContent, {
+      parser: 'espree',
 
-    // Prettier options
-    printWidth: 80,
-    tabWidth: 2,
-    useTabs: false,
-    semi: false,
-    singleQuote: true,
-    trailingComma: 'none',
-    bracketSpacing: false,
-    arrowParens: 'avoid'
-  }).trim()
+      // Prettier options
+      printWidth: 120,
+      tabWidth: 2,
+      useTabs: false,
+      semi: false,
+      singleQuote: true,
+      trailingComma: 'none',
+      bracketSpacing: true,
+      arrowParens: 'avoid'
+    })
+    .trim()
 }
 
 // Compare two JavaScript files
@@ -34,4 +36,4 @@ function compareFiles(actualFile, expectedFile) {
   assert.equal(formatFile(actualFile), formatFile(expectedFile))
 }
 
-module.exports = {getTempFolder, formatFile, compareFiles}
+module.exports = { getTempFolder, formatFile, compareFiles }