Skip to content

Commit ec025cd

Browse files
committed
runCliJson should return unknown since it's unvalidated
1 parent d5b8046 commit ec025cd

2 files changed

Lines changed: 24 additions & 26 deletions

File tree

lib/entry-points.js

Lines changed: 8 additions & 9 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/codeql.ts

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -514,13 +514,9 @@ async function getCodeQLForCmd(
514514
async getVersion() {
515515
let result = util.getCachedCodeQlVersion(cmd);
516516
if (result === undefined) {
517-
result = await runCliJson<VersionInfo>(
518-
cmd,
519-
["version", "--format=json"],
520-
{
521-
noStreamStdout: true,
522-
},
523-
);
517+
result = (await runCliJson(cmd, ["version", "--format=json"], {
518+
noStreamStdout: true,
519+
})) as VersionInfo;
524520
util.cacheCodeQlVersion(cmd, result);
525521
}
526522
return result;
@@ -728,7 +724,7 @@ async function getCodeQLForCmd(
728724
filterToLanguagesWithQueries: boolean;
729725
} = { filterToLanguagesWithQueries: false },
730726
) {
731-
return runCliJson<ResolveLanguagesOutput>(cmd, [
727+
return (await runCliJson(cmd, [
732728
"resolve",
733729
"languages",
734730
"--format=betterjson",
@@ -741,7 +737,7 @@ async function getCodeQLForCmd(
741737
? ["--filter-to-languages-with-queries"]
742738
: []),
743739
...getExtraOptionsFromEnv(["resolve", "languages"]),
744-
]);
740+
])) as ResolveLanguagesOutput;
745741
},
746742
async resolveBuildEnvironment(
747743
workingDir: string | undefined,
@@ -757,7 +753,10 @@ async function getCodeQLForCmd(
757753
if (workingDir !== undefined) {
758754
codeqlArgs.push("--working-dir", workingDir);
759755
}
760-
return await runCliJson<ResolveBuildEnvironmentOutput>(cmd, codeqlArgs);
756+
return (await runCliJson(
757+
cmd,
758+
codeqlArgs,
759+
)) as ResolveBuildEnvironmentOutput;
761760
},
762761
async databaseRunQueries(
763762
databasePath: string,
@@ -959,9 +958,9 @@ async function getCodeQLForCmd(
959958
...getExtraOptionsFromEnv(["resolve", "queries"]),
960959
...queries,
961960
];
962-
return await runCliJson<string[]>(cmd, codeqlArgs, {
961+
return (await runCliJson(cmd, codeqlArgs, {
963962
noStreamStdout: true,
964-
});
963+
})) as string[];
965964
},
966965
async resolveDatabase(
967966
databasePath: string,
@@ -973,9 +972,9 @@ async function getCodeQLForCmd(
973972
"--format=json",
974973
...getExtraOptionsFromEnv(["resolve", "database"]),
975974
];
976-
return await runCliJson<ResolveDatabaseOutput>(cmd, codeqlArgs, {
975+
return (await runCliJson(cmd, codeqlArgs, {
977976
noStreamStdout: true,
978-
});
977+
})) as ResolveDatabaseOutput;
979978
},
980979
async mergeResults(
981980
sarifFiles: string[],
@@ -1140,14 +1139,14 @@ async function runCli(
11401139
* @param opts.noStreamStdout Optional boolean to indicate whether to stream the command's standard output.
11411140
* @returns The parsed JSON output from the command.
11421141
*/
1143-
async function runCliJson<T>(
1142+
async function runCliJson(
11441143
cmd: string,
11451144
args: string[] = [],
11461145
opts: { stdin?: string; noStreamStdout?: boolean } = {},
1147-
): Promise<T> {
1146+
): Promise<unknown> {
11481147
const output = await runCli(cmd, args, opts);
11491148
try {
1150-
return JSON.parse(output) as T;
1149+
return JSON.parse(output);
11511150
} catch (e) {
11521151
throw Error(
11531152
`Unexpected output from codeql ${args.join(" ")}: ${getErrorMessage(e)}`,

0 commit comments

Comments
 (0)