feat(cli): retire the PIP demo pair (sma/momentum) to the generic path (#159 cut 4)
The two legacy pip-scored demos — the SMA-cross (`sma`/`Strategy::SmaCross`) and momentum (`momentum`/`Strategy::Momentum`) — and the whole built-in *synthetic* run/sweep/walkforward/mc + `macd` surface they anchored are dropped, not migrated: both terminated in a `SimBroker(pip_size)` construction-arg node absent from the zero-arg std vocabulary and reported the legacy pip yardstick C10 retired with the realistic broker. With the R-family already migrated to data (cuts 1-3) the `Strategy` enum held only these two, so the retirement collapses the entire built-in-strategy machinery: `Strategy`, `HarnessKind`, `strategy_from`, `run_sweep`/`sweep_family`/`momentum_*`, `run_sample*`/`sample_*`/`sma_cross`, the built-in `walkforward_family`/`sweep_over`/`run_oos` and `run_mc`/`mc_family`, the `macd` composite cluster, and `run_dispatch`/`run_args_from`/`RunCmd.--harness`. The generic surfaces survive untouched: `aura run <bp.json>`, `aura sweep <bp.json> --axis`, `aura walkforward|mc --strategy r-sma --real`, `aura campaign`, `aura generalize`. Grammar consequence (derived forks logged on #159): `run`/`sweep` now require an explicit blueprint (no synthetic default); `sma`/`momentum` join the retired r-breakout/r-meanrev tokens in the generic `Usage:` error, pinned by a new negative (`sweep_and_walkforward_reject_retired_pip_tokens_as_unrecognized`); bare `aura graph` renders the shipped `examples/r_sma_open.json` (loaded via `blueprint_from_json`), and the render self-containment test with it; the `sample-model.json` viewer corpus + its two `.mjs` tests stay frozen as static test data. `member_key`/`MAX_KEY`/`fnv1a64` were removed too. The plan wrongly listed `member_key` as a survivor; after the built-in sweep/mc retire it has no production caller (campaign_run derives its own content key), so it and its unit tests are genuinely dead. The spec-compliance gate surfaced this plan-deviation; it was ratified after verifying the tree stays green. Verified: `cargo build --workspace`, `cargo test --workspace` (0 failed), and `cargo clippy --workspace --all-targets -- -D warnings` all clean. refs #159
This commit is contained in:
+76
-1613
File diff suppressed because it is too large
Load Diff
@@ -225,7 +225,6 @@ pub fn render_chart_html(data: &ChartData, mode: ChartMode) -> String {
|
|||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::*;
|
use super::*;
|
||||||
use crate::sample_blueprint;
|
|
||||||
|
|
||||||
/// `aura graph`'s page is self-contained (no remote fetch), embeds the
|
/// `aura graph`'s page is self-contained (no remote fetch), embeds the
|
||||||
/// deterministic model as the viewer's data source, carries the Graphviz-WASM
|
/// deterministic model as the viewer's data source, carries the Graphviz-WASM
|
||||||
@@ -234,7 +233,13 @@ mod tests {
|
|||||||
/// against the prototype.
|
/// against the prototype.
|
||||||
#[test]
|
#[test]
|
||||||
fn render_html_is_self_contained_and_embeds_the_model() {
|
fn render_html_is_self_contained_and_embeds_the_model() {
|
||||||
let html = render_html(&sample_blueprint());
|
let env = crate::project::Env::std();
|
||||||
|
let bp = aura_engine::blueprint_from_json(
|
||||||
|
include_str!("../examples/r_sma_open.json"),
|
||||||
|
&|t| env.resolve(t),
|
||||||
|
)
|
||||||
|
.expect("the shipped r-sma example reloads into a renderable blueprint");
|
||||||
|
let html = render_html(&bp);
|
||||||
// a complete HTML document
|
// a complete HTML document
|
||||||
assert!(html.starts_with("<!doctype html>"), "not an HTML doc");
|
assert!(html.starts_with("<!doctype html>"), "not an HTML doc");
|
||||||
assert!(html.trim_end().ends_with("</html>"), "HTML doc not closed");
|
assert!(html.trim_end().ends_with("</html>"), "HTML doc not closed");
|
||||||
|
|||||||
+164
-1076
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user