Skip to content

fix: differentiate telemetry between refs/defs, log codeintel actions…

Warren Gifford requested to merge refs-vs-defs into master

Created by: dadlerj

… on hovers, only log telemetry on codeintel from integrations

Fixes three issues:

  • Sourcegraph can now differentiate between refs and other types of codeintel (so the activation step is only "completed" when a user actually does a find-refs)
  • Contributed actions executed in integrations (e.g. browser extensions) now ONLY log a user action if they are a code intel action (i.e., have an _eventName of 'hover', 'findReferences', 'goToDefinition', or 'goToDefinition.preloaded'). There's a TODO here of logging other event types (need to rethink this API, and whether servers should even log other types of actions).
  • Hovers now log user codeintel actions. This means they are included in the code intelligence metrics on the site-admin > Usage statistics page.

Addresses part of https://github.com/sourcegraph/sourcegraph/issues/2024

Merge request reports