Files in this chat — persistent per-chat artifact registry

[oxoxDev]

Summary

A persistent "Files in this chat" surface inside the chat view that lists every artifact the agent has generated in the current thread, with per-row Download / Reveal-in-folder / Delete affordances. Survives app restarts (for files created in-session) so the user can return to a chat and still grab a deck they made earlier.

Problem

The inline ArtifactCard shipped in #2779 (PR #3017) sits above the composer as a single ephemeral card per artifact:

• Scrolling, sending the next turn, or restarting the app loses the card.
• The artifactsByThread Redux slice is in-memory only (app/src/store/index.ts:158 registers chatRuntimeReducer directly, no persistReducer), so files vanish across app restarts even though the underlying payload is still on disk under <workspace>/artifacts/<uuid>/.
• There is no place to enlist all files involved in a chat.

User feedback (live dev:app test, 2026-05-30): "there should be a place to enlist all the files involved in a chat" — flagged after a generated climate-change-an-overview.pptx was visible once, then the user couldn't find it again after a new turn was sent.

Solution

Frontend-only addition layered on top of the artifact pipeline already shipped in #2779:

1. ChatFilesChip (new component) — header chip with paperclip icon + count badge. Hidden when artifact count = 0. Owns popover open/close + click-outside + Esc.
2. ChatFilesPanel (new component) — popover panel listing per-thread artifacts (title + kind icon + size + Download + Reveal + Delete with confirm modal). Uses useT() + Tailwind + inline SVGs (mirrors ArtifactCard.tsx style — no new icon dep).
3. Persistence — wrap chatRuntimeReducer with persistReducer + a createTransform that keeps only status === 'ready' entries on storage write (in_progress / failed stay session-scoped — should not survive a restart).
4. Delete flow — frontend deleteArtifact() service + removeArtifactForThread reducer; optimistic FE removal with rollback on RPC error. RPC openhuman.ai_delete_artifact already exists.
5. Above-composer card on ready — filter status === 'ready' OUT of the existing above-composer block (delegated to the panel). in_progress / failed keep showing there for live feedback.

Known backend gap (deferred to a follow-up)

ArtifactMeta (src/openhuman/artifacts/types.rs:80-102) has no thread_id field, and openhuman.ai_list_artifacts is workspace-scoped (only offset / limit). So a true cold-load-from-RPC for a single thread is not possible today. This issue ships persisted-slice + RPC fetch for the artifacts the slice knows about; a follow-up should persist thread_id into ArtifactMeta + add a thread_id filter param on ai_list_artifacts for full cold-load support.

Files (estimated)

File	New/Modified	Est. LOC
app/src/components/chat/ChatFilesPanel.tsx	NEW	200
app/src/components/chat/ChatFilesChip.tsx	NEW	90
app/src/components/chat/__tests__/ChatFilesPanel.test.tsx	NEW	220
app/src/components/chat/__tests__/ChatFilesChip.test.tsx	NEW	90
app/src/store/__tests__/chatRuntimeSlice.test.ts	MODIFIED	+80
app/src/services/__tests__/artifactDownloadService.test.ts	NEW	120
app/src/pages/Conversations.tsx	MODIFIED	+30 / -20
app/src/store/chatRuntimeSlice.ts	MODIFIED	+35
app/src/store/index.ts	MODIFIED	+40
app/src/services/artifactDownloadService.ts	MODIFIED	+35
app/src/lib/i18n/en.ts	MODIFIED	+12
app/src/lib/i18n/{ar,bn,de,es,fr,hi,id,it,ko,pl,pt,ru,zh-CN}.ts × 13	MODIFIED	+156

Total estimated: 1100–1700 LOC (FE-only; 0 Rust, 0 Tauri).

Acceptance criteria

