new OCI compat mode flag

Signed-off-by: Brian DeHamer <bdehamer@github.com>
another take on an empty config
2025-04-10 09:41:02 -07:00 · 2025-04-10 08:47:16 -07:00 · 2025-04-09 17:41:13 -07:00 · 2025-04-09 16:41:18 -07:00 · 2025-04-09 14:43:34 -07:00 · 2025-04-09 10:56:02 -07:00
31 changed files with 76407 additions and 78278 deletions
@@ -0,0 +1,10 @@
+rules:
+  document-end: disable
+  document-start:
+    level: warning
+    present: false
+  line-length:
+    level: warning
+    max: 80
+    allow-non-breakable-words: true
+    allow-non-breakable-inline-mappings: true
@@ -1,14 +1,13 @@
 import eslint from '@eslint/js'
 import importplugin from 'eslint-plugin-import'
 import jestplugin from 'eslint-plugin-jest'
-import path from 'node:path'
 import tseslint from 'typescript-eslint'

 export default tseslint.config(
  // Ignore non-project files
  {
    name: 'ignore',
-    ignores: ['.github', 'dist', 'coverage', '**/*.json', 'jest.setup.js', 'eslint.config.mjs']
+    ignores: ['.github', 'dist', 'coverage', '**/*.json', 'jest.setup.js']
  },
  // Use recommended rules from ESLint, TypeScript, and other plugins
  eslint.configs.recommended,
@@ -22,7 +21,7 @@ export default tseslint.config(
    languageOptions: {
      ecmaVersion: 2023,
      parserOptions: {
-        project: [ './tsconfig.lint.json' ]
+        project: ['./.github/linters/tsconfig.json', './tsconfig.json']
      }
    },
    rules: {
@@ -89,13 +88,6 @@ export default tseslint.config(
          allowObject: true
        }
      ]
-    },
-    settings: {
-      'import/resolver': {
-        typescript: {
-          project: './tsconfig.lint.json'
-        }
-      }
    }
  }
 )
@@ -0,0 +1,9 @@
+{
+  "$schema": "https://json.schemastore.org/tsconfig",
+  "extends": "../../tsconfig.json",
+  "compilerOptions": {
+    "noEmit": true
+  },
+  "include": ["../../__tests__/**/*", "../../src/**/*"],
+  "exclude": ["../../dist", "../../node_modules", "../../coverage", "*.json"]
+}
@@ -28,11 +28,11 @@ jobs:
    steps:
      - name: Checkout
        id: checkout
-        uses: actions/checkout@v6.0.2
+        uses: actions/checkout@v4

      - name: Setup Node.js
        id: setup-node
-        uses: actions/setup-node@v6.2.0
+        uses: actions/setup-node@v4
        with:
          node-version-file: .node-version
          cache: npm
@@ -60,7 +60,7 @@ jobs:
      - if: ${{ failure() && steps.diff.outcome == 'failure' }}
        name: Upload Artifact
        id: upload
-        uses: actions/upload-artifact@v6
+        uses: actions/upload-artifact@v4
        with:
          name: dist
          path: dist/
@@ -21,11 +21,11 @@ jobs:
    steps:
      - name: Checkout
        id: checkout
-        uses: actions/checkout@0c366fd6a839edf440554fa01a7085ccba70ac98 # v5.0.1
+        uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1

      - name: Setup Node.js
        id: setup-node
-        uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0
+        uses: actions/setup-node@b39b52d1213e96004bfcb1c61a8a6fa8ab84f3e8 # v4.0.1
        with:
          node-version-file: .node-version
          cache: npm
@@ -58,7 +58,7 @@ jobs:
    steps:
      - name: Checkout
        id: checkout
-        uses: actions/checkout@0c366fd6a839edf440554fa01a7085ccba70ac98 # v5.0.1
+        uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
      - name: Calculate subject digest
        id: subject
        env:
@@ -32,19 +32,19 @@ jobs:
    steps:
      - name: Checkout
        id: checkout
-        uses: actions/checkout@v6.0.2
+        uses: actions/checkout@v4

      - name: Initialize CodeQL
        id: initialize
-        uses: github/codeql-action/init@v4
+        uses: github/codeql-action/init@v3
        with:
          languages: ${{ matrix.language }}
          source-root: src

      - name: Autobuild
        id: autobuild
-        uses: github/codeql-action/autobuild@v4
+        uses: github/codeql-action/autobuild@v3

      - name: Perform CodeQL Analysis
        id: analyze
-        uses: github/codeql-action/analyze@v4
+        uses: github/codeql-action/analyze@v3
@@ -0,0 +1,50 @@
+name: Lint Codebase
+
+on:
+  pull_request:
+    branches:
+      - main
+  push:
+    branches:
+      - main
+
+permissions:
+  contents: read
+  packages: read
+  statuses: write
+
+jobs:
+  lint:
+    name: Lint Codebase
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        id: checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        id: setup-node
+        uses: actions/setup-node@v4
+        with:
+          node-version-file: .node-version
+          cache: npm
+
+      - name: Install Dependencies
+        id: install
+        run: npm ci
+
+      - name: Lint Codebase
+        id: super-linter
+        uses: super-linter/super-linter/slim@v7.2.1
+        env:
+          DEFAULT_BRANCH: main
+          FILTER_REGEX_EXCLUDE: dist/**/*
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          TYPESCRIPT_DEFAULT_STYLE: prettier
+          VALIDATE_ALL_CODEBASE: true
+          VALIDATE_JAVASCRIPT_STANDARD: false
+          VALIDATE_TYPESCRIPT_STANDARD: false
+          VALIDATE_JSCPD: false
@@ -0,0 +1,22 @@
+name: 'Publish Immutable Action Version'
+
+on:
+  release:
+    types: [published]
+
+permissions: {}
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
+      packages: write
+
+    steps:
+      - name: Checking out
+        uses: actions/checkout@v4
+      - name: Publish
+        id: publish
+        uses: actions/publish-immutable-action@v0.0.4
@@ -1 +1 @@
-24.5.0
+20.6.0
@@ -18,12 +18,6 @@ Once the attestation has been created and signed, it will be uploaded to the GH
 attestations API and associated with the repository from which the workflow was
 initiated.

-When an attestation is created, the attestation is stored on the local
-filesystem used by the runner. For each attestation created, the filesystem path
-will be appended to the file `${RUNNER_TEMP}/created_attestation_paths.txt`.
-This can be used to gather all attestations created by all jobs during a the
-workflow.
-
 Attestations can be verified using the [`attestation` command in the GitHub
 CLI][5].

@@ -52,13 +46,11 @@ attest:
   permissions:
     id-token: write
     attestations: write
