GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — ci(release): skip npm packages an earlier attempt already staged by mikolalysenko · Pull Request #1499 · SocketDev/socket-patch

Conversation

@mikolalysenko

@mikolalysenko Mikola Lysenko (mikolalysenko) commented Oct 13, 2026 •
edited by cursor Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Why

5.0.1's main npm package (@socketsecurity/socket-patch) never staged: the release run got E401 on it after its publisher was fixed. The npm-only retry (run 38449949240) then failed on the first platform package with 409 Cannot stage previously published version "5.0.1", because the 14 platform packages were already staged.

The "already staged" fallback runs npm stage list. That can't authenticate in CI, because the OIDC exchange only happens on publish. So the job never reached the main package.

What

Both stage steps in release.yml's npm-publish job now use one inline stage() helper. If the version is already published, it skips. If the registry answers with that 409 message, it counts the package as staged and moves on. Any other failure still fails the step.

The helper is inline, not a script, because the job checks out the release tag (v5.0.1), which doesn't have any new script.

After merge

Actions → Release → npm-retry-version: 5.0.1 stages @socketsecurity/[email protected] from CI, with signed provenance like the platform packages got. Then approve it with 2FA after the 14 platform packages.

actionlint on release.yml passes.

🤖 Generated with Claude Code


Note

Low Risk
Release workflow-only change to npm staging retry behavior; no runtime or registry credential changes beyond skipping known 409/published cases.

Overview
The npm-publish job’s platform and main stage-publish steps now share an inline stage() helper that makes partial retries idempotent.

Instead of treating npm stage list as the “already staged” signal (which fails in CI because OIDC only runs on publish), a failed npm stage publish is skipped when npm view shows the version is already live, or when registry output contains Cannot stage previously published version (409 for a staged-but-unapproved version). Those packages are still recorded in staged-packages.txt; any other error still fails the step. The helper stays inline so npm-retry-version runs work when the job checks out an older release tag.

Reviewed by Cursor Bugbot for commit e226d75. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@mikolalysenko @claude
ci(release): skip npm packages an earlier attempt already staged
e226d75
A retry of 5.0.1's npm publish (run 38449949240) failed on the first
platform package with `409 Cannot stage previously published version`:
the 14 platform packages were already staged by the release run. The
fallback that should have skipped them runs `npm stage list`, which
can't authenticate in CI because OIDC is exchanged only on publish, so
the job never reached the main package.

Treat that 409 message as "already staged" in both stage steps, through
one inline stage() helper. It stays inline because the job checks out
the release tag, which predates any new script.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@mikolalysenko
Mikola Lysenko (mikolalysenko) enabled auto-merge October 13, 2026 18:55
Tanmay182003
Tanmay Singla (Tanmay182003) approved these changes Oct 13, 2026
View reviewed changes
Tanmay182003
Tanmay Singla (Tanmay182003) approved these changes Oct 13, 2026
View reviewed changes
@mikolalysenko Mikola Lysenko (mikolalysenko) mentioned this pull request Oct 13, 2026
Swarm health manager log #1382
Open
@mikolalysenko
Mikola Lysenko (mikolalysenko) added this pull request to the merge queue Oct 13, 2026
Merged via the queue into main with commit 30de647 Oct 13, 2026
55 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@mikolalysenko
Mikola Lysenko (mikolalysenko) deleted the ci/npm-retry-skip-staged branch October 13, 2026 19:17
@mikolalysenko Mikola Lysenko (mikolalysenko) mentioned this pull request Oct 13, 2026
v5.0.0 not published: registry trusted publishers still name release.yml #1456
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Tanmay182003 Tanmay Singla (Tanmay182003) Tanmay182003 approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@mikolalysenko @Tanmay182003
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.