Skip to content

Fix desktop API host initialization ordering - #1886

Open
matthewpenkala wants to merge 1 commit into
storytold:mainfrom
matthewpenkala:contrib/await-desktop-api-host-bootstrap
Open

Fix desktop API host initialization ordering#1886
matthewpenkala wants to merge 1 commit into
storytold:mainfrom
matthewpenkala:contrib/await-desktop-api-host-bootstrap

Conversation

@matthewpenkala

Copy link
Copy Markdown

Summary

  • Install the native-configured Storyteller API host before mounting desktop REST consumers.
  • Share concurrent host synchronization, cache only accepted hosts, and keep failures retryable.
  • Render a fail-closed startup error when the native host is unavailable.
  • Run a host-change session refresh after installation without making that refresh block the app shell.

Why

Desktop startup previously launched host synchronization as fire-and-forget work from the rendered component tree. REST consumers could therefore mount against a default or stale host before Tauri configuration arrived.

The bootstrap now establishes the required host first while leaving web startup unchanged.

Validation

  • Focused host-sync and bootstrap suites: 16/16 passed.
  • Concurrent callers, retry behavior, cache invalidation, missing-host failure, and noncritical session-refresh paths are covered.
  • git diff --check and contribution-scope scans pass.

Origin and scope

This change was isolated from a broader downstream integration and reduced to a generic, self-contained upstream contribution. Private deployment/release-specific material and artifacts are intentionally excluded.

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

👷 Deploy request for artcraft-webapp pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1008d8b

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

Deploy Preview for artcraft-dashboard ready!

Name Link
🔨 Latest commit 1008d8b
🔍 Latest deploy log https://app.netlify.com/projects/artcraft-dashboard/deploys/6a7b93958e29c900081646a0
😎 Deploy Preview https://deploy-preview-1886--artcraft-dashboard.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

Deploy Preview for storyteller-docs ready!

Name Link
🔨 Latest commit 1008d8b
🔍 Latest deploy log https://app.netlify.com/projects/storyteller-docs/deploys/6a7b9395e742390008c4bf58
😎 Deploy Preview https://deploy-preview-1886--storyteller-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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