mirror of
https://github.com/explosion/spaCy.git
synced 2026-01-13 03:56:18 +03:00
What was wrong
`apiDetails.baseClass` in [website/pages/[...listPathPage].tsx](cci:7://file:///c:/Users/T2430514/Downloads/spaCy/website/pages/%5B...listPathPage%5D.tsx:0:0-0:0) was hard-coded to `null`, ignoring the `api_base_class` and `api_base_class_title` values provided in page front-matter.
As a result, API reference pages could not render “extends …” information or generate correct cross-links, hurting usability and discoverability of the docs.
## How it’s fixed
The placeholder line was replaced with real logic:
```ts
const baseClass = mdx.frontmatter.api_base_class
? {
title: mdx.frontmatter.api_base_class_title ?? mdx.frontmatter.title,
slug: mdx.frontmatter.api_base_class,
}
: null;
```
apiDetails.baseClass now receives this object, allowing the UI to show accurate base-class data whenever it’s supplied, while still defaulting to null when it isn’t.
Co-Authored-By: S. M. Mohiuddin Khan Shiam <147746955+mohiuddin-khan-shiam@users.noreply.github.com>
154 lines
4.7 KiB
TypeScript
154 lines
4.7 KiB
TypeScript
import type { GetStaticPaths, GetStaticProps } from 'next'
|
|
import { serialize } from 'next-mdx-remote/serialize'
|
|
import fs from 'fs'
|
|
import { MDXRemote, MDXRemoteSerializeResult } from 'next-mdx-remote'
|
|
import path from 'path'
|
|
import Layout from '../src/templates'
|
|
import remarkPlugins from '../plugins/index.mjs'
|
|
|
|
import recordSection from '../meta/recordSections'
|
|
import { sidebarUsageFlat } from '../meta/sidebarFlat'
|
|
|
|
type ApiDetails = {
|
|
stringName: string | null
|
|
baseClass: {
|
|
title: string
|
|
slug: string
|
|
} | null
|
|
trainable: string | null
|
|
}
|
|
|
|
export type PropsPageBase = {
|
|
/**
|
|
* TODO: This is only here for legacy support of the old code base
|
|
* It should be refactort to pass the file path and page path instead.
|
|
*/
|
|
slug: string
|
|
sectionTitle: string | null
|
|
theme: string | null
|
|
section: string
|
|
isIndex: boolean
|
|
}
|
|
|
|
export type PropsPage = PropsPageBase & {
|
|
mdx: MDXRemoteSerializeResult
|
|
apiDetails: ApiDetails
|
|
}
|
|
|
|
const PostPage = ({ mdx: mdx, ...props }: PropsPage) => {
|
|
return (
|
|
<Layout {...props}>
|
|
<MDXRemote {...mdx} />
|
|
</Layout>
|
|
)
|
|
}
|
|
|
|
export default PostPage
|
|
|
|
type ParsedUrlQuery = {
|
|
listPathPage: Array<string>
|
|
}
|
|
|
|
export const getStaticPaths: GetStaticPaths<ParsedUrlQuery> = async () => {
|
|
// This function needs to be defined inside `getStaticPath` to be executed in executed in the correct context
|
|
const loadFolder = (pathBase: Array<string> = []): Array<{ params: ParsedUrlQuery }> =>
|
|
fs
|
|
.readdirSync(path.join('docs', ...pathBase), { withFileTypes: true })
|
|
.flatMap((dirent: fs.Dirent) => {
|
|
if (dirent.isDirectory()) {
|
|
return loadFolder([...pathBase, dirent.name])
|
|
}
|
|
if (!dirent.name.includes('.mdx') || dirent.name[0] === '_') {
|
|
return []
|
|
}
|
|
|
|
return {
|
|
params: {
|
|
listPathPage:
|
|
dirent.name === 'index.mdx'
|
|
? pathBase
|
|
: [...pathBase, dirent.name.replace('.mdx', '')],
|
|
},
|
|
}
|
|
})
|
|
|
|
return {
|
|
paths: loadFolder(),
|
|
fallback: false,
|
|
}
|
|
}
|
|
|
|
const getPathFileWithExtension = (listPathFile: ReadonlyArray<string>) =>
|
|
`${path.join(...listPathFile)}.mdx`
|
|
|
|
export const getStaticProps: GetStaticProps<PropsPage, ParsedUrlQuery> = async (args) => {
|
|
if (!args.params) {
|
|
return { notFound: true }
|
|
}
|
|
|
|
const listPathFile = ['docs', ...args.params.listPathPage]
|
|
const isIndex = fs.existsSync(getPathFileWithExtension(listPathFile)) !== true
|
|
const listPathFileWithIndex = isIndex ? [...listPathFile, 'index'] : listPathFile
|
|
const pathFileWithIndexAndExtension = getPathFileWithExtension(listPathFileWithIndex)
|
|
|
|
const mdx = await serialize(fs.readFileSync(pathFileWithIndexAndExtension, 'utf-8'), {
|
|
parseFrontmatter: true,
|
|
mdxOptions: { remarkPlugins },
|
|
})
|
|
|
|
if (!mdx.frontmatter) {
|
|
throw new Error(`Frontmatter missing for ${pathFileWithIndexAndExtension}`)
|
|
}
|
|
|
|
const parentFolder =
|
|
listPathFileWithIndex.length > 1
|
|
? listPathFileWithIndex[listPathFileWithIndex.length - 2]
|
|
: null
|
|
const section = mdx.frontmatter.section ?? parentFolder
|
|
const sectionMeta = section ? recordSection[section] ?? null : null
|
|
|
|
// Determine base class details from frontmatter, if provided
|
|
const baseClass = mdx.frontmatter.api_base_class
|
|
? {
|
|
title: mdx.frontmatter.api_base_class_title ?? mdx.frontmatter.title,
|
|
slug: mdx.frontmatter.api_base_class,
|
|
}
|
|
: null
|
|
|
|
const apiDetails: ApiDetails = {
|
|
stringName: mdx.frontmatter.api_string_name ?? null,
|
|
baseClass,
|
|
trainable: mdx.frontmatter.api_trainable ?? null,
|
|
}
|
|
|
|
const slug = `/${args.params.listPathPage.join('/')}`
|
|
|
|
const next =
|
|
section === 'usage'
|
|
? sidebarUsageFlat.find((item, index) => {
|
|
return (
|
|
index > 0 && sidebarUsageFlat[index - 1].url === slug && item.url[0] === '/'
|
|
)
|
|
})
|
|
: undefined
|
|
|
|
return {
|
|
props: {
|
|
...mdx.frontmatter,
|
|
slug,
|
|
mdx,
|
|
sectionTitle: sectionMeta?.title ?? null,
|
|
theme: sectionMeta?.theme ?? null,
|
|
section: section,
|
|
apiDetails: apiDetails,
|
|
isIndex,
|
|
next: next
|
|
? {
|
|
slug: next.url,
|
|
title: next.text,
|
|
}
|
|
: null,
|
|
},
|
|
}
|
|
}
|