Modify BattleRunner.java to connect to external server and emit structured output #133

New Issue

2026-08-30T11:38:43+02:00

SirStone commented

2026-08-30 11:38:43 +02:00

Wayfinder child of: #126

Context

The current tools/battle_runner/RunBattle.java uses BattleRunner.create() with .embeddedServer(). The test framework (#126) needs it to connect to an external TR server (started separately by the Nim framework with -I flag) and print structured output to stdout that Nim can parse.

Requirements

Modify BattleRunner.create() to connect to an external server URL (passed as CLI arg or env var)
Define a structured stdout output format (recommend JSON-lines) that includes: round results, winners, scores, per-bot results, total ticks
This output format is the contract that #127's parser implements — co-design them
Accept battle configuration as CLI args: bot paths, number of rounds, arena size
Exit cleanly when battle completes

Blocked by

Nothing

Blocks

#127, #130

## Wayfinder child of: #126 ### Context The current `tools/battle_runner/RunBattle.java` uses `BattleRunner.create()` with `.embeddedServer()`. The test framework (#126) needs it to connect to an **external** TR server (started separately by the Nim framework with `-I` flag) and print structured output to stdout that Nim can parse. ### Requirements - Modify `BattleRunner.create()` to connect to an external server URL (passed as CLI arg or env var) - Define a structured stdout output format (recommend JSON-lines) that includes: round results, winners, scores, per-bot results, total ticks - This output format is the **contract** that #127's parser implements — co-design them - Accept battle configuration as CLI args: bot paths, number of rounds, arena size - Exit cleanly when battle completes ### Blocked by Nothing ### Blocks #127, #130

SirStone added the wayfinder:map label 2026-08-30 11:38:50 +02:00

SirStone referenced this issue

2026-08-30 11:46:07 +02:00

Wayfinder: Shared integration test framework for garages #126

SirStone commented

2026-08-30 11:46:17 +02:00

Spec addition from grilling

The structured stdout format is the contract between Java and Nim. Recommend JSON-lines, one JSON object per event:

{"event":"ready"} — server connected
{"event":"round_end","round":1,"results":[{"name":"Bot1","score":100,"rank":1},...]}
{"event":"battle_end","winners":["Bot1"],"total_rounds":10,"results":[...]}

This format must be co-designed with #127 (the Nim parser). Implement parser and emitter together or define the schema first.

BattleRunner process timeout: 30s after last output line.

## Spec addition from grilling The structured stdout format is the **contract** between Java and Nim. Recommend JSON-lines, one JSON object per event: - `{"event":"ready"}` — server connected - `{"event":"round_end","round":1,"results":[{"name":"Bot1","score":100,"rank":1},...]}` - `{"event":"battle_end","winners":["Bot1"],"total_rounds":10,"results":[...]}` This format must be co-designed with #127 (the Nim parser). Implement parser and emitter together or define the schema first. BattleRunner process timeout: **30s** after last output line.

SirStone added wayfinder:task and removed wayfinder:map labels 2026-08-30 11:48:28 +02:00

SirStone referenced this issue

2026-08-30 11:50:53 +02:00

Wayfinder: Shared integration test framework for garages #126

SirStone referenced this issue

2026-08-30 11:50:54 +02:00

Define BattleResult type and server output parser #127

SirStone commented

2026-08-30 12:05:57 +02:00