fix: address cr -- import entire module rather than only used exports

Signed-off-by: Naseem <[email protected]>

Author: Naseem

plugins/node/opentelemetry-plugin-redis/src/redis.ts

@@ -15,7 +15,7 @@
  */
 
 import { BasePlugin } from '@opentelemetry/core';
-import type * as redis from 'redis';
+import type * as redisTypes from 'redis';
 import * as shimmer from 'shimmer';
 import {
   getTracedCreateClient,
@@ -24,7 +24,7 @@ import {
 } from './utils';
 import { VERSION } from './version';
 
-export class RedisPlugin extends BasePlugin<typeof redis> {
+export class RedisPlugin extends BasePlugin<typeof redisTypes> {
   static readonly COMPONENT = 'redis';
   readonly supportedVersions = ['^2.6.0']; // equivalent to >= 2.6.0 <3
 

plugins/node/opentelemetry-plugin-redis/src/types.ts

@@ -14,15 +14,15 @@
  * limitations under the License.
  */
 
-import type { Callback } from 'redis';
+import type * as redisTypes from 'redis';
 
 // exported from
 // https://github.com/NodeRedis/node_redis/blob/master/lib/command.js
 export interface RedisCommand {
   command: string;
   args: string[];
   buffer_args: boolean;
-  callback: Callback<unknown>;
+  callback: redisTypes.Callback<unknown>;
   call_on_write: boolean;
 }
 

plugins/node/opentelemetry-plugin-redis/src/utils.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import type { RedisClient } from 'redis';
+import type * as redisTypes from 'redis';
 import { Tracer, SpanKind, Span, CanonicalCode } from '@opentelemetry/api';
 import { RedisCommand, RedisPluginClientTypes } from './types';
 import { EventEmitter } from 'events';
@@ -37,8 +37,8 @@ const endSpan = (span: Span, err?: Error | null) => {
 };
 
 export const getTracedCreateClient = (tracer: Tracer, original: Function) => {
-  return function createClientTrace(this: RedisClient) {
-    const client: RedisClient = original.apply(this, arguments);
+  return function createClientTrace(this: redisTypes.RedisClient) {
+    const client: redisTypes.RedisClient = original.apply(this, arguments);
     return tracer.bind(client);
   };
 };
@@ -47,7 +47,7 @@ export const getTracedCreateStreamTrace = (
   tracer: Tracer,
   original: Function
 ) => {
-  return function create_stream_trace(this: RedisClient) {
+  return function create_stream_trace(this: redisTypes.RedisClient) {
     if (!this.stream) {
       Object.defineProperty(this, 'stream', {
         get() {