Skip to content

Commit acb6b8b

Browse files
committed
Remove enableAccessPathSuggestions config
Access path suggestions will now be enabled for all languages which support it (which is currently only Ruby).
1 parent 8a3a6d7 commit acb6b8b

File tree

2 files changed

+10
-28
lines changed

2 files changed

+10
-28
lines changed

extensions/ql-vscode/src/config.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -737,10 +737,6 @@ const LLM_GENERATION_DEV_ENDPOINT = new Setting(
737737
const MODEL_EVALUATION = new Setting("evaluation", MODEL_SETTING);
738738
const MODEL_PACK_LOCATION = new Setting("packLocation", MODEL_SETTING);
739739
const ENABLE_PYTHON = new Setting("enablePython", MODEL_SETTING);
740-
const ENABLE_ACCESS_PATH_SUGGESTIONS = new Setting(
741-
"enableAccessPathSuggestions",
742-
MODEL_SETTING,
743-
);
744740

745741
export type ModelConfigPackVariables = {
746742
database: string;
@@ -758,7 +754,6 @@ export interface ModelConfig {
758754
variables: ModelConfigPackVariables,
759755
): string;
760756
enablePython: boolean;
761-
enableAccessPathSuggestions: boolean;
762757
}
763758

764759
export class ModelConfigListener extends ConfigListener implements ModelConfig {
@@ -813,10 +808,6 @@ export class ModelConfigListener extends ConfigListener implements ModelConfig {
813808
public get enablePython(): boolean {
814809
return !!ENABLE_PYTHON.getValue<boolean>();
815810
}
816-
817-
public get enableAccessPathSuggestions(): boolean {
818-
return !!ENABLE_ACCESS_PATH_SUGGESTIONS.getValue<boolean>();
819-
}
820811
}
821812

822813
const GITHUB_DATABASE_SETTING = new Setting("githubDatabase", ROOT_SETTING);

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

Lines changed: 10 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -348,17 +348,11 @@ export class ModelEditorView extends AbstractWebview<
348348
withProgress((progress) => this.loadMethods(progress), {
349349
cancellable: false,
350350
}),
351-
// Only load access path suggestions if the feature is enabled
352-
this.modelConfig.enableAccessPathSuggestions
353-
? withProgress(
354-
(progress) => this.loadAccessPathSuggestions(progress),
355-
{
356-
cancellable: false,
357-
location: ProgressLocation.Window,
358-
title: "Loading access path suggestions",
359-
},
360-
)
361-
: undefined,
351+
withProgress((progress) => this.loadAccessPathSuggestions(progress), {
352+
cancellable: false,
353+
location: ProgressLocation.Window,
354+
title: "Loading access path suggestions",
355+
}),
362356
]);
363357
void telemetryListener?.sendUIInteraction("model-editor-switch-modes");
364358

@@ -416,14 +410,11 @@ export class ModelEditorView extends AbstractWebview<
416410
await this.generateModeledMethodsOnStartup();
417411
}),
418412
this.loadExistingModeledMethods(),
419-
// Only load access path suggestions if the feature is enabled
420-
this.modelConfig.enableAccessPathSuggestions
421-
? withProgress((progress) => this.loadAccessPathSuggestions(progress), {
422-
cancellable: false,
423-
location: ProgressLocation.Window,
424-
title: "Loading access path suggestions",
425-
})
426-
: undefined,
413+
withProgress((progress) => this.loadAccessPathSuggestions(progress), {
414+
cancellable: false,
415+
location: ProgressLocation.Window,
416+
title: "Loading access path suggestions",
417+
}),
427418
]);
428419
}
429420

0 commit comments

Comments
 (0)