@get-dx/cli 0.6.0 → 0.6.1

package/dist/commands/studio/reports.js

@@ -367,9 +367,9 @@ function studioReportToYaml(report) {
         description: report.description ?? "",
         markdown_notes: report.markdown_notes ?? "",
         view_access_type: report.view_access_type,
-        viewer_emails: [],
+        viewer_emails: report.viewer_emails ?? [],
         edit_access_type: report.edit_access_type,
-        editor_emails: [],
+        editor_emails: report.editor_emails ?? [],
         tiles: report.tiles.map((tile) => ({
             id: tile.id,
             title: tile.title,

package/dist/config.js

@@ -2,7 +2,7 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { CliError, EXIT_CODES } from "./errors.js";
-const DEFAULT_API_BASE_URL = "https://api.getdx.com";
+export const DEFAULT_API_BASE_URL = "https://api.getdx.com";
 const DEFAULT_WEB_BASE_URL = "https://app.getdx.com";
 // --- Config file I/O ---------------------------------------------------------
 export function getConfigPath() {
@@ -14,7 +14,16 @@ export function readConfig() {
         return {};
     }
     const content = fs.readFileSync(configPath, "utf8");
-    const raw = JSON.parse(content);
+    let raw;
+    try {
+        raw = JSON.parse(content);
+    }
+    catch {
+        // The config file can be left empty or truncated if a previous `dx`
+        // process was killed mid-write, or two processes wrote concurrently.
+        // Treat it like a missing config rather than crashing every command.
+        return {};
+    }
     const stored = {};
     if (typeof raw.webBaseUrl === "string") {
         stored.webBaseUrl = raw.webBaseUrl;
@@ -46,8 +55,15 @@ export function persistVersionPromptSelection(prompt) {
     writeConfig(config);
 }
 export function writeConfig(config) {
-    fs.mkdirSync(getConfigDir(), { recursive: true });
-    fs.writeFileSync(getConfigPath(), JSON.stringify(config, null, 2) + "\n", "utf8");
+    const configDir = getConfigDir();
+    fs.mkdirSync(configDir, { recursive: true });
+    const configPath = getConfigPath();
+    // Write to a temp file and rename into place so a crash or a race between
+    // concurrent `dx` processes can't leave config.json empty or truncated —
+    // renames are atomic, so readers always see either the old or new content.
+    const tempPath = path.join(configDir, `.config.json.${process.pid}.${Date.now()}.tmp`);
+    fs.writeFileSync(tempPath, JSON.stringify(config, null, 2) + "\n", "utf8");
+    fs.renameSync(tempPath, configPath);
 }
 export function persistBaseUrls(apiBaseUrl, webBaseUrl) {
     writeConfig({

package/dist/loginViaBrowser.js

@@ -49,7 +49,16 @@ export async function loginViaBrowser(webBaseUrl) {
                 error: new CliError(`Authentication failed: token exchange failed with status ${tokenExchangeResponse.status}`),
             };
         }
-        const body = (await tokenExchangeResponse.json());
+        let body;
+        try {
+            body = (await tokenExchangeResponse.json());
+        }
+        catch {
+            return {
+                type: "ERROR",
+                error: new CliError("Authentication failed: token exchange returned an invalid response"),
+            };
+        }
         return {
             type: "SUCCESS",
             token: body.access_token,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@get-dx/cli",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "DX CLI",
   "author": "DX <developers@getdx.com>",
   "bugs": {