From c2144c78fb09caffd4afc9a02fccd77317da1f83 Mon Sep 17 00:00:00 2001
From: tylerslaton <mtslaton1@gmail.com>
Date: Tue, 19 Mar 2024 15:41:16 -0400
Subject: [PATCH] feat: add loading spinner for search and limit number of
 reindexes per hour

Signed-off-by: tylerslaton <mtslaton1@gmail.com>
---
 src/lib/db.ts                    |  5 +++--
 src/pages/search.vue             | 13 ++++++++++++-
 src/server/api/[...slug].post.ts | 10 +++++++++-
 3 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/src/lib/db.ts b/src/lib/db.ts
index 5553af8..318d67f 100644
--- a/src/lib/db.ts
+++ b/src/lib/db.ts
@@ -4,7 +4,7 @@ import { PrismaClient, Prisma } from '@prisma/client';
 
 const prisma = new PrismaClient();
 
-export const getToolsForUrl = async (url: string): Promise<{ tools: Tool[], examples: ToolExample[] }> => {
+export const getToolsForUrl = async (url: string): Promise<{ tools: Tool[], examples: ToolExample[], lastIndexedAt: Date }> => {
     const toolEntry = await prisma.toolEntry.findFirst({
         where: {
             reference: url
@@ -15,11 +15,12 @@ export const getToolsForUrl = async (url: string): Promise<{ tools: Tool[], exam
     });
 
     if (!toolEntry) {
-        return { tools: [], examples: [] };
+        return { tools: [], examples: [], lastIndexedAt: new Date() };
     }
 
     return {
         tools: toolEntry.content as Tool[],
+        lastIndexedAt: toolEntry.lastIndexedAt,
         examples: toolEntry.examples.map((example) => ({
             name: example.name,
             url: example.url,
diff --git a/src/pages/search.vue b/src/pages/search.vue
index 32f70db..9b23c90 100644
--- a/src/pages/search.vue
+++ b/src/pages/search.vue
@@ -1,6 +1,6 @@
 <template>
     <div>
-        <div v-if="!error.status" class="m-10 md:m-24 mt-36 prose">
+        <div v-if="!error.status && !loading" class="m-10 md:m-24 mt-36 prose">
             <h1 class="text-3xl font-bold mb-8">Search Results</h1>
             <div v-for="(tools, url) in searchResults" :key="url" class="mb-8">
                 <a class="text-xl font-semibold mb-4 block" :href="`/${url}`">
@@ -10,6 +10,14 @@
                 
             </div>
         </div>
+
+        <div v-else-if="loading" class="flex items-center justify-center h-screen">
+            <div class="flex flex-col items-center justify-center h-screen">
+                <div class="animate-spin rounded-full h-16 w-16 border-t-4 border-b-6 border-gray-900"></div>
+                <p class="mt-10 text-gray-900 text-xl font-semibold">Loading search...</p>
+            </div>
+        </div>
+
         <Error
             class="flex flex-col items-center justify-center h-screen"
             v-else
@@ -26,6 +34,7 @@ import type { Tool } from '@/lib/types';
 const router = useRouter();
 const searchResults = ref({} as Record<string, Tool[]>);
 const error = ref({status: 0, message: ''});
+const loading = ref(true);
 
 const fetchData = async (q: string) => {
     // if the query is a github valid github url, we want to redirect to that tool's page
@@ -38,9 +47,11 @@ const fetchData = async (q: string) => {
     const results = await fetch(`/api/search?q=${q}`, { method: 'GET' });
     if (!results.ok) {
         error.value = { status: results.status, message: results.statusText };
+        loading.value = false;
         return;
     }
     searchResults.value = await results.json() as Record<string, Tool[]>;
+    loading.value = false;
 };
 
 onMounted(async () => fetchData(useRoute().query.q as string));
diff --git a/src/server/api/[...slug].post.ts b/src/server/api/[...slug].post.ts
index a5e9376..5b0251f 100644
--- a/src/server/api/[...slug].post.ts
+++ b/src/server/api/[...slug].post.ts
@@ -30,10 +30,17 @@ export default defineEventHandler(async (event) => {
     
     // if the tool is already indexed and force is not true, return the tool
     let entry = await db.getToolsForUrl(url);
-    if (entry.tools.length > 0 && !getQuery(event).force) {
+
+    // if the tool is already indexed and the last index time is less than 1 hour ago, return the tool
+    if (entry.tools.length > 0 && entry.lastIndexedAt && (Date.now() - Number(entry.lastIndexedAt)) < 3600000) {
+        // add headers to communicate that the response is cached and when it was last indexed
         setResponseHeader(event, "Content-Type", "application/json");
+        setResponseHeader(event, "Cached-Response", "true")
+        setResponseHeader(event, "Last-Indexed-At", entry.lastIndexedAt.toISOString())
+        setResponseStatus(event, 200)
         return entry;
     }
+    setResponseHeader(event, "Cached-Response", "false")
 
     // grab the owner, repo and subdirs from the URL if they exist
     const [owner, repo, ...subdirs] = url.replace(/^(https?:\/\/)?(www\.)?github\.com\//, "").split("/");
@@ -80,6 +87,7 @@ export default defineEventHandler(async (event) => {
 
     // upsert the tool into the database and return the tool
     setResponseHeader(event, "Content-Type", "application/json");
+    setResponseStatus(event, 201)
     return await db.upsertToolForUrl(url, 
         parsedTools, 
         await getExamples(owner, repo)