refactor: update references after renaming connect-monorepo to metamask-connect#28906
refactor: update references after renaming connect-monorepo to metamask-connect#28906ffmcgee725 wants to merge 3 commits into
Conversation
|
✅ E2E Fixture Validation — Schema is up to date |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3055e21. Configure here.
|
cortisiko
left a comment
There was a problem hiding this comment.
there is a cursor bot high sev comment, can you address and then run the mm connect tests? Feel free to ping when you are ready for a review
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
PR template — items to address before "Ready for review"Warnings — informational, address before merging:
See docs/readme/ready-for-review.md for the full Definition of Ready for Review. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
None of these changes touch any MetaMask Mobile app source code, Detox test specs, page objects, selectors, or shared test infrastructure. No E2E test tags are warranted. Performance Test Selection: |
|




Description
Updates all references from
MetaMask/connect-monorepotoMetaMask/metamask-connectahead of the GitHub repo rename.Motivation
The
connect-monorepoGitHub repo is being renamed tometamask-connectto match the external product name. While GitHub auto-redirects old URLs, redirects break if a new repo is ever created with the old name. This PR proactively updates all references.Changes
scripts/fetch-rn-playground-apk.shREPO="MetaMask/connect-monorepo"variable (used for GitHub API calls to fetch the playground APK), plus header commentstests/performance/mm-connect/utils.js../metamask-connect/...), error message instructions, and commentstests/performance/mm-connect/README.mdtests/performance/README.mdNot changed:
attribution.txt— containswalletconnect/walletconnect-monoreporeferences (different project)Changelog
CHANGELOG entry: null
Related issues
Manual testing steps
./scripts/fetch-rn-playground-apk.sh— verify it resolves the GitHub Release fromMetaMask/metamask-connect(will work once the repo is renamed; until then, the old URL auto-redirects)grep -r 'connect-monorepo' tests/ scripts/returns onlywalletconnect-monorepohits (inattribution.txt)Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Documentation and test-helper path/API repo strings only; no wallet or production runtime behavior changes.
Overview
Renames the React Native Playground dependency from
connect-monorepotometamask-connectacross performance-test docs, APK fetch tooling, and path resolution so setup matches the upcoming GitHub repo rename.fetch-rn-playground-apk.shnow pulls release assets fromMetaMask/metamask-connect.tests/performance/mm-connect/utils.tslooks for a locally built APK under../metamask-connect/...and updates the “APK not found” instructions accordingly.tests/performance/README.mdandtests/performance/mm-connect/README.mdreplace clone paths, build commands, GitHub links, and CI env var wording (e.g.RN_PLAYGROUND_APK_VERSIONpins a metamask-connect release).Reviewed by Cursor Bugbot for commit 01b27f1. Bugbot is set up for automated code reviews on this repo. Configure here.