Fix diagnose log_dir_path value

Use the `logFilePath`'s value as a base and get the parent directory
path to check.

This way the `log_dir_path` and `appsignal.log` paths don't point at
different directories. This could have happen because the `logFilePath`
getter (used by `appsignal.log`) does a permission check and falls back
on the system `/tmp` dir if the user configured path can't be written
to.

packages/nodejs/.changesets/fix-diagnose-log_dir_path-path.md

@@ -0,0 +1,5 @@
+---
+bump: "patch"
+---
+
+Fix the diagnose's `log_dir_path` path check. It now always checks the actual log file's parent directory, rather than the configured path. These two values may differ as the package does a permission check to see if the `logPath` is writable or not.

packages/nodejs/src/diagnose.ts

@@ -148,7 +148,7 @@ export class DiagnoseTool {
         path: process.cwd()
       },
       log_dir_path: {
-        path: this.#config.data.logPath!.replace("/appsignal.log", "")
+        path: path.dirname(logFilePath)
       },
       "appsignal.log": {
         path: logFilePath,