Skip to content

fix: run npm ci in ts/ in release-verify gate - #15

Merged
abienkowski merged 1 commit into
mainfrom
fix/release-npm-ci
Aug 18, 2026
Merged

fix: run npm ci in ts/ in release-verify gate#15
abienkowski merged 1 commit into
mainfrom
fix/release-npm-ci

Conversation

@abienkowski

Copy link
Copy Markdown
Collaborator

Problem

The Release workflow (release.yml) has been failing on every push to main (#12, #13, #14) at the release-verify job's npm ci step:

npm error code EUSAGE
npm error The `npm ci` command can only install with an existing package-lock.json...

npm ci ran at the repo root, which has no package-lock.json (it lives in ts/). Because release-verify is the first job, all downstream jobs (version, release-{go,rs,ts}, publish-release) were skipped — so no release could be published.

Fix

Scope npm ci to the ts/ working directory, mirroring the CI fix already merged in #14.

Verification

Part of epic #4; unblocks #6 and #8.

The release-verify job ran `npm ci` at the repo root, where there is no
package-lock.json, so it failed with EUSAGE and blocked the entire release
pipeline (version/build/sign/publish jobs were skipped). Mirror the CI fix
from #14 by scoping npm ci to the ts/ working directory.
@abienkowski
abienkowski merged commit 0238d46 into main Aug 18, 2026
6 checks passed
@abienkowski
abienkowski deleted the fix/release-npm-ci branch August 18, 2026 04:59
abienkowski added a commit that referenced this pull request Aug 18, 2026
The publish-release job has no checkout step, so gh CLI cannot infer the
target repository from git context:

  failed to run git: fatal: not a git repository (or any of the parent
  directories): .git

This was the third and final blocker in the release pipeline (after #15's
npm ci fix and #16's Quint install fix). v0.2.7 was manually published via
'gh release edit' after confirming all its artifacts (Go/Rust/TS
binaries+archive, 6 SBOMs, Cosign signatures, public GHCR images) were
already valid. This fix ensures the *next* release (v0.2.8+) auto-publishes
without manual intervention.
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