Skip to content

Conversation

@jamadeo
Copy link
Collaborator

@jamadeo jamadeo commented Oct 3, 2025

This works for me but it's a little flaky. It seems to fail showing the dev tools if there's another instance of goose also with the devtools running.

It also works once I open the chrome dev tools and then refresh the app.

@jamadeo jamadeo requested a review from alexhancock October 3, 2025 14:14
@zanesq
Copy link
Collaborator

zanesq commented Oct 3, 2025

Thanks for doing this! I tested locally but didn't see any react devtools show up in inspector. Closed all other goose windows. Maybe I'm missing it where should it show up in devtools?

@jamadeo
Copy link
Collaborator Author

jamadeo commented Oct 3, 2025

Ugh yeah it was flaky for me too. @zanesq did you try refreshing with the chrome dev tools open? I had to do that too

@jamadeo
Copy link
Collaborator Author

jamadeo commented Oct 3, 2025

And do you see any logs from node? There were some mysterious errors from a service worker layer about not being able to delete an extension database when it wasn't working

@zanesq
Copy link
Collaborator

zanesq commented Oct 6, 2025

Just tried again and refresh didn't show the react dev tools profile or components tabs. I do have it installed in chrome and can see the tabs on other websites with react etc.

I do see some errore from node that could be related but not sure if those are on main or not.

[30030:1005/093428.209302:ERROR:extensions/browser/extensions_browser_client.cc:72] Extension Error:
  OTR:     false
  Level:   1
  Source:  manifest.json
  Message: Service worker registration failed. Status code: 2
  ID:      iedlfkhnniicohgmoebfngiecdmggbge
  Type:    ManifestError
[30030:1005/093429.047249:ERROR:components/services/storage/service_worker/service_worker_storage.cc:1732] Failed to delete the database: Database IO error
[30030:1005/093429.211030:ERROR:components/services/storage/service_worker/service_worker_storage.cc:1732] Failed to delete the database: Database IO error
09:38:40.038 › from renderer: [UNHANDLED REJECTION] SecurityError: Failed to construct 'SharedWorker': Access to the script at 'blob:http://localhost:5173/ee6cfff7-c7c5-4b56-a54a-312ada4a01b7' is denied by the document's Content Security Policy.
[30030:1006/084200.408321:ERROR:CONSOLE:2] "Electron sandboxed_renderer.bundle.js script failed to run", source: node:electron/js2c/sandbox_bundle (2)
[30030:1006/084200.408347:ERROR:CONSOLE:2] "TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))", source: node:electron/js2c/sandbox_bundle (2)
[30030:1006/084200.513678:ERROR:CONSOLE:1] "Request Autofill.enable failed. {"code":-32601,"message":"'Autofill.enable' wasn't found"}", source: devtools://devtools/bundled/core/protocol_client/protocol_client.js (1)
[30030:1006/084200.513711:ERROR:CONSOLE:1] "Request Autofill.setAddresses failed. {"code":-32601,"message":"'Autofill.setAddresses' wasn't found"}", source: devtools://devtools/bundled/core/protocol_client/protocol_client.js (1)

@jamadeo
Copy link
Collaborator Author

jamadeo commented Oct 6, 2025

@zanesq thanks for that -- the "Failed to delete the database: Database IO error" is what I saw when it wouldn't work. Then I quit all other goose processes and did not see the error. I'm not sure what is going on there -- maybe something to do with the electron session persistence?

I do however see this even when it does work:

[84510:1006/121858.460288:ERROR:CONSOLE:2] "Electron sandboxed_renderer.bundle.js script failed to run", source: node:electron/js2c/sandbox_bundle (2)
[84510:1006/121858.460364:ERROR:CONSOLE:2] "TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))", source: node:electron/js2c/sandbox_bundle (2)
[84510:1006/121858.643086:ERROR:CONSOLE:1] "Request Autofill.enable failed. {"code":-32601,"message":"'Autofill.enable' wasn't found"}", source: devtools://devtools/bundled/core/protocol_client/protocol_client.js (1)
[84510:1006/121858.643139:ERROR:CONSOLE:1] "Request Autofill.setAddresses failed. {"code":-32601,"message":"'Autofill.setAddresses' wasn't found"}", source: devtools://devtools/bundled/core/protocol_client/protocol_client.js (1)

