cschlaefke/Shuffle
1
0
Fork 0
Code Issues 33 Pull requests 2 Projects Releases Packages Wiki Activity Actions

feat(catalog): Fehlendes Spiel melden – Backend (KAT-04, 13a) #67

Open
cschlaefke wants to merge 6 commits from feature/13a-fehlendes-spiel-backend into main
pull from: feature/13a-fehlendes-spiel-backend
merge into: cschlaefke:main
Conversation 0 Commits 6 Files changed 10 +1222 -73
cschlaefke commented 2026-11-04 18:54:25 +00:00
Owner
Copy link

Refs #13

Teil 13a (Backend) von „Fehlendes Spiel melden“ nach docs/specs/13-fehlendes-spiel-melden.md.

Inhalt

  • Migration 20261104100000_game_reports_user.sql (additiv): strip_leading_article, normalize_game_title, report_day, find_similar_catalog_games, report_missing_game; game_reports.result_seen_at; Insert-Recht auf 6 Spalten, Update nur result_seen_at; Policies; Trigger game_reports_daily_limit (10 je Nutzer und Berliner Tag, Advisory-Lock); Grants erst entziehen, dann gezielt erteilen.
  • Entscheidung Chris (2026-11-04, Option B): Es gibt kein „abgelehnt mit Verweis“ mehr. Mit Verweis wird merged verwendet (hängt direkt um), rejected nur ohne Ziel. Deshalb entfällt adopt_report_target (keine security-definer-Funktion in 13a). admin_resolve_report weist rejected mit Ziel ab (report_reject_has_target), dazu Check game_reports_rejected_no_target. Ändert Spec-F3 aus Spec #2; als Nachtrag/Spec-F10 dokumentiert.
  • pgTAP: neue 10_game_reports_user.test.sql (190 Tests), 08 angepasst (rejected-mit-Ziel-Fixtures → merged, neue Tests); nach sauberem db reset: 1004 pgTAP-Tests, 41 Deno-Tests grün.
  • Docs/Specs: Spec 13, Spec #2 (Nachtrag), katalog-pflege.md §6, datenmodell-r1.md, supabase/README.md, docs/specs/13-umsetzung.md.

Review

code-reviewer: APPROVE (Stand vor Option B; Befunde S2/N2/N4 eingearbeitet). Die Änderung für Option B wird gesondert nachgeprüft, bis dahin kein Label.

Für 13b / nicht geprüft

  • App kennt nur noch pending / approved / merged / rejected (ohne Ziel); Edition-Hinweis bei approved und merged.
  • Nicht geprüft: Zugriff über REST/HTTP, echte Parallelität des Tageslimits (Advisory-Lock, manuell/T1), Eigentümer postgres auf Staging/Prod.
  • 08 nutzt weiter disable trigger für Fixtures (begründet im Nachtrag).

🤖 Generated with Claude Code

