Skip to content

fix: reset navigation stack when showing execution prompt#2391

Open
broody wants to merge 1 commit intomainfrom
fix/execution-prompt-navigation-state
Open

fix: reset navigation stack when showing execution prompt#2391
broody wants to merge 1 commit intomainfrom
fix/execution-prompt-navigation-state

Conversation

@broody
Copy link
Contributor

@broody broody commented Feb 7, 2026

Summary

Fixes state issue where the execution prompt incorrectly showed a back button on the second display instead of the close button.

Changes

  • Added reset: true to execute route navigation to clear navigation history
  • Updated type signatures to include the reset option
  • Updated all relevant tests to expect the new behavior

Testing

All 16 tests passing ✅

Fixes state issue where execution prompt incorrectly showed back button
on second display instead of close button. Now uses reset: true to clear
navigation history when opening execute route, ensuring consistent UX.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
controller-example-next Ready Ready Preview Feb 7, 2026 1:03am
keychain Ready Ready Preview Feb 7, 2026 1:03am
keychain-storybook Ready Ready Preview Feb 7, 2026 1:03am

Request Review

@codecov
Copy link

codecov bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 19.81%. Comparing base (0e418ec) to head (87de3c2).
⚠️ Report is 77 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2391      +/-   ##
==========================================
- Coverage   20.89%   19.81%   -1.08%     
==========================================
  Files         307      325      +18     
  Lines       31079    33140    +2061     
  Branches      880      883       +3     
==========================================
+ Hits         6493     6567      +74     
- Misses      24574    26561    +1987     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant