bundle dissolved-verb translator args into an invocation struct #214

New Issue

2026-07-06T18:08:47+02:00

Brummel commented

2026-07-06 18:08:47 +02:00

The dissolved-verb translator/runner functions in
crates/aura-cli/src/verb_sugar.rs carry 10-11 positional args and each need a
#[allow(clippy::too_many_arguments)] (translate_generalize,
run_generalize_sugar). This grows as more verbs dissolve onto the same shape
(walkforward, mc next in the #210 milestone).

Bundle the common invocation shape (fast, slow, stop_length, stop_k, metric,
name, symbols, from_ms, to_ms, blueprint_canonical) into an invocation struct
shared across the dissolved verbs, dropping the allows. Best done ONCE after the
remaining verbs dissolve, so the struct covers the settled common shape.

context: generalize dissolution (#210) architect drift review, [low]
depends on: the walkforward + mc dissolutions (#210 milestone) — do this after

The dissolved-verb translator/runner functions in `crates/aura-cli/src/verb_sugar.rs` carry 10-11 positional args and each need a `#[allow(clippy::too_many_arguments)]` (`translate_generalize`, `run_generalize_sugar`). This grows as more verbs dissolve onto the same shape (walkforward, mc next in the #210 milestone). Bundle the common invocation shape (fast, slow, stop_length, stop_k, metric, name, symbols, from_ms, to_ms, blueprint_canonical) into an invocation struct shared across the dissolved verbs, dropping the allows. Best done ONCE after the remaining verbs dissolve, so the struct covers the settled common shape. context: generalize dissolution (#210) architect drift review, [low] depends on: the walkforward + mc dissolutions (#210 milestone) — do this after

Brummel added the idea label 2026-07-06 18:08:47 +02:00

Brummel referenced this issue

2026-07-06 18:13:42 +02:00

Dissolve the orchestration verbs into the campaign path (sweep, mc, walkforward, generalize) #210

Brummel commented

2026-07-06 23:12:06 +02:00

The walkforward dissolution (#210) landed the THIRD instance of this shape — // (13-arg fns, an 8-element tuple, a third near-identical ). That is the rule-of-three trip point flagged by the cycle-close architect. Deferral still holds by design: the bundle is best cut ONCE over the settled common shape, and one verb remains (mc's R-bootstrap path, #210 decision 7). Do this bundle refactor right after mc dissolves — it will carry all four verbs.

Brummel commented

2026-07-06 23:12:44 +02:00

(Reposting cleanly — the previous comment lost its inline code spans to a shell-quoting artifact.)

The walkforward dissolution (#210) landed the THIRD instance of this shape:
translate_walkforward / register_generated_wf / run_walkforward_sugar
(13-argument functions, an 8-element walkforward_args_from return tuple, and a
third near-identical GeneratedWalkforward struct beside GeneratedSweep /
GeneratedGeneralize). That is the rule-of-three trip point the cycle-close
architect flagged.

Deferral still holds by design: the bundle is best cut ONCE over the settled
common shape, and one verb remains — mc's R-bootstrap path (#210 decision 7). Do
this refactor right after mc dissolves, so the shared invocation struct covers all
four verbs at once.

(Reposting cleanly — the previous comment lost its inline code spans to a shell-quoting artifact.) The walkforward dissolution (#210) landed the THIRD instance of this shape: `translate_walkforward` / `register_generated_wf` / `run_walkforward_sugar` (13-argument functions, an 8-element `walkforward_args_from` return tuple, and a third near-identical `GeneratedWalkforward` struct beside `GeneratedSweep` / `GeneratedGeneralize`). That is the rule-of-three trip point the cycle-close architect flagged. Deferral still holds by design: the bundle is best cut ONCE over the settled common shape, and one verb remains — mc's R-bootstrap path (#210 decision 7). Do this refactor right after mc dissolves, so the shared invocation struct covers all four verbs at once.

Brummel referenced this issue

2026-07-07 00:29:26 +02:00

Dissolve the orchestration verbs into the campaign path (sweep, mc, walkforward, generalize) #210

Brummel referenced this issue from a commit

2026-07-07 09:53:51 +02:00

audit: cycle close generalize dissolution — drift-clean (#210)

Brummel referenced this issue from a commit

2026-07-07 09:53:51 +02:00

feat(cli): walkforward campaign translator + summary reducer (#210 T1-T2)

Brummel referenced this issue from a commit

2026-07-07 09:53:51 +02:00

audit: cycle close walkforward dissolution — drift-clean (#210)