Skip to content

remove legacy build pages, unused deps#154

Merged
vigneshrajsb merged 2 commits intomainfrom
codex/remove-build-pages
Mar 31, 2026
Merged

remove legacy build pages, unused deps#154
vigneshrajsb merged 2 commits intomainfrom
codex/remove-build-pages

Conversation

@vigneshrajsb
Copy link
Copy Markdown
Contributor

@vigneshrajsb vigneshrajsb commented Mar 31, 2026

What changed

  • remove all Next.js web pages under src/pages/builds/
  • delete the page-only src/components/logs and src/components/ai-agent frontend trees that were only used by those routes
  • remove the page-only packages that existed only for those routes: axios, partial-json, react-markdown, react-shiki, rehype-raw, and remark-gfm
  • remove additional unused packages: @types/react-syntax-highlighter, react-syntax-highlighter, bcryptjs, isomorphic-unfetch, js-cookie, kubernetes-client, seedrandom, inquirer, mockdate, only-allow, and promptfoo
  • simplify the activity stream status table back to Service / Branch / Status so it no longer emits links to removed /builds/... pages
  • remove stale jest.mock('axios') setup from the GitHub test files

Why

The /builds/* pages are no longer meant to be served from this app. Users navigate to the relevant functionality from a separate UI, so keeping these routes, internal status-comment links, and page-only dependencies in this repo only preserves dead surface area.

Impact

  • removes the legacy /builds/:uuid/* web routes from this app
  • drops the unused frontend code and additional unreferenced dependencies tied to that surface area or otherwise unused in the repo
  • removes dead Build Pipeline and Deploy Logs columns from the activity-stream comment table

Validation

  • pnpm exec prettier --check package.json pnpm-lock.yaml
  • pnpm run lint
  • pnpm run build
  • pnpm test

@vigneshrajsb vigneshrajsb changed the title [codex] remove legacy build pages remove legacy build pages, unused deps Mar 31, 2026
@vigneshrajsb vigneshrajsb marked this pull request as ready for review March 31, 2026 17:57
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner March 31, 2026 17:57
@vigneshrajsb vigneshrajsb merged commit 2e63de2 into main Mar 31, 2026
1 check passed
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.

2 participants