Skip to content

feat: unclutter the agent History conversation list - #620

Open
michael-on-code wants to merge 1 commit into
mainfrom
feat/unclutter-agent-chat-history-discussions-list
Open

michael-on-code wants to merge 1 commit into
mainfrom
feat/unclutter-agent-chat-history-discussions-list

Conversation

@michael-on-code

Copy link
Copy Markdown
Contributor

feat(history): unclutter the agent History conversation list

Description

The agent-settings History tab (Edit Agent → Activity → History) packed up to seven things into every conversation row: a full timestamp ("Oct 26, 2026, 3:44 PM · 4 days ago"), the owner squeezed into a fixed max-w-[100px] box on the right, the title, a sentiment thumb next to grey topic chips, a preview line, a separate row of Documents/Tools chips, and a chevron. Nothing stood out, so the list was hard to scan.

This PR gives each row the same mail-style layout as the SDK's Profile → History tab (iblai-web-frontend, companion PR). The data, the filters and the preview pane stay the same.

Before                                                   After
┌───────────────────────────────────────────────┐        ┌───────────────────────────────────────────────┐
│ Oct 26, 2026, 3:44 PM · 4 days ago  Michael A… │       │ (M)  Lesson 1.2.1: Terror of Death     Oct 26 │
│ Lesson 1.2.1: Terror of Death               > │        │      How may I assist you today?              │
│ 👍 [Death]                                    │        │      Michael Animashaun · Death   [Docs·1] 👍 │
│ How may I assist you today?                   │        └───────────────────────────────────────────────┘
│ [Documents · 1] [Tools · 2]                   │
└───────────────────────────────────────────────┘

Checklist

  • Tests were added/updated according to the feature/bugfix/change made
  • Version was rolled according to semver requirements
  • API endpoints openapi schema was updated if applicable

Preview

image

@michael-on-code michael-on-code added priority run-tests Trigger PR validation pipeline labels Oct 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

❌ PR E2E failed

23 tests failed against prod 1.340.0.

Open the report: every failure has a screenshot and a trace. Comment /retest <spec> to re-run one file in minutes.

Tests What went wrong Example
1 Check failed: Error: expect(locator).toHaveText(expected) failed Locator: getByRole('dialog').filter({ hasText: 'Edit Agent' }).getByRole('table').locator('tbody tr').filter({ has: getByRole('cell', { name: 'e2e-apikey-1793402750001-zm1vx', exact: true }) }).locator('td').nth(2) admin creates an API key with an expiration date picked via the calendar, and the row shows that exact date · screenshot · trace
13 Check failed: Error: expect(locator).toBeVisible() failed Locator: getByRole('button', { name: 'Open Accessibility Menu' }) — also failing on os#​616, os#​619, probably not yours user clicks the accessibility button to open the menu, a drag does not open it, and a click after a drag does · screenshot · trace
6 Check failed: Error: Could not observe the tenant org-metadata GET (`/api/core/orgs/<org>/metadata/`) that the app itself fires on load via `providers/index.tsx`'s `useTenantMetadata`. This must FAIL, not skip — it is not gated by DM_URL or any per-environment setup. Underlying error: TimeoutError: page.waitForRe — also failing on os#​616, probably not yours newly registered user goes to sidebar and clicks the help button to open docs link · screenshot · trace
1 Check failed: Error: Report generation did not complete: neither CSV editor nor deferred-notification toast appeared within 60s — also failing on os#​616, os#​619, probably not yours admin goes to Data Reports tab and verifies other download buttons are disabled while generating · trace
1 Check failed: Error: expect(locator).toHaveAttribute(expected) failed Locator: getByRole('dialog', { name: 'User Profile' }).getByRole('switch', { name: /^Chat History Export/i }) — also failing on os#​616, os#​619, probably not yours admin opens the tenant Advanced tab and the Chat History Export switch is present, ON by default, and persists when toggled · screenshot · trace

Full report, traces and screenshots

798 passed · 23 failed · 201 skipped

Details
  • Where it stopped: running the tests

Pipeline run, for maintainers: 41126877791

commit 68fc3838 · suite mentor (full) on stg1 · tested against prod 1.340.0 · How to read this

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority run-tests Trigger PR validation pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant