feat: sync latest work and apply peer feedback improvements

.env.example

@@ -1,4 +1,8 @@
 # Kopier til .env og tilpass. .env skal aldri committes.
+# `npm run setup` gjør kopieringen for deg (og overskriver aldri en eksisterende .env).
+#
+# Bruker eller passord annerledes enn under? Sett dem i DATABASE_URL og TEST_DATABASE_URL, f.eks.
+#   postgres://it2810:hemmelig@localhost:5432/project2
 
 # Backend
 PORT=3001
@@ -16,13 +20,20 @@ IMDB_MIN_VOTES=100
 # Frontend (valgfri – standard er <base>/graphql)
 VITE_GRAPHQL_URL=
 
-# Bilder og handling fra TMDB (https://www.themoviedb.org/settings/api). Uten nøkkel vises
-# plassholdere i stedet for plakater.
+# Bilder og handling fra TMDB (https://www.themoviedb.org/settings/api).
+# La stå tom i utvikling: `npm run dev` starter da en falsk TMDB-server (port 3999) og en falsk
+# Internet Archive (port 3998) og setter URL-ene under selv, så plakater og gratisfilmer vises uten
+# nøkler. Med ekte nøkkel brukes ekte TMDB. (Backend alene, uten `npm run dev`, viser plassholdere.)
 TMDB_API_KEY=
-# Overstyres i utvikling/E2E for å bruke den lokale falske TMDB-serveren (npm run tmdb:mock -w backend):
+# Settes automatisk av `npm run dev` når nøkkelen er tom. Manuelt (npm run tmdb:mock -w backend):
 # TMDB_API_URL=http://localhost:3999/3
 # TMDB_IMAGE_URL=http://localhost:3999/t/p
 
-# Lovlige gratisfilmer fra Internet Archive (npm run db:archive). Standard er den ekte tjenesten;
-# pek mot den falske serveren (npm run archive:mock -w backend) i utvikling/E2E:
+# Lovlige gratisfilmer fra Internet Archive (npm run db:archive). Standard er den ekte tjenesten.
+# `npm run dev` bruker den falske serveren (port 3998) når TMDB_API_KEY er tom og denne er utkommentert.
+# Manuelt (npm run archive:mock -w backend):
 # ARCHIVE_URL=http://localhost:3998
+
+# Antall syntetiske titler som `npm run setup` genererer hvis data/ er tom (standard 20000).
+# 120000 gir det store settet, men tar lengre tid.
+# FIXTURE_SIZE=20000

.gitignore

@@ -14,5 +14,6 @@ blob-report/
 .vite/
 *.tsbuildinfo
 .DS_Store
+.claude/DONE
 .claude/DONE.local
 data/e2e/

.gitlab-ci.yml

@@ -13,6 +13,7 @@ default:
 stages:
   - check
   - test
+  - e2e
 
 variables:
   HUSKY: '0'
@@ -39,3 +40,42 @@ unit-tests:
     TEST_DATABASE_URL: postgres://postgres@postgres:5432/project2_test
   script:
     - npm test
+
+# Bygger begge pakkene slik at en ødelagt produksjonsbygging oppdages før deploy.
+build:
+  stage: test
+  script:
+    - npm run build
+  artifacts:
+    paths:
+      - frontend/dist/
+    expire_in: 1 week
+
+# E2E mot ekte backend og database. Playwright-bildet har nettleserne ferdig installert, så
+# `playwright install` trengs ikke. Versjonen må følge @playwright/test i package-lock.json.
+# Playwright starter selv falsk TMDB, falsk Internet Archive, backend og frontend (se
+# playwright.config.ts), og global-setup.ts migrerer og fyller databasen med syntetiske data.
+e2e:
+  stage: e2e
+  # Bygger selv via webServer i playwright.config.ts; dist fra build-jobben trengs ikke.
+  dependencies: []
+  image: mcr.microsoft.com/playwright:v1.63.0-noble
+  services:
+    - name: postgres:16
+      alias: postgres
+  variables:
+    POSTGRES_DB: project2_e2e
+    POSTGRES_USER: postgres
+    POSTGRES_HOST_AUTH_METHOD: trust
+    E2E_DATABASE_URL: postgres://postgres@postgres:5432/project2_e2e
+  script:
+    - npm run test:e2e
+  artifacts:
+    # always: en flaky test (feiler først, består på retry) gir også trace i test-results/.
+    when: always
+    reports:
+      junit: test-results/junit.xml
+    paths:
+      - playwright-report/
+      - test-results/
+    expire_in: 1 week

.gitlab/issue_templates/Oppgave.md

@@ -0,0 +1,22 @@
+## Mål
+
+<!-- Hva skal være annerledes når oppgaven er ferdig, og hvorfor trengs det? -->
+
+## Ferdig når
+
+<!-- Konkrete, sjekkbare kriterier. -->
+
+- [ ]
+- [ ] Tester er lagt til eller oppdatert, og lint, typecheck, `npm test` og E2E er grønne
+- [ ] Merge request er godkjent av en annen i gruppa
+
+## Ansvarlig
+
+<!-- Hvem gjør oppgaven, og hvem reviewer? -->
+
+- Ansvarlig: @
+- Reviewer: @
+
+## Notater
+
+<!-- Lenker, skjermbilder, avgrensninger. Valgfritt. -->

.gitlab/merge_request_templates/Default.md

@@ -0,0 +1,33 @@
+## Oppsummering
+
+<!-- Hva endres, og hvorfor? To–tre setninger holder. -->
+
+## Issue
+
+Closes #
+
+<!-- «Closes #12» lukker issuet automatisk når MR-en merges til standardgrenen. -->
+
+## Slik er det testet
+
+<!-- Hvilke tester er lagt til eller endret? Hva er prøvd for hånd (nettleser, skjermleser, mobil)? -->
+
+-
+
+## Sjekkliste
+
+- [ ] `npm run lint` er grønn
+- [ ] `npm run typecheck` er grønn
+- [ ] `npm test` er grønn (backend mot PostgreSQL og komponenttester)
+- [ ] `npm run test:e2e` er grønn, eller endringen berører ikke brukerflyter
+- [ ] Tilgjengelighet: tastatur, synlig fokus og skjermlesertekst er sjekket for nye eller endrede elementer
+- [ ] Utypisk bruk er testet der det er relevant: tomt søk, ingen treff, spesialtegn, lange strenger, nettverksfeil
+- [ ] Dokumentasjon i `docs/` er oppdatert hvis oppførsel eller oppsett er endret
+- [ ] Ingen hemmeligheter (`.env`, nøkler) i diffen
+
+## Review
+
+Reviewer: @
+
+<!-- Reviewer kommenterer i MR-en og merker funn som blokkerende, bør fikses eller valgfritt.
+Funn rettes med nye commits, ikke force-push. -->

.prettierignore

@@ -5,3 +5,4 @@ playwright-report
 test-results
 package-lock.json
 data
+.claude

CLAUDE.md

@@ -4,24 +4,24 @@ Du jobber autonomt. Nicolay ser ikke på skjermen før prosjektet er ferdig. Ikk
 
 ## Roller
 
-- **Hovedsesjonen (Opus) er daglig leder.** Den planlegger, deler opp arbeid, delegerer til subagenter, verifiserer og krysser av i `PLAN.md`. Lederen skriver helst ikke produksjonskode selv.
+- **Hovedsesjonen (Opus) er daglig leder.** Den planlegger, deler opp arbeid, delegerer til subagenter, verifiserer og krysser av i `docs/prosess/plan.md`. Lederen skriver helst ikke produksjonskode selv.
 - **Subagenter (Sonnet):** `backend-utvikler`, `frontend-utvikler`, `tester`, `reviewer`. Gi dem små, avgrensede oppgaver med tydelig ferdigkriterium. Kjør uavhengige oppgaver parallelt.
 
 ## Arbeidsløkke (hver runde)
 
-1. Les `PLAN.md` og `BLOCKERS.md`.
+1. Les `docs/prosess/plan.md` og `docs/prosess/blokkeringer.md`.
 2. Velg neste uavkryssede punkt (eller flere uavhengige).
 3. Deleger → la `reviewer` gå gjennom endringen → la `tester` kjøre relevante tester.
 4. Kjør selv `npm run lint && npm run typecheck && npm test` i berørte pakker. Rødt = ikke ferdig.
-5. Commit, kryss av i `PLAN.md`, skriv én linje i `docs/ki-logg.md`.
+5. Commit, kryss av i `docs/prosess/plan.md`, skriv én linje i `docs/ki/ki-logg.md`.
 6. Er alle MVP-kriterier oppfylt og verifisert: opprett filen `.claude/DONE`.
 
 ## Git
 
 - Jobb alltid på grener fra `mvp`. Aldri commit eller push til `main`, aldri force-push.
 - Én gren per oppgave (`feat/…`, `fix/…`, `test/…`), merge til `mvp` når review og tester er grønne.
 - Små commits med Conventional Commits-meldinger på engelsk.
-- Hvis `glab` er innlogget: opprett issue per PLAN-punkt og MR mot `mvp`, reviewer kommenterer i MR-en. Ellers: gjør det samme lokalt og loggfør i `docs/ki-logg.md`.
+- Hvis `glab` er innlogget: opprett issue per PLAN-punkt og MR mot `mvp`, reviewer kommenterer i MR-en. Ellers: gjør det samme lokalt og loggfør i `docs/ki/ki-logg.md`.
 
 ## Tekniske rammer (ikke avvik uten å skrive begrunnelse i beslutninger.md)
 
@@ -44,4 +44,4 @@ Du jobber autonomt. Nicolay ser ikke på skjermen før prosjektet er ferdig. Ikk
 
 - Ingen `sudo`, ingen endringer utenfor repoet, ingen hemmeligheter i git (`.env` i `.gitignore`, `.env.example` i repo).
 - README.md skrives av gruppa. Agentene legger bare fakta og installasjonssteg i `docs/`, som gruppa kan bygge README på.
-- Står du fast på noe som krever Nicolay (passord, VM, VPN, tilganger): skriv det i `BLOCKERS.md` og gå videre.
+- Står du fast på noe som krever Nicolay (passord, VM, VPN, tilganger): skriv det i `docs/prosess/blokkeringer.md` og gå videre.