You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Phase 3 of the roadmap: a declarative trace-testing pipeline for the behavior trees under edit, built on the exec core (#72, now merged to devel). This is the "pipeline for testing the behavior tree" from the review.
Rebased onto devel; conflicts resolved (CHANGELOG.md, plus the latched-status tick.ts refinement kept over devel's original). Now targets devel directly.
What's included
src/btcpp/exec/trace.ts — TraceScenario / TraceAssertion DSL and runScenario(doc, scenario):
Mocks: per-leaf outcomes (fixed or per-tick scripts) keyed by instance name / registered ID.
Assertions: node status at a path, rootStatus, or blackboard subset — at a specific tick or the final tick. Missing (never-ticked) nodes read as IDLE.
Records the full per-tick trace so both live and end-of-run expectations work; failures report the actual value.
Sample *.trace.json fixtures: simple_sequence (happy + failing-branch), fallback_recovery (primary→backup recovery), subtree_running (two-tick RUNNING action through a SubTree).
src/test/unit/trace.test.ts — unit tests for the runner plus a discovery runner that executes every fixtures/**/*.trace.json against its sibling .xml, turning fixtures into a CI behavior gate.
Included engine refinement
Improved the exec core to a Groot-like latched-status model: internal-state reset (for re-ticking) is separated from visible halt (IDLE), so a failed node keeps its FAILURE status in the snapshot instead of being clobbered. Benefits both traces and the overlays. All exec-core and verifier tests still pass with this version.
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Risk: medium. Cursor Bugbot was not present on this PR; CI checks (pre-commit, Linux/macOS/Windows builds) all passed. Approving the trace-testing pipeline changes with unit tests and fixtures; no reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
Base automatically changed from
cursor/exec-tick-core-2d56 to
develJuly 2, 2026 21:51
The reason will be displayed to describe this comment to others. Learn more.
Risk: medium. Cursor Bugbot was not present on this PR; pre-commit passed and there are no unresolved automated findings. Approving the trace-testing pipeline changes; no reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Summary
Phase 3 of the roadmap: a declarative trace-testing pipeline for the behavior trees under edit, built on the exec core (#72, now merged to
devel). This is the "pipeline for testing the behavior tree" from the review.What's included
src/btcpp/exec/trace.ts—TraceScenario/TraceAssertionDSL andrunScenario(doc, scenario):statusat apath,rootStatus, orblackboardsubset — at a specifictickor the final tick. Missing (never-ticked) nodes read asIDLE.*.trace.jsonfixtures:simple_sequence(happy + failing-branch),fallback_recovery(primary→backup recovery),subtree_running(two-tick RUNNING action through a SubTree).src/test/unit/trace.test.ts— unit tests for the runner plus a discovery runner that executes everyfixtures/**/*.trace.jsonagainst its sibling.xml, turning fixtures into a CI behavior gate.Included engine refinement
Improved the exec core to a Groot-like latched-status model: internal-state reset (for re-ticking) is separated from visible halt (IDLE), so a failed node keeps its
FAILUREstatus in the snapshot instead of being clobbered. Benefits both traces and the overlays. All exec-core and verifier tests still pass with this version.Testing
npm run test:unit— 77 passed (trace + exec + verifier together)npm run check-typesnpm run lintnpm run compile