Repository navigation
fix: TS18048 in GamePhaseComponents.test.tsx - #324
Conversation
npm audit fix (ohne --force) behebt das einzige High-Finding (browserslist <=4.28.6, GHSA-c83g-rgw3-j3cx) sowie @xmldom/xmldom (moderate), beide ohne Breaking Changes. Verbleibende 21 moderate Findings (decode-uri-component/uuid über die Expo-Toolchain) bleiben bewusst offen — audit-level=high blockiert davon nichts mehr. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AS54CHuJc8Hm28KnyRqpz2
…ch toBeTruthy) npx tsc --noEmit schlägt auf testing fehl: 'hintBtn' is possibly 'undefined' (TS18048). expect(hintBtn).toBeTruthy() narrowt den Typ für TypeScript nicht, daher non-null assertion für den Property-Zugriff im nächsten Statement. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AS54CHuJc8Hm28KnyRqpz2
… testing) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AS54CHuJc8Hm28KnyRqpz2
🔀 Mergeability-Report✅ Konfliktfrei mergebar ( CI-Checks: ✅ keine fehlgeschlagenen Checks ⏳ mergeability / mergeability Merge-Checkliste
Kostenlos generiert von reusable-mergeability.yml · keine API-Kosten · review-gate: |
|
Fix existiert bereits in PR #323 (npm audit fix) — dort ist dieser TS18048-Fix bereits gemerged, daher jetzt umgekehrt auch hier portiert (Commit 819168e; beide Branches sind inhaltlich identisch, PR #323 enthält den TS-Fix zusätzlich). Generated by Claude Code |
Befund
npx tsc --noEmitschlägt auftestingfehl:expect(hintBtn).toBeTruthy()narrowt den Typ für TypeScript nicht — der nachfolgende Property-Zugriff bleibtpossibly undefined. Dadurch schlägt der🔍 Code Quality & Linting-Check in CI auf jedem PR gegentestingfehl (aufgefallen als Teil von PR #323, unabhängig von dessen eigentlicher Änderung).Fix
Non-null assertion für den Property-Zugriff im nächsten Statement (etabliertes Muster für diesen Fall).
Test plan
npx tsc --noEmit→ Exit 0npm test -- GamePhaseComponents→ 15/15 grün🤖 Generated with Claude Code
https://claude.ai/code/session_01AS54CHuJc8Hm28KnyRqpz2
Generated by Claude Code