Skip to content

fix: resolve hydration mismatch on sample load and format block names#26

Merged
madawei2699 merged 2 commits into
mainfrom
fix/hydration-followup
Jun 13, 2026
Merged

fix: resolve hydration mismatch on sample load and format block names#26
madawei2699 merged 2 commits into
mainfrom
fix/hydration-followup

Conversation

@madawei2699

Copy link
Copy Markdown
Contributor

Resolves #25. Removes direct window.location access, moves query param loading to useEffect, and replaces underscores in block ID names.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the playground page by removing synchronous sample loading from initial state initializers and deferring it to a useEffect hook. It also updates the block composition display to replace underscores with spaces. The review feedback highlights a potential memory leak and race condition in the asynchronous sample loading hook due to an uncleared setTimeout, and suggests simplifying the block name formatting logic using a regular expression.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/app/playground/page.tsx Outdated
Comment thread src/app/playground/page.tsx Outdated
…eout cleanup and regex block name formatting
@madawei2699 madawei2699 merged commit 39192bd into main Jun 13, 2026
1 check passed
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.

Fix example-gallery sample loading hydration and review follow-ups

1 participant