Skip to content

Commit 0feef36

Browse files
committed
Clean up temporary file in finally block
1 parent 35267bb commit 0feef36

File tree

1 file changed

+12
-7
lines changed

1 file changed

+12
-7
lines changed

extensions/ql-vscode/src/variant-analysis/code-scanning-pack.ts

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@ export async function resolveCodeScanningQueryPack(
2929
// Resolve queries
3030
void logger.log(`Resolving queries for pack: ${packName}`);
3131

32-
const suiteFile = await file({
33-
postfix: ".qls",
34-
});
35-
const suitePath = suiteFile.path;
3632
const suiteYaml: SuiteInstruction[] = [
3733
{
3834
import: `codeql-suites/${language}-code-scanning.qls`,
@@ -45,11 +41,20 @@ export async function resolveCodeScanningQueryPack(
4541
},
4642
},
4743
];
48-
await outputFile(suitePath, dump(suiteYaml), "utf8");
4944

50-
const resolvedQueries = await cliServer.resolveQueries(suitePath);
45+
let resolvedQueries: string[];
46+
const suiteFile = await file({
47+
postfix: ".qls",
48+
});
49+
const suitePath = suiteFile.path;
50+
51+
try {
52+
await outputFile(suitePath, dump(suiteYaml), "utf8");
5153

52-
await suiteFile.cleanup();
54+
resolvedQueries = await cliServer.resolveQueries(suitePath);
55+
} finally {
56+
await suiteFile.cleanup();
57+
}
5358

5459
if (resolvedQueries.length === 0) {
5560
throw Error(

0 commit comments

Comments
 (0)