contentIndex: optimize non-RSS feeds

jackyzha0 · Nov 13, 2024 · bda8569 · bda8569
1 parent d3c386c
commit bda8569
Showing 1 changed file with 15 additions and 17 deletions.
diff --git a/quartz/plugins/emitters/contentIndex.ts b/quartz/plugins/emitters/contentIndex.ts
@@ -24,6 +24,7 @@ import {
   Entry,
   defaultFeed,
   compareTreeNodes,
+  isContentDetails,
 } from "./contentIndex/indexTree"
 import DepGraph from "../../depgraph"
 import chalk from "chalk"
@@ -39,7 +40,7 @@ export type ContentDetails = {
   date?: Date
   description?: string
   slug?: FullSlug
-  noRSS: boolean
+  noRSS?: boolean
 }
 
 interface Options {
@@ -70,9 +71,7 @@ function generateSiteMap(cfg: GlobalConfiguration, idx: ContentIndex): string {
     <loc>https://${joinSegments(base, encodeURI(simplifySlug(content.slug!)))}</loc>
     ${content.date && `<lastmod>${content.date.toISOString()}</lastmod>`}
   </url>`
-  let urls = (idx.spread().filter((e) => typeof e != "string") as ContentDetails[])
-    .map((e) => createURLEntry(e))
-    .join("")
+  let urls = (idx.spread() as ContentDetails[]).map((e) => createURLEntry(e)).join("")
   return `<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" xmlns:xhtml="http://www.w3.org/1999/xhtml">${urls}
 </urlset>`
 }
@@ -261,19 +260,18 @@ export const ContentIndex: QuartzEmitterPlugin<Partial<Options>> = (opts) => {
       // Generate ContentIndex
       const fp = joinSegments("static", "contentIndex") as FullSlug
       const simplifiedIndex = Object.fromEntries(
-        (indexTree.spread().filter((e) => typeof e != "string") as ContentDetails[]).map(
-          (content) => {
-            // remove description and from content index as nothing downstream
-            // actually uses it. we only keep it in the index as we need it
-            // for the RSS feed
-            delete content.description
-            delete content.date
-
-            var slug = content.slug
-            delete content.slug
-            return [slug, content]
-          },
-        ),
+        (indexTree.spread() as ContentDetails[]).map((content) => {
+          // remove description and from content index as nothing downstream
+          // actually uses it. we only keep it in the index as we need it
+          // for the RSS feed
+          delete content.description
+          delete content.date
+          delete content.noRSS
+
+          var slug = content.slug
+          delete content.slug
+          return [slug, content]
+        }),
       )
       emitted.push(
         write({