Why This Matters
integration_health.py is now a long mixed file containing provider validators, raw HTTP probes, and client-backed checks.
Read First
app/cli/wizard/integration_health.py
tests/cli/wizard/test_integration_health.py
Scope
- Move validator functions into smaller modules grouped by style, for example
client_validators.py, http_probe_validators.py, and mcp_validators.py.
- Keep
app/cli/wizard/integration_health.py as the stable import surface by re-exporting the same public functions.
- Do not change the return strings more than necessary in the same PR.
Tests / Coverage
- split the existing test module only if it becomes easier to read
- keep import paths stable in tests while you move the implementations
- add at least one test that imports through the old public module path
Acceptance Criteria
- the file is broken into smaller modules
- public imports from
app.cli.wizard.integration_health still work
- existing validator behavior stays the same
Why This Matters
integration_health.pyis now a long mixed file containing provider validators, raw HTTP probes, and client-backed checks.Read First
app/cli/wizard/integration_health.pytests/cli/wizard/test_integration_health.pyScope
client_validators.py,http_probe_validators.py, andmcp_validators.py.app/cli/wizard/integration_health.pyas the stable import surface by re-exporting the same public functions.Tests / Coverage
Acceptance Criteria
app.cli.wizard.integration_healthstill work