Jeroen Schweitzer jpmschweitzer
  • Joined on 2025-11-14
jpmschweitzer commented on pull request jpmschweitzer/settled-reach#70 2026-02-25 16:54:10 +01:00
feat(client): save/load client UI — F5/F6 quicksave/quickload (#554)

Review: client → main (type: code) — Round 2

Hoshe (Code Quality): REQUEST_CHANGES → overridden (false negative)

Hoshe read pre-fix files from the main worktree instead of…

jpmschweitzer pushed to client at jpmschweitzer/settled-reach 2026-02-25 15:43:59 +01:00
3725a3df5e fix(client): address PR #70 review — event leak, quit flush, notification color
jpmschweitzer commented on pull request jpmschweitzer/settled-reach#70 2026-02-25 15:41:21 +01:00
feat(client): save/load client UI — F5/F6 quicksave/quickload (#554)

Review: client → main (type: code) — Round 1

Hoshe (Code Quality): REQUEST_CHANGES

Core save/load flow is architecturally sound — one-shot carry-forward, protocol versioning, D-085…

jpmschweitzer created pull request jpmschweitzer/settled-reach#70 2026-02-25 13:16:19 +01:00
feat(client): save/load client UI — F5/F6 quicksave/quickload (#554)
jpmschweitzer pushed to client at jpmschweitzer/settled-reach 2026-02-25 13:15:09 +01:00
e1ea07e746 feat(client): save/load client UI — F5/F6 quicksave/quickload (#554)
jpmschweitzer pushed to main at jpmschweitzer/settled-reach 2026-02-25 13:11:38 +01:00
704415fa99 chore(meta): update changelog
jpmschweitzer pushed to main at jpmschweitzer/settled-reach 2026-02-25 13:10:26 +01:00
af3398ea34 fix(ci): address PR #69 review — stale comments, dead CLI flags
jpmschweitzer closed pull request jpmschweitzer/settled-reach#69 2026-02-25 13:09:52 +01:00
feat(ci): Sprint 19 test infrastructure and protocol handshake
jpmschweitzer pushed to main at jpmschweitzer/settled-reach 2026-02-25 13:09:51 +01:00
a0ed10468b Merge remote-tracking branch 'origin/ci'
fcd2fd3cf7 fix(skills): pr-review must run from main worktree, no workarounds
b6c4ecb302 feat(ci): protocol version handshake client + IPC benchmark (#556, #342)
1bd528e521 Merge remote-tracking branch 'origin/main' into ci
01b354569b chore(config): allowlist cargo and test runner commands in settings
Compare 9 commits »
jpmschweitzer pushed to main at jpmschweitzer/settled-reach 2026-02-25 13:08:48 +01:00
51bad2e606 fix(client): bump protocol version to 15
jpmschweitzer commented on pull request jpmschweitzer/settled-reach#69 2026-02-25 13:08:11 +01:00
feat(ci): Sprint 19 test infrastructure and protocol handshake

Review: ci → main (type: code) — Round 1

Hoshe (Code Quality): REQUEST_CHANGES

Solid infrastructure work. D-030 three-layer IPC architecture faithfully implemented. Scripts well-structur…

jpmschweitzer closed pull request jpmschweitzer/settled-reach#67 2026-02-25 13:07:23 +01:00
feat(client): sprint 19 — test infra, session management, debug overlay
jpmschweitzer pushed to main at jpmschweitzer/settled-reach 2026-02-25 13:07:21 +01:00
cf4919913e Merge remote-tracking branch 'origin/client'
e7f1e80307 fix(client): POI category key mismatch — use poi_category from wire format
16e1cc3a09 chore(skills): tighten sprint agent spawn rules
ab654c8865 test(client): add sprint 18 test suites for examine display and minimap
83a244fc2c fix(client): address PR #67 round 2 — broken tests, BBCode escape, expiry loop
Compare 12 commits »
jpmschweitzer pushed to planning at jpmschweitzer/settled-reach 2026-02-25 13:06:20 +01:00
2493e2b0b5 chore(meta): update changelog
fdad0958b3 docs(workshops): add outcomes files for 5 completed workshops
e9ca3cadf0 Merge remote-tracking branch 'origin/server'
23fbfdbfc5 fix(simulation): PR #68 review — version bump, tracing warns, test coverage
f874979ff9 docs(decisions): file 6 D-records and 10 Q-records from workshop backlog
Compare 55 commits »
jpmschweitzer pushed to client at jpmschweitzer/settled-reach 2026-02-25 13:04:03 +01:00
e51650ec74 Merge origin/main into client (resolve CHANGELOG conflict)
2493e2b0b5 chore(meta): update changelog
fdad0958b3 docs(workshops): add outcomes files for 5 completed workshops
e9ca3cadf0 Merge remote-tracking branch 'origin/server'
23fbfdbfc5 fix(simulation): PR #68 review — version bump, tracing warns, test coverage
Compare 11 commits »
jpmschweitzer pushed to ci at jpmschweitzer/settled-reach 2026-02-25 13:02:07 +01:00
fcd2fd3cf7 fix(skills): pr-review must run from main worktree, no workarounds
jpmschweitzer created pull request jpmschweitzer/settled-reach#69 2026-02-25 12:58:27 +01:00
feat(ci): Sprint 19 test infrastructure and protocol handshake
jpmschweitzer pushed to ci at jpmschweitzer/settled-reach 2026-02-25 12:58:06 +01:00
b6c4ecb302 feat(ci): protocol version handshake client + IPC benchmark (#556, #342)
1bd528e521 Merge remote-tracking branch 'origin/main' into ci
01b354569b chore(config): allowlist cargo and test runner commands in settings
81ec337e4d feat(ci): IPC round-trip timing benchmark scaffold (#342)
732b5f3adc fix(simulation): resolve duplicate examine_result field and regenerate fixtures
Compare 111 commits »
jpmschweitzer commented on pull request jpmschweitzer/settled-reach#67 2026-02-25 12:58:06 +01:00
feat(client): sprint 19 — test infra, session management, debug overlay

Review Round 3: client → main (type: code)

Hoshe (Code Quality): REQUEST_CHANGES

Round 2 critical fix confirmed — _test_input_queue proxy is present and correctly wired. BBCode escape…

jpmschweitzer pushed to client at jpmschweitzer/settled-reach 2026-02-25 12:57:31 +01:00
e7f1e80307 fix(client): POI category key mismatch — use poi_category from wire format