Open
Conversation
- Add comprehensive outputSchema to get and list order tools - Add additional filter parameters to listOrders (authorizedDate, invoicedDate, utmSource, sellerNames, paymentNames, etc.) - Improve input descriptions with examples and valid values - Update client.ts to support new filter parameters
- Added new tools for managing collections, including creating, updating, deleting, and retrieving collections and their products. - Introduced types for collections and their responses to enhance type safety and clarity in the codebase. - Updated package.json and bun.lock to include necessary dependencies for Cloudflare Workers types. ndencies for Cloudflare Workers types.
🚀 Preview Deployments Ready!Your changes have been deployed to preview environments: 📦
|
Contributor
There was a problem hiding this comment.
3 issues found across 66 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="vtex/server/tools/collection/add-sku.ts">
<violation number="1" location="vtex/server/tools/collection/add-sku.ts:13">
P2: Use the same `skuId` input shape as other collection tools to avoid callers failing validation with the wrong field casing.</violation>
</file>
<file name="vtex/scripts/download-schemas.ts">
<violation number="1" location="vtex/scripts/download-schemas.ts:48">
P2: Ensure the script exits with a non-zero code on failure; currently the error is only logged, so automation can’t detect a failed download.</violation>
</file>
<file name="object-storage/package.json">
<violation number="1" location="object-storage/package.json:8">
P1: The dev script is now empty (`bun run` without a target), so running `bun run dev` won’t start the server and will exit with an error. Restore the intended dev command.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
…error handling in schema download - Modified the dev script in package.json to enable hot reloading for the server. - Improved error handling in downloadSchemas function to log errors and exit the process on failure. - Corrected casing of SKU ID in add-sku.ts for consistency and clarity.
viktormarinho
approved these changes
Feb 6, 2026
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
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.
Summary by cubic
Adds a new VTEX Commerce MCP that integrates Catalog, Collections, Pricing, Orders, Inventory, and Warehouses, exposing tools to manage products, SKUs, collections, orders, stock, and prices. Includes HTTP MCP config schema, typed client, OpenAPI scripts, and local dev improvements (hot reload and better schema download error handling).
New Features
Migration
bun run download-schemasthenbun run openapito refresh generated clients.bun run build/npm run deploy).Written for commit b55f45a. Summary will update on new commits.