Skip to content

PR 2825 sequence diagram#2871

Draft
bc-fetisov wants to merge 1 commit intocanaryfrom
cursor/pr-2825-sequence-diagram-4fc6
Draft

PR 2825 sequence diagram#2871
bc-fetisov wants to merge 1 commit intocanaryfrom
cursor/pr-2825-sequence-diagram-4fc6

Conversation

@bc-fetisov
Copy link

What/Why?

This PR adds a new documentation artifact, core/pr-2825-graphql-proxy-sequence-diagram.md, which provides an analysis and a Mermaid sequence diagram for the withGraphqlProxy middleware introduced in bigcommerce/catalyst#2825.

The purpose is to clearly illustrate the request flow, middleware ordering, and key behaviors of the GraphQL proxy, including pass-through scenarios, validation, and successful proxying to the Storefront GraphQL API. This helps in understanding the middleware's integration and functionality.

Testing

This PR introduces a new documentation file. Testing involves reviewing the content of core/pr-2825-graphql-proxy-sequence-diagram.md for accuracy, clarity, and correct representation of the withGraphqlProxy middleware's behavior as implemented in PR #2825.

Migration

No files were moved, and no breaking changes were introduced. This PR adds a new documentation file.


Open in Cursor Open in Web

Co-authored-by: Andrii Fetisov <bc-fetisov@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Feb 10, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@vercel
Copy link

vercel bot commented Feb 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Feb 10, 2026 5:20pm

Request Review

@changeset-bot
Copy link

changeset-bot bot commented Feb 10, 2026

⚠️ No Changeset found

Latest commit: e7e8351

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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

Comments