SirStone/SirRoboGarage
1
0
Fork 0
Code Issues 28 Pull Requests Actions Projects Releases Wiki Activity

Define BattleResult type and server output parser #127

New Issue
Closed
opened 2026-08-30 10:22:01 +02:00 by SirStone · 2 comments
SirStone commented 2026-08-30 10:22:01 +02:00
Owner
Copy Link
Copy Source

Wayfinder child of: #126

Design the BattleResult type (rounds, winners, scores, ticks, per-bot results) and implement parsing of the Java battle runner's stdout into it. This is the contract the whole framework is built on.

Blocked by: nothing

Grilling Revisions

Parser file: lives in common_libs/test_framework/parser.nim (not test_framework.nim). Named parser.nim to avoid confusion with test files.

BattleResult fields — full score breakdown:

  • Per bot: rank, name, totalScore, survival, lastSurvivorBonus, bulletDamage, bulletKillBonus, ramDamage, ramKillBonus, firstPlaces, secondPlaces, thirdPlaces
  • Per round: seq[RoundResult] with roundNumber, turnCount, winner

Output contract — stdout/stderr split:

  • Parser only reads stdout (lifecycle markers + results block).
  • Diagnostic output goes to stderr — parser never sees it.
  • Runner must be updated to print per-round winner in the results block.

Tests: common_libs/test_framework/tests/test_parser.nim

**Wayfinder child of:** #126 Design the `BattleResult` type (rounds, winners, scores, ticks, per-bot results) and implement parsing of the Java battle runner's stdout into it. This is the contract the whole framework is built on. **Blocked by:** nothing ### Grilling Revisions **Parser file**: lives in `common_libs/test_framework/parser.nim` (not `test_framework.nim`). Named `parser.nim` to avoid confusion with test files. **BattleResult fields — full score breakdown:** - Per bot: `rank`, `name`, `totalScore`, `survival`, `lastSurvivorBonus`, `bulletDamage`, `bulletKillBonus`, `ramDamage`, `ramKillBonus`, `firstPlaces`, `secondPlaces`, `thirdPlaces` - Per round: `seq[RoundResult]` with `roundNumber`, `turnCount`, `winner` **Output contract — stdout/stderr split:** - Parser only reads stdout (lifecycle markers + results block). - Diagnostic output goes to stderr — parser never sees it. - Runner must be updated to print per-round winner in the results block. **Tests**: `common_libs/test_framework/tests/test_parser.nim`
SirStone added the wayfinder:task label 2026-08-30 10:22:01 +02:00
SirStone commented 2026-08-30 11:50:54 +02:00
Author
Owner
Copy Link
Copy Source

Dependency correction

Blocked by: #133 (not "nothing" as originally stated). The parser cannot be implemented until #133 defines the structured stdout output format from BattleRunner.java.

## Dependency correction Blocked by: #133 (not "nothing" as originally stated). The parser cannot be implemented until #133 defines the structured stdout output format from BattleRunner.java.
SirStone commented 2026-08-30 12:05:53 +02:00
Author
Owner
Copy Link
Copy Source

Implemented in commit 57a1915.

Implemented in commit 57a1915.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ready-for-agent
wayfinder:grilling
Wayfinder design question to grill
 wayfinder:map
Wayfinder map issue
 wayfinder:prototype
wayfinder:research
Wayfinder research question
 wayfinder:task
Wayfinder implementation task
No Label wayfinder:task
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
SirStone
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SirStone/SirRoboGarage#127
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?