-
Notifications
You must be signed in to change notification settings - Fork 46.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DevTools Bug] Cannot add node "476" because a node with that id is already in the Store. #21636
Comments
Hi @Bodyhealer. I'm no table to reproduce the error following the steps you've described. bug-21636-repro-Kapture.2021-06-07.at.08.56.19.mp4Any suggestions? |
I've got this error today as well on Firefox 89. P.S.: Clicking on |
@rdcavanha You're seeing this bug using OP's codebase? Or a different app? |
Completely different app. |
@rdcavanha Could you share your code then? I'm still looking for a way to repro this issue. |
@bvaughn Sorry, I can't share the code as it is a private repo. However, I've noticed that after I refreshed my Firefox profile the issue seems to be gone. I've tried using the React extension on the same app as before and was unable to reproduce the bug again. This leads me to believe that it's probably some sort of conflict with another extension (just a guess). Unfortunately I can't say which one exactly as I no longer have all the extensions installed due to the profile refresh. The fact that this has very low occurrence seems to support this idea of extension conflict as well. |
Thanks for the extra info. |
[Win10 64-bit] [Firefox 89.0]
|
macOS 11.3.1 Macbook Pro M1, Firefox 89.0.1.
Source is private, sorry |
Unfortunately errors after the extension is first installed or removed might be something that we can't do much about. (Sometimes a full reload of the tab and/or the devtools console is required before an extension will work correctly.) Maybe there's a way for React/DevTools to more gracefully handle this edge case, but it's not a super high priority given how uncommon it is and how easy it is to fix (reload once). |
Reloading the whole browser did work, but tab / console refresh didn't work @bvaughn. Thanks! |
I faced this issue when I had two devtools frontend open, one in the browser extension and a devtools standalone, closing one stopped giving this error. Maybe that's what is causing this. |
The image file is missing |
I'm able to periodically reproduce this bug, so I can take a look next time it comes up! |
Just reproduced in Chrome using standalone react-devtools simultaneously with the browser extension. Even though the steps to reproduce might not be the same as the original reporter, as well as being misuse of the extension, I hope they might giva a clue to the root cause.
Prerequisites:
Steps to reproduce:
Edit: |
@rdcavanha I'm using the Chrome extension and got the same error. It went away when I changed one of my classNames, I'm guessing one of my extensions was using the same className. |
getting the same error with react18 alpha steps to reproduce
open firefox developer edition |
@noahehall Followed the steps above but
I don't know what Looks like the error is coming from |
@bvaughn sorry about that - jsync is internal to the repo (/root/packages/tools/jsync) give me a second to do a fresh clone and determine exactly what needs to be done to run the app @bvaughn here are the exact steps taken on a fresh clone, please ignore the previous directions then open |
@bvaughn hah! Upraded node to 16.9.1 and recieved the same error as you. the the real issue (with jsync) was a preinstall script inside jsync/package.json (ment to automatically sync root pkgjsons to child pkjsons via npm link which creates the jsync bin cmd) This has been fixed and you should be able to follow the steps and reproduce (react-devtools issue) without error sorry but nodeproto, by definition, is filled with prototypes and things (thus relying on tools like react-devtools is paramount) thx for your help on this issue! nh |
@noahehall Am I checking the
|
the branch extract-nodeproto-utica |
there's a pkjson script that copies the flowconfig , you should comment it |
There is no
Okay. I'll try this. |
@bvaughn your patience is emmy-award winning. I was able to resolve this issue internally (thus for other users facing this issue, its likely a configuration problem) so what exactly was the issue? shit if i know... but the develop branch works both:
working on the problem you were facing running my nodeproto framework, I ended up
|
Thank you for the update, @noahehall 👍🏼 I'm glad you were able to get the issue resolved on your end. We'll keep this issue open for others in case more repros pop up in the future 😄 |
I got the same issue, then I relaized I have the same extention dulicated, I used Edge and I installed both React Dev Tool extensions for both chrom and edge. After I disabled / remove the Chrome extention ( because I used Edge browser ). The issue was gone. |
Thank you for the extra information @virajtk |
Same problem here :( |
Merging #22570 into this issue, which seems to be another instance of the same problem. |
@Delice0 I'm sorry to hear you're seeing this. Unfortunately without instructions on how to repro, this isn't particularly useful for tracking down and fixing the issue. |
The same issue with the last RN and node 16.13.1 |
Please take a moment to read other comments above (like #21636 (comment)) before adding an addition one, folks. @KonstantinZhukovskij Without any repro instructions, "same issue" doesn't really give us any way to help. |
With React Native bug reproduction: https://github.com/AllanChain/rn-dev-bug Steps:
"Cannot add node" error will appear, Edit: I figured out that it's the incorrect version of |
Maybe this bug gone. For now I'm using v4.22.0-0229baee2 |
Here is another repro: #23035 (comment) Thanks @junkboy0315! I realized that upgrading react-devtools-core >= 4.22.0 breaks react-native-debugger. Minimal reproductive sample (zip) This sample works fine with debugger at first. (it uses react-devtools-core v4.21.0)
|
Possible fixed by #23156. |
Closing this issue since we're releasing a new version today (with the fix from #23156). Released on Jan 24 as version 4.23.0: Give it a couple of days to propagate to all of the browsers. (Chrome usually takes the longest.) |
Thank you @bvaughn. Based on working example I was able to fix DevTool in Expo 44. I was following this article but it did not work out on my end. |
Website or app
https://zhivika.ru/product/26406103_plastyr_perc_bperf_10smh18sm_n
Repro steps
How often does this bug happen?
Every time
DevTools package (automated)
react-devtools-extensions
DevTools version (automated)
4.13.5-0ae5290b54
Error message (automated)
Cannot add node "476" because a node with that id is already in the Store.
Error call stack (automated)
No response
Error component stack (automated)
No response
GitHub query string (automated)
The text was updated successfully, but these errors were encountered: