Skip to content

Conversation

@pulpdrew
Copy link
Contributor

Closes HDX-3117

Summary

This PR fixes a crash when loading or navigating away from the dashboard page. It occurred when a non-auto granularity was specified, and was caused by an infinite re-render loop due to continuously watching and resetting the granularity query parameter.

Before

Screen.Recording.2025-12-29.at.9.56.09.AM.mov

After

Screen.Recording.2025-12-29.at.9.55.32.AM.mov

@changeset-bot
Copy link

changeset-bot bot commented Dec 29, 2025

🦋 Changeset detected

Latest commit: e97b5f6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/app Patch
@hyperdx/api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 29, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
hyperdx-v2-oss-app Ready Ready Preview, Comment Dec 29, 2025 2:56pm

@claude
Copy link

claude bot commented Dec 29, 2025

✅ No critical issues found.

The fix correctly resolves the infinite re-render loop by:

  • Removing the values prop that was causing the form to re-sync from URL state on every render
  • Keeping defaultValues for initial state from URL params
  • Maintaining the one-way sync from form → URL via the existing useEffect

The change is minimal, focused, and includes a proper changeset.

@github-actions
Copy link
Contributor

E2E Test Results

All tests passed • 51 passed • 4 skipped • 693s

Status Count
✅ Passed 51
❌ Failed 0
⚠️ Flaky 0
⏭️ Skipped 4

Tests ran across 4 shards in parallel.

View full report →

@teeohhem teeohhem self-requested a review December 29, 2025 15:05
@kodiakhq kodiakhq bot merged commit 5062d80 into main Dec 29, 2025
14 checks passed
@kodiakhq kodiakhq bot deleted the drew/fix-dashboard-infinite-rerender branch December 29, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants