Revert "RPP: create new annotations AI agent"

This reverts commit 9995b96.

Reason for revert: I forgot that the backend CL needs to land before this can land!

Original change's description:
> RPP: create new annotations AI agent
>
> Required mostly for metrics so we can track its usage accurately rather
> than bundled in with the existing "Ask AI" integration.
>
> (Googlers: see the attached bug for a link to the backend CL, this needs
> to land first before this can land).
>
> Bug: 406795908
> Change-Id: Ibf0fff042f72a6e622e9d9d9e87769a71feb23af
> Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6407494
> Reviewed-by: Nikolay Vitkov <[email protected]>
> Commit-Queue: Jack Franklin <[email protected]>

Bug: 406795908
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: Ic85d030aa8542818168b23b6c60f582aa68eaf52
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6405418
Bot-Commit: Rubber Stamper <[email protected]>
Auto-Submit: Jack Franklin <[email protected]>
Reviewed-by: Yang Guo <[email protected]>
Commit-Queue: Rubber Stamper <[email protected]>

Author: jackfranklin

config/gni/devtools_grd_files.gni

@@ -989,7 +989,6 @@ grd_files_debug_sources = [
   "front_end/models/ai_assistance/agents/NetworkAgent.js",
   "front_end/models/ai_assistance/agents/PatchAgent.js",
   "front_end/models/ai_assistance/agents/PerformanceAgent.js",
-  "front_end/models/ai_assistance/agents/PerformanceAnnotationsAgent.js",
   "front_end/models/ai_assistance/agents/PerformanceInsightsAgent.js",
   "front_end/models/ai_assistance/agents/StylingAgent.js",
   "front_end/models/ai_assistance/data_formatters/FileFormatter.js",

front_end/core/host/AidaClient.ts

@@ -124,8 +124,6 @@ export enum ClientFeature {
   CHROME_NETWORK_AGENT = 7,
   // Chrome AI Assistance Performance Agent.
   CHROME_PERFORMANCE_AGENT = 8,
-  // Chrome AI Annotations Performance Agent
-  CHROME_PERFORMANCE_ANNOTATIONS_AGENT = 20,
   // Chrome AI Assistance File Agent.
   CHROME_FILE_AGENT = 9,
   // Chrome AI Patch Agent.

front_end/models/ai_assistance/BUILD.gn

@@ -19,7 +19,6 @@ devtools_module("ai_assistance") {
     "agents/NetworkAgent.ts",
     "agents/PatchAgent.ts",
     "agents/PerformanceAgent.ts",
-    "agents/PerformanceAnnotationsAgent.ts",
     "agents/PerformanceInsightsAgent.ts",
     "agents/StylingAgent.ts",
     "data_formatters/FileFormatter.ts",
@@ -77,7 +76,6 @@ ts_library("unittests") {
     "agents/NetworkAgent.test.ts",
     "agents/PatchAgent.test.ts",
     "agents/PerformanceAgent.test.ts",
-    "agents/PerformanceAnnotationsAgent.test.ts",
     "agents/PerformanceInsightsAgent.test.ts",
     "agents/StylingAgent.test.ts",
     "data_formatters/FileFormatter.test.ts",

front_end/models/ai_assistance/agents/PerformanceAgent.test.ts

@@ -226,4 +226,22 @@ self: 3
       assert.isFalse(enhancedQuery3.includes(mockAiCallTree.serialize()));
     });
   });
+
+  describe('generating an AI entry label', () => {
+    it('generates a label from the final answer and trims newlines', async function() {
+      const agent = new PerformanceAgent({
+        aidaClient: mockAidaClient([[{
+          explanation: 'hello world\n',
+        }]]),
+      });
+      const {parsedTrace} = await TraceLoader.traceEngine(this, 'web-dev-with-commit.json.gz');
+      const evalScriptEvent = parsedTrace.Renderer.allTraceEntries.find(
+          event => event.name === Trace.Types.Events.Name.EVALUATE_SCRIPT && event.ts === 122411195649);
+      assert.exists(evalScriptEvent);
+      const aiCallTree = TimelineUtils.AICallTree.AICallTree.fromEvent(evalScriptEvent, parsedTrace);
+      assert.isOk(aiCallTree);
+      const label = await agent.generateAIEntryLabel(aiCallTree);
+      assert.strictEqual(label, 'hello world');
+    });
+  });
 });

front_end/models/ai_assistance/agents/PerformanceAgent.ts

