Skip to content

fix(docker): build browser inference in dev stack - #879

Merged
JArmandoAnaya merged 1 commit into
mainfrom
fix/docker-browser-inference-dev
Sep 17, 2026
Merged

JArmandoAnaya merged 1 commit into
mainfrom
fix/docker-browser-inference-dev

Conversation

@JArmandoAnaya

Copy link
Copy Markdown
Contributor

Summary

  • include @visionset/browser-inference in the Compose app image and build it before Vite starts
  • preserve built library output while watch builds run, preventing Vite from observing an empty ORT asset directory
  • add a regression check for the Compose browser-inference build contract

Validation

  • docker compose -f docker/compose.yaml -f docker/compose.gpu.yaml up --build -d
  • app source and ORT asset served through nginx (HTTP 200)
  • docker compose -f docker/compose.yaml -f docker/compose.gpu.yaml config --quiet
  • node --test tests/scripts/compose_browser_inference.test.mjs

pnpm test:scripts requires a Node 24 checkout with local dependencies; the host checkout has neither.

@JArmandoAnaya
JArmandoAnaya merged commit d69e279 into main Sep 17, 2026
32 checks passed
@JArmandoAnaya
JArmandoAnaya deleted the fix/docker-browser-inference-dev branch September 17, 2026 22:54
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