Fix npm release workflow failures - #2422
Conversation
WalkthroughThe PR updates CI/CD workflow configuration and package manifests, including enhanced swapfile management in the npm release workflow and version rollbacks across related packages with repository metadata additions. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai assess this PR size classification for the totality of the PR with the following criterias and report it in your comment: S/M/L PR Classification Guidelines:This guide helps classify merged pull requests by effort and complexity rather than just line count. The goal is to assess the difficulty and scope of changes after they have been completed. Small (S)Characteristics:
Review Effort: Would have taken 5-10 minutes Examples:
Medium (M)Characteristics:
Review Effort: Would have taken 15-30 minutes Examples:
Large (L)Characteristics:
Review Effort: Would have taken 45+ minutes Examples:
Additional Factors to ConsiderWhen deciding between sizes, also consider:
Notes:
|
|
The author of this PR is on the CodeRabbit Free Plan. In order to use the Chat feature, please upgrade the PR author to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. |
Motivation
The npm release workflow has been failing for approximately two months due to multiple issues:
Missing repository field: The
ui-componentspackage.json was missing therepositoryfield, causing npm OIDC publish to fail with a 422 error (npm requires the repository URL to match the git remote for provenance attestation)Version mismatch: Package versions in the repo (0.0.1-alpha.197) were out of sync with what's actually published on npm (0.0.1-alpha.196), causing dependency resolution failures during CI
Swap file creation: The swap file creation step could fail with "Text file busy" if the file already existed from a previous run
Solution
repositoryfield topackages/ui-components/package.jsonto match the orderbook package configuration0.0.1-alpha.196to match what's published on npm, allowing the workflow to properly bump and publishChecks
By submitting this for review, I'm confirming I've done the following:
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.