Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Bad intelligence on new rev in GitHub PR file lists #383

Open
efritz opened this issue Apr 4, 2020 · 1 comment
Open

Bad intelligence on new rev in GitHub PR file lists #383

efritz opened this issue Apr 4, 2020 · 1 comment

Comments

@efritz
Copy link
Contributor

efritz commented Apr 4, 2020

We're recently discovered some odd behavior when viewing results from go-langserver via the Sourcegraph browser extension. When on a GitHub PR file list in which a Go file has been added in the new rev, the Go langserver fails to tokenize any Go file in the new revision and no code intelligence appears. This does not seem to affect the original revision.

The failing line is a build context import which always returns a scanner error (expected 'package' but got EOF). Additional debugging shows that the build context attempts to open and return the content of a zero-length file (the file in the new rev). When looking at the temp directories, the zip archives appear to be correct.

See https://github.com/sourcegraph/sourcegraph/issues/9556 for some additional context.

@efritz
Copy link
Contributor Author

efritz commented Apr 4, 2020

Update: this also appears to happen on diff views on Sourcegraph.com, which also points to this being a langserver issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant