# refactor\(web\): the pull requests column lives in its own component · PR \#22671 · pingdotgg/t3code

[View on GitCafe](https://git.cafe/pingdotgg/t3code/pull/22671)

Repository: [pingdotgg/t3code](https://git.cafe/pingdotgg/t3code)

Visibility: public

State: merged

Source repository: pingdotgg/t3code

Source branch: refactor/split-pull-requests

Target branch: main

Head commit: 0f7b1fa34cfd384b42d234f9485a91dbb2d8d188

[Discussion and reviews](https://git.cafe/pingdotgg/t3code/pull/22671)

[Commits](https://git.cafe/pingdotgg/t3code/pull/22671/commits?format=markdown)

[Files changed](https://git.cafe/pingdotgg/t3code/pull/22671/diffs?format=markdown)

## Description

---

`routes/_chat.pull-requests.tsx` was 2,502 lines: the route, the page's state, and the column's header, search and filter chrome all in one file.

This moves `PullRequestsColumn` (with its expandable search and refresh control) into `components/pullRequest/PullRequestsColumn.tsx`, `CompactFilterMenu` into `PullRequestCompactFilterMenu.tsx`, and the state/involvement/sort option lists into `pullRequestListOptions.ts`. No behavior change; the column was already its own component, so render boundaries are unchanged.

| File | Before | After |
|---|---|---|
| routes/_chat.pull-requests.tsx | 2,502 | 2,100 |
| PullRequestsColumn.tsx | - | 293 |
| PullRequestCompactFilterMenu.tsx | - | 93 |
| pullRequestListOptions.ts | - | 40 |

Part of the pull request page split stack: #22671 → #22672 → #22702 → #22703 → #22704 → #22705 → #22706.

Measured with the rc-init render counter against a dev server seeded from real data (136 pull requests listed), fresh browser per side, main `6a6c84f990` vs the stack top. Opening a PR detail: 6482 / 947 / 1249 renders on main vs 6331 / 1187 / 956 on the stack (noise, the first open includes data arriving). Switching Summary/Timeline/Code: 2-13 renders per switch on both sides. Holding Shift 3 times on the list: 14952 vs 14937 renders; the page and column no longer re-render (6 + 6 before, 0 after), the rows still do because their speed-mode prop really changes.

Tests run on this layer: `tsc --noEmit` for apps/web, `vp lint` on the changed files, `vp test run src/reactCompilerCoverage.test.ts -t pullRequest` (every new file compiles with 0 bailouts), `vp test run src/components/pullRequest/pullRequestsRouteSearch.test.ts`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

