Add same-origin proxy mode for Cypress compatibility#22
Merged
ctrlaltdylan merged 5 commits intomainfrom Mar 2, 2026
Merged
Conversation
Implements Admin API request interception with three modes:
- 'mock': Returns mock data for products, orders, customers (default)
- { proxy: string }: Forwards requests to app's backend proxy
- { accessToken: string }: Direct requests to Shopify with token
Changes:
- Add AdminApiConfig type to src/types/index.ts
- Add /mock-admin-api endpoint with GraphQL and REST handlers
- Update App Bridge to intercept fetch('/admin/api/...') calls
- Add adminApi option to CLI init template
- Update README with API support matrix and configuration docs
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add `files` field to package.json and create admin-frame/.npmignore so that dist/, admin-frame/dist/, and app-bridge/dist/ are included in the published tarball instead of being excluded by .gitignore. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When --proxy is passed, mount http-proxy-middleware at /__proxy/ to reverse-proxy the app through mock-bridge. This makes the iframe same-origin, which is required for Cypress (which cannot interact with cross-origin iframes). WebSocket proxying is included for HMR. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Allows running directly from the feature branch via: npx github:ctrlaltdylan/mock-bridge#http-proxy-middleware Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
--proxyCLI flag that mountshttp-proxy-middlewareat/__proxy/to reverse-proxy the app through mock-bridge--proxyUsage
Or test this branch directly:
Then in Cypress:
Test plan
--proxyflag, confirm app loads via/__proxy/(same-origin iframe)--proxy, confirm no behavior change🤖 Generated with Claude Code