P0b: document.currentScript.parentElement null für Dynamic Scripts #75

New issue

Closed

opened 2026-06-18 17:17:55 +00:00 by Artur · 1 comment

Artur commented

2026-06-18 17:17:55 +00:00

Owner

P0b: `document.currentScript.parentElement` null für Dynamic Scripts

Priority: HIGH
Betrifft: kit.svelte.dev, web.dev, speedtest.net, ALLE Seiten mit Inline-Scripts im <head>
Impact: Scripts crashen beim Start. SvelteKit-Seite kann nicht bootstrappen.

Problembeschreibung

Der DynamicScriptHandler führt Scripts aus, BEVOR sie im DOM attached sind.
document.currentScript wird zwar korrekt gesetzt (Sprint 12), aber das Script-Element
hat zu diesem Zeitpunkt .parentElement === null.

Aktueller Ablauf:

Parser findet <script>-Tag
ScriptLoader.fetchAndExecute() → fetch URL → get content
ExecutionRealm.execute(code, scriptElement) → setzt __currentScript
Script läuft: document.currentScript.parentElement → null (nicht attached)
TypeError → Script bricht ab → App bootstrapped nicht

Gewünschtes Verhalten

Im Browser:

Parser öffnet <script>-Tag
Script wird in den DOM-Baum eingehängt (parent vorhanden)
Script wird ausgeführt → currentScript.parentElement = korrektes DOM-Element

Option A: Script vor execute() attachien (EMPFEHLUNG)

// dynamic-scripts.ts — vor execute():
if (!scriptElement.parentNode) {
  const target = scriptElement.getAttribute('data-target') ?? document.head;
  const anchor = document.querySelector(target) ?? document.head.lastChild;
  if (anchor && anchor.parentNode) {
    anchor.parentNode.insertBefore(scriptElement, anchor.nextSibling);
  }
}

Akzeptanzkriterien

document.currentScript.parentElement ist niemals null während execution
kit.svelte.dev lädt ohne diesen TypeError
Kein Nebeneffekt auf andere Script-Ausführungen
Unit-Test: Script vor execute in DOM attached

Betroffene Dateien

Datei	Änderung
`src/js/dynamic-scripts.ts`	Script-Attachment vor execute
`tests/unit/sprint16-currentscript.test.ts`	5+ Tests

## P0b: `document.currentScript.parentElement` null für Dynamic Scripts **Priority:** HIGH **Betrifft:** kit.svelte.dev, web.dev, speedtest.net, ALLE Seiten mit Inline-Scripts im <head> **Impact:** Scripts crashen beim Start. SvelteKit-Seite kann nicht bootstrappen. ### Problembeschreibung Der DynamicScriptHandler führt Scripts aus, BEVOR sie im DOM attached sind. `document.currentScript` wird zwar korrekt gesetzt (Sprint 12), aber das Script-Element hat zu diesem Zeitpunkt `.parentElement === null`. **Aktueller Ablauf:** 1. Parser findet `<script>`-Tag 2. ScriptLoader.fetchAndExecute() → fetch URL → get content 3. ExecutionRealm.execute(code, scriptElement) → setzt `__currentScript` 4. Script läuft: `document.currentScript.parentElement` → **null** (nicht attached) 5. TypeError → Script bricht ab → App bootstrapped nicht ### Gewünschtes Verhalten Im Browser: 1. Parser öffnet `<script>`-Tag 2. Script wird in den DOM-Baum eingehängt (parent vorhanden) 3. Script wird ausgeführt → `currentScript.parentElement` = korrektes DOM-Element ### Option A: Script vor execute() attachien (EMPFEHLUNG) ```typescript // dynamic-scripts.ts — vor execute(): if (!scriptElement.parentNode) { const target = scriptElement.getAttribute('data-target') ?? document.head; const anchor = document.querySelector(target) ?? document.head.lastChild; if (anchor && anchor.parentNode) { anchor.parentNode.insertBefore(scriptElement, anchor.nextSibling); } } ``` ### Akzeptanzkriterien - [ ] `document.currentScript.parentElement` ist niemals null während execution - [ ] kit.svelte.dev lädt ohne diesen TypeError - [ ] Kein Nebeneffekt auf andere Script-Ausführungen - [ ] Unit-Test: Script vor execute in DOM attached ### Betroffene Dateien | Datei | Änderung | |-------|----------| | `src/js/dynamic-scripts.ts` | Script-Attachment vor execute | | `tests/unit/sprint16-currentscript.test.ts` | 5+ Tests |

Artur added the

bug

label

2026-06-18 17:17:55 +00:00

glow was assigned by Artur

2026-06-18 17:17:55 +00:00

Artur referenced this issue from a commit

2026-06-18 17:26:51 +00:00

Sprint 16: Convergence Gap Fixes — ES Module, HTML Filter, currentScript, Animate

Artur closed this issue

2026-06-18 17:27:08 +00:00

Artur commented

2026-06-18 17:27:08 +00:00

Author

Owner

✅ Sprint 16 implementiert — document.currentScript.parentElement Fix

Was wurde gemacht:

execution-realm.ts:execute(): Nur __currentScript setzen wenn scriptElement.parentNode existiert
parser.ts:makeScriptElement(): Fakes Script-Element hat jetzt parentNode = document.head + parentElement Getter
Verhindert TypeError bei SvelteKit/web.dev/speedtest Inline-Scripts

Commit: eca1765

✅ **Sprint 16 implementiert** — `document.currentScript.parentElement` Fix **Was wurde gemacht:** - `execution-realm.ts:execute()`: Nur `__currentScript` setzen wenn `scriptElement.parentNode` existiert - `parser.ts:makeScriptElement()`: Fakes Script-Element hat jetzt `parentNode = document.head` + `parentElement` Getter - Verhindert `TypeError` bei SvelteKit/web.dev/speedtest Inline-Scripts **Commit:** `eca1765`

Artur referenced this issue

2026-06-18 17:27:10 +00:00

P2c: Dynamic Script Attachment — Script vor execute in DOM einhängen #83

Artur referenced this issue from a commit

2026-06-18 17:36:02 +00:00

Sprint 16: ES Module Dependency Resolution via Bun Plugin

Artur referenced this issue

2026-06-18 18:17:58 +00:00

Phase 2: Production Engine — Epic (ES Module, Performance, API Completeness) #56