Template-Refresh aus dem Framework-Review (Pins, Prod-Gerüst, ci_host, skip_if_exists, Render-Job) - #14
Merged
Conversation
… ci_host, skip_if_exists, Render-Job - Paket-Pins aktuell (integrations 0.2.2, auth 0.1.1, lti 0.1.4, ui 0.6.0 — bisher u. a. integrations 0.1.0 ohne clean_media_url-Fix). - api.ts: same-origin im Prod-Build statt localhost-Fallback; 204 und FormData korrekt; init-Spread überschreibt Header nicht mehr. - App-Shell: PreferencesMenu aus @basicbar/ui statt 90-Zeilen-Eigenbau. - settings: ImproperlyConfigured bei DEBUG=0 mit Dev-SECRET_KEY. - Prod-Gerüst: Root-Dockerfile (SPA gebacken), docker-compose.prod.yml, .env.prod.example, .dockerignore; Dev-Dockerfile ohne libpq-dev/gcc, CMD uvicorn; gunicorn raus. - Neue Frage ci_host: GitHub Actions (CI + GHCR-Release) oder GitLab-CI, bedingt gerendert. - _skip_if_exists für Identitäts-/Inhaltsdateien, _message_after_copy. - ruff-Config (Import-Sortierung), echte Tests in common/tests.py, Kopierreste bereinigt, manage.py ausführbar (EXE001). - Repo-CI: template-probe rendert mit LTI und fährt ruff, check, makemigrations --check, Tests gegen PostgreSQL, Frontend-Build; Paket-Jobs als Matrix mit Caches. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This was referenced Oct 1, 2026
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.
Zweiter PR aus dem Framework-Review (nach #13): das Copier-Template holt die Pakete ein, bekommt das Prod-Gerüst der reifen Tools und wird in der CI endlich selbst gerendert.
Was sich ändert
Pins & Code — integrations 0.2.2 / auth 0.1.1 / lti 0.1.4 / ui 0.6.0 (bisher 0.1.0/0.1.0/0.1.2/0.2.1 — das Template generierte Tools ohne den
clean_media_url-Sicherheitsfix).api.tsfällt im Prod-Build auf same-origin stattlocalhostzurück, behandelt 204/FormData und verliert bei eigenenheadersnicht mehr den CSRF-Token. Die 90-Zeilen-SettingsMenu-Kopie weichtPreferencesMenuaus @basicbar/ui. Settings werfenImproperlyConfigured, wennDJANGO_DEBUG=0mit dem Dev-SECRET_KEYstartet.Prod-Gerüst (bisher nur in abstimmbar/ausleihbar, nahezu identisch — Rule of Two): Root-
Dockerfile(Multi-Stage, SPA gebacken),docker-compose.prod.yml(Pflichtvariablen per:?, migrate+collectstatic beim Start, Caddy serviert aus Volumes),.env.prod.example,.dockerignorefür beide Build-Kontexte. Dev-Dockerfile ohnelibpq-dev/gcc(alles Wheels, ≈150 MB weniger),gunicornraus.Copier — neue Frage
ci_host(github/gitlab, bedingt gerenderte.github/workflows/{ci,release}.ymlbzw..gitlab-ci.yml);_skip_if_existsfür Identitäts-/Inhaltsdateien (tailwind.config.js, App.tsx, locales, README/CLAUDE.md, Keycloak-Realm, Caddyfile, .env.prod.example) — damit entfallen z. B. erkennbars 11 Hue-Konflikte beim nächstencopier update;_message_after_copymit den ersten Schritten. Ports/Hue-Defaults rücken hinter erkennbar auf.Qualität — ruff-Config mit Import-Sortierung, echte Tests in
common/tests.py, Kopierreste (Pagination-Docstring,accounts.oidc-Verweis,CMD runserver) bereinigt,manage.pyausführbar. Repo-CI: Jobtemplate-proberendert das Template (mit LTI) und fährt ruff,check,makemigrations --check, Testsuite gegen PostgreSQL und Frontend-Build der generierten App; die drei Paket-Jobs sind jetzt eine Matrix mit pip-/npm-Cache.Verifiziert (lokal)
Beide
ci_host-Varianten gerendert (bedingte CI-Dateien korrekt, keine Jinja-Reste); generiertes Backend im Container: ruff sauber,checkOK,makemigrations --checkOK, 9 Tests grün; Frontendnpm install && npm run buildgrün (der Chunk-Size-Hinweis ist das TipTap-im-Bundle-Thema des Reviews → ui-PR).Nicht enthalten
Bestandstools werden nicht automatisch aktualisiert —
copier update --vcs-ref HEADist ein bewusster Schritt pro Tool (Changelog nennt, wo Konflikte zu erwarten sind).🤖 Generated with Claude Code