Revert 2e660819 #3154
Revert 2e660819 #3154
Conversation
|
dosubot
bot
added
the
size:L
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
| <div className="rounded-full size-1.5 bg-lime-400" /> | ||
| <div>Running</div> | ||
| {status.pid && <div className="whitespace-nowrap">(PID: {status.pid})</div>} | ||
| <Button |
There was a problem hiding this comment.
Consider adding an aria-label (e.g. aria-label="Abort command") to the abort Button for improved accessibility.
| </div> | ||
| )} | ||
| {lines.length > 0 && ( | ||
| <Button variant="ghost" size="icon" onClick={() => setIsExpanded(!isExpanded)}> |
There was a problem hiding this comment.
Consider adding an aria-expanded attribute to the toggle Button so that its expanded state is communicated to assistive technologies.
| <Button variant="ghost" size="icon" onClick={() => setIsExpanded(!isExpanded)}> | |
| <Button variant="ghost" size="icon" aria-expanded={isExpanded} onClick={() => setIsExpanded(!isExpanded))}> |
* Fix unit test set-up * run tests on CI * update failing tests * Update node Update the coverage job node version
Context
Implementation
Screenshots
How to Test
Get in Touch
Important
Reverts commit 2e66081, undoing changes to command execution, terminal process management, and related UI components.
executeCommandTool()inexecuteCommandTool.ts, affecting command execution flow and error handling.ExecaTerminalProcessinExecaTerminalProcess.ts, affecting process management and signal handling.commandExecutionStatusSchemainindex.ts, affecting status tracking.ChatRow.tsxandCommandExecution.tsx, affecting UI display of command execution results.combineCommandSequences.ts, affecting command and output message handling.This description was created by
for 8a80ccc. You can customize this summary. It will automatically update as commits are pushed.