refactor(sonar): clear isolated shell and dart-marker leftovers

- replace the last shell theme-action ternary with a small helper
- switch dart-marker internal geometry/debug helpers to object parameters and tighten one overlay guard
- validate with lint, changelog check, tests, and Sonar on xConfig (31 -> 26 open issues)

Author: thomasasen

CHANGELOG.md

@@ -21,6 +21,9 @@ direkt zu einer versionierten Release-Sektion.
 
 ### Fixed
 
+- Nutzerwirkung: Keine beabsichtigte sichtbare Verhaltensänderung; xConfig-Themenaktionen, Cricket-Highlighting und Dart-Marker-Darts laufen fachlich weiter auf denselben Pfaden, drücken in dieser Sonar-Welle aber einige kleine Guard- und Debug-Signaturen direkter aus.
+  Technik: `xconfig-ui/shell-view` ersetzt den letzten verschachtelten Hintergrundbild-Hinweis durch einen kleinen Action-Text-Helper, `cricket-highlighter` bereinigt einen lokalen Shape-Guard per Optional-Chaining, und `dart-marker-darts` bündelt zwei rein interne Debug-/Geometrie-Signaturen auf kleine Objektparameter statt 8-Parameter-Helfer; die bestehenden Shell-, Cricket- und Dart-Marker-Regressionen bleiben dabei der fachliche Schutz gegen Drift.
+
 - Nutzerwirkung: Keine beabsichtigte sichtbare Verhaltensänderung; die xConfig-Shell reagiert weiterhin gleich auf Menü-, Settings-, Update- und Lifecycle-Aktionen, drückt die internen Controller-Pfade in dieser Sonar-Welle aber klarer getrennt aus.
   Technik: `xconfig-ui/action-controller`, `render-controller` und `lifecycle-controller` ziehen ihre lokalen Action-, Render- und Lifecycle-Helfer in äußere Hilfsfunktionen, bereinigen dabei einen verschachtelten Select-Ternary und lassen die öffentliche Controller-API sowie das bestehende Event-, Queue-, Modal- und Teardown-Verhalten unverändert; die vorhandenen Action- und Shell-Regressionen bleiben dabei der Schutz gegen Drift.
 

src/features/cricket-highlighter/logic.js

@@ -470,7 +470,7 @@ function ensurePresentationPatterns(overlay, visualConfig) {
 }
 
 function applyShapeStyle(shape, presentation, targetLabel) {
-  if (!shape || !shape.classList || !shape.style) {
+  if (!shape?.classList || !shape.style) {
     return;
   }
   const normalizedPresentation = resolvePresentationToken(presentation);

src/features/dart-marker-darts/logic.js

@@ -286,7 +286,7 @@ function ensureOverlaySvg(state, documentRef) {
   }
 
   let overlay = state.overlayNode;
-  if (overlay && overlay.isConnected) {
+  if (overlay?.isConnected) {
     if (String(overlay.tagName || "").toLowerCase() === "svg") {
       return overlay;
     }
@@ -854,7 +854,16 @@ function getFlightOffsets(center, boardCenter, dartLength) {
   return { start, mid };
 }
 
-function buildGeometryPayload(marker, index, screenPoint, overlayRect, svgRect, groupRect, entry, extra = {}) {
+function buildGeometryPayload({
+  marker,
+  index,
+  screenPoint,
+  overlayRect,
+  svgRect,
+  groupRect,
+  entry,
+  extra = {},
+}) {
   const payload = {
     markerKey: buildMarkerKey(marker),
     index,
@@ -1107,16 +1116,16 @@ function buildOverlaySignature(overlayRect, dartLength) {
   ].join("|");
 }
 
-function maybeEmitBoardAndOverlayDebug(
+function maybeEmitBoardAndOverlayDebug({
   state,
   featureDebug,
   board,
   boardRect,
   groupRect,
   overlayRect,
   dartLength,
-  clipPath
-) {
+  clipPath,
+}) {
   const boardSignature = buildBoardSignature(board, boardRect, groupRect);
   if (state.lastBoardSignature !== boardSignature) {
     state.lastBoardSignature = boardSignature;
@@ -1334,16 +1343,16 @@ export function updateDartMarkerDarts(options = {}) {
     y: boardRect.top + boardRect.height / 2 - overlayRect.top,
   };
 
-  maybeEmitBoardAndOverlayDebug(
+  maybeEmitBoardAndOverlayDebug({
     state,
     featureDebug,
     board,
     boardRect,
     groupRect,
     overlayRect,
     dartLength,
-    clipPath
-  );
+    clipPath,
+  });
 
   const markerSet = new Set(markers);
   let removed = 0;
@@ -1417,15 +1426,15 @@ export function updateDartMarkerDarts(options = {}) {
       visualConfig,
     });
 
-    const geometryPayload = buildGeometryPayload(
+    const geometryPayload = buildGeometryPayload({
       marker,
       index,
       screenPoint,
       overlayRect,
-      boardRect,
+      svgRect: boardRect,
       groupRect,
-      entry
-    );
+      entry,
+    });
     emitDebug(state, featureDebug, "geometry-apply", geometryPayload);
 
     if (isNew) {

src/features/xconfig-ui/shell-view.js

@@ -803,6 +803,18 @@ function buildDartDesignOptionLayout(
   return layout;
 }
 
+function resolveThemeActionNoteText(action) {
+  if (action === "clearThemeBackground") {
+    return "Entfernt das gespeicherte Bild für dieses Theme.";
+  }
+
+  if (action === "uploadThemeBackground") {
+    return "Öffnet die Dateiauswahl und speichert das Bild für dieses Theme.";
+  }
+
+  return "";
+}
+
 function buildFeatureField(documentRef, feature, field) {
   const fieldId = `ad-xconfig-field-${feature.featureKey}-${field.key || field.action}`;
 
@@ -825,14 +837,7 @@ function buildFeatureField(documentRef, feature, field) {
       },
     });
     wrapper.appendChild(button);
-    const noteText = String(
-      field.description ||
-        (field.action === "clearThemeBackground"
-          ? "Entfernt das gespeicherte Bild für dieses Theme."
-          : field.action === "uploadThemeBackground"
-            ? "Öffnet die Dateiauswahl und speichert das Bild für dieses Theme."
-            : "")
-    ).trim();
+    const noteText = String(field.description || resolveThemeActionNoteText(field.action)).trim();
     if (noteText) {
       wrapper.appendChild(createElement(documentRef, "p", {
         className: "ad-xconfig-note",