Skip to content

fix(ci): verify published beta integrity and wait for registry propagation - #74

Merged
rdlabo merged 1 commit into
mainfrom
fix/release-integrity-json
Nov 11, 2026
Merged

rdlabo merged 1 commit into
mainfrom
fix/release-integrity-json

Conversation

@rdlabo

@rdlabo rdlabo commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Duplicate beta release jobs can receive npm E409 while a successful publication is still propagating. Once visible, npm may return dist.integrity as a singleton JSON array. The existing string comparison rejects the matching archive and leaves the release run failed.

Accept the matching digest as either a string or exactly one array entry. Mismatched and ambiguous responses continue to fail. Extend E409 metadata checks to a bounded five minutes without repeating npm publish.

Add seven regression tests that execute the workflow's actual publication logic with a fake registry and timers. They cover matching archives, malformed responses, delayed visibility, the five-minute bound, other errors and immediate integrity mismatch failures. Run these through the existing ESM CI entry point. Build, lint and the full ESM checks passed. The public beta archive matches its immutable CI artifact by SHA-512.

@github-actions

Copy link
Copy Markdown
Contributor

Playwright test results

passed  265 passed

Details

stats  265 tests across 18 suites
duration  3 minutes, 4 seconds
commit  13bbae8
info  This detailed result covers Ionic 9 only. Ionic 8 runs against the same screenshots in a separate matrix job; check the workflow run for both results. To update the screenshots, comment with /update-screenshots.

github-actions Bot added a commit that referenced this pull request Nov 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://webview-native-ui.github.io/ionic-theme-md3/pr-74/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

@rdlabo
rdlabo merged commit 6f1ed97 into main Nov 11, 2026
16 of 17 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