Sprint 27: Tech-Debt — dynamic-scripts.ts aufräumen (zwei Queue-Systeme) #89

New issue

Closed

opened 2026-06-18 20:37:51 +00:00 by Artur · 1 comment

Artur commented

2026-06-18 20:37:51 +00:00

Owner

Problembeschreibung

dynamic-scripts.ts hat zwei verschiedene Queue-Systeme die parallel existieren und teilweise denselben Zweck erfüllen:

Queue 1: _dynamicQueue (Set)

private _dynamicQueue: Set<string> = new Set();  // URL-basiert
private _dynamicScripts: Map<string, any> = new Map();  // URL → element

Genutzt von: _patchAppendChild(), _patchSrcSetter(), MutationObserver

Queue 2: chunkQueue (Array)

private chunkQueue: Array<{element, url, resolve, reject}> = [];

Genutzt von: _queueDynamic() via _processQueue()

Problem: _dynamicQueue wird NIRGENDS verarbeitet. _processQueue() iteriert nur über chunkQueue. Die Daten in _dynamicQueue sind tot.

Aufräum-Plan

_dynamicQueue und _dynamicScripts aus dem Code entfernen
_queueDynamic() behalten — pusht korrekt in chunkQueue
_urlQueue() in _scheduleQueue() umbenennen (spiegelt wider was es tut)
Prüfen ob _dynamicQueue-Entries irgendwo DOCH gelesen werden

Akzeptanzkriterien

_dynamicQueue und _dynamicScripts entfernt
Alle Tests weiterhin grün
Kein anderer Code referenziert diese Properties
Crawl 19/19 korpus

Betroffene Dateien

Datei	Änderung
`src/js/dynamic-scripts.ts`	`_dynamicQueue`, `_dynamicScripts` entfernen

## Problembeschreibung `dynamic-scripts.ts` hat zwei verschiedene Queue-Systeme die parallel existieren und teilweise denselben Zweck erfüllen: **Queue 1: `_dynamicQueue` (Set)** ```typescript private _dynamicQueue: Set<string> = new Set(); // URL-basiert private _dynamicScripts: Map<string, any> = new Map(); // URL → element ``` Genutzt von: `_patchAppendChild()`, `_patchSrcSetter()`, MutationObserver **Queue 2: `chunkQueue` (Array)** ```typescript private chunkQueue: Array<{element, url, resolve, reject}> = []; ``` Genutzt von: `_queueDynamic()` via `_processQueue()` **Problem:** `_dynamicQueue` wird NIRGENDS verarbeitet. `_processQueue()` iteriert nur über `chunkQueue`. Die Daten in `_dynamicQueue` sind tot. ## Aufräum-Plan 1. `_dynamicQueue` und `_dynamicScripts` aus dem Code entfernen 2. `_queueDynamic()` behalten — pusht korrekt in `chunkQueue` 3. `_urlQueue()` in `_scheduleQueue()` umbenennen (spiegelt wider was es tut) 4. Prüfen ob `_dynamicQueue`-Entries irgendwo DOCH gelesen werden ## Akzeptanzkriterien - [ ] `_dynamicQueue` und `_dynamicScripts` entfernt - [ ] Alle Tests weiterhin grün - [ ] Kein anderer Code referenziert diese Properties - [ ] Crawl 19/19 korpus ## Betroffene Dateien | Datei | Änderung | |-------|----------| | `src/js/dynamic-scripts.ts` | `_dynamicQueue`, `_dynamicScripts` entfernen |

Artur added the

housekeeping

label

2026-06-18 20:37:51 +00:00

Artur referenced this issue

2026-06-19 08:50:06 +00:00

Issue #96: SubtleCrypto — Chrome-Kompatibilität für AWS WAF PoW #91

Artur referenced this issue

2026-06-19 08:52:53 +00:00

Issue #99: PageNetworkManager — Unified Network Stack + Shared-State Fix (fixes #89) #94

Artur commented

2026-06-19 10:17:33 +00:00

Author

Owner

Bereits erledigt ✅

_dynamicQueue und _dynamicScripts existieren nicht mehr in dynamic-scripts.ts — wurden in Sprint 23b mitbereinigt. Der Code hat nur noch chunkQueue und processingElements (instance-level).

**Bereits erledigt** ✅ `_dynamicQueue` und `_dynamicScripts` existieren nicht mehr in `dynamic-scripts.ts` — wurden in Sprint 23b mitbereinigt. Der Code hat nur noch `chunkQueue` und `processingElements` (instance-level).

Artur closed this issue

2026-06-19 10:17:33 +00:00