fix: show delete button consistently on task cards (#6494)

- Remove size condition from delete button visibility
- Keep size display conditional (only show when size > 0)
- Update tests to reflect new behavior
- Add tests for undefined size and no item cases

Author: MuriloFP

webview-ui/src/components/chat/TaskActions.tsx

@@ -36,7 +36,7 @@ export const TaskActions = ({ item, buttonsDisabled }: TaskActionsProps) => {
 					onClick={(e) => copyWithFeedback(item.task, e)}
 				/>
 			)}
-			{!!item?.size && item.size > 0 && (
+			{item && (
 				<>
 					<div className="flex items-center">
 						<IconButton
@@ -53,7 +53,11 @@ export const TaskActions = ({ item, buttonsDisabled }: TaskActionsProps) => {
 								}
 							}}
 						/>
-						<span className="ml-1 text-xs text-vscode-foreground opacity-85">{prettyBytes(item.size)}</span>
+						{!!item.size && item.size > 0 && (
+							<span className="ml-1 text-xs text-vscode-foreground opacity-85">
+								{prettyBytes(item.size)}
+							</span>
+						)}
 					</div>
 					{deleteTaskId && (
 						<DeleteTaskDialog

webview-ui/src/components/chat/__tests__/TaskActions.spec.tsx

@@ -361,10 +361,27 @@ describe("TaskActions", () => {
 			expect(screen.getByText("1024 B")).toBeInTheDocument()
 		})
 
-		it("does not render delete button when item has no size", () => {
+		it("renders delete button even when item has no size", () => {
 			const itemWithoutSize = { ...mockItem, size: 0 }
 			render(<TaskActions item={itemWithoutSize} buttonsDisabled={false} />)
 
+			const deleteButton = screen.queryByLabelText("Delete Task (Shift + Click to skip confirmation)")
+			expect(deleteButton).toBeInTheDocument()
+			// File size should not be displayed when size is 0
+			expect(screen.queryByText("0 B")).not.toBeInTheDocument()
+		})
+
+		it("renders delete button when item has undefined size", () => {
+			const itemWithUndefinedSize = { ...mockItem, size: undefined }
+			render(<TaskActions item={itemWithUndefinedSize} buttonsDisabled={false} />)
+
+			const deleteButton = screen.queryByLabelText("Delete Task (Shift + Click to skip confirmation)")
+			expect(deleteButton).toBeInTheDocument()
+		})
+
+		it("does not render delete button when item is undefined", () => {
+			render(<TaskActions item={undefined} buttonsDisabled={false} />)
+
 			const deleteButton = screen.queryByLabelText("Delete Task (Shift + Click to skip confirmation)")
 			expect(deleteButton).not.toBeInTheDocument()
 		})