@saastemly/voidcommerce 0.22.0 → 0.22.1

package/dist/cli.js

@@ -25,7 +25,7 @@ import {
   version,
   voidAppsIn,
   wrangler
-} from "./index-2kqz80nx.js";
+} from "./index-51zee2vf.js";
 import {
   LOCAL_KEY_FILE,
   PRIVATE_KEY_VAR,
@@ -126,7 +126,7 @@ ${color.bold("This will destroy, on Cloudflare:")}
 ` + `  ${color.cyan(`vc teardown --cloudflare --confirm ${project.manifest.shop.domain}`)}`);
   }
   console.log(`▸ wrangler delete (${plan.worker})`);
-  const deleted = await wrangler(bin, ["delete", "--name", plan.worker, "--force"], app, true);
+  const deleted = await wrangler(bin, ["delete", "--name", plan.worker, "--force"], app);
   if (deleted.code !== 0 && !/not found|does not exist|10007|10090/i.test(deleted.out)) {
     return fail(`could not delete the worker:
 ${deleted.out.trim().split(`
@@ -135,7 +135,7 @@ ${deleted.out.trim().split(`
   }
   console.log(`${color.green("✓")} worker gone${/not found|does not exist/i.test(deleted.out) ? color.dim(" (it was already)") : ""}`);
   for (const queue of plan.queues) {
-    const dropped = await wrangler(bin, ["queues", "delete", queue], app, true);
+    const dropped = await wrangler(bin, ["queues", "delete", queue], app);
     if (dropped.code !== 0 && !/not found|does not exist|11009|11018/i.test(dropped.out)) {
       console.error(`${color.yellow("!")} queue "${queue}" could not be deleted: ${dropped.out.trim().split(`
 `).slice(-1)[0]}`);
@@ -144,7 +144,7 @@ ${deleted.out.trim().split(`
     }
   }
   if (plan.d1 && !opts.keepData) {
-    const dropped = await wrangler(bin, ["d1", "delete", plan.d1, "--skip-confirmation"], app, true);
+    const dropped = await wrangler(bin, ["d1", "delete", plan.d1, "--skip-confirmation"], app);
     if (dropped.code !== 0 && !/not found|does not exist|7404/i.test(dropped.out)) {
       return fail(`the worker and queues are gone but D1 "${plan.d1}" is not:
 ${dropped.out.trim().split(`

package/dist/{index-2kqz80nx.js → index-51zee2vf.js}

@@ -1383,7 +1383,7 @@ import color from "picocolors";
 // package.json
 var package_default = {
   name: "@saastemly/voidcommerce",
-  version: "0.22.0",
+  version: "0.22.1",
   description: "Void, with a shop in it. `vc init` walks you through Better Auth, betterCommerce and every plugin; everything else passes through to `void`.",
   type: "module",
   license: "MIT",

package/dist/index.js

@@ -52,7 +52,7 @@ import {
   routeProblem,
   strictDependencies,
   upsertJsonc
-} from "./index-2kqz80nx.js";
+} from "./index-51zee2vf.js";
 import {
   allEnvKeys,
   envSummary,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saastemly/voidcommerce",
-  "version": "0.22.0",
+  "version": "0.22.1",
   "description": "Void, with a shop in it. `vc init` walks you through Better Auth, betterCommerce and every plugin; everything else passes through to `void`.",
   "type": "module",
   "license": "MIT",

package/src/deploy/teardown.ts

@@ -116,12 +116,18 @@ export async function teardownCloudflare(project: Project, opts: TeardownOptions
 	}
 
 	/**
+	 * Every call CAPTURES its output rather than streaming it, because
+	 * idempotency depends on reading what wrangler said. `wrangler(..., true)`
+	 * inherits stdio, which leaves `out` empty — so "this worker does not
+	 * exist" could never be recognised and a second teardown failed where it
+	 * should have been a no-op.
+	 *
 	 * Order matters. The worker goes FIRST: while it is live it serves
 	 * customers, and a shop that answers with a database that no longer
 	 * exists is worse than a shop that does not answer.
 	 */
 	console.log(`▸ wrangler delete (${plan.worker})`);
-	const deleted = await wrangler(bin, ["delete", "--name", plan.worker, "--force"], app, true);
+	const deleted = await wrangler(bin, ["delete", "--name", plan.worker, "--force"], app);
 	// "not found" is success here: teardown is idempotent by design.
 	if (deleted.code !== 0 && !/not found|does not exist|10007|10090/i.test(deleted.out)) {
 		return fail(`could not delete the worker:\n${deleted.out.trim().split("\n").slice(-3).join("\n")}`);
@@ -129,7 +135,7 @@ export async function teardownCloudflare(project: Project, opts: TeardownOptions
 	console.log(`${color.green("✓")} worker gone${/not found|does not exist/i.test(deleted.out) ? color.dim(" (it was already)") : ""}`);
 
 	for (const queue of plan.queues) {
-		const dropped = await wrangler(bin, ["queues", "delete", queue], app, true);
+		const dropped = await wrangler(bin, ["queues", "delete", queue], app);
 		if (dropped.code !== 0 && !/not found|does not exist|11009|11018/i.test(dropped.out)) {
 			console.error(`${color.yellow("!")} queue "${queue}" could not be deleted: ${dropped.out.trim().split("\n").slice(-1)[0]}`);
 		} else {
@@ -138,7 +144,7 @@ export async function teardownCloudflare(project: Project, opts: TeardownOptions
 	}
 
 	if (plan.d1 && !opts.keepData) {
-		const dropped = await wrangler(bin, ["d1", "delete", plan.d1, "--skip-confirmation"], app, true);
+		const dropped = await wrangler(bin, ["d1", "delete", plan.d1, "--skip-confirmation"], app);
 		if (dropped.code !== 0 && !/not found|does not exist|7404/i.test(dropped.out)) {
 			return fail(`the worker and queues are gone but D1 "${plan.d1}" is not:\n${dropped.out.trim().split("\n").slice(-3).join("\n")}`);
 		}