Design/ux v1#2
Merged
Merged
Conversation
- compose.yaml runs postgres 16 (host port 5433 to avoid agent-platform clash) + redis 7 with healthchecks and named volumes. - Makefile auto-loads .env so run-api / migrate-up / etc. work without a manual `export $(grep ...)` step. - `make up` is detached: builds binaries, brings up infra, applies migrations, starts both go services in the background, polls /health until healthy, then returns to the prompt. PIDs in .pids/, logs in .logs/ (gitignored). - `make down` stops go services via PID files (with pkill fallback) and runs docker compose down. - `make logs` tails both go service logs interleaved; `make logs-infra` tails compose; `make ps` shows everything in one view. - shoplit-redirect dev DSN points at the same shoplit user as the API; the prod-only readonly role is provisioned during deployment (Plan 5).
Locks the design direction for the frontend milestone (Plan 5 pulled
forward):
- editorial / magazine mood (LTK + Pinterest + Glossier reference)
- minimal customization per cart (cover image + accent color + bio)
- vertical stack of large product cards on /c/{slug}, 2-col at ≥768px
- mobile-first; light theme only in v1
- Next.js 14 + Tailwind + shadcn/ui + Fraunces/Inter
- 7 pages designed: /, /login, /dashboard, /dashboard/carts/new,
/dashboard/carts/[id], /c/{slug}, /legal/*
- mocked data layer (lib/api-client.ts ⇄ lib/mocks.ts) so real backend
wiring in later plans is a one-file swap
Spec sits alongside the architecture spec; complements rather than
replaces it.
Bootstrap Next.js 14 in web/ → tokens + fonts + shadcn → typed mocks + api-client → shared components → 7 pages → final verification with Lighthouse + axe a11y. Each task is self-contained with full code blocks; the design spec remains the source of truth for visual details.
- start.sh now launches `pnpm dev` from web/ alongside the two go services; auto-runs `pnpm install` on first run if node_modules is missing. Polls /:3000 with a 30s timeout (vs 8s for go) to account for next.js cold-start compile. - stop.sh kills the web pid + pkills any stray `next dev` / `next-server` processes as a fallback. - Makefile: new `run-web` and `web-install` targets for ad-hoc use; logs/ps now include web; help text updated. - README: Node + pnpm added to prerequisites; workflow rewritten to mention http://localhost:3000 as the entry point. Verified end-to-end: make up brings all three services to HTTP 200 within ~5s; make down stops cleanly with no orphan node/go processes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.