Skip to content

Improve Memory header controls (codex) - #2525

Merged
numnx merged 1 commit into
feature/CODUX-163-bugfixesfrom
task/feature-codux-163-bugfixes-t01-codex-mrabpsi0
Jul 7, 2026
Merged

Improve Memory header controls (codex)#2525
numnx merged 1 commit into
feature/CODUX-163-bugfixesfrom
task/feature-codux-163-bugfixes-t01-codex-mrabpsi0

Conversation

@numnx

@numnx numnx commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

🧩 Task: Improve Memory header controls

Automated task PR opened by Code UX for T01.

📋 Summary

Sprint: Sprint 163 · Bugfixes · Task: T01

Fix the Memory page header, self-reflection sprint-start gating and invocation display, and sprint completion PR timing while keeping tests and docs current.

🤖 Model & Provider

Provider Codex CLI
Model gpt-5.5

⏱️ Timing

Started 2026-07-07 07:24:33 UTC
Finished 2026-07-07 07:32:21 UTC
Duration 7m 48s

📊 Token Usage

Metric Value
Input tokens 186,268
Cached input tokens 1,391,744
Output tokens 19,306
Total tokens 1,597,318
Invocations 1
Tool calls 44
Active time 7m 21s

Included usage estimate (subscription/local login): $2.21

✅ QA Review

⏳ QA review runs after this PR is opened. Check the task card in the dashboard for the outcome once it lands.

📝 Full Task Prompt
## Objective
Improve the Memory page header section called out in CODUXCODUX-96 by making the tier summary, filter controls, and actions easier to scan without losing the current accessibility and danger-delete contracts.

## Scope
- `dashboard/src/v2/MemoryPage.tsx`
- `dashboard/src/v2/components/memory/MemoryFilters.tsx`
- `dashboard/src/v2/components/memory/__tests__/MemoryFilters.accessibility.test.tsx`
- `tests/dashboard/v2/memory-page.test.tsx`
- `docs/dashboard/design-system-memory.md`
- `docs/dashboard/memory.md`
- `docs-web/user/dashboard/memory.md`

## Implementation Requirements
1. Refactor the `MemoryFilters` rendered inside `PageHeader.actions` into a polished header control layout with a clear tier/count summary for Short Term and Long Term, a compact current-scope line, and separate rows or groups for selectors and actions so the controls no longer read as one long crowded string.
2. Preserve and improve the existing controls: tier tabs with counts, sprint selector for short-term memory, agent selector, Add Memory, Model Catalog pressed state with active-model status, and Danger Delete Off/Armed state with persistent explanatory copy.
3. Ensure the header wraps cleanly on narrow screens using `min-w-0`, `flex-wrap`, stable button/control sizing, and no horizontal overflow; keep visible focus rings, `role="tablist"`, `aria-selected`, `aria-pressed`, live status announcements, and disabled reason copy.
4. Update or add focused tests that assert the new visible count/summary copy, accessible names, disabled reasons, keyboard tab navigation, model catalog pressed state, and danger delete copy remain correct.
5. Update canonical and public Memory docs to describe the improved header/filter bar behavior and any changed user-visible wording.

## Constraints
- Do not change memory API requests, graph rendering, search behavior, batch delete behavior, or embedding model business logic.
- Keep danger delete semantics intact: graph and inspector single-memory deletes are immediate only when armed, and sidebar card deletion remains separately guarded by its existing arm/cancel flow.
- Keep existing `activeTierSignal`, `selectedSprintIdSignal`, and `selectedAgentPresetIdSignal` contracts; do not introduce a new state store.
- Do not use new external UI libraries.

## Verification
- Run `pnpm exec vitest run dashboard/src/v2/components/memory/__tests__/MemoryFilters.accessibility.test.tsx tests/dashboard/v2/memory-page.test.tsx`.
- Run `pnpm run typecheck:dashboard`.
- Success means the Memory header controls are visually grouped, responsive, accessible by role/name, and all focused tests pass.
🌿 Branch Info

Base: feature/CODUX-163-bugfixes
Head: task/feature-codux-163-bugfixes-t01-codex-mrabpsi0


🤖 Generated by Code UX

@numnx
numnx merged commit 56cc687 into feature/CODUX-163-bugfixes Jul 7, 2026
6 checks passed
@numnx
numnx deleted the task/feature-codux-163-bugfixes-t01-codex-mrabpsi0 branch July 7, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant