Skip to content

feat(memory): add actor and session lists - #1955

Merged
notgitika merged 1 commit into
refactorfrom
feat/memory-actor-session-lists
Aug 13, 2026
Merged

feat(memory): add actor and session lists#1955
notgitika merged 1 commit into
refactorfrom
feat/memory-actor-session-lists

Conversation

@notgitika

@notgitika notgitika commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

This PR adds first-class actor and session list commands for AgentCore Memory.

Adds agentcore memory actor list and agentcore memory session list, with required selectors and pagination

Also adds corresponding TUI flows from the top-level Memory menu and a selected Memory's detail actions.

Reuses the actor/session picker components for the existing event flow.

Covers command behavior, direct TUI navigation, pagination, empty/retry states, and the existing event/record flows.

I did targeted memory tests command line + TUI. typecheck, lint, formatting, and bundle all pass.

@github-actions github-actions Bot added the size/l PR size: L label Aug 7, 2026
@github-actions github-actions Bot added agentcore-harness-reviewing AgentCore Harness review in progress size/l PR size: L and removed agentcore-harness-reviewing AgentCore Harness review in progress size/l PR size: L labels Aug 7, 2026
@notgitika
notgitika force-pushed the feat/memory-record-tui branch from e740afe to 99737d3 Compare August 11, 2026 15:22
@notgitika
notgitika force-pushed the feat/memory-actor-session-lists branch from 71041ca to 5146394 Compare August 11, 2026 15:33
@github-actions github-actions Bot added size/l PR size: L and removed size/l PR size: L labels Aug 11, 2026
Base automatically changed from feat/memory-record-tui to refactor August 11, 2026 21:38
@notgitika
notgitika force-pushed the feat/memory-actor-session-lists branch from 5146394 to 86c2e7b Compare August 11, 2026 21:41
@codecov-commenter

codecov-commenter commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.58974% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.71%. Comparing base (531d068) to head (7931b5c).

Files with missing lines Patch % Lines
src/handlers/memory/session/list/screen.tsx 56.09% 18 Missing ⚠️
src/handlers/memory/actor/screen.tsx 50.00% 1 Missing ⚠️
src/handlers/memory/session/screen.tsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           refactor    #1955      +/-   ##
============================================
- Coverage     96.78%   96.71%   -0.07%     
============================================
  Files           326      335       +9     
  Lines         18061    18290     +229     
============================================
+ Hits          17480    17689     +209     
- Misses          581      601      +20     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@notgitika
notgitika force-pushed the feat/memory-actor-session-lists branch from 86c2e7b to 7931b5c Compare August 12, 2026 00:59
@notgitika
notgitika merged commit 0a485da into refactor Aug 13, 2026
11 checks passed
@notgitika
notgitika deleted the feat/memory-actor-session-lists branch August 13, 2026 16:39
@jariy17

jariy17 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Nice work! My worry is that adding actor and session to the top-level memory menu crowds it — customers still need to pick a memory first for those to be meaningful. Should it live under the selected memory rather than alongside it?

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

Labels

size/l PR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants