Skip to content

fix(dashboard/memory): bound KV row output - #7494

Merged
houko merged 1 commit into
mainfrom
fix/dashboard-memory-agent-kv-rows
Aug 25, 2026
Merged

houko merged 1 commit into
mainfrom
fix/dashboard-memory-agent-kv-rows

Conversation

@houko

@houko houko commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Substantive changes

  • Replace raw per-agent KV query errors with fixed localized UI copy.
  • Centralize visible-cell and hover-preview truncation semantics.
  • Add focused coverage for error redaction and both preview bounds.

Verification

  • mise exec -- pnpm test --run src/pages/Memory/components/AgentKvRows.test.tsx (2 tests passed)
  • mise exec -- pnpm typecheck
  • mise exec -- pnpm exec eslint src/pages/Memory/components/AgentKvRows.tsx src/pages/Memory/components/AgentKvRows.test.tsx src/pages/Memory/constants.ts
  • mise exec -- pnpm lint
  • mise exec -- pnpm test --run (102 files, 1077 tests passed)
  • mise exec -- pnpm build
  • mise exec -- pnpm test:i18n-parity (known baseline only: Polish and Ukrainian each retain the same eight extra plural keys; Korean and Chinese match English)

Out-of-scope follow-ups

  • The inaccurate full-tooltip promise in Memory/constants.ts remains pending because open fix: align memory embedding provider catalog #7472 owns that file. Re-audit the preview contract on its post-merge source instead of stacking a conflicting edit.
  • Other Memory page audit findings remain separate component-level clusters in the handoff inventory.

@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed labels Aug 15, 2026
@houko
houko merged commit 5260a1b into main Aug 25, 2026
38 checks passed
@houko
houko deleted the fix/dashboard-memory-agent-kv-rows branch August 25, 2026 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant