Skip to content

Create Artifact Metadata Storage Record on registry push#313

Merged
malancas merged 37 commits intomainfrom
create-storage-records-on-registry-push
Dec 18, 2025
Merged

Create Artifact Metadata Storage Record on registry push#313
malancas merged 37 commits intomainfrom
create-storage-records-on-registry-push

Conversation

@malancas
Copy link
Contributor

@malancas malancas commented Dec 10, 2025

Adds support for creating artifact storage records if the push-to-registry and create-storage-record options are set to true. If the Action fails to create a storage record, it gracefully handles the error and continues.

Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
malancas and others added 7 commits December 15, 2025 11:17
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
…actions/attest into create-storage-records-on-registry-push
Signed-off-by: Meredith Lancaster <malancas@github.com>
malancas and others added 9 commits December 16, 2025 07:32
Signed-off-by: Meredith Lancaster <malancas@github.com>
…actions/attest into create-storage-records-on-registry-push
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
@malancas malancas marked this pull request as ready for review December 17, 2025 23:44
@malancas malancas requested a review from a team as a code owner December 17, 2025 23:44
Copilot AI review requested due to automatic review settings December 17, 2025 23:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for creating artifact metadata storage records when pushing attestations to a registry. The feature is controlled by a new create-storage-record input that defaults to true when push-to-registry is enabled.

Key changes:

  • New create-storage-record input parameter and corresponding logic to create storage records via the @actions/attest library
  • Graceful error handling for storage record creation failures (logs warnings instead of failing the workflow)
  • New storage-record-ids output to expose the created storage record IDs

Reviewed changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/main.ts Added createStorageRecord input field, output handling for storage record IDs, and logging for created storage records
src/index.ts Added input parsing for the new create-storage-record parameter
src/attest.ts Implemented storage record creation logic with protocol validation, error handling, and integration with the @actions/attest library
package.json Updated version to 3.1.0 and bumped @actions/attest dependency to ^2.1.0
action.yml Added create-storage-record input definition and storage-record-ids output definition
tests/main.test.ts Added test coverage for successful storage record creation and graceful failure handling
README.md Updated documentation to describe the new feature, required permissions, and outputs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

malancas and others added 5 commits December 17, 2025 15:48
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Signed-off-by: Meredith Lancaster <malancas@github.com>
Copy link
Collaborator

@bdehamer bdehamer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need run npm run bundle and check in the result /dist changes, but this looks good otherwise.

Signed-off-by: Meredith Lancaster <malancas@github.com>
@malancas malancas requested a review from bdehamer December 18, 2025 18:16
@malancas malancas merged commit 7667f58 into main Dec 18, 2025
5 checks passed
@malancas malancas deleted the create-storage-records-on-registry-push branch December 18, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants