fix(rust): drop the stray spaces in the running-server handoff line #34

wtclaude · 2026-09-26T15:09:47Z

wtclaude commented

2026-09-26 15:09:47 +00:00

Found re-walking Rust phase 18 §34.3 step 7 on Debian 12 with the released v0.3.1 binary. The step passed; this is the only thing it turned up.

The defect

After install --game rust on a running server, the handoff printed:

Rust server "beta" is set up. The plugin loads now; `doctor --game rust --server-id beta` confirms it                  connected.

When a source line was wrapped, a run of spaces ended up inside the string literal of the D156 line (installer#29). The existing test only checked fragments of the sentence, and the broken version contained them all, so nothing caught it.

The change

The fix: one line in src/rustgame/sidecar.rs.
The test: the_print_config_document_parses now compares the whole first line of the running-server handoff. I checked that it fails against the pre-fix literal (the assertion shows the gap) and passes with the fix.
The sweep: I scanned every string literal in src/ for the same shape, a single-line literal with a long gap mid-sentence. This was the only one. The other hits are deliberate: aligned columns (verified sha256) and indentation after a \n.

cargo fmt --check, cargo clippy --all-targets -- -D warnings and cargo test are green (163 + 9).

Merging releases v0.3.2.

AI-assisted: Claude Code (Claude Opus 5.5)

🤖 Generated with Claude Code

https://claude.ai/code/session_01E14m6SuuY6i1vASFeGDBeY

Found re-walking Rust phase 18 §34.3 step 7 on Debian 12 with the released v0.3.1 binary. The step passed; this is the only thing it turned up. ## The defect After `install --game rust` on a running server, the handoff printed: ``` Rust server "beta" is set up. The plugin loads now; `doctor --game rust --server-id beta` confirms it connected. ``` When a source line was wrapped, a run of spaces ended up inside the string literal of the D156 line (installer#29). The existing test only checked fragments of the sentence, and the broken version contained them all, so nothing caught it. ## The change - **The fix**: one line in `src/rustgame/sidecar.rs`. - **The test**: `the_print_config_document_parses` now compares the whole first line of the running-server handoff. I checked that it **fails against the pre-fix literal** (the assertion shows the gap) and passes with the fix. - **The sweep**: I scanned every string literal in `src/` for the same shape, a single-line literal with a long gap mid-sentence. This was the only one. The other hits are deliberate: aligned columns (`verified sha256`) and indentation after a `\n`. `cargo fmt --check`, `cargo clippy --all-targets -- -D warnings` and `cargo test` are green (163 + 9). Merging releases v0.3.2. - [x] AI-assisted: Claude Code (Claude Opus 5.5) 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01E14m6SuuY6i1vASFeGDBeY

wtclaude added 1 commit 2026-09-26 15:09:48 +00:00

fix(rust): drop the stray spaces in the running-server handoff line

PR Checks / rust-gates (pull_request) Successful in 57s

Details

d61f46ffb6

The D156 line printed "confirms it                  connected." — a
wrapped source line left a run of spaces inside the string literal.
Found by the phase 18 step 7 re-walk on Debian 12.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E14m6SuuY6i1vASFeGDBeY

wtclaude added 1 commit 2026-09-26 15:14:29 +00:00

test(rust): pin the running-server handoff line whole

PR Checks / rust-gates (pull_request) Successful in 1m0s

Details

9d73367e63

The existing assertions checked fragments ("doctor --game rust
--server-id alpha"), which the stray-space version also contained. The
whole first line is now compared, so a wrapped source line cannot put
its indent back into the sentence. Fails against the pre-fix literal.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E14m6SuuY6i1vASFeGDBeY

whitlocktech approved these changes 2026-09-26 15:41:19 +00:00

whitlocktech merged commit 3a3676a22d into main

2026-09-26 15:41:25 +00:00

whitlocktech deleted branch fix/rust-handoff-spaces

2026-09-26 15:41:26 +00:00

whitlocktech referenced this issue from a commit

2026-09-26 15:41:26 +00:00

Merge pull request 'fix(rust): drop the stray spaces in the running-server handoff line' (#34) from fix/rust-handoff-spaces into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RunicGateway/installer#34