Add block scoping to all switch case statements in layout.ts

[Copilot]

The switch statement in layout.ts processes different HTML tag types. Only the SCRIPT case had block scoping to prevent variable name collisions between cases. All other cases declare variables (e.g., linkElement, styleElement, imgElement) without block scope, risking unintentional variable shadowing.

Changes

• Wrapped all case blocks in the main tag processing switch statement with curly braces
• Ensures variables declared in each case (e.g., let linkElement, let styleElement) are scoped to that case only
• Prevents potential variable conflicts between different tag processing paths

// Before: unscoped variable declarations
case "LINK":
    let linkElement = this.element(node.id) as HTMLLinkElement;
    // ...
    break;
case "STYLE":
    let styleElement = this.element(node.id) as HTMLStyleElement;
    // ...
    break;

// After: each case properly scoped
case "LINK": {
    let linkElement = this.element(node.id) as HTMLLinkElement;
    // ...
    break;
}
case "STYLE": {
    let styleElement = this.element(node.id) as HTMLStyleElement;
    // ...
    break;
}

This aligns all cases with the pattern already established for the SCRIPT case.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.