Skip to content

fix(release): tolerate npm dist-tag propagation lag - #95

Merged
0xjcf merged 3 commits into
betafrom
fas/beta-dist-tag-verification
Jul 18, 2026
Merged

fix(release): tolerate npm dist-tag propagation lag#95
0xjcf merged 3 commits into
betafrom
fas/beta-dist-tag-verification

Conversation

@0xjcf

@0xjcf 0xjcf commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • prefer online npm registry reads during post-publish dist-tag verification
  • retry stale registry snapshots for a bounded 14-second convergence window
  • distinguish dist-tag write failures from post-write verification lag
  • recommend an online recheck before asking for a fresh OTP repair
  • cover convergence, persistent mismatch, and recovery guidance deterministically

Why

The beta.9 packages published successfully, but the immediate npm view reads returned stale dist-tags. The wrapper treated that temporary registry propagation lag as a release failure and printed OTP repair commands even though no repair was ultimately needed.

Verification

  • node --test scripts/release-beta.test.mjs (9/9)
  • pnpm run test:scripts (31/31)
  • pnpm release:beta --dry-run (inert publish, all tests and builds passed)
  • fas validate-task (fast lane passed)
  • fas verify --full (all seven lanes passed)
  • pre-push pnpm test:full (641 package tests, 31 script tests, and all 9 example roots passed; Voice Workbench 272/272)

No changeset is included because this changes repository release tooling only; beta.9 package contents and published tags remain unchanged.

@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 65c2f99f-3073-4a38-b5d4-fbd3b2d9d49b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fas/beta-dist-tag-verification

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.16%. Comparing base (03f4e93) to head (acf8b5a).
⚠️ Report is 5 commits behind head on beta.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             beta      #95   +/-   ##
=======================================
  Coverage   90.16%   90.16%           
=======================================
  Files          38       38           
  Lines        3181     3181           
  Branches      948      944    -4     
=======================================
  Hits         2868     2868           
  Misses        290      290           
  Partials       23       23           
Flag Coverage Δ
unittests 90.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@0xjcf
0xjcf marked this pull request as ready for review July 18, 2026 21:40
@0xjcf
0xjcf merged commit c1f1ef5 into beta Jul 18, 2026
6 checks passed
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