Codestin Search App

SasSwart · 2026-07-22T14:04:41Z

Frontend for the AI session network summary. Adds Network calls, Blocked network requests, and Top domains rows to the Session summary card on the individual AI session detail page, driven by the network fields on the session threads response.

Renders "Disabled" when network monitoring was not active and "No activity" when there were no calls. Covered by Storybook stories for each state.

PR map (merge strictly bottom-up)

This change is a 4-PR stack. Each PR depends on all the ones below it, so merge in this exact order:

feat: add network calls summary to AI session threads API #27417 — backend network summary
feat: show network request summary on AI session detail card #27418 — frontend summary rows
feat: add network calls list to AI session threads API #27425 — backend per-call list network_call_logs
feat(site): show network calls list on AI session detail #27426 — frontend network-calls panel

Refs AIGOV-463

🤖 Generated with Claude Code

linear-code · 2026-07-22T14:04:45Z

AIGOV-463

SasSwart

Self review complete. I need to check up on some things but its good enough for external review in the mean time.

coderagents · 2026-07-28T13:52:29Z

Documentation Check

Updates Needed

docs/ai-coder/ai-gateway/audit.md - Addressed in 804a1e9. The Session detail section now documents the Session summary card's Network requests, Blocked network requests, and Top domains rows. Verified against the head revision:
- Row labels match SessionSummaryTable.tsx exactly (Network requests, Blocked network requests, Top domains), and the card name matches the Session summary heading in SessionThreadsPageView.tsx
- Disabled / No activity conditions match the networkCalls === undefined and total === 0 branches
- "appears only when the session made at least one request" matches the total > 0 guard
- "When the session contacted more than one distinct domain, a +N more count follows" matches the totalCount > 1 guard
- No emdash, endash, or -- in the added prose
docs/ai-coder/ai-gateway/audit.md - Terminology split resolved in 804a1e9. The rename is now complete across every user-facing string, and the docs follow it:
- ListSessionsPageView.tsx header is now Network Requests, matching the docs' Network Requests column reference
- The Disabled tooltip and the badge tooltip now read "Network request monitoring was not active for this session." and "Total requests"
- The two non-numeric states are shared via the new NetworkRequestStates.tsx, so the list and detail card cannot drift apart again
- The Sessions list paragraph was updated from "network calls" to "network requests" throughout
Note: call survives only in non-user-facing identifiers (the network_calls JSON field, AIBridgeSessionNetworkCallSummary, NetworkCallBadges). No prose defect, but readers moving between the auto-generated docs/reference/api/ schemas and this page will see network_calls described as "network requests". Worth a one-line mention in the prose if that mismatch is expected to persist.
docs/images/aibridge/sessions.png - New: this screenshot shows the sessions list, whose column header changed from Network Calls to Network Requests in 804a1e9. If the header is visible in the capture, the image is now stale and contradicts the prose directly above it.
docs/images/aibridge/session_detail.png - Still outstanding. Verify whether this screenshot captures the Session summary card. If it does, it is now stale, since the card gained three rows.

Automated review via Coder Agents

EhabY

Few small comments, otherwise LGTM

EhabY · 2026-07-29T09:16:24Z

+	let networkCallsValue: ReactNode;
+	if (networkCalls === undefined) {
+		networkCallsValue = (
+			<span className="inline-flex items-center gap-1 whitespace-nowrap text-content-secondary">
+				Disabled
+				<InfoTooltip message="Network call monitoring was not active for this session." />
+			</span>
+		);
+	} else if (networkCalls.total === 0) {
+		networkCallsValue = (
+			<span className="whitespace-nowrap text-content-secondary">
+				No activity
+			</span>
+		);
+	} else {
+		networkCallsValue = (
+			<Badge>{networkCalls.total.toLocaleString("en-US")}</Badge>
+		);
+	}


This seems to be almost identical to NetworkCallBadges.tsx:20-33, I wonder if we can use a shared component so that if they drift then they drift together

EhabY · 2026-07-29T09:19:39Z

