🎨 phpstan level 8

[garak]

No description provided.

[Copilot]

Pull request overview

This pull request raises the PHPStan strictness to level 8 and applies code changes across the chess engine and output renderers to satisfy the tighter static analysis requirements.

Changes:

• Bump PHPStan configuration from level 7 to level 8 and update dev tooling versions in composer.json.
• Add null-guards/assertions to output rendering loops and coordinate rendering helpers.
• Adjust core engine code in Chess.php to address nullable types in move handling, attack detection, castling, and repetition logic.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
src/Output/UnicodeOutput.php	Adds a null key guard during board iteration for PHPStan.
src/Output/AsciiOutput.php	Adds a null key guard during board iteration for PHPStan.
src/Output/SvgOutput.php	Skips iteration when board key is null (and when piece is null).
src/Output/ImagineOutput.php	Adds null guards/assertions; changes board drawing and piece placement iteration.
src/Entry.php	Tightens phpdoc for castling array type.
src/Chess.php	Introduces nullable-handling changes in promotions/captures, history indexing, castling generation, attacked(), repetition detection, and SAN generation.
phpstan.neon	Increases PHPStan level to 8.
composer.json	Updates dev dependency constraints (phpstan, cs-fixer, phpbench, etc.).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.