Skip to content

Conversation

@vc-ci
Copy link
Contributor

@vc-ci vc-ci commented Jan 29, 2026

ci: Automated update vc-module-x-api from PR undefined undefined


Note

Medium Risk
Medium risk because it changes where VirtoCommerce.Xapi is sourced from (blob vs GitHub releases), which can alter deployed module bits and compatibility at runtime despite being a config-only change.

Overview
Updates backend/packages.json to pull VirtoCommerce.Xapi from the Azure Blob package source at version 3.1001.0 (zip), removing the prior 3.1000.0 GitHub releases module entry.

Also includes minor JSON formatting/whitespace adjustments and leaves the file without a trailing newline.

Written by Cursor Bugbot for commit 7c225ee. This will update automatically on new commits. Configure here.

@CLAassistant
Copy link

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.

Copy link

@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.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

},
{
"Id": "VirtoCommerce.Xapi",
"BlobName": "VirtoCommerce.Xapi_3.1001.0.zip"
Copy link

Choose a reason for hiding this comment

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

Blob name missing PR suffix suggests automation failure

Medium Severity

The BlobName for VirtoCommerce.Xapi is VirtoCommerce.Xapi_3.1001.0.zip, which lacks the -pr-XX-XXXX suffix present in all other AzureBlob entries. Combined with the PR description showing "from PR undefined undefined", this suggests the automation failed to capture source PR information. The blob artifact may not exist in the prerelease container, or the name may be incorrect, potentially causing module loading failures at deployment time.

Fix in Cursor Fix in Web

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.

3 participants