Remove custom outputDir pathing now that we're fetching the apiIndex at

Author: wise-king-sullyman

.gitignore

@@ -35,4 +35,6 @@ textContent/*.mdx
 
 coverage/
 
-.wrangler/
+.wrangler/
+
+temp

cli/cli.ts

@@ -34,6 +34,7 @@ try {
     .replace('file://', '')
 } catch (e: any) {
   if (e.code === 'ERR_MODULE_NOT_FOUND') {
+    console.log('@patternfly/patternfly-doc-core not found, using current directory as astroRoot')
     astroRoot = process.cwd()
   } else {
     console.error('Error resolving astroRoot', e)
@@ -87,29 +88,6 @@ async function transformMDContentToMDX() {
   }
 }
 
-async function updateTsConfigOutputDirPath(program: Command) {
-  const { verbose } = program.opts()
-  const tsConfigPath = join(astroRoot, 'tsconfig.json')
-
-  try {
-    const tsConfigFile = await readFile(tsConfigPath, 'utf-8')
-    const tsConfig = JSON.parse(tsConfigFile)
-    const formattedOutputDir = join(absoluteOutputDir, '*')
-
-    tsConfig.compilerOptions.paths['outputDir/*'] = [formattedOutputDir]
-
-    await writeFile(tsConfigPath, JSON.stringify(tsConfig, null, 2))
-
-    if (verbose) {
-      console.log(
-        `Updated tsconfig.json with outputDir path: ${formattedOutputDir}`,
-      )
-    }
-  } catch (e: any) {
-    console.error('Error updating tsconfig.json with outputDir path:', e)
-  }
-}
-
 async function initializeApiIndex(program: Command) {
   const { verbose } = program.opts()
   const templateIndexPath = join(astroRoot, 'cli', 'templates', 'apiIndex.json')
@@ -150,7 +128,6 @@ async function buildProject(program: Command): Promise<DocsConfig | undefined> {
     )
     return config
   }
-  await updateTsConfigOutputDirPath(program)
   await updateContent(program)
   await generateProps(program, true)
   await initializeApiIndex(program)
@@ -230,7 +207,6 @@ program.command('init').action(async () => {
 })
 
 program.command('start').action(async () => {
-  await updateTsConfigOutputDirPath(program)
   await updateContent(program)
   await initializeApiIndex(program)
 

jest.config.ts

@@ -16,7 +16,6 @@ const config: Config = {
   testPathIgnorePatterns: ['/node_modules/', '/__tests__/helpers/'],
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
   moduleNameMapper: {
-    '^outputDir/(.*)$': '<rootDir>/dist/$1',
     '\\.(css|less)$': '<rootDir>/src/__mocks__/styleMock.ts',
     '^astro:content$': '<rootDir>/src/__mocks__/astro-content.ts',
     '(.+)\\.js': '$1',

package-lock.json

@@ -65,7 +65,7 @@
     "@patternfly/quickstarts": "^6.0.0",
     "@types/react": "^18.3.23",
     "@types/react-dom": "^18.3.7",
-    "astro": "5.15.9",
+    "astro": "^5.15.9",
     "change-case": "5.4.4",
     "commander": "^13.1.0",
     "glob": "^11.0.3",

package.json

@@ -137,7 +137,7 @@ it('returns 400 error when version parameter is missing', async () => {
   jest.restoreAllMocks()
 })
 
-it('excludes content entries that have no section field', async () => {
+it('returns sections array that matches the API index', async () => {
   global.fetch = jest.fn(() =>
     Promise.resolve({
       ok: true,
@@ -151,8 +151,10 @@ it('excludes content entries that have no section field', async () => {
   } as any)
   const body = await response.json()
 
-  // Should only include sections from entries that have data.section
-  expect(body.length).toBeGreaterThan(0)
+  // Verify the returned sections exactly match the indexed sections
+  // The API index generation process filters out entries without section fields
+  expect(body).toEqual(mockApiIndex.sections.v6)
+  expect(body).toEqual(['components', 'layouts', 'utilities'])
 
   jest.restoreAllMocks()
 })

src/__tests__/pages/api/__tests__/[version].test.ts

@@ -28,8 +28,9 @@ export const GET: APIRoute = async ({ params, url }) => {
 
     return createJsonResponse(pages)
   } catch (error) {
+    const details = error instanceof Error ? error.message : String(error)
     return createJsonResponse(
-      { error: 'Failed to load API index', details: error },
+      { error: 'Failed to load API index', details },
       500
     )
   }

src/pages/api/[version]/[section].ts

@@ -10,8 +10,9 @@ export const GET: APIRoute = async ({ url }) => {
     const index = await fetchApiIndex(url)
     versions = index.versions
   } catch (error) {
+    const details = error instanceof Error ? error.message : String(error)
     return createJsonResponse(
-      { error: 'Failed to load API index', details: error },
+      { error: 'Failed to load API index', details },
       500
     )
   }

src/pages/api/openapi.json.ts

@@ -7,12 +7,7 @@
     "module": "ESNext",
     "moduleResolution": "bundler",
     "importHelpers": true,
-    "verbatimModuleSyntax": false,
-    "paths": {
-      "outputDir/*": [
-        "/Users/ausulliv/repos/patternfly-doc-core/dist/*"
-      ]
-    }
+    "verbatimModuleSyntax": false
   },
   "include": [
     ".astro/types.d.ts",