🧑‍💻 @muddlebee has entered the contributor hall of fame. Merged. Done. Shipped. Go touch grass (then come back with another PR). 🌱

👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.

Greptile Summary

• Removes the monkeypatch.setattr(flow, "build_demo_action_response", ...) call from test_run_wizard_claude_code_skips_api_key_and_runs_cli_onboarding because build_demo_action_response was removed from flow and patching a non-existent attribute raises AttributeError in CI.
• build_demo_action_response still exists in app/cli/wizard/validation.py; the removal is scoped only to the flow module-level attribute, so no other tests are affected.
• No logic or assertions were changed; the test's coverage and intent remain intact.

Confidence Score: 5/5

Safe to merge — single-line test cleanup with no logic changes.

The change removes exactly one obsolete monkeypatch that caused an AttributeError; all 26 tests pass, no assertions were altered, and there are no new risks introduced.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Test as test_run_wizard_claude_code_...
    participant flow as flow module
    participant store as wizard_store

    Test->>flow: monkeypatch select_prompt, get_store_path, probe_local_target
    Test->>flow: monkeypatch _run_cli_llm_onboarding, save_local_config, sync_provider_env, save_llm_api_key
    Note over Test,flow: build_demo_action_response patch removed (no longer on flow)
    Test->>flow: run_wizard()
    flow->>flow: _render_saved_summary → _render_next_steps
    flow->>store: save_local_config(...)
    flow-->>Test: exit_code = 0
    Test->>Test: assert cli_onboarding_providers == ["claude-code"]
    Test->>Test: assert saved_llm_keys == []

_{Reviews (1): Last reviewed commit: "test(wizard): drop stale build_demo stub..." | Re-trigger Greptile}