Skip to content

[Engine] TriggerPolicy cadence execution and interval reset#128

Merged
learncold merged 1 commit into
mainfrom
codex/issue-95-trigger-policy-interval
Apr 13, 2026
Merged

[Engine] TriggerPolicy cadence execution and interval reset#128
learncold merged 1 commit into
mainfrom
codex/issue-95-trigger-policy-interval

Conversation

@learncold
Copy link
Copy Markdown
Owner

Summary

  • Wire SystemDescriptor.triggerPolicy into SystemScheduler::executePhase() so phase execution now evaluates descriptor-owned cadence instead of relying on runtime call-site filtering.
  • Add deterministic Interval cadence support with intervalTicks, including scheduler-owned cadence state that resets on initialize() and stop().
  • Extend scheduler/runtime tests and update the runtime UML explanation so the implementation matches the documented engine contract.

Related Issue

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • Additional checks:
  • cmake --preset windows-debug-no-app
  • cmake --build --preset build-no-app-debug
  • ctest --preset test-no-app-debug --output-on-failure

Risks / Follow-up

  • Interval currently fires on the first applicable phase tick and then repeats every intervalTicks; if the engine later needs offset-aware cadence, that should be introduced as an explicit descriptor field rather than hidden scheduler behavior.

@learncold learncold marked this pull request as ready for review April 13, 2026 11:24
@learncold learncold merged commit 58a5233 into main Apr 13, 2026
4 checks passed
@learncold learncold deleted the codex/issue-95-trigger-policy-interval branch April 13, 2026 11:32
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.

Task-TriggerPolicy 실행 조건과 Interval cadence 적용

1 participant