[DrJones/Perf] require a new conversation for new context

- Each "Ask AI" for the Performance agent results in a new conversation
- Context switches by navigation would also require a new conversation (upon the user action).

Bug: 370436840, 377601855
Change-Id: Id96454fbaa34fcfe03b9e638c70dbcf9223c7f6a
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5999262
Commit-Queue: Paul Irish <[email protected]>
Reviewed-by: Paul Irish <[email protected]>
Commit-Queue: Alex Rudenko <[email protected]>

Author: OrKoN

front_end/panels/freestyler/AiAgent.ts

@@ -172,6 +172,7 @@ export abstract class AiAgent<T> {
    * historical conversations.
    */
   #origin?: string;
+  #context?: ConversationContext<T>;
 
   constructor(opts: AgentOptions) {
     this.#aidaClient = opts.aidaClient;
@@ -194,6 +195,10 @@ export abstract class AiAgent<T> {
     return this.#origin;
   }
 
+  get context(): ConversationContext<T>|undefined {
+    return this.#context;
+  }
+
   get title(): string|undefined {
     return [...this.#history.values()]
         .flat()
@@ -402,6 +407,10 @@ STOP`;
     if (options.selected && this.#origin === undefined && options.selected) {
       this.#origin = options.selected.getOrigin();
     }
+    // Remember if the context that is set.
+    if (options.selected && !this.#context) {
+      this.#context = options.selected;
+    }
     const id = this.#runId++;
 
     const response = {

front_end/panels/freestyler/FreestylerPanel.ts

@@ -577,9 +577,8 @@ export class FreestylerPanel extends UI.Panel.Panel {
       return;
     }
 
-    if (!this.#currentAgent) {
-      this.#currentAgent = this.#createAgent(targetAgentType);
-    } else if (this.#currentAgent.type !== targetAgentType || this.#currentAgent.isHistoryEntry) {
+    if (!this.#currentAgent || this.#currentAgent.type !== targetAgentType || this.#currentAgent.isHistoryEntry ||
+        targetAgentType === AgentType.DRJONES_PERFORMANCE) {
       this.#currentAgent = this.#createAgent(targetAgentType);
     }
     this.#viewProps.agentType = this.#currentAgent.type;
@@ -680,11 +679,14 @@ export class FreestylerPanel extends UI.Panel.Panel {
     this.#viewProps.selectedContext = currentContext;
     if (!currentContext) {
       this.#viewProps.blockedByCrossOrigin = false;
+      this.#viewProps.requiresNewConversation = false;
       this.doUpdate();
       return;
     }
     this.#viewProps.blockedByCrossOrigin = !currentContext.isOriginAllowed(this.#currentAgent.origin);
     this.#viewProps.isReadOnly = this.#currentAgent.isHistoryEntry;
+    this.#viewProps.requiresNewConversation = this.#currentAgent.type === AgentType.DRJONES_PERFORMANCE &&
+        Boolean(this.#currentAgent.context) && this.#currentAgent.context !== currentContext;
     this.doUpdate();
   }
 

front_end/panels/freestyler/components/FreestylerChatUi.ts

@@ -115,6 +115,10 @@ const UIStringsNotTranslate = {
    * @description Placeholder text for the input shown when the conversation is blocked because a cross-origin context was selected.
    */
   crossOriginError: 'To talk about data from another origin, start a new chat',
+  /**
+   * @description Placeholder text for the input shown when the conversation is blocked because a cross-origin context was selected.
+   */
+  newConversationError: 'To talk about this data, start a new chat',
   /**
    *@description Title for the send icon button.
    */
@@ -276,6 +280,7 @@ export interface Props {
   agentType?: AgentType;
   isReadOnly: boolean;
   blockedByCrossOrigin: boolean;
+  requiresNewConversation?: boolean;
 }
 
 // The model returns multiline code blocks in an erroneous way with the language being in new line.
@@ -363,7 +368,7 @@ export class FreestylerChatUi extends HTMLElement {
   }
 
   #isTextInputDisabled = (): boolean => {
-    if (this.#props.blockedByCrossOrigin) {
+    if (this.#props.blockedByCrossOrigin || this.#props.requiresNewConversation) {
       return true;
     }
     const isAidaAvailable = this.#props.aidaAvailability === Host.AidaClient.AidaAccessPreconditions.AVAILABLE;
@@ -890,6 +895,9 @@ export class FreestylerChatUi extends HTMLElement {
     if (state === State.CONSENT_VIEW || !agentType) {
       return i18nString(UIStrings.followTheSteps);
     }
+    if (this.#props.requiresNewConversation) {
+      return lockedString(UIStringsNotTranslate.newConversationError);
+    }
     if (this.#props.blockedByCrossOrigin) {
       return lockedString(UIStringsNotTranslate.crossOriginError);
     }
@@ -944,7 +952,7 @@ export class FreestylerChatUi extends HTMLElement {
       ></devtools-button>`;
       // clang-format on
     }
-    if (this.#props.blockedByCrossOrigin) {
+    if (this.#props.blockedByCrossOrigin || this.#props.requiresNewConversation) {
       // clang-format off
       return html`<devtools-button
         class="chat-input-button"