Conversation
PR SummaryMedium Risk Overview Build skipping is moved from In Reviewed by Cursor Bugbot for commit 2a638df. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 553b1f8. Configure here.
| run: ./build.sh "${{ needs.validate.outputs.commit_hash }}" "${{ needs.validate.outputs.version_name }}" "${{ matrix.arch }}" | ||
|
|
||
| - name: Upload build artifact | ||
| if: steps.check_release.outputs.skip != 'true' |
There was a problem hiding this comment.
Publish job fails when all artifacts already exist
High Severity
When all requested artifacts already exist in the release, both build matrix jobs skip the upload-artifact step, producing zero workflow artifacts. The publish job's actions/download-artifact@v8 step then fails with "No artifacts found" (this is documented v4+ behavior), which blocks the deploy job since it declares needs: [validate, publish]. This defeats the PR's stated goal of deploying to all environments on re-runs where artifacts already exist.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 553b1f8. Configure here.
|
|
||
| release_exists = f"firecracker-{arch}" in release_assets | ||
|
|
||
| print(f"Release: {arch} artifact {'exists' if release_exists else 'missing'}", file=sys.stderr) |
There was a problem hiding this comment.
Dead --github-repo argument after removing its only consumer
Low Severity
The --github-repo argument is still defined in the argument parser, but args.github_repo is never referenced anywhere after this diff removed check_existing_artifacts, which was its only consumer. This is dead code left behind during the refactoring that could confuse future developers into thinking repository context is still used by the validation script.
Reviewed by Cursor Bugbot for commit 553b1f8. Configure here.


No description provided.