Skip to content

feat: [MCP] BigCommerce Integration#12262

Open
Angelebeats wants to merge 2 commits intoactivepieces:mainfrom
Angelebeats:feature/bigcommerce-mcp
Open

feat: [MCP] BigCommerce Integration#12262
Angelebeats wants to merge 2 commits intoactivepieces:mainfrom
Angelebeats:feature/bigcommerce-mcp

Conversation

@Angelebeats
Copy link
Copy Markdown
Contributor

@Angelebeats Angelebeats commented Mar 29, 2026 •
edited
Loading

Description

This PR upgrades the existing BigCommerce piece to meet the MCP (Model Context Protocol) standards as part of the Activepieces bounty program.

Changes:

  • Renamed to BigCommerce [MCP] for better discoverability.
  • Added Actions:
    • Update a Product
    • Delete a Product
    • Get Order (Fetch specific order details)
    • List Orders (Filter by Customer/Status)
    • List Categories (Fetch all store categories)
  • Enhanced Service Layer: Updated requests.ts with V3/V2 API endpoints for the new actions.
  • Registration: Added @activepieces/piece-bigcommerce to tsconfig.base.json path mappings.
  • Quality: Added .eslintrc.json for linting compliance.

Fixes #9847

Latest Updates (Mar 29):

  • Fixed Data Overwrite Risk: Added null/undefined filters in Update Product action.
  • Fixed Lint Path: Corrected .eslintrc.json relative path (4 levels deep).
  • Standardized Metadata: Corrected Display Name to 'BigCommerce' and author ID to 'Angelebeats'.
  • Fixed Code Formatting: Added missing trailing newline to tsconfig.base.json.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 29, 2026 •
edited
Loading

Confidence Score: 5/5

PR is safe to merge — all previous review concerns are resolved and no new issues were found.

All P0/P1 findings from earlier rounds have been addressed: null-overwrite risk in Update Product is fixed, list-orders.ts exists and is registered, the eslintrc path is correct, the non-standard display name suffix is removed, and the spurious tsconfig entry is gone. No new correctness or security issues were identified.

No files require special attention.

Vulnerabilities

No security concerns identified. Auth tokens are passed via the X-Auth-Token header through the existing fireHttpRequest helper, consistent with the rest of the piece. No secrets are hardcoded or logged.

Important Files Changed

Filename Overview
packages/pieces/community/bigcommerce/src/lib/actions/update-a-product.ts Update Product action with proper null filtering to prevent overwriting existing fields with nulls; the filter is applied both in the action and redundantly via filter: true in requests.ts, but this is harmless.
packages/pieces/community/bigcommerce/src/lib/actions/delete-a-product.ts Clean, minimal Delete Product action; correctly delegates to the V3 catalog endpoint.
packages/pieces/community/bigcommerce/src/lib/actions/get-order.ts Clean Get Order action fetching from the V2 orders endpoint; no issues found.
packages/pieces/community/bigcommerce/src/lib/actions/list-orders.ts List Orders action with optional customer_id/status_id filters; null-guard checks are correct for optional Number props.
packages/pieces/community/bigcommerce/src/lib/actions/list-categories.ts List Categories action with optional name filter; correctly constructs query string before calling the V3 catalog categories endpoint.
packages/pieces/community/bigcommerce/src/lib/common/requests.ts Service layer extended with updateProduct, deleteProduct, fetchOrders, fetchOrder, and fetchCategories; filterNulls helper correctly placed after exported const; any types on existing methods replaced with unknown.
packages/pieces/community/bigcommerce/src/index.ts Registers all five new actions; displayName corrected to 'BigCommerce' without the non-standard [MCP] suffix; author added.
packages/pieces/community/bigcommerce/.eslintrc.json Extends ../../../../.eslintrc.json (4 levels up to repo root) — now matches the standard used by all sibling community pieces.
tsconfig.base.json Adds @activepieces/piece-bigcommerce path mapping in correct alphabetical position; the spurious @activepieces/piece-canva entry from an earlier version is no longer present.
packages/pieces/community/bigcommerce/package.json Version bumped from 0.1.4 to 0.1.5 — correct patch bump for non-breaking additions.

Reviews (11): Last reviewed commit: "Merge branch 'main' into feature/bigcomm..." | Re-trigger Greptile