• ChatFilesChip in header — paperclip + count badge appears in the chat header when at least one artifact exists for the active thread; hidden when count = 0; aria-label chat.files.chip.aria ("{count} files in this chat").
• ChatFilesPanel popover — opens on chip click; closes on Esc + click-outside; each row shows kind icon + title + human-readable size; Download / Reveal / Delete buttons per row.
• Delete with confirm — Delete row → confirm modal → optimistic remove from slice + RPC openhuman.ai_delete_artifact → on RPC error, re-insert + show toast chat.files.delete.failed.
• Cross-restart persistence — generated .pptx survives app quit + relaunch; only status === 'ready' entries are persisted (in_progress / failed are session-only via createTransform outbound filter).
• Above-composer card delegates ready to panel — the existing ArtifactCard render block keeps in_progress / failed cards visible there but hides ready cards (they live in the panel only). No double-rendering.
• Empty state — panel shows chat.files.panel.empty ("No files yet. Ask the agent to generate one.") when count = 0 (chip is hidden, so this is only seen if the panel is open while the last artifact is deleted).
• i18n parity — 12 new keys × 14 locales; pnpm i18n:check (parity) and pnpm i18n:english:check (no English leaks) both green for the new keys.
• Diff coverage ≥ 80% — the implementing PR meets the changed-lines coverage gate per .github/workflows/pr-ci.yml. New Vitest suites cover: panel populated render, optimistic delete + RPC rollback, download/reveal delegation, confirm-modal cancel path, chip hidden-when-zero, chip count badge, slice persist-transform filter.

Related

• Parent epic: Add multimodal support for files and PPT generation #1535 "Add multimodal support for files and PPT generation"
• Sub-tasks of Add multimodal support for files and PPT generation #1535:
  • Define artifact storage types, config, and persistence layer #2776 ✅ merged (artifact storage / artifacts domain)
  • Extend multimodal input to accept document and file attachments beyond images #2777 (multimodal file attachments) — PR feat(agent/multimodal): support [FILE:…] markers with text extraction (#2777) #2954 open
  • Add PPT generation tool using python-pptx in the Python runtime #2778 (presentation tool backed by python-pptx) — PR feat(tools): add generate_presentation tool (native rust engine, ppt-rs) (#2778) #3016 open
  • Surface artifact downloads and generation progress in the React UI #2779 (inline ArtifactCard + Tauri download) — PR feat(chat): ArtifactCard + Tauri download + backend artifact events (#2779) #3017 open ← this issue extends that work
  • Wire PPT generation and file attachments into agent definitions and error handling #2780 (orchestrator wiring) — pending
• Backend follow-up (separate, not part of this issue): persist thread_id into ArtifactMeta + add thread_id filter param on openhuman.ai_list_artifacts so artifacts from old sessions (pre-persistence) become listable.

[hemanth1999k]

Interested in picking this up. Holding off on opening a PR until #3017 merges — the substrate this layers on (the ArtifactCard, the artifactsByThread slice, artifactDownloadService, and the openhuman.ai_delete_artifact RPC) all live in that PR's branch rather than main today, so any work here would be unmergeable until then. Will start once #3017 lands. If anyone else wants to grab this in the meantime, no objection.

[oxoxDev]

@hemanth1999k thanks for flagging — this is already in flight as #3026, stacked on #3017 for exactly the reasons you called out (artifactsByThread slice, ArtifactCard, artifactDownloadService, openhuman.ai_delete_artifact RPC all live in #3017's branch, not on main). Your dependency analysis matches mine; PR will rebase onto main once #3017 lands.

If you're looking for adjacent #1535 work, a few unclaimed follow-ups surfaced during implementation:

1. ArtifactMeta.thread_id field + ai_list_artifacts thread filter param — flagged in feat(chat): Files in this chat — persistent per-chat artifact panel (#3024) #3026's PR body. Today the persisted Redux slice covers per-thread cold-load on app restart; this would close the gap for true cold-load-from-RPC (e.g. iOS client, cross-device sync).
2. Image embedding in slides via ppt-rs::add_image — pairs a Composio file fetch / artifact marker with a slide image placeholder. Useful for 'deck with chart screenshots' requests.
3. Per-slide source attribution (SlideSpec.sources: Vec<String> rendered as a 'Sources:' footer in speaker notes) — pairs with the grounding rule in Wire PPT generation and file attachments into agent definitions and error handling #2780 / feat(orchestrator): wire generate_presentation + require grounding (#2780) #3029.

Happy to open any of those as separate issues if you want to pick one up. Closing #3024 in favor of #3026.