feat: add page for ai-bridge interception logs #20331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

jakehwll merged 70 commits into main from jakehwll/routing-ai-governance

Nov 12, 2025

Contributor

jakehwll commented Oct 16, 2025

Relates #20287

This pull-request introduces a basic routing for AI Governance's Request Logs feature. Currently we're just pulling back the basics from the database and rendering it into the table. Nothing exciting.

The idea is to extend further upon the /aigovernance route so it has been appropriately wrapped with a <AIGovernanceLayout /> to introduce a navigation later.

github-actions bot assigned jakehwll

jakehwll changed the title ~~feat(ai-governance): implement request logs routing~~ feat: implement request logs routing

jakehwll force-pushed the jakehwll/routing-ai-governance branch from 321ce8c to cd9b38b Compare

October 21, 2025 23:23

jakehwll changed the base branch from main to dean/aibridge-list-with-user

October 21, 2025 23:23

Base automatically changed from dean/aibridge-list-with-user to main

October 22, 2025 05:18

jakehwll mentioned this pull request

fix: rename total to count in AIBridgeListInterceptionsResponse #20410

Merged

jakehwll added a commit that referenced this pull request


          fix: rename total to count in AIBridgeListInterceptionsResponse (…

d455f6e

…#20410)

Thanks to the great work in #20393, we’ve successfully introduced
offset-based pagination for this endpoint. However, the frontend expects
a `count` field in the response rather than `total`. This PR updates the
response payload to rename the returned key to `count` for consistency
with frontend expectations and existing API patterns.

This is necessary to unblock the work in #20331

jakehwll and others added 23 commits

October 23, 2025 02:27


          feat!: impl /aigovernance to Admin Settings (hardcoded on)

cfc4aa9


          feat: add /aigovernance routes and layout

134495a


          feat: feed some dummy content

6cd0819


          feat: implement <RequestLogsRow />

1f9fddf


          feat: query interceptions from backend

0231d38


          feat: render interceptions in frontend

bc71782


          fix: omit status column (lacking data)

6e05475


          fix: ensure that we're checking for aibridge for ai governance menu

afab507


          feat: empty and loading states on <RequestLogsPageView /> table

828d381


          feat: add <Paywall /> to /ai-governance/request-logs

ba289ec


          chore: add missing todo

b7280f5


          feat: link documentation to /ai-coder/ai-bridge

5abbac5


          fix: menu checking for aibridge experiment on

811d0c5


          fix: prefer to use .toLocalString() over string

6a74e80


          feat: add storybook generation

ebf2e91


          chore: resolve build

37df869


          feat: use usePaginatedQuery() and <PaginationContainer />

f282470


          feat: implement pagination

4f09456


          fix: improve opening button

baad816


          fix: show useravatar and username

32ae292


          fix: improve rendered content from json blob

0d4bd15


          chore: remove redundant onclick

5dc3679


          fix: improve the gaps in content

Member

aslilac commented Oct 30, 2025

also, the pr title may pass the bots vibe check but it's not very clear to me. maybe something like...

feat: add page for ai-bridge interception logs

jaaydenh reviewed

View reviewed changes

site/src/pages/AIGovernancePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx Outdated Show resolved Hide resolved

jaaydenh reviewed

View reviewed changes

site/src/pages/AIGovernancePage/RequestLogsPage/RequestLogsPageView.tsx Show resolved Hide resolved

jaaydenh reviewed

View reviewed changes

site/src/pages/AIGovernancePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx Outdated Show resolved Hide resolved

jaaydenh reviewed

View reviewed changes

site/src/pages/AIGovernancePage/AIGovernanceHelpTooltip.tsx Outdated Show resolved Hide resolved

jakehwll changed the title ~~feat: implement request logs routing~~ feat: add page for ai-bridge interception logs


          Merge branch 'main' into jakehwll/routing-ai-governance

f5909e5

jakehwll mentioned this pull request

chore: update paywall grammar mistake #20602

Merged

jakehwll added a commit that referenced this pull request


          chore: update paywall grammar mistake (#20602)

0f8f67e

Resolved issue from #20331 where the article `an` was incorrectly used
before words not beginning with a vowel sound. Updated affected
instances to use the correct article `a`.

```diff
- You need an Premium license to use this feature.
+ You need a Premium license to use this feature.
```

This was already correct on the following pages.

*
[`ConnectionLogPageView.tsx`](https://github.com/coder/coder/blob/7182c53df7648cd7db8551629226c4a0a1cc8559/site/src/pages/ConnectionLogPage/ConnectionLogPageView.tsx#L130)
*
[`GroupsPageView.tsx`](https://github.com/coder/coder/blob/7182c53df7648cd7db8551629226c4a0a1cc8559/site/src/pages/GroupsPage/GroupsPageView.tsx#L49)

david-fraley approved these changes

View reviewed changes

jaaydenh approved these changes

View reviewed changes

jakehwll removed the cherry-pick/v2.28 label

jakehwll added 8 commits

November 11, 2025 23:03


          fix: prefer FC<ProviderFilterProps>

4fb2c36


          fix: typo

e955388


          fix: resolve using size={} (prefer class)

45c1529


          fix: remove /docs prepended

2ae8d74


          fix: resolve missing key

a0239a3


          chore: remove children from <AIGovernanceLayout />

e1dbdb1


          fix: let type inference do its thing

14654d3


          fix: resolve interceptionQuery on Storyies

a115cc9

jakehwll requested a review from Emyrk as a code owner

November 11, 2025 23:04

jakehwll added a commit that referenced this pull request


          chore: update paywall grammar mistake (#20602)

a62f27d

Resolved issue from #20331 where the article `an` was incorrectly used
before words not beginning with a vowel sound. Updated affected
instances to use the correct article `a`.

```diff
- You need an Premium license to use this feature.
+ You need a Premium license to use this feature.
```

This was already correct on the following pages.

*
[`ConnectionLogPageView.tsx`](https://github.com/coder/coder/blob/7182c53df7648cd7db8551629226c4a0a1cc8559/site/src/pages/ConnectionLogPage/ConnectionLogPageView.tsx#L130)
*
[`GroupsPageView.tsx`](https://github.com/coder/coder/blob/7182c53df7648cd7db8551629226c4a0a1cc8559/site/src/pages/GroupsPage/GroupsPageView.tsx#L49)

jakehwll force-pushed the jakehwll/routing-ai-governance branch from a2a08c7 to a115cc9 Compare

November 11, 2025 23:10


          Merge branch 'main' into jakehwll/routing-ai-governance

99b0eb5

jakehwll removed the request for review from Emyrk

November 11, 2025 23:15

jakehwll added 2 commits

November 11, 2025 23:57


          chore: resolve lint

ea66dc1


          chore: commit bump

8bbfca3

jakehwll merged commit 3bb7975 into main

31 checks passed

jakehwll deleted the jakehwll/routing-ai-governance branch

November 12, 2025 00:41

github-actions bot locked and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet