[object Object]

← back to Designer Wallcoverings

docs: correct stale --dry-run comment across 9 write-scripts

7c89163b0d74b0334bcbcc7507b290940f9413de · 2026-09-15 08:23:17 -0700 · Steve

The comment claimed '--dry-run is now the default and is accepted but ignored',
but the guard was changed to (args.includes('--dry-run') || !args.includes('--apply')),
which HONORS an explicit --dry-run and lets it override --apply. Comment now matches code.
Found by DTD contrarian review of HEAD; no code change, comment-only.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019ttR6L74ty1TWAR4eeWXyT

Files touched

Diff

commit 7c89163b0d74b0334bcbcc7507b290940f9413de
Author: Steve <steve@designerwallcoverings.com>
Date:   Tue Sep 15 08:23:17 2026 -0700

    docs: correct stale --dry-run comment across 9 write-scripts
    
    The comment claimed '--dry-run is now the default and is accepted but ignored',
    but the guard was changed to (args.includes('--dry-run') || !args.includes('--apply')),
    which HONORS an explicit --dry-run and lets it override --apply. Comment now matches code.
    Found by DTD contrarian review of HEAD; no code change, comment-only.
    
    Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
    Claude-Session: https://claude.ai/code/session_019ttR6L74ty1TWAR4eeWXyT
---
 DW-Programming/command54-shopify-push.js          | 2 +-
 DW-Programming/fabricut-activate.js               | 2 +-
 DW-Programming/hollywood-shopify-push.js          | 2 +-
 DW-Programming/justindavid-shopify-push.js        | 2 +-
 scripts/interior-design-tags.js                   | 2 +-
 scripts/populate-mfr-sku-metafield.js             | 2 +-
 shopify/scripts/add-missing-samples-v2.js         | 2 +-
 shopify/scripts/create-missing-sample-variants.js | 2 +-
 shopify/scripts/thibaut-photo-fix.js              | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/DW-Programming/command54-shopify-push.js b/DW-Programming/command54-shopify-push.js
index d04aafcb..7d43adf4 100644
--- a/DW-Programming/command54-shopify-push.js
+++ b/DW-Programming/command54-shopify-push.js
@@ -16,7 +16,7 @@
  *   node command54-shopify-push.js --limit 100      # Queue first 100
  *   node command54-shopify-push.js --all             # Queue ALL push-ready
  *   node command54-shopify-push.js --dry-run         # Preview only (no DB writes)
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  *   node command54-shopify-push.js --dry-run --limit 5  # Preview 5 products
  *   node command54-shopify-push.js --reconcile       # Link completed jobs to vendor_catalog
  *   node command54-shopify-push.js --status          # Show push status
diff --git a/DW-Programming/fabricut-activate.js b/DW-Programming/fabricut-activate.js
index 134ced29..ebd4ba5b 100644
--- a/DW-Programming/fabricut-activate.js
+++ b/DW-Programming/fabricut-activate.js
@@ -10,7 +10,7 @@
  *      is fed by the controlled TSV feed (real retail) only.
  *
  * Usage: node fabricut-activate.js [--dry-run] [--limit N]
- * Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ * Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  */
 const { Pool } = require('pg');
 const https = require('https');
diff --git a/DW-Programming/hollywood-shopify-push.js b/DW-Programming/hollywood-shopify-push.js
index b21837a5..c413e658 100644
--- a/DW-Programming/hollywood-shopify-push.js
+++ b/DW-Programming/hollywood-shopify-push.js
@@ -23,7 +23,7 @@
  *   node hollywood-shopify-push.js --limit 100      # Queue first 100
  *   node hollywood-shopify-push.js --all             # Queue ALL 8,091 push-ready
  *   node hollywood-shopify-push.js --dry-run         # Preview only (no DB writes)
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  *   node hollywood-shopify-push.js --dry-run --limit 5  # Preview 5 products
  *   node hollywood-shopify-push.js --reconcile       # Link completed jobs to vendor_catalog
  *   node hollywood-shopify-push.js --status          # Show push status
diff --git a/DW-Programming/justindavid-shopify-push.js b/DW-Programming/justindavid-shopify-push.js
index 2b0d7cb1..bc46b8a6 100644
--- a/DW-Programming/justindavid-shopify-push.js
+++ b/DW-Programming/justindavid-shopify-push.js
@@ -14,7 +14,7 @@
  *   node justindavid-shopify-push.js              # Queue all ready products
  *   node justindavid-shopify-push.js --limit 100  # Queue first 100
  *   node justindavid-shopify-push.js --dry-run    # Preview only
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  *   node justindavid-shopify-push.js --reconcile  # Update tracking from completed jobs
  *   node justindavid-shopify-push.js --status     # Show push status
  */
diff --git a/scripts/interior-design-tags.js b/scripts/interior-design-tags.js
index 425db09a..bd643732 100644
--- a/scripts/interior-design-tags.js
+++ b/scripts/interior-design-tags.js
@@ -18,7 +18,7 @@
  *   node scripts/interior-design-tags.js --limit 50         # Limit per table
  *   node scripts/interior-design-tags.js --shopify-only     # Skip Gemini, just push to Shopify
  *   node scripts/interior-design-tags.js --dry-run          # Preview without saving
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  */
 
 const { Pool } = require('pg');
diff --git a/scripts/populate-mfr-sku-metafield.js b/scripts/populate-mfr-sku-metafield.js
index 1a85120b..a5ef3f7d 100644
--- a/scripts/populate-mfr-sku-metafield.js
+++ b/scripts/populate-mfr-sku-metafield.js
@@ -11,7 +11,7 @@
  * 3. Handle patterns
  *
  * Usage: node populate-mfr-sku-metafield.js [--dry-run] [--vendor=VendorName]
- * Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ * Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  */
 
 const SHOPIFY_STORE = 'designer-laboratory-sandbox';
diff --git a/shopify/scripts/add-missing-samples-v2.js b/shopify/scripts/add-missing-samples-v2.js
index 119851f0..8962c0c3 100644
--- a/shopify/scripts/add-missing-samples-v2.js
+++ b/shopify/scripts/add-missing-samples-v2.js
@@ -10,7 +10,7 @@
  *   node add-missing-samples-v2.js              # Run (resumes from progress)
  *   node add-missing-samples-v2.js --fresh      # Ignore progress, full rescan
  *   node add-missing-samples-v2.js --dry-run    # Scan only, no creates
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  *   node add-missing-samples-v2.js --scan-only  # Scan + report, skip creates
  */
 
diff --git a/shopify/scripts/create-missing-sample-variants.js b/shopify/scripts/create-missing-sample-variants.js
index aed8d587..bf402f37 100644
--- a/shopify/scripts/create-missing-sample-variants.js
+++ b/shopify/scripts/create-missing-sample-variants.js
@@ -9,7 +9,7 @@
  *   node create-missing-sample-variants.js           # Start/resume from progress file
  *   node create-missing-sample-variants.js --fresh    # Ignore progress, start from beginning
  *   node create-missing-sample-variants.js --dry-run  # Just show what would be created
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  */
 
 const https = require('https');
diff --git a/shopify/scripts/thibaut-photo-fix.js b/shopify/scripts/thibaut-photo-fix.js
index 78f43049..1baca67c 100644
--- a/shopify/scripts/thibaut-photo-fix.js
+++ b/shopify/scripts/thibaut-photo-fix.js
@@ -10,7 +10,7 @@
  *            present does it delete the recorded low-res media ids (so the
  *            2520 becomes the sole / featured image). Safe + idempotent.
  *   --dry-run  with either pass: report, no writes.
- *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is now the default and is accepted but ignored.
+ *   Default run-mode is DRY (2026-09-14): this script is DRY BY DEFAULT and no longer acts without --apply. Pass --apply to WRITE; --dry-run is the default and, if passed explicitly, overrides --apply (dry-run always wins).
  */
 const fs = require('fs');
 const path = require('path');

← 7a561a32 auto-data-snapshot: 2026-09-15T07:41:08 (5 data files) — DW-  ·  back to Designer Wallcoverings  ·  TK-11539: apply weight guard to Schumacher sample-variant ch 335c4269 →