Hide internal chat tool messages#50
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e4f0ee5590
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| .page-content-chat .adclaw-operate-card { | ||
| display: none !important; |
There was a problem hiding this comment.
Keep approval cards visible on chat pages
This selector hides every adclaw-operate-card, not just reasoning/tool trace rows, so approval UIs are also suppressed; when an MCP/tool flow emits an approval request card, users cannot see or act on it and the run can stall waiting for input. This affects any environment where approval-gated tool execution is enabled.
Useful? React with 👍 / 👎.
| .join("\n"); | ||
| } | ||
|
|
||
| const INTERNAL_OUTPUT_TYPES = new Set([ |
There was a problem hiding this comment.
Filter all internal runtime event message types
The internal-type filter is incomplete: it excludes some tool/runtime events but leaves other non-chat event types unfiltered, so shouldRenderOutputMessage can still surface internal execution traffic in history when those message types appear. To match the stated behavior, expand this set (or switch to an explicit allowlist of user-visible message types).
Useful? React with 👍 / 👎.
Summary
Scope
Validation