Skip to content

Commit 30615da

Browse files
committed
Align trusted publish job with known-good workflow
1 parent 264f2eb commit 30615da

2 files changed

Lines changed: 21 additions & 2 deletions

File tree

.github/workflows/tagged_release.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,10 @@ jobs:
4444
uses: actions/setup-node@v4
4545
with:
4646
node-version: 20
47+
registry-url: 'https://registry.npmjs.org'
48+
49+
- name: Upgrade npm for trusted publishing
50+
run: npm install -g npm@latest && npm --version
4751
architecture: ${{ matrix.node_arch }}
4852
cache: npm
4953

@@ -135,5 +139,3 @@ jobs:
135139
136140
- name: Publish tagged release
137141
run: npm publish --provenance
138-
env:
139-
NODE_AUTH_TOKEN: ''

RUNNER-FIX-HISTORY.md

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -426,6 +426,23 @@ This file records the CI/workflow fix iterations so another agent can continue f
426426
- `.github/workflows/tagged_release.yaml`
427427
- remove `registry-url` from publish job `actions/setup-node` step.
428428
- explicitly clear `NODE_AUTH_TOKEN` in publish step to force tokenless trusted publishing path.
429+
430+
### Iteration AL (in progress)
431+
- GitHub run checked: `22653458636` (`Build & Publish tagged release`) for tag `v0.4.5`.
432+
- Outcome:
433+
- Build matrix and GitHub release creation succeeded.
434+
- npm publish failed with `ENEEDAUTH` (no npm auth in effective publish context).
435+
- Cross-check performed:
436+
- compared against working workflow in `/Users/steph/Workspace/repeato-native-cv/.github/workflows/publish.yml`.
437+
- key differences in working workflow publish job:
438+
- `actions/setup-node` includes `registry-url: https://registry.npmjs.org`.
439+
- npm is upgraded before publish.
440+
- no forced empty `NODE_AUTH_TOKEN` override.
441+
- Current local fix:
442+
- `.github/workflows/tagged_release.yaml`
443+
- restore `registry-url` in publish job `setup-node` step.
444+
- add `Upgrade npm for trusted publishing` step.
445+
- remove forced `NODE_AUTH_TOKEN: ''` override from publish step.
429446
## Current Hypothesis
430447
Primary remaining blocker has shifted from crash/fatal errors to CLI capability variance on the Windows runner (notably `tsv` config availability).
431448

0 commit comments

Comments
 (0)