Unwrap the walkforward path's UnknownKnob Debug frame into prose #269

Closed
opened 2026-07-13 22:52:23 +02:00 by claude · 0 comments
Collaborator

The walkforward verb path still frames its unknown-axis rejection in the Debug register: the UnknownKnob wrapper prints UnknownKnob("<message>") around a payload that is itself already prose (wrapped from override_paths' own String error). Two tests pin the literal UnknownKnob substring on stderr (aura_walkforward_emits_an_unknown_axis_rejection_once, aura_walkforward_over_a_blueprint_rejects_an_unknown_axis in crates/aura-cli/tests/cli_run.rs), so the frame survives.

This is the same defect family the sweep path shed with #247 (its KindMismatch/MissingKnob now render as prose — commit 414448b): no Rust identifier should reach a user's stderr. The fix is to unwrap the already-prose payload at the walkforward boundary (or render the variant in the #247 register) and update the two pinned tests to assert the prose instead of the Debug frame.

Found while scoping #247: the issue named only the sweep path's two variants, so the walkforward frame was deliberately left out of that slice.

The walkforward verb path still frames its unknown-axis rejection in the Debug register: the `UnknownKnob` wrapper prints `UnknownKnob("<message>")` around a payload that is itself already prose (wrapped from `override_paths`' own String error). Two tests pin the literal `UnknownKnob` substring on stderr (`aura_walkforward_emits_an_unknown_axis_rejection_once`, `aura_walkforward_over_a_blueprint_rejects_an_unknown_axis` in `crates/aura-cli/tests/cli_run.rs`), so the frame survives. This is the same defect family the sweep path shed with #247 (its `KindMismatch`/`MissingKnob` now render as prose — commit 414448b): no Rust identifier should reach a user's stderr. The fix is to unwrap the already-prose payload at the walkforward boundary (or render the variant in the #247 register) and update the two pinned tests to assert the prose instead of the Debug frame. Found while scoping #247: the issue named only the sweep path's two variants, so the walkforward frame was deliberately left out of that slice.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Brummel/Aura#269