Merge pull request #3576 from github/robertbrignull/disable-llm-proxima

robertbrignull · web-flow · commit 29c36584f8b1 · 2024-04-22T16:47:56.000+01:00
Disable LLM model generation when not using github.com
diff --git a/extensions/ql-vscode/src/config.ts b/extensions/ql-vscode/src/config.ts
@@ -815,15 +815,18 @@ export interface ModelConfig {
 
 export class ModelConfigListener extends ConfigListener implements ModelConfig {
   protected handleDidChangeConfiguration(e: ConfigurationChangeEvent): void {
-    this.handleDidChangeConfigurationForRelevantSettings([MODEL_SETTING], e);
+    this.handleDidChangeConfigurationForRelevantSettings(
+      [MODEL_SETTING, VSCODE_GITHUB_ENTERPRISE_URI_SETTING],
+      e,
+    );
   }
 
   public get flowGeneration(): boolean {
     return !!FLOW_GENERATION.getValue<boolean>();
   }
 
   public get llmGeneration(): boolean {
-    return !!LLM_GENERATION.getValue<boolean>();
+    return !!LLM_GENERATION.getValue<boolean>() && !hasEnterpriseUri();
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -815,15 +815,18 @@ export interface ModelConfig {`
`815`	`815`
`816`	`816`	`export class ModelConfigListener extends ConfigListener implements ModelConfig {`
`817`	`817`	`protected handleDidChangeConfiguration(e: ConfigurationChangeEvent): void {`
`818`		`- this.handleDidChangeConfigurationForRelevantSettings([MODEL_SETTING], e);`
	`818`	`+ this.handleDidChangeConfigurationForRelevantSettings(`
	`819`	`+ [MODEL_SETTING, VSCODE_GITHUB_ENTERPRISE_URI_SETTING],`
	`820`	`+ e,`
	`821`	`+ );`
`819`	`822`	`}`
`820`	`823`
`821`	`824`	`public get flowGeneration(): boolean {`
`822`	`825`	`return !!FLOW_GENERATION.getValue<boolean>();`
`823`	`826`	`}`
`824`	`827`
`825`	`828`	`public get llmGeneration(): boolean {`
`826`		`- return !!LLM_GENERATION.getValue<boolean>();`
	`829`	`+ return !!LLM_GENERATION.getValue<boolean>() && !hasEnterpriseUri();`
`827`	`830`	`}`
`828`	`831`
`829`	`832`	`/**`