Skip to content

Fix worker web UI navigation route fallback#201

Merged
DeadWaveWave merged 1 commit intoDeadWaveWave:mainfrom
Xinyuexyyyyy:fix-web-ui-spa-fallback
Apr 29, 2026
Merged

Fix worker web UI navigation route fallback#201
DeadWaveWave merged 1 commit intoDeadWaveWave:mainfrom
Xinyuexyyyyy:fix-web-ui-spa-fallback

Conversation

@Xinyuexyyyyy
Copy link
Copy Markdown
Contributor

Summary

  • serve the Web UI entrypoint for extensionless browser navigation paths such as /canvas
  • keep API/auth/debug/static asset paths from falling back to the Web UI
  • apply the same navigation-path check to the Web UI login gate

Tests

  • npm exec -- vitest run tests/unit/app/webUiAssets.spec.ts tests/contract/controlSurface/controlSurfaceHttpServer.webShell.spec.ts --reporter=dot
  • npm exec -- tsc -b tsconfig.node.json tsconfig.web.json --noEmit
  • npm exec -- prettier --check src/app/main/controlSurface/webUiAssets.ts src/app/main/controlSurface/http/webUiEntryGate.ts tests/unit/app/webUiAssets.spec.ts tests/contract/controlSurface/controlSurfaceHttpServer.webShell.spec.ts
  • git diff --check

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 29, 2026

CLA assistant check
All committers have signed the CLA.

@Xinyuexyyyyy Xinyuexyyyyy force-pushed the fix-web-ui-spa-fallback branch from fe4fe70 to babc18a Compare April 29, 2026 14:19
@DeadWaveWave DeadWaveWave merged commit 8e09bc7 into DeadWaveWave:main Apr 29, 2026
6 of 7 checks passed
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.

3 participants