Include mkdirSync in try/catch

github · Oct 5, 2023 · 3b2fee4 · 3b2fee4
1 parent a144bf5
commit 3b2fee4
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/lib/diagnostics.js b/lib/diagnostics.js
diff --git a/lib/diagnostics.js.map b/lib/diagnostics.js.map
diff --git a/src/diagnostics.ts b/src/diagnostics.ts
@@ -93,15 +93,15 @@ export function addDiagnostic(
     "codeql-action",
   );
 
-  // Create the directory if it doesn't exist yet.
-  mkdirSync(diagnosticsPath, { recursive: true });
+  try {
+    // Create the directory if it doesn't exist yet.
+    mkdirSync(diagnosticsPath, { recursive: true });
 
-  const jsonPath = path.resolve(
-    diagnosticsPath,
-    `codeql-action-${diagnostic.timestamp}.json`,
-  );
+    const jsonPath = path.resolve(
+      diagnosticsPath,
+      `codeql-action-${diagnostic.timestamp}.json`,
+    );
 
-  try {
     writeFileSync(jsonPath, JSON.stringify(diagnostic));
   } catch (err) {
     logger.warning(`Unable to write diagnostic message to database: ${err}`);