Register winston instrumentation and winston seperate module

[elebiodaslingshot]

I am trying to write up a logger and tracer module for our application. I have tracing working, but I can not for the life of me get the winston logger instrumentation working.

registerInstrumentations({
    instrumentations: [
    new MongoDBInstrumentation(),
    new HttpInstrumentation(),
    new WinstonInstrumentation({
        logHook: (span, record) => {
            console.log("LOGHOOK SPAN ADD")
            if (span) {
              console.log("SPAN IN HERE")
              console.log(span)
              const context = span.spanContext();
              const traceIdEnd = context.traceId.slice(context.traceId.length / 2);
              record["dd.trace_id"] = toNumberString(fromString(traceIdEnd, 16), undefined);
              record["dd.span_id"] = toNumberString(fromString(context.spanId, 16), undefined);
            }
        }
    })

I call this from inside a private class on module load, which I do before winston require, and then in my logger module I do the following

require('./tracing')
import winston from "winston";

const logger = winston.createLogger({
  level: process.env.LOG_LEVEL || 'info',
  format: winston.format.combine(
    loggingFormat(),
    winston.format.timestamp({  
      format: 'YYYY-MM-DD hh:mm:ss.SSS A' // 2022-01-25 03:23:10.350 PM 
    }),
    winston.format.json()),
  transports: [new winston.transports.Console()],
});

But not logs are being injected with the trace and span id's. Is this not how the instrumentation should be initialized? Do I have to consolidate to a single module?