@tmdb-graphql-api/resolvers 0.1.2-unstable.5 → 0.1.2-unstable.6

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tmdb-graphql-api/resolvers",
   "description": "The TMDB GraphQL resolvers module.",
-  "version": "0.1.2-unstable.5",
+  "version": "0.1.2-unstable.6",
   "author": "Dylan Aubrey",
   "license": "MIT",
   "homepage": "https://github.com/badbatch/themoviedb-graphql-api",

package/src/helpers/buildObjArgsResolver.ts

@@ -26,16 +26,33 @@ export const buildObjArgsResolver =
     };
 
     const childLogger = logger.child(newCtx.data);
-    childLogger.info(`Resolving ${fieldPath}`);
+    const loggingEnabled = !obj || fieldName === 'id';
+    const id = obj && 'id' in obj && obj.id ? obj.id : 'id' in args ? args.id : undefined;
+
+    const enrichLogMessage = (message: string) => {
+      const enrichedMessage = `${message} ${fieldPath}`;
+      return id ? `${enrichedMessage} ${String(id)}` : enrichedMessage;
+    };
+
+    if (loggingEnabled) {
+      childLogger.info(enrichLogMessage('Resolving'));
+    }
+
     const { data, errors, headers, source = 'api' } = await callback(obj, args, newCtx, info);
 
     if (errors?.length) {
-      const message = `Failed to resolve ${fieldPath}`;
-      childLogger.error(message, { errors });
+      const message = enrichLogMessage('Failed to resolve');
+
+      if (loggingEnabled) {
+        childLogger.error(message, { errors });
+      }
+
       throw new GraphQLError(message, { nodes: fieldNodes, originalError: errors[0] });
     }
 
-    childLogger.info(`Successfully resolved ${fieldPath} from ${source}`);
+    if (loggingEnabled) {
+      childLogger.info(enrichLogMessage(`Successfully resolved from ${source}`));
+    }
 
     if (headers) {
       setCacheMetadata(fieldPath, headers);