create-contractor-site 2.1.1 → 2.2.0

package/scripts/smoke-test.mjs

@@ -6,10 +6,12 @@
  * 1. Missing target argument
  * 2. Existing non-empty target
  * 3. Target equal to / inside template root
- * 4. Duplicate service input normalization + slug uniqueness + business/services alignment
- * 5. Service-area name parse/dedupe (Chesapeake duplicate-slug case) + areas rebuild
+ * 4. Published DEFAULT_TEMPLATE_REF / CREATE_CONTRACTOR_TEMPLATE_REF contract (v2.2.0)
+ * 5. Duplicate service input normalization + slug uniqueness + business/services alignment
+ * 6. Service-area name parse/dedupe (Chesapeake duplicate-slug case) + areas rebuild
  *    without leaking stale template county/ZIP metadata into new areas
- * 6. Temp-target --yes scaffold (install/validate/build) unless SKIP_CLI_E2E=1
+ * 7. CREATE_CONTRACTOR_SITE_ANSWERS_JSON real CLI spawn (copy+replace, skip setup)
+ * 8. Temp-target --yes scaffold (install/validate/build) unless SKIP_CLI_E2E=1
  */
 
 import { spawnSync } from 'node:child_process';
@@ -26,9 +28,30 @@ import {
   replaceTargetData,
   slugify,
 } from '../src/replace-data.mjs';
-import { buildAnswers } from '../src/prompts.mjs';
+import {
+  buildAnswers,
+  DEFAULT_FREE_ESTIMATE,
+  DEFAULT_HOURS,
+  DEFAULT_INSURANCE,
+  DEFAULT_LICENSE,
+  DEFAULT_PAYMENT_METHODS,
+  DEFAULT_SITE_TYPE,
+  DEFAULT_YEARS_EXPERIENCE,
+  normalizeDirectories,
+  normalizeHours,
+  normalizePaymentMethods,
+  normalizeSiteType,
+  normalizeSocial,
+  optionalText,
+  requiredText,
+  SITE_TYPES,
+  SOCIAL_NETWORK_KEYS,
+} from '../src/prompts.mjs';
 import { isSameOrInside, validateTarget, TargetValidationError } from '../src/validate-target.mjs';
-import { findLocalTemplateRoot } from '../src/copy-template.mjs';
+import {
+  DEFAULT_TEMPLATE_REF,
+  findLocalTemplateRoot,
+} from '../src/copy-template.mjs';
 import { isVersionAtLeast, resolveCommandPath } from '../src/run-command.mjs';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
@@ -63,6 +86,32 @@ function assert(condition, message) {
   if (!condition) throw new Error(message);
 }
 