-     artifact-metadata: write
   ```

   The `id-token` permission gives the action the ability to mint the OIDC token
   necessary to request a Sigstore signing certificate. The `attestations`
-   permission is necessary to persist the attestation. The `artifact-metadata`
-   permission is necessary to create the artifact storage record.
+   permission is necessary to persist the attestation.

 1. Add the following to your workflow after your artifact has been built:

@@ -120,12 +112,6 @@ See [action.yml](action.yml)
    # the "subject-digest" parameter be specified. Defaults to false.
    push-to-registry:

-    # Whether to create a storage record for the artifact.
-    # Requires that push-to-registry is set to true.
-    # Requires that the "subject-name" parameter specify the fully-qualified
-    # image name. Defaults to true.
-    create-storage-record:
-
    # Whether to attach a list of generated attestations to the workflow run
    # summary page. Defaults to true.
    show-summary:
@@ -139,12 +125,11 @@ See [action.yml](action.yml)

 <!-- markdownlint-disable MD013 -->

-| Name                 | Description                                                    | Example                                          |
-| -------------------  | -------------------------------------------------------------- | ------------------------------------------------ |
-| `attestation-id`     | GitHub ID for the attestation                                  | `123456`                                         |
-| `attestation-url`    | URL for the attestation summary                                | `https://github.com/foo/bar/attestations/123456` |
-| `bundle-path`        | Absolute path to the file containing the generated attestation | `/tmp/attestation.json`                          |
-| `storage-record-ids` | GitHub IDs for the storage records                             | `987654`                                         |
+| Name              | Description                                                    | Example                                          |
+| ----------------- | -------------------------------------------------------------- | ------------------------------------------------ |
+| `attestation-id`  | GitHub ID for the attestation                                  | `123456`                                         |
+| `attestation-url` | URL for the attestation summary                                | `https://github.com/foo/bar/attestations/123456` |
+| `bundle-path`     | Absolute path to the file containing the generated attestation | `/tmp/attestation.json`                          |

 <!-- markdownlint-enable MD013 -->

@@ -278,10 +263,6 @@ fully-qualified image name (e.g. "ghcr.io/user/app" or
 "acme.azurecr.io/user/app"). Do NOT include a tag as part of the image name --
 the specific image being attested is identified by the supplied digest.

-If the `push-to-registry` option is set to true, the Action will also
-emit an Artifact Metadata Storage Record. If you do not want to emit a
-storage record, set `create-storage-record` to `false`.
-
 > **NOTE**: When pushing to Docker Hub, please use "docker.io" as the registry
 > portion of the image name.

@@ -300,7 +281,6 @@ jobs:
      packages: write
      contents: read
      attestations: write
-      artifact-metadata: write
    env:
      REGISTRY: ghcr.io
      IMAGE_NAME: ${{ github.repository }}
@@ -2,26 +2,20 @@
 * Unit tests for the action's entrypoint, src/index.ts
 */

-import { jest, describe, expect, beforeEach } from '@jest/globals'
+import * as core from '@actions/core'
+import * as main from '../src/main'

