From 5dc8f182017eac357800f01d943d3fe9e70948a4 Mon Sep 17 00:00:00 2001 From: Stefan Mogeritsch Date: Mon, 19 Jan 2026 17:05:57 +0100 Subject: [PATCH] docs(journal): update session log with local dev setup and infra improvements Enhanced the session log to include local development environment setup, DI fixes, tracing updates, and Docker Compose optimizations. Reflected improvements in architecture, observability, and build results. --- docs/99_Journal/2026-01-19_Session_Log.md | 26 +++++++++++++++++------ 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/docs/99_Journal/2026-01-19_Session_Log.md b/docs/99_Journal/2026-01-19_Session_Log.md index 6c0dd102..2926ae15 100644 --- a/docs/99_Journal/2026-01-19_Session_Log.md +++ b/docs/99_Journal/2026-01-19_Session_Log.md @@ -12,7 +12,7 @@ status: COMPLETED # Session Log: 19. Jänner 2026 ## Zielsetzung -Abschluss des "Trace Bullet" (Ping Feature) durch Abarbeitung der offenen Punkte aus dem Handover vom 17.01. und Bereinigung der Frontend-Struktur. +Abschluss des "Trace Bullet" (Ping Feature) durch Abarbeitung der offenen Punkte aus dem Handover vom 17.01., Bereinigung der Frontend-Struktur und Inbetriebnahme der lokalen Entwicklungsumgebung. ## Durchgeführte Arbeiten @@ -20,9 +20,19 @@ Abschluss des "Trace Bullet" (Ping Feature) durch Abarbeitung der offenen Punkte * **Migration:** Tests aus `at.mocode.clients.pingfeature` wurden in die Clean Architecture Struktur (`at.mocode.ping.feature.data` und `presentation`) migriert. * **Cleanup:** Das alte Package `at.mocode.clients.pingfeature` wurde vollständig entfernt (inkl. Tests). * **Integration Test:** Ein neuer `PingSyncIntegrationTest` wurde erstellt, der den Datenfluss vom API-Client bis zum Repository verifiziert. +* **Dependency Injection Fixes:** + * `SyncModule` nutzt nun korrekt den `named("apiClient")` HttpClient. + * `localDbModule` wurde in `main.kt` (Desktop Shell) registriert, um `DatabaseProvider` verfügbar zu machen. +* **Code Quality:** Unused Imports, Variables und redundante Elvis-Operatoren wurden bereinigt. ### 2. Infrastructure & Observability -* **Tracing Fix:** Der `ping-service` hatte die Tracing-Dependencies (`monitoring-client`) nicht eingebunden. Dies wurde in der `build.gradle.kts` korrigiert. Nun sollten Traces lückenlos in Zipkin erscheinen. +* **Tracing Fix:** Der `ping-service` hatte die Tracing-Dependencies (`monitoring-client`) nicht eingebunden. Dies wurde in der `build.gradle.kts` korrigiert. +* **Local Dev Config:** + * `monitoring-defaults.properties` wurde angepasst, damit Zipkin lokal unter `localhost:9411` angesprochen wird (statt `zipkin:9411`). + * `application.yaml` des `ping-service` wurde bereinigt, um zentrale Defaults zu nutzen. +* **Docker Compose:** + * `docker-compose.yaml` wurde strukturiert und um **Mailpit** erweitert. + * Services (Postgres, Redis, Consul, Zipkin, Keycloak) laufen stabil. ### 3. Build & Contracts * **API Visibility:** `contracts:ping-api` exportiert nun `core-domain` via `api` statt `implementation`. Dies behebt die Compiler-Warnung `Cannot access 'Syncable'`. @@ -31,10 +41,12 @@ Abschluss des "Trace Bullet" (Ping Feature) durch Abarbeitung der offenen Punkte * **Architecture:** Neue Datei `docs/01_Architecture/02_Frontend_Architecture.md` erstellt, die die Modularisierungsstrategie und Clean Architecture Vorgaben festhält. ## Ergebnisse -* Der Build ist **GRÜN**. -* Die Architektur ist konsistent (keine Legacy-Pakete mehr im Ping-Feature). -* Observability ist im Backend sichergestellt. +* **Build:** GRÜN. +* **Backend:** Gateway und Ping-Service laufen lokal und kommunizieren mit der Docker-Infrastruktur. Tracing funktioniert. +* **Frontend:** Desktop-App startet erfolgreich. +* **Architektur:** Konsistent und sauber (keine Legacy-Pakete mehr im Ping-Feature). ## Nächste Schritte (Ausblick) -* Beginn der Arbeit an den Fachdomänen (Veranstaltungen/Events). -* Migration des `auth-feature` auf die neue Architektur bei nächster Gelegenheit. +* **Frontend Debugging:** Analyse der verbleibenden Laufzeit-Probleme in der Desktop-App. +* **Feature Development:** Beginn der Arbeit an den Fachdomänen (Veranstaltungen/Events). +* **Auth Migration:** Migration des `auth-feature` auf die neue Architektur bei nächster Gelegenheit.