Builds on the proposed Fields package in Labkit

Elliot Forbesrequested to merge
ef-building-on-fields-package into master

What does this MR do?

Refactors Labkit to use the centralized Fields module for field constants instead of scattered definitions across modules. Replaces Labkit::Context::CORRELATION_ID_KEY references with Labkit::Fields::CORRELATION_ID.

Why?

  • Centralizes field constant management
  • Reduces duplication
  • Establishes consistent pattern for field definitions
  • Maintains existing contracts

Changes

  • Removed LOG_KEY constant from CorrelationId module
  • Updated all references to use Labkit::Fields::CORRELATION_ID
  • Added require_relative "fields" to Context module
  • Updated corresponding tests

Type of change

  • Refactoring (non-breaking change that improves code structure)
Edited by Elliot Forbes

Merge request reports