Merge pull request #118 from timlrx/v2

Fix kbar fetch to load relative to base url and add back starting actions

Author: timlrx

.changeset/pre.json

@@ -21,6 +21,7 @@
     "tricky-carpets-fly",
     "unlucky-keys-join",
     "weak-swans-heal",
+    "wicked-maps-rest",
     "yellow-ligers-sort"
   ]
 }

.changeset/wicked-maps-rest.md

@@ -0,0 +1,5 @@
+---
+'pliny': patch
+---
+
+Fix kbar fetch to load relative to base url and add back starting actions

packages/pliny/CHANGELOG.md

@@ -1,5 +1,11 @@
 # pliny
 
+## 0.1.0-beta.10
+
+### Patch Changes
+
+- 9f87280: Fix kbar fetch to load relative to base url and add back starting actions
+
 ## 0.1.0-beta.9
 
 ### Patch Changes

packages/pliny/add-use-client.mjs

@@ -16,7 +16,11 @@ import globby from 'globby'
     }
   }
   // Handle ui differently as they are not split
-  const clientPaths = await globby(['ui/NewsletterForm.js', 'ui/Pre.js'])
+  const clientPaths = await globby([
+    'ui/NewsletterForm.js',
+    'ui/BlogNewsletterForm.js',
+    'ui/Pre.js',
+  ])
   for (const path of clientPaths) {
     console.log(path)
     const data = fs.readFileSync(path)

packages/pliny/package.json

@@ -2,7 +2,7 @@
   "name": "pliny",
   "description": "Main entry point for pliny components",
   "homepage": "https://github.com/timlrx/pliny",
-  "version": "0.1.0-beta.9",
+  "version": "0.1.0-beta.10",
   "type": "module",
   "exports": {
     "./*": "./*",

packages/pliny/src/config.ts

@@ -98,7 +98,7 @@ const sampleConfig: PlinyConfig = {
   search: {
     provider: 'kbar', // kbar or algolia
     kbarConfig: {
-      searchDocumentsPath: 'search.json', // path to load documents to search
+      searchDocumentsPath: 'search.json', // path to load documents to search relative to public folder
     },
     // algoliaConfig: {
     //   // The application ID provided by Algolia

packages/pliny/src/search/KBar.tsx

@@ -39,20 +39,6 @@ export const KBarSearchProvider: FC<{
   const [modalLoaded, setModalLoaded] = useState(false)
   const [dataLoaded, setDataLoaded] = useState(false)
 
-  const startingActions = useMemo(() => {
-    return Array.isArray(defaultActions)
-      ? defaultActions
-      : [
-          {
-            id: 'homepage',
-            name: 'Homepage',
-            keywords: '',
-            section: 'Home',
-            perform: () => router.push('/'),
-          },
-        ]
-  }, [defaultActions, router])
-
   const importDocSearchModalIfNeeded = useCallback(() => {
     if (KBarModal) {
       return Promise.resolve()
@@ -73,7 +59,18 @@ export const KBarSearchProvider: FC<{
       }
     }
     const mapPosts = (posts: CoreContent<MDXDocument>[]) => {
-      const actions: Action[] = []
+      const startingActions = Array.isArray(defaultActions)
+        ? defaultActions
+        : [
+            {
+              id: 'homepage',
+              name: 'Homepage',
+              keywords: '',
+              section: 'Home',
+              perform: () => router.push('/'),
+            },
+          ]
+      const actions: Action[] = startingActions
       for (const post of posts) {
         actions.push({
           id: post.path,
@@ -87,7 +84,11 @@ export const KBarSearchProvider: FC<{
       return actions
     }
     async function fetchData() {
-      const res = await fetch(searchDocumentsPath)
+      const url =
+        searchDocumentsPath.indexOf('://') > 0 || searchDocumentsPath.indexOf('//') === 0
+          ? searchDocumentsPath
+          : new URL(searchDocumentsPath, window.location.origin)
+      const res = await fetch(url)
       const json = await res.json()
       const actions = mapPosts(json)
       setSearchActions(actions)
@@ -103,14 +104,7 @@ export const KBarSearchProvider: FC<{
       /*removes event listener on cleanup*/
       window.removeEventListener('keydown', handleKeyDown)
     }
-  }, [
-    importDocSearchModalIfNeeded,
-    modalLoaded,
-    dataLoaded,
-    startingActions,
-    router,
-    searchDocumentsPath,
-  ])
+  }, [importDocSearchModalIfNeeded, modalLoaded, dataLoaded, router, searchDocumentsPath])
 
   return (
     <>