@@ -201,11 +201,7 @@ export class CallTreeContext extends ConversationContext<TimelineUtils.AICallTre
 export class PerformanceAgent extends AiAgent<TimelineUtils.AICallTree.AICallTree> {
   override readonly type = AgentType.PERFORMANCE;
   readonly preamble = preamble;
-
-  // We have to set the type of clientFeature here to be the entire enum
-  // because in PerformanceAnnotationsAgent.ts we override it.
-  // TODO(b/406961576): split the agents apart rather than have one extend the other.
-  readonly clientFeature: Host.AidaClient.ClientFeature = Host.AidaClient.ClientFeature.CHROME_PERFORMANCE_AGENT;
+  readonly clientFeature = Host.AidaClient.ClientFeature.CHROME_PERFORMANCE_AGENT;
   get userTier(): string|undefined {
     return Root.Runtime.hostConfig.devToolsAiAssistancePerformanceAgent?.userTier;
   }
@@ -255,4 +251,33 @@ export class PerformanceAgent extends AiAgent<TimelineUtils.AICallTree.AICallTre
     const perfEnhancementQuery = treeStr ? `${treeStr}\n\n# User request\n\n` : '';
     return `${perfEnhancementQuery}${query}`;
   }
+
+  /**
+   * Used in the Performance panel to automatically generate a label for a selected entry.
+   */
+  async generateAIEntryLabel(callTree: TimelineUtils.AICallTree.AICallTree): Promise<string> {
+    const context = new CallTreeContext(callTree);
+    const response = await Array.fromAsync(this.run(AI_LABEL_GENERATION_PROMPT, {selected: context}));
+    const lastResponse = response.at(-1);
+    if (lastResponse && lastResponse.type === ResponseType.ANSWER && lastResponse.complete === true) {
+      return lastResponse.text.trim();
+    }
+    throw new Error('Failed to generate AI entry label');
+  }
 }
+
+const AI_LABEL_GENERATION_PROMPT = `## Instruction:
+Generate a concise label (max 60 chars, single line) describing the *user-visible effect* of the selected call tree's activity, based solely on the provided call tree data.
+
+## Strict Constraints:
+- Output must be a single line of text.
+- Maximum 60 characters.
+- No full stops.
+- Focus on user impact, not internal operations.
+- Do not include the name of the selected event.
+- Do not make assumptions about when the activity happened.
+- Base the description only on the information present within the call tree data.
+- Prioritize brevity.
+- Only include third-party script names if their identification is highly confident.
+- Always use "responsiveness" rather than "user interaction responsiveness".
+`;

front_end/models/ai_assistance/agents/PerformanceAnnotationsAgent.test.ts

@@ -8,7 +8,6 @@ export * from './agents/AiAgent.js';
 export * from './agents/FileAgent.js';
 export * from './agents/NetworkAgent.js';
 export * from './agents/PerformanceAgent.js';
-export * from './agents/PerformanceAnnotationsAgent.js';
 export * from './agents/PerformanceInsightsAgent.js';
 export * from './agents/StylingAgent.js';
 export * from './agents/PatchAgent.js';

front_end/models/ai_assistance/agents/PerformanceAnnotationsAgent.ts

@@ -590,7 +590,7 @@ describeWithEnvironment('Overlays', () => {
       const generateButton = elementsWrapper.querySelector<HTMLElement>('.ai-label-button');
       assert.isOk(generateButton, 'could not find "Generate label" button');
       assert.isTrue(generateButton.classList.contains('enabled'));
-      const agent = new AiAssistanceModels.PerformanceAnnotationsAgent({
+      const agent = new AiAssistanceModels.PerformanceAgent({
         aidaClient: mockAidaClient([[{
           explanation: 'This is an interesting entry',
           metadata: {

front_end/models/ai_assistance/ai_assistance.ts

@@ -189,7 +189,7 @@ export class EntryLabelOverlay extends HTMLElement {
   #callTree: Utils.AICallTree.AICallTree|null = null;
   // Creates or gets the setting if it exists.
   #aiAnnotationsEnabledSetting = Common.Settings.Settings.instance().createSetting('ai-annotations-enabled', false);
-  #agent = new AiAssistanceModels.PerformanceAnnotationsAgent({
+  #performanceAgent = new AiAssistanceModels.PerformanceAgent({
     aidaClient: new Host.AidaClient.AidaClient(),
     serverSideLoggingEnabled: isAiAssistanceServerSideLoggingEnabled(),
   });
@@ -249,8 +249,8 @@ export class EntryLabelOverlay extends HTMLElement {
   /**
    * So we can provide a mocked agent in tests. Do not call this method outside of a test!
    */
-  overrideAIAgentForTest(agent: AiAssistanceModels.PerformanceAnnotationsAgent): void {
-    this.#agent = agent;
+  overrideAIAgentForTest(agent: AiAssistanceModels.PerformanceAgent): void {
+    this.#performanceAgent = agent;
   }
 
   connectedCallback(): void {
@@ -521,7 +521,7 @@ export class EntryLabelOverlay extends HTMLElement {
         this.#focusInputBox();
         void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#boundRender);
 
-        this.#label = await this.#agent.generateAIEntryLabel(this.#callTree);
+        this.#label = await this.#performanceAgent.generateAIEntryLabel(this.#callTree);
         this.dispatchEvent(new EntryLabelChangeEvent(this.#label));
         this.#inputField.innerText = this.#label;