Skip to content

feat: add doctor command with 3S-* structured diagnostics - #2

Open
supportersimulator wants to merge 14 commits into
mainfrom
feature/doctor-command
Open

feat: add doctor command with 3S-* structured diagnostics#2
supportersimulator wants to merge 14 commits into
mainfrom
feature/doctor-command

Conversation

@supportersimulator

Copy link
Copy Markdown
Owner

Summary

  • Adds 3surgeons doctor CLI command with 16 structured diagnostic codes (3S-PY-OLD, 3S-MCP-MISS, etc.)
  • DiagnosticCode enum + DiagnosticResult dataclass + 4 check functions + run_all_checks() aggregator
  • MCP launcher (bin/3surgeons-mcp) now emits structured JSON errors and auto-bootstraps venv when no runtime found
  • CI extended to matrix [3.10, 3.12] with doctor contract validation and launcher smoke tests
  • Host setup docs with platform-specific instructions and 3S-* error code troubleshooting table
  • Diagnostics wired into existing setup-check command output

Test Plan

  • 663 tests passing locally (0 failures)
  • test_diagnostics.py — 19 tests covering all codes, check functions, aggregator
  • test_cli.py — 5 doctor command tests (pass/fail/json/exit-code) + setup-check integration
  • test_launcher.sh — 3 launcher smoke tests (error output, executable, bootstrap)
  • CI matrix validates on Python 3.10 and 3.12

🤖 Generated with Claude Code

supportersimulator and others added 14 commits March 9, 2026 16:13
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…t, fix starlette

- Matrix: [3.11, 3.12] → [3.10, 3.12] with fail-fast: false
- Add 3.9 expected-fail job (verifies 3S-PY-OLD doctor code)
- Add doctor contract check (validates JSON schema in CI)
- Add launcher smoke test step
- Fix starlette CI failure: install .[test,http] instead of .[test]

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace icon.png (128x128) and icon.svg with higher-quality versions
- Add assets/logo-256.png for README and docs
- Add assets/social-preview.png (1280x640) for GitHub social preview
- Embed logo in README header

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant