infinitered / infinitered/reactotron
Can't push to arrays in `reactotron-core-client`
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 15.6k
- Forks
- 972
- PR merge metrics
- No merged PRs in 30d
Description
Error when using the latest beta release and upgrading React Native.

### Steps to reproduce the issue.
1. Checkout to https://github.com/infinitered/ignite/pull/2377
2. Checkout to https://github.com/infinitered/reactotron/pull/1288
3. Update this metro config in `ignite/boilerplate/metro.config.js`
5. Run `npx zx scripts/install-workspace-packages-in-target.mjs ~/Code/ignite/boilerplate/` from reactotron repo
6. Run `yarn start` and `yarn ios` in ignite/boilerplate repo
### Context
Tried getting around this by updating the property

But this caused another issue where non-core plugins weren't getting initialized correctly like in reactotron-mst.
Seems like that mutating class properties causes some funky business with prototypes: https://stackoverflow.com/questions/57037585/why-we-cannot-add-a-new-property-to-an-object-constructor-the-same-way-we-add-a
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the failure using the referenced Ignite and Reactotron pull requests, the change in ignite/boilerplate/metro.config.js, the workspace-package install command, and yarn start/yarn ios. Investigate the array behavior in reactotron-core-client alongside the reported prototype issue. Done means arrays can be pushed after the upgrade and non-core plugins such as reactotron-mst still initialize correctly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- devtools, mobile-dev
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100