CedarCopilot wants to merge 1 commit into staging from fix/table-read-result-pill
Live on prod, watching, 2 days left
Libra has verdicts on 0 of 1 tracked behaviors on prod; 1 is still being checked. Libra checks hourly for 3 days after each deploy.
create/import tool-result, mirroring attachTableToThread's context-chip gate , but read is the action a "show me the table" follow-up in a new conversation actually calls, and it still fell through to null. All the user saw was the tool-call's own inert Table: <path> argument row (describeToolCall), which looks like a chip but isn't clickable , while the agent's own text claimed "attached above as a chip."read to TableResultRenderer's gate. Tradeoff called out in the code comment: an incidental internal read (e.g. a row-scoped fan-out peeking at other rows) can now also surface a pill , acceptable since the common path this covers is the user-facing one.pnpm --filter @zero/mail run types , no errors in the changed file or new symbols (remaining errors are pre-existing generated-artifact gaps in a fresh worktree, unrelated to this change).🤖 Generated with Claude Code
The PR appears safe to merge, with a non-blocking transcript-clutter issue for table fan-out operations.
### Issue 1
apps/mail/modules/cedar-os/src/components/renderers/messageRenderers.tsx:359
Including every `read` result also exposes internal row-agent reads during a table `fill`. Each row agent reads before writing, and those tool results are added to the visible main-thread transcript, so one fill can display many redundant Open-table pills. Restrict this affordance to user-facing reads or suppress subagent read results.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
This PR extends table tool-result rendering so successful read operations display the existing clickable Open-table pill.
WriteDocumentResultRenderer for read results.set, add_rows, and fill excluded.Reviews (1) · Last reviewed commit: "fix(chat): the table pill also shows whe..."
Libra has not measured any production surfaces for this change yet.