Check that the database exists before writing diagnostics to it

github · Apr 11, 2024 · d4e7b0e · d4e7b0e
1 parent 96f44cb
commit d4e7b0e
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 21 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
@@ -1,4 +1,4 @@
-import { mkdirSync, writeFileSync } from "fs";
+import { existsSync, mkdirSync, writeFileSync } from "fs";
 import path from "path";
 
 import { Config } from "./config-utils";
@@ -87,23 +87,31 @@ export function addDiagnostic(
   diagnostic: DiagnosticMessage,
 ) {
   const logger = getActionsLogger();
+  const databasePath = getCodeQLDatabasePath(config, language);
   const diagnosticsPath = path.resolve(
-    getCodeQLDatabasePath(config, language),
+    databasePath,
     "diagnostic",
     "codeql-action",
   );
 
-  try {
-    // Create the directory if it doesn't exist yet.
-    mkdirSync(diagnosticsPath, { recursive: true });
+  // Check that the database exists before writing to it.
+  if (existsSync(databasePath)) {
+    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`,
+      );
 
-    writeFileSync(jsonPath, JSON.stringify(diagnostic));
-  } catch (err) {
-    logger.warning(`Unable to write diagnostic message to database: ${err}`);
+      writeFileSync(jsonPath, JSON.stringify(diagnostic));
+    } catch (err) {
+      logger.warning(`Unable to write diagnostic message to database: ${err}`);
+    }
+  } else {
+    logger.info(
+      `Writing a diagnostic for ${language}, but the database at ${databasePath} does not exist yet.`,
+    );
   }
 }