Skip to content

Commit 54e1275

Browse files
authored
Merge pull request #3387 from github/koesie10/update-ruby-generate-query-constraints
Update Ruby model generation query constraints
2 parents 7a2688e + 873ccaa commit 54e1275

File tree

5 files changed

+10
-8
lines changed

5 files changed

+10
-8
lines changed

extensions/ql-vscode/src/model-editor/languages/models-as-data.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export type ModelsAsDataLanguagePredicate<T> = {
3333
};
3434

3535
type ModelsAsDataLanguageModelGeneration = {
36-
queryConstraints: QueryConstraints;
36+
queryConstraints: (mode: Mode) => QueryConstraints;
3737
filterQueries?: (queryPath: string) => boolean;
3838
parseResults: (
3939
// The path to the query that generated the results.

extensions/ql-vscode/src/model-editor/languages/ruby/index.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -172,9 +172,10 @@ export const ruby: ModelsAsDataLanguage = {
172172
},
173173
},
174174
modelGeneration: {
175-
queryConstraints: {
176-
"query path": "queries/modeling/GenerateModel.ql",
177-
},
175+
queryConstraints: (mode) => ({
176+
kind: "table",
177+
"tags contain all": ["modeleditor", "generate-model", modeTag(mode)],
178+
}),
178179
parseResults: parseGenerateModelResults,
179180
},
180181
accessPathSuggestions: {

extensions/ql-vscode/src/model-editor/languages/static/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,9 @@ export const staticLanguage: ModelsAsDataLanguage = {
141141
},
142142
},
143143
modelGeneration: {
144-
queryConstraints: {
144+
queryConstraints: () => ({
145145
"tags contain": ["modelgenerator"],
146-
},
146+
}),
147147
filterQueries: filterFlowModelQueries,
148148
parseResults: parseFlowModelResults,
149149
},

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -619,7 +619,7 @@ export class ModelEditorView extends AbstractWebview<
619619

620620
try {
621621
await runGenerateQueries({
622-
queryConstraints: modelGeneration.queryConstraints,
622+
queryConstraints: modelGeneration.queryConstraints(mode),
623623
filterQueries: modelGeneration.filterQueries,
624624
parseResults: (queryPath, results) =>
625625
modelGeneration.parseResults(

extensions/ql-vscode/test/vscode-tests/no-workspace/model-editor/generate.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import { runGenerateQueries } from "../../../../src/model-editor/generate";
1313
import { ruby } from "../../../../src/model-editor/languages/ruby";
1414
import type { ModeledMethod } from "../../../../src/model-editor/modeled-method";
1515
import { EndpointType } from "../../../../src/model-editor/method";
16+
import { Mode } from "../../../../src/model-editor/shared/mode";
1617

1718
describe("runGenerateQueries", () => {
1819
const modelsAsDataLanguage = ruby;
@@ -125,7 +126,7 @@ describe("runGenerateQueries", () => {
125126
};
126127

127128
await runGenerateQueries({
128-
queryConstraints: modelGeneration.queryConstraints,
129+
queryConstraints: modelGeneration.queryConstraints(Mode.Framework),
129130
filterQueries: modelGeneration.filterQueries,
130131
parseResults: (queryPath, results) =>
131132
modelGeneration.parseResults(

0 commit comments

Comments
 (0)