Skip to content

fix(desktop,app): preserve shared skill modal on warm deeplinks#1033

Closed
jcllobet wants to merge 7 commits intodevfrom
issue-1022-deeplink-logging-h1-h4-h5
Closed

fix(desktop,app): preserve shared skill modal on warm deeplinks#1033
jcllobet wants to merge 7 commits intodevfrom
issue-1022-deeplink-logging-h1-h4-h5

Conversation

@jcllobet
Copy link
Collaborator

Summary

  • combine the h1 desktop single-instance handoff fix, the h4 warm-deeplink dedupe fix, and the h5 modal-state cleanup for shared bundle imports
  • forward second launches into the already-running desktop app and stop dropping follow-up warm deeplinks before dispatch
  • close competing create-worker and connect-remote modal state before opening the shared-skill flow so the shared skill modal can surface immediately
  • keep [issue-1022][h1], [issue-1022][h4], and [issue-1022][h5] logs to inspect the full handoff, dispatch, and modal-visibility chain

Testing

  • pnpm --filter @different-ai/openwork exec cargo check --manifest-path src-tauri/Cargo.toml
  • pnpm typecheck

Notes

  • This branch is the fullest combination of the current hypotheses. I have not yet validated the desktop already-open modal behavior end-to-end in this exact combined branch from this environment.

@vercel
Copy link

vercel bot commented Mar 19, 2026

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

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment Mar 19, 2026 4:24am
openwork-den Ready Ready Preview, Comment Mar 19, 2026 4:24am
openwork-den-worker-proxy Ready Ready Preview, Comment Mar 19, 2026 4:24am
openwork-landing Ready Ready Preview, Comment, Open in v0 Mar 19, 2026 4:24am
openwork-share Ready Ready Preview, Comment Mar 19, 2026 4:24am

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

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