mass-push-etsy: don't pass --commit to bundle-gen in --dry-run (no state mutation / no file writes on dry-run)

bee700ddc0be97259d75553cb0ef41a0cc038eda · 2026-06-03 20:59:15 -0700 · Steve Abrams

Files touched

M scripts/mass-push-etsy.js

Diff

commit bee700ddc0be97259d75553cb0ef41a0cc038eda
Author: Steve Abrams <steve@designerwallcoverings.com>
Date:   Wed Jun 3 20:59:15 2026 -0700

    mass-push-etsy: don't pass --commit to bundle-gen in --dry-run (no state mutation / no file writes on dry-run)
---
 scripts/mass-push-etsy.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/scripts/mass-push-etsy.js b/scripts/mass-push-etsy.js
index 45cce49..7db9eac 100644
--- a/scripts/mass-push-etsy.js
+++ b/scripts/mass-push-etsy.js
@@ -117,8 +117,13 @@ async function main() {
       '--from-bucket',
       '--by', opt.by,
       '--byte-budget', String(opt.byteBudget),
-      '--commit',
     ];
+    // --commit writes listing.json + flips bucket rows queued→exported in PG.
+    // A --dry-run must NOT mutate state or write files (Steve: "only create
+    // files when we upload to etsy"). Only commit on a real drain. When
+    // dry-running, the generator just prints the bundle table — which is all
+    // the validation needs.
+    if (!opt.dryRun) genArgs.push('--commit');
     const r = spawnSync('node', genArgs, { stdio: 'inherit' });
     if (r.status !== 0) { console.error('  ✗ bundle-gen failed (exit ' + r.status + ')'); process.exit(1); }
   } else {

← 94fd03d yolo ledger: T4 live building cards + batch-2 recovery · back to Wallco Ai · log: etsy option-C validated (all 12 bundles under 90MB by a d247186 →