Skip to content

Conversation

Copy link

Copilot AI commented Nov 17, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

This PR makes small, low-risk fixes to constants/extraRpcs.js:

  1. Fix duplicate object property 'trackingDetails' in the moonbeam (chainId 128) entry by combining the two privacy statements into a single trackingDetails string. The previous duplicate key could lead to confusion and may be flagged by linters.

  2. Fix a typo in the holesky wss URL under chainId 17000: change 'wss://etherem-holesky-rpc.publicnode.com' to 'wss://ethereum-holesky-rpc.publicnode.com'.

  3. Add missing https:// scheme to an entry that used 'public-trx-mainnet.fastnode.io' (under chainId 728126428) by changing it to 'https://public-trx-mainnet.fastnode.io'.

These changes are limited to formatting/typo corrections to avoid changing RPC lists or semantics broadly. No RPCs are removed. The PR includes the updated constants/extraRpcs.js file.

Please review and let me know if you'd like additional cleanup (e.g., merge duplicated chainId entries or remove placeholder strings like 'rpcWorking:false').

This pull request was created as a result of the following prompt from Copilot chat.

This PR makes small, low-risk fixes to constants/extraRpcs.js:

  1. Fix duplicate object property 'trackingDetails' in the moonbeam (chainId 128) entry by combining the two privacy statements into a single trackingDetails string. The previous duplicate key could lead to confusion and may be flagged by linters.

  2. Fix a typo in the holesky wss URL under chainId 17000: change 'wss://etherem-holesky-rpc.publicnode.com' to 'wss://ethereum-holesky-rpc.publicnode.com'.

  3. Add missing https:// scheme to an entry that used 'public-trx-mainnet.fastnode.io' (under chainId 728126428) by changing it to 'https://public-trx-mainnet.fastnode.io'.

These changes are limited to formatting/typo corrections to avoid changing RPC lists or semantics broadly. No RPCs are removed. The PR includes the updated constants/extraRpcs.js file.

Please review and let me know if you'd like additional cleanup (e.g., merge duplicated chainId entries or remove placeholder strings like 'rpcWorking:false').


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Nov 17, 2025
Copilot stopped work on behalf of leeli01284-collab due to an error November 17, 2025 19:48
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.

1 participant