-// Mock modules before importing them
-const runMock = jest.fn<() => Promise<void>>()
-
-jest.unstable_mockModule('../src/main', () => ({
-  run: runMock
-}))
-
-jest.unstable_mockModule('@actions/core', () => ({
-  getInput: jest.fn(() => ''),
-  getBooleanInput: jest.fn(() => false)
-}))
+// Mock the action's entrypoint
+const runMock = jest.spyOn(main, 'run').mockImplementation()
+const getBooleanInputMock = jest.spyOn(core, 'getBooleanInput')

 describe('index', () => {
  beforeEach(() => {
-    jest.clearAllMocks()
+    getBooleanInputMock.mockImplementation(() => false)
  })
-  it('calls run when imported', async () => {
-    await import('../src/index.js')
+  it('calls run when imported', () => {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
+    require('../src/index')

    expect(runMock).toHaveBeenCalled()
  })
@@ -5,110 +5,38 @@
 * Specifically, the inputs listed in `action.yml` should be set as environment
 * variables following the pattern `INPUT_<INPUT_NAME>`.
 */
-import {
-  jest,
-  describe,
-  expect,
-  beforeEach,
-  afterEach,
-  it
-} from '@jest/globals'
-import type { RunInputs } from '../src/main.js'
+import * as core from '@actions/core'
+import * as github from '@actions/github'
+import { mockFulcio, mockRekor, mockTSA } from '@sigstore/mock'
+import * as oci from '@sigstore/oci'
+import fs from 'fs/promises'
+import nock from 'nock'
+import os from 'os'
+import path from 'path'
+import { MockAgent, setGlobalDispatcher } from 'undici'
+import { SEARCH_PUBLIC_GOOD_URL } from '../src/endpoints'
+import * as main from '../src/main'

-// Create mock functions for core
-const infoMock = jest.fn()
-const warningMock = jest.fn()
-const startGroupMock = jest.fn()
-const endGroupMock = jest.fn()
-const setOutputMock = jest.fn()
-const setFailedMock = jest.fn()
-const summaryWriteMock = jest.fn<() => Promise<void>>()
+// Mock the GitHub Actions core library
+const infoMock = jest.spyOn(core, 'info')
+const startGroupMock = jest.spyOn(core, 'startGroup')
+const setOutputMock = jest.spyOn(core, 'setOutput')
+const setFailedMock = jest.spyOn(core, 'setFailed')

-// Create a mock summary object
-const mockSummary = {
-  addHeading: jest.fn().mockReturnThis(),
-  addRaw: jest.fn().mockReturnThis(),
-  addTable: jest.fn().mockReturnThis(),
-  addSeparator: jest.fn().mockReturnThis(),
-  addLink: jest.fn().mockReturnThis(),
-  addBreak: jest.fn().mockReturnThis(),
-  addList: jest.fn().mockReturnThis(),
-  write: summaryWriteMock.mockResolvedValue(undefined)
-}
+// Ensure that setFailed doesn't set an exit code during tests
+setFailedMock.mockImplementation(() => {})

-// Mock @actions/core before importing
-jest.unstable_mockModule('@actions/core', () => ({
-  info: infoMock,
-  warning: warningMock,
-  startGroup: startGroupMock,
-  endGroup: endGroupMock,
-  setOutput: setOutputMock,
-  setFailed: setFailedMock,
-  summary: mockSummary
-}))
+const summaryWriteMock = jest.spyOn(core.summary, 'write')
+summaryWriteMock.mockImplementation(async () => Promise.resolve(core.summary))

-// Create mocks for OCI and attest modules
-/* eslint-disable @typescript-eslint/no-explicit-any */
-const getRegistryCredentialsMock = jest.fn<(...args: any[]) => any>()
-const attachArtifactToImageMock = jest.fn<(...args: any[]) => any>()
-const createStorageRecordMock = jest.fn<(...args: any[]) => any>()
-const attestMock = jest.fn<(...args: any[]) => any>()
-/* eslint-enable @typescript-eslint/no-explicit-any */
-
-// Mock @sigstore/oci
-jest.unstable_mockModule('@sigstore/oci', () => ({
-  getRegistryCredentials: getRegistryCredentialsMock,
-  attachArtifactToImage: attachArtifactToImageMock
-}))
-
-// Mock @actions/attest
-jest.unstable_mockModule('@actions/attest', () => ({
-  attest: attestMock,
-  createStorageRecord: createStorageRecordMock
-}))
-
-// Create a mutable context object for @actions/github
-const mockContext: Record<string, unknown> = {}
-
-// Mock for getOctokit to return a mock octokit client
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-const mockReposGet = jest.fn<(...args: any[]) => any>()
-const mockOctokit = {
-  rest: {
-    repos: {
-      get: mockReposGet
-    }
-  }
-}
-
-jest.unstable_mockModule('@actions/github', () => ({
-  context: mockContext,
-  getOctokit: jest.fn(() => mockOctokit)
-}))
-
-// Helper to set the mocked GitHub context
-function setGHContext(context: object): void {
-  Object.keys(mockContext).forEach(key => delete mockContext[key])
-  Object.assign(mockContext, context)
-}
-
-// Now import the modules after mocking
-const { mockFulcio, mockRekor, mockTSA } = await import('@sigstore/mock')
-const fs = (await import('fs/promises')).default
-const nock = (await import('nock')).default
-const os = (await import('os')).default
-const path = (await import('path')).default
-const { MockAgent, setGlobalDispatcher } = await import('undici')
-const { SEARCH_PUBLIC_GOOD_URL } = await import('../src/endpoints.js')
-const { run } = (await import('../src/main.js')) as {
-  run: (inputs: RunInputs) => Promise<void>
-}
+// Mock the action's main function
+const runMock = jest.spyOn(main, 'run')

 // MockAgent for mocking @actions/github
 const mockAgent = new MockAgent()
 setGlobalDispatcher(mockAgent)

-const defaultInputs: RunInputs = {
+const defaultInputs: main.RunInputs = {
  predicate: '',
  predicateType: '',
  predicatePath: '',
@@ -117,15 +45,16 @@ const defaultInputs: RunInputs = {
  subjectPath: '',
  subjectChecksums: '',
  pushToRegistry: false,
-  createStorageRecord: true,
  showSummary: true,
  githubToken: '',
  privateSigning: false
 }

 describe('action', () => {
-  // Capture original environment variables so we can restore them after each test
+  // Capture original environment variables and GitHub context so we can restore
+  // them after each test
  const originalEnv = process.env
+  const originalContext = { ...github.context }

  // Mock OIDC token endpoint
  const tokenURL = 'https://token.url'
@@ -137,66 +66,30 @@ describe('action', () => {
    'base64'
  )}.}`

-  const subjectName = 'ghcr.io/registry/foo/bar'
+  const subjectName = 'registry/foo/bar'
  const subjectDigest =
    'sha256:7d070f6b64d9bcc530fe99cc21eaaa4b3c364e0b2d367d7735671fa202a03b32'
  const predicate = '{}'
  const predicateType = 'https://in-toto.io/attestation/release/v0.1'

  const attestationID = '1234567890'
-  const storageRecordID = 987654321

  beforeEach(() => {
    jest.clearAllMocks()

-    // Set up default GitHub context with empty payload
-    setGHContext({
-      payload: {},
-      repo: { owner: 'test-owner', repo: 'test-repo' }
-    })
-
-    // Set up default return value for attestMock (without tlogID for private/GitHub sigstore)
-    attestMock.mockResolvedValue({
-      attestationID,
-      bundle: {
-        mediaType: 'application/vnd.dev.sigstore.bundle.v0.3+json',
-        verificationMaterial: {
-          certificate: {
-            rawBytes: Buffer.from(
-              '-----BEGIN CERTIFICATE-----\ntest\n-----END CERTIFICATE-----'
-            ).toString('base64')
-          },
-          tlogEntries: []
-        },
-        content: {}
-      },
-      certificate:
-        '-----BEGIN CERTIFICATE-----\ntest\n-----END CERTIFICATE-----'
-    })
-
-    // Set up default return value for createStorageRecordMock (returns array of record IDs)
-    createStorageRecordMock.mockResolvedValue([storageRecordID])
-
    nock(tokenURL)
      .get('/')
      .query({ audience: 'sigstore' })
      .reply(200, { value: oidcToken })

-    const pool = mockAgent.get('https://api.github.com')
-    pool
+    mockAgent
+      .get('https://api.github.com')
      .intercept({
        path: /^\/repos\/.*\/.*\/attestations$/,
        method: 'post'
      })
      .reply(201, { id: attestationID })

-    pool
-      .intercept({
-        path: /^\/orgs\/.*\/artifacts\/metadata\/storage-record$/,
-        method: 'post'
-      })
-      .reply(200, { storage_records: [{ id: storageRecordID }] })
-
    process.env = {
      ...originalEnv,
      ACTIONS_ID_TOKEN_REQUEST_URL: tokenURL,
@@ -209,12 +102,12 @@ describe('action', () => {
    // Restore the original environment
    process.env = originalEnv

-    // Clear the github context
-    setGHContext({ payload: {}, repo: { owner: '', repo: '' } })
+    // Restore the original github.context
+    setGHContext(originalContext)
  })

  describe('when ACTIONS_ID_TOKEN_REQUEST_URL is not set', () => {
-    const inputs: RunInputs = {
+    const inputs: main.RunInputs = {
      ...defaultInputs,
      subjectDigest,
      subjectName,
@@ -229,8 +122,9 @@ describe('action', () => {
    })

    it('sets a failed status', async () => {
-      await run(inputs)
+      await main.run(inputs)

+      expect(runMock).toHaveReturned()
      expect(setFailedMock).toHaveBeenCalledWith(
        new Error(
          'missing "id-token" permission. Please add "permissions: id-token: write" to your workflow.'
@@ -241,8 +135,9 @@ describe('action', () => {

  describe('when no inputs are provided', () => {
    it('sets a failed status', async () => {
-      await run(defaultInputs)
+      await main.run(defaultInputs)

+      expect(runMock).toHaveReturned()
      expect(setFailedMock).toHaveBeenCalledWith(
        new Error(
          'One of subject-path, subject-digest, or subject-checksums must be provided'
@@ -252,7 +147,7 @@ describe('action', () => {
  })

  describe('when the repository is private', () => {
-    const inputs: RunInputs = {
+    const inputs: main.RunInputs = {
      ...defaultInputs,
      subjectDigest,
      subjectName,
@@ -276,9 +171,10 @@ describe('action', () => {
    })

    it('invokes the action w/o error', async () => {
-      await run(inputs)
+      await main.run(inputs)

-      expect(setFailedMock).not.toHaveBeenCalled()
+      expect(runMock).toHaveReturned()
+      expect(setFailedMock).not.toHaveBeenCalledWith()
      expect(infoMock).toHaveBeenNthCalledWith(
        1,
        expect.stringMatching(
@@ -321,7 +217,10 @@ describe('action', () => {
  })

  describe('when the repository is public', () => {
-    const inputs: RunInputs = {
+    const getRegCredsSpy = jest.spyOn(oci, 'getRegistryCredentials')
+    const attachArtifactSpy = jest.spyOn(oci, 'attachArtifactToImage')
+
+    const inputs: main.RunInputs = {
      ...defaultInputs,
      subjectDigest,
      subjectName,
@@ -344,51 +243,26 @@ describe('action', () => {
      })
      await mockRekor({ baseURL: 'https://rekor.sigstore.dev' })

-      getRegistryCredentialsMock.mockImplementation(() => ({
+      getRegCredsSpy.mockImplementation(() => ({
        username: 'username',
        password: 'password'
      }))
-      attachArtifactToImageMock.mockResolvedValue({
-        digest: 'sha256:123456',
-        mediaType: 'application/vnd.cncf.notary.v2',
-        size: 123456
-      })
-
-      // Set up attestMock with tlogID for public good sigstore
-      attestMock.mockResolvedValue({
-        attestationID,
-        bundle: {
-          mediaType: 'application/vnd.dev.sigstore.bundle.v0.3+json',
-          verificationMaterial: {
-            certificate: {
-              rawBytes: Buffer.from(
-                '-----BEGIN CERTIFICATE-----\ntest\n-----END CERTIFICATE-----'
-              ).toString('base64')
-            },
-            tlogEntries: [{ logIndex: '123' }]
-          },
-          content: {}
-        },
-        certificate:
-          '-----BEGIN CERTIFICATE-----\ntest\n-----END CERTIFICATE-----',
-        tlogID: '123'
-      })
-
-      // Mock the repos.get API call for repoOwnerIsOrg check
-      mockReposGet.mockResolvedValue({
-        data: { owner: { type: 'Organization' } }
-      })
+      attachArtifactSpy.mockImplementation(async () =>
+        Promise.resolve({
+          digest: 'sha256:123456',
+          mediaType: 'application/vnd.cncf.notary.v2',
+          size: 123456
+        })
+      )
    })

    it('invokes the action w/o error', async () => {
-      await run(inputs)
+      await main.run(inputs)

+      expect(runMock).toHaveReturned()
      expect(setFailedMock).not.toHaveBeenCalled()
-      expect(getRegistryCredentialsMock).toHaveBeenCalledWith(subjectName)
-      expect(attachArtifactToImageMock).toHaveBeenCalled()
-      expect(attestMock).toHaveBeenCalled()
-      expect(createStorageRecordMock).toHaveBeenCalled()
-      expect(warningMock).not.toHaveBeenCalled()
+      expect(getRegCredsSpy).toHaveBeenCalledWith(subjectName)
+      expect(attachArtifactSpy).toHaveBeenCalled()
      expect(infoMock).toHaveBeenNthCalledWith(
        1,
        expect.stringMatching(
@@ -419,14 +293,6 @@ describe('action', () => {
        6,
        expect.stringMatching(attestationID)
      )
-      expect(infoMock).toHaveBeenNthCalledWith(
-        9,
-        expect.stringMatching('Storage record created')
-      )
-      expect(infoMock).toHaveBeenNthCalledWith(
-        10,
-        expect.stringMatching('Storage record IDs: 987654321')
-      )
      expect(setOutputMock).toHaveBeenNthCalledWith(
        1,
        'bundle-path',
@@ -442,62 +308,6 @@ describe('action', () => {
        'attestation-url',
        expect.stringContaining(`foo/bar/attestations/${attestationID}`)
      )
-      expect(setOutputMock).toHaveBeenNthCalledWith(
-        4,
-        'storage-record-ids',
-        expect.stringMatching(storageRecordID.toString())
-      )
-      expect(setFailedMock).not.toHaveBeenCalled()
-    })
-
-    it('catches error when storage record creation fails and continues', async () => {
-      // Mock the createStorageRecord function and throw an error
-      createStorageRecordMock.mockRejectedValueOnce(
-        new Error('Failed to persist storage record: Not Found')
-      )
-
-      await run(inputs)
-
-      expect(attestMock).toHaveBeenCalled()
-      expect(createStorageRecordMock).toHaveBeenCalled()
-      expect(setFailedMock).not.toHaveBeenCalled()
-      expect(warningMock).toHaveBeenNthCalledWith(
-        1,
-        expect.stringMatching('Failed to create storage record')
-      )
-    })
-
-    it('does not create a storage record when the repo is owned by a user', async () => {
-      // Mock the repos.get API to return a user-owned repo
-      mockReposGet.mockResolvedValueOnce({ data: { owner: { type: 'User' } } })
-
-      await run(inputs)
-
-      expect(setFailedMock).not.toHaveBeenCalled()
-      expect(getRegistryCredentialsMock).toHaveBeenCalledWith(subjectName)
-      expect(attachArtifactToImageMock).toHaveBeenCalled()
-      expect(attestMock).toHaveBeenCalled()
-      expect(createStorageRecordMock).not.toHaveBeenCalled()
-      expect(warningMock).not.toHaveBeenCalled()
-      expect(infoMock).toHaveBeenCalledWith(
-        expect.stringMatching(
-          `Attestation created for ${subjectName}@${subjectDigest}`
-        )
-      )
-      expect(infoMock).not.toHaveBeenCalledWith(
-        expect.stringMatching('Storage record created')
-      )
-      expect(infoMock).not.toHaveBeenCalledWith(
-        expect.stringMatching('Storage record IDs: 987654321')
-      )
-      expect(setOutputMock).toHaveBeenCalledWith(
-        'attestation-id',
-        expect.stringMatching(attestationID)
-      )
-      expect(setOutputMock).not.toHaveBeenCalledWith(
-        'storage-record-ids',
-        expect.stringMatching(storageRecordID.toString())
-      )
      expect(setFailedMock).not.toHaveBeenCalled()
    })
  })
@@ -541,15 +351,16 @@ describe('action', () => {
    })

    it('invokes the action w/o error', async () => {
-      const inputs: RunInputs = {
+      const inputs: main.RunInputs = {
        ...defaultInputs,
        subjectPath: path.join(dir, `${filename}-*`),
        predicateType,
        predicate,
        githubToken: 'gh-token'
      }
-      await run(inputs)
+      await main.run(inputs)

+      expect(runMock).toHaveReturned()
      expect(setFailedMock).not.toHaveBeenCalled()
      expect(infoMock).toHaveBeenNthCalledWith(
        1,
@@ -588,15 +399,16 @@ describe('action', () => {
    })

    it('sets a failed status', async () => {
-      const inputs: RunInputs = {
+      const inputs: main.RunInputs = {
        ...defaultInputs,
        subjectPath: path.join(dir, `${filename}-*`),
        predicateType,
        predicate,
        githubToken: 'gh-token'
      }
-      await run(inputs)
+      await main.run(inputs)

+      expect(runMock).toHaveReturned()
      expect(setFailedMock).toHaveBeenCalledWith(
        new Error(
          'Too many subjects specified. The maximum number of subjects is 1024.'
@@ -605,3 +417,9 @@ describe('action', () => {
    })
  })
 })
+
+// Stubbing the GitHub context is a bit tricky. We need to use
+// `Object.defineProperty` because `github.context` is read-only.
+function setGHContext(context: object): void {
+  Object.defineProperty(github, 'context', { value: context })
+}
@@ -1,7 +1,7 @@
 import fs from 'fs/promises'
 import os from 'os'
 import path from 'path'
-import { predicateFromInputs, PredicateInputs } from '../src/predicate.js'
+import { predicateFromInputs, PredicateInputs } from '../src/predicate'

 describe('subjectFromInputs', () => {
  const blankInputs: PredicateInputs = {
@@ -1,4 +1,4 @@
-import { highlight, mute } from '../src/style.js'
+import { highlight, mute } from '../src/style'

 describe('style', () => {
  describe('highlight', () => {
@@ -6,7 +6,7 @@ import {
  formatSubjectDigest,
  subjectFromInputs,
  SubjectInputs
-} from '../src/subject.js'
+} from '../src/subject'

 describe('subjectFromInputs', () => {
  const blankInputs: SubjectInputs = {
@@ -264,15 +264,10 @@ describe('subjectFromInputs', () => {
        expect(subjects).toBeDefined()
        expect(subjects).toHaveLength(3)

-        subjects.forEach(
-          (
-            subject: { name: string; digest: Record<string, string> },
-            i: number
-          ) => {
-            expect(subject.name).toEqual(`${filename}-${i}`)
-            expect(subject.digest).toEqual({ sha256: expectedDigest })
-          }
-        )
+        subjects.forEach((subject, i) => {
+          expect(subject.name).toEqual(`${filename}-${i}`)
+          expect(subject.digest).toEqual({ sha256: expectedDigest })
+        })
      })
    })

@@ -478,13 +473,6 @@ badline
              '187dcd1506a170337415589ff00c8743f19d41cc31fca246c2739dfd450d0b9d'
          }
        })
-        expect(subjects).toContainEqual({
-          name: 'demo_0.0.1_darwin_arm64',
-          digest: {
-            sha512:
-              '5d8b4751ef31f9440d843fcfa4e53ca2e25b1cb1f13fd355fdc7c24b41fe645293291ea9297ba3989078abb77ebbaac66be073618a9e4974dbd0361881d4c718'
-          }
-        })
      })
    })
  })
@@ -492,8 +480,8 @@ badline
  describe('when specifying a subject checksums string', () => {
    const checksums = `
 f861e68a080799ca83104630b56abb90d8dbcc5f8b5a8639cb691e269838f29e  demo_0.0.1_linux_386
-187dcd1506a170337415589ff00c8743f19d41cc31fca246c2739dfd450d0b9d *demo_0.0.1_linux_amd64
-9ecbf449e286a8a8748c161c52aa28b6b2fc64ab86f94161c5d1b3abc18156c5 demo_0.0.1_linux_arm64`
+187dcd1506a170337415589ff00c8743f19d41cc31fca246c2739dfd450d0b9d  demo_0.0.1_linux_amd64
+9ecbf449e286a8a8748c161c52aa28b6b2fc64ab86f94161c5d1b3abc18156c5  demo_0.0.1_linux_arm64`

    it('returns the multiple subjects', async () => {
      const inputs: SubjectInputs = {
@@ -512,20 +500,6 @@ f861e68a080799ca83104630b56abb90d8dbcc5f8b5a8639cb691e269838f29e  demo_0.0.1_lin
            'f861e68a080799ca83104630b56abb90d8dbcc5f8b5a8639cb691e269838f29e'
        }
      })
-      expect(subjects).toContainEqual({
-        name: 'demo_0.0.1_linux_amd64',
-        digest: {
-          sha256:
-            '187dcd1506a170337415589ff00c8743f19d41cc31fca246c2739dfd450d0b9d'
-        }
-      })
-      expect(subjects).toContainEqual({
-        name: 'demo_0.0.1_linux_arm64',
-        digest: {
-          sha256:
-            '9ecbf449e286a8a8748c161c52aa28b6b2fc64ab86f94161c5d1b3abc18156c5'
-        }
-      })
    })
  })

@@ -53,12 +53,6 @@ inputs:
      the "subject-digest" parameter be specified. Defaults to false.
    default: false
    required: false
-  create-storage-record:
-    description: >
-      Whether to create a storage record for the artifact.
-      Requires that push-to-registry is set to true. Defaults to true.
-    default: true
-    required: false
  show-summary:
    description: >
      Whether to attach a list of generated attestations to the workflow run
@@ -77,9 +71,7 @@ outputs:
    description: 'The ID of the attestation.'
  attestation-url:
    description: 'The URL for the attestation summary.'
-  storage-record-ids:
-    description: 'The IDs of the storage records created for the artifact.'

 runs:
-  using: node24
+  using: node20
  main: ./dist/index.js
@@ -0,0 +1,287 @@
+"use strict";
+exports.id = 184;
+exports.ids = [184];
+exports.modules = {
+
+/***/ 91184:
+/***/ ((__unused_webpack___webpack_module__, __webpack_exports__, __webpack_require__) => {
+
+/* harmony export */ __webpack_require__.d(__webpack_exports__, {
+/* harmony export */   "default": () => (/* binding */ pMap)
+/* harmony export */ });
+/* unused harmony exports pMapIterable, pMapSkip */
+async function pMap(
+	iterable,
+	mapper,
+	{
+		concurrency = Number.POSITIVE_INFINITY,
+		stopOnError = true,
+		signal,
+	} = {},
+) {
+	return new Promise((resolve, reject_) => {
+		if (iterable[Symbol.iterator] === undefined && iterable[Symbol.asyncIterator] === undefined) {
+			throw new TypeError(`Expected \`input\` to be either an \`Iterable\` or \`AsyncIterable\`, got (${typeof iterable})`);
+		}
+
+		if (typeof mapper !== 'function') {
+			throw new TypeError('Mapper function is required');
+		}
+
+		if (!((Number.isSafeInteger(concurrency) && concurrency >= 1) || concurrency === Number.POSITIVE_INFINITY)) {
+			throw new TypeError(`Expected \`concurrency\` to be an integer from 1 and up or \`Infinity\`, got \`${concurrency}\` (${typeof concurrency})`);
+		}
+
+		const result = [];
+		const errors = [];
+		const skippedIndexesMap = new Map();
+		let isRejected = false;
+		let isResolved = false;
+		let isIterableDone = false;
+		let resolvingCount = 0;
+		let currentIndex = 0;
+		const iterator = iterable[Symbol.iterator] === undefined ? iterable[Symbol.asyncIterator]() : iterable[Symbol.iterator]();
+
+		const reject = reason => {
+			isRejected = true;
+			isResolved = true;
+			reject_(reason);
+		};
+
+		if (signal) {
+			if (signal.aborted) {
+				reject(signal.reason);
+			}
+
+			signal.addEventListener('abort', () => {
+				reject(signal.reason);
+			});
+		}
+
+		const next = async () => {
+			if (isResolved) {
+				return;
+			}
+
+			const nextItem = await iterator.next();
+
+			const index = currentIndex;
+			currentIndex++;
+
+			// Note: `iterator.next()` can be called many times in parallel.
+			// This can cause multiple calls to this `next()` function to
+			// receive a `nextItem` with `done === true`.
+			// The shutdown logic that rejects/resolves must be protected
+			// so it runs only one time as the `skippedIndex` logic is
+			// non-idempotent.
+			if (nextItem.done) {
+				isIterableDone = true;
+
+				if (resolvingCount === 0 && !isResolved) {
+					if (!stopOnError && errors.length > 0) {
+						reject(new AggregateError(errors)); // eslint-disable-line unicorn/error-message
+						return;
+					}
+
+					isResolved = true;
+
+					if (skippedIndexesMap.size === 0) {
+						resolve(result);
+						return;
+					}
+
+					const pureResult = [];
+
+					// Support multiple `pMapSkip`'s.
+					for (const [index, value] of result.entries()) {
+						if (skippedIndexesMap.get(index) === pMapSkip) {
+							continue;
+						}
+
+						pureResult.push(value);
+					}
+
+					resolve(pureResult);
+				}
+
+				return;
+			}
+
+			resolvingCount++;
+
+			// Intentionally detached
+			(async () => {
+				try {
+					const element = await nextItem.value;
+
+					if (isResolved) {
+						return;
+					}
+
+					const value = await mapper(element, index);
+
+					// Use Map to stage the index of the element.
+					if (value === pMapSkip) {
+						skippedIndexesMap.set(index, value);
+					}
+
+					result[index] = value;
+
+					resolvingCount--;
+					await next();
+				} catch (error) {
+					if (stopOnError) {
+						reject(error);
+					} else {
+						errors.push(error);
+						resolvingCount--;
+
+						// In that case we can't really continue regardless of `stopOnError` state
+						// since an iterable is likely to continue throwing after it throws once.
+						// If we continue calling `next()` indefinitely we will likely end up
+						// in an infinite loop of failed iteration.
+						try {
+							await next();
+						} catch (error) {
+							reject(error);
+						}
+					}
+				}
+			})();
+		};
+
+		// Create the concurrent runners in a detached (non-awaited)
+		// promise. We need this so we can await the `next()` calls
+		// to stop creating runners before hitting the concurrency limit
+		// if the iterable has already been marked as done.
+		// NOTE: We *must* do this for async iterators otherwise we'll spin up
+		// infinite `next()` calls by default and never start the event loop.
+		(async () => {
+			for (let index = 0; index < concurrency; index++) {
+				try {
+					// eslint-disable-next-line no-await-in-loop
+					await next();
+				} catch (error) {
+					reject(error);
+					break;
+				}
+
+				if (isIterableDone || isRejected) {
+					break;
+				}
+			}
+		})();
+	});
+}
+
+function pMapIterable(
+	iterable,
+	mapper,
+	{
+		concurrency = Number.POSITIVE_INFINITY,
+		backpressure = concurrency,
+	} = {},
+) {
+	if (iterable[Symbol.iterator] === undefined && iterable[Symbol.asyncIterator] === undefined) {
+		throw new TypeError(`Expected \`input\` to be either an \`Iterable\` or \`AsyncIterable\`, got (${typeof iterable})`);
+	}
+
+	if (typeof mapper !== 'function') {
+		throw new TypeError('Mapper function is required');
+	}
+
+	if (!((Number.isSafeInteger(concurrency) && concurrency >= 1) || concurrency === Number.POSITIVE_INFINITY)) {
+		throw new TypeError(`Expected \`concurrency\` to be an integer from 1 and up or \`Infinity\`, got \`${concurrency}\` (${typeof concurrency})`);
+	}
+
+	if (!((Number.isSafeInteger(backpressure) && backpressure >= concurrency) || backpressure === Number.POSITIVE_INFINITY)) {
+		throw new TypeError(`Expected \`backpressure\` to be an integer from \`concurrency\` (${concurrency}) and up or \`Infinity\`, got \`${backpressure}\` (${typeof backpressure})`);
+	}
+
+	return {
+		async * [Symbol.asyncIterator]() {
+			const iterator = iterable[Symbol.asyncIterator] === undefined ? iterable[Symbol.iterator]() : iterable[Symbol.asyncIterator]();
+
+			const promises = [];
+			let runningMappersCount = 0;
+			let isDone = false;
+			let index = 0;
+
+			function trySpawn() {
+				if (isDone || !(runningMappersCount < concurrency && promises.length < backpressure)) {
+					return;
+				}
+
+				const promise = (async () => {
+					const {done, value} = await iterator.next();
+
+					if (done) {
+						return {done: true};
+					}
+
+					runningMappersCount++;
+
+					// Spawn if still below concurrency and backpressure limit
+					trySpawn();
+
+					try {
+						const returnValue = await mapper(await value, index++);
+
+						runningMappersCount--;
+
+						if (returnValue === pMapSkip) {
+							const index = promises.indexOf(promise);
+
+							if (index > 0) {
+								promises.splice(index, 1);
+							}
+						}
+
+						// Spawn if still below backpressure limit and just dropped below concurrency limit
+						trySpawn();
+
+						return {done: false, value: returnValue};
+					} catch (error) {
+						isDone = true;
+						return {error};
+					}
+				})();
+
+				promises.push(promise);
+			}
+
+			trySpawn();
+
+			while (promises.length > 0) {
+				const {error, done, value} = await promises[0]; // eslint-disable-line no-await-in-loop
+
+				promises.shift();
+
+				if (error) {
+					throw error;
+				}
+
+				if (done) {
+					return;
+				}
+
+				// Spawn if just dropped below backpressure limit and below the concurrency limit
+				trySpawn();
+
+				if (value === pMapSkip) {
+					continue;
+				}
+
+				yield value;
+			}
+		},
+	};
+}
+
+const pMapSkip = Symbol('skip');
+
+
+/***/ })
+
+};
+;
@@ -1,6 +1,7 @@
-export const id = 606;
-export const ids = [606];
-export const modules = {
+"use strict";
+exports.id = 606;
+exports.ids = [606];
+exports.modules = {

 /***/ 606:
 /***/ ((__unused_webpack___webpack_module__, __webpack_exports__, __webpack_require__) => {
@@ -18,7 +19,7 @@ async function pMap(
 		signal,
 	} = {},
 ) {
-	return new Promise((resolve_, reject_) => {
+	return new Promise((resolve, reject_) => {
 		if (iterable[Symbol.iterator] === undefined && iterable[Symbol.asyncIterator] === undefined) {
 			throw new TypeError(`Expected \`input\` to be either an \`Iterable\` or \`AsyncIterable\`, got (${typeof iterable})`);
 		}
@@ -41,24 +42,10 @@ async function pMap(
 		let currentIndex = 0;
 		const iterator = iterable[Symbol.iterator] === undefined ? iterable[Symbol.asyncIterator]() : iterable[Symbol.iterator]();

-		const signalListener = () => {
-			reject(signal.reason);
-		};
-
-		const cleanup = () => {
-			signal?.removeEventListener('abort', signalListener);
-		};
-
-		const resolve = value => {
-			resolve_(value);
-			cleanup();
-		};
-
 		const reject = reason => {
 			isRejected = true;
 			isResolved = true;
 			reject_(reason);
-			cleanup();
 		};

 		if (signal) {
@@ -66,7 +53,9 @@ async function pMap(
 				reject(signal.reason);
 			}

-			signal.addEventListener('abort', signalListener, {once: true});
+			signal.addEventListener('abort', () => {
+				reject(signal.reason);
+			});
 		}

 		const next = async () => {
@@ -214,32 +203,31 @@ function pMapIterable(
 			const iterator = iterable[Symbol.asyncIterator] === undefined ? iterable[Symbol.iterator]() : iterable[Symbol.asyncIterator]();

 			const promises = [];
-			let pendingPromisesCount = 0;
+			let runningMappersCount = 0;
 			let isDone = false;
 			let index = 0;

 			function trySpawn() {
-				if (isDone || !(pendingPromisesCount < concurrency && promises.length < backpressure)) {
+				if (isDone || !(runningMappersCount < concurrency && promises.length < backpressure)) {
 					return;
 				}

-				pendingPromisesCount++;
-
 				const promise = (async () => {
 					const {done, value} = await iterator.next();

 					if (done) {
-						pendingPromisesCount--;
 						return {done: true};
 					}

+					runningMappersCount++;
+
 					// Spawn if still below concurrency and backpressure limit
 					trySpawn();

 					try {
 						const returnValue = await mapper(await value, index++);

-						pendingPromisesCount--;
+						runningMappersCount--;

 						if (returnValue === pMapSkip) {
 							const index = promises.indexOf(promise);
@@ -254,7 +242,6 @@ function pMapIterable(

 						return {done: false, value: returnValue};
 					} catch (error) {
-						pendingPromisesCount--;
 						isDone = true;
 						return {error};
 					}
@@ -297,3 +284,4 @@ const pMapSkip = Symbol('skip');
 /***/ })

 };
+;
@@ -1,3 +0,0 @@
-{
-  "type": "module"
-}
@@ -1,37 +0,0 @@
-export default {
-  preset: "ts-jest",
-  verbose: true,
-  clearMocks: true,
-  testEnvironment: 'node',
-  moduleFileExtensions: ['js', 'ts'],
-  testMatch: ['**/*.test.ts'],
-  testPathIgnorePatterns: [
-    "/node_modules/",
-    "/dist/"
-  ],
-  transform: {
-    '^.+\\.ts$': [
-      'ts-jest',
-      {
-        useESM: true,
-        diagnostics: {
-          ignoreCodes: [151002]
-        }
-      }
-    ]
-  },
-  coverageReporters: [
-    "json-summary",
-    "text",
-    "lcov"
-  ],
-  collectCoverage: true,
-  collectCoverageFrom: [
-    "./src/**"
-  ],
-  extensionsToTreatAsEsm: ['.ts'],
-  transformIgnorePatterns: ['node_modules/(?!(@actions)/)'],
-  moduleNameMapper: {
-    '^(\\.{1,2}/.*)\\.js$': '$1'
-  },
-}
@@ -0,0 +1 @@
+process.stdout.write = jest.fn()
@@ -1,8 +1,7 @@
 {
  "name": "actions/attest",
  "description": "Generate signed attestations for workflow artifacts",
-  "version": "3.2.0",
-  "type": "module",
+  "version": "2.2.1",
  "author": "",
  "private": true,
  "homepage": "https://github.com/actions/attest",
@@ -21,50 +20,80 @@
    ".": "./dist/index.js"
  },
  "engines": {
-    "node": ">=24"
+    "node": ">=20"
  },
  "scripts": {
    "bundle": "npm run format:write && npm run package",
-    "ci-test": "node --experimental-vm-modules node_modules/jest/bin/jest.js",
+    "ci-test": "jest",
    "format:write": "prettier --write **/*.ts",
    "format:check": "prettier --check **/*.ts",
-    "lint:eslint": "npx eslint",
-    "lint:markdown": "npx markdownlint --config .markdown-lint.yml \"*.md\"",
+    "lint:eslint": "npx eslint . -c ./.github/linters/eslint.config.mjs",
+    "lint:markdown": "npx markdownlint --config .github/linters/.markdown-lint.yml \"*.md\"",
    "lint": "npm run lint:eslint && npm run lint:markdown",
    "package": "ncc build src/index.ts --license licenses.txt",
    "package:watch": "npm run package -- --watch",
-    "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js",
+    "test": "jest",
    "all": "npm run format:write && npm run lint && npm run test && npm run package"
  },
  "license": "MIT",
+  "jest": {
+    "preset": "ts-jest",
+    "setupFilesAfterEnv": [
+      "./jest.setup.js"
+    ],
+    "verbose": true,
+    "clearMocks": true,
+    "testEnvironment": "node",
+    "moduleFileExtensions": [
+      "js",
+      "ts"
+    ],
+    "testMatch": [
+      "**/*.test.ts"
+    ],
+    "testPathIgnorePatterns": [
+      "/node_modules/",
+      "/dist/"
+    ],
+    "transform": {
+      "^.+\\.ts$": "ts-jest"
+    },
+    "coverageReporters": [
+      "json-summary",
+      "text",
+      "lcov"
+    ],
+    "collectCoverage": true,
+    "collectCoverageFrom": [
+      "./src/**"
+    ]
+  },
  "dependencies": {
-    "@actions/attest": "^2.2.1",
-    "@actions/core": "^2.0.2",
-    "@actions/github": "^7.0.0",
+    "@actions/attest": "^1.6.0",
+    "@actions/core": "^1.11.1",
+    "@actions/github": "^6.0.0",
    "@actions/glob": "^0.5.0",
-    "@sigstore/oci": "^0.6.0",
+    "@sigstore/oci": "^0.4.0",
    "csv-parse": "^5.6.0"
  },
  "devDependencies": {
-    "@eslint/js": "^9.39.2",
-    "@jest/globals": "^30.2.0",
-    "@sigstore/mock": "^0.11.0",
-    "@types/jest": "^30.0.0",
+    "@eslint/js": "^9.23.0",
+    "@sigstore/mock": "^0.10.0",
+    "@types/jest": "^29.5.14",
    "@types/make-fetch-happen": "^10.0.4",
-    "@types/node": "^25.2.0",
-    "@vercel/ncc": "^0.38.4",
-    "eslint": "^9.39.2",
-    "eslint-import-resolver-typescript": "^4.4.4",
-    "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jest": "^29.12.1",
-    "jest": "^30.2.0",
-    "js-yaml": "^4.1.1",
-    "markdownlint-cli": "^0.47.0",
+    "@types/node": "^22.13.14",
+    "@vercel/ncc": "^0.38.3",
+    "eslint": "^9.23.0",
+    "eslint-plugin-import": "^2.31.0",
+    "eslint-plugin-jest": "^28.11.0",
+    "jest": "^29.7.0",
+    "js-yaml": "^4.1.0",
+    "markdownlint-cli": "^0.44.0",
    "nock": "^13.5.6",
-    "prettier": "^3.8.1",
-    "ts-jest": "^29.4.6",
-    "typescript": "^5.9.3",
-    "typescript-eslint": "^8.54.0",
-    "undici": "^7.20.0"
+    "prettier": "^3.5.3",
+    "ts-jest": "^29.3.1",
+    "typescript": "^5.8.2",
+    "typescript-eslint": "^8.29.0",
+    "undici": "^5.28.5"
  }
 }
@@ -1,14 +1,6 @@
-import {
-  Attestation,
-  Predicate,
-  Subject,
-  attest,
-  createStorageRecord
-} from '@actions/attest'
+import { Attestation, Predicate, Subject, attest } from '@actions/attest'
 import { attachArtifactToImage, getRegistryCredentials } from '@sigstore/oci'
-import { formatSubjectDigest } from './subject.js'
-import * as core from '@actions/core'
-import * as github from '@actions/github'
+import { formatSubjectDigest } from './subject'

 const OCI_TIMEOUT = 30000
 const OCI_RETRY = 3
@@ -16,7 +8,6 @@ const OCI_RETRY = 3
 export type SigstoreInstance = 'public-good' | 'github'
 export type AttestResult = Attestation & {
  attestationDigest?: string
-  storageRecordIds?: number[]
 }

 export const createAttestation = async (
@@ -25,7 +16,6 @@ export const createAttestation = async (
  opts: {
    sigstoreInstance: SigstoreInstance
    pushToRegistry: boolean
-    createStorageRecord: boolean
    githubToken: string
  }
 ): Promise<AttestResult> => {
@@ -43,11 +33,10 @@ export const createAttestation = async (
  if (subjects.length === 1 && opts.pushToRegistry) {
    const subject = subjects[0]
    const credentials = getRegistryCredentials(subject.name)
-    const subjectDigest = formatSubjectDigest(subject)
    const artifact = await attachArtifactToImage({
      credentials,
      imageName: subject.name,
-      imageDigest: subjectDigest,
+      imageDigest: formatSubjectDigest(subject),
      artifact: Buffer.from(JSON.stringify(attestation.bundle)),
      mediaType: attestation.bundle.mediaType,
      annotations: {
@@ -59,78 +48,7 @@ export const createAttestation = async (

    // Add the attestation's digest to the result
    result.attestationDigest = artifact.digest
-
-    // Because creating a storage record requires the 'artifact-metadata:write'
-    // permission, we wrap this in a try/catch to avoid failing the entire
-    // attestation process if the token does not have the correct permissions.
-    if (opts.createStorageRecord) {
-      try {
-        const token = opts.githubToken
-        const isOrg = await repoOwnerIsOrg(token)
-        if (!isOrg) {
-          // The Artifact Metadata Storage Record API is only available to
-          // organizations. So if the repo owner is not an organization,
-          // storage record creation should not be attempted.
-          return result
-        }
-
-        const registryUrl = getRegistryURL(subject.name)
-        const artifactOpts = {
-          name: subject.name,
-          digest: subjectDigest
-        }
-        const packageRegistryOpts = {
-          registryUrl
-        }
-        const records = await createStorageRecord(
-          artifactOpts,
-          packageRegistryOpts,
-          token
-        )
-
-        if (!records || records.length === 0) {
-          core.warning('No storage records were created.')
-        }
-
-        result.storageRecordIds = records
-      } catch (error) {
-        core.warning(`Failed to create storage record: ${error}`)
-        core.warning(
-          'Please check that the "artifact-metadata:write" permission has been included'
-        )
-      }
-    }
  }

  return result
 }
-
-// Call the GET /repos/{owner}/{repo} endpoint to determine if the repo
-// owner is an organization. This is used to determine if storage
-// record creation should be attempted.
-export const repoOwnerIsOrg = async (githubToken: string): Promise<boolean> => {
-  const octokit = github.getOctokit(githubToken)
-  const { data: repo } = await octokit.rest.repos.get({
-    owner: github.context.repo.owner,
-    repo: github.context.repo.repo
-  })
-  return repo.owner?.type === 'Organization'
-}
-
-function getRegistryURL(subjectName: string): string {
-  let url: URL
-
-  try {
-    url = new URL(subjectName)
-  } catch {
-    url = new URL(`https://${subjectName}`)
-  }
-
-  if (url.protocol !== 'https:') {
-    throw new Error(
-      `Unsupported protocol ${url.protocol} in subject name ${subjectName}`
-    )
-  }
-
-  return url.origin
-}
@@ -2,7 +2,7 @@
 * The entrypoint for the action.
 */
 import * as core from '@actions/core'
-import { run, RunInputs } from './main.js'
+import { run, RunInputs } from './main'

 const inputs: RunInputs = {
  subjectPath: core.getInput('subject-path'),
@@ -13,7 +13,6 @@ const inputs: RunInputs = {
  predicate: core.getInput('predicate'),
  predicatePath: core.getInput('predicate-path'),
  pushToRegistry: core.getBooleanInput('push-to-registry'),
-  createStorageRecord: core.getBooleanInput('create-storage-record'),
  showSummary: core.getBooleanInput('show-summary'),
  githubToken: core.getInput('github-token'),
  // undocumented -- not part of public interface
@@ -3,25 +3,23 @@ import * as github from '@actions/github'
 import fs from 'fs'
 import os from 'os'
 import path from 'path'
-import { AttestResult, SigstoreInstance, createAttestation } from './attest.js'
-import { SEARCH_PUBLIC_GOOD_URL } from './endpoints.js'
-import { PredicateInputs, predicateFromInputs } from './predicate.js'
-import * as style from './style.js'
+import { AttestResult, SigstoreInstance, createAttestation } from './attest'
+import { SEARCH_PUBLIC_GOOD_URL } from './endpoints'
+import { PredicateInputs, predicateFromInputs } from './predicate'
+import * as style from './style'
 import {
  SubjectInputs,
  formatSubjectDigest,
  subjectFromInputs
-} from './subject.js'
+} from './subject'

 import type { Subject } from '@actions/attest'

 const ATTESTATION_FILE_NAME = 'attestation.json'
-const ATTESTATION_PATHS_FILE_NAME = 'created_attestation_paths.txt'

 export type RunInputs = SubjectInputs &
  PredicateInputs & {
    pushToRegistry: boolean
-    createStorageRecord: boolean
    githubToken: string
    showSummary: boolean
    privateSigning: boolean
@@ -70,7 +68,6 @@ export async function run(inputs: RunInputs): Promise<void> {
    const att = await createAttestation(subjects, predicate, {
      sigstoreInstance,
      pushToRegistry: inputs.pushToRegistry,
-      createStorageRecord: inputs.createStorageRecord,
      githubToken: inputs.githubToken
    })

@@ -82,31 +79,11 @@ export async function run(inputs: RunInputs): Promise<void> {
      flag: 'a'
    })

-    const baseDir = process.env.RUNNER_TEMP
-    /* istanbul ignore else */
-    if (baseDir) {
-      const outputSummaryPath = path.join(baseDir, ATTESTATION_PATHS_FILE_NAME)
-      // Append the output path to the attestations paths file
-      fs.appendFileSync(outputSummaryPath, outputPath + os.EOL, {
-        encoding: 'utf-8',
-        flag: 'a'
-      })
-    } else {
-      core.warning(
-        'RUNNER_TEMP environment variable is not set. Cannot write attestation paths file.'
-      )
-    }
-
-    /* istanbul ignore else */
    if (att.attestationID) {
      core.setOutput('attestation-id', att.attestationID)
      core.setOutput('attestation-url', attestationURL(att.attestationID))
    }
-    if (att.storageRecordIds) {
-      core.setOutput('storage-record-ids', att.storageRecordIds.join(','))
-    }

-    /* istanbul ignore else */
    if (inputs.showSummary) {
      await logSummary(att)
    }
@@ -164,7 +141,6 @@ const logAttestation = (
    core.info(`${SEARCH_PUBLIC_GOOD_URL}?logIndex=${attestation.tlogID}`)
  }

-  /* istanbul ignore else */
  if (attestation.attestationID) {
    core.info(style.highlight('Attestation uploaded to repository'))
    core.info(attestationURL(attestation.attestationID))
@@ -174,18 +150,12 @@ const logAttestation = (
    core.info(style.highlight('Attestation uploaded to registry'))
    core.info(`${subjects[0].name}@${attestation.attestationDigest}`)
  }
-
-  if (attestation.storageRecordIds && attestation.storageRecordIds.length > 0) {
-    core.info(style.highlight('Storage record created'))
-    core.info(`Storage record IDs: ${attestation.storageRecordIds.join(',')}`)
-  }
 }

 // Attach summary information to the GitHub Actions run
 const logSummary = async (attestation: AttestResult): Promise<void> => {
  const { attestationID } = attestation

-  /* istanbul ignore else */
  if (attestationID) {
    const url = attestationURL(attestationID)
    core.summary.addHeading('Attestation Created', 3)
@@ -181,14 +181,8 @@ const getSubjectFromChecksumsString = (checksums: string): Subject[] => {
      continue
    }

-    // It's common for checksum records to have a leading flag character before
-    // the artifact name. It will be either a '*' or a space.
-    const flag_and_name = record.slice(delimIndex + 1)
-    const name =
-      flag_and_name.startsWith('*') || flag_and_name.startsWith(' ')
-        ? flag_and_name.slice(1)
-        : flag_and_name
-
+    // Swallow the type identifier character at the beginning of the name
+    const name = record.slice(delimIndex + 2)
    const digest = record.slice(0, delimIndex)

    if (!HEX_STRING_RE.test(digest)) {
@@ -5,7 +5,6 @@
    "module": "NodeNext",
    "rootDir": "./src",
    "moduleResolution": "NodeNext",
-    "isolatedModules": true,
    "baseUrl": "./",
    "sourceMap": true,
    "outDir": "./dist",
@@ -16,5 +15,5 @@
    "skipLibCheck": true,
    "newLine": "lf"
  },
-  "exclude": ["./dist", "./node_modules", "./__tests__", "./coverage", "./jest.config.ts"]
+  "exclude": ["./dist", "./node_modules", "./__tests__", "./coverage"]
 }
@@ -1,10 +0,0 @@
-{
-  "$schema": "https://json.schemastore.org/tsconfig",
-  "extends": "./tsconfig.json",
-  "compilerOptions": {
-    "noEmit": true,
-    "rootDir": "."
-  },
-  "include": ["./__tests__/**/*", "./src/**/*", "./jest.config.ts"],
-  "exclude": ["./dist", "./node_modules", "./coverage", "*.json"]
-}
Author	SHA1	Message	Date
Brian DeHamer	8cb9193776	new OCI compat mode flag Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-10 09:41:02 -07:00
Brian DeHamer	99cf707746	another take on an empty config Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-10 08:47:16 -07:00
Brian DeHamer	6e1e49a5d4	try empty descriptor Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-09 17:41:13 -07:00
Brian DeHamer	c8edbd7693	redefine empty blob Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-09 16:41:18 -07:00
Brian DeHamer	f4f308e094	tweak logging Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-09 14:43:34 -07:00
Brian DeHamer	3a105d9b34	dump probe response headers Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-09 10:56:02 -07:00
Brian DeHamer	bb7f52f719	more debug logging Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-09 10:33:50 -07:00
Brian DeHamer	6d85257406	experiment w/ OCI header auth Signed-off-by: Brian DeHamer <bdehamer@github.com>	2025-04-03 17:47:14 -07:00
@@ -1 +1 @@
 .5.0
 .6.0