Conversation
Analysis + concept only, no implementation (Ticket T-20260926-456950433). Documents PromptBoard's ItemType model, ProfiPrompt's current Prompt/Version model, and a minimal additive item_type field proposal (default "prompt", no new table, no schema version bump, migration-safe via existing .get(key, default) pattern already used in prompt_from_dict()). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PTbvD41MCVmnQWaobvHfCk
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: CI rot (python 3.10/3.11/3.12), Ursache liegt nicht im Diff; Konzept inhaltlich schlüssig.
CI-Diagnose (Log python 3.12 gelesen; 3.10/3.11 schlagen im selben Job fehl): tests/test_assets_and_icons.py:14: from PIL import Image führt zu ModuleNotFoundError: No module named 'PIL', Collection-Abbruch (collected 169 items / 1 error). Der PR fügt nur docs/KONZEPT_PROMPT-UNTERKATEGORIEN.md hinzu, der Fehler besteht auf dem Basisstand ebenso. Patch-Vorschlag: Pillow>=10 in die CI-Installation bzw. die dev/test-Extras der pyproject.toml aufnehmen (oder den Test mit pytest.importorskip("PIL") absichern).
Konzept-Review
- Additives
item_type: str = "prompt"mit.get(..., default)-Migration und unveränderterprofiprompt-library-v1passt zur bestehenden Stabilitätsregel inEXPORTFORMAT.md. - Risiko, das im Dokument fehlt: Ältere ProfiPrompt-Versionen lesen nur bekannte Felder und schreiben beim nächsten Speichern
prompts.jsonohneitem_typezurück. Bei Mehrgeräte-/OneDrive-Betrieb mit gemischten Versionen geht die Typangabe dann stillschweigend verloren. Vorschlag: im Konzept "Downgrade/Mischbetrieb" als Risiko ausweisen und ggf. unbekannte Felder beim Round-Trip erhalten. - Die offenen Fragen (Schreibweise, Prompt- vs. Versionsebene, SKILL.md-Export, PromptBoard-Adapter) sind sinnvoll gestellt, die Entscheidung liegt beim Nutzer.
Hinweis: Der PR ist als "Nicht mergen" markiert. Keine Credentials oder Nutzerpfade im Diff.
Generated by Claude Code
|
Review (Claude Sonnet): Nicht gemergt. Das Dokument ist eine interne Planungs-/Entscheidungsvorlage (Ticket-ID T-20260926-456950433, wörtlicher Nutzerauftrag, Abschnitt 'Offene Nutzerfragen', PR-Beschreibung selbst: 'Nicht mergen - Ablage/Review'). Solche Planungsdateien gehören nicht in das öffentliche Repo. Vorschlag: Inhalt in die lokale Aufgabenliste/den Planungsordner übernehmen und den PR schließen, oder erst nach Entscheidung eine öffentliche, bereinigte Fassung (docs/) einreichen. |
|
Interne Planungsvorlage – Inhalt in lokale Projektplanung übernommen; nicht für das öffentliche Repo. |
1 participant
Zusammenfassung
item_type: str = "prompt"-Feld vor (kein neues Enum-Storage, keine neue Datei, kein Schema-Versionssprung, migrationssicher über das bereits vorhandene.get(key, default)-Muster inprompt_from_dict())Nicht mergen
Dieser PR dient der Ablage/Review des Konzepts, nicht der Umsetzung.
🤖 Generated with Claude Code
https://claude.ai/code/session_01PTbvD41MCVmnQWaobvHfCk