feat(routes): hide /work directory from the public site

Author: danielnaab

src/build/routes.ts

@@ -7,17 +7,21 @@ export type Route = {
 }
 
 export function allRoutes(catalog: Catalog): Route[] {
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  const _catalog = catalog
   const routes: Route[] = [
     { path: '/', view: 'home' },
-    { path: '/work/', view: 'work-index' },
-    { path: '/work/health/', view: 'health' },
+    // Work directory is disabled until approved content is ready.
+    // Uncomment to restore the public catalog, detail pages, and health page.
+    // { path: '/work/', view: 'work-index' },
+    // { path: '/work/health/', view: 'health' },
     { path: '/commitment/', view: 'commitment' },
     { path: '/about/', view: 'about' },
     { path: '/design-system/', view: 'design-system' },
   ]
-  for (const entry of catalog) {
-    if (entry.hidden) continue
-    routes.push({ path: `/work/${entry.name}/`, view: 'work-detail', slug: entry.name })
-  }
+  // for (const entry of catalog) {
+  //   if (entry.hidden) continue
+  //   routes.push({ path: `/work/${entry.name}/`, view: 'work-detail', slug: entry.name })
+  // }
   return routes
 }

tests/a11y/pages.test.ts

@@ -7,8 +7,6 @@ const DIST = join(process.cwd(), 'dist')
 
 const pages = [
   'index.html',
-  'work/index.html',
-  'work/health/index.html',
   'commitment/index.html',
   'about/index.html',
   'design-system/index.html',

tests/build/routes.test.ts

@@ -0,0 +1,21 @@
+import { describe, test, expect } from 'bun:test'
+import { allRoutes } from '../../src/build/routes'
+import { fixtureCatalog } from '../fixtures/catalog'
+
+describe('allRoutes', () => {
+  test('produces the expected public routes', () => {
+    const paths = allRoutes(fixtureCatalog).map((r) => r.path)
+    expect(paths).toContain('/')
+    expect(paths).toContain('/commitment/')
+    expect(paths).toContain('/about/')
+    expect(paths).toContain('/design-system/')
+  })
+
+  test('does not produce any /work/ routes', () => {
+    const paths = allRoutes(fixtureCatalog).map((r) => r.path)
+    for (const path of paths) {
+      expect(path.startsWith('/work/')).toBe(false)
+    }
+    expect(paths).not.toContain('/work/')
+  })
+})

tests/build/smoke.test.ts

@@ -19,8 +19,6 @@ beforeAll(async () => {
 describe('build smoke', () => {
   const expectedPaths = [
     'index.html',
-    'work/index.html',
-    'work/health/index.html',
     'commitment/index.html',
     'about/index.html',
   ]
@@ -35,8 +33,8 @@ describe('build smoke', () => {
     })
   }
 
-  test('produces a work directory', () => {
+  test('does not produce a work directory', () => {
     const workDir = join(outDir, 'work')
-    expect(existsSync(workDir)).toBe(true)
+    expect(existsSync(workDir)).toBe(false)
   })
 })