Rename to processedByAutoModel

Author: robertbrignull

extensions/ql-vscode/src/common/interface-types.ts

@@ -543,8 +543,8 @@ interface SetInProgressMethodsMessage {
   methods: string[];
 }
 
-interface SetSentToLLMMethodsMessage {
-  t: "setSentToLLMMethods";
+interface SetProcessedByAutoModelMethodsMessage {
+  t: "setProcessedByAutoModelMethods";
   methods: string[];
 }
 
@@ -631,7 +631,7 @@ export type ToModelEditorMessage =
   | SetModeledMethodsMessage
   | SetModifiedMethodsMessage
   | SetInProgressMethodsMessage
-  | SetSentToLLMMethodsMessage
+  | SetProcessedByAutoModelMethodsMessage
   | RevealMethodMessage
   | SetAccessPathSuggestionsMessage;
 

extensions/ql-vscode/src/model-editor/auto-modeler.ts

@@ -161,7 +161,7 @@ export class AutoModeler {
           );
 
           // Let the UI know which methods have been sent to the LLM
-          this.modelingStore.addSentToLLMMethods(
+          this.modelingStore.addProcessedByAutoModelMethods(
             this.databaseItem,
             candidateSignatures,
           );

extensions/ql-vscode/src/model-editor/model-editor-view.ts

@@ -836,14 +836,16 @@ export class ModelEditorView extends AbstractWebview<
     );
 
     this.push(
-      this.modelingEvents.onSentToLLMMethodsChanged(async (event) => {
-        if (event.dbUri === this.databaseItem.databaseUri.toString()) {
-          await this.postMessage({
-            t: "setSentToLLMMethods",
-            methods: Array.from(event.methods),
-          });
-        }
-      }),
+      this.modelingEvents.onProcessedByAutoModelMethodsChanged(
+        async (event) => {
+          if (event.dbUri === this.databaseItem.databaseUri.toString()) {
+            await this.postMessage({
+              t: "setProcessedByAutoModelMethods",
+              methods: Array.from(event.methods),
+            });
+          }
+        },
+      ),
     );
 
     this.push(

extensions/ql-vscode/src/model-editor/modeling-events.ts

@@ -42,15 +42,15 @@ interface SelectedMethodChangedEvent {
   readonly modeledMethods: readonly ModeledMethod[];
   readonly isModified: boolean;
   readonly isInProgress: boolean;
-  readonly hasBeenSentToLLM: boolean;
+  readonly processedByAutoModel: boolean;
 }
 
 interface InProgressMethodsChangedEvent {
   readonly dbUri: string;
   readonly methods: ReadonlySet<string>;
 }
 
-interface SentToLLMMethodsChangedEvent {
+interface ProcessedByAutoModelMethodsChangedEvent {
   readonly dbUri: string;
   readonly methods: ReadonlySet<string>;
 }
@@ -75,7 +75,7 @@ export class ModelingEvents extends DisposableObject {
   public readonly onModifiedMethodsChanged: AppEvent<ModifiedMethodsChangedEvent>;
   public readonly onSelectedMethodChanged: AppEvent<SelectedMethodChangedEvent>;
   public readonly onInProgressMethodsChanged: AppEvent<InProgressMethodsChangedEvent>;
-  public readonly onSentToLLMMethodsChanged: AppEvent<SentToLLMMethodsChangedEvent>;
+  public readonly onProcessedByAutoModelMethodsChanged: AppEvent<ProcessedByAutoModelMethodsChangedEvent>;
   public readonly onRevealInModelEditor: AppEvent<RevealInModelEditorEvent>;
   public readonly onFocusModelEditor: AppEvent<FocusModelEditorEvent>;
 
@@ -89,7 +89,7 @@ export class ModelingEvents extends DisposableObject {
   private readonly onModifiedMethodsChangedEventEmitter: AppEventEmitter<ModifiedMethodsChangedEvent>;
   private readonly onSelectedMethodChangedEventEmitter: AppEventEmitter<SelectedMethodChangedEvent>;
   private readonly onInProgressMethodsChangedEventEmitter: AppEventEmitter<InProgressMethodsChangedEvent>;
-  private readonly onSentToLLMMethodsChangedEventEmitter: AppEventEmitter<SentToLLMMethodsChangedEvent>;
+  private readonly onProcessedByAutoModelMethodsChangedEventEmitter: AppEventEmitter<ProcessedByAutoModelMethodsChangedEvent>;
   private readonly onRevealInModelEditorEventEmitter: AppEventEmitter<RevealInModelEditorEvent>;
   private readonly onFocusModelEditorEventEmitter: AppEventEmitter<FocusModelEditorEvent>;
 
@@ -149,11 +149,11 @@ export class ModelingEvents extends DisposableObject {
     this.onInProgressMethodsChanged =
       this.onInProgressMethodsChangedEventEmitter.event;
 
-    this.onSentToLLMMethodsChangedEventEmitter = this.push(
-      app.createEventEmitter<SentToLLMMethodsChangedEvent>(),
+    this.onProcessedByAutoModelMethodsChangedEventEmitter = this.push(
+      app.createEventEmitter<ProcessedByAutoModelMethodsChangedEvent>(),
     );
-    this.onSentToLLMMethodsChanged =
-      this.onSentToLLMMethodsChangedEventEmitter.event;
+    this.onProcessedByAutoModelMethodsChanged =
+      this.onProcessedByAutoModelMethodsChangedEventEmitter.event;
 
     this.onRevealInModelEditorEventEmitter = this.push(
       app.createEventEmitter<RevealInModelEditorEvent>(),
@@ -240,7 +240,7 @@ export class ModelingEvents extends DisposableObject {
     modeledMethods: ModeledMethod[],
     isModified: boolean,
     isInProgress: boolean,
-    hasBeenSentToLLM: boolean,
+    processedByAutoModel: boolean,
   ) {
     this.onSelectedMethodChangedEventEmitter.fire({
       databaseItem,
@@ -249,7 +249,7 @@ export class ModelingEvents extends DisposableObject {
       modeledMethods,
       isModified,
       isInProgress,
-      hasBeenSentToLLM,
+      processedByAutoModel,
     });
   }
 
@@ -263,11 +263,11 @@ export class ModelingEvents extends DisposableObject {
     });
   }
 
-  public fireSentToLLMMethodsChangedEvent(
+  public fireProcessedByAutoModelMethodsChangedEvent(
     dbUri: string,
     methods: ReadonlySet<string>,
   ) {
-    this.onSentToLLMMethodsChangedEventEmitter.fire({
+    this.onProcessedByAutoModelMethodsChangedEventEmitter.fire({
       dbUri,
       methods,
     });

extensions/ql-vscode/src/model-editor/modeling-store.ts

@@ -14,7 +14,7 @@ interface InternalDbModelingState {
   modeledMethods: Record<string, ModeledMethod[]>;
   modifiedMethodSignatures: Set<string>;
   inProgressMethods: Set<string>;
-  sentToLLMMethods: Set<string>;
+  processedByAutoModelMethods: Set<string>;
   selectedMethod: Method | undefined;
   selectedUsage: Usage | undefined;
 }
@@ -27,7 +27,7 @@ interface DbModelingState {
   readonly modeledMethods: Readonly<Record<string, readonly ModeledMethod[]>>;
   readonly modifiedMethodSignatures: ReadonlySet<string>;
   readonly inProgressMethods: ReadonlySet<string>;
-  readonly sentToLLMMethods: ReadonlySet<string>;
+  readonly processedByAutoModelMethods: ReadonlySet<string>;
   readonly selectedMethod: Method | undefined;
   readonly selectedUsage: Usage | undefined;
 }
@@ -39,7 +39,7 @@ interface SelectedMethodDetails {
   readonly modeledMethods: readonly ModeledMethod[];
   readonly isModified: boolean;
   readonly isInProgress: boolean;
-  readonly hasBeenSentToLLM: boolean;
+  readonly processedByAutoModel: boolean;
 }
 
 export class ModelingStore extends DisposableObject {
@@ -62,7 +62,7 @@ export class ModelingStore extends DisposableObject {
       mode,
       modeledMethods: {},
       modifiedMethodSignatures: new Set(),
-      sentToLLMMethods: new Set(),
+      processedByAutoModelMethods: new Set(),
       selectedMethod: undefined,
       selectedUsage: undefined,
       inProgressMethods: new Set(),
@@ -305,15 +305,17 @@ export class ModelingStore extends DisposableObject {
     const modeledMethods = dbState.modeledMethods[method.signature] ?? [];
     const isModified = dbState.modifiedMethodSignatures.has(method.signature);
     const isInProgress = dbState.inProgressMethods.has(method.signature);
-    const hasBeenSentToLLM = dbState.sentToLLMMethods.has(method.signature);
+    const processedByAutoModel = dbState.processedByAutoModelMethods.has(
+      method.signature,
+    );
     this.modelingEvents.fireSelectedMethodChangedEvent(
       dbItem,
       method,
       usage,
       modeledMethods,
       isModified,
       isInProgress,
-      hasBeenSentToLLM,
+      processedByAutoModel,
     );
   }
 
@@ -342,11 +344,14 @@ export class ModelingStore extends DisposableObject {
     });
   }
 
-  public addSentToLLMMethods(dbItem: DatabaseItem, sentToLLMMethods: string[]) {
-    this.changeSentToLLMMethods(dbItem, (state) => {
-      state.sentToLLMMethods = new Set([
-        ...state.sentToLLMMethods,
-        ...sentToLLMMethods,
+  public addProcessedByAutoModelMethods(
+    dbItem: DatabaseItem,
+    processedByAutoModelMethods: string[],
+  ) {
+    this.changeProcessedByAutoModelMethods(dbItem, (state) => {
+      state.processedByAutoModelMethods = new Set([
+        ...state.processedByAutoModelMethods,
+        ...processedByAutoModelMethods,
       ]);
     });
   }
@@ -371,7 +376,9 @@ export class ModelingStore extends DisposableObject {
         selectedMethod.signature,
       ),
       isInProgress: dbState.inProgressMethods.has(selectedMethod.signature),
-      hasBeenSentToLLM: dbState.sentToLLMMethods.has(selectedMethod.signature),
+      processedByAutoModel: dbState.processedByAutoModelMethods.has(
+        selectedMethod.signature,
+      ),
     };
   }
 
@@ -429,17 +436,17 @@ export class ModelingStore extends DisposableObject {
     );
   }
 
-  private changeSentToLLMMethods(
+  private changeProcessedByAutoModelMethods(
     dbItem: DatabaseItem,
     updateState: (state: InternalDbModelingState) => void,
   ) {
     const state = this.getState(dbItem);
 
     updateState(state);
 
-    this.modelingEvents.fireSentToLLMMethodsChangedEvent(
+    this.modelingEvents.fireProcessedByAutoModelMethodsChangedEvent(
       dbItem.databaseUri.toString(),
-      state.sentToLLMMethods,
+      state.processedByAutoModelMethods,
     );
   }
 }

extensions/ql-vscode/src/view/model-editor/ModelEditor.tsx

@@ -148,7 +148,7 @@ export function ModelEditor({
             setInProgressMethods(new Set(msg.methods));
             break;
           }
-          case "setSentToLLMMethods": {
+          case "setProcessedByAutoModelMethods": {
             // TODO: set state
             break;
           }