Strangler Stage 1: tame the worst outliers

Labels: good first issue, code-quality, refactor

Stage 1 -- tame the worst outliers

Tighten Ruff thresholds from baseline to Stage 1 targets. This stage focuses on the small number of extreme outliers that gate every metric.

Threshold changes

Key files to refactor

mcp_integrator.py::install (owns 3 metrics simultaneously):

• 269 statements (target: 200)
• 108 branches (target: 60)
• 97 complexity (target: 50)
• Extract phases into private helpers (_validate, _resolve, _deploy, _hooks)

install/validation.py (complexity 65, target 50)

commands/install.py (18 args on two functions, target 15):

• Reduce via parameter objects or context dataclasses

marketplace/publisher.py (16 returns, target 12):

• Early-return consolidation

File length (3 files > 1400 lines):

• github_downloader.py (2316), commands/install.py (1680), skill_integrator.py (1365)

How to verify

# Update thresholds in pyproject.toml, then:
uv run ruff check src/ tests/

# Update MAX_LINES in .github/workflows/ci.yml, then check:
find src/ -name '*.py' | xargs wc -l | awk '$1 > 1400'

References

• Parent: Strangler-fig: progressively tighten Ruff complexity thresholds #1076
• Supersedes refactor: Stage 1 strangler-fig -- extract mcp_integrator.py::install #1004