Conversation
Contributor
Review Summary by QodoAdd v3 release workflow with validation and publishing
WalkthroughsDescription• Add v3 release workflow triggered manually from vibe3 branch • Validate branch, notify Slack, build and test before release • Generate versions using Lerna with conventional commits • Publish packages to npm with v3 dist-tag Diagramflowchart LR
A["Workflow Dispatch"] --> B["Validate Branch"]
B --> C["Notify Slack"]
B --> D["Build"]
D --> E["Test"]
E --> F["Release"]
F --> G["Generate Versions"]
G --> H["Publish to npm"]
File Changes1. .github/workflows/release-v3.yml
|
Contributor
Code Review by Qodo
1. Publish is dry-run
|
![qodo-free-for-open-source-projects[bot]](https://avatars.githubusercontent.com/in/1068000?s=60&v=4){kind=small}
Comment on lines
+81
to
+82
| - name: Publish to npm with v3 dist-tag | ||
| run: yarn lerna publish from-package --dist-tag v3 --dry-run -y |
Contributor
There was a problem hiding this comment.
1. Publish is dry-run 🐞 Bug ✓ Correctness
release-v3.yml runs yarn lerna publish ... --dry-run, so no packages are actually published even though the workflow also bumps versions and creates a GitHub release, leaving the repo/releases ahead of npm. This can produce a broken release state where consumers cannot install the released versions.
Agent Prompt
### Issue description
The v3 release workflow does not publish anything because it runs `yarn lerna publish ... --dry-run`, despite creating version bumps/GitHub releases.
### Issue Context
This can create GitHub tags/releases and bumped versions without corresponding npm artifacts.
### Fix Focus Areas
- .github/workflows/release-v3.yml[81-82]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Contributor
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
https://monday.monday.com/boards/10027056258/pulses/10027056856