and this looks related:

electron/electron#41614

but it doesn't seem to break anything.

@jamadeo
Copy link
Collaborator Author

jamadeo commented Oct 6, 2025

Also when you say

Closed all other goose windows

did you quit Goose entirely? I think if the window is closed but the app is still running, it does seem to still conflict

@zanesq
Copy link
Collaborator

zanesq commented Oct 6, 2025

Okay tried again exited all goose and I did see it after refreshing 👍 does seem flaky. Guess it's better than no react dev tools so approving.

@jamadeo jamadeo merged commit f68c4bf into main Oct 10, 2025
11 checks passed
@jamadeo jamadeo deleted the jackamadeo/react-dev-tools branch October 10, 2025 02:31
tlongwell-block added a commit that referenced this pull request Oct 10, 2025
* origin/main:
  Improve Rust analysis output for `analyze` tool (#5072)
  Remove duplicate prepare_reply_context call (#5063)
  install react dev tools in development (#4979)
  Doc: Added powershell installation link to the guide (#5012)
  draft of new blog post about automating more automation (#5038)
  Subagent extension selection behavior fix (#5093)
  Add dev and alpha environment indicator (#5092)
  docs: add content carousel (#5086)
  Applied server side call to parse and save recipe (#5022)
  feat(prompt-library): add Code Documentation Migrator intermediate prompt (#4996) (#5051)
  Add Messy Column Fixer recipe (#5062)
  Cleanup temp files (#5081)
  add openmetadata recipe (#5076)
  Fix Hacktoberfest Leaderboard (#5080)
  adding brand guidelines to AGENTS.md (#4887)
  Fix: Prevent cross-contamination of cache data across analysis modes for `analyze` tool (#5075)
  fix: remove circular reference (#5018)
  Introduced a new prompt for content amplification that integrates multi-step workflows using official Goose extensions. Closes Issue #4998 (#5050)
  Add hint for focus mode when used on file paths for `analyze` tool (#5069)
  fix: use dynamic port allocation for OAuth server (#5019)
zanesq added a commit that referenced this pull request Oct 13, 2025
…sion-streaming

* 'main' of github.com:block/goose: (37 commits)
  Clear deeplinks after use (#5128)
  Revert "Fix gpt-5 input context limit (#4619)" (#5135)
  fix: missing cmake and protobuf for windows build, deduplicate sh/pws… (#5028)
  Fix bedrock tool input schema (#5064)
  Add self-test recipe for goose validation (#5111)
  fix: modifies openai request logic for reasoning models (#4221) (#4294)
  Fix race condition threat when set_param and set_secret of c… (#5109)
  Clean room implementation of the chat process (#5079)
  Bump rmcp (#5096)
  set version in an env variable for testing (#5100)
  fix : enhance fuzzy file search in goose desktop (#5071)
  Make async (#5126)
  docs: unlist tutorials for extensions with archived or moved servers (#5116)
  Add API Documentation Generator prompt (#5001)
  Add flag for enabling eleven labs voice dictation (#5095)
  force re-render fields to pick up custom params usage in instructions (#5112)
  Remove isUserInputDisabled (#5115)
  Improve Rust analysis output for `analyze` tool (#5072)
  Remove duplicate prepare_reply_context call (#5063)
  install react dev tools in development (#4979)
  ...

# Conflicts:
#	ui/desktop/src/components/BaseChat2.tsx
#	ui/desktop/src/hooks/useChatStream.ts
@alexhancock alexhancock mentioned this pull request Oct 17, 2025
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.

3 participants