greptile-apps[bot]
greptile-apps bot reviewed Mar 29, 2026
View reviewed changes
@Angelebeats Angelebeats force-pushed the feature/bigcommerce-mcp branch from 8c1a2c4 to 3a93fac Compare March 29, 2026 10:20
greptile-apps[bot]
greptile-apps bot reviewed Mar 29, 2026
View reviewed changes
@abuaboud abuaboud added the 🌟 feature new major feature label Apr 1, 2026
@Angelebeats Angelebeats force-pushed the feature/bigcommerce-mcp branch from 4763e97 to 019e830 Compare April 1, 2026 16:08
@onyedikachi-david
Copy link
Copy Markdown
Collaborator

onyedikachi-david commented Apr 2, 2026

@Angelebeats Thanks for contributing, was this tested, if yes, please share the credentials, you can send them to my discord handle -- abi.encode(onyedikachi), also undo the changes that are out of scope with the feat and bump the piece version bump docs

@kishanprmr kishanprmr removed the 🌟 feature new major feature label Apr 2, 2026
@Angelebeats Angelebeats force-pushed the feature/bigcommerce-mcp branch 2 times, most recently from 886c612 to 8dba14f Compare April 2, 2026 13:01
@Angelebeats
Copy link
Copy Markdown
Contributor Author

Angelebeats commented Apr 2, 2026

Updated the PR to address feedback:

  • Bumped version to 0.1.5.
  • Refactored ilterNulls to be applied only to Product Creation/Update actions as requested.
  • Cleaned up unrelated metadata files (:Zone.Identifier) from the branch.
  • Validated all v3 catalog endpoint mappings.

@Angelebeats Angelebeats force-pushed the feature/bigcommerce-mcp branch 3 times, most recently from 9942deb to d6ff9fd Compare April 2, 2026 13:47
@Angelebeats Angelebeats force-pushed the feature/bigcommerce-mcp branch from d6ff9fd to 46a3919 Compare April 2, 2026 13:47
@Angelebeats
Copy link
Copy Markdown
Contributor Author

Angelebeats commented Apr 2, 2026

@onyedikachi-david I've updated the PR to remove out-of-scope files and fix the versioning. For testing, I've prepared a trial store. The credentials (Store Hash and Access Token) are being sent to your Discord (�bi.encode(onyedikachi)) by the contributor. Please let us know if you need anything else to proceed with the merge.

@Angelebeats
Copy link
Copy Markdown
Contributor Author

Angelebeats commented Apr 3, 2026

Hi @onyedikachi-david, I have sent a friend request to you on Discord (my handle is Angelebeats) to provide the test credentials and report for this BigCommerce MCP PR. Please accept it so we can move forward with the verification. Thank you!

@Angelebeats
Copy link
Copy Markdown
Contributor Author

Angelebeats commented Apr 7, 2026

@onyedikachi-david Hi, I sent a friend request on Discord (Angelebeats) but it hasn't been accepted yet. Is there another way (e.g., email or a secure link) I can send the Store Hash and Token to you so we can finalize the testing? Thank you!

@onyedikachi-david
Copy link
Copy Markdown
Collaborator

onyedikachi-david commented Apr 8, 2026

@onyedikachi-david Hi, I sent a friend request on Discord (Angelebeats) but it hasn't been accepted yet. Is there another way (e.g., email or a secure link) I can send the Store Hash and Token to you so we can finalize the testing? Thank you!

I am yet to see your request, please join Activepieces discord server, if you haven't, let me know when you leave a message in the general channel. https://discord.gg/ScPCK6Mp

@Angelebeats
Copy link
Copy Markdown
Contributor Author

Angelebeats commented Apr 8, 2026

@onyedikachi-david PR has been fully cleaned up and rebased.

  • Reverted all out-of-scope changes (including tsconfig.base.json and metadata files).
  • Piece version bumped to 0.1.5 in package.json.
  • All 5 new BigCommerce actions and type safety fixes are included.
    The PR is now pure and focused only on the BigCommerce piece upgrade. Ready for merge. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@onyedikachi-david onyedikachi-david

Labels

🧩 area/third-party-pieces

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[MPC] BigCommerce

4 participants

@Angelebeats @onyedikachi-david @abuaboud @kishanprmr