+			{networkCalls !== undefined && networkCalls.total > 0 && (
+				<div className="flex items-center justify-between">
+					<dt className="shrink-0 font-normal whitespace-nowrap">
+						Blocked network requests
+					</dt>


I suppose total always includes the blocked requests? (guaranteed by the server)

EhabY · 2026-07-29T09:30:16Z

Nit: I do wonder how useful the block signage here when the column says blocked anyway:

Also should we use the same terminalogy? Like Network Calls but then Blocked Network Requests, are the Calls different from the Requests?

github-actions · 2026-07-30T10:11:41Z

Docs preview

Check off each page once it's been reviewed. If a page changes in a later push, its checkbox clears automatically so it gets a fresh look. Pages not yet wired into the docs navigation aren't listed here.

Expose total and blocked network calls plus top destination domains on the AI session threads endpoint. Total and blocked reuse the existing Agent Firewall aggregation from the sessions list query; top domains are a new GetAIBridgeSessionTopDomains aggregation over boundary logs using the same interception-window correlation. Refs AIGOV-463 Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

…fetch Add multi-interception and shared-firewall-session tests asserting the network_calls summary total/blocked and top-domains counts partition correctly across consecutive windows and do not bleed across AI sessions that share one firewall session. Reduce the top-domains fetch to a single row, since the summary card renders only the most-contacted domain plus a "+N more" count derived from NetworkDomainCount (a window aggregate independent of the row cap). Document that the domain-extraction regex assumes scheme+host(+port) detail without userinfo or IPv6 literal hosts, and add a port-suffixed test row that pins host stripping. Refs AIGOV-463 Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

…uence number

Co-authored-by: Cian Johnston <[email protected]>

Add Network calls, Blocked network requests, and Top domains rows to the Session summary card, driven by the network fields on the session threads response. Renders Disabled when monitoring was not active and No activity when there were no calls. Refs AIGOV-463 Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

Backend for the AI session network summary. Exposes total/blocked network calls and top destination domains on the session threads endpoint (`GET /api/v2/ai-gateway/sessions/{id}`). Total and blocked reuse the existing Agent Firewall aggregation from the sessions list query, so the numbers match the sessions table. Top domains are a new server-side aggregation (`GetAIBridgeSessionTopDomains`) over boundary logs, using the same interception-window correlation. There is no network-error state, matching the current data model. Frontend consuming these fields is in a separate stacked PR. ### PR map (merge strictly bottom-up) This change is a 4-PR stack. Each PR depends on all the ones below it, so merge in this exact order: 1. #27417 — backend network summary (base `main`) 2. #27418 — frontend summary rows (base #27417) 3. #27425 — backend per-call list `network_call_logs` (base #27418) 4. #27426 — frontend network-calls panel (base #27425) Refs AIGOV-463 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]> Co-authored-by: Cian Johnston <[email protected]>

github-actions Bot assigned SasSwart Jul 22, 2026

SasSwart force-pushed the aigov-463-frontend branch from da2a0a6 to 734e63e Compare July 23, 2026 09:55

This was referenced Jul 23, 2026

feat: add network calls summary to AI session threads API #27417

Merged

feat: add network calls list to AI session threads API #27425

Merged

feat(site): show network calls list on AI session detail #27426

Merged

SasSwart force-pushed the aigov-463-frontend branch from 734e63e to 3a7959b Compare July 28, 2026 12:32

SasSwart commented Jul 28, 2026

View reviewed changes

Comment thread site/src/pages/AIBridgePage/SessionThreadsPage/SessionSummaryTable.tsx

Comment thread site/src/pages/AIBridgePage/SessionThreadsPage/SessionSummaryTable.tsx

SasSwart requested a review from EhabY July 28, 2026 13:49

SasSwart marked this pull request as ready for review July 28, 2026 13:49

EhabY approved these changes Jul 29, 2026

View reviewed changes

SasSwart requested a review from a team as a code owner July 30, 2026 10:11

SasSwart and others added 5 commits July 30, 2026 10:16

perf(coderd/database): bound network-call windows with a sentinel seq…

56621bd

…uence number

Update coderd/database/queries/aibridge.sql

d4fb320

Co-authored-by: Cian Johnston <[email protected]>

make gen

16d4f5d

SasSwart and others added 2 commits July 30, 2026 10:18

standardise terminology as network requests instead of network calls

9825f4f

SasSwart force-pushed the aigov-463-frontend branch from 6c2b3be to 9825f4f Compare July 30, 2026 10:19

SasSwart changed the title ~~feat(site): show network calls summary on AI session detail card~~ feat: show network request summary on AI session detail card Jul 30, 2026

fix: share network request states and align terminology

804a1e9

Base automatically changed from aigov-463-network-summary to main July 30, 2026 11:09

SasSwart merged commit 3f3fd1c into main Jul 30, 2026
36 checks passed

SasSwart deleted the aigov-463-frontend branch July 30, 2026 11:23

github-actions Bot locked and limited conversation to collaborators Jul 30, 2026

Navigation Menu

Uh oh!

feat: show network request summary on AI session detail card - #27418

feat: show network request summary on AI session detail card#27418
SasSwart merged 8 commits into
mainfrom
aigov-463-frontend

SasSwart commented Jul 22, 2026 •

edited

Loading

Uh oh!

linear-code Bot commented Jul 22, 2026

Uh oh!

SasSwart left a comment

Uh oh!

Uh oh!

Uh oh!

coderagents Bot commented Jul 28, 2026 •

edited

Loading

Uh oh!

EhabY left a comment

Uh oh!

Uh oh!

EhabY Jul 29, 2026

Uh oh!

EhabY Jul 29, 2026

Uh oh!

Uh oh!

Uh oh!

EhabY commented Jul 29, 2026

Uh oh!

github-actions Bot commented Jul 30, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

SasSwart commented Jul 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR map (merge strictly bottom-up)

Uh oh!

linear-code Bot commented Jul 22, 2026

Uh oh!

SasSwart left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderagents Bot commented Jul 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Documentation Check

Updates Needed

Uh oh!

EhabY left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

EhabY Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

EhabY Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

EhabY commented Jul 29, 2026

Uh oh!

github-actions Bot commented Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Docs preview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

SasSwart commented Jul 22, 2026 •

edited

Loading

coderagents Bot commented Jul 28, 2026 •

edited

Loading

github-actions Bot commented Jul 30, 2026 •

edited

Loading