Skip to content

Use npm trusted publishing#17

Merged
roodboi merged 1 commit into
mainfrom
codex/trusted-publishing
Jul 11, 2026
Merged

Use npm trusted publishing#17
roodboi merged 1 commit into
mainfrom
codex/trusted-publishing

Conversation

@roodboi

@roodboi roodboi commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

What changed

  • grant the publish job the OIDC permission required by npm trusted publishing
  • remove the long-lived NPM_TOKEN from the workflow
  • extend the workflow security test to require OIDC and reject npm secrets

Why

The 4.1.0 release reached npm but failed because the standalone repository was still using a token that cannot publish schema-stream. Island AI already publishes this package through npm's repository-bound trusted publisher; the standalone workflow should use the same short-lived credential model.

The npm trusted-publisher binding must target hack-dance/schema-stream, publish.yml, and the PUBLISH environment.

Validation

  • bun run format
  • bun run lint
  • bun test tests/workflow-security.test.ts
  • bun run type-check
  • git diff --check

@roodboi
roodboi merged commit e763daa into main Jul 11, 2026
4 checks passed
@roodboi
roodboi deleted the codex/trusted-publishing branch July 11, 2026 00:33
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.

1 participant