fix: gate website validation for clean checkouts

Author: rockyshi1993

package.json

@@ -275,6 +275,9 @@
     "scripts/test-r10-control-plane-honesty.js",
     "scripts/test-process-enforcement-e2e.js",
     "scripts/test-docs-surface-inventory.js",
+    "scripts/run-website-build-if-present.js",
+    "scripts/check-generated-site-links-if-present.js",
+    "scripts/check-generated-site-links.js",
     "scripts/test-delivery-honesty.js",
     "scripts/test-ecr-closure.js",
     "scripts/lib/docs-surface-inventory.js",

scripts/check-generated-site-links-if-present.js

@@ -0,0 +1,33 @@
+#!/usr/bin/env node
+'use strict'
+
+const fs = require('fs')
+const path = require('path')
+const { scanGeneratedSite } = require('./check-generated-site-links')
+
+const ROOT = path.resolve(__dirname, '..')
+
+function main(options = {}) {
+  const root = options.root ? path.resolve(options.root) : ROOT
+  const websitePackage = path.join(root, 'website', 'package.json')
+  if (!fs.existsSync(websitePackage)) {
+    console.log('Generated site link check skipped: website/package.json not present')
+    return 0
+  }
+
+  try {
+    const result = scanGeneratedSite({ rootDir: path.join(root, 'website', 'dist') })
+    console.log(`Generated site links: html=${result.htmlCount} missing=${result.missing.length} uniqueTargets=${result.uniqueTargets.length}`)
+    for (const item of result.missing) {
+      console.error(`MISSING ${item.source} -> ${item.href} (${item.target})`)
+    }
+    return result.missing.length > 0 ? 2 : 0
+  } catch (error) {
+    console.error(error.message)
+    return 1
+  }
+}
+
+if (require.main === module) process.exitCode = main()
+
+module.exports = { main }

scripts/run-website-build-if-present.js

@@ -0,0 +1,41 @@
+#!/usr/bin/env node
+'use strict'
+
+const fs = require('fs')
+const path = require('path')
+const { CheckedCommandError, runChecked } = require('./lib/checked-command')
+
+const ROOT = path.resolve(__dirname, '..')
+
+function main(options = {}) {
+  const root = options.root ? path.resolve(options.root) : ROOT
+  const websitePackage = path.join(root, 'website', 'package.json')
+  if (!fs.existsSync(websitePackage)) {
+    console.log('Website build skipped: website/package.json not present')
+    return 0
+  }
+
+  try {
+    const evidence = runChecked('npm', ['--prefix', 'website', 'run', 'build'], {
+      cwd: root,
+      timeoutMs: 240000,
+      summaryLimit: 12000
+    })
+    if (evidence.stdout) process.stdout.write(evidence.stdout)
+    if (evidence.stderr) process.stderr.write(evidence.stderr)
+    return 0
+  } catch (error) {
+    if (error instanceof CheckedCommandError) {
+      if (error.evidence.stdout) process.stdout.write(error.evidence.stdout)
+      if (error.evidence.stderr) process.stderr.write(error.evidence.stderr)
+      console.error(error.message)
+      return error.evidence.exitCode || 1
+    }
+    console.error(error.message)
+    return 1
+  }
+}
+
+if (require.main === module) process.exitCode = main()
+
+module.exports = { main }

scripts/test-generated-site-links.js

@@ -7,8 +7,11 @@ const fs = require('fs')
 const os = require('os')
 const path = require('path')
 const { scanGeneratedSite } = require('./check-generated-site-links')
+const { main: checkGeneratedSiteLinksIfPresent } = require('./check-generated-site-links-if-present')
+const { main: runWebsiteBuildIfPresent } = require('./run-website-build-if-present')
 
 const rootDir = fs.mkdtempSync(path.join(os.tmpdir(), 'devcodex-generated-links-'))
+const cleanRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'devcodex-generated-links-clean-'))
 
 try {
   fs.mkdirSync(path.join(rootDir, 'guide'), { recursive: true })
@@ -30,7 +33,11 @@ try {
   assert.strictEqual(broken.missing[0].target, 'missing')
   assert.deepStrictEqual(broken.uniqueTargets, ['missing'])
 
+  assert.strictEqual(runWebsiteBuildIfPresent({ root: cleanRoot }), 0)
+  assert.strictEqual(checkGeneratedSiteLinksIfPresent({ root: cleanRoot }), 0)
+
   console.log('Generated site link checker tests passed')
 } finally {
   fs.rmSync(rootDir, { recursive: true, force: true })
+  fs.rmSync(cleanRoot, { recursive: true, force: true })
 }

scripts/validation-manifest.json

@@ -4285,12 +4285,9 @@
       "schemaVersion": "ValidationNodeV1",
       "id": "website-build",
       "owner": "website-build",
-      "command": "npm",
+      "command": "node",
       "args": [
-        "--prefix",
-        "website",
-        "run",
-        "build"
+        "scripts/run-website-build-if-present.js"
       ],
       "dependencies": [
         "generated-site-precheck",
@@ -4300,6 +4297,7 @@
         "website/**",
         "README.md",
         "scripts/test-generated-site-links.js",
+        "scripts/run-website-build-if-present.js",
         "scripts/check-generated-site-links.js"
       ],
       "consumers": [
@@ -4331,7 +4329,7 @@
       "owner": "website-validation",
       "command": "node",
       "args": [
-        "scripts/check-generated-site-links.js"
+        "scripts/check-generated-site-links-if-present.js"
       ],
       "dependencies": [
         "generated-site-precheck",
@@ -4340,6 +4338,7 @@
       "inputs": [
         "website/**",
         "README.md",
+        "scripts/check-generated-site-links-if-present.js",
         "scripts/check-generated-site-links.js"
       ],
       "consumers": [],