Skip to content

[10/16] Desktop: editor simulator on the desktop host, editor-only natives removed - #29

Draft
richardelms-stash wants to merge 1 commit into
desktop/02-editmode-testsfrom
desktop/03-editor-rewire
Draft

richardelms-stash wants to merge 1 commit into
desktop/02-editmode-testsfrom
desktop/03-editor-rewire

Conversation

@richardelms-stash

@richardelms-stash richardelms-stash commented Aug 26, 2026 •

Copy link
Copy Markdown

TL;DR

  • The editor simulator now presents checkouts through the real desktop host in a standalone window; the old editor-only native plugins are deleted.
  • Play mode on a Standalone build target uses the exact code path that ships in players.

stash-unity stack PR 3 of 5 (targets desktop/02-editmode-tests, #28). Linear: LAU-299

What

  • Editor/StashEditorPluginWindow.cs: the simulator presents the checkout through the stash-native desktop host in window mode (presentation: "window", width / height from the device presets, new "Desktop card (480x720)" preset) and lets the play-mode StashNative instance drain the native events, so per-call callbacks and global events fire exactly as on a device. The reflection contract with StashNative.OpenEditorTestWindow now carries the card / modal config (it was dropped before). Simulate Callbacks buttons and the dismiss-on-close path stay.
  • Runtime/StashNative.cs: in play mode with a Standalone build target the desktop host opens directly in window mode with the desktop sizing (what ships is what is tested); mobile targets keep the phone-preset simulator; the desktop event drain also runs in the editor.
  • Editor/Windows/ and Editor/macOS/ deleted (WebView2 launcher DLL and the arm64-only WKWebView launcher bundle with its EditorApplication.delayCall-off-thread callback path). The public Stash.Editor.Windows.WebViewLauncher / Stash.Editor.macOS.WebViewLauncher editor types go away; the CHANGELOG entry in PR 5 calls this out.

Verified

  • Unity 6000.3.21f1: editor assembly compiles, EditMode suite 15 / 15.
  • dotnet format whitespace --verify-no-changes clean.
  • Manual (for the reviewer): play mode with a macOS or Windows Standalone target opens the host's window presentation; with a mobile target the simulator panel opens it at the preset size; closing either delivers OnDialogDismissed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FuG7FjNCGRKxPchjZ8AuUV


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

…-only natives

The Window/Stash test window now presents the checkout through the
stash-native desktop host in window mode at the selected device preset
and routes its events into StashNative, so per-call callbacks and global
events fire in the editor as on a device; the card config is no longer
dropped on the way in. The Simulate buttons stay. Play mode with a
Standalone build target opens the desktop host directly with the desktop
sizing; mobile targets keep the phone-preset simulator.

Editor/Windows (WebView2 launcher DLL) and Editor/macOS (WKWebView
launcher bundle, arm64 only) are removed: one webview codebase per OS,
a universal mac binary and sound threading. The Stash.Editor.*.
WebViewLauncher types are gone (called out in the CHANGELOG).
@richardelms-stash
richardelms-stash force-pushed the desktop/03-editor-rewire branch from 8ab622d to d1f82b1 Compare August 26, 2026 11:16
@richardelms-stash richardelms-stash changed the title Desktop: editor simulator on the desktop host, editor-only natives removed [10/16] Desktop: editor simulator on the desktop host, editor-only natives removed Aug 26, 2026
@richardelms-stash

Copy link
Copy Markdown
Author

/ripley review

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