@financial-times/cp-content-pipeline-schema 2.7.0 → 2.9.0

package/src/resolvers/content-tree/references/Tweet.ts

@@ -1,3 +1,4 @@
+import { OperationalError } from '@dotcom-reliability-kit/errors'
 import { TweetResolvers } from '../../../generated'
 import isError from '../../../helpers/isError'
 
@@ -10,9 +11,13 @@ export const Tweet = {
       return tweet.html
     } catch (error) {
       if (isError(error)) {
-        context.logger.error({
+        context.logger.warn({
           event: 'RECOVERABLE_ERROR',
-          error,
+          error: new OperationalError({
+            code: 'TWEET_EMBED_ERROR',
+            message: `Failed to get HTML embed for tweet ${parent.reference.id}`,
+            cause: error,
+          }),
         })
       }
       return null

package/src/resolvers/content-tree/references/Video.ts

@@ -1,15 +1,29 @@
 import { uuidFromUrl } from '../../../helpers/metadata'
 import { VideoReferenceResolvers } from '../../../generated'
+import { OperationalError } from '@dotcom-reliability-kit/errors'
 
 export const Video = {
   // This will override the original id from the parent that is a URL.
   id: (parent) => uuidFromUrl(parent.reference.id),
 
   async title(parent, _args, context) {
-    const capiResponse = await context.dataSources.capi.getContent(
-      uuidFromUrl(parent.reference.id)
-    )
-    return capiResponse.title()
+    try {
+      const capiResponse = await context.dataSources.capi.getContent(
+        uuidFromUrl(parent.reference.id)
+      )
+
+      return capiResponse.title()
+    } catch (error) {
+      if (error instanceof Error) {
+        throw new OperationalError({
+          code: 'VIDEO_CONTENT_FETCH_ERROR',
+          message: `Error getting content object for video ${parent.reference.id}`,
+          cause: error,
+        })
+      }
+
+      throw error
+    }
   },
 
   type(parent) {

package/src/resolvers/content.ts

@@ -80,6 +80,7 @@ const resolvers = {
       }
     },
     isPinned: (parent) => parent.isPinned(),
+    authors: (parent) => parent.authors(),
   },
 
   LiveBlogPackage: {

package/src/resolvers/core.ts

@@ -2,6 +2,7 @@ import fs from 'fs'
 import path from 'path'
 import { QueryResolvers, MutationResolvers } from '../generated'
 import { CapiResponse } from '../model/CapiResponse'
+import { BaseError, HttpError } from '@dotcom-reliability-kit/errors'
 
 const packageJson = JSON.parse(
   fs.readFileSync(path.resolve(__dirname, '../../package.json'), 'utf-8')
@@ -13,7 +14,23 @@ const resolvers = {
   Query: {
     version: () => version,
     async content(_, args, context) {
-      return context.dataSources.capi.getContent(args.uuid)
+      try {
+        return await context.dataSources.capi.getContent(args.uuid)
+      } catch (error) {
+        if (
+          error instanceof BaseError &&
+          error.data.upstreamStatusCode === 404
+        ) {
+          throw new HttpError({
+            code: 'CONTENT_NOT_FOUND',
+            message: `Content ${args.uuid} not found in Content API`,
+            statusCode: 404,
+            relatesToSystems: ['up-ica'],
+          })
+        }
+
+        throw error
+      }
     },
     contentFromJSON(_, { content }, context) {
       return CapiResponse.fromJSON(content, context)

package/src/resolvers/index.ts

@@ -9,6 +9,7 @@ import { default as richText } from './richText'
 import { default as scalars } from './scalars'
 import { default as teaser } from './teaser'
 import { default as topper } from './topper'
+import { default as person } from './person'
 import { resolvers as references } from './content-tree/references'
 import { Resolvers } from '../generated'
 
@@ -25,6 +26,7 @@ const resolvers = {
   ...scalars,
   ...teaser,
   ...topper,
+  ...person,
 } satisfies Resolvers
 
 export default resolvers

package/src/resolvers/person.ts

@@ -0,0 +1,12 @@
+import { PersonResolvers } from '../generated'
+const resolvers = {
+  Person: {
+    headshot: (parent) => parent.headshot(),
+    prefLabel: (parent) => parent.prefLabel(),
+    streamPage: (parent) => parent.streamPage(),
+  },
+} satisfies {
+  Person: PersonResolvers
+}
+
+export default resolvers