v1.11.1 release

update debug build
Fixes #262 normalize artifact paths
2022-10-04 12:30:45 -04:00 · 2022-10-04 08:09:17 -04:00 · 2022-10-04 08:08:09 -04:00 · 2022-10-02 18:19:43 -04:00 · 2022-10-02 17:35:42 -04:00 · 2022-10-02 15:15:54 -04:00
32 changed files with 7286 additions and 3595 deletions
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -11,7 +11,7 @@ jobs:
  check_pr:
    runs-on: ubuntu-latest
    steps:
-      - uses: actions/checkout@v2.4.0
+      - uses: actions/checkout@v3

      - name: "yarn install"
        run: yarn install
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -5,7 +5,7 @@ jobs:
  check_pr:
    runs-on: ubuntu-latest
    steps:
-    - uses: actions/checkout@v2.4.0
+    - uses: actions/checkout@v3

    - name: "yarn install"
      run: yarn install
--- a/.gitignore
+++ b/.gitignore
@@ -93,3 +93,6 @@ fabric.properties
 .idea/**/markdown-navigator/

 # End of https://www.gitignore.io/api/webstorm
+
+# Coverage
+coverage
--- a/README.md
+++ b/README.md
@@ -3,36 +3,40 @@
 This action will create a GitHub release and optionally upload an artifact to it.

 ## Action Inputs
- **allowUpdates**: An optional flag which indicates if we should update a release if it already exists. Defaults to `false`.
- **artifactErrorsFailBuild**: An optional flag which indicates if artifact read or upload errors should fail the build.  
- **artifact** (deprecated): An optional set of paths representing artifacts to upload to the release. This may be a single path or a comma delimited list of paths (or globs).
- **artifacts**: An optional set of paths representing artifacts to upload to the release. This may be a single path or a comma delimited list of paths (or globs).
- **artifactContentType**: The content type of the artifact. Defaults to `raw`.
- **body**: An optional body for the release.
- **bodyFile**: An optional body file for the release. This should be the path to the file.
- **commit**: An optional commit reference. This will be used to create the tag if it does not exist.
- **discussionCategory**: When provided this will generate a discussion of the specified category. The category must exist otherwise this will cause the action to fail. This isn't used with draft releases.
- **draft**: Optionally marks this release as a draft release. Set to `true` to enable.
- **generateReleaseNotes**: Indicates if release notes should be automatically generated. Set to `true` to enable.
- **name**: An optional name for the release. If this is omitted the tag will be used.
- **omitBody**: Indicates if the release body should be omitted.
- **omitBodyDuringUpdate**: Indicates if the release body should be omitted during updates. The body will still be applied for newly created releases. This will preserve the existing body during updates.
- **omitName**: Indicates if the release name should be omitted.
- **omitNameDuringUpdate**: Indicates if the release name should be omitted during updates. The name will still be applied for newly created releases. This will preserve the existing name during updates.
- **omitPrereleaseDuringUpdate**: Indicates if the prerelease flag should be omitted during updates. The prerelease flag will still be applied for newly created releases.
-  This will preserve the existing prerelease state during updates.
- **owner**: Optionally specify the owner of the repo where the release should be generated. Defaults to current repo's owner. Example - `ncipollo`.
- **prerelease**: Optionally marks this release as prerelease. Set to true to enable.
- **removeArtifacts**: Indicates if existing release artifacts should be removed. Defaults to `false`.
- **replacesArtifacts**: Indicates if existing release artifacts should be replaced. Defaults to `true`.
- **repo**: Optionally specify the repo where the release should be generated. Defaults to current repo. Example - `release-action`.
- **tag**: An optional tag for the release. If this is omitted the git ref will be used (if it is a tag).
- **token**: The GitHub token. Typically, this will be `${{ secrets.GITHUB_TOKEN }}`. If you are using a personal access token it should have access to the `repo` scope.
+| Input name                 | Description                                                                               	                                                                                                                                                         | Required 	 | Default Value        |
+|----------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|------------|----------------------|
+| allowUpdates               | An optional flag which indicates if we should update a release if it already exists. Defaults to false.                                                                                                                                             | false      | ""                   |
+| artifactErrorsFailBuild    | An optional flag which indicates if artifact read or upload errors should fail the build.                                                                                                                                                           | false      | ""                   |
+| artifacts                  | An optional set of paths representing artifacts to upload to the release. This may be a single path or a comma delimited list of paths (or globs)                                                                                                   | false      | ""                   |
+| artifactContentType        | The content type of the artifact. Defaults to raw                                                                                                                                                                                                   | false      | ""                   |
+| body                       | An optional body for the release.                                                                                                                                                                                                                   | false      | ""                   |
+| bodyFile                   | An optional body file for the release. This should be the path to the file.                                                                                                                                                                         | false      | ""                   |
+| commit                     | An optional commit reference. This will be used to create the tag if it does not exist.                                                                                                                                                             | false      | ""                   |
+| discussionCategory         | When provided this will generate a discussion of the specified category. The category must exist otherwise this will cause the action to fail. This isn't used with draft releases                                                                  | false      | ""                   |
+| draft                      | Optionally marks this release as a draft release. Set to true to enable.                                                                                                                                                                            | false      | ""                   |
+| generateReleaseNotes       | Indicates if release notes should be automatically generated.                                                                                                                                                                                       | false      | false                |
+| name                       | An optional name for the release. If this is omitted the tag will be used.                                                                                                                                                                          | false      | ""                   |
+| omitBody                   | Indicates if the release body should be omitted.                                                                                                                                                                                                    | false      | false                |
+| omitBodyDuringUpdate       | Indicates if the release body should be omitted during updates. The body will still be applied for newly created releases. This will preserve the existing body during updates.                                                                     | false      | false                |
+| omitDraftDuringUpdate      | Indicates if the draft flag should be omitted during updates. The draft flag will still be applied for newly created releases. This will preserve the existing draft state during updates.                                                          | false      | false                |
+| omitName                   | Indicates if the release name should be omitted.                                                                                                                                                                                                    | false      | false                |
+| omitNameDuringUpdate       | Indicates if the release name should be omitted during updates. The name will still be applied for newly created releases. This will preserve the existing name during updates.                                                                     | false      | false                |
+| omitPrereleaseDuringUpdate | Indicates if the prerelease flag should be omitted during updates. The prerelease flag will still be applied for newly created releases. This will preserve the existing prerelease state during updates.                                           | false      | false                |
+| owner                      | Optionally specify the owner of the repo where the release should be generated. Defaults to current repo's owner.                                                                                                                                   | false      | "current repo owner" |
+| prerelease                 | Optionally marks this release as prerelease. Set to true to enable.                                                                                                                                                                                 | false      | ""                   |
+| removeArtifacts            | Indicates if existing release artifacts should be removed.                                                                                                                                                                                          | false      | false                |
+| replacesArtifacts          | Indicates if existing release artifacts should be replaced.                                                                                                                                                                                         | false      | true                 |
+| repo                       | Optionally specify the repo where the release should be generated.                                                                                                                                                                                  | false      | current repo         |
+| tag                        | An optional tag for the release. If this is omitted the git ref will be used (if it is a tag).                                                                                                                                                      | false      | ""                   |
+| token                      | The GitHub token. This will default to the GitHub app token. This is primarily useful if you want to use your personal token (for targeting other repos, etc).  If you are using a personal access token it should have access to the `repo` scope. | false      | github.token         |
+| updateOnlyUnreleased       | When allowUpdates is enabled, this will fail the action if the release it is updating is not a draft or a prerelease.                                                                                                                               | false      | false                |

 ## Action Outputs
