fix: improve doctor command error messages by SimplyLiz · Pull Request #124 · SimplyLiz/CodeMCP

SimplyLiz · 2026-01-29T11:59:10Z

Summary

SCIP "not found" message now shows which indexer to use (e.g., "uses scip-go")
Add ckb index as primary suggested fix for missing SCIP index
Filter LSP checks to only show servers relevant to detected project languages
Map JavaScript projects to typescript LSP server (they share typescript-language-server)

Before

⚠ scip: SCIP index not found
⚠ lsp: LSP commands not found: dart (dart), go (gopls), python (pylsp), typescript (typescript-language-server)

After

Go project:

✓ lsp: LSP ready: go (starts on-demand)

TypeScript project:

⚠ scip: SCIP index not found — run 'ckb index' (uses scip-typescript)
  → ckb index
  → npm install -g @sourcegraph/scip-typescript && scip-typescript index
⚠ lsp: LSP not installed for typescript — install typescript-language-server: npm install -g typescript-language-server typescript

Test plan

go build ./cmd/ckb passes
go test ./internal/query/... -run Doctor passes
Verified output on Go project (CKB/src) — shows only go LSP
Verified output on TypeScript project (website) — shows scip-typescript and typescript LSP

🤖 Generated with Claude Code

- SCIP "not found" message now shows which indexer to use (e.g., "uses scip-go") - Add 'ckb index' as primary suggested fix for missing SCIP index - Filter LSP checks to only show servers relevant to detected project languages - Map JavaScript projects to typescript LSP server (they share typescript-language-server) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

github-actions · 2026-01-29T12:01:41Z

🟢 Change Impact Analysis

Metric	Value
Risk Level	LOW 🟢
Files Changed	2
Symbols Changed	66
Directly Affected	1
Transitively Affected	49

Blast Radius: 0 modules, 1 files, 50 unique callers

📝 Changed Symbols (66)

Symbol	File	Type	Confidence
`Engine#filterRelevantLspServers`	`internal/query/doctor.go`	modified	100%
`Engine#scipNotFoundMessage`	`internal/query/doctor.go`	modified	100%
`config`	`internal/query/doctor.go`	modified	100%
`delivery`	`internal/query/doctor.go`	modified	100%
`id`	`internal/query/doctor.go`	modified	100%
`id`	`internal/query/doctor.go`	modified	100%
`id1`	`internal/query/doctor.go`	modified	100%
`id2`	`internal/query/doctor.go`	modified	100%
`input`	`internal/query/doctor.go`	modified	100%
`langToLspServer`	`internal/query/doctor.go`	modified	100%
`now`	`internal/query/doctor.go`	modified	100%
`now`	`internal/query/doctor.go`	modified	100%
`t`	`internal/query/doctor.go`	modified	100%
`t`	`internal/query/doctor.go`	modified	100%
`t`	`internal/query/doctor.go`	modified	100%
…	+51 more

🎯 Affected Downstream (20)

Symbol	Module	Distance	Kind
`unknown`	``	1	type-dependency
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/collectDiagnostics().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/getChangeImpactStatus().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/runDoctor().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/runIndex().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/runIndexWatchLoop().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/cmd/ckb`/runStatus().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/api`/Server#handleDoctorFix().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/api`/Server#handleLanguageQuality().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/daemon`/RefreshManager#RunFullReindex().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/daemon`/RefreshManager#RunIncrementalRefreshWithTrigger().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/project`/TestDetectLanguage().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/project`/TestDetectLanguage_DepthLimit().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/project`/TestDetectLanguage_GlobPatterns().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/project`/TestDetectLanguage_SkipsIgnoredDirs().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/project`/TestProjectQualityReport().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/query`/Engine#Doctor().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/query`/Engine#checkLsp().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/query`/Engine#checkScip().`	``	2	transitive-caller
`scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc` github.com/SimplyLiz/CodeMCP/internal/query`/Engine#scipNotFoundMessage().`	``	2	transitive-caller

Recommendations

⚠️ test: Significant transitive impact (49 symbols). Run comprehensive test suite.
- Action: Run full test suite before merging

⚠️ Index is 0 commit(s) behind HEAD. Results may be incomplete.

_{Generated by CKB}

Addresses GO-2026-4339 (net/url) and GO-2026-4340 (crypto/tls) standard library vulnerabilities. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

github-actions · 2026-01-29T12:02:45Z

🔐 Security Audit Results

⚠️ Security gate passed with warnings - 7 issue(s) found (review recommended)

Category	Findings
🔑 Secrets	✅ 0
🛡️ SAST	✅ 0
📦 Dependencies	⚠️ 7
📜 Licenses	✅ 0

📦 Dependency Vulnerabilities

Found 7 vulnerability(ies) across 2 scanner(s)

Details

Trivy (4 findings)

CVE-2026-22036 (MEDIUM): undici - undici: Undici: Denial of Service via excessive de...
CVE-2025-54410 (LOW): github.com/docker/docker - github.com/moby/moby: Moby's Firewalld reload remo...
GHSA-vrw8-fxc6-2r93 (MEDIUM): github.com/go-chi/chi/v5 - chi Allows Host Header Injection which Leads to Op...
CVE-2025-47908 (MEDIUM): github.com/rs/cors - github.com/rs/cors: Denial of service via maliciou...

OSV-Scanner (3 findings)

github.com/docker/docker: 2 vulnerabilities
github.com/go-chi/chi/v5: 1 vulnerabilities
github.com/rs/cors: 2 vulnerabilities

_{Generated by CKB Security Audit | View Details | Security Tab}

github-actions · 2026-01-29T12:03:00Z

CKB Analysis

🎯 66 changed → 49 affected · 🔥 1 hotspot · 📊 1 complex · 📚 143 stale

Risk factors: Touches 1 hotspot(s)

Metric	Value
Impact Analysis	66 symbols → 49 affected	🟢
Doc Coverage	9.090909090909092%	⚠️
Complexity	1 violations	⚠️
Coupling	0 gaps	✅
Blast Radius	0 modules, 1 files	✅
Index	indexed (8s)	🆕

🎯 Change Impact Analysis · 🟢 LOW · 66 changed → 20 affected

Metric	Value
Symbols Changed	66
Directly Affected	1
Transitively Affected	49
Modules in Blast Radius	0
Files in Blast Radius	1

Symbols changed in this PR:

Engine#filterRelevantLspServers [modified] (100%) — internal/query/doctor.go
Engine#scipNotFoundMessage [modified] (100%) — internal/query/doctor.go
config [modified] (100%) — internal/query/doctor.go
delivery [modified] (100%) — internal/query/doctor.go
id [modified] (100%) — internal/query/doctor.go
id [modified] (100%) — internal/query/doctor.go
id1 [modified] (100%) — internal/query/doctor.go
id2 [modified] (100%) — internal/query/doctor.go
input [modified] (100%) — internal/query/doctor.go
langToLspServer [modified] (100%) — internal/query/doctor.go
… and 56 more

Downstream symbols affected:
Direct callers (1):

unknown
Transitive callers (19):
scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc github.com/SimplyLiz/CodeMCP/cmd/ckb/collectDiagnostics(). (depth 2)
scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc github.com/SimplyLiz/CodeMCP/cmd/ckb/getChangeImpactStatus(). (depth 2)
scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc github.com/SimplyLiz/CodeMCP/cmd/ckb/runDoctor(). (depth 2)
scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc github.com/SimplyLiz/CodeMCP/cmd/ckb/runIndex(). (depth 2)
scip-go gomod github.com/SimplyLiz/CodeMCP be3ca2ea09bc github.com/SimplyLiz/CodeMCP/cmd/ckb/runIndexWatchLoop(). (depth 2)
… and 14 more transitive callers

Recommendations:

⚠️ Significant transitive impact (49 symbols). Run comprehensive test suite.
- Action: Run full test suite before merging

⚠️ Index is undefined commit(s) behind HEAD. Results may be incomplete. Run ckb index to refresh.

🔥 Hotspots · 1 volatile files

File	Churn Score
`internal/query/doctor.go`	10.25

📊 Complexity · 1 violations

File	Cyclomatic	Cognitive
`internal/query/doctor.go`	13	⚠️ 29

💡 Quick wins · 10 suggestions

🟡 Add tests → internal/query/engine_test.go
🟡 Add tests → internal/incremental/store_test.go
🟡 Add tests → internal/api/index_processor.go
🟡 Add tests → internal/backends/git/diff.go
🟡 Add tests → internal/query/doctor.go
🟢 Assign backup owner → internal/query/engine_test.go
🟢 Assign backup owner → internal/incremental/store_test.go
🟢 Assign backup owner → internal/api/index_processor.go

📚 Stale docs · 143 broken references

docs/CONTRIBUTING.md:108 → errors.New
docs/CONTRIBUTING.md:108 → errors.SYMBOL_NOT_FOUND
docs/CONTRIBUTING.md:111 → errors.Wrap
docs/backlog/index-refresh-improvements.md:36 → watcher.Add
docs/backlog/index-refresh-improvements.md:36 → filepath.Join
docs/backlog/index-refresh-improvements.md:97 → time.Second
docs/featureplans/change-impact-analysis-checklist.md:208 → coverage.out
docs/featureplans/change-impact-analysis-checklist.md:214 → lcov.info
docs/featureplans/change-impact-analysis-checklist.md:309 → coverage.out
docs/featureplans/change-impact-analysis.md:448 → scip.SCIPIndex
docs/featureplans/change-impact-analysis.md:478 → context.Context
docs/featureplans/change-impact-analysis.md:669 → coverage.out
docs/ideas.md:16 → provenance.backends
docs/ideas.md:21 → queryPolicy.coalesceWindowMs
docs/ideas.md:302 → confidence.factors

_{Generated by CKB · Run details}

codecov · 2026-01-29T12:08:11Z

Codecov Report

❌ Patch coverage is 66.66667% with 15 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/query/doctor.go	66.6%	15 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           develop    #124   +/-   ##
=======================================
  Coverage     45.1%   45.1%           
=======================================
  Files          350     350           
  Lines        59747   59773   +26     
=======================================
+ Hits         26957   26974   +17     
- Misses       30963   30973   +10     
+ Partials      1827    1826    -1

Flag	Coverage Δ
unit	`45.1% <66.6%> (+<0.1%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

fix: bump Go to 1.24.12 for security fixes

28a4413

Addresses GO-2026-4339 (net/url) and GO-2026-4340 (crypto/tls) standard library vulnerabilities. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

SimplyLiz merged commit 8eae83b into develop Jan 29, 2026
22 checks passed

SimplyLiz deleted the fix/doctor-messages branch January 29, 2026 12:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: improve doctor command error messages#124

fix: improve doctor command error messages#124
SimplyLiz merged 2 commits intodevelopfrom
fix/doctor-messages

SimplyLiz commented Jan 29, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

codecov bot commented Jan 29, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

SimplyLiz commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Before

After

Test plan

Uh oh!

github-actions bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🟢 Change Impact Analysis

Recommendations

Uh oh!

github-actions bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔐 Security Audit Results

📦 Dependency Vulnerabilities

Uh oh!

github-actions bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CKB Analysis

Uh oh!

codecov bot commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

SimplyLiz commented Jan 29, 2026 •

edited

Loading

github-actions bot commented Jan 29, 2026 •

edited

Loading

github-actions bot commented Jan 29, 2026 •

edited

Loading

github-actions bot commented Jan 29, 2026 •

edited

Loading

codecov bot commented Jan 29, 2026 •

edited

Loading