Skip to content

Commit 6d7fbfc

Browse files
Merge pull request #2903 from github/robertbrignull/multiple-models-feature-flag
Add feature flag for showing multiple modelings
2 parents 55af9bc + e10e3ad commit 6d7fbfc

File tree

7 files changed

+16
-1
lines changed

7 files changed

+16
-1
lines changed

extensions/ql-vscode/src/config.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -709,6 +709,7 @@ const MODEL_SETTING = new Setting("model", ROOT_SETTING);
709709
const FLOW_GENERATION = new Setting("flowGeneration", MODEL_SETTING);
710710
const LLM_GENERATION = new Setting("llmGeneration", MODEL_SETTING);
711711
const EXTENSIONS_DIRECTORY = new Setting("extensionsDirectory", MODEL_SETTING);
712+
const SHOW_MULTIPLE_MODELS = new Setting("showMultipleModels", MODEL_SETTING);
712713

713714
export function showFlowGeneration(): boolean {
714715
return !!FLOW_GENERATION.getValue<boolean>();
@@ -723,3 +724,7 @@ export function getExtensionsDirectory(languageId: string): string | undefined {
723724
languageId,
724725
});
725726
}
727+
728+
export function showMultipleModels(): boolean {
729+
return !!SHOW_MULTIPLE_MODELS.getValue<boolean>();
730+
}

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,11 @@ import {
3434
import { Method, Usage } from "./method";
3535
import { ModeledMethod } from "./modeled-method";
3636
import { ExtensionPack } from "./shared/extension-pack";
37-
import { showFlowGeneration, showLlmGeneration } from "../config";
37+
import {
38+
showFlowGeneration,
39+
showLlmGeneration,
40+
showMultipleModels,
41+
} from "../config";
3842
import { Mode } from "./shared/mode";
3943
import { loadModeledMethods, saveModeledMethods } from "./modeled-method-fs";
4044
import { pickExtensionPack } from "./extension-pack-picker";
@@ -365,6 +369,7 @@ export class ModelEditorView extends AbstractWebview<
365369
extensionPack: this.extensionPack,
366370
showFlowGeneration: showFlowGeneration(),
367371
showLlmButton,
372+
showMultipleModels: showMultipleModels(),
368373
mode: this.mode,
369374
},
370375
});

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,6 @@ export interface ModelEditorViewState {
55
extensionPack: ExtensionPack;
66
showFlowGeneration: boolean;
77
showLlmButton: boolean;
8+
showMultipleModels: boolean;
89
mode: Mode;
910
}

extensions/ql-vscode/src/stories/model-editor/LibraryRow.stories.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,7 @@ LibraryRow.args = {
214214
extensionPack: createMockExtensionPack(),
215215
showFlowGeneration: true,
216216
showLlmButton: true,
217+
showMultipleModels: true,
217218
mode: Mode.Application,
218219
},
219220
hideModeledMethods: false,

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ ModelEditor.args = {
3030
},
3131
showFlowGeneration: true,
3232
showLlmButton: true,
33+
showMultipleModels: true,
3334
mode: Mode.Application,
3435
},
3536
initialMethods: [

extensions/ql-vscode/src/view/model-editor/__tests__/LibraryRow.spec.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ describe(LibraryRow.name, () => {
2020
mode: Mode.Application,
2121
showFlowGeneration: false,
2222
showLlmButton: false,
23+
showMultipleModels: false,
2324
extensionPack: createMockExtensionPack(),
2425
};
2526

extensions/ql-vscode/src/view/model-editor/__tests__/ModeledMethodsList.spec.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ describe(ModeledMethodsList.name, () => {
4848
mode: Mode.Application,
4949
showFlowGeneration: false,
5050
showLlmButton: false,
51+
showMultipleModels: false,
5152
extensionPack: createMockExtensionPack(),
5253
};
5354

0 commit comments

Comments
 (0)