Skip to content

Conversation

@SimplyLiz
Copy link
Owner

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
Copy link

github-actions bot commented Jan 29, 2026

🟢 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
Copy link

github-actions bot commented Jan 29, 2026

🔐 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
Copy link

github-actions bot commented Jan 29, 2026

CKB Analysis

Risk Files +94 -24 Modules

🎯 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:

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
📚 Stale docs · 143 broken references

Generated by CKB · Run details

@codecov
Copy link

codecov bot commented Jan 29, 2026

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.

@SimplyLiz SimplyLiz merged commit 8eae83b into develop Jan 29, 2026
22 checks passed
@SimplyLiz SimplyLiz deleted the fix/doctor-messages branch January 29, 2026 12:38
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.

2 participants