diff --git a/internal/lsp/cache/analysis.go b/internal/lsp/cache/analysis.go index 872f1d23e9..c51094dafe 100644 --- a/internal/lsp/cache/analysis.go +++ b/internal/lsp/cache/analysis.go @@ -207,7 +207,7 @@ func runAnalysis(ctx context.Context, snapshot *snapshot, analyzer *analysis.Ana } defer func() { if r := recover(); r != nil { - event.Log(ctx, fmt.Sprintf("analysis panicked: %s", r), tag.Package.Of(pkg.PkgPath())) + event.Log(ctx, fmt.Sprintf("analysis panicked: %s", r), tag.Package.Of(pkg.ID())) data.err = errors.Errorf("analysis %s for package %s panicked: %v", analyzer.Name, pkg.PkgPath(), r) } }() diff --git a/internal/lsp/cache/load.go b/internal/lsp/cache/load.go index cdcc3aea07..308ce553c2 100644 --- a/internal/lsp/cache/load.go +++ b/internal/lsp/cache/load.go @@ -141,7 +141,10 @@ func (s *snapshot) load(ctx context.Context, allowNetwork bool, scopes ...interf } for _, pkg := range pkgs { if !containsDir || s.view.Options().VerboseOutput { - event.Log(ctx, "go/packages.Load", tag.Snapshot.Of(s.ID()), tag.PackagePath.Of(pkg.PkgPath), tag.Files.Of(pkg.CompiledGoFiles)) + event.Log(ctx, "go/packages.Load", + tag.Snapshot.Of(s.ID()), + tag.Package.Of(pkg.ID), + tag.Files.Of(pkg.CompiledGoFiles)) } // Ignore packages with no sources, since we will never be able to // correctly invalidate that metadata. diff --git a/internal/lsp/debug/tag/tag.go b/internal/lsp/debug/tag/tag.go index 15fc6807a3..ff2f2ecd38 100644 --- a/internal/lsp/debug/tag/tag.go +++ b/internal/lsp/debug/tag/tag.go @@ -19,7 +19,7 @@ var ( File = keys.NewString("file", "") Directory = keys.New("directory", "") URI = keys.New("URI", "") - Package = keys.NewString("package", "") + Package = keys.NewString("package", "") // Package ID PackagePath = keys.NewString("package_path", "") Query = keys.New("query", "") Snapshot = keys.NewUInt64("snapshot", "")