john/McRogueFace
1
0
Fork 0
Code Issues 26 Pull requests Projects Releases Wiki Activity

mcrfpy.Trigger enum — behavior trigger type identifiers #298

New issue
Closed
opened 2026-03-15 00:30:28 +00:00 by john · 1 comment
john commented 2026-03-15 00:30:28 +00:00
Owner
Copy link

Summary

Add a string-evaluating enum for trigger types passed to entity step() callbacks.

Enum values

class Trigger(Enum):
    DONE      # behavior completed its task
    BLOCKED   # next move blocked by collide-label entity
    TARGET    # entity with target label entered FOV

Trigger semantics

Value When fired data argument to step()
DONE Behavior completed (path arrived, sleep finished, custom every turn) Cell arrived at, or None for sleep/custom
BLOCKED Next move blocked by entity with collide label (blocked_cell, blocking_entity) tuple
TARGET Entity with target_label spotted in FOV The spotted entity

Priority

When multiple triggers apply in the same turn: TARGET > BLOCKED > DONE

Implementation

Follow the existing IntEnum pattern (PyInputState.h/cpp):

// PyTrigger.h
enum class TriggerType : int {
    DONE = 0,
    BLOCKED,
    TARGET
};

Files to create/modify

  • src/PyTrigger.h/cpp — new enum type
  • src/McRFPy_API.cpp — register type
## Summary Add a string-evaluating enum for trigger types passed to entity `step()` callbacks. ## Enum values ```python class Trigger(Enum): DONE # behavior completed its task BLOCKED # next move blocked by collide-label entity TARGET # entity with target label entered FOV ``` ## Trigger semantics | Value | When fired | `data` argument to `step()` | |-------|-----------|----------------------------| | `DONE` | Behavior completed (path arrived, sleep finished, custom every turn) | Cell arrived at, or `None` for sleep/custom | | `BLOCKED` | Next move blocked by entity with collide label | `(blocked_cell, blocking_entity)` tuple | | `TARGET` | Entity with `target_label` spotted in FOV | The spotted entity | ## Priority When multiple triggers apply in the same turn: `TARGET > BLOCKED > DONE` ## Implementation Follow the existing IntEnum pattern (`PyInputState.h/cpp`): ```cpp // PyTrigger.h enum class TriggerType : int { DONE = 0, BLOCKED, TARGET }; ``` ## Files to create/modify - `src/PyTrigger.h/cpp` — new enum type - `src/McRFPy_API.cpp` — register type
john commented 2026-03-15 20:18:30 +00:00
Author
Owner
Copy link

Roadmap context

Part of the Grid & Entity Overhaul Roadmap (docs/GRID_ENTITY_OVERHAUL_ROADMAP.md), Phase 2 (Entity Data Model Extensions). Leaf dependency — can be implemented in parallel with #295, #296, #297. Required by #299 (step callback).

## Roadmap context Part of the Grid & Entity Overhaul Roadmap (`docs/GRID_ENTITY_OVERHAUL_ROADMAP.md`), **Phase 2** (Entity Data Model Extensions). Leaf dependency — can be implemented in parallel with #295, #296, #297. Required by #299 (step callback).
john closed this issue 2026-03-16 11:21:02 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
threedee
emscripten-mcrogueface
rogueliketutorial25
alpha_presentable
alpha_streamline_2
alpha_streamline_1
interpreter_mode
grid_entity_integration
reprs_and_member_names
break_up_ui_h
standardize_font_handling
standardize_vector_handling
standardize_color_handling
standardize_texture_handling
raii_pyobjects
rebase-for-7DRL_Feb-2024
engjam_uicollection
engjam_ui_overhaul
0.2.3-prerelease-7drl2026-emscripten
Labels
Clear labels   
Alpha Release Requirement

This issue is a blocker to 0.1 Alpha release of McRogueFace.

  
Bugfix

   
Demo Target

Functionality to demonstrate; Depends on new features, but this issue isn't for writing them.

  
Documentation

   
Major Feature

Significant time and effort required.

  
Minor Feature

Some effort required to create or overhaul functionality.

  
priority:tier1-active

Current development focus - critical path to v1.0

  
priority:tier2-foundation

Important foundation work - not blocking v1.0

  
priority:tier3-future

Future features - deferred until after v1.0

  
priority:tier4-deferred

   
Refactoring & Cleanup

no new functionality, just improving the codebase.

  
system:animation

Animation and property interpolation

  
system:documentation

Documentation infrastructure

  
system:grid

Grid system and spatial containers

  
system:input

Input handling and events

  
system:performance

Performance optimization and profiling

  
system:procgen

Procedural Generation (Noise, BSP, Heightmap)

  
system:python-binding

Python/C++ binding layer

  
system:rendering

Rendering pipeline and visuals

  
system:ui-hierarchy

UI component hierarchy and composition

  
Tiny Feature

quick and easy - a few lines or with little interconnection around the codebase

  
workflow:blocked

Blocked by other work - waiting on dependencies

  
workflow:needs-benchmark

Needs performance testing and benchmarks

  
workflow:needs-documentation

Needs documentation before or after implementation

No labels
Alpha Release Requirement
Bugfix
Demo Target
Documentation
Major Feature
Minor Feature
priority:tier1-active
priority:tier2-foundation
priority:tier3-future
priority:tier4-deferred
Refactoring & Cleanup
system:animation
system:documentation
system:grid
system:input
system:performance
system:procgen
system:python-binding
system:rendering
system:ui-hierarchy
Tiny Feature
workflow:blocked
workflow:needs-benchmark
workflow:needs-documentation
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#299 Entity step() callback — Python-side turn handler
john/McRogueFace
#300 Behavior data struct and primitive implementations
john/McRogueFace
Reference
john/McRogueFace#298
No description provided.