address PR feedback

Author: wise-king-sullyman

src/__tests__/pages/api/__tests__/[version]/[section]/[page]/[tab].test.ts

@@ -131,76 +131,106 @@ jest.mock('../../../../../../../utils/apiIndex/get', () => ({
   }),
 }))
 
+/**
+ * Mock fetchApiIndex to return the same data as getApiIndex
+ */
+jest.mock('../../../../../../../utils/apiIndex/fetch', () => ({
+  fetchApiIndex: jest.fn().mockResolvedValue({
+    versions: ['v6'],
+    sections: {
+      v6: ['components'],
+    },
+    pages: {
+      'v6::components': ['alert'],
+    },
+    tabs: {
+      'v6::components::alert': ['react', 'html', 'react-demos'],
+    },
+  }),
+}))
+
 beforeEach(() => {
   jest.clearAllMocks()
 })
 
-it('returns markdown/MDX content as plain text', async () => {
+it('redirects to /text endpoint', async () => {
+  const mockRedirect = jest.fn((path: string) => new Response(null, { status: 302, headers: { Location: path } }))
   const response = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
       tab: 'react',
     },
+    url: new URL('http://localhost/api/v6/components/alert/react'),
+    redirect: mockRedirect,
   } as any)
-  const body = await response.text()
 
-  expect(response.status).toBe(200)
-  expect(response.headers.get('Content-Type')).toBe('text/plain; charset=utf-8')
-  expect(typeof body).toBe('string')
-  expect(body).toContain('Alert Component')
+  expect(mockRedirect).toHaveBeenCalledWith('/api/v6/components/alert/react/text')
+  expect(response.status).toBe(302)
 })
 
-it('returns different content for different tabs', async () => {
+it('redirects to /text endpoint for different tabs', async () => {
+  const mockRedirect = jest.fn((path: string) => new Response(null, { status: 302, headers: { Location: path } }))
+
   const reactResponse = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
       tab: 'react',
     },
+    url: new URL('http://localhost/api/v6/components/alert/react'),
+    redirect: mockRedirect,
   } as any)
-  const reactBody = await reactResponse.text()
 
+  expect(reactResponse.status).toBe(302)
+  expect(mockRedirect).toHaveBeenCalledWith('/api/v6/components/alert/react/text')
+
+  mockRedirect.mockClear()
   const htmlResponse = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
       tab: 'html',
     },
+    url: new URL('http://localhost/api/v6/components/alert/html'),
+    redirect: mockRedirect,
   } as any)
-  const htmlBody = await htmlResponse.text()
 
-  expect(reactBody).toContain('React Alert')
-  expect(htmlBody).toContain('HTML')
-  expect(reactBody).not.toEqual(htmlBody)
+  expect(htmlResponse.status).toBe(302)
+  expect(mockRedirect).toHaveBeenCalledWith('/api/v6/components/alert/html/text')
 })
 
-it('returns demo content for demos tabs', async () => {
+it('redirects demos tabs to /text endpoint', async () => {
+  const mockRedirect = jest.fn((path: string) => new Response(null, { status: 302, headers: { Location: path } }))
   const response = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
       tab: 'react-demos',
     },
+    url: new URL('http://localhost/api/v6/components/alert/react-demos'),
+    redirect: mockRedirect,
   } as any)
-  const body = await response.text()
 
-  expect(response.status).toBe(200)
-  expect(body).toContain('demos')
+  expect(response.status).toBe(302)
+  expect(mockRedirect).toHaveBeenCalledWith('/api/v6/components/alert/react-demos/text')
 })
 
 it('returns 404 error for nonexistent version', async () => {
+  const mockRedirect = jest.fn()
   const response = await GET({
     params: {
       version: 'v99',
       section: 'components',
       page: 'alert',
       tab: 'react',
     },
+    url: new URL('http://localhost/api/v99/components/alert/react'),
+    redirect: mockRedirect,
   } as any)
   const body = await response.json()
 
@@ -210,13 +240,16 @@ it('returns 404 error for nonexistent version', async () => {
 })
 
 it('returns 404 error for nonexistent section', async () => {
+  const mockRedirect = jest.fn()
   const response = await GET({
     params: {
       version: 'v6',
       section: 'invalid',
       page: 'alert',
       tab: 'react',
     },
+    url: new URL('http://localhost/api/v6/invalid/alert/react'),
+    redirect: mockRedirect,
   } as any)
   const body = await response.json()
 
@@ -225,13 +258,16 @@ it('returns 404 error for nonexistent section', async () => {
 })
 
 it('returns 404 error for nonexistent page', async () => {
+  const mockRedirect = jest.fn()
   const response = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'nonexistent',
       tab: 'react',
     },
+    url: new URL('http://localhost/api/v6/components/nonexistent/react'),
+    redirect: mockRedirect,
   } as any)
   const body = await response.json()
 
@@ -241,13 +277,16 @@ it('returns 404 error for nonexistent page', async () => {
 })
 
 it('returns 404 error for nonexistent tab', async () => {
+  const mockRedirect = jest.fn()
   const response = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
       tab: 'nonexistent',
     },
+    url: new URL('http://localhost/api/v6/components/alert/nonexistent'),
+    redirect: mockRedirect,
   } as any)
   const body = await response.json()
 
@@ -257,12 +296,15 @@ it('returns 404 error for nonexistent tab', async () => {
 })
 
 it('returns 400 error when required parameters are missing', async () => {
+  const mockRedirect = jest.fn()
   const response = await GET({
     params: {
       version: 'v6',
       section: 'components',
       page: 'alert',
     },
+    url: new URL('http://localhost/api/v6/components/alert'),
+    redirect: mockRedirect,
   } as any)
   const body = await response.json()
 

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

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import type { APIRoute } from 'astro'
 import { fetchApiIndex } from '../../../../../utils/apiIndex/fetch'
 import { createJsonResponse, createIndexKey } from '../../../../../utils/apiHelpers'

src/pages/api/[version]/[section]/[page]/[tab]/examples.ts

@@ -1,36 +1,9 @@
-import type { APIRoute, GetStaticPaths } from 'astro'
+import type { APIRoute } from 'astro'
 import { createJsonResponse, createIndexKey } from '../../../../../../utils/apiHelpers'
-import { getApiIndex } from '../../../../../../utils/apiIndex/get'
 import { fetchApiIndex } from '../../../../../../utils/apiIndex/fetch'
 
 export const prerender = false
 
-export const getStaticPaths: GetStaticPaths = async () => {
-  // Use the pre-generated index file
-  const index = await getApiIndex()
-
-  const paths: { params: { version: string; section: string; page: string; tab: string } }[] = []
-
-  // Build paths from index structure
-  for (const version of index.versions) {
-    for (const section of index.sections[version] || []) {
-      const sectionKey = createIndexKey(version, section)
-      for (const page of index.pages[sectionKey] || []) {
-        const pageKey = createIndexKey(version, section, page)
-        for (const tab of index.tabs[pageKey] || []) {
-          // Only create paths for tabs that have examples
-          const tabKey = createIndexKey(version, section, page, tab)
-          if (index.examples[tabKey] && index.examples[tabKey].length > 0) {
-            paths.push({ params: { version, section, page, tab } })
-          }
-        }
-      }
-    }
-  }
-
-  return paths
-}
-
 export const GET: APIRoute = async ({ params, url }) => {
   const { version, section, page, tab } = params