Skip to content

Dashboard preview blank state#8952

Draft
ericokuma wants to merge 2 commits intomainfrom
cursor/APP-758-dashboard-preview-blank-state-8521
Draft

Dashboard preview blank state#8952
ericokuma wants to merge 2 commits intomainfrom
cursor/APP-758-dashboard-preview-blank-state-8521

Conversation

@ericokuma
Copy link
Contributor

Fixes APP-758

Prevents the dashboard preview from going blank during reconciliation when project files are edited externally.

Why this change?
Previously, when validSpec became null during reconciliation, the frontend immediately reflected this, causing the dashboard preview to disappear. This change ensures a smoother user experience by retaining the last valid dashboard state until reconciliation completes.

What was changed?
Implemented a cached selector (createCachedExploreValidSpecSelector) that holds onto the last complete valid explore and metricsView spec. This cached spec is returned when the current response has a missing validSpec and resources are actively reconciling. Once reconciliation is idle, the cache is no longer used, allowing actual invalid states to surface. This selector was integrated into useExploreValidSpec and getExploreValidSpecQueryOptions.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

Linear Issue: APP-758

Open in Web Open in Cursor 

Co-authored-by: ericokuma <ericokuma@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Mar 2, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

Co-authored-by: ericokuma <ericokuma@users.noreply.github.com>
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