feat: add interactive mode to strategy-builder CLI

[hardyjosh]

Motivation

Constructing a non-interactive raindex strategy-builder invocation requires knowing the strategy key, deployment key, which select-tokens apply, which fields the deployment asks for, and the presets for each field. That is the exact flow the webapp walks users through visually. Asking a human or agent to read the raw .rain YAML to assemble those flags is a poor UX.

Solution

Add -i / --interactive to strategy-builder. It drives a TUI wizard in the terminal's alternate-screen buffer that mirrors the webapp flow:

1. Enter owner address
2. Pick a strategy (scrollable list with name + description, arrow navigation, paging)
3. Pick a deployment
4. Select tokens for each slot (with balance lookup)
5. Fill each field (preset picker with custom-value escape hatch)
6. Optionally add deposits (with preset amounts)
7. Generate calldata, print to stdout or save to a file

Built on crossterm rather than dialoguer because dialoguer's cursor-up redraw maths miscounts multi-line wrapped items and corrupts the terminal. The alt-screen avoids all of that — the wizard owns its own screen and leaves the main terminal untouched.

All prior selections are shown as context at the top of every screen so the user can see how choices accumulate.

Checks

• made this PR as small as possible
• unit-tested any new functionality
• linked any relevant issues or PRs
• included screenshots (if this involves a front-end change)

Summary by CodeRabbit

• New Features
  • Added interactive mode to the strategy builder CLI command with a guided wizard for configuring strategies and deployments.
  • Users can now select from available strategies, configure deployment variants, choose token approvals, and input deposits through an interactive prompt.
  • Added support for exporting generated deployment calldata to stdout or a specified file.

[coderabbitai]

Walkthrough

This pull request introduces an interactive CLI mode for the strategy builder that guides users through strategy deployment configuration. New dependencies are added for terminal UI and HTTP requests. The interactive flow fetches available strategies, collects user inputs for owner, strategy selection, and builder parameters, then generates and outputs deployment transaction calldata.

Changes

Cohort / File(s)	Summary
Dependencies crates/cli/Cargo.toml	Added CLI dependencies: console, crossterm, and dialoguer for terminal UI; added workspace reqwest for HTTP requests.
Terminal UI Components crates/cli/src/commands/strategy_builder/select.rs	New module providing interactive terminal selection and text input primitives with word-wrapping, ANSI styling, and scrollable list rendering.
Interactive Mode Implementation crates/cli/src/commands/strategy_builder/interactive.rs	New 580-line interactive wizard that orchestrates strategy fetching, user input collection, and deployment calldata generation with progress tracking and terminal state management.
Strategy Builder Module crates/cli/src/commands/strategy_builder/mod.rs	Added --interactive flag and submodules; changed strategy, deployment, and owner from required to optional fields; updated execution flow to delegate to interactive mode when enabled.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant CLI as StrategyBuilder<br/>(mod.rs)
    participant Interactive as Interactive<br/>Wizard
    participant UI as Terminal UI<br/>(select.rs)
    participant Registry as Registry API
    participant Calldata as Calldata<br/>Generator

    User->>CLI: Run with --interactive flag
    CLI->>Interactive: run_interactive(registry_url)
    
    Interactive->>Registry: Fetch available strategies
    Registry-->>Interactive: Strategy list
    
    Interactive->>UI: Render strategy selection
    UI->>User: Display selectable list
    User->>UI: Select strategy + deployment variant
    UI-->>Interactive: Selected indices
    
    Interactive->>UI: Prompt owner address input
    User->>UI: Enter owner address
    UI-->>Interactive: Owner address
    
    Interactive->>UI: Prompt order tokens selection
    User->>UI: Select tokens (optional)
    UI-->>Interactive: Token choices
    
    Interactive->>UI: Prompt builder field inputs
    User->>UI: Fill required fields
    UI-->>Interactive: Field values
    
    Interactive->>UI: Prompt deposit amounts
    User->>UI: Enter deposits
    UI-->>Interactive: Deposit values
    
    Interactive->>Calldata: Generate deployment calldata<br/>(with approvals + deployment)
    Calldata-->>Interactive: Calldata lines
    
    Interactive->>UI: Prompt output mode (stdout/file)
    User->>UI: Choose output destination
    UI-->>Interactive: Output preference
    
    Interactive->>User: Write calldata to stdout or file
    User-->>Interactive: Output complete

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 64.71% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: adding an interactive mode to the strategy-builder CLI, which aligns with the core objective of the changeset.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch interactive-strategy-builder

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[hardyjosh]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• feat: template fallback operator ${expr || 'default'} #2551 : 2 dependent PRs (#2548 , #2552 )
• feat: add --tokens flag to list deployment-specific tokens #2549
• feat: add interactive mode to strategy-builder CLI #2546 👈 (View in Graphite)
• feat: add strategy-builder CLI subcommand #2544
• Consolidate JS API gui module into common crate and rename GUI to OrderBuilder #2479
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[hardyjosh]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[hardyjosh]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@crates/cli/src/commands/strategy_builder/interactive.rs`:
- Around line 165-185: Replace the catch-all `_ => Ok(calldata_lines)` in the
match on `output_choice` with an explicit `0 => Ok(calldata_lines)` arm to
self-document the intended case; also add a fallback arm (e.g., `_ =>
unreachable!("invalid output_choice")` or return a descriptive error) to catch
unexpected indices at runtime. Update the match that references `output_choice`,
`calldata_lines`, and `path` accordingly so the two intended branches are
explicit and any other values are handled explicitly.
- Around line 29-37: Wrap the terminal setup/teardown with a panic-safe guard so
raw mode and cursor/screen state are always restored if run_wizard panics: after
calling terminal::enable_raw_mode() and execute!(w,
terminal::EnterAlternateScreen, cursor::Hide), create a small RAII guard type
(or use std::panic::catch_unwind) whose Drop implementation calls execute!(w,
terminal::LeaveAlternateScreen, cursor::Show) and terminal::disable_raw_mode();
then call run_wizard(&mut w, &registry, &mut progress).await normally and let
the guard handle cleanup (or explicitly dismiss the guard) to ensure w, raw
mode, and cursor state are restored even on panic.

In `@crates/cli/src/commands/strategy_builder/select.rs`:
- Around line 284-292: The code risks underflow when computing positions like
cols as u16 - 3 and rows as u16 - 2; change these to use saturating subtraction
on the usize values before casting (e.g., let xpos = cols.saturating_sub(3) as
u16 and let ypos = rows.saturating_sub(2) as u16) and use xpos/ypos in the
execute! calls so cursor::MoveTo gets a safe u16; update both places where
cols/rows are adjusted and keep existing variables (scroll, total_header,
count_items_fitting, items) unchanged.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 70e78f15-8e44-4f16-805a-db1cd610bebd

📥 Commits

Reviewing files that changed from the base of the PR and between 72e5a29 and 9a36b8f.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (4)

• crates/cli/Cargo.toml
• crates/cli/src/commands/strategy_builder/interactive.rs
• crates/cli/src/commands/strategy_builder/mod.rs
• crates/cli/src/commands/strategy_builder/select.rs