Attachment chips in user messages now open the doc viewer
nwhitehouse fixes a silent UX gap: the filename chips shown next to a user prompt were display-only. Two commits make them clickable when a `document_id` is present, with the second commit catching a missed render path.
The first commit (e67b072f) adds an onOpenDocument prop to UserMessage and conditionally renders each chip as a <button> when both document_id and the handler are present. Clicking opens the side-panel doc viewer via the same openDocument call the assistant's "Read " blocks use. versionId is left null at the chip level; DocView resolves to the document's current version at view time.
The second commit (4f480f20) came 30 minutes later. UserMessage is rendered from two routes - /assistant via ChatView and /projects/[id]/assistant/chat/[chatId] directly - and only ChatView got the handler in the first pass. The project chat page needed the same wire-up.
That missed-route pattern is the thing to remember when porting. Any component rendered from multiple top-level routes needs the handler passed from each one. It's easy to test one and assume coverage.
Spotted something wrong? Or know the PR text has fresher detail than the writeup above?