Clean up some of the code

Author: Nathan Turinski

extension.bundle.ts

@@ -41,7 +41,7 @@ export interface IStartFuncProcessResult {
      * An async iterable stream of terminal output from the function host task.
      * This stream provides real-time access to the output of the `func host start` command,
      * allowing consumers to monitor host status, capture logs, and detect errors.
-     * 
+     *
      * The stream will be undefined if the host failed to start or if output streaming is not available.
      * Consumers should iterate over the stream asynchronously to read output lines as they are produced.
      * The stream remains active for the lifetime of the function host process.
@@ -169,7 +169,6 @@ async function startFuncTask(context: IActionContext, workspaceFolder: vscode.Wo
         const funcPort: string = await getFuncPortFromTaskOrProject(context, funcTask, workspaceFolder);
         let statusRequestTimeout: number = intervalMs;
         const maxTime: number = Date.now() + timeoutInSeconds * 1000;
-
         while (Date.now() < maxTime) {
             if (taskError !== undefined) {
                 throw taskError;
@@ -200,7 +199,6 @@ async function startFuncTask(context: IActionContext, workspaceFolder: vscode.Wo
                         }
                     }
                 }
-
             }
 
             await delay(intervalMs);

src/commands/pickFuncProcess.ts

@@ -5,43 +5,20 @@
 
 import { registerCommand, type IActionContext } from '@microsoft/vscode-azext-utils';
 import * as vscode from 'vscode';
+import { getRecentLogsPlainText } from '../funcCoreTools/funcHostErrorUtils';
 import { onRunningFuncTasksChanged, refreshFuncHostDebugContext, runningFuncTaskMap, type IRunningFuncTask } from '../funcCoreTools/funcHostTask';
 import { localize } from '../localize';
 import { stripAnsiControlCharacters } from '../utils/ansiUtils';
-import { openCopilotChat } from '../utils/copilotChat';
 import { FuncHostDebugViewProvider, type IHostErrorNode, type IHostTaskNode } from './FunctionHostDebugView';
 
 const viewId = 'azureFunctions.funcHostDebugView';
 
 function isHostTaskNode(node: unknown): node is IHostTaskNode {
-    if (!node || typeof node !== 'object') {
-        return false;
-    }
-
-    const n = node as Partial<IHostTaskNode>;
-    const scope = (n as { workspaceFolder?: unknown }).workspaceFolder;
-    const hasValidScope = typeof scope === 'object' || typeof scope === 'number';
-
-    return n.kind === 'hostTask'
-        && hasValidScope
-        && typeof n.portNumber === 'string'
-        && (n.cwd === undefined || typeof n.cwd === 'string');
+    return !!node && typeof node === 'object' && (node as IHostTaskNode).kind === 'hostTask';
 }
 
 function isHostErrorNode(node: unknown): node is IHostErrorNode {
-    if (!node || typeof node !== 'object') {
-        return false;
-    }
-
-    const n = node as Partial<IHostErrorNode>;
-    const scope = (n as { workspaceFolder?: unknown }).workspaceFolder;
-    const hasValidScope = typeof scope === 'object' || typeof scope === 'number';
-
-    return n.kind === 'hostError'
-        && hasValidScope
-        && typeof n.portNumber === 'string'
-        && typeof n.message === 'string'
-        && (n.cwd === undefined || typeof n.cwd === 'string');
+    return !!node && typeof node === 'object' && (node as IHostErrorNode).kind === 'hostError';
 }
 
 async function tryOpenDebugViewOnFirstFuncHostError(): Promise<void> {
@@ -74,16 +51,6 @@ async function tryOpenDebugViewOnFirstFuncHostError(): Promise<void> {
     }
 }
 
-function getRecentLogs(task: IRunningFuncTask | undefined, limit: number = 250): string {
-    const logs = task?.logs ?? [];
-    const recent = logs.slice(Math.max(0, logs.length - limit));
-    return recent.join('');
-}
-
-function getRecentLogsPlainText(task: IRunningFuncTask | undefined, limit: number = 250): string {
-    return stripAnsiControlCharacters(getRecentLogs(task, limit));
-}
-
 export function registerFunctionHostDebugView(context: vscode.ExtensionContext): void {
     const provider = new FuncHostDebugViewProvider();
 
@@ -167,7 +134,7 @@ export function registerFunctionHostDebugView(context: vscode.ExtensionContext):
             errorContext,
         ].filter((l): l is string => Boolean(l)).join('\n');
 
-        await openCopilotChat(prompt);
+        await vscode.commands.executeCommand('workbench.action.chat.open', { mode: 'agent', query: prompt });
     });
 
     registerCommand('azureFunctions.funcHostDebug.refresh', async (actionContext: IActionContext) => {

src/debug/registerFunctionHostDebugView.ts

@@ -4,6 +4,17 @@
  *--------------------------------------------------------------------------------------------*/
 
 import { stripAnsiControlCharacters } from '../utils/ansiUtils';
+import { type IRunningFuncTask } from './funcHostTask';
+
+export function getRecentLogs(task: IRunningFuncTask | undefined, limit: number = 250): string {
+    const logs = task?.logs ?? [];
+    const recent = logs.slice(Math.max(0, logs.length - limit));
+    return recent.join('');
+}
+
+export function getRecentLogsPlainText(task: IRunningFuncTask | undefined, limit: number = 250): string {
+    return stripAnsiControlCharacters(getRecentLogs(task, limit));
+}
 
 /**
  * Regex that matches a Functions-host timestamp prefix, e.g. `[2026-03-11T19:57:44.622Z]`.
@@ -21,21 +32,6 @@ function normalizeWhitespace(text: string): string {
     return text.replace(/\s+/g, ' ').trim();
 }
 
-export interface FuncHostErrorContextOptions {
-    /**
-     * Number of log lines to include before an error line
-     */
-    before?: number;
-    /**
-     * Number of log lines to include after an error line
-     */
-    after?: number;
-    /**
-     * Maximum number of log lines to return (keeps the most recent)
-     */
-    max?: number;
-}
-
 // Detect red/bright-red foreground in any of the common SGR forms:
 //   Basic 4-bit:    \x1b[31m  (red)  /  \x1b[91m  (bright red), with optional extra params
 //   256-color:      \x1b[38;5;1m (red) / \x1b[38;5;9m (bright red)
@@ -95,65 +91,3 @@ export function addErrorLinesFromChunk(errorLogs: string[], rawChunk: string): b
 
     return added;
 }
-
-/**
- * Extracts context for only a single relevant error line (as selected in the UI).
- *
- * @param errorMessage A plain-text error line (ANSI/control chars already removed).
- */
-export function extractFuncHostErrorContextForErrorMessage(
-    logs: readonly string[],
-    errorMessage: string,
-    options?: FuncHostErrorContextOptions
-): string[] {
-    const target = (errorMessage ?? '').trim();
-    if (!target) {
-        return [];
-    }
-
-    const before = options?.before ?? 5;
-    const after = options?.after ?? 15;
-    const max = options?.max ?? 250;
-
-    let bestIndex = -1;
-    let bestScore = 0;
-
-    for (let i = 0; i < logs.length; i++) {
-        const line = logs[i];
-        if (!isFuncHostErrorLog(line)) {
-            continue;
-        }
-
-        const plain = stripAnsiControlCharacters(line).trim();
-        if (!plain) {
-            continue;
-        }
-
-        let score = 0;
-        if (plain === target) {
-            score = 2;
-        } else if (plain.includes(target) || target.includes(plain)) {
-            score = 1;
-        }
-
-        if (score > 0 && (score > bestScore || (score === bestScore && i > bestIndex))) {
-            bestScore = score;
-            bestIndex = i;
-        }
-    }
-
-    if (bestIndex < 0) {
-        return [];
-    }
-
-    const start = Math.max(0, bestIndex - before);
-    const end = Math.min(logs.length - 1, bestIndex + after);
-
-    const result = logs.slice(start, end + 1);
-
-    if (result.length > max) {
-        return result.slice(result.length - max);
-    }
-
-    return result;
-}