Skip to content

feat: add hexagon entity type and hook#73

Merged
Cazka merged 1 commit intoCazka:mainfrom
wakka810:hexagon
Dec 13, 2025
Merged

feat: add hexagon entity type and hook#73
Cazka merged 1 commit intoCazka:mainfrom
wakka810:hexagon

Conversation

@wakka810
Copy link
Contributor

Adds a new Hexagon entity type and its color constant.

Copilot AI review requested due to automatic review settings December 12, 2025 13:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for a new hexagon entity type to the game's entity management system. The changes introduce the Hexagon entity type alongside its distinctive color (#35c5db) and implements a polygon hook to detect and track hexagon entities rendered on screen.

Key Changes:

  • Added Hexagon enum value to EntityType and corresponding color to EntityColor
  • Implemented #hexagonHook() method that detects hexagons with radius 100 using a 6-sided polygon hook

Reviewed changes

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

File Description
src/types/entity.ts Adds Hexagon to EntityType enum and EntityColor enum with cyan color value
src/extensions/entity_manager.ts Implements hexagon detection hook following existing polygon hook pattern

The implementation is clean and follows the established patterns in the codebase. The hexagon hook is consistent with the existing triangle, square, and pentagon hooks in structure and logic. No issues were identified during the review.


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

@Cazka
Copy link
Owner

Cazka commented Dec 13, 2025

thanks for the contribution! looks great.

@Cazka Cazka merged commit cb9968e into Cazka:main Dec 13, 2025
6 checks passed
@wakka810 wakka810 deleted the hexagon branch December 15, 2025 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants