@groundfloorcloud/cli 0.2.0 → 0.2.1

package/README.md

@@ -180,7 +180,7 @@ wrong identity for testing Coderunner Vault/Files as the workload.
 | `--app-id <uuid>` | Bind a **new** coderunner to an App as a helper. Ignored on reuse. |
 | `--secret <NAME>` | Inject a workspace secret (repeatable). Only works where the platform has injection enabled; prefer runtime reads via M2M `secrets:read`. |
 | `--api-url <url>` | Control Plane base URL override. |
-| `-y, --yes` | Skip the reuse confirmation (required without a terminal). |
+| `-y, --yes` | Skip the reuse confirmation (required without a terminal) and the prompt shown with `-c` when the deploy would drop previously deployed env variables (key names only; values are not stored). |
 | `-p, --path <dir>` | Directory to package (default: cwd). |
 | `--git <url>` / `--ref <ref>` | Clone and deploy a remote repo. |
 | `--subdir <dir>` | Package a subdirectory of the source. |

package/dist/index.js

@@ -752,6 +752,26 @@ async function getDeployment(apiUrl, workspaceId, coderunnerId) {
   }
   return await res.json();
 }
+async function getDeploymentById(apiUrl, workspaceId, coderunnerId, deploymentId) {
+  const token = await getValidAccessToken();
+  const res = await fetch(
+    `${apiUrl}/v1/workspaces/${encodeURIComponent(
+      workspaceId
+    )}/coderunners/${encodeURIComponent(
+      coderunnerId
+    )}/deployments/${encodeURIComponent(deploymentId)}`,
+    {
+      headers: {
+        Authorization: `Bearer ${token}`,
+        Accept: "application/json"
+      }
+    }
+  );
+  if (!res.ok) {
+    throw new CpError(res.status, await parseError(res));
+  }
+  return await res.json();
+}
 function deleteCoderunnerDeployment(apiUrl, workspaceId, coderunnerId, deploymentId) {
   return cpDelete(
     apiUrl,
@@ -2041,9 +2061,12 @@ function priorEnvKeys(coderunner2) {
   const keys = manifest?.coderunner?.deploy?.env_keys;
   return Array.isArray(keys) ? keys.map(String) : [];
 }
-function envRedeployWarning(priorKeys, sentKeys) {
+function envKeysNotResent(priorKeys, sentKeys) {
   const sent = new Set(sentKeys);
-  const missing = priorKeys.filter((k) => !sent.has(k)).sort();
+  return priorKeys.filter((k) => !sent.has(k)).sort();
+}
+function envRedeployWarning(priorKeys, sentKeys) {
+  const missing = envKeysNotResent(priorKeys, sentKeys);
   const base = 'Environment variable VALUES are not stored by Groundfloor: every deploy must resend them with -e KEY=VALUE or groundfloor.json "env".';
   if (priorKeys.length > 0) {
     if (missing.length === 0) return [];
@@ -2076,6 +2099,7 @@ var VERSION_POLL_INTERVAL_MS = 2e3;
 var VERSION_POLL_TIMEOUT_MS = 5 * 60 * 1e3;
 var DEPLOY_POLL_INTERVAL_MS = 2e3;
 var DEPLOY_POLL_TIMEOUT_MS = 5 * 60 * 1e3;
+var DEPLOY_TERMINAL_FAILURE = /* @__PURE__ */ new Set([4, 5, 6]);
 function slugify(input) {
   const slug = input.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "");
   return slug || "app";
@@ -2235,19 +2259,30 @@ async function deployCommand(opts) {
     const injectSecretKeys = (opts.secret ?? []).map((key) => key.trim()).filter(Boolean);
     let coderunnerId = opts.coderunner;
     if (coderunnerId && !opts.skipDeploy) {
+      let known;
       try {
-        const known = (await listCoderunners(cfg.apiUrl, ws)).find(
+        known = (await listCoderunners(cfg.apiUrl, ws)).find(
           (c) => c.id === coderunnerId
         );
-        if (known) {
-          for (const line of envRedeployWarning(
-            priorEnvKeys(known),
-            Object.keys(envVariable)
-          )) {
-            log(line);
+      } catch {
+      }
+      if (known) {
+        const prior = priorEnvKeys(known);
+        const sentKeys = Object.keys(envVariable);
+        const warning = envRedeployWarning(prior, sentKeys);
+        if (envKeysNotResent(prior, sentKeys).length > 0) {
+          if (io.isTTY && !opts.yes) {
+            await confirmAction({
+              summary: warning,
+              question: "Deploy anyway and clear these variables?",
+              io
+            });
+          } else {
+            for (const line of warning) log(line);
           }
+        } else {
+          for (const line of warning) log(line);
         }
-      } catch {
       }
     }
     if (opts.appId && coderunnerId) {
@@ -2400,24 +2435,43 @@ async function deployCommand(opts) {
       throw err;
     }
     log("Waiting for deployment...");
+    const newDeploymentId = result.deployment_id || void 0;
+    const readNewDeployment = async () => {
+      if (newDeploymentId) {
+        try {
+          return await getDeploymentById(
+            cfg.apiUrl,
+            ws,
+            coderunnerId,
+            newDeploymentId
+          );
+        } catch (err) {
+          if (err instanceof CpError && err.status === 404) return null;
+          throw err;
+        }
+      }
+      const current = await getDeployment(cfg.apiUrl, ws, coderunnerId);
+      return current?.version_id === uploaded.version_id ? current : null;
+    };
     const deployDeadline = Date.now() + DEPLOY_POLL_TIMEOUT_MS;
     let deployOk = false;
     while (Date.now() < deployDeadline) {
-      const deployment = await getDeployment(cfg.apiUrl, ws, coderunnerId);
+      const deployment = await readNewDeployment();
       const dStatus = deployment?.status ?? 0;
       if (dStatus === 3) {
         deployOk = true;
         break;
       }
-      if (dStatus === 4) {
+      if (DEPLOY_TERMINAL_FAILURE.has(dStatus)) {
+        process.stderr.write("\n");
         return await failDeploy({
           apiUrl: cfg.apiUrl,
           workspaceId: ws,
-          headline: `Deployment failed for coderunner ${coderunnerId} (${statusLabel(4)}).`,
+          headline: `Deployment failed for coderunner ${coderunnerId} (${statusLabel(dStatus)}).`,
           coderunnerId,
           versionId: uploaded.version_id,
-          deploymentId: result.deployment_id ?? deployment?.id,
-          status: 4
+          deploymentId: newDeploymentId ?? deployment?.id,
+          status: dStatus
         });
       }
       process.stderr.write(".");
@@ -2426,9 +2480,9 @@ async function deployCommand(opts) {
     process.stderr.write("\n");
     if (!deployOk) {
       let lastStatus = null;
-      let deploymentId = result.deployment_id;
+      let deploymentId = newDeploymentId;
       try {
-        const latest = await getDeployment(cfg.apiUrl, ws, coderunnerId);
+        const latest = await readNewDeployment();
         lastStatus = latest?.status ?? null;
         deploymentId = latest?.id ?? deploymentId;
       } catch {
@@ -4164,7 +4218,7 @@ program.command("deploy").description(
   []
 ).option(
   "-y, --yes",
-  "Confirm updating an existing same-named coderunner (required with no interactive terminal). Does not bypass the saved-default-workspace check: pass -w"
+  "Confirm updating an existing same-named coderunner (required with no interactive terminal), and skip the prompt about previously deployed env variables that this deploy does not resend (-c). Does not bypass the saved-default-workspace check: pass -w"
 ).option("-m, --message <text>", "Version description").option("--skip-deploy", "Upload a new version but do not deploy it").action((opts) => run(() => deployCommand(opts)));
 program.command("usage").description(
   "Show account quota/usage and the billing estimate (read-only). The account is the billing parent of the workspace, not the workspace itself"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groundfloorcloud/cli",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Groundfloor gf CLI — sign in, deploy coderunners, publish Shell apps, and manage secrets, files, and Dataplane.",
   "type": "module",
   "bin": {