Skip to content
Snippets Groups Projects

codeintel: Add support for diagnostics

Merged Administrator requested to merge lsif-diagnostics into master

Created by: efritz

This adds resolvers for LSIF-powered diagnostic results and closes https://github.com/sourcegraph/sourcegraph/issues/10838.

This should be good enough to start code insights, but there are some efficiency notes I've left in the implementation for things we can do to improve the situation once we hit a large enough sample that it matters.

I also plan on adding an after parameter so that we can do legitimate paging instead of just expanding the first parameter 2x on request.

Merge request reports

Merged by avatar (Jul 25, 2025 2:14pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: codecov[bot]

    Codecov Report

    Merging #11233 into master will not change coverage. The diff coverage is n/a.

    @@           Coverage Diff           @@
    ##           master   #11233   +/-   ##
    =======================================
      Coverage   46.78%   46.78%           
    =======================================
      Files        1392     1392           
      Lines       78709    78709           
      Branches     6762     6762           
    =======================================
      Hits        36825    36825           
      Misses      38373    38373           
      Partials     3511     3511           
    Flag Coverage Δ
    #go 51.07% <0.00%> (ø)
    #storybook 6.92% <0.00%> (ø)
    #typescript 35.53% <0.00%> (ø)
    #unit 46.63% <0.00%> (ø)
Please register or sign in to reply
Loading