Skip to content

chore(identity-webhook): npm publish CI and v0.3.0 package prep#60

Open
eliteprox wants to merge 2 commits into
mainfrom
chore/npm-publish-ci
Open

chore(identity-webhook): npm publish CI and v0.3.0 package prep#60
eliteprox wants to merge 2 commits into
mainfrom
chore/npm-publish-ci

Conversation

@eliteprox

@eliteprox eliteprox commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Publish @livepeer/clearinghouse-identity-webhook to npm on v*.*.* tags via OIDC trusted publishing (release.yml).
  • One-time bootstrap-publish.yml for the first scoped package create (npm OIDC cannot create a missing package).
  • bump-version.yml + dependabot.yml; CI npm pack --dry-run.
  • Package prep: public exports (./protocol, ./verifiers), TypeScript types, MIT LICENSE, RELEASING docs.

Out of scope

Test plan

  • cd identity-webhook && npm ci && npm test && npm pack --dry-run --ignore-scripts
  • Merge PR
  • Run bootstrap-publish once with NPM_TOKEN for v0.3.0 (or create the package on npmjs.com manually)
  • Configure npm trusted publishing for release.yml, then delete NPM_TOKEN
  • Push later v*.*.* tags and verify release publishes via OIDC

One-time npm setup (after merge)

See RELEASING.md.

Copilot AI review requested due to automatic review settings July 8, 2026 21:18

Copilot AI left a comment

Copy link
Copy Markdown

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 prepares identity-webhook/ for its first public npm release and adds CI/CD automation to publish @livepeer/clearinghouse-identity-webhook on semver tags using npm OIDC trusted publishing.

Changes:

  • Add GitHub Actions workflows for tag-based npm publishing (release.yml), manual semver bumps (bump-version.yml), and extend CI with npm pack --dry-run.
  • Update identity-webhook/package.json for v0.3.0 publishing metadata (exports map, files list, repository/homepage/keywords, public access).
  • Add release documentation plus legacy env helpers and tests to support embedded/legacy integration.

Reviewed changes

Copilot reviewed 10 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
README.md Documents npm package publishing location and links to releasing guide.
identity-webhook/protocol.d.ts Adds public TypeScript declarations for the webhook protocol module.
identity-webhook/package.json Prepares package for public publish (version bump, exports/files, metadata).
identity-webhook/package-lock.json Updates lockfile version fields for v0.3.0.
identity-webhook/legacy-env.test.mjs Adds tests for legacy env mapping + an end-to-end authorize flow.
identity-webhook/legacy-env.mjs Introduces legacy env variable mapping helpers for OIDC verifier config.
identity-webhook/legacy-env.d.ts Adds TypeScript declarations for legacy env helper entrypoint.
identity-webhook/docs/RELEASING.md Documents OIDC trusted publishing setup and release process.
.github/workflows/release.yml Implements tag-triggered npm publish via OIDC + GitHub Release creation.
.github/workflows/ci.yml Extends CI to include npm pack --dry-run for identity-webhook changes.
.github/workflows/bump-version.yml Adds manual workflow to bump version, commit, tag, and push.
.github/REPO_SETUP.md Updates repo automation checklist to reflect new workflows.
.github/dependabot.yml Adds weekly Dependabot updates for identity-webhook npm deps and GH Actions.
Files not reviewed (1)
  • identity-webhook/package-lock.json: Generated file

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

Comment thread identity-webhook/package.json Outdated
Comment thread identity-webhook/package.json Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/release.yml
@eliteprox

Copy link
Copy Markdown
Collaborator Author

@rickstaa I'd like to publish this to npm as @livepeer/clearinghouse-identity-webhook

Copilot AI review requested due to automatic review settings July 8, 2026 21:47
@eliteprox
eliteprox requested a review from rickstaa July 8, 2026 21:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • identity-webhook/package-lock.json: Generated file

Comment thread identity-webhook/package.json Outdated
Comment thread identity-webhook/package.json
@eliteprox

Copy link
Copy Markdown
Collaborator Author

Added a livepeer-scoped bootstrap-publish workflow for the first npm create (OIDC alone E404s on a missing package), plus RELEASING.md steps. After the first publish, configure trusted publishing and delete NPM_TOKEN.

Stacked follow-ups (coming next):

  1. OIDC JWKS discovery
  2. Composite app_*.pmth_* token exchange

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 15 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • identity-webhook/package-lock.json: Generated file

Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/bootstrap-publish.yml Outdated
Add release, bump-version, bootstrap-publish, and dependabot workflows for
publishing @livepeer/clearinghouse-identity-webhook via npm trusted
publishing (OIDC). Prepare package exports/types and RELEASING docs; add
repo MIT LICENSE. Checkout publish tags via refs/tags/ for safety.
Copilot AI review requested due to automatic review settings July 11, 2026 00:35
@eliteprox
eliteprox force-pushed the chore/npm-publish-ci branch from e2805f8 to 6a0a865 Compare July 11, 2026 00:35
@eliteprox eliteprox changed the title chore(identity-webhook): add npm publish CI and prepare v0.3.0 chore(identity-webhook): npm publish CI and v0.3.0 package prep Jul 11, 2026
@eliteprox

Copy link
Copy Markdown
Collaborator Author

Cleaned this PR to publish-only: removed legacy-env packaging, kept workflows + package exports/types + LICENSE. Tag checkouts now use refs/tags/….

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 13 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • identity-webhook/package-lock.json: Generated file

Comment on lines +24 to +28
"files": [
"protocol.mjs",
"protocol.d.ts",
"verifiers.mjs",
"verifiers.d.ts",
…ackage

Introduce MIT License and comprehensive README documentation for the @livepeer/clearinghouse-identity-webhook package, detailing installation, usage, and environment configuration for identity verification. This enhances clarity and accessibility for developers integrating the identity webhook into their projects.
Copilot AI review requested due to automatic review settings July 13, 2026 19:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants