fix 404 from search result (#2181)

.changeset/kind-clouds-dance.md

@@ -0,0 +1,6 @@
+---
+'nextra-theme-docs': patch
+'nextra': patch
+---
+
+skip search indexing for 404/500 pages

docs/pages/404.mdx

@@ -0,0 +1,5 @@
+import { NotFoundPage } from 'nextra-theme-docs'
+
+# 404: Page Not Found
+
+<NotFoundPage />

docs/pages/_meta.json

@@ -25,5 +25,12 @@
     "theme": {
       "typesetting": "article"
     }
+  },
+  "404": {
+    "type": "page",
+    "theme": {
+      "timestamp": false,
+      "typesetting": "article"
+    }
   }
 }

docs/pages/docs/guide.mdx

@@ -1,4 +1,5 @@
 import {
+  BoxIcon,
   BrushIcon,
   DropperIcon,
   FilesIcon,
@@ -34,4 +35,9 @@ available in all themes.
   <Card icon={<GlobeIcon />} title="Next.js I18n" href="/docs/guide/i18n" />
   <Card icon={<BrushIcon />} title="Custom CSS" href="/docs/guide/custom-css" />
   <Card icon={<DropperIcon />} title="Advanced" href="/docs/guide/advanced" />
+  <Card
+    icon={<BoxIcon />}
+    title="Built-in Components"
+    href="/docs/guide/built-ins"
+  />
 </Cards>

packages/nextra-theme-docs/src/components/index.ts

@@ -10,10 +10,10 @@ export { Input } from './input'
 export { LocaleSwitch } from './locale-switch'
 export { NavLinks } from './nav-links'
 export { Navbar } from './navbar'
-export { NotFoundPage } from './not-found'
+export { NotFoundPage } from './404'
 export { Search } from './search'
 export { Select } from './select'
-export { ServerSideErrorPage } from './server-side-error'
+export { ServerSideErrorPage } from './500'
 export { Sidebar } from './sidebar'
 export { SkipNavContent, SkipNavLink } from './skip-nav'
 export { ThemeSwitch } from './theme-switch'

packages/nextra/src/compile.ts

@@ -17,6 +17,7 @@ import {
   CODE_BLOCK_FILENAME_REGEX,
   CWD,
   DEFAULT_LOCALE,
+  ERROR_ROUTES,
   MARKDOWN_URL_EXTENSION_REGEX
 } from './constants'
 import {
@@ -90,22 +91,35 @@ export async function compileMdx(
 
   const structurizedData = Object.create(null)
 
+  const {
+    staticImage,
+    flexsearch,
+    readingTime,
+    latex,
+    codeHighlight,
+    defaultShowCopyCode,
+    route = '',
+    locale,
+    mdxOptions
+  } = loaderOptions
+
   let searchIndexKey: string | null = null
-  if (typeof loaderOptions.flexsearch === 'object') {
-    if (loaderOptions.flexsearch.indexKey) {
-      searchIndexKey = loaderOptions.flexsearch.indexKey(
-        filePath,
-        loaderOptions.route || '',
-        loaderOptions.locale
-      )
+  if (
+    ERROR_ROUTES.has(route) ||
+    route === '/_app' /* remove this check in v3 */
+  ) {
+    /* skip */
+  } else if (typeof flexsearch === 'object') {
+    if (flexsearch.indexKey) {
+      searchIndexKey = flexsearch.indexKey(filePath, route, locale)
       if (searchIndexKey === '') {
-        searchIndexKey = loaderOptions.locale || DEFAULT_LOCALE
+        searchIndexKey = locale || DEFAULT_LOCALE
       }
     } else {
-      searchIndexKey = loaderOptions.locale || DEFAULT_LOCALE
+      searchIndexKey = locale || DEFAULT_LOCALE
     }
-  } else if (loaderOptions.flexsearch) {
-    searchIndexKey = loaderOptions.locale || DEFAULT_LOCALE
+  } else if (flexsearch) {
+    searchIndexKey = locale || DEFAULT_LOCALE
   }
 
   const {
@@ -116,23 +130,14 @@ export async function compileMdx(
     rehypePlugins,
     rehypePrettyCodeOptions
   }: MdxOptions = {
-    ...loaderOptions.mdxOptions,
+    ...mdxOptions,
     // You can override MDX options in the frontMatter too.
     ...frontMatter.mdxOptions
   }
 
   const format =
     _format === 'detect' ? (filePath.endsWith('.mdx') ? 'mdx' : 'md') : _format
 
-  const {
-    staticImage,
-    flexsearch,
-    readingTime,
-    latex,
-    codeHighlight,
-    defaultShowCopyCode
-  } = loaderOptions
-
   // https://github.com/shuding/nextra/issues/1303
   const isFileOutsideCWD =
     !isPageImport && path.relative(CWD, filePath).startsWith('..')