Skip to content

Add release workflow publishing to npm - #6

Merged
dkanas merged 1 commit into
mainfrom
release-workflow
Aug 6, 2026
Merged

Add release workflow publishing to npm#6
dkanas merged 1 commit into
mainfrom
release-workflow

Conversation

@dkanas

@dkanas dkanas commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Adds a publish workflow: runs after a successful CI run on main, publishes only when the package.json version changed, then tags the release as v<version>. workflow_dispatch forces a publish.

Switches publishConfig to registry.npmjs.org with public access, so npx @contractbook/mcp works as the README and setup command describe.

Authentication is npm trusted publishing via OIDC.

@dkanas
dkanas force-pushed the release-workflow branch 2 times, most recently from 4b260b5 to 4366b80 Compare August 6, 2026 07:17
Runs on pushes to main, publishes only when the package.json version
changed, then tags the release. workflow_dispatch forces a publish.

Uses a push trigger rather than workflow_run. workflow_run grants write
permissions and OIDC access while running in the base repository, so
checking out the triggering run's head_sha would execute pull request code
from forks in a privileged context. A push to main cannot originate from a
fork.

Switches publishConfig from GitHub Packages to registry.npmjs.org with
public access. The previous target could not serve `npx @contractbook/mcp`,
which is what the README and the setup command tell users to run.

Aligns the CI workflow's action versions with the publish workflow.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@dkanas
dkanas force-pushed the release-workflow branch from 4366b80 to 7658203 Compare August 6, 2026 08:28
@dkanas
dkanas merged commit 910d647 into main Aug 6, 2026
1 check passed
@dkanas dkanas self-assigned this Aug 6, 2026
@dkanas
dkanas deleted the release-workflow branch August 6, 2026 09:19
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