Improve attachment handling in the Studio Code composer#3925
Conversation
…ling' into stu-1892-improve-attachment-handling
sejas
left a comment
There was a problem hiding this comment.
Thanks for improving the previews of the attachments. I like the new preview on hover, and the auto-scroll in text files. I suggest making the ✖︎ remove icon more visible. Currently it's hard to spot. I tested the PR in dark and light modes.
@shaunandrews, I'm not approving this PR because I noticed that it modifies apps/hosted files, and I'm not sure if that's a desired goal of this PR.
| Before | After Dark | After Light |
|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
📊 Performance Test ResultsComparing 0306243 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
It was initially intentional, as I wanted to cover the hosted variant of the UI as well. But since that's all up the air still, and I'm not entirely sure how we'll handle attachments in that environment, I've removed those changes. The hosted version of the UI will simply continue to ignore attachments until we figure out the details there. |
…-1892-improve-attachment-handling
Ah sorry! You mentioned it before, but I wasn't seeing that behavior in the Agentic UI. I looked in the default UI and saw it there, and fixed it. Thanks for the review! |








Related issues
Proposed Changes
Screenshots
Testing Instructions
Validated locally:
npx eslint --fix ...npm test -- tools/common/ai/tests/composer-attachments.test.ts apps/studio/src/components/studio-code-session/composer/index.test.tsx apps/ui/src/data/core/connectors/web/index.test.tsnpm run typecheck/private/tmp/verify-studio-file-attachment-tiles.mjsPre-merge Checklist