Skip to content

Split app/cli/wizard/integration_health.py into smaller validator modules #873

@VaibhavUpreti

Description

@VaibhavUpreti

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

  1. Move validator functions into smaller modules grouped by style, for example client_validators.py, http_probe_validators.py, and mcp_validators.py.
  2. Keep app/cli/wizard/integration_health.py as the stable import surface by re-exporting the same public functions.
  3. 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

Metadata

Metadata

Assignees

Labels

maintainabilityImproves long-term code maintainabilityrefactorCodebase cleanup without behavior change

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions