Skip to content

fix: expose docs deep-dive routes in sidebar#57

Open
scaleborg wants to merge 1 commit intomainfrom
fix/docs-deep-dive-sidebar-nav
Open

fix: expose docs deep-dive routes in sidebar#57
scaleborg wants to merge 1 commit intomainfrom
fix/docs-deep-dive-sidebar-nav

Conversation

@scaleborg
Copy link
Owner

Summary

  • Add 6 deep-dive documentation pages as sub-items under Docs > How It Works
  • Extend Docs header active state to highlight on deep-dive routes
  • Add mcp_ecosystem ViewId and pathname detection for /how-mcp-works

Routes added to sidebar

  • Ingestion Pipeline → /how-ingestion-works
  • Chat Pipeline → /how-chat-works
  • Career Intelligence → /how-career-works
  • Observability → /how-monitor-works
  • Architecture Reference → /how-architecture-works
  • MCP Ecosystem → /how-mcp-works

Not changed

  • /how-gpu-works intentionally left in Reference > GPUs (different domain)
  • No router changes - all routes already existed
  • No other sidebar sections modified

Test plan

  • npx tsc --noEmit - 0 errors
  • pnpm lint - 0 errors
  • Playwright: all 6 routes navigate correctly from sidebar
  • Playwright: SubNavItem shows [active] on each route
  • Playwright: Docs header data-active="true" strictly verified on all 6

🤖 Generated with Claude Code

Add 6 deep-dive documentation pages as sub-items under
Docs > How It Works in the sidebar navigation:

- Ingestion Pipeline → /how-ingestion-works
- Chat Pipeline → /how-chat-works
- Career Intelligence → /how-career-works
- Observability → /how-monitor-works
- Architecture Reference → /how-architecture-works
- MCP Ecosystem → /how-mcp-works

/how-gpu-works intentionally left in Reference > GPUs (different domain).

/how-mcp-works required new mcp_ecosystem ViewId and pathname detection
(was the only how-* route with zero sidebar infrastructure).

Docs header active state extended to highlight on all 6 deep-dive routes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant