fix(cli): don't sys.exit at import when OptionsAI is unavailable (#6)

[bradsmithmba]

Summary

src/cli/main.py called sys.exit(1) at module scope when OptionsAI failed to import:

    except ImportError as e:
        typer.echo(f"Error importing OptionsAI: {e}", err=True)
        typer.echo("Please ensure all dependencies are installed ...", err=True)
        sys.exit(1)

sys.exit raises SystemExit, so any module that merely imports src.cli.main terminates the entire process. Under pytest this aborts collection:

INTERNALERROR> File ".../tests/cli/test_commands.py", line 9, in <module>
INTERNALERROR>   from src.cli.main import app
INTERNALERROR> File ".../src/cli/main.py", line 38, in <module>
INTERNALERROR>   sys.exit(1)
INTERNALERROR> SystemExit: 1
no tests collected

Closes #6.

Fix

Capture the import error instead of exiting, and report it at command execution time inside get_options_ai() — preserving the original user-facing message and exit code 1 for real CLI usage, while making the module safe to import:

    except ImportError as e:
        OptionsAI = None
        _options_ai_import_error = e

def get_options_ai(config_path=None):
    if OptionsAI is None:
        console.print(f"[red]OptionsAI is unavailable: {_options_ai_import_error}[/red]")
        console.print("Please ensure all dependencies are installed ...")
        raise typer.Exit(1)
    ...

Testing

	clean master	this branch
pytest tests/cli/ collection	INTERNALERROR: SystemExit, 0 collected	collects normally
import src.cli.main	terminates process	imports cleanly

The cli command tests now collect (9 pass, 19 fail). The 19 failures occur because the app-level @app.callback() calls get_options_ai() on every invocation and OptionsAI cannot be constructed while the feature pipeline is broken — that is blocked by #1 and is not introduced by this change. They will pass once #1 lands.

🤖 Generated with Claude Code