Skip to content

fix(desktop): move enterprise creator skill seeding off startup path#776

Merged
benjaminshafii merged 1 commit intodifferent-ai:devfrom
IanShaw027:fix/async-enterprise-seed
Mar 7, 2026
Merged

fix(desktop): move enterprise creator skill seeding off startup path#776
benjaminshafii merged 1 commit intodifferent-ai:devfrom
IanShaw027:fix/async-enterprise-seed

Conversation

@IanShaw027
Copy link
Contributor

Summary

Fix a startup hang caused by synchronous enterprise creator skill seeding.

On starter workspace bootstrap, OpenWork was downloading the
openwork-enterprise zip on the startup critical path. When GitHub connectivity
was slow or stalled, the desktop app could stay stuck in [workspace] bootstrap
and show a long white screen before any of the engine services started.

I verified this by comparing startup behavior with and without the enterprise
seed marker:

  • without the marker, startup stayed in bootstrap while the main process held
    GitHub HTTPS connections
  • with the marker present, startup quickly moved on and started the downstream
    services

Changes

  • move enterprise creator skill seeding into a background thread
  • keep starter workspace local file seeding unchanged
  • add in-process deduping so the same workspace does not spawn duplicate seed jobs
  • add network timeouts for the enterprise zip download
  • keep the existing success marker behavior

@vercel
Copy link

vercel bot commented Mar 7, 2026

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

Project Deployment Actions Updated (UTC)
openwork-software Ready Ready Preview, Comment Mar 7, 2026 3:30am

@vercel
Copy link

vercel bot commented Mar 7, 2026

@IanShaw027 is attempting to deploy a commit to the 0 Finance Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

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

@benjaminshafii benjaminshafii merged commit fa3bc8b into different-ai:dev Mar 7, 2026
11 of 13 checks passed
caoergou added a commit to caoergou/openwork that referenced this pull request Mar 7, 2026
PR different-ai#776 introduced a JavaScript module (local-file-path.impl.js) without
a corresponding type declaration, causing TS7016 error when running
typecheck with strict settings.

This adds the missing .d.ts file to provide type information for the
normalizeLocalFilePath function.
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.

2 participants