Skip to content

fix: copy repo-root fixtures into web-console test image #1151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sakibsadmanshajib merged 2 commits into from
Aug 25, 2026
Merged

fix: copy repo-root fixtures into web-console test image #1151

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d50e7bd
fix: copy repo-root fixtures into web-console test image
sakibsadmanshajib Aug 25, 2026
b2b7826
fix: cover new web-console test fixtures in deploy-demo-box paths filter
sakibsadmanshajib Aug 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions .github/workflows/deploy-demo-box.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,18 @@ on:
# here is exactly the kind of change that must not wait for an
# unrelated commit to ship it.
- '.dockerignore'
# deploy/docker/Dockerfile.web-console COPYs these three repo-root
# paths so tests/unit/control-plane-host.test.ts and
# tests/unit/chat-coverage-lib.test.ts have their fixtures inside the
# image (PR #1151): .env.example, all of deploy/ (the hostname-drift
# test scans the whole tree recursively, so this is deliberately not
# narrowed to deploy/docker/** or deploy/litellm/** above), and one
# frozen coverage-run artifact under docs/proof/. Same blind spot as
# every other entry in this list: without one, a change to any of
# these would merge and never trigger a deploy.
- '.env.example'
- 'deploy/**'
- 'docs/proof/chat-interaction-coverage-2026-08-10/coverage.run.json'
# A schema-only merge used to match nothing here, so it deployed nothing
# and applied nothing. That is literally how PR #624's
# 20260731_01_tenant_billing_account_backfill_v2.sql and PR #602's
Expand Down
12 changes: 12 additions & 0 deletions deploy/docker/Dockerfile.web-console
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,18 @@ RUN set -eux; \

COPY apps/web-console/ ./

# Repo-root fixtures a handful of unit tests read directly (control-plane-host,
# chat-coverage-lib, model-catalog-table): the build context is the repo root
# but only apps/web-console/ is copied above, so anything a test reads outside
# that tree is otherwise an ENOENT inside the image, or (model-catalog-table)
# an `existsSync` false that lets the test go quiet via `skipIf` instead.
# Copy exactly the paths those tests use, not the whole repo, to keep the
# image small.
COPY .env.example /app/.env.example
COPY deploy/ /app/deploy/
COPY docs/proof/chat-interaction-coverage-2026-08-10/coverage.run.json /app/docs/proof/chat-interaction-coverage-2026-08-10/coverage.run.json
COPY supabase/migrations/ /app/supabase/migrations/

EXPOSE 3000

CMD ["npm", "run", "dev", "--", "--hostname", "0.0.0.0", "--port", "3000"]
Loading