# refactor\(web\): the pull request list's rows read the speed-mode modifier themselves · PR \#22704 · pingdotgg/t3code

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

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

Visibility: public

State: merged

Source repository: pingdotgg/t3code

Source branch: refactor/split-pull-requests-list-groups

Target branch: refactor/split-pull-requests-detail-hook

Head commit: 39c2453342f21a0f56d05bf184a749337a230098

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

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

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

## Description

---

The page subscribed to the Shift modifier at the top of the route, so every Shift press re-rendered the whole page and the column only to pass `speedMode` to the rows.

The grouped rows move into `PullRequestListGroups.tsx`, which reads the modifier itself. With 136 rows listed, 3 Shift taps: `PullRequestsRouteView` 6 → 0 renders, `PullRequestsColumn` 6 → 0; the rows still re-render because their speed actions really change.

| File | Before | After |
|---|---|---|
| routes/_chat.pull-requests.tsx | 2,100 | 2,034 |
| PullRequestListGroups.tsx | - | 121 |

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)