@blankdotpage/cli 0.1.0 → 0.1.2

package/README.md

@@ -22,7 +22,6 @@ By default, CLI commands target:
 Override target endpoint with either:
 
 - `--app-url https://your-preview.fly.dev`
-- `BLANKPAGE_CLI_APP_URL=https://your-preview.fly.dev`
 - `BLANKPAGE_APP_URL=https://your-preview.fly.dev`
 
 ## Commands

package/dist/index.js

@@ -3,7 +3,6 @@
 // src/index.ts
 import fs2 from "node:fs/promises";
 import os2 from "node:os";
-import { spawn } from "node:child_process";
 import { URL } from "node:url";
 
 // src/lib/args.ts
@@ -41,10 +40,67 @@ function booleanFlag(flags, key) {
   return flags[key] === true;
 }
 
+// src/lib/browser.ts
+import {
+  spawn
+} from "node:child_process";
+function getLaunchCommand(platform, url) {
+  if (platform === "darwin") {
+    return { command: "open", args: [url] };
+  }
+  if (platform === "linux") {
+    return { command: "xdg-open", args: [url] };
+  }
+  if (platform === "win32") {
+    return { command: "cmd", args: ["/c", "start", "", url] };
+  }
+  return null;
+}
+function toError(error) {
+  return error instanceof Error ? error : new Error(String(error));
+}
+function openBrowser(url, options = {}) {
+  const platform = options.platform ?? process.platform;
+  const launch = getLaunchCommand(platform, url);
+  if (!launch) {
+    return false;
+  }
+  const spawnCommand = options.spawnCommand ?? ((command, args, spawnOptions) => spawn(command, args, spawnOptions));
+  const onError = options.onError ?? (() => {
+  });
+  try {
+    const child = spawnCommand(launch.command, launch.args, {
+      stdio: "ignore",
+      detached: true
+    });
+    child.once("error", (error) => {
+      onError(toError(error));
+    });
+    child.unref();
+    return true;
+  } catch (error) {
+    onError(toError(error));
+    return false;
+  }
+}
+
 // src/lib/config.ts
 import os from "node:os";
 import path from "node:path";
 import fs from "node:fs/promises";
+function isObject(value) {
+  return typeof value === "object" && value !== null;
+}
+function readOptionalString(input, key) {
+  const value = input[key];
+  if (value === void 0) {
+    return void 0;
+  }
+  if (typeof value !== "string") {
+    throw new Error(`Invalid CLI config value for "${key}"`);
+  }
+  return value;
+}
 function resolveConfigPath() {
   if (process.env.BLANKPAGE_CLI_CONFIG_PATH) {
     return process.env.BLANKPAGE_CLI_CONFIG_PATH;
@@ -55,15 +111,39 @@ async function loadConfig() {
   const configPath = resolveConfigPath();
   try {
     const raw = await fs.readFile(configPath, "utf8");
-    return JSON.parse(raw);
-  } catch {
-    return {};
+    let parsed;
+    try {
+      parsed = JSON.parse(raw);
+    } catch (error) {
+      throw new Error(
+        `Failed to parse CLI config file at ${configPath}: ${error instanceof Error ? error.message : String(error)}`
+      );
+    }
+    if (!isObject(parsed)) {
+      throw new Error(
+        `Invalid CLI config file at ${configPath}: expected an object`
+      );
+    }
+    return {
+      appUrl: readOptionalString(parsed, "appUrl"),
+      token: readOptionalString(parsed, "token"),
+      tokenExpiresAt: readOptionalString(parsed, "tokenExpiresAt")
+    };
+  } catch (error) {
+    if (error.code === "ENOENT") {
+      return {};
+    }
+    throw error;
   }
 }
 async function saveConfig(config) {
   const configPath = resolveConfigPath();
-  await fs.mkdir(path.dirname(configPath), { recursive: true });
-  await fs.writeFile(configPath, JSON.stringify(config, null, 2), "utf8");
+  await fs.mkdir(path.dirname(configPath), { recursive: true, mode: 448 });
+  await fs.writeFile(configPath, JSON.stringify(config, null, 2), {
+    encoding: "utf8",
+    mode: 384
+  });
+  await fs.chmod(configPath, 384);
 }
 
 // src/lib/http.ts
@@ -195,7 +275,7 @@ function generatePageId() {
 
 // src/index.ts
 function getDefaultAppUrl() {
-  return process.env.BLANKPAGE_CLI_APP_URL || process.env.BLANKPAGE_APP_URL || "https://blank.page";
+  return process.env.BLANKPAGE_APP_URL || "https://blank.page";
 }
 function resolveAppUrl({
   appUrlFlag,
@@ -212,23 +292,6 @@ function requireToken(token) {
 function sleep(ms) {
   return new Promise((resolve) => setTimeout(resolve, ms));
 }
-function maybeOpenBrowser(url) {
-  const platform = process.platform;
-  if (platform === "darwin") {
-    spawn("open", [url], { stdio: "ignore", detached: true }).unref();
-    return;
-  }
-  if (platform === "linux") {
-    spawn("xdg-open", [url], { stdio: "ignore", detached: true }).unref();
-    return;
-  }
-  if (platform === "win32") {
-    spawn("cmd", ["/c", "start", "", url], {
-      stdio: "ignore",
-      detached: true
-    }).unref();
-  }
-}
 function output(json, payload, text) {
   if (json) {
     printJson(payload);
@@ -265,13 +328,16 @@ async function commandLogin(ctx) {
     {
       event: "login_started",
       authorizeUrl: started.authorizeUrl,
-      challengeId: started.challengeId,
-      challengeSecret: started.challengeSecret
+      challengeId: started.challengeId
     },
     `Open this URL to authorize: ${started.authorizeUrl}`
   );
   if (!noBrowser) {
-    maybeOpenBrowser(started.authorizeUrl);
+    openBrowser(started.authorizeUrl, {
+      onError: (error) => {
+        printError(`Could not open browser automatically: ${error.message}`);
+      }
+    });
   }
   const deadline = Date.now() + timeoutSeconds * 1e3;
   while (Date.now() < deadline) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blankdotpage/cli",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "CLI for Blank Page",
   "type": "module",
   "files": [