add is promise utility

Author: mikeldking

js/packages/openinference-core/src/decorators/index.ts

@@ -66,19 +66,21 @@ export function chain<
 >(options: TraceDecoratorOptions = {}) {
   const {
     tracer: _tracer,
-    name = "chain",
+    name: optionsName,
     processInput: _processInput,
     processOutput: _processOutput,
   } = options;
   const tracer: OITracer = _tracer ? wrapTracer(_tracer) : getTracer();
   const processInput = _processInput ?? defaultProcessInput;
   const processOutput = _processOutput ?? defaultProcessOutput;
   // TODO: infer the name from the target
-  return function (originalMethod: Fn, _ctx: ClassMethodDecoratorContext) {
+  return function (originalMethod: Fn, ctx: ClassMethodDecoratorContext) {
+    const methodName = String(ctx.name);
+    const spanName = optionsName || methodName || "chain";
     const wrappedFn = function (this: unknown, ...args: unknown[]) {
       const input = processInput(args);
       return tracer.startActiveSpan(
-        name,
+        spanName,
         {
           attributes: {
             [OPENINFERENCE_SPAN_KIND]: OpenInferenceSpanKind.CHAIN,

js/packages/openinference-core/src/utils/typeUtils.ts

@@ -35,6 +35,20 @@ export function isObjectWithStringKeys(
   );
 }
 
+/**
+ * Type guard for if a function is a Promise
+ * @param value
+ * @returns true if it is a Promise
+ */
+export function isPromise<T = unknown>(value: unknown): value is Promise<T> {
+  return (
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    !!value &&
+    (value as any)?.then === "function" &&
+    (value as any)?.catch === "function"
+  );
+}
+
 /**
  * Type guard to determine whether or not a value is an object with string keys and attribute values.
  * @param value
@@ -51,7 +65,7 @@ export function isAttributes(value: unknown): value is Attributes {
 
 /**
  * A type check function to ensure that a switch or set of conditionals is exhaustive.
- * Typscript will throw an error if the switch or conditionals are not exhaustive.
+ * Typescript will throw an error if the switch or conditionals are not exhaustive.
  * @example
  *  ```typescript
  * type MyType = "a" | "b";