Skip to content

feat: add lsp version and gitlab version to internal events

Joey Khabierequested to merge
jkhabie_add_fields_to_internal_events into main

What does this merge request do and why?

This change enhances the internal events tracking system to capture version information from two sources: the GitLab instance and the language server (a code editing tool).

The interceptor now extracts the GitLab version and language server version from the request context and includes them in event data - the GitLab version goes into a dedicated field, while the language server version is stored in an "extra" data section. The order of interceptors was also adjusted to ensure version information is available when needed.

Comprehensive tests were added to verify the system correctly handles various scenarios: when versions are present, missing, or invalid. This allows the service to better track which versions of tools users are running, which is valuable for debugging, analytics, and understanding feature adoption across different software versions.

issue:

related #1571

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request checklist

  • Tests added for new functionality. If not, please raise an issue to follow up.
  • Documentation added/updated, if needed.
  • If this change requires executor implementation: verified that issues/MRs exist for both Go executor and Node executor or confirmed that changes are backward-compatible and don't break existing executor functionality.
Edited by Joey Khabie

Merge request reports