Zum Hauptinhalt springen

Bestehende Technical Debt

Dokumentierte Technical Debt, deaktivierte Features, bekannte Einschränkungen und bewusste Architektur-Entscheidungen.

Deprecated Code​

ItemDateiPriorität
Flux Sidebar stashable Propertyresources/views/flux/sidebar/index.blade.phpNiedrig
ItemDateiPriorität
autoLinkExistingProfiles() deaktiviertapp/Services/Social/SocialProfileLinkSyncer.php:72Mittel
autoApproveMatchingLinks() deaktiviertapp/Jobs/ImportContactLinkProfile.php:125Mittel
Back-Reference Auto-Approval entferntapp/Jobs/ImportContactLinkProfile.php:274Mittel

Problem: Die automatische Freigabe von Kontaktlinks produzierte zu viele Falsch-Positive. Handle-Matching allein (ohne Bidirektionalitäts-Check oder Uniqueness-Scoring) führte zu falschen Cross-Platform-Verknüpfungen.

Lösung für Reaktivierung:

  1. Confidence-Score implementieren (z.B. Handle-Uniqueness-Check, bidirektionale Link-Verifizierung)
  2. Nur Links mit Score > Schwellenwert automatisch freigeben
  3. Links unter Schwellenwert zur manuellen Prüfung markieren

Migration: 2026_02_22_100001_reset_auto_approved_contact_links.php setzt alle auto-genehmigten Links zurück.

Sentry ViewEngineDecorator – Laravel 12 Inkompatibilität​

ERLEDIGT (2026-03-06): Sentry wurde vollständig entfernt und durch Flare (spatie/laravel-flare) + Nightwatch (laravel/nightwatch) ersetzt. Der ViewEngineDecorator-Workaround ist nicht mehr relevant.

ScoreChart: ApexCharts statt Flux UI Chart (Revert)​

EigenschaftWert
PrioritätMittel
Aufwand~2-4 Stunden (wenn Flux Charts die Features unterstützen)
StatusBewusster Revert, ApexCharts funktioniert stabil

Kernprobleme mit Flux UI Charts für ScoreChart:

  1. Zwei Serien (solid + dashed): Flux <flux:chart> unterstützt keine unterschiedlichen Linien-Stile
  2. Gap-Interpolation: Kein Konzept für Serien-Überlappung bei Lücken-Interpolation
  3. Custom Tooltips: Nur Standard-Tooltip, kein Custom-HTML
  4. Feste Y-Achse 0-100: Flux Charts skalieren automatisch, keine feste Range
  5. domain Prop Bug: Typ-Konvertierung zwischen PHP-Array und String scheiterte

Voraussetzungen für erneute Migration: Flux Charts muss mehrere Serien mit unterschiedlichen dashArray-Werten, feste domain/range und Custom-Tooltip-Templates unterstützen.

Location: app/Livewire/Watchers/ScoreChart.php, resources/views/livewire/watchers/score-chart.blade.php

Watchers\Show: defensive $range Property nach Plan-70 Revert​

EigenschaftWert
PrioritätNiedrig
Aufwand~5 Minuten (Property + Doc-Block entfernen)
StatusÜbergangs-Workaround, ab 2026-05-13 entfernen

Kontext: Plan 70 (Charts-Migration) hat in Commit 00a07d8 (29.04.2026 20:46 UTC) den HasTimeRange-Trait zu App\Livewire\Watchers\Show hinzugefügt — und damit eine public string $range = '30d'; Property. Etwa eine Stunde später (b5cd205, 21:48 UTC) wurde der Trait wieder entfernt, weil der page-level Range-Selector mit den Sub-Component-Charts kollidierte.

Folge: Browser-Tabs, die in dem 1-Stunden-Fenster geladen wurden, halten Snapshots mit der alten $range-Property. Bei jeder Livewire-Action senden sie das Feld mit, der Server kennt es nicht mehr → PublicPropertyNotFoundException → 500. Tritt auch nach Hard-Refresh auf, weil mehrere Sub-Components (ScoreChart, VideoScoreChart, AggregatedVideoChart, VideoStatsChart) selbst wire:click="$set('range', ...)"-Buttons haben — bei Race-Conditions im Frontend kann das Update fälschlich an den Parent geroutet werden.

Workaround:

  • app/Livewire/Watchers/Show.php: public ?string $range = null; (nicht im Template gebunden, absorbiert nur das alte Feld)
  • resources/js/app.js: 500-Error-Hook erkennt PublicPropertyNotFoundException im Response-Body und triggert genau einen Page-Reload pro Session (sessionStorage-Guard gegen Endless-Loops)

Entfernen ab 2026-05-13 (14 Tage nach Deploy = Browser-Caches sind durch). Property löschen, Tech-Debt-Eintrag entfernen. JS-Hook bleibt als generelle Sicherheit gegen ähnliche Refactoring-Bugs.

Location: app/Livewire/Watchers/Show.php (Zeilen 70–80), resources/js/app.js (handleStaleSnapshot-Funktion)

Bekannte Einschränkungen​

ItemBereichPriorität
SQLite-Kompatibilität in TestsTest-Suite (5 Tests übersprungen)Mittel
YouTube is_verified nicht via APIDatenqualitätNiedrig

Architektur-Entscheidungen​

ItemBereichPriorität
Queue Driver: database statt redisInfrastrukturMittel
Cache Driver: database statt redisInfrastruktur — Redis-Migration geplant (siehe PLAN.md)Niedrig