for tasks with problem matchers, don't pass back output, only succinct reply (microsoft#269568)

fixes microsoft#269563

Author: meganrogge

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/taskHelpers.ts

@@ -224,9 +224,8 @@ export async function taskProblemPollFn(execution: IExecution, token: Cancellati
 							? new Range(marker.startLineNumber, marker.startColumn, marker.endLineNumber, marker.endColumn)
 							: undefined
 					});
-					const label: string = uri ? uri.path.split('/').pop() ?? uri.toString() : '';
 					const message = marker.message ?? '';
-					problemList.push(`Problem: ${message} in ${label} coming from ${owner}`);
+					problemList.push(`Problem: ${message} in ${uri.fsPath} coming from ${owner} starting on line ${marker.startLineNumber}${marker.startColumn ? `, column ${marker.startColumn} and ending on line ${marker.endLineNumber}${marker.endColumn ? `, column ${marker.endColumn}` : ''}` : ''}`);
 				}
 			}
 			if (problemList.length === 0) {

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/monitoring/outputMonitor.ts

@@ -105,6 +105,7 @@ export class OutputMonitor extends Disposable implements IOutputMonitor {
 
 		let modelOutputEvalResponse;
 		let resources;
+		let output;
 
 		let extended = false;
 		try {
@@ -134,6 +135,7 @@ export class OutputMonitor extends Disposable implements IOutputMonitor {
 						} else {
 							resources = idleResult.resources;
 							modelOutputEvalResponse = idleResult.modelOutputEvalResponse;
+							output = idleResult.output;
 						}
 						break;
 					}
@@ -149,7 +151,7 @@ export class OutputMonitor extends Disposable implements IOutputMonitor {
 		} finally {
 			this._pollingResult = {
 				state: this._state,
-				output: this._execution.getOutput(),
+				output: output ?? this._execution.getOutput(),
 				modelOutputEvalResponse: token.isCancellationRequested ? 'Cancelled' : modelOutputEvalResponse,
 				pollDurationMs: Date.now() - pollStartTime,
 				resources
@@ -161,7 +163,7 @@ export class OutputMonitor extends Disposable implements IOutputMonitor {
 	}
 
 
-	private async _handleIdleState(token: CancellationToken): Promise<{ resources?: ILinkLocation[]; modelOutputEvalResponse?: string; shouldContinuePollling: boolean }> {
+	private async _handleIdleState(token: CancellationToken): Promise<{ resources?: ILinkLocation[]; modelOutputEvalResponse?: string; shouldContinuePollling: boolean; output?: string }> {
 		const confirmationPrompt = await this._determineUserInputOptions(this._execution, token);
 
 		if (confirmationPrompt?.detectedRequestForFreeFormInput) {
@@ -208,7 +210,7 @@ export class OutputMonitor extends Disposable implements IOutputMonitor {
 		const custom = await this._pollFn?.(this._execution, token, this._taskService);
 		const resources = custom?.resources;
 		const modelOutputEvalResponse = await this._assessOutputForErrors(this._execution.getOutput(), token);
-		return { resources, modelOutputEvalResponse, shouldContinuePollling: false };
+		return { resources, modelOutputEvalResponse, shouldContinuePollling: false, output: custom?.output };
 	}
 
 	private async _handleTimeoutState(command: string, invocationContext: IToolInvocationContext | undefined, extended: boolean, token: CancellationToken): Promise<boolean> {