Add minesweeper command #543

Merged

Vylpes merged 1 commit from feature/7-minesweeper-command into release/3.5.0

2026-09-16 13:26:30 +01:00

Vylpes commented

2025-12-23 16:09:21 +00:00

Owner

Add minesweeper command
It is a 9x9 grid with 10 mines
Add tests

- Add minesweeper command - It is a 9x9 grid with 10 mines - Add tests #7

Vylpes added 1 commit

2025-12-23 16:09:22 +00:00

Add minesweeper command

Test / build (push) Successful in 50s

Details

796525ca68

Vylpes requested review from Smithy-bot

2025-12-23 16:09:31 +00:00

Smithy-bot was assigned by Vylpes

2025-12-23 16:09:36 +00:00

Smithy-bot approved these changes

2026-08-29 14:35:07 +01:00

Smithy-bot left a comment

Approve — the minesweeper command is a clean, self-contained addition and looks good to merge.

This adds a /minesweeper slash command (9×9, 10 mines), registers it in registry.ts, and covers the command with unit tests.

What looks good

Grid construction is correct (independent rows, Fisher–Yates for unique mine positions, 8-neighbor counts).
Discord spoiler formatting (||:emoji:||) matches the usual click-to-reveal minesweeper pattern; a 9×9 board stays well under the 2000-character message limit.
Registration follows the same import/RegisterCommand pattern as the other commands.
Tests cover constructor config, execute wiring, mine counts, adjacency, and emoji mapping.

Non-blocking

Unused grid fixture in the adjacency test can be dropped.
generateGrid does not guard mineCount > width * height; fine while the command hardcodes 9/9/10.

No merge-blocking issues.

**Approve** — the minesweeper command is a clean, self-contained addition and looks good to merge. This adds a `/minesweeper` slash command (9×9, 10 mines), registers it in `registry.ts`, and covers the command with unit tests. **What looks good** - Grid construction is correct (independent rows, Fisher–Yates for unique mine positions, 8-neighbor counts). - Discord spoiler formatting (`||:emoji:||`) matches the usual click-to-reveal minesweeper pattern; a 9×9 board stays well under the 2000-character message limit. - Registration follows the same import/`RegisterCommand` pattern as the other commands. - Tests cover constructor config, `execute` wiring, mine counts, adjacency, and emoji mapping. **Non-blocking** - Unused `grid` fixture in the adjacency test can be dropped. - `generateGrid` does not guard `mineCount > width * height`; fine while the command hardcodes 9/9/10. No merge-blocking issues.

Smithy-bot referenced this pull request

2026-08-29 16:24:58 +01:00

Add cursor/* branches to test workflow #559

Vylpes merged commit 634cf62610 into release/3.5.0

2026-09-16 13:26:30 +01:00

Vylpes deleted branch feature/7-minesweeper-command

2026-09-16 13:26:31 +01:00

Vylpes referenced this pull request from a commit

2026-09-16 13:26:31 +01:00

Merge pull request 'Add minesweeper command' (#543) from feature/7-minesweeper-command into release/3.5.0

Vylpes commented

2026-09-16 13:27:49 +01:00