Skip to content

pr-vcptcore-qa-07-04-2026-a66d#5646

Open
vc-ci wants to merge 1 commit intovcptcore-qafrom
pr-vcptcore-qa-07-04-2026-a66d
Open

pr-vcptcore-qa-07-04-2026-a66d#5646
vc-ci wants to merge 1 commit intovcptcore-qafrom
pr-vcptcore-qa-07-04-2026-a66d

Conversation

@vc-ci
Copy link
Copy Markdown
Contributor

@vc-ci vc-ci commented Apr 7, 2026

ci: Automated update vc-module-catalog-publishing from PR undefined undefined


Note

Low Risk
Low risk manifest change that only alters which CatalogPublishing package artifact is pulled; main risk is deploying an unintended module version if the new blob name is incorrect.

Overview
Updates backend/packages.json to pull VirtoCommerce.CatalogPublishing from VirtoCommerce.CatalogPublishing_3.1002.0.zip instead of the previous PR build (3.1001.0-pr-72-48ca.zip).

Also applies minor JSON whitespace/formatting cleanup (including file ending newline change).

Reviewed by Cursor Bugbot for commit 2af9bd0. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2af9bd0. Configure here.

},
{
"Id": "VirtoCommerce.CatalogPublishing",
"BlobName": "VirtoCommerce.CatalogPublishing_3.1002.0.zip"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Release artifact referenced in pre-release blob container

Medium Severity

The CatalogPublishing BlobName changed from a PR build (_3.1001.0-pr-72-48ca.zip) to a release-style version (_3.1002.0.zip) within the vc3prerelease AzureBlob source. Every other module in this source uses a -pr- suffixed artifact. The PR description confirms the automation broke: "from PR undefined undefined" — the CI script failed to resolve the source PR, likely producing an incorrect artifact reference that may not exist in the pre-release container.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2af9bd0. Configure here.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

2 participants