[allozaur]

Overview

A tool call that was stopped before it returned now shows as interrupted, instead of spinning on "Executing command..." for good.

Demo

Recorded at 39d6e3f; click the preview for the full video.

Additional information

Calls skipped by a steering message are unchanged: they already get a "Tool execution was interrupted by a new user message." result, shown as plain output.

Cause

The chat-level isStreaming flag was passed to every assistant message, so an older message treated its unresolved call as still pending while a newer reply streamed. Separately, exec_shell_command creates its result message before running, so stopping it mid-run left an empty result that rendered as a bare title with no status.

What changed

• ChatMessageAgenticContent only treats the last assistant message as streaming.
• ToolCallBlock marks a finished call without a result as interrupted, shown next to the existing failed / incomplete labels. exec_shell_command opts in to treating an empty result the same way.
• The exec_shell_command block shows a stop icon with "Command interrupted before it returned output" instead of the spinner.

Testing

• tests/client/exec-shell-command-streaming.svelte.test.ts covers a call left without a result after streaming stops, a command stopped before any output, and an earlier message while a newer reply streams.
• Tests that render a live message now pass isLastAssistantMessage: true.
• Checked in the browser against a local model: a sleep 40 command stopped mid-run shows the interrupted label.

Demo script

test.describe('pr-31314', () => {
	test('demo', async ({ demo, page }) => {
		test.setTimeout(240_000);

		const input = page.getByRole('textbox', { name: 'Type a message...' });
		const stop = page.getByRole('button', { exact: true, name: 'Stop' });
		const allow = page.getByRole('button', { name: 'Allow once' });
		const call = page.locator('[data-slot="collapsible-trigger"]').filter({ hasText: 'sleep 40' });

		await demo.type(
			input,
			'Run this exact shell command with the exec tool and nothing else: sleep 40 && echo done',
			300
		);
		await demo.press('Enter', 300);

		if (await allow.waitFor({ timeout: 120_000 }).then(() => true, () => false)) {
			await demo.click(allow, 600);
		}

		await expect(page.getByText('Executing command...').first()).toBeVisible({ timeout: 60_000 });
		await demo.caption('A long-running command is executing', 2000);

		await demo.click(stop, 800);
		await expect(call.getByText('interrupted')).toBeVisible({ timeout: 10_000 });
		await demo.caption('Stopping it marks the call as interrupted', 2400);

		await demo.click(call, 800);
		await expect(page.getByText('Command interrupted before it returned output')).toBeVisible();
		await demo.caption('Instead of a spinner that never ends', 2400);

		await demo.type(input, 'Never mind, just say hi', 300);
		await demo.press('Enter', 300);
		await expect(stop).toBeVisible({ timeout: 60_000 });
		await expect(call.getByText('interrupted')).toBeVisible();
		await demo.caption('It stays interrupted while the next reply streams', 2600);
	});
});

Requirements

• I have read and agree with the contributing guidelines
• AI usage disclosure: YES, Claude (Opus 5.5 Medium)

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.