diff --git a/app/renderer/lib/types.ts b/app/renderer/lib/types.ts
index 9f0bdb32..14e2d80f 100644
--- a/app/renderer/lib/types.ts
+++ b/app/renderer/lib/types.ts
@@ -220,7 +220,7 @@ export type MenubarPayload = {
skills: Array<{ name: string; turns: number; cost: number }>
subagents: Array<{ name: string; calls: number; cost: number }>
mcpServers: Array<{ name: string; calls: number }>
- // Spend by referenced pull request (top 20 by cost), attributed at turn
+ // Spend by referenced pull request (every PR, cost-descending), attributed at turn
// granularity. Optional: older CLIs omit it, and it is absent when no PR links
// were observed. Rows carry attributed cost/calls and ARE summable;
// `attributedCost + unattributedCost === distinctCost`. `approx` marks a row
@@ -250,8 +250,6 @@ export type MenubarPayload = {
subagentSessions?: number
attributedCost?: number
unattributedCost?: number
- otherPrCount?: number
- otherPrCost?: number
}
}
optimize: {
diff --git a/app/renderer/sections/PullRequests.test.tsx b/app/renderer/sections/PullRequests.test.tsx
index ae0dcf51..69efc422 100644
--- a/app/renderer/sections/PullRequests.test.tsx
+++ b/app/renderer/sections/PullRequests.test.tsx
@@ -53,8 +53,6 @@ const SAMPLE: PrPayload = {
distinctSessions: 3,
attributedCost: 330.75,
unattributedCost: 45.3,
- otherPrCount: 0,
- otherPrCost: 0,
}
// Get the button-role row wrapping a given PR link, for click/keyboard toggling.
@@ -71,26 +69,27 @@ describe('PullRequests', () => {
openExternal.mockResolvedValue(undefined)
})
- it('renders PR rows as a table with linked labels, cost, and a date span', async () => {
+ it('renders PR cards with linked labels, cost, activity, and a date span', async () => {
getOverview.mockResolvedValue(makePayload(SAMPLE))
render()
const link = await screen.findByRole('link', { name: 'getagentseal/codeburn#780' })
expect(link).toHaveAttribute('href', 'https://github.com/getagentseal/codeburn/pull/780')
expect(screen.getByText('$240.50')).toBeInTheDocument()
- expect(screen.getByText('512')).toBeInTheDocument()
+ expect(screen.getByText('512 calls')).toBeInTheDocument()
expect(screen.getByText(expectedSpan(SAMPLE.rows[0]!.firstStarted, SAMPLE.rows[0]!.lastEnded))).toBeInTheDocument()
// A same-day PR collapses its span to a single label.
expect(screen.getByText(expectedSpan(SAMPLE.rows[1]!.firstStarted, SAMPLE.rows[1]!.lastEnded))).toBeInTheDocument()
})
- it('renders the Models column with a "+N" overflow tag', async () => {
+ it('renders every model explicitly instead of hiding models behind an overflow count', async () => {
getOverview.mockResolvedValue(makePayload(SAMPLE))
render()
- // Three models collapse to the first two plus a count of the rest.
- expect(await screen.findByText('fable, opus +1')).toBeInTheDocument()
- // A single model renders as-is, with no overflow tag.
+ expect(await screen.findByText('fable')).toBeInTheDocument()
+ expect(screen.getByText('opus')).toBeInTheDocument()
+ expect(screen.getByText('haiku')).toBeInTheDocument()
+ expect(screen.queryByText('+1')).toBeNull()
expect(screen.getByText('sonnet')).toBeInTheDocument()
})
@@ -178,10 +177,11 @@ describe('PullRequests', () => {
getOverview.mockResolvedValue(makePayload(SAMPLE))
render()
- const note = await screen.findByText(/attributed to the rows above/)
- expect(note.textContent).toContain('$330.75')
- expect(note.textContent).toContain('3 PR-linked sessions')
- expect(note.textContent).toContain('summable')
+ expect(await screen.findByText('Attributed spend')).toBeInTheDocument()
+ expect(screen.getByText('$330.75')).toBeInTheDocument()
+ expect(screen.getByLabelText('Pull request attribution summary')).toHaveTextContent('Linked sessions3')
+ const note = screen.getByText(/attributed turn by turn/)
+ expect(note.textContent).toContain('without double counting')
expect(screen.getByText(/Not tied to a specific PR/).textContent).toContain('$45.30')
})
@@ -189,16 +189,17 @@ describe('PullRequests', () => {
getOverview.mockResolvedValue(makePayload({ ...SAMPLE, subagentSessions: 32 }))
render()
- const note = await screen.findByText(/attributed to the rows above/)
- expect(note.textContent).toContain('3 PR-linked sessions')
- expect(note.textContent).toContain('32 folded-in subagent runs')
+ expect(await screen.findByText('Folded agent runs')).toBeInTheDocument()
+ expect(screen.getByText('32')).toBeInTheDocument()
+ const note = screen.getByText(/attributed turn by turn/)
+ expect(note.textContent).toContain('32 subagent runs are included')
})
it('omits the subagent note when none were folded', async () => {
getOverview.mockResolvedValue(makePayload(SAMPLE))
render()
- const note = await screen.findByText(/attributed to the rows above/)
+ const note = await screen.findByText(/attributed turn by turn/)
expect(note.textContent).not.toContain('subagent')
})
@@ -260,23 +261,23 @@ describe('PullRequests', () => {
expect(screen.queryByText(/Not tied to a specific PR/)).toBeNull()
})
- it('renders an Other (N more PRs) reconciliation row when PRs are capped', async () => {
- const cappedPayload: PrPayload = {
- rows: [
- { url: 'https://github.com/getagentseal/codeburn/pull/780', label: 'getagentseal/codeburn#780', cost: 200, savingsUSD: 0, sessions: 3, calls: 512, firstStarted: '2026-07-01T10:00:00Z', lastEnded: '2026-07-03T18:00:00Z', models: ['fable'], categories: [{ name: 'Feature work', cost: 200 }] },
- ],
- distinctCost: 288.4,
- distinctSessions: 4,
- attributedCost: 288.4,
- unattributedCost: 0,
- otherPrCount: 5,
- otherPrCost: 88.4,
- }
- getOverview.mockResolvedValue(makePayload(cappedPayload))
+ it('renders the complete PR list without an opaque Other row', async () => {
+ const manyRows = Array.from({ length: 32 }, (_, index) => ({
+ ...SAMPLE.rows[0]!,
+ url: `https://github.com/getagentseal/codeburn/pull/${800 + index}`,
+ label: `getagentseal/codeburn#${800 + index}`,
+ }))
+ getOverview.mockResolvedValue(makePayload({
+ ...SAMPLE,
+ rows: manyRows,
+ attributedCost: manyRows.reduce((sum, row) => sum + row.cost, 0),
+ }))
render()
- expect(await screen.findByText('Other (5 more PRs)')).toBeInTheDocument()
- expect(screen.getByText('$88.40')).toBeInTheDocument()
+ expect(await screen.findByText('getagentseal/codeburn#800')).toBeInTheDocument()
+ expect(screen.getByText('getagentseal/codeburn#831')).toBeInTheDocument()
+ expect(screen.getByText('32 total')).toBeInTheDocument()
+ expect(screen.queryByText(/Other \(/)).toBeNull()
})
it('shows the quiet empty state (never a fake table) when no PR links exist', async () => {
diff --git a/app/renderer/sections/PullRequests.tsx b/app/renderer/sections/PullRequests.tsx
index cd6bb45b..c14aa8e8 100644
--- a/app/renderer/sections/PullRequests.tsx
+++ b/app/renderer/sections/PullRequests.tsx
@@ -27,10 +27,12 @@ function sessionWord(n: number): string {
return n === 1 ? 'session' : 'sessions'
}
-// Up to two short model names, then a "+N" overflow tag; empty for no models.
-function modelsLabel(models: string[]): string {
- if (models.length <= 2) return models.join(', ')
- return `${models.slice(0, 2).join(', ')} +${models.length - 2}`
+function ModelChips({ models }: { models: string[] }) {
+ return (
+
+ {models.map(model => {model})}
+
+ )
}
function openPr(event: MouseEvent, url: string): void {
@@ -41,7 +43,7 @@ function openPr(event: MouseEvent, url: string): void {
// Keyboard activation for the button-role row, guarded so Enter/Space fired on
// the inner link (its own control) never doubles up as a row toggle.
-function rowKeyDown(event: KeyboardEvent, toggle: () => void): void {
+function rowKeyDown(event: KeyboardEvent, toggle: () => void): void {
if (event.target !== event.currentTarget) return
if (event.key === 'Enter' || event.key === ' ') {
event.preventDefault()
@@ -73,7 +75,7 @@ function PullRequestsPage({ pullRequests, staleError }: { pullRequests?: PullReq
return (
<>
{staleError && }
-
+
{pullRequests && pullRequests.rows.length > 0
?
: PR links are captured as sessions are parsed. Once a session references a pull request, it appears here.}
@@ -83,7 +85,7 @@ function PullRequestsPage({ pullRequests, staleError }: { pullRequests?: PullReq
}
function PrTable({ pullRequests }: { pullRequests: PullRequests }) {
- const { rows, distinctCost, distinctSessions, subagentSessions, attributedCost, unattributedCost, otherPrCount, otherPrCost } = pullRequests
+ const { rows, distinctCost, distinctSessions, subagentSessions, attributedCost, unattributedCost } = pullRequests
const [expandedUrl, setExpandedUrl] = useState(null)
// Reset any open expansion when the PR set changes (a period/provider switch or
// a refresh that alters the list): a stale expandedUrl would otherwise linger
@@ -95,59 +97,51 @@ function PrTable({ pullRequests }: { pullRequests: PullRequests }) {
// payload omits it, so the rows are not summable and the footer must differ.
const summable = attributedCost !== undefined
const unattributed = unattributedCost ?? 0
- const otherCount = otherPrCount ?? 0
- const otherCost = otherPrCost ?? 0
- // Reconcile to the visible numbers: sum the rounded row costs (plus any
- // capped-away remainder) so the footer total equals what the eye adds up.
- const displayedAttributed = rows.reduce((sum, row) => sum + Number(row.cost.toFixed(2)), 0) + otherCost
+ // Reconcile to the visible numbers: every PR is present, so the summary is
+ // exactly the sum of the rounded cards a person can inspect below.
+ const displayedAttributed = rows.reduce((sum, row) => sum + Number(row.cost.toFixed(2)), 0)
return (
<>
-
-
-
-
-
Pull request
-
Models
-
Cost
-
Sessions
-
Calls
-
Active
-
-
-
-
- {rows.map(pr => (
- setExpandedUrl(current => current === pr.url ? null : pr.url)}
- />
- ))}
-
- {otherCount > 0 && (
- // A muted summary line, kept out of the sorted rows: its cost is an
- // aggregate of the capped-away PRs and can exceed a visible row.
-
-
-
Other ({otherCount.toLocaleString('en-US')} more PRs)
- {formatUsd(displayedAttributed)} attributed to the rows above, across {distinctSessions.toLocaleString('en-US')} PR-linked {sessionWord(distinctSessions)}
- {subagentSessions ? ` + ${subagentSessions.toLocaleString('en-US')} folded-in subagent ${subagentSessions === 1 ? 'run' : 'runs'}` : ''}.
- {' '}Each turn's cost goes to the PR it was working on, so the rows are summable.
+ Costs are attributed turn by turn, so every row adds up without double counting.
+ {subagentSessions ? ` ${subagentSessions.toLocaleString('en-US')} subagent ${subagentSessions === 1 ? 'run is' : 'runs are'} included in the PR where the work happened.` : ''}