+/** @param {string} file */
+function readJson(file) {
+  return JSON.parse(fs.readFileSync(file, 'utf8'));
+}
+
+/** Seed template data files into a temp target. @param {string} prefix */
+function seedDataDir(prefix) {
+  assert(REPO_ROOT, 'expected local template root');
+  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
+  const dataSrc = path.join(REPO_ROOT, 'src', 'data');
+  const dataDst = path.join(tmp, 'src', 'data');
+  fs.mkdirSync(dataDst, { recursive: true });
+  for (const f of [
+    'business.json',
+    'site.json',
+    'services.json',
+    'areas.json',
+    'navigation.json',
+    'landings.json',
+    'directories.json',
+  ]) {
+    fs.copyFileSync(path.join(dataSrc, f), path.join(dataDst, f));
+  }
+  return { tmp, dataDst };
+}
+
 /**
  * @param {string[]} args
  * @param {{ env?: NodeJS.ProcessEnv, cwd?: string }} [opts]
@@ -95,6 +144,43 @@ async function main() {
     assert(!isVersionAtLeast('10.9.9', '11.1.2'), 'older major should fail');
   });
 
+  await test(
+    'DEFAULT_TEMPLATE_REF / CREATE_CONTRACTOR_TEMPLATE_REF published contract is v2.2.0',
+    () => {
+      const copySrcPath = path.join(PKG_ROOT, 'src', 'copy-template.mjs');
+      const copySrc = fs.readFileSync(copySrcPath, 'utf8');
+      // Pin the source fallback so a silent ref bump cannot ship without updating this test.
+      assert(
+        /CREATE_CONTRACTOR_TEMPLATE_REF\s*\|\|\s*['"]v2\.2\.0['"]/.test(copySrc),
+        'copy-template.mjs DEFAULT_TEMPLATE_REF fallback must be v2.2.0',
+      );
+      const expected =
+        process.env.CREATE_CONTRACTOR_TEMPLATE_REF || 'v2.2.0';
+      assert(
+        DEFAULT_TEMPLATE_REF === expected,
+        `DEFAULT_TEMPLATE_REF must equal CREATE_CONTRACTOR_TEMPLATE_REF or v2.2.0 (got ${DEFAULT_TEMPLATE_REF})`,
+      );
+      if (!process.env.CREATE_CONTRACTOR_TEMPLATE_REF) {
+        assert(
+          DEFAULT_TEMPLATE_REF === 'v2.2.0',
+          `published default template ref must be v2.2.0 (got ${DEFAULT_TEMPLATE_REF})`,
+        );
+      }
+
+      const help = runCli(['--help']);
+      assert(help.status === 0, `help exit 0, got ${help.status}`);
+      const helpOut = `${help.stdout}\n${help.stderr}`;
+      assert(
+        /CREATE_CONTRACTOR_TEMPLATE_REF/i.test(helpOut),
+        'help must document CREATE_CONTRACTOR_TEMPLATE_REF',
+      );
+      assert(
+        /v2\.2\.0/.test(helpOut),
+        'help must document default template ref v2.2.0',
+      );
+    },
+  );
+
   await test('missing target argument exits 1 with usage', () => {
     const result = runCli([]);
     assert(result.status === 1, `expected exit 1, got ${result.status}`);
@@ -299,6 +385,7 @@ async function main() {
       'areas.json',
       'navigation.json',
       'landings.json',
+      'directories.json',
     ]) {
       fs.copyFileSync(path.join(dataSrc, file), path.join(dataDst, file));
     }
@@ -376,6 +463,594 @@ async function main() {
     fs.rmSync(tmp, { recursive: true, force: true });
   });
 
+  await test('buildAnswers normalizes blank text trust fields and foundedYear', () => {
+    const answers = buildAnswers({
+      businessName: 'Trust Co',
+      primaryServices: ['Masonry'],
+      freeEstimate: '   ',
+      yearsExperience: '\t',
+      license: '',
+      insurance: null,
+      foundedYear: '  ',
+    });
+    assert(answers.freeEstimate === DEFAULT_FREE_ESTIMATE, 'freeEstimate default');
+    assert(answers.yearsExperience === DEFAULT_YEARS_EXPERIENCE, 'yearsExperience default');
+    assert(answers.license === DEFAULT_LICENSE, 'license default');
+    assert(answers.insurance === DEFAULT_INSURANCE, 'insurance default');
+    assert(answers.foundedYear === '', 'foundedYear blank → ""');
+    assert(requiredText('  x  ', 'y') === 'x', 'requiredText trims');
+    assert(optionalText('  2012 ') === '2012', 'optionalText trims');
+    assert(optionalText(undefined) === '', 'optionalText missing → ""');
+  });
+
+  await test('normalizePaymentMethods CSV/array blank → defaults never []', () => {
+    assert(
+      JSON.stringify(normalizePaymentMethods('Cash,  Check , ,Credit Card')) ===
+        JSON.stringify(['Cash', 'Check', 'Credit Card']),
+      'CSV trims and drops blanks',
+    );
+    assert(
+      JSON.stringify(normalizePaymentMethods(['  Visa ', '', 'Cash'])) ===
+        JSON.stringify(['Visa', 'Cash']),
+      'array trims and drops blanks',
+    );
+    assert(
+      JSON.stringify(normalizePaymentMethods('   ')) ===
+        JSON.stringify(DEFAULT_PAYMENT_METHODS),
+      'whitespace CSV → defaults',
+    );
+    assert(
+      JSON.stringify(normalizePaymentMethods([])) ===
+        JSON.stringify(DEFAULT_PAYMENT_METHODS),
+      'empty array → defaults',
+    );
+    assert(
+      JSON.stringify(normalizePaymentMethods(null)) ===
+        JSON.stringify(DEFAULT_PAYMENT_METHODS),
+      'null → defaults',
+    );
+    assert(normalizePaymentMethods('').length > 0, 'never empty list');
+  });
+
+  await test('normalizeHours keeps 3-row {days,time} shape with safe defaults', () => {
+    const fromBlank = normalizeHours(undefined);
+    assert(fromBlank.length === 3, 'default length 3');
+    assert(
+      JSON.stringify(fromBlank) === JSON.stringify(DEFAULT_HOURS),
+      'missing → DEFAULT_HOURS',
+    );
+    assert(
+      JSON.stringify(normalizeHours([])) === JSON.stringify(DEFAULT_HOURS),
+      'empty array → defaults',
+    );
+
+    const partial = normalizeHours([
+      { days: '  ', time: '9:00 AM - 5:00 PM' },
+      '  10:00 AM - 1:00 PM ',
+      { days: 'Sunday', time: '  ' },
+    ]);
+    assert(partial.length === 3, 'partial still length 3');
+    assert(partial[0].days === DEFAULT_HOURS[0].days, 'blank days → default days');
+    assert(partial[0].time === '9:00 AM - 5:00 PM', 'weekday time kept');
+    assert(partial[1].days === DEFAULT_HOURS[1].days, 'string row uses default days');
+    assert(partial[1].time === '10:00 AM - 1:00 PM', 'string row trims time');
+    assert(partial[2].days === 'Sunday', 'sunday days kept');
+    assert(partial[2].time === DEFAULT_HOURS[2].time, 'blank time → default');
+
+    const compact = buildAnswers({
+      businessName: 'Hours Co',
+      primaryServices: ['Masonry'],
+      hoursWeekday: '8-5',
+      hoursSaturday: '  ',
+      hoursSunday: 'Closed',
+    });
+    assert(compact.hours.length === 3, 'compact → 3 rows');
+    assert(compact.hours[0].time === '8-5', 'compact weekday');
+    assert(compact.hours[1].time === DEFAULT_HOURS[1].time, 'blank sat → default');
+    assert(compact.hours[2].time === 'Closed', 'compact sunday');
+    assert(
+      JSON.stringify(compact.paymentMethods) === JSON.stringify(DEFAULT_PAYMENT_METHODS),
+      'omitted payments → defaults',
+    );
+  });
+
+  await test('replaceTargetData writes trust + payment + hours + empty social', () => {
+    assert(REPO_ROOT, 'expected local template root');
+    const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'ccs-trust-'));
+    const dataSrc = path.join(REPO_ROOT, 'src', 'data');
+    const dataDst = path.join(tmp, 'src', 'data');
+    fs.mkdirSync(dataDst, { recursive: true });
+    for (const file of [
+      'business.json',
+      'site.json',
+      'services.json',
+      'areas.json',
+      'navigation.json',
+      'landings.json',
+      'directories.json',
+    ]) {
+      fs.copyFileSync(path.join(dataSrc, file), path.join(dataDst, file));
+    }
+
+    const original = JSON.parse(
+      fs.readFileSync(path.join(dataDst, 'business.json'), 'utf8'),
+    );
+    const originalDirs = JSON.parse(
+      fs.readFileSync(path.join(dataDst, 'directories.json'), 'utf8'),
+    );
+
+    const answers = buildAnswers({
+      businessName: 'Trust Builders',
+      primaryServices: ['Masonry'],
+      freeEstimate: 'Free Quote Today',
+      yearsExperience: '20+',
+      license: 'VA Class A #12345',
+      insurance: 'Bonded and insured.',
+      foundedYear: '',
+      paymentMethods: 'Cash, Zelle',
+      hours: [
+        { days: 'Monday - Friday', time: '8:00 AM - 4:00 PM' },
+        { days: 'Saturday', time: 'Closed' },
+        { days: 'Sunday', time: 'Closed' },
+      ],
+    });
+    replaceTargetData(tmp, answers);
+
+    const business = JSON.parse(
+      fs.readFileSync(path.join(dataDst, 'business.json'), 'utf8'),
+    );
+    const site = JSON.parse(fs.readFileSync(path.join(dataDst, 'site.json'), 'utf8'));
+    const directories = JSON.parse(
+      fs.readFileSync(path.join(dataDst, 'directories.json'), 'utf8'),
+    );
+    assert(business._instructions, '_instructions preserved');
+    assert(
+      JSON.stringify(Object.keys(business._instructions)) ===
+        JSON.stringify(Object.keys(original._instructions)),
+      '_instructions keys stable',
+    );
+    assert(business.free_estimate === 'Free Quote Today', 'free_estimate written');
+    assert(business.years_experience === '20+', 'years_experience written');
+    assert(business.license === 'VA Class A #12345', 'license written');
+    assert(business.insurance === 'Bonded and insured.', 'insurance written');
+    assert(
+      Object.prototype.hasOwnProperty.call(business, 'founded_year'),
+      'founded_year key must remain',
+    );
+    assert(business.founded_year === '', 'founded_year skip → ""');
+    assert(
+      JSON.stringify(business.payment_methods) === JSON.stringify(['Cash', 'Zelle']),
+      'payment_methods written',
+    );
+    assert(Array.isArray(business.hours) && business.hours.length === 3, 'hours length 3');
+    assert(business.hours[0].days === 'Monday - Friday', 'hours days shape');
+    assert(business.hours[0].time === '8:00 AM - 4:00 PM', 'hours time written');
+    assert(business.hours[1].time === 'Closed', 'saturday closed');
+    assert(JSON.stringify(business.social) === '{}', 'omitted social → {}');
+    assert(site.features.enable_directories === false, 'no dirs → enable false');
+    assert(directories.directories.length >= 1, 'directories min(1) when none');
+    assert(directories.directories.length === originalDirs.directories.length);
+    assert(directories._instructions && directories.variant === originalDirs.variant);
+    fs.rmSync(tmp, { recursive: true, force: true });
+  });
+
+  await test('normalizeSocial/directories + replace populate/none parity', () => {
+    // Social blank omission + population (object + compact CSV)
+    assert(JSON.stringify(normalizeSocial(undefined)) === '{}', 'missing social → {}');
+    assert(
+      JSON.stringify(normalizeSocial({ facebook: '  ', instagram: null, x: '' })) === '{}',
+      'blank social keys omitted',
+    );
+    const populated = normalizeSocial({
+      facebook: ' https://facebook.com/acme ',
+      instagram: '',
+      youtube: 'https://youtube.com/@acme',
+      unknown: 'https://nope.example',
+    });
+    assert(
+      JSON.stringify(populated) ===
+        JSON.stringify({
+          facebook: 'https://facebook.com/acme',
+          youtube: 'https://youtube.com/@acme',
+        }),
+    );
+    const compact = normalizeSocial(
+      'facebook=https://facebook.com/a, google-business=https://g.co/a, ,tiktok=',
+    );
+    assert(compact.facebook === 'https://facebook.com/a' && compact.google_business === 'https://g.co/a');
+    assert(!Object.prototype.hasOwnProperty.call(compact, 'tiktok'));
+    assert(SOCIAL_NETWORK_KEYS.includes('linkedin'));
+
+    // Directories provided vs none
+    assert(normalizeDirectories(undefined).length === 0);
+    assert(normalizeDirectories([]).length === 0);
+    const fromCsv = normalizeDirectories(
+      'Google Business|https://g.co/acme,  BBB|https://bbb.org/acme ,badrow',
+    );
+    assert(fromCsv.length === 2 && fromCsv[0].url === 'https://g.co/acme' && fromCsv[0].initials);
+    const fromRows = normalizeDirectories([
+      { name: 'Angi', url: 'https://angi.com/x', initials: 'A' },
+      { name: '  ', url: 'https://skip.example' },
+      { name: 'Houzz', url: '  ' },
+    ]);
+    assert(fromRows.length === 1 && fromRows[0].initials === 'A');
+
+    const none = buildAnswers({ businessName: 'No Social', primaryServices: ['Masonry'] });
+    assert(JSON.stringify(none.social) === '{}' && none.directories.length === 0);
+    assert(none.enableDirectories === false);
+    const withDirs = buildAnswers({
+      businessName: 'With Dirs',
+      primaryServices: ['Masonry'],
+      social: { facebook: 'https://facebook.com/x', instagram: '  ' },
+      directories: 'Google|https://g.co/x',
+    });
+    assert(withDirs.social.facebook === 'https://facebook.com/x');
+    assert(!Object.prototype.hasOwnProperty.call(withDirs.social, 'instagram'));
+    assert(withDirs.directories.length === 1 && withDirs.enableDirectories === true);
+
+    // directories:[] + enableDirectories:true → feature false
+    const forcedEmpty = buildAnswers({
+      businessName: 'Forced Empty Dirs',
+      primaryServices: ['Masonry'],
+      directories: [],
+      enableDirectories: true,
+    });
+    assert(forcedEmpty.directories.length === 0 && forcedEmpty.enableDirectories === false);
+
+    // replace: populated social + directories
+    const seeded = seedDataDir('ccs-social-');
+    replaceTargetData(
+      seeded.tmp,
+      buildAnswers({
+        businessName: 'Social Co',
+        primaryServices: ['Patios'],
+        social: {
+          facebook: 'https://facebook.com/socialco',
+          instagram: '',
+          youtube: 'https://youtube.com/@socialco',
+        },
+        directories: [
+          { name: 'Google Business', url: 'https://g.co/socialco', initials: 'G' },
+          { name: 'Angi', url: 'https://angi.com/socialco' },
+        ],
+      }),
+    );
+    const business = readJson(path.join(seeded.dataDst, 'business.json'));
+    const site = readJson(path.join(seeded.dataDst, 'site.json'));
+    const directories = readJson(path.join(seeded.dataDst, 'directories.json'));
+    assert(
+      JSON.stringify(business.social) ===
+        JSON.stringify({
+          facebook: 'https://facebook.com/socialco',
+          youtube: 'https://youtube.com/@socialco',
+        }),
+    );
+    assert(site.features.enable_directories === true);
+    assert(directories.directories.length === 2 && directories.directories[0].initials === 'G');
+    assert(directories.directories[1].initials && directories._instructions);
+
+    // replace: directories:[] + enable true → flag false + min(1) placeholder rows
+    const forced = seedDataDir('ccs-dirs-force-');
+    const origLen = readJson(path.join(forced.dataDst, 'directories.json')).directories.length;
+    replaceTargetData(
+      forced.tmp,
+      buildAnswers({
+        businessName: 'Force Enable Empty',
+        primaryServices: ['Masonry'],
+        directories: [],
+        enableDirectories: true,
+      }),
+    );
+    const siteForced = readJson(path.join(forced.dataDst, 'site.json'));
+    const dirsForced = readJson(path.join(forced.dataDst, 'directories.json'));
+    assert(siteForced.features.enable_directories === false, 'empty+force → enable false');
+    assert(dirsForced.directories.length >= 1 && dirsForced.directories.length === origLen);
+    fs.rmSync(forced.tmp, { recursive: true, force: true });
+    fs.rmSync(seeded.tmp, { recursive: true, force: true });
+  });
+
+  await test('normalizeSiteType canonical + human-ish aliases; invalid → multipage', () => {
+    assert(DEFAULT_SITE_TYPE === 'multipage', 'default site type is multipage');
+    assert(JSON.stringify(SITE_TYPES) === JSON.stringify(['one-page', 'multipage', 'seo']));
+    assert(normalizeSiteType(undefined) === 'multipage', 'missing → multipage');
+    assert(normalizeSiteType('') === 'multipage', 'blank → multipage');
+    assert(normalizeSiteType('  ') === 'multipage', 'whitespace → multipage');
+    assert(normalizeSiteType('one-page') === 'one-page');
+    assert(normalizeSiteType('multipage') === 'multipage');
+    assert(normalizeSiteType('seo') === 'seo');
+    assert(normalizeSiteType('one page') === 'one-page', 'one page alias');
+    assert(normalizeSiteType('onepage') === 'one-page', 'onepage alias');
+    assert(normalizeSiteType('multi page') === 'multipage', 'multi page alias');
+    assert(normalizeSiteType('multi-page') === 'multipage', 'multi-page alias');
+    assert(normalizeSiteType('SEO') === 'seo', 'case fold');
+    assert(normalizeSiteType('not-a-type') === 'multipage', 'invalid → multipage');
+    assert(normalizeSiteType('blog') === 'multipage', 'unknown → multipage');
+
+    const omitted = buildAnswers({ businessName: 'Type Co', primaryServices: ['Masonry'] });
+    assert(omitted.siteType === 'multipage', 'buildAnswers omits → multipage');
+    assert(buildAnswers({
+      businessName: 'One Page Co',
+      primaryServices: ['Masonry'],
+      siteType: 'one page',
+    }).siteType === 'one-page');
+    assert(buildAnswers({
+      businessName: 'SEO Co',
+      primaryServices: ['Masonry'],
+      siteType: 'seo',
+    }).siteType === 'seo');
+  });
+
+  await test('replaceTargetData writes site_type from answers (default multipage)', () => {
+    assert(REPO_ROOT, 'expected local template root');
+    const seeded = seedDataDir('ccs-sitetype-');
+    try {
+      const originalSite = readJson(path.join(seeded.dataDst, 'site.json'));
+      replaceTargetData(
+        seeded.tmp,
+        buildAnswers({ businessName: 'Type Default Co', primaryServices: ['Masonry'] }),
+      );
+      const siteDefault = readJson(path.join(seeded.dataDst, 'site.json'));
+      assert(siteDefault.site_type === 'multipage', `default write multipage, got ${siteDefault.site_type}`);
+      assert(siteDefault._instructions, '_instructions preserved');
+      assert(
+        JSON.stringify(Object.keys(siteDefault._instructions)) ===
+          JSON.stringify(Object.keys(originalSite._instructions)),
+        '_instructions keys stable',
+      );
+
+      // one-page
+      const one = seedDataDir('ccs-sitetype-one-');
+      replaceTargetData(
+        one.tmp,
+        buildAnswers({
+          businessName: 'One Page Site',
+          primaryServices: ['Patios'],
+          siteType: 'one-page',
+        }),
+      );
+      assert(readJson(path.join(one.dataDst, 'site.json')).site_type === 'one-page');
+      fs.rmSync(one.tmp, { recursive: true, force: true });
+
+      // seo
+      const seo = seedDataDir('ccs-sitetype-seo-');
+      replaceTargetData(
+        seo.tmp,
+        buildAnswers({
+          businessName: 'SEO Site',
+          primaryServices: ['Patios'],
+          siteType: 'seo',
+        }),
+      );
+      assert(readJson(path.join(seo.dataDst, 'site.json')).site_type === 'seo');
+      fs.rmSync(seo.tmp, { recursive: true, force: true });
+    } finally {
+      fs.rmSync(seeded.tmp, { recursive: true, force: true });
+    }
+  });
+
+  await test('--help documents trust/payment/hours/social/directories/siteType defaults', () => {
+    const help = runCli(['--help']);
+    assert(help.status === 0, `help exit 0, got ${help.status}`);
+    const helpOut = `${help.stdout}\n${help.stderr}`;
+    assert(/freeEstimate/i.test(helpOut), 'help lists freeEstimate');
+    assert(/foundedYear/i.test(helpOut), 'help lists foundedYear');
+    assert(/empty string/i.test(helpOut), 'help documents foundedYear empty string');
+    assert(/paymentMethods/i.test(helpOut), 'help lists paymentMethods');
+    assert(/hoursWeekday/i.test(helpOut), 'help lists compact hours keys');
+    assert(/social/i.test(helpOut), 'help lists social');
+    assert(/directories/i.test(helpOut), 'help lists directories');
+    assert(/enable_directories/i.test(helpOut), 'help mentions enable_directories');
+    assert(/siteType/i.test(helpOut), 'help lists siteType');
+    assert(/multipage/i.test(helpOut), 'help documents multipage default');
+    assert(/one-page/i.test(helpOut), 'help lists one-page');
+    assert(/CREATE_CONTRACTOR_SITE_ANSWERS_JSON/i.test(helpOut), 'help lists JSON env');
+  });
+
+  await test('CREATE_CONTRACTOR_SITE_ANSWERS_JSON CLI path defaults intake and siteType', () => {
+    assert(REPO_ROOT, 'expected local template root');
+    // Fresh non-existent target (CLI rejects existing dirs that are non-empty).
+    const targetDir = path.join(
+      fs.mkdtempSync(path.join(os.tmpdir(), 'ccs-jsonenv-parent-')),
+      'site',
+    );
+
+    const payload = {
+      businessName: 'JSON Env Masonry',
+      primaryServices: ['Patios'],
+      // Blank/whitespace trust fields → buildAnswers defaults; foundedYear → ""
+      freeEstimate: '   ',
+      yearsExperience: '\t',
+      license: '',
+      insurance: null,
+      foundedYear: '   ',
+      // Blank payments + empty hours → defaults (never [])
+      paymentMethods: '  ,  ',
+      hours: [],
+      social: { facebook: '  ', instagram: '' }, // blank → {}
+      directories: [], // none → enable false + min(1)
+    };
+
+    const result = runCli([targetDir], {
+      env: {
+        CREATE_CONTRACTOR_TEMPLATE_ROOT: REPO_ROOT,
+        CREATE_CONTRACTOR_SITE_ANSWERS_JSON: JSON.stringify(payload),
+        NODE_ENV: 'test',
+        CREATE_CONTRACTOR_SITE_SKIP_SETUP: '1',
+      },
+    });
+
+    const out = `${result.stdout}\n${result.stderr}`;
+    try {
+      assert(result.status === 0, `CLI exit 0, got ${result.status}:\n${out.slice(-2000)}`);
+      assert(/Data replace complete/i.test(out), 'expected skip-setup success banner');
+
+      const businessPath = path.join(targetDir, 'src/data/business.json');
+      assert(fs.existsSync(businessPath), 'expected generated business.json');
+      const business = JSON.parse(fs.readFileSync(businessPath, 'utf8'));
+      const site = readJson(path.join(targetDir, 'src/data/site.json'));
+      const directories = readJson(path.join(targetDir, 'src/data/directories.json'));
+
+      assert(business.name === 'JSON Env Masonry', 'env business name via CLI');
+      assert(
+        business.free_estimate === DEFAULT_FREE_ESTIMATE,
+        `free_estimate default via CLI, got ${business.free_estimate}`,
+      );
+      assert(
+        business.years_experience === DEFAULT_YEARS_EXPERIENCE,
+        `years_experience default via CLI, got ${business.years_experience}`,
+      );
+      assert(
+        business.license === DEFAULT_LICENSE,
+        `license default via CLI, got ${business.license}`,
+      );
+      assert(
+        business.insurance === DEFAULT_INSURANCE,
+        `insurance default via CLI, got ${business.insurance}`,
+      );
+      assert(
+        Object.prototype.hasOwnProperty.call(business, 'founded_year'),
+        'founded_year key must remain',
+      );
+      assert(business.founded_year === '', 'CLI founded_year blank → ""');
+      assert(
+        JSON.stringify(business.payment_methods) === JSON.stringify(DEFAULT_PAYMENT_METHODS),
+        'CLI blank payments → defaults',
+      );
+      assert(business.payment_methods.length > 0, 'payment_methods never []');
+      assert(
+        JSON.stringify(business.hours) === JSON.stringify(DEFAULT_HOURS),
+        'CLI empty hours → DEFAULT_HOURS',
+      );
+      assert(business.hours.every((h) => h.days && h.time), 'hours rows shaped');
+      assert(JSON.stringify(business.social) === '{}', 'CLI blank social → {}');
+      assert(site.features.enable_directories === false, 'CLI no dirs → enable false');
+      assert(directories.directories.length >= 1, 'CLI directories min(1)');
+      assert(site.site_type === 'multipage', `CLI omitted siteType → multipage, got ${site.site_type}`);
+    } finally {
+      const parent = path.dirname(targetDir);
+      try {
+        fs.rmSync(parent, { recursive: true, force: true });
+      } catch {
+        console.warn(`    (could not fully remove ${parent})`);
+      }
+    }
+  });
+
+  await test('CREATE_CONTRACTOR_SITE_ANSWERS_JSON can set siteType one-page and seo', () => {
+    assert(REPO_ROOT, 'expected local template root');
+
+    for (const { label, siteType, expected } of [
+      { label: 'one-page', siteType: 'one page', expected: 'one-page' },
+      { label: 'seo', siteType: 'seo', expected: 'seo' },
+    ]) {
+      const targetDir = path.join(
+        fs.mkdtempSync(path.join(os.tmpdir(), `ccs-sitetype-${label}-`)),
+        'site',
+      );
+      const result = runCli([targetDir], {
+        env: {
+          CREATE_CONTRACTOR_TEMPLATE_ROOT: REPO_ROOT,
+          CREATE_CONTRACTOR_SITE_ANSWERS_JSON: JSON.stringify({
+            businessName: `JSON ${label}`,
+            primaryServices: ['Patios'],
+            siteType,
+          }),
+          NODE_ENV: 'test',
+          CREATE_CONTRACTOR_SITE_SKIP_SETUP: '1',
+        },
+      });
+      const out = `${result.stdout}\n${result.stderr}`;
+      try {
+        assert(result.status === 0, `CLI ${label} exit 0, got ${result.status}:\n${out.slice(-1500)}`);
+        const site = readJson(path.join(targetDir, 'src/data/site.json'));
+        assert(site.site_type === expected, `CLI ${label} site_type, got ${site.site_type}`);
+      } finally {
+        try {
+          fs.rmSync(path.dirname(targetDir), { recursive: true, force: true });
+        } catch {
+          console.warn(`    (could not fully remove ${path.dirname(targetDir)})`);
+        }
+      }
+    }
+  });
+
+  await test('CREATE_CONTRACTOR_SITE_ANSWERS_JSON compact payment/hours/social/dirs path', () => {
+    assert(REPO_ROOT, 'expected local template root');
+    const targetDir = path.join(
+      fs.mkdtempSync(path.join(os.tmpdir(), 'ccs-jsoncompact-parent-')),
+      'site',
+    );
+
+    const payload = {
+      businessName: 'Compact Hours Co',
+      primaryServices: ['Patios'],
+      paymentMethods: ['Cash', '  ', 'ACH'],
+      hoursWeekday: '9:00 AM - 5:00 PM',
+      hoursSaturday: 'Closed',
+      hoursSunday: 'Closed',
+      social: 'facebook=https://facebook.com/compact,instagram=https://instagram.com/compact',
+      directories: 'Google Business|https://g.co/compact,BBB|https://bbb.org/compact',
+    };
+
+    const result = runCli([targetDir], {
+      env: {
+        CREATE_CONTRACTOR_TEMPLATE_ROOT: REPO_ROOT,
+        CREATE_CONTRACTOR_SITE_ANSWERS_JSON: JSON.stringify(payload),
+        NODE_ENV: 'test',
+        CREATE_CONTRACTOR_SITE_SKIP_SETUP: '1',
+      },
+    });
+
+    const out = `${result.stdout}\n${result.stderr}`;
+    try {
+      assert(result.status === 0, `CLI exit 0, got ${result.status}:\n${out.slice(-2000)}`);
+      const business = JSON.parse(
+        fs.readFileSync(path.join(targetDir, 'src/data/business.json'), 'utf8'),
+      );
+      const site = readJson(path.join(targetDir, 'src/data/site.json'));
+      const directories = readJson(path.join(targetDir, 'src/data/directories.json'));
+      assert(
+        JSON.stringify(business.payment_methods) === JSON.stringify(['Cash', 'ACH']),
+        `compact payments, got ${JSON.stringify(business.payment_methods)}`,
+      );
+      assert(business.hours.length === 3, 'compact hours length');
+      assert(business.hours[0].days === 'Monday - Friday', 'weekday days label');
+      assert(business.hours[0].time === '9:00 AM - 5:00 PM', 'compact weekday time');
+      assert(business.hours[1].time === 'Closed', 'compact saturday');
+      assert(business.hours[2].time === 'Closed', 'compact sunday');
+      // Trust defaults still apply when omitted
+      assert(business.free_estimate === DEFAULT_FREE_ESTIMATE, 'trust default still works');
+      assert(business.founded_year === '', 'founded_year still ""');
+      assert(business.social.facebook === 'https://facebook.com/compact');
+      assert(business.social.instagram === 'https://instagram.com/compact');
+      assert(site.features.enable_directories === true, 'compact dirs → enable true');
+      assert(directories.directories.length === 2);
+      assert(directories.directories[0].url === 'https://g.co/compact');
+    } finally {
+      const parent = path.dirname(targetDir);
+      try {
+        fs.rmSync(parent, { recursive: true, force: true });
+      } catch {
+        console.warn(`    (could not fully remove ${parent})`);
+      }
+    }
+  });
+
+  await test('skip setup guard requires NODE_ENV test', () => {
+    const parent = fs.mkdtempSync(path.join(os.tmpdir(), 'ccs-skip-guard-'));
+    const targetDir = path.join(parent, 'site');
+    try {
+      const result = runCli(['--yes', targetDir], {
+        env: { CREATE_CONTRACTOR_TEMPLATE_ROOT: REPO_ROOT || '', CREATE_CONTRACTOR_SITE_SKIP_SETUP: '1', NODE_ENV: '' },
+      });
+      const out = `${result.stdout}\n${result.stderr}`;
+      assert(result.status === 1, `expected guard failure, got ${result.status}`);
+      assert(/requires NODE_ENV=test/i.test(out), `expected guard message:\n${out}`);
+    } finally {
+      fs.rmSync(parent, { recursive: true, force: true });
+    }
+  });
+
   await test('replaceTargetData Chesapeake case keeps unique area slugs', () => {
     assert(REPO_ROOT, 'expected local template root');
     const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'ccs-areas-'));
@@ -389,6 +1064,7 @@ async function main() {
       'areas.json',
       'navigation.json',
       'landings.json',
+      'directories.json',
     ]) {
       fs.copyFileSync(path.join(dataSrc, file), path.join(dataDst, file));
     }
@@ -467,6 +1143,38 @@ async function main() {
         fs.readFileSync(path.join(targetDir, 'src/data/business.json'), 'utf8'),
       );
       assert(business.name === 'Acme Masonry', 'expected replaced business name');
+      // --yes omits trust/payment/hours → buildAnswers defaults (not sample-specific facts)
+      assert(business.free_estimate === DEFAULT_FREE_ESTIMATE, 'E2E free_estimate default');
+      assert(
+        business.years_experience === DEFAULT_YEARS_EXPERIENCE,
+        'E2E years_experience default',
+      );
+      assert(business.license === DEFAULT_LICENSE, 'E2E license default');
+      assert(business.insurance === DEFAULT_INSURANCE, 'E2E insurance default');
+      assert(business.founded_year === '', 'E2E founded_year omitted → ""');
+      assert(
+        JSON.stringify(business.payment_methods) === JSON.stringify(DEFAULT_PAYMENT_METHODS),
+        'E2E payment_methods defaults',
+      );
+      assert(business.payment_methods.length > 0, 'E2E payment_methods never []');
+      assert(
+        JSON.stringify(business.hours) === JSON.stringify(DEFAULT_HOURS),
+        'E2E hours defaults',
+      );
+      assert(
+        business.hours.every((h) => typeof h.days === 'string' && typeof h.time === 'string'),
+        'E2E hours shape',
+      );
+      assert(JSON.stringify(business.social) === '{}', 'E2E omitted social → {}');
+      const siteE2E = readJson(path.join(targetDir, 'src/data/site.json'));
+      const dirsE2E = readJson(path.join(targetDir, 'src/data/directories.json'));
+      assert(siteE2E.features.enable_directories === false, 'E2E no dirs → enable false');
+      assert(dirsE2E.directories.length >= 1, 'E2E directories min(1)');
+      assert(
+        siteE2E.site_type === 'multipage',
+        `E2E --yes site_type multipage, got ${siteE2E.site_type}`,
+      );
+      // Full scaffold already ran validate:data + build (includes route gate) successfully.
 
       const services = JSON.parse(
         fs.readFileSync(path.join(targetDir, 'src/data/services.json'), 'utf8'),