Add Task Count and Limit Charts Per RowPublic

Got it. I’m going to find the “grouped” jobs view implementation for the job details page, identify which tasks are included in that grouped rendering, then add a small count label like “N tasks” (pluralized) based on the same filtered task set so it always matches what you’re seeing.

I found the likely job-details client component (JobSessionsClient.tsx). Next I’ll inspect how it renders “grouped” vs “ungrouped” and where the list of relevant tasks is computed, then I’ll add the N tasks label in that grouped header.

Next I’m going to zero in on the “grouped” rendering inside JobSessionsClient.tsx and identify where the list of “relevant tasks” is computed, so the N tasks label reflects exactly what’s shown (including filters like model/query params).

I found that the grouped view already computes the exact task groups it’s showing (filteredTaskGroups). Next I’ll add a small summary line at the top of that grouped list that displays the count as “N task(s)”, so it always matches the current filters (env/model/status/search/min-sessions/model-compare).

Done

In the grouped view, I added a small summary line showing the number of relevant task groups as “N task(s)”. It’s based on the same filteredTaskGroups list the UI renders, so it tracks your current filters (env/model/status/search/min-sessions/model-compare).

• Changed: src/app/(platform)/dashboard/sessions/components/GroupedSessionsView.tsx

Makes sense — instead of showing the task count above the grouped list, I’ll wire the grouped view to report the current “filtered task group count” up to JobSessionsClient, then render that right-aligned in the filters row (and remove the old in-list label).

I’m going to (1) remove the count from inside the grouped list, (2) have GroupedSessionsView report the current filtered task-count upward, then (3) render that count as a fixed element at the far right of the filter row so it stays aligned even when the filters scroll.

Updated placement

The “N tasks” label now renders on the right side of the filter dropdown row (so it stays aligned even when the filter chips scroll), instead of inside the grouped list.

• Grouped view reports count upward: GroupedSessionsView now calls onTaskCountChange(filteredTaskGroups.length) once it’s not in the initial loading state.
• Filter bar shows it on the right: JobSessionsClient displays the count as a fixed element to the right of the scrollable filters (only in grouped mode).

Files changed

• src/app/(platform)/dashboard/sessions/components/GroupedSessionsView.tsx
• src/app/(platform)/dashboard/jobs/[jobId]/JobSessionsClient.tsx

I’ll tweak the filter-row layout so the task count has a bit more breathing room on the right (between it and the search/tabs area), and also add a touch of right padding inside the horizontally-scrollable filters so the last pill isn’t jammed against the edge.

Added more right-side spacing

• Filters scroll area now has extra right padding (pr-3) so the last filter isn’t tight against the edge.
• “N tasks” now has a right margin (mr-3) so there’s more space before the search/tabs section.

Updated in src/app/(platform)/dashboard/jobs/[jobId]/JobSessionsClient.tsx.

I’ll update the Charts tab grid layout so it never goes beyond 2 charts per row (even on XL/2XL screens). That’s just a Tailwind class change in the Charts tab section of JobSessionsClient.tsx.

Updated: Charts tab max 2 per row

In the Charts tab, I changed the grid classes so it’s always 1 column on small screens and 2 columns at md and up (no more 3–4 columns on larger breakpoints).

• File: src/app/(platform)/dashboard/jobs/[jobId]/JobSessionsClient.tsx

Yep — they should still be in your working tree. I’ll double‑check your current git diff/status to confirm all the UI tweaks (task count placement/spacing + charts 2-per-row) are still present and uncommitted.

Yep — the code changes are still in place.

• Charts tab is capped at 2 per row: JobSessionsClient.tsx now has grid grid-cols-1 md:grid-cols-2 gap-4.
• Grouped “N tasks” is to the right of the filters + extra right spacing: the filters scroller has pr-3, and the count has mr-3.
• Grouped task count wiring exists: GroupedSessionsView exposes onTaskCountChange, and JobSessionsClient passes onTaskCountChange={setGroupedTaskCount}.

One note: git status currently says “working tree clean”, so these changes appear to already be committed (or otherwise not showing as local diffs). If you want, tell me what commit you expect them to be in and I’ll locate it in git log / git show.

They’re committed locally, but it looks like your branch hasn’t been pushed yet — so GitHub won’t show them. I’m going to check what commit you’re ahead by and whether origin/nico/eng-975-fix-job-charts is missing it.

Yep — I’ll finish the checks that got interrupted and confirm on GitHub which commit nico/eng-975-fix-job-charts currently points to, plus whether there’s an open PR using that branch.

Yes — your changes are reflected on nico/eng-975-fix-job-charts on GitHub.

• GitHub branch head SHA: 7f439aed692e76abbd7a1cb4effaf718fc36cbf5 (matches your local HEAD)
• Latest commit message: 7f439aed6 fix charts

The reason you may not be seeing them is likely that the PR you’re looking at isn’t using the latest commit, or you’re viewing the PR from an older commit in the UI.

Also: there’s an open PR for this branch, but it’s currently showing title “Increase OG image title padding…” (PR #1694). If you tell me the PR number/URL you’re looking at (or want me to open #1694 details), I can verify exactly which commit GitHub says it’s on and what files changed in that PR view.