- **id**: The identifier of the created release.
- **html_url**: The HTML URL of the release.
- **upload_url**: The URL for uploading assets to the release.
+| Output name | Description                                   |
+|-------------|-----------------------------------------------|
+| id          | The identifier of the created release.        |
+| html_url    | The HTML URL of the release.                  |
+| upload_url  | The URL for uploading assets to the release.  |

 ## Example
 This example will create a release when a tag is pushed:
@@ -49,16 +53,19 @@ jobs:

  build:
    runs-on: ubuntu-latest
+    permissions:
+      contents: write
    steps:
    - uses: actions/checkout@v2
    - uses: ncipollo/release-action@v1
      with:
        artifacts: "release.tar.gz,foo/*.txt"
        bodyFile: "body.md"
-        token: ${{ secrets.GITHUB_TOKEN }}
+        token: ${{ secrets.YOUR_GITHUB_TOKEN }}

 ```

 ## Notes
 - You must provide a tag either via the action input or the git ref (i.e push / create a tag). If you do not provide a tag the action will fail.
 - If the tag of the release you are creating does not yet exist, you should set both the `tag` and `commit` action inputs. `commit` can point to a commit hash or a branch name (ex - `main`).
+- In the example above only required permissions for the action specified (which is `contents: write`). If you add other actions to the same workflow you should expand `permissions` block accordingly.
--- a/tests/Action.test.ts
+++ b/tests/Action.test.ts
@@ -22,20 +22,22 @@ const artifacts = [
 ]

 const createBody = 'createBody'
+const createDraft = true
 const createName = 'createName'
 const commit = 'commit'
 const discussionCategory = 'discussionCategory'
-const draft = true
 const generateReleaseNotes = true
 const id = 100
 const createPrerelease = true
-const updatePrerelease = false
 const releaseId = 101
 const replacesArtifacts = true
 const tag = 'tag'
 const token = 'token'
 const updateBody = 'updateBody'
+const updateDraft = false
 const updateName = 'updateName'
+const updatePrerelease = false
+const updateOnlyUnreleased = false
 const url = 'http://api.example.com'

 describe("Action", () => {
@@ -56,7 +58,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease)
@@ -76,7 +78,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease)
@@ -101,7 +103,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease
@@ -121,7 +123,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease
@@ -164,7 +166,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease
@@ -215,7 +217,7 @@ describe("Action", () => {
            updateBody,
            commit,
            discussionCategory,
-            draft,
+            updateDraft,
            updateName,
            updatePrerelease
        )
@@ -239,7 +241,7 @@ describe("Action", () => {
            createBody,
            commit,
            discussionCategory,
-            draft,
+            createDraft,
            generateReleaseNotes,
            createName,
            createPrerelease
@@ -266,7 +268,7 @@ describe("Action", () => {
            updateBody,
            commit,
            discussionCategory,
-            draft,
+            updateDraft,
            updateName,
            updatePrerelease
        )
@@ -285,7 +287,7 @@ describe("Action", () => {
            updateBody,
            commit,
            discussionCategory,
-            draft,
+            updateDraft,
            updateName,
            updatePrerelease
        )
@@ -304,7 +306,7 @@ describe("Action", () => {
            updateBody,
            commit,
            discussionCategory,
-            draft,
+            updateDraft,
            updateName,
            updatePrerelease
        )
@@ -316,7 +318,9 @@ describe("Action", () => {
        expect(applyReleaseDataMock).toBeCalledWith({id: releaseId, upload_url: url})
    }

-    function createAction(allowUpdates: boolean, hasArtifact: boolean, removeArtifacts: boolean = false): Action {
+    function createAction(allowUpdates: boolean,
+                          hasArtifact: boolean,
+                          removeArtifacts: boolean = false): Action {
        let inputArtifact: Artifact[]
        if (hasArtifact) {
            inputArtifact = artifacts
@@ -362,11 +366,11 @@ describe("Action", () => {
                allowUpdates: allowUpdates,
                artifactErrorsFailBuild: true,
                artifacts: inputArtifact,
+                createdDraft: createDraft,
                createdReleaseBody: createBody,
                createdReleaseName: createName,
                commit: commit,
                discussionCategory: discussionCategory,
-                draft: draft,
                generateReleaseNotes: true,
                owner: "owner",
                createdPrerelease: createPrerelease,
@@ -375,9 +379,11 @@ describe("Action", () => {
                repo: "repo",
                tag: tag,
                token: token,
+                updatedDraft: updateDraft,
                updatedReleaseBody: updateBody,
                updatedReleaseName: updateName,
-                updatedPrerelease: updatePrerelease
+                updatedPrerelease: updatePrerelease,
+                updateOnlyUnreleased: updateOnlyUnreleased
            }
        })
        const MockOutputs = jest.fn<Outputs, any>(() => {
--- a/tests/ArtifactUploader.test.ts
+++ b/tests/ArtifactUploader.test.ts
@@ -18,6 +18,7 @@ const uploadMock = jest.fn()

 jest.mock('fs', () => {
    return {
+        promises: {},
        readFileSync: () => fileContents,
        statSync: () => {
            return {size: contentLength}
--- a/tests/Inputs.test.ts
+++ b/tests/Inputs.test.ts
@@ -108,7 +108,7 @@ describe('Inputs', () => {

        it('returns input.artifacts with default contentType', () => {
            mockGetInput.mockReturnValueOnce('art1')
-                .mockReturnValueOnce('raw')
+                .mockReturnValueOnce('')
                .mockReturnValueOnce('false')

            expect(inputs.artifacts).toEqual(artifacts)
@@ -128,6 +128,17 @@ describe('Inputs', () => {
        })
    })

+    describe('createdDraft', () => {
+        it('returns false', () => {
+            expect(inputs.createdDraft).toBe(false)
+        })
+
+        it('returns true', () => {
+            mockGetInput.mockReturnValue('true')
+            expect(inputs.createdDraft).toBe(true)
+        })
+    })
+
    describe('createdReleaseBody', () => {
        it('returns input body', () => {
            mockGetInput
@@ -174,7 +185,7 @@ describe('Inputs', () => {
            mockGetInput
                .mockReturnValueOnce('true')
                .mockReturnValueOnce('name')
-            expect(inputs.createdReleaseBody).toBeUndefined()
+            expect(inputs.createdReleaseName).toBeUndefined()
        })

        it('returns tag', () => {
@@ -198,17 +209,6 @@ describe('Inputs', () => {
        })
    })

-    describe('draft', () => {
-        it('returns false', () => {
-            expect(inputs.draft).toBe(false)
-        })
-
-        it('returns true', () => {
-            mockGetInput.mockReturnValue('true')
-            expect(inputs.draft).toBe(true)
-        })
-    })
-
    describe('generateReleaseNotes', () => {
        it('returns returns true', function () {
            mockGetInput.mockReturnValue("true")
@@ -294,10 +294,38 @@ describe('Inputs', () => {
            expect(inputs.tag).toBe('sha-tag')
        })
        it('throws if no tag', () => {
+            context.ref = ""
            expect(() => inputs.tag).toThrow()
        })
    })

+    describe('updatedDraft', () => {
+        it('returns false', () => {
+            expect(inputs.updatedDraft).toBe(false)
+        })
+
+        it('returns true', () => {
+            mockGetInput
+                .mockReturnValueOnce('false')
+                .mockReturnValue('true')
+            expect(inputs.updatedDraft).toBe(true)
+        })
+
+        it('returns true when omitted is blank', () => {
+            mockGetInput
+                .mockReturnValueOnce('')
+                .mockReturnValue('true')
+            expect(inputs.updatedDraft).toBe(true)
+        })
+
+        it('returns undefined when omitted for update', () => {
+            mockGetInput
+                .mockReturnValueOnce('true')
+                .mockReturnValueOnce('true')
+            expect(inputs.updatedDraft).toBeUndefined()
+        })
+    })
+
    describe('updatedReleaseBody', () => {
        it('returns input body', () => {
            mockGetInput
@@ -402,6 +430,17 @@ describe('Inputs', () => {
        })
    })

+    describe('updateOnlyUnreleased', () => {
+        it('returns false', () => {
+            expect(inputs.updateOnlyUnreleased).toBe(false)
+        })
+
+        it('returns true', () => {
+            mockGetInput.mockReturnValueOnce('true')
+            expect(inputs.updateOnlyUnreleased).toBe(true)
+        })
+    })
+
    function createGlobber(): ArtifactGlobber {
        const MockGlobber = jest.fn<ArtifactGlobber, any>(() => {
            return {
--- a/tests/Integration.test.ts
+++ b/tests/Integration.test.ts
@@ -41,11 +41,11 @@ describe.skip('Integration Test', () => {
                allowUpdates: true,
                artifactErrorsFailBuild: false,
                artifacts: artifacts(),
+                createdDraft: false,
                createdReleaseBody: "This release was generated by release-action's integration test",
                createdReleaseName: "Releases Action Integration Test",
                commit: undefined,
                discussionCategory: 'Release',
-                draft: false,
                generateReleaseNotes: true,
                owner: "ncipollo",
                createdPrerelease: false,
@@ -54,9 +54,11 @@ describe.skip('Integration Test', () => {
                repo: "actions-playground",
                tag: "release-action-test",
                token: getToken(),
+                updatedDraft: false,
                updatedReleaseBody: "This release was generated by release-action's integration test",
                updatedReleaseName: "Releases Action Integration Test",
-                updatedPrerelease: false
+                updatedPrerelease: false,
+                updateOnlyUnreleased: false
            }
        })
        return new MockInputs();
--- a/tests/ReleaseValidator.test.ts
+++ b/tests/ReleaseValidator.test.ts
@@ -0,0 +1,74 @@
+import {ReleaseValidator} from "../src/ReleaseValidator";
+
+describe("validateReleaseUpdate", () => {
+    describe("updateOnlyUnreleased is disabled", () => {
+        const validator = new ReleaseValidator(false)
+        it('should not throw', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+    })
+    describe("updateOnlyUnreleased is enabled", () => {
+        const validator = new ReleaseValidator(true)
+        it('should throw if neither draft or prerelease are enabled', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).toThrow()
+        })
+        
+        it('should not throw if draft is enabled', () => {
+            const releaseResponse = {
+                draft: true,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should not throw if prerelease is enabled', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: true,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should not throw if draft & prerelease is enabled', () => {
+            const releaseResponse = {
+                draft: true,
+                prerelease: true,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should default error message release name to release', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: null
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).toThrow(`Tried to update "release" which is neither a draft or prerelease. (updateOnlyUnreleased is on)`)
+        })
+    })
+})
--- a/action.yml
+++ b/action.yml
@@ -59,6 +59,10 @@ inputs:
    description: 'Indicates if the release body should be omitted during updates. The body will still be applied for newly created releases. This will preserve the existing body during updates.'
    required: false
    default: 'false'
+  omitDraftDuringUpdate:
+    description: 'Indicates if the draft flag should be omitted during updates. The draft flag will still be applied for newly created releases. This will preserve the existing draft state during updates.'
+    required: false
+    default: 'false'
  omitName:
    description: 'Indicates if the release name should be omitted.'
    required: false
@@ -90,7 +94,7 @@ inputs:
  repo:
    description: "Optionally specify the repo where the release should be generated. Defaults to current repo"
    required: false
-    default: ''  
+    default: ''
  tag:
    description: 'An optional tag for the release. If this is omitted the git ref will be used (if it is a tag).'
    required: false
@@ -99,6 +103,10 @@ inputs:
    description: 'The Github token.'
    required: false
    default: ${{ github.token }}
+  updateOnlyUnreleased:
+    description: "When allowUpdates is enabled, this will fail the action if the release it is updating is not a draft or a prerelease."
+    required: false
+    default: 'false'
 outputs:
  id:
    description: 'The identifier of the created release.'
@@ -107,7 +115,7 @@ outputs:
  upload_url:
    description: 'The URL for uploading assets to the release.'
 runs:
-  using: 'node12'
+  using: 'node16'
  main: 'dist/index.js'
 branding:
  icon: 'tag'
--- a/dist/index.js
+++ b/dist/index.js
--- a/dist/index.js.map
+++ b/dist/index.js.map
--- a/dist/licenses.txt
+++ b/dist/licenses.txt
@@ -484,28 +484,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.


-concat-map
-MIT
-This software is released under the MIT license:
-
-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.
-
-
 deprecation
 ISC
 The ISC License
@@ -576,7 +554,7 @@ glob
 ISC
 The ISC License

-Copyright (c) Isaac Z. Schlueter and Contributors
+Copyright (c) 2009-2022 Isaac Z. Schlueter and Contributors

 Permission to use, copy, modify, and/or distribute this software for any
 purpose with or without fee is hereby granted, provided that the above
@@ -590,12 +568,6 @@ WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
 ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
 IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

-## Glob Logo
-
-Glob's logo created by Tanya Brassie <http://tanyabrassie.com/>, licensed
-under a Creative Commons Attribution-ShareAlike 4.0 International License
-https://creativecommons.org/licenses/by-sa/4.0/
-

 inflight
 ISC
@@ -665,7 +637,7 @@ minimatch
 ISC
 The ISC License

-Copyright (c) Isaac Z. Schlueter and Contributors
+Copyright (c) 2011-2022 Isaac Z. Schlueter and Contributors

 Permission to use, copy, modify, and/or distribute this software for any
 purpose with or without fee is hereby granted, provided that the above
@@ -725,30 +697,8 @@ ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
 IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.


-path-is-absolute
+tr46
 MIT
-The MIT License (MIT)
-
-Copyright (c) Sindre Sorhus <sindresorhus@gmail.com> (sindresorhus.com)
-
-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.
-

 tunnel
 MIT
@@ -799,6 +749,60 @@ The above copyright notice and this permission notice shall be included in all c
 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.


+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.
+
+
+webidl-conversions
+BSD-2-Clause
+# The BSD 2-Clause License
+
+Copyright (c) 2014, Domenic Denicola
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met:
+
+1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.
+
+2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+
+whatwg-url
+MIT
+The MIT License (MIT)
+
+Copyright (c) 2015–2016 Sebastian Mayr
+
+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.
+
+
 wrappy
 ISC
 The ISC License
--- a/dist/sourcemap-register.js
+++ b/dist/sourcemap-register.js
--- a/jest.config.js
+++ b/jest.config.js
@@ -1,11 +0,0 @@
-module.exports = {
-  clearMocks: true,
-  moduleFileExtensions: ['js', 'ts'],
-  testEnvironment: 'node',
-  testMatch: ['**/*.test.ts'],
-  testRunner: 'jest-circus/runner',
-  transform: {
-    '^.+\\.ts$': 'ts-jest'
-  },
-  verbose: true
-}
--- a/lib/Action.js
+++ b/lib/Action.js
@@ -11,6 +11,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Action = void 0;
 const GithubError_1 = require("./GithubError");
+const ReleaseValidator_1 = require("./ReleaseValidator");
 class Action {
    constructor(inputs, outputs, releases, uploader, artifactDestroyer) {
        this.inputs = inputs;
@@ -18,6 +19,7 @@ class Action {
        this.releases = releases;
        this.uploader = uploader;
        this.artifactDestroyer = artifactDestroyer;
+        this.releaseValidator = new ReleaseValidator_1.ReleaseValidator(inputs.updateOnlyUnreleased);
    }
    perform() {
        return __awaiter(this, void 0, void 0, function* () {
@@ -45,6 +47,8 @@ class Action {
                catch (error) {
                    return yield this.checkForMissingReleaseError(error);
                }
+                // Fail if this isn't an unreleased release & updateOnlyUnreleased is enabled.
+                this.releaseValidator.validateReleaseUpdate(getResponse.data);
                return yield this.updateRelease(getResponse.data.id);
            }
            else {
@@ -64,7 +68,7 @@ class Action {
    }
    updateRelease(id) {
        return __awaiter(this, void 0, void 0, function* () {
-            return yield this.releases.update(id, this.inputs.tag, this.inputs.updatedReleaseBody, this.inputs.commit, this.inputs.discussionCategory, this.inputs.draft, this.inputs.updatedReleaseName, this.inputs.updatedPrerelease);
+            return yield this.releases.update(id, this.inputs.tag, this.inputs.updatedReleaseBody, this.inputs.commit, this.inputs.discussionCategory, this.inputs.updatedDraft, this.inputs.updatedReleaseName, this.inputs.updatedPrerelease);
        });
    }
    static noPublishedRelease(error) {
@@ -93,7 +97,7 @@ class Action {
    }
    createRelease() {
        return __awaiter(this, void 0, void 0, function* () {
-            return yield this.releases.create(this.inputs.tag, this.inputs.createdReleaseBody, this.inputs.commit, this.inputs.discussionCategory, this.inputs.draft, this.inputs.generateReleaseNotes, this.inputs.createdReleaseName, this.inputs.createdPrerelease);
+            return yield this.releases.create(this.inputs.tag, this.inputs.createdReleaseBody, this.inputs.commit, this.inputs.discussionCategory, this.inputs.createdDraft, this.inputs.generateReleaseNotes, this.inputs.createdReleaseName, this.inputs.createdPrerelease);
        });
    }
 }
--- a/lib/ArtifactDestroyer.js
+++ b/lib/ArtifactDestroyer.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
--- a/lib/ArtifactGlobber.js
+++ b/lib/ArtifactGlobber.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
@@ -28,6 +32,7 @@ const Globber_1 = require("./Globber");
 const Artifact_1 = require("./Artifact");
 const untildify_1 = __importDefault(require("untildify"));
 const ArtifactPathValidator_1 = require("./ArtifactPathValidator");
+const PathNormalizer_1 = require("./PathNormalizer");
 class FileArtifactGlobber {
    constructor(globber = new Globber_1.FileGlobber()) {
        this.globber = globber;
@@ -36,6 +41,7 @@ class FileArtifactGlobber {
        const split = /[,\n]/;
        return artifact.split(split)
            .map(path => path.trimStart())
+            .map(path => PathNormalizer_1.PathNormalizer.normalizePath(path))
            .map(path => FileArtifactGlobber.expandPath(path))
            .map(pattern => this.globPattern(pattern, errorsFailBuild))
            .map((globResult) => FileArtifactGlobber.validatePattern(errorsFailBuild, globResult[1], globResult[0]))
--- a/lib/ArtifactPathValidator.js
+++ b/lib/ArtifactPathValidator.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
--- a/lib/ArtifactUploader.js
+++ b/lib/ArtifactUploader.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
--- a/lib/Inputs.js
+++ b/lib/Inputs.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
@@ -50,14 +54,6 @@ class CoreInputs {
        const allow = core.getInput('artifactErrorsFailBuild');
        return allow == 'true';
    }
-    get createdReleaseBody() {
-        if (CoreInputs.omitBody)
-            return undefined;
-        return this.body;
-    }
-    static get omitBody() {
-        return core.getInput('omitBody') == 'true';
-    }
    get body() {
        const body = core.getInput('body');
        if (body) {
@@ -69,6 +65,30 @@ class CoreInputs {
        }
        return '';
    }
+    get createdDraft() {
+        const draft = core.getInput('draft');
+        return draft == 'true';
+    }
+    get createdPrerelease() {
+        const preRelease = core.getInput('prerelease');
+        return preRelease == 'true';
+    }
+    get createdReleaseBody() {
+        if (CoreInputs.omitBody)
+            return undefined;
+        return this.body;
+    }
+    static get omitBody() {
+        return core.getInput('omitBody') == 'true';
+    }
+    get createdReleaseName() {
+        if (CoreInputs.omitName)
+            return undefined;
+        return this.name;
+    }
+    static get omitName() {
+        return core.getInput('omitName') == 'true';
+    }
    get commit() {
        const commit = core.getInput('commit');
        if (commit) {
@@ -76,11 +96,6 @@ class CoreInputs {
        }
        return undefined;
    }
-    get createdReleaseName() {
-        if (CoreInputs.omitName)
-            return undefined;
-        return this.name;
-    }
    get discussionCategory() {
        const category = core.getInput('discussionCategory');
        if (category) {
@@ -88,9 +103,6 @@ class CoreInputs {
        }
        return undefined;
    }
-    static get omitName() {
-        return core.getInput('omitName') == 'true';
-    }
    get name() {
        const name = core.getInput('name');
        if (name) {
@@ -98,10 +110,6 @@ class CoreInputs {
        }
        return this.tag;
    }
-    get draft() {
-        const draft = core.getInput('draft');
-        return draft == 'true';
-    }
    get generateReleaseNotes() {
        const generate = core.getInput('generateReleaseNotes');
        return generate == 'true';
@@ -113,18 +121,6 @@ class CoreInputs {
        }
        return this.context.repo.owner;
    }
-    get createdPrerelease() {
-        const preRelease = core.getInput('prerelease');
-        return preRelease == 'true';
-    }
-    static get omitPrereleaseDuringUpdate() {
-        return core.getInput('omitPrereleaseDuringUpdate') == 'true';
-    }
-    get updatedPrerelease() {
-        if (CoreInputs.omitPrereleaseDuringUpdate)
-            return undefined;
-        return this.createdPrerelease;
-    }
    get removeArtifacts() {
        const removes = core.getInput('removeArtifacts');
        return removes == 'true';
@@ -155,6 +151,22 @@ class CoreInputs {
    get token() {
        return core.getInput('token', { required: true });
    }
+    get updatedDraft() {
+        if (CoreInputs.omitDraftDuringUpdate)
+            return undefined;
+        return this.createdDraft;
+    }
+    static get omitDraftDuringUpdate() {
+        return core.getInput('omitDraftDuringUpdate') == 'true';
+    }
+    get updatedPrerelease() {
+        if (CoreInputs.omitPrereleaseDuringUpdate)
+            return undefined;
+        return this.createdPrerelease;
+    }
+    static get omitPrereleaseDuringUpdate() {
+        return core.getInput('omitPrereleaseDuringUpdate') == 'true';
+    }
    get updatedReleaseBody() {
        if (CoreInputs.omitBody || CoreInputs.omitBodyDuringUpdate)
            return undefined;
@@ -168,6 +180,9 @@ class CoreInputs {
            return undefined;
        return this.name;
    }
+    get updateOnlyUnreleased() {
+        return core.getInput('updateOnlyUnreleased') == 'true';
+    }
    static get omitNameDuringUpdate() {
        return core.getInput('omitNameDuringUpdate') == 'true';
    }
--- a/lib/Main.js
+++ b/lib/Main.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
--- a/lib/Outputs.js
+++ b/lib/Outputs.js
@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
    if (k2 === undefined) k2 = k;
    o[k2] = m[k];
--- a/lib/PathNormalizer.js
+++ b/lib/PathNormalizer.js
@@ -0,0 +1,13 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.PathNormalizer = void 0;
+const path_1 = __importDefault(require("path"));
+class PathNormalizer {
+    static normalizePath(pathString) {
+        return pathString.split(path_1.default.sep).join("/");
+    }
+}
+exports.PathNormalizer = PathNormalizer;
--- a/lib/ReleaseValidator.js
+++ b/lib/ReleaseValidator.js
@@ -0,0 +1,18 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ReleaseValidator = void 0;
+class ReleaseValidator {
+    constructor(updateOnlyUnreleased) {
+        this.updateOnlyUnreleased = updateOnlyUnreleased;
+    }
+    validateReleaseUpdate(releaseResponse) {
+        var _a;
+        if (!this.updateOnlyUnreleased) {
+            return;
+        }
+        if (!releaseResponse.draft && !releaseResponse.prerelease) {
+            throw new Error(`Tried to update "${(_a = releaseResponse.name) !== null && _a !== void 0 ? _a : "release"}" which is neither a draft or prerelease. (updateOnlyUnreleased is on)`);
+        }
+    }
+}
+exports.ReleaseValidator = ReleaseValidator;
--- a/package.json
+++ b/package.json
@@ -7,6 +7,7 @@
  "scripts": {
    "build": "tsc",
    "clean": "rm -rf lib/*",
+    "coverage": "jest --coverage",
    "debug": "yarn clean && yarn install && yarn build && yarn package",
    "package": "ncc build --source-map --license licenses.txt",
    "release": "yarn clean && yarn install --production && yarn build && yarn package",
@@ -23,19 +24,48 @@
  ],
  "author": "GitHub",
  "license": "MIT",
+  "jest": {
+    "clearMocks": true,
+    "collectCoverage": true,
+    "coveragePathIgnorePatterns": [
+      "src/Globber.ts",
+      "src/Releases.ts"
+    ],
+    "coverageThreshold": {
+      "global": {
+        "branches": 95,
+        "functions": 100,
+        "lines": 100,
+        "statements": 100
+      }
+    },
+    "moduleFileExtensions": [
+      "js",
+      "ts"
+    ],
+    "testEnvironment": "node",
+    "testMatch": [
+      "**/*.test.ts"
+    ],
+    "testRunner": "jest-circus/runner",
+    "transform": {
+      "^.+\\.ts$": "ts-jest"
+    },
+    "verbose": true
+  },
  "dependencies": {
-    "@actions/core": "^1.6.0",
-    "@actions/github": "^5.0.0",
-    "@types/glob": "^7.2.0",
-    "glob": "^7.2.0",
+    "@actions/core": "^1.10.0",
+    "@actions/github": "^5.1.1",
+    "@types/glob": "^8.0.0",
+    "glob": "^8.0.3",
    "untildify": "^4.0.0"
  },
  "devDependencies": {
-    "@types/jest": "^27.0.2",
-    "@types/node": "^16.11.6",
-    "jest": "^27.3.1",
-    "jest-circus": "^27.3.1",
-    "ts-jest": "^27.0.7",
-    "typescript": "^4.4.4"
+    "@types/jest": "^28.1.1",
+    "@types/node": "^18.7.23",
+    "jest": "^28.1.1",
+    "jest-circus": "^29.1.2",
+    "ts-jest": "^28.0.5",
+    "typescript": "^4.8.4"
  }
 }
--- a/src/Action.ts
+++ b/src/Action.ts
@@ -10,13 +10,16 @@ import {ArtifactUploader} from "./ArtifactUploader";
 import {GithubError} from "./GithubError";
 import {Outputs} from "./Outputs";
 import {ArtifactDestroyer} from "./ArtifactDestroyer";
+import {ReleaseValidator} from "./ReleaseValidator";

 export class Action {
    private inputs: Inputs
    private outputs: Outputs
    private releases: Releases
-    private artifactDestroyer: ArtifactDestroyer
    private uploader: ArtifactUploader
+    private artifactDestroyer: ArtifactDestroyer
+    
+    private releaseValidator: ReleaseValidator

    constructor(inputs: Inputs,
                outputs: Outputs,
@@ -28,6 +31,7 @@ export class Action {
        this.releases = releases
        this.uploader = uploader
        this.artifactDestroyer = artifactDestroyer
+        this.releaseValidator = new ReleaseValidator(inputs.updateOnlyUnreleased)
    }

    async perform() {
@@ -56,6 +60,9 @@ export class Action {
            } catch (error: any) {
                return await this.checkForMissingReleaseError(error)
            }
+            
+            // Fail if this isn't an unreleased release & updateOnlyUnreleased is enabled.
+            this.releaseValidator.validateReleaseUpdate(getResponse.data)

            return await this.updateRelease(getResponse.data.id)
        } else {
@@ -78,7 +85,7 @@ export class Action {
            this.inputs.updatedReleaseBody,
            this.inputs.commit,
            this.inputs.discussionCategory,
-            this.inputs.draft,
+            this.inputs.updatedDraft,
            this.inputs.updatedReleaseName,
            this.inputs.updatedPrerelease
        )
@@ -113,7 +120,7 @@ export class Action {
            this.inputs.createdReleaseBody,
            this.inputs.commit,
            this.inputs.discussionCategory,
-            this.inputs.draft,
+            this.inputs.createdDraft,
            this.inputs.generateReleaseNotes,
            this.inputs.createdReleaseName,
            this.inputs.createdPrerelease
--- a/src/ArtifactGlobber.ts
+++ b/src/ArtifactGlobber.ts
@@ -3,6 +3,7 @@ import {Globber, FileGlobber} from "./Globber";
 import {Artifact} from "./Artifact";
 import untildify from "untildify";
 import {ArtifactPathValidator} from "./ArtifactPathValidator";
+import {PathNormalizer} from "./PathNormalizer";

 export interface ArtifactGlobber {
    globArtifactString(artifact: string, contentType: string, errorsFailBuild: boolean): Artifact[]
@@ -19,6 +20,7 @@ export class FileArtifactGlobber implements ArtifactGlobber {
        const split = /[,\n]/
        return artifact.split(split)
            .map(path => path.trimStart())
+            .map(path => PathNormalizer.normalizePath(path))
            .map(path => FileArtifactGlobber.expandPath(path))
            .map(pattern => this.globPattern(pattern, errorsFailBuild))
            .map((globResult) => FileArtifactGlobber.validatePattern(errorsFailBuild, globResult[1], globResult[0]))
--- a/src/Inputs.ts
+++ b/src/Inputs.ts
@@ -9,21 +9,23 @@ export interface Inputs {
    readonly artifactErrorsFailBuild: boolean
    readonly artifacts: Artifact[]
    readonly commit?: string
+    readonly createdDraft: boolean
+    readonly createdPrerelease: boolean
    readonly createdReleaseBody?: string
    readonly createdReleaseName?: string
    readonly discussionCategory?: string
-    readonly draft: boolean
    readonly generateReleaseNotes: boolean
    readonly owner: string
-    readonly createdPrerelease: boolean
    readonly removeArtifacts: boolean
    readonly replacesArtifacts: boolean
    readonly repo: string
    readonly tag: string
    readonly token: string
+    readonly updatedDraft?: boolean
    readonly updatedReleaseBody?: string
    readonly updatedReleaseName?: string
    readonly updatedPrerelease?: boolean
+    readonly updateOnlyUnreleased: boolean
 }

 export class CoreInputs implements Inputs {
@@ -61,15 +63,6 @@ export class CoreInputs implements Inputs {
        return allow == 'true'
    }

-    get createdReleaseBody(): string | undefined {
-        if (CoreInputs.omitBody) return undefined
-        return this.body
-    }
-
-    private static get omitBody(): boolean {
-        return core.getInput('omitBody') == 'true'
-    }
-
    private get body(): string | undefined {
        const body = core.getInput('body')
        if (body) {
@@ -84,6 +77,34 @@ export class CoreInputs implements Inputs {
        return ''
    }

+    get createdDraft(): boolean {
+        const draft = core.getInput('draft')
+        return draft == 'true'
+    }
+
+    get createdPrerelease(): boolean {
+        const preRelease = core.getInput('prerelease')
+        return preRelease == 'true'
+    }
+
+    get createdReleaseBody(): string | undefined {
+        if (CoreInputs.omitBody) return undefined
+        return this.body
+    }
+
+    private static get omitBody(): boolean {
+        return core.getInput('omitBody') == 'true'
+    }
+
+    get createdReleaseName(): string | undefined {
+        if (CoreInputs.omitName) return undefined
+        return this.name
+    }
+
+    private static get omitName(): boolean {
+        return core.getInput('omitName') == 'true'
+    }
+
    get commit(): string | undefined {
        const commit = core.getInput('commit')
        if (commit) {
@@ -92,11 +113,6 @@ export class CoreInputs implements Inputs {
        return undefined
    }

-    get createdReleaseName(): string | undefined {
-        if (CoreInputs.omitName) return undefined
-        return this.name
-    }
-
    get discussionCategory(): string | undefined {
        const category = core.getInput('discussionCategory')
        if (category) {
@@ -105,10 +121,6 @@ export class CoreInputs implements Inputs {
        return undefined
    }

-    private static get omitName(): boolean {
-        return core.getInput('omitName') == 'true'
-    }
-
    private get name(): string | undefined {
        const name = core.getInput('name')
        if (name) {
@@ -117,11 +129,6 @@ export class CoreInputs implements Inputs {

        return this.tag
    }
-
-    get draft(): boolean {
-        const draft = core.getInput('draft')
-        return draft == 'true'
-    }
    
    get generateReleaseNotes(): boolean {
        const generate = core.getInput('generateReleaseNotes')
@@ -135,20 +142,7 @@ export class CoreInputs implements Inputs {
        }
        return this.context.repo.owner
    }
-
-    get createdPrerelease(): boolean {
-        const preRelease = core.getInput('prerelease')
-        return preRelease == 'true'
-    }
-
-    private static get omitPrereleaseDuringUpdate(): boolean {
-        return core.getInput('omitPrereleaseDuringUpdate') == 'true'
-    }
-
-    get updatedPrerelease(): boolean | undefined {
-        if (CoreInputs.omitPrereleaseDuringUpdate) return undefined
-        return this.createdPrerelease
-    }
+    
    get removeArtifacts(): boolean {
        const removes = core.getInput('removeArtifacts')
        return removes == 'true'
@@ -185,6 +179,24 @@ export class CoreInputs implements Inputs {
        return core.getInput('token', {required: true})
    }

+    get updatedDraft(): boolean | undefined {
+        if (CoreInputs.omitDraftDuringUpdate) return undefined
+        return this.createdDraft
+    }
+
+    private static get omitDraftDuringUpdate(): boolean {
+        return core.getInput('omitDraftDuringUpdate') == 'true'
+    }
+    
+    get updatedPrerelease(): boolean | undefined {
+        if (CoreInputs.omitPrereleaseDuringUpdate) return undefined
+        return this.createdPrerelease
+    }
+
+    private static get omitPrereleaseDuringUpdate(): boolean {
+        return core.getInput('omitPrereleaseDuringUpdate') == 'true'
+    }
+
    get updatedReleaseBody(): string | undefined {
        if (CoreInputs.omitBody || CoreInputs.omitBodyDuringUpdate) return undefined
        return this.body
@@ -198,6 +210,10 @@ export class CoreInputs implements Inputs {
        if (CoreInputs.omitName || CoreInputs.omitNameDuringUpdate) return undefined
        return this.name
    }
+    
+    get updateOnlyUnreleased(): boolean {
+        return core.getInput('updateOnlyUnreleased') == 'true'
+    }

    private static get omitNameDuringUpdate(): boolean {
        return core.getInput('omitNameDuringUpdate') == 'true'
--- a/src/PathNormalizer.ts
+++ b/src/PathNormalizer.ts
@@ -0,0 +1,7 @@
+import path from "path";
+
+export class PathNormalizer {
+    static normalizePath(pathString: string): string {
+        return pathString.split(path.sep).join("/")
+    } 
+}
--- a/src/ReleaseValidator.ts
+++ b/src/ReleaseValidator.ts
@@ -0,0 +1,20 @@
+export class ReleaseValidator {
+    constructor(private updateOnlyUnreleased: boolean) {
+    }
+
+    validateReleaseUpdate(releaseResponse: ReleaseStageArguments) {
+        if (!this.updateOnlyUnreleased) {
+            return
+        }
+
+        if (!releaseResponse.draft && !releaseResponse.prerelease) {
+            throw new Error(`Tried to update "${releaseResponse.name ?? "release"}" which is neither a draft or prerelease. (updateOnlyUnreleased is on)`)
+        }
+    }
+}
+
+export type ReleaseStageArguments = {
+    draft: boolean
+    name: string | null
+    prerelease: boolean
+}
--- a/yarn.lock
+++ b/yarn.lock