feat: add admin-web Docker build & push workflow with semantic-release #11

Merged

millianlmx merged 8 commits from feat/admin-web-docker into main 2026-07-19 16:10:08 +02:00

Conversation 3 Commits 8 Files Changed 20 +8757 -85

millianlmx commented 2026-07-12 22:41:56 +02:00

Owner

Copy Link

No description provided.

millianlmx added 5 commits 2026-07-13 15:52:11 +02:00

feat: add admin-web Docker build & push workflow with semantic-release 0166398ae1

ci: trigger workflow on PR with build-only validation 56739edf65

fix(ci): remove sudo from docker install — Gitea ubuntu runner is root without sudo 46d95661ce

test(admin-web): fix 14 broken vitest tests and exclude e2e from run ... 094bd59470

- vitest.config.ts: add explicit exclude (e2e/**, playwright-report/**,
  test-results/**) spread with configDefaults.exclude to keep
  node_modules/dist filtering intact
- app/page.test.tsx: waitFor for async stats resolution, replace
  getAllByRole('article') with 'a [data-slot=card]' scoped queries
  (Getting Started card has no icon so we scope to the 3 stat cards inside
  <Link>), drop console.error assertion (production fetchStats destructures
  { count } from the resolved payload without throwing on
  { count: null, error }, so console.error is never called on that path)
- components/workout-form.test.tsx: mock sonner (toast.error/success with
  all methods stubbed) to replace obsolete window.alert assertions, wait for
  the custom Select trainer dropdown to mount before clicking, fill the
  exercise-name-0 input (default exercises state is [{name:""}] which fails
  validation), make insert/update mocks resolve on a macrotask so React can
  render the transient 'Saving...' loading state
- app/login/page.test.tsx: replace getByRole('heading') with getByText
  (shadcn CardTitle renders a <div>, not a semantic heading), extend sonner
  mock with toast.error, fix invalid-credentials mock to return a real
  Error instance (production supabase-js returns AuthError extends Error,
  so the err instanceof Error branch surfaces the real message)
- components/ui/dialog.test.tsx: skip jsdom-incompatible click-outside test
  (Radix Dialog closes via pointerDown on overlay, not reproducible in jsdom;
  covered by the Escape test + Playwright E2E), replace aria-modal assertion
  with role=dialog + aria-labelledby (modern Radix no longer emits aria-modal)

Scope: tests + config only. No production code change.

ci: gate admin-web docker build on unit tests and remove admin-web job from ci.yml ... 8236eb35ae

- admin-web-docker.yml: add 'test' job (Node 22, npm ci, tsc --noEmit,
  vitest run) gated by the anti-[skip ci] guard, wire needs: [test] on
  build-validation and semantic-release, and needs: [semantic-release, test]
  on build-and-push. Tests now run as a gate before Docker build validation
  on PRs and before semantic-release on main.
- ci.yml: drop the redundant admin-web-test job (now covered by the test
  job in admin-web-docker.yml on PRs), remove the admin-web path filter
  and output from the changes job, simplify deploy-functions.needs.
- scripts/verify-admin-web-docker.sh: update check_workflow_jobs to expect
  4 jobs (test + build-validation + semantic-release + build-and-push) and
  check_workflow_needs to expect build-and-push needs [semantic-release, test]
  so the verify script stays green with the new job topology.

millianlmx force-pushed feat/admin-web-docker from d6d5f40359 to 8236eb35ae 2026-07-13 15:52:11 +02:00 Compare

millianlmx closed this pull request 2026-07-13 15:57:22 +02:00

millianlmx reopened this pull request 2026-07-13 15:57:24 +02:00

millianlmx added 1 commit 2026-07-13 16:00:04 +02:00

chore: re-trigger CI after rebase 30693928b5

millianlmx added 1 commit 2026-07-13 16:13:33 +02:00

fix(test): resolve unhandled rejection in sidebar logout error test ... 20f01899df

The 'should handle logout errors gracefully' test mocked supabase.auth.signOut
to reject (throw), which does not match real supabase-js behaviour (signOut
resolves with { error }, never throws). The rejecting mock caused an unhandled
promise rejection in the async handleLogout onClick handler: tolerated locally
(vitest exit 0) but failing in CI (npx vitest run exits 1).

Switch to a realistic resolved-with-error mock and assert the user is still
redirected to /login (true graceful handling). No production code change.

millianlmx commented 2026-07-13 16:18:27 +02:00

Author

Owner

Copy Link

📱 Prêt à tester !

L'app est déployée sur l'iPhone (USB/WiFi).

• Teste les changements
• Reply LGTM pour merger
• Reply KO pour bloquer

## 📱 Prêt à tester ! L'app est déployée sur l'iPhone (USB/WiFi). - Teste les changements - Reply **LGTM** pour merger - Reply **KO** pour bloquer

millianlmx added 1 commit 2026-07-19 15:56:21 +02:00

fix(youtube-worker): sync package-lock.json with @google/genai dep and harden Dockerfile ... a4c6a123a0

Commit edcd857 added @google/genai to package.json without running
npm install, leaving package-lock.json stale. CI's strict npm ci
correctly refused; the Dockerfile's lenient npm install --production
silently masked the desync in prod.

- Regenerate package-lock.json (@google/genai@1.52.0 + 37 transitive deps)
- Dockerfile: npm install --production -> npm ci --omit=dev (strict)
- Dockerfile: drop * glob on package-lock.json* so missing lock fails loud

millianlmx commented 2026-07-19 16:06:55 +02:00

Author

Owner

Copy Link

📱 Prêt à tester !

L'app est déployée sur l'iPhone (USB/WiFi).

• Teste les changements
• Reply LGTM pour merger
• Reply KO pour bloquer

## 📱 Prêt à tester ! L'app est déployée sur l'iPhone (USB/WiFi). - Teste les changements - Reply **LGTM** pour merger - Reply **KO** pour bloquer

millianlmx commented 2026-07-19 16:09:39 +02:00

Author

Owner

Copy Link

LGTM

LGTM

millianlmx merged commit dabd33c8de into main 2026-07-19 16:10:08 +02:00

millianlmx referenced this issue from a commit 2026-07-19 16:10:10 +02:00

Merge pull request 'feat: add admin-web Docker build & push workflow with semantic-release' (#11) from feat/admin-web-docker into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

millianlmx

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: millianlmx/tabatago#11