refactor: update styling and remove unused status classes in AcpToolCallCard component#1169
Conversation
…allCard component
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f7ce1239ac
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| data-testid="acp-tool-call-card" | ||
| data-expanded={expanded ? 'true' : 'false'} | ||
| className="rounded-2xl border border-black/10 bg-surface-modal px-4 py-3 shadow-sm dark:border-white/10" | ||
| className="rounded-lg px-1 py-0.5" |
There was a problem hiding this comment.
Because this changes the user-visible ACP tool-card styling, it falls under /workspace/ClawX/AGENTS.md, which says: “Any user-visible UI change should include or update an Electron E2E spec in the same PR.” This commit only updates the two chat components, so the revised tool-call rendering has no Electron E2E coverage to catch regressions in the collapsed/expanded UI.
Useful? React with 👍 / 👎.
1 participant
Summary
Modify the tool component style.
![Uploading ScreenShot_2026-07-15_170334_430.png…]()
Related Issue(s)
Type of Change
Validation
Checklist