• Will Chen's avatar
    Fix preview navigation forward/back buttons and add E2E test (#2372) · 82e9ddb7
    Will Chen 提交于
    ## Summary
    - Add E2E test for preview navigation forward/back buttons
    - Fix back/forward navigation by sending target URL instead of using
    `history.back()`/`history.forward()` which don't work reliably in
    Electron iframes
    - Update dyad-shim.js to handle URL-based navigation with
    `location.replace`
    - Add popstate event handler to notify parent of navigation changes
    
    ## Test plan
    - Run `npm run e2e -- --grep "forward and back"` to verify the new test
    passes
    - Run `npm run e2e -- --grep "refresh"` to verify all refresh-related
    tests pass
    - Manually test back/forward navigation buttons in the preview panel
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    <!-- devin-review-badge-begin -->
    
    ---
    
    <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2372">
      <picture>
    <source media="(prefers-color-scheme: dark)"
    srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
    <img
    src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
    alt="Open with Devin">
      </picture>
    </a>
    <!-- devin-review-badge-end -->
    
    <!-- CURSOR_SUMMARY -->
    ---
    
    > [!NOTE]
    > Ensures preview back/forward navigation works reliably in Electron
    iframes and preserves routes across refresh/HMR. Adds targeted E2E
    coverage.
    > 
    > - In `PreviewIframe.tsx`, switch back/forward handling to send the
    target URL via `postMessage` and update local `navigationHistory`,
    `currentHistoryPosition`, and `preservedUrls`; keep
    `currentIframeUrlRef` in sync.
    > - In `worker/dyad-shim.js`, add `popstate` handler to notify parent
    (`replaceState` message) and handle `navigate` messages by validating
    http/https URLs and using `location.replace`, with fallback to
    `history.go(±1)`.
    > - Add E2E test `refresh.spec.ts` for preview forward/back buttons,
    verifying initial disabled state, navigation to About, back to Home,
    then forward to About again.
    > 
    > <sup>Written by [Cursor
    Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
    7f9f790158262b9e3100ead96bf25f8e3ee8cac7. This will update automatically
    on new commits. Configure
    [here](https://cursor.com/dashboard?tab=bugbot).</sup>
    <!-- /CURSOR_SUMMARY -->
    
    <!-- This is an auto-generated description by cubic. -->
    ---
    ## Summary by cubic
    Fixes preview forward/back navigation and preserves the current route on
    refresh and HMR so the preview behaves like a real browser. Adds E2E
    tests for both flows.
    
    - **Bug Fixes**
    - Make back/forward work in Electron by posting the target URL to the
    iframe and navigating with location.replace; notify parent on popstate;
    validate http/https URLs to block unsafe protocols.
    - Track and restore the current route across refresh and HMR via
    previewCurrentUrlAtom and a URL ref; clear preserved URL on root path;
    reset on app switch and validate same-origin URLs.
    
    - **E2E Tests**
      - Add a multi-page react-router fixture (Home/About).
    - Add tests for route preservation on refresh and for forward/back
    buttons, including enabled/disabled states.
    
    <sup>Written for commit 7f9f790158262b9e3100ead96bf25f8e3ee8cac7.
    Summary will update on new commits.</sup>
    
    <!-- End of auto-generated description by cubic. -->
    
    ---------
    Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
    82e9ddb7
名称
最后提交
最后更新
..
fixtures 正在载入提交数据...
helpers 正在载入提交数据...
snapshots 正在载入提交数据...
1.spec.ts 正在载入提交数据...
add_prompt_deep_link.spec.ts 正在载入提交数据...
annotator.spec.ts 正在载入提交数据...
app_search.spec.ts 正在载入提交数据...
app_storage_path.spec.ts 正在载入提交数据...
approve.spec.ts 正在载入提交数据...
astro.spec.ts 正在载入提交数据...
attach_image.spec.ts 正在载入提交数据...
auto_approve.spec.ts 正在载入提交数据...
auto_update.spec.ts 正在载入提交数据...
azure_provider_settings.spec.ts 正在载入提交数据...
azure_send_message.spec.ts 正在载入提交数据...
backup.spec.ts 正在载入提交数据...
capacitor.spec.ts 正在载入提交数据...
chat_input.spec.ts 正在载入提交数据...
chat_mode.spec.ts 正在载入提交数据...
chat_search.spec.ts 正在载入提交数据...
concurrent_chat.spec.ts 正在载入提交数据...
context_limit_banner.spec.ts 正在载入提交数据...
context_manage.spec.ts 正在载入提交数据...
context_window.spec.ts 正在载入提交数据...
copy_app.spec.ts 正在载入提交数据...
copy_chat.spec.ts 正在载入提交数据...
debugging_logs.spec.ts 正在载入提交数据...
default_chat_mode.spec.ts 正在载入提交数据...
delete_app.spec.ts 正在载入提交数据...
delete_provider.spec.ts 正在载入提交数据...
dump_messages.spec.ts 正在载入提交数据...
dyad_tags_parsing.spec.ts 正在载入提交数据...
edit_code.spec.ts 正在载入提交数据...
edit_custom_models.spec.ts 正在载入提交数据...
edit_provider.spec.ts 正在载入提交数据...
engine.spec.ts 正在载入提交数据...
env_var.spec.ts 正在载入提交数据...
favorite_app.spec.ts 正在载入提交数据...
file_tree_search.spec.ts 正在载入提交数据...
fix_error.spec.ts 正在载入提交数据...
git_collaboration.spec.ts 正在载入提交数据...
github-import.spec.ts 正在载入提交数据...
github.spec.ts 正在载入提交数据...
import.spec.ts 正在载入提交数据...
import_in_place.spec.ts 正在载入提交数据...
lm_studio.spec.ts 正在载入提交数据...
local_agent_advanced.spec.ts 正在载入提交数据...
local_agent_ask.spec.ts 正在载入提交数据...
local_agent_auto.spec.ts 正在载入提交数据...
local_agent_basic.spec.ts 正在载入提交数据...
local_agent_code_search.spec.ts 正在载入提交数据...
local_agent_consent.spec.ts 正在载入提交数据...
local_agent_file_upload.spec.ts 正在载入提交数据...
local_agent_grep.spec.ts 正在载入提交数据...
local_agent_list_files.spec.ts 正在载入提交数据...
local_agent_read_logs.spec.ts 正在载入提交数据...
local_agent_run_type_checks.spec.ts 正在载入提交数据...
local_agent_search_replace.spec.ts 正在载入提交数据...
local_agent_summarize.spec.ts 正在载入提交数据...
logs_server.spec.ts 正在载入提交数据...
main.spec.ts 正在载入提交数据...
mcp.spec.ts 正在载入提交数据...
mention_app.spec.ts 正在载入提交数据...
mention_files.spec.ts 正在载入提交数据...
new_chat.spec.ts 正在载入提交数据...
nodejs_path_configuration.spec.ts 正在载入提交数据...
ollama.spec.ts 正在载入提交数据...
partial_response.spec.ts 正在载入提交数据...
performance_monitor.spec.ts 正在载入提交数据...
preview_iframe.spec.ts 正在载入提交数据...
problems.spec.ts 正在载入提交数据...
prompt_library.spec.ts 正在载入提交数据...
rebuild.spec.ts 正在载入提交数据...
refresh.spec.ts 正在载入提交数据...
reject.spec.ts 正在载入提交数据...
release_channel.spec.ts 正在载入提交数据...
rename_app.spec.ts 正在载入提交数据...
rename_edit.spec.ts 正在载入提交数据...
restart.spec.ts 正在载入提交数据...
retry.spec.ts 正在载入提交数据...
security_review.spec.ts 正在载入提交数据...
select_component.spec.ts 正在载入提交数据...
setup.spec.ts 正在载入提交数据...
smart_context_balanced.spec.ts 正在载入提交数据...
smart_context_deep.spec.ts 正在载入提交数据...
smart_context_options.spec.ts 正在载入提交数据...
supabase_branch.spec.ts 正在载入提交数据...
supabase_client.spec.ts 正在载入提交数据...
supabase_migrations.spec.ts 正在载入提交数据...
supabase_stale_ui.spec.ts 正在载入提交数据...
switch_apps.spec.ts 正在载入提交数据...
switch_versions.spec.ts 正在载入提交数据...
telemetry.spec.ts 正在载入提交数据...
template-community.spec.ts 正在载入提交数据...
template-create-nextjs.spec.ts 正在载入提交数据...
theme_selection.spec.ts 正在载入提交数据...
themes_management.spec.ts 正在载入提交数据...
thinking_budget.spec.ts 正在载入提交数据...
toggle_screen_sizes.spec.ts 正在载入提交数据...
turbo_edits_options.spec.ts 正在载入提交数据...
turbo_edits_v2.spec.ts 正在载入提交数据...
uncommitted_files_banner.spec.ts 正在载入提交数据...
undo.spec.ts 正在载入提交数据...
version_integrity.spec.ts 正在载入提交数据...
visual_editing.spec.ts 正在载入提交数据...