@playpilot/tpi 8.33.2 → 8.34.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.33.2",
+	"version": "8.34.0",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/lib/api/externalPages.ts

@@ -116,21 +116,34 @@ export async function pollLinkInjections(
 /**
  * Re-run fetchLinkInjections but with run_ai enabled. This makes the call to update AI results.
  * This happens when the content_modified_time has changed since the last response, likely meaning the article was edited.
- * Returns the response as is if ai is not enabled or content_modified_time from response matches current time.
+ * Returns the response as if ai is not enabled or content_modified_time from response matches current time.
+ *
+ * Will also re-run the AI if the previous attempt failed on recent articles with a cut off for how often the AI will re-run in a given time period.
  */
 export async function runAiBasedOnResponse(pageText: string, response: LinkInjectionResponse): Promise<LinkInjectionResponse> {
   if (!pageText) return response
   if (!response.ai_enabled) return response
   if (response.ai_running) return response
 
+  const rerun = async (): Promise<LinkInjectionResponse> => await fetchLinkInjections({ params: { run_ai: true }, method: 'POST' })
+
   const currentModifiedTime = getPageMetaData().content_modified_time
   const responseModifiedTime = response.content_modified_time ? getDatetime(response.content_modified_time) : null
   const isCurrentTimeEqualToResponseTime = currentModifiedTime && responseModifiedTime === currentModifiedTime
 
+  if (currentModifiedTime && response.ai_last_run && response.ai_progress_message === 'Failed') {
+    const tenMinutesAgo = new Date(Date.now() - 600_000)
+    const isArticleModifiedInLastDay = new Date().getTime() - new Date(currentModifiedTime).getTime() < 86_400_000
+
+    if (isArticleModifiedInLastDay && new Date(response.ai_last_run) < tenMinutesAgo) {
+      return await rerun()
+    }
+  }
+
   if (response.ai_last_run && isCurrentTimeEqualToResponseTime) return response
   if (new Date(currentModifiedTime || 0) <= new Date(responseModifiedTime || 0)) return response
 
-  return await fetchLinkInjections({ params: { run_ai: true }, method: 'POST' })
+  return await rerun()
 }
 
 export async function saveLinkInjections(linkInjections: LinkInjection[]): Promise<LinkInjection[]> {

package/src/tests/lib/api/externalPages.test.js

@@ -379,6 +379,38 @@ describe('$lib/api/externalPages', () => {
       await runAiBasedOnResponse('Some text', response)
       expect(api).toHaveBeenCalled()
     })
+
+    it('Should call api if ai_enabled is true, response content_modified_time time is equal to given time but response failed', async () => {
+      const now = new Date()
+      document.body.innerHTML = `<meta content="${now}" property="article:modified_time">`
+      const response = { ai_enabled: true, content_modified_time: now, ai_progress_message: 'Failed', ai_last_run: '2020-03-20T00:00:00Z' }
+
+      // @ts-ignore
+      await runAiBasedOnResponse('Some text', response)
+      expect(api).toHaveBeenCalled()
+    })
+
+    it('Should not call api if ai_enabled is true, response content_modified_time time is equal to given time, response failed, but ai was run in the last 10 minutes', async () => {
+      const now = new Date()
+      const fiveMinutesAgo = new Date(now.getTime() - 300_000)
+      document.body.innerHTML = `<meta content="${now}" property="article:modified_time">`
+      const response = { ai_enabled: true, content_modified_time: now, ai_progress_message: 'Failed', ai_last_run: fiveMinutesAgo }
+
+      // @ts-ignore
+      await runAiBasedOnResponse('Some text', response)
+      expect(api).not.toHaveBeenCalled()
+    })
+
+    it('Should not call api if ai_enabled is true, response content_modified_time time is equal to given time, response failed, but article modified time is past 24 hours ago', async () => {
+      const now = new Date()
+      const moreThanADayAgo = new Date(now.getTime() - 90_000_000)
+      document.body.innerHTML = `<meta content="${moreThanADayAgo}" property="article:modified_time">`
+      const response = { ai_enabled: true, content_modified_time: moreThanADayAgo, ai_progress_message: 'Failed', ai_last_run: '2020-03-20T00:00:00Z' }
+
+      // @ts-ignore
+      await runAiBasedOnResponse('Some text', response)
+      expect(api).not.toHaveBeenCalled()
+    })
   })
 
   describe('saveLinkInjections', () => {