Skip to content

refactor(web): organize devtools components#8

Open
tomerqodo wants to merge 6 commits intoaugment_full_base_refactorweb_organize_devtools_components_pr8from
augment_full_head_refactorweb_organize_devtools_components_pr8
Open

refactor(web): organize devtools components#8
tomerqodo wants to merge 6 commits intoaugment_full_base_refactorweb_organize_devtools_components_pr8from
augment_full_head_refactorweb_organize_devtools_components_pr8

Conversation

@tomerqodo
Copy link
Copy Markdown

Benchmark PR from agentic-review-benchmarks#8

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Jan 26, 2026

🤖 Augment PR Summary

Summary: Refactors and reorganizes the web devtools integration, separating React Scan and TanStack devtools into dedicated subfolders.

Changes:

  • Added lazy-loaded ReactScanLoader and TanStackDevtoolsLoader wrappers guarded by IS_DEV
  • Moved the React Scan component into components/devtools/react-scan
  • Moved the TanStack devtools wrapper into components/devtools/tanstack
  • Updated web/app/layout.tsx and the query client initializer to consume the new loaders

Technical Notes: Devtools are now loaded via React.lazy + Suspense to avoid impacting non-dev builds and to keep dev-only code paths isolated.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

const TanStackDevtoolsWrapper = lazy(() =>
import('./devtools').then(module => ({
default: module.TanStackDevtoolsWrapper,
})).catch(() => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

import('./devtools') errors are currently swallowed without any logging, which can make devtools silently disappear in development if the chunk fails to load. Consider logging the caught error (similar to ReactScanLoader) so failures are diagnosable.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

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.

2 participants