Refs #13 Teil 13a (Backend) von „Fehlendes Spiel melden“ nach `docs/specs/13-fehlendes-spiel-melden.md`. ## Inhalt - Migration `20261104100000_game_reports_user.sql` (additiv): `strip_leading_article`, `normalize_game_title`, `report_day`, `find_similar_catalog_games`, `report_missing_game`; `game_reports.result_seen_at`; Insert-Recht auf 6 Spalten, Update nur `result_seen_at`; Policies; Trigger `game_reports_daily_limit` (10 je Nutzer und Berliner Tag, Advisory-Lock); Grants erst entziehen, dann gezielt erteilen. - **Entscheidung Chris (2026-11-04, Option B):** Es gibt kein „abgelehnt mit Verweis“ mehr. Mit Verweis wird `merged` verwendet (hängt direkt um), `rejected` nur ohne Ziel. Deshalb **entfällt `adopt_report_target`** (keine security-definer-Funktion in 13a). `admin_resolve_report` weist `rejected` mit Ziel ab (`report_reject_has_target`), dazu Check `game_reports_rejected_no_target`. Ändert Spec-F3 aus Spec #2; als Nachtrag/Spec-F10 dokumentiert. - pgTAP: neue `10_game_reports_user.test.sql` (190 Tests), `08` angepasst (rejected-mit-Ziel-Fixtures → merged, neue Tests); nach sauberem `db reset`: 1004 pgTAP-Tests, 41 Deno-Tests grün. - Docs/Specs: Spec 13, Spec #2 (Nachtrag), `katalog-pflege.md` §6, `datenmodell-r1.md`, `supabase/README.md`, `docs/specs/13-umsetzung.md`. ## Review code-reviewer: APPROVE (Stand vor Option B; Befunde S2/N2/N4 eingearbeitet). **Die Änderung für Option B wird gesondert nachgeprüft**, bis dahin kein Label. ## Für 13b / nicht geprüft - App kennt nur noch pending / approved / merged / rejected (ohne Ziel); Edition-Hinweis bei approved und merged. - Nicht geprüft: Zugriff über REST/HTTP, echte Parallelität des Tageslimits (Advisory-Lock, manuell/T1), Eigentümer `postgres` auf Staging/Prod. - `08` nutzt weiter `disable trigger` für Fixtures (begründet im Nachtrag). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
cschlaefke added 5 commits 2026-11-04 18:54:25 +00:00 
Refs #13

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Refs #13

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
feat(db): kein 'abgelehnt mit Verweis' mehr, adopt_report_target entfällt (Entscheidung Chris, Refs #13)
All checks were successful
CI / flutter (pull_request) Successful in 1m35s
Details
CI / web-build (pull_request) Successful in 1m5s
Details
CI / supabase (pull_request) Successful in 10s
Details
CI / supabase-tests (pull_request) Successful in 43s
Details
CI / e2e (pull_request) Successful in 3m21s
Details
3af18073a6 
admin_resolve_report weist rejected mit Ziel ab (report_reject_has_target),
Check game_reports_rejected_no_target, Tests und Docs angepasst.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
cschlaefke added 1 commit 2026-11-04 19:03:52 +00:00
docs(db): Review-Nachbesserungen PR #67 (Spec, Nachtrag, Release-Checkliste, Testkorrekturen) Refs #13
All checks were successful
CI / flutter (pull_request) Successful in 2m9s
Details
CI / web-build (pull_request) Successful in 1m33s
Details
CI / supabase (pull_request) Successful in 10s
Details
CI / supabase-tests (pull_request) Successful in 37s
Details
CI / e2e (pull_request) Successful in 4m14s
Details
ec42421312 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
cschlaefke added the
status/ready-for-merge
 label 2026-11-04 20:39:51 +00:00
All checks were successful
CI / flutter (pull_request) Successful in 2m9s
Details
CI / web-build (pull_request) Successful in 1m33s
Details
CI / supabase (pull_request) Successful in 10s
Details
CI / supabase-tests (pull_request) Successful in 37s
Details
CI / e2e (pull_request) Successful in 4m14s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/13a-fehlendes-spiel-backend:feature/13a-fehlendes-spiel-backend
git checkout feature/13a-fehlendes-spiel-backend

Merge

Merge the changes and update on Forgejo.
git checkout main
git merge --no-ff feature/13a-fehlendes-spiel-backend
git checkout main
git merge --ff-only feature/13a-fehlendes-spiel-backend
git checkout feature/13a-fehlendes-spiel-backend
git rebase main
git checkout main
git merge --no-ff feature/13a-fehlendes-spiel-backend
git checkout main
git merge --squash feature/13a-fehlendes-spiel-backend
git checkout main
git merge --ff-only feature/13a-fehlendes-spiel-backend
git checkout main
git merge feature/13a-fehlendes-spiel-backend
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
epic/katalog

Epic KAT: Spielekatalog

  
epic/konto

Epic KTO: Konto & Profil

  
epic/partien

Epic PAR: Partien & Statistik

  
epic/querschnitt

Epic QUE: Querschnitt

  
epic/sammlung

Epic SAM: Sammlung & Listen

  
needs-clarification

Fachliche Frage offen

  
prio/could

könnte

  
prio/must

MVP – muss

  
prio/should

sollte

  
size/l

3 Tage

  
size/m

1–3 Tage

  
size/s

< 1 Tag

  
size/xl

Mehr als 3 Tage

  
status/in-progress

In Umsetzung

  
status/ready-for-merge

Review bestanden, Merge durch Chris

  
status/spec-review

Spec wartet auf Freigabe

  
type/bug

Fehler

  
type/epic

Epic

  
type/story

Fachliche User Story

  
type/tech

Technische Aufgabe

No labels
epic/katalog
epic/konto
epic/partien
epic/querschnitt
epic/sammlung
needs-clarification
prio/could
prio/must
prio/should
size/l
size/m
size/s
size/xl
status/in-progress
status/ready-for-merge
status/spec-review
type/bug
type/epic
type/story
type/tech
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: cschlaefke/Shuffle#67
No description provided.
Delete branch "feature/13a-fehlendes-spiel-backend"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?