Skip to content

fix: pass DevTools config through createReconciler - #632

Merged
thejustinwalsh merged 2 commits into
pixijs:mainfrom
AlaricBaraou:fix-devTools
Dec 1, 2025
Merged

fix: pass DevTools config through createReconciler#632
thejustinwalsh merged 2 commits into
pixijs:mainfrom
AlaricBaraou:fix-devTools

Conversation

@AlaricBaraou

Copy link
Copy Markdown
Contributor

Should fix: #631

We had the same issue in @react-three/fiber
And this PR fix it pmndrs/react-three-fiber#3594

I found out that injectIntoDevTools is not the correct way to pass the DevTools config anymore. It changed in that PR https://github.com/facebook/react/pull/30522/files

I did not test this PR as I'm not familiar with your repo but it should work for you too 👍

@thejustinwalsh

Copy link
Copy Markdown
Collaborator

Thanks @AlaricBaraou! Giving it a try now.

@thejustinwalsh

Copy link
Copy Markdown
Collaborator

I see in the r3f thread you commented abot the types not being updated yet.
I will need to add the same @ts-ignore or @ts-expect-error comments here too.

@thejustinwalsh
thejustinwalsh requested a review from trezy November 29, 2025 07:01
@thejustinwalsh thejustinwalsh self-assigned this Nov 29, 2025
@thejustinwalsh thejustinwalsh changed the title Pass DevTools config through createReconciler fix: pass DevTools config through createReconciler Nov 29, 2025
@pkg-pr-new

pkg-pr-new Bot commented Dec 1, 2025

Copy link
Copy Markdown
npm i https://pkg.pr.new/pixijs/pixi-react/@pixi/react@632

commit: 18c12a4

@thejustinwalsh

Copy link
Copy Markdown
Collaborator

Tested against React 19.0.0 and my full app is now in DevTools including the @pixi/react root. @AlaricBaraou thanks for the PR!

@thejustinwalsh
thejustinwalsh added this pull request to the merge queue Dec 1, 2025
Merged via the queue into pixijs:main with commit 873b900 Dec 1, 2025
8 checks passed
@github-actions

github-actions Bot commented Dec 1, 2025

Copy link
Copy Markdown

🎉 This PR is included in version 8.0.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken React Dev Tools

2 participants