chrome-devtools-frontend 1.0.1638082 → 1.0.1640418

package/front_end/models/ai_assistance/AiConversation.ts

@@ -13,6 +13,7 @@ import type * as NetworkTimeCalculator from '../network_time_calculator/network_
 
 import {AccessibilityAgent, AccessibilityContext} from './agents/AccessibilityAgent.js';
 import {
+  type AgentOptions,
   type AiAgent,
   type AllowedOriginResult,
   type ContextDetail,
@@ -29,6 +30,7 @@ import {NetworkAgent, RequestContext} from './agents/NetworkAgent.js';
 import {PerformanceAgent, PerformanceTraceContext} from './agents/PerformanceAgent.js';
 import {StorageAgent, StorageContext} from './agents/StorageAgent.js';
 import {NodeContext, StylingAgent} from './agents/StylingAgent.js';
+import {AiAgent2} from './AiAgent2.js';
 import {AiHistoryStorage, ConversationType, type SerializedConversation} from './AiHistoryStorage.js';
 import type {ChangeManager} from './ChangeManager.js';
 
@@ -356,35 +358,27 @@ export class AiConversation {
       allowedOrigin: this.allowedOrigin,
       history,
     };
+
+    this.#agent = Root.Runtime.hostConfig.devToolsAiV2Architecture?.enabled ? new AiAgent2(options) :
+                                                                              this.#createV1Agent(type, options);
+  }
+
+  #createV1Agent(type: ConversationType, options: AgentOptions): AiAgent<unknown> {
     switch (type) {
-      case ConversationType.STYLING: {
-        this.#agent = new StylingAgent(options);
-        break;
-      }
-      case ConversationType.NETWORK: {
-        this.#agent = new NetworkAgent(options);
-        break;
-      }
-      case ConversationType.FILE: {
-        this.#agent = new FileAgent(options);
-        break;
-      }
-      case ConversationType.PERFORMANCE: {
-        this.#agent = new PerformanceAgent(options);
-        break;
-      }
-      case ConversationType.ACCESSIBILITY: {
-        this.#agent = new AccessibilityAgent(options);
-        break;
-      }
-      case ConversationType.STORAGE: {
-        this.#agent = new StorageAgent(options);
-        break;
-      }
-      case ConversationType.NONE: {
-        this.#agent = new ContextSelectionAgent(options);
-        break;
-      }
+      case ConversationType.STYLING:
+        return new StylingAgent(options);
+      case ConversationType.NETWORK:
+        return new NetworkAgent(options);
+      case ConversationType.FILE:
+        return new FileAgent(options);
+      case ConversationType.PERFORMANCE:
+        return new PerformanceAgent(options);
+      case ConversationType.ACCESSIBILITY:
+        return new AccessibilityAgent(options);
+      case ConversationType.STORAGE:
+        return new StorageAgent(options);
+      case ConversationType.NONE:
+        return new ContextSelectionAgent(options);
       default:
         Platform.assertNever(type, 'Unknown conversation type');
     }

package/front_end/models/ai_assistance/AiOrigins.ts

@@ -0,0 +1,46 @@
+// Copyright 2026 The Chromium Authors
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import * as Common from '../../core/common/common.js';
+import type * as Platform from '../../core/platform/platform.js';
+
+/**
+ * Returns true if the origin is considered opaque and should be blocked from
+ * AI assistance to prevent potential data leakage.
+ *
+ * @see https://crbug.com/513732588
+ */
+export function isOpaqueOrigin(origin: string): boolean {
+  /**
+   * Origins starting with 'about' (like about:blank or about:srcdoc) are
+   * considered opaque. 'about://' is the sentinel used by DevTools
+   * ParsedURL.securityOrigin() for these.
+   */
+  return origin === 'null' || origin === 'data:' || origin.startsWith('about') || origin.startsWith('detached');
+}
+
+/**
+ * Extracts the origin from a context URL or identifier.
+ * Handles special cases like "detached" nodes and trace identifiers.
+ */
+export function extractContextOrigin(contextURL: string): string {
+  if (isOpaqueOrigin(contextURL)) {
+    return contextURL;
+  }
+  if (contextURL.startsWith('trace-')) {
+    return contextURL;
+  }
+  return Common.ParsedURL.ParsedURL.extractOrigin(contextURL as Platform.DevToolsPath.UrlString);
+}
+
+/**
+ * Determines if two origins are equivalent and safe to be used together.
+ * Opaque origins are never equivalent to anything, not even themselves.
+ */
+export function areOriginsEquivalent(origin1: string, origin2: string): boolean {
+  if (isOpaqueOrigin(origin1) || isOpaqueOrigin(origin2)) {
+    return false;
+  }
+  return origin1 === origin2;
+}

package/front_end/models/ai_assistance/README.md

@@ -22,6 +22,22 @@ When the user begins a conversation with the AI, we want to include information
 
 To deal with the work to take an object that is the AI agent's context and turn it into a text representation that can be sent to the AI, we use _formatters_. These are typically classes with `static` methods that can take in objects and return their text representation that we want to pass to the AI.
 
+## Future Architecture (V2) - WIP
+
+We are currently working on migrating the DevTools AI Assistance from a siloed multi-agent architecture to a unified, skill-based single-agent architecture (`AIAgent2`).
+
+In this new architecture:
+- A single agent (`AIAgent2`) handles multiple domains.
+- Capabilities are defined as **Skills** in Markdown files.
+- The agent can dynamically load skills as needed via a `learnSkill` tool.
+
+This work is currently in progress and behind a feature flag.
+
+### Skills Build System
+
+To support dynamic loading of skills, we generate JavaScript files from Markdown files containing skill definitions.
+We use a nested `BUILD.gn` file in the `skills/` subdirectory specifically for this purpose. This ensures that GN's `target_gen_dir` points to `gen/front_end/models/ai_assistance/skills/`, placing the generated `.skill.js` files in the same relative structure as their source `.md` files. This allows TypeScript files in the `skills/` directory (like `map.ts`) to import the generated files using relative paths (e.g., `./styling.skill.js`) seamlessly.
+
 ## Performance specific documentation
 
 ### `TimelineUtils.AIContext.AgentFocus`

package/front_end/models/ai_assistance/StorageItem.ts

@@ -2,35 +2,39 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-interface StorageItemOriginOnly {
-  origin: string;
-  storageKey?: string;
-  storageType?: never;
-  key?: never;
+export class StorageItem {
+  constructor(
+      /**
+       * The origin of the top-level primary page target being inspected.
+       * Used to restrict AI agent tools from accessing unauthorized pages.
+       */
+      readonly primaryTargetOrigin: string,
+      /**
+       * The origin of the selected storage or cookie item (if any).
+       * If no item is selected, this is the same as primaryTargetOrigin.
+       */
+      readonly origin: string,
+  ) {
+  }
 }
 
-interface StorageItemWithKey {
-  origin: string;
-  storageKey?: string;
-  storageType: 'cookie'|'localStorage'|'sessionStorage';
-  key: string;
+export class DOMStorageItem extends StorageItem {
+  constructor(
+      primaryTargetOrigin: string,
+      origin: string,
+      /** The storage key partition identifier used by the browser storage engine. */
+      readonly storageKey: string,
+      /** The sub-category of DOM storage: 'localStorage' or 'sessionStorage'. */
+      readonly type: string,
+      /** The optional specific key of the selected item in this storage partition. */
+      readonly key?: string,
+  ) {
+    super(primaryTargetOrigin, origin);
+  }
 }
 
-export type StorageItemData = StorageItemOriginOnly|StorageItemWithKey;
-
-// The StorageItem is used as context for the Ai Assistance.
-// If the user selects a row in e.g. the cookies table, the storageType and key
-// will be populated.
-export class StorageItem {
-  readonly origin: string;
-  readonly storageKey?: string;
-  readonly storageType?: 'cookie'|'localStorage'|'sessionStorage';
-  readonly key?: string;
-
-  constructor(data: StorageItemData) {
-    this.origin = data.origin;
-    this.storageKey = data.storageKey;
-    this.storageType = data.storageType;
-    this.key = data.key;
+export class CookieItem extends StorageItem {
+  constructor(primaryTargetOrigin: string, origin: string, readonly name?: string) {
+    super(primaryTargetOrigin, origin);
   }
 }

package/front_end/models/ai_assistance/agents/AccessibilityAgent.ts

@@ -93,8 +93,8 @@ export class AccessibilityContext extends ConversationContext<LHModel.ReporterTy
     return this.#lh.finalUrl ?? this.#lh.finalDisplayedUrl;
   }
 
-  override getOrigin(): string {
-    return new URL(this.#url()).origin;
+  override getURL(): string {
+    return this.#url();
   }
 
   override getItem(): LHModel.ReporterTypes.ReportJSON {

package/front_end/models/ai_assistance/agents/AiAgent.ts

@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 import * as Host from '../../../core/host/host.js';
+import type {UrlString} from '../../../core/platform/DevToolsPath.js';
 import * as Root from '../../../core/root/root.js';
 import type * as SDK from '../../../core/sdk/sdk.js';
 import type * as Protocol from '../../../generated/protocol.js';
@@ -10,6 +11,7 @@ import type * as LHModel from '../../lighthouse/lighthouse.js';
 import type * as TextUtils from '../../text_utils/text_utils.js';
 import type * as Trace from '../../trace/trace.js';
 import type * as Workspace from '../../workspace/workspace.js';
+import {areOriginsEquivalent, extractContextOrigin, isOpaqueOrigin} from '../AiOrigins.js';
 import {debugLog, isStructuredLogEnabled} from '../debug.js';
 
 const MAX_SUGGESTION_LENGTH = 200;
@@ -36,21 +38,6 @@ export const enum ErrorType {
   CROSS_ORIGIN = 'cross-origin'
 }
 
-/**
- * Returns true if the origin is considered opaque and should be blocked from
- * AI assistance to prevent potential data leakage.
- *
- * @see https://crbug.com/513732588
- */
-export function isOpaqueOrigin(origin: string): boolean {
-  /**
-   * Origins starting with 'about' (like about:blank or about:srcdoc) are
-   * considered opaque. 'about://' is the sentinel used by DevTools
-   * ParsedURL.securityOrigin() for these.
-   */
-  return origin === 'null' || origin === 'data:' || origin.startsWith('about') || origin.startsWith('detached');
-}
-
 export const enum MultimodalInputType {
   SCREENSHOT = 'screenshot',
   UPLOADED_IMAGE = 'uploaded-image',
@@ -197,10 +184,14 @@ export interface ConversationSuggestion {
 export type ConversationSuggestions = [ConversationSuggestion, ...ConversationSuggestion[]];
 
 export abstract class ConversationContext<T> {
-  abstract getOrigin(): string;
+  abstract getURL(): string;
   abstract getItem(): T;
   abstract getTitle(): string;
 
+  getOrigin(): string {
+    return extractContextOrigin(this.getURL());
+  }
+
   /**
    * Returns true if this data context (e.g., a DOM node or Network Request) is
    * allowed to be included in a conversation that is locked to the provided
@@ -214,17 +205,14 @@ export abstract class ConversationContext<T> {
    * If undefined, the conversation has not yet been locked to an origin.
    */
   isOriginAllowed(establishedOrigin: string|undefined): boolean {
-    const dataOrigin = this.getOrigin();
-    // Opaque origins are never allowed to be used as context.
-    if (isOpaqueOrigin(dataOrigin)) {
-      return false;
-    }
+    const origin = this.getOrigin();
     // If no origin is established yet, this context will be the one to lock the conversation.
+    // Opaque origins are never allowed to be used as context.
     if (!establishedOrigin) {
-      return true;
+      return !isOpaqueOrigin(origin);
     }
     // Only allow data that matches the origin the conversation is already locked to.
-    return dataOrigin === establishedOrigin;
+    return areOriginsEquivalent(origin, establishedOrigin);
   }
 
   /**
@@ -321,6 +309,13 @@ export interface SourceFileAiWidget {
   };
 }
 
+export interface SourceFilesListAiWidget {
+  name: 'SOURCE_FILES_LIST';
+  data: {
+    uiSourceCodes: Workspace.UISourceCode.UISourceCode[],
+  };
+}
+
 export interface LighthouseReportAiWidget {
   name: 'LIGHTHOUSE_REPORT';
   data: {
@@ -344,9 +339,21 @@ export interface NetworkRequestGeneralHeadersAiWidget {
   };
 }
 
-export type AiWidget = ComputedStyleAiWidget|CoreVitalsAiWidget|StylePropertiesAiWidget|DomTreeAiWidget|
-    PerformanceTraceAiWidget|PerfInsightAiWidget|TimelineRangeSummaryAiWidget|BottomUpTreeAiWidget|SourceFileAiWidget|
-    LighthouseReportAiWidget|TimelineEventSummaryAiWidget|NetworkRequestGeneralHeadersAiWidget;
+// If the line and column are not provided, we assume the whole file was sent to the agent.s
+export interface SourceCodeAiWidget {
+  name: 'SOURCE_CODE';
+  data: {
+    url: UrlString,
+    code: string,
+    line?: number,
+    column?: number,
+  };
+}
+
+export type AiWidget =
+    ComputedStyleAiWidget|CoreVitalsAiWidget|StylePropertiesAiWidget|DomTreeAiWidget|PerformanceTraceAiWidget|
+    PerfInsightAiWidget|TimelineRangeSummaryAiWidget|BottomUpTreeAiWidget|SourceFileAiWidget|LighthouseReportAiWidget|
+    TimelineEventSummaryAiWidget|NetworkRequestGeneralHeadersAiWidget|SourceCodeAiWidget|SourceFilesListAiWidget;
 
 export type FunctionCallHandlerResult<Result> = {
   requiresApproval: true,
@@ -409,6 +416,13 @@ interface AidaFetchResult {
   rpcId?: Host.AidaClient.RpcGlobalId;
 }
 
+class CrossOriginError extends Error {
+  constructor() {
+    super('Cross-origin navigation detected');
+    this.name = 'CrossOriginError';
+  }
+}
+
 /**
  * AiAgent is a base class for implementing an interaction with AIDA
  * that involves one or more requests being sent to AIDA optionally
@@ -504,6 +518,14 @@ export abstract class AiAgent<T> {
     this.#facts.clear();
   }
 
+  /**
+   * Clears any subclass-specific caches. This is called when a run encounters
+   * an error (e.g., cross-origin navigation, abort, or execution error) to
+   * prevent unvalidated cached data from being replayed in subsequent runs.
+   */
+  clearCache(): void {
+  }
+
   popPendingMultimodalInput(): MultimodalInput|undefined {
     return undefined;
   }
@@ -778,11 +800,6 @@ export abstract class AiAgent<T> {
                   },
               );
 
-          if ('result' in result && result.result === 'BLOCKED_CROSS_ORIGIN') {
-            yield this.#createErrorResponse(ErrorType.CROSS_ORIGIN);
-            break;
-          }
-
           if (options.signal?.aborted) {
             yield this.#createErrorResponse(ErrorType.ABORT);
             break;
@@ -808,6 +825,10 @@ export abstract class AiAgent<T> {
           };
           request = this.buildRequest(query, Host.AidaClient.Role.ROLE_UNSPECIFIED);
         } catch (err) {
+          if (err instanceof CrossOriginError) {
+            yield this.#createErrorResponse(ErrorType.CROSS_ORIGIN);
+            break;
+          }
           debugLog('Error handling function call', err);
           yield this.#createErrorResponse(ErrorType.UNKNOWN);
           break;
@@ -876,8 +897,19 @@ export abstract class AiAgent<T> {
       }
     }
 
+    const isOriginBlocked = (): boolean => {
+      const allowedOriginResult = this.#allowedOrigin?.();
+      return Boolean(allowedOriginResult && 'blocked' in allowedOriginResult);
+    };
+
     let result = await call.handler(args, options);
 
+    // Check 1: After first handler execution.
+    // Navigation could have occurred during the async handler execution.
+    if (isOriginBlocked()) {
+      throw new CrossOriginError();
+    }
+
     if ('requiresApproval' in result) {
       if (code) {
         yield {
@@ -926,17 +958,21 @@ export abstract class AiAgent<T> {
       // Re-check allowed origin after the approval await to prevent a TOCTOU (Time-of-Check
       // to Time-of-Use) race condition where the page might have navigated cross-origin
       // while the user was confirming the action.
-      const allowedOriginResult = this.#allowedOrigin?.();
-      if (allowedOriginResult && 'blocked' in allowedOriginResult) {
-        return {
-          result: 'BLOCKED_CROSS_ORIGIN',
-        };
+      // Check 2: After waiting for user approval.
+      if (isOriginBlocked()) {
+        throw new CrossOriginError();
       }
 
       result = await call.handler(args, {
         ...options,
         approved: true,
       });
+
+      // Check 3: After second handler execution (approved run).
+      // Navigation could have occurred during the async execution of the approved action.
+      if (isOriginBlocked()) {
+        throw new CrossOriginError();
+      }
     }
 
     if ('result' in result) {
@@ -1012,6 +1048,7 @@ export abstract class AiAgent<T> {
 
   #createErrorResponse(error: ErrorType): ResponseData {
     this.#removeLastRunParts();
+    this.clearCache();
     if (error !== ErrorType.ABORT) {
       Host.userMetrics.actionTaken(Host.UserMetrics.Action.AiAssistanceError);
     }

package/front_end/models/ai_assistance/agents/ContextSelectionAgent.snapshot.txt

@@ -91,7 +91,7 @@ Content:
     },
     {
       "name": "performanceRecordAndReload",
-      "description": "Records a new performance trace. Use this to measure and debug performance metrics and Core Web Vitals like Largest Contentful Paint (LCP), Interaction to Next Paint (INP), and Cumulative Layout Shift (CLS).",
+      "description": "Records a new performance trace. Use this to measure, analyze, and debug page performance, general performance issues, performance metrics, and Core Web Vitals like Largest Contentful Paint (LCP), Interaction to Next Paint (INP), and Cumulative Layout Shift (CLS).",
       "parameters": {
         "type": 6,
         "description": "",
@@ -102,7 +102,7 @@ Content:
     },
     {
       "name": "runLighthouseAudits",
-      "description": "Records a Lighthouse audit on the current page. Use this to debug accessibility, SEO, and best practices. (For performance metrics like LCP, use performanceRecordAndReload instead).",
+      "description": "Records a Lighthouse audit on the current page. Use this to debug accessibility, SEO, and best practices. (For any performance-related questions or performance issues, do NOT use this; use performanceRecordAndReload instead).",
       "parameters": {
         "type": 6,
         "description": "",

package/front_end/models/ai_assistance/agents/ContextSelectionAgent.ts

@@ -12,6 +12,7 @@ import * as Logs from '../../logs/logs.js';
 import * as NetworkTimeCalculator from '../../network_time_calculator/network_time_calculator.js';
 import type * as Trace from '../../trace/trace.js';
 import * as Workspace from '../../workspace/workspace.js';
+import {isOpaqueOrigin} from '../AiOrigins.js';
 import {debugLog} from '../debug.js';
 
 import {AccessibilityContext} from './AccessibilityAgent.js';
@@ -20,7 +21,6 @@ import {
   AiAgent,
   type AllowedOriginResult,
   type ContextResponse,
-  isOpaqueOrigin,
   type RequestOptions,
 } from './AiAgent.js';
 import {FileContext} from './FileAgent.js';
@@ -35,19 +35,26 @@ const lockedString = i18n.i18n.lockedString;
  * chrome_preambles.gcl). Sync local changes with the server-side.
  */
 const preamble = `
-You are a Web Development Assistant integrated into Chrome DevTools. Your tone is educational, supportive, and technically precise.
-You aim to help developers of all levels, prioritizing teaching web concepts as the primary entry point for any solution.
+You are an advanced Web Development Assistant and AI routing agent integrated into Chrome DevTools. Your tone is educational, supportive, and technically precise. You aim to help developers of all levels, prioritizing teaching web concepts as the primary entry point for any solution.
+
+Your role is to understand the user's query, identify the appropriate specialized agent to handle it, and select the relevant context from the page to assist that agent.
+
+# Workflow
+1.  **Analyze**: Understand the user's intent and what they are trying to achieve.
+2.  **Classify**: Determine which specialized agent is best suited for the task (e.g., StylingAgent for CSS/styling issues, NetworkAgent for network requests, FileAgent for source files, PerformanceAgent for performance details, AccessibilityAgent for accessibility reports, or StorageAgent for storage issues).
+3.  **Gather Context**: Identify what information the specialized agent will need. Proactively use your tools to find and select this context (e.g., finding the relevant DOM node, network request, file, or performance trace). Always try to select a single specific context before answering the question.
+4.  **Delegate**: Once context is selected, hand over to the specialized agent. If you are unable to delegate or gather more information, provide a comprehensive guide on how to fix the issue using Chrome DevTools, explaining how and why, or suggest any panel/flow that may help.
 
 # Considerations
 * Determine what is the domain of the question - styling, network, sources, performance or other part of DevTools.
-* For questions about web performance metrics (e.g., LCP, INP, CLS) or page speed, use performanceRecordAndReload to record a performance trace.
-* Proactively try to gather additional data. If a select specific data can be selected, select one.
-* Always try select single specific context before answering the question.
+* For questions about performance (e.g., general performance issues, page speed, performance metrics like LCP, INP, CLS), use performanceRecordAndReload to record a performance trace.
+* Proactively try to gather additional data. If a specific piece of data can be selected, select it.
+* Always try to select a single specific context before answering the question.
 * Avoid making assumptions without sufficient evidence, and always seek further clarification if needed.
 * When presenting solutions, clearly distinguish between the primary cause and contributing factors.
 * Please answer only if you are sure about the answer. Otherwise, explain why you're not able to answer.
 * If you are unable to gather more information provide a comprehensive guide to how to fix the issue using Chrome DevTools and explain how and why.
-* You can suggest any panel or flow in Chrome DevTools that may help the user out
+* You can suggest any panel or flow in Chrome DevTools that may help the user out.
 
 # Formatting Guidelines
 * Use Markdown for all code snippets.
@@ -55,7 +62,7 @@ You aim to help developers of all levels, prioritizing teaching web concepts as
 * **CRITICAL**: Use the precision of Strunk & White, the brevity of Hemingway, and the simple clarity of Vonnegut. Don't add repeated information, and keep the whole answer short.
 
 * **CRITICAL** If a tool returns an empty list, immediately pivot to the next logical tool (e.g., from sources to network).
-* **CRITICAL** Always exhaust all possible way to find and select context from different domains.
+* **CRITICAL** Always exhaust all possible ways to find and select context from different domains.
 * **CRITICAL** NEVER write full Python programs - you should only write individual statements that invoke a single function from the provided library.
 * **CRITICAL** NEVER output text before a function call. Always do a function call first.
 * **CRITICAL** You are a debugging assistant in DevTools. NEVER provide answers to questions of unrelated topics such as legal advice, financial advice, personal opinions, medical advice, religion, race, politics, sexuality, gender, or any other non web-development topics. Answer "Sorry, I can't answer that. I'm best at questions about debugging web pages." to such questions.
@@ -261,6 +268,7 @@ export class ContextSelectionAgent extends AiAgent<never> {
         const origin = allowedOriginResult.origin;
 
         const files: Array<{file: string, id: number | undefined}> = [];
+        const uiSourceCodes: Workspace.UISourceCode.UISourceCode[] = [];
         for (const file of ContextSelectionAgent.getUISourceCodes()) {
           const fileUrl = file.url();
           const fileOrigin = Common.ParsedURL.ParsedURL.extractOrigin(fileUrl);
@@ -273,10 +281,17 @@ export class ContextSelectionAgent extends AiAgent<never> {
             file: file.fullDisplayName(),
             id: ContextSelectionAgent.uiSourceCodeId.get(file),
           });
+          uiSourceCodes.push(file);
         }
 
         return {
           result: files,
+          widgets: [{
+            name: 'SOURCE_FILES_LIST',
+            data: {
+              uiSourceCodes,
+            },
+          }],
         };
       },
     });
@@ -342,7 +357,7 @@ export class ContextSelectionAgent extends AiAgent<never> {
 
     this.declareFunction('performanceRecordAndReload', {
       description:
-          'Records a new performance trace. Use this to measure and debug performance metrics and Core Web Vitals like Largest Contentful Paint (LCP), Interaction to Next Paint (INP), and Cumulative Layout Shift (CLS).',
+          'Records a new performance trace. Use this to measure, analyze, and debug page performance, general performance issues, performance metrics, and Core Web Vitals like Largest Contentful Paint (LCP), Interaction to Next Paint (INP), and Cumulative Layout Shift (CLS).',
       parameters: {
         type: Host.AidaClient.ParametersTypes.OBJECT,
         description: '',
@@ -379,7 +394,7 @@ export class ContextSelectionAgent extends AiAgent<never> {
 
     this.declareFunction<{mode: LHSupportedRunMode}>('runLighthouseAudits', {
       description:
-          'Records a Lighthouse audit on the current page. Use this to debug accessibility, SEO, and best practices. (For performance metrics like LCP, use performanceRecordAndReload instead).',
+          'Records a Lighthouse audit on the current page. Use this to debug accessibility, SEO, and best practices. (For any performance-related questions or performance issues, do NOT use this; use performanceRecordAndReload instead).',
       parameters: {
         type: Host.AidaClient.ParametersTypes.OBJECT,
         description: '',

package/front_end/models/ai_assistance/agents/ConversationSummaryAgent.ts

@@ -105,7 +105,7 @@ export class ConversationSummaryContext extends ConversationContext<string> {
     this.#conversation = conversation;
   }
 
-  getOrigin(): string {
+  override getURL(): string {
     return 'devtools://ai-assistance';
   }
 

package/front_end/models/ai_assistance/agents/FileAgent.ts

@@ -83,8 +83,8 @@ export class FileContext extends ConversationContext<Workspace.UISourceCode.UISo
     this.#file = file;
   }
 
-  override getOrigin(): string {
-    return new URL(this.#file.url()).origin;
+  override getURL(): string {
+    return this.#file.url();
   }
 
   override getItem(): Workspace.UISourceCode.UISourceCode {

package/front_end/models/ai_assistance/agents/GreenDevAgent.ts

@@ -101,7 +101,7 @@ export class GreenDevContext extends ConversationContext<string> {
     this.#context = context;
   }
 
-  getOrigin(): string {
+  override getURL(): string {
     return 'devtools://ai-assistance';
   }
 

package/front_end/models/ai_assistance/agents/NetworkAgent.ts

@@ -2,7 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import * as Common from '../../../core/common/common.js';
 import * as Host from '../../../core/host/host.js';
 import * as i18n from '../../../core/i18n/i18n.js';
 import * as Root from '../../../core/root/root.js';
@@ -111,13 +110,13 @@ export class RequestContext extends ConversationContext<SDK.NetworkRequest.Netwo
   }
 
   /**
-   * Note: this is not the literal origin of the network request. This origin
+   * Note: this is not the literal origin of the network request. This URL
    * is used to determine when we should force the user to start a new AI
    * conversation when the context changes. We allow a single AI conversation to
    * inspect all network requests that were made for that given target URL.
    */
-  override getOrigin(): string {
-    return Common.ParsedURL.ParsedURL.extractOrigin(this.#request.documentURL);
+  override getURL(): string {
+    return this.#request.documentURL;
   }
 
   override getItem(): SDK.NetworkRequest.NetworkRequest {