Skip JSDoc, SassDoc and Rollup stats in development

Updates Review app UI to hide links when not available

Author: colinrotherham

package.json

@@ -20,7 +20,7 @@
     "build-release": "./bin/build-release.sh",
     "publish-release": "./bin/publish-release.sh",
     "publish-preview": "./bin/publish-preview.sh",
-    "predev": "npm run build",
+    "predev": "npm run build -- --ignore-scripts",
     "dev": "concurrently \"npm run dev --workspace @govuk-frontend/review\" \"npm run dev --workspace govuk-frontend\" --kill-others --names \"app,pkg\" --prefix-colors \"red.dim,blue.dim\"",
     "build": "npm run build --workspace govuk-frontend --workspace @govuk-frontend/review",
     "build:app": "cross-env-shell NODE_ENV=production npm run build --workspace @govuk-frontend/review",

packages/govuk-frontend-review/src/app.mjs

@@ -1,3 +1,6 @@
+import { stat } from 'fs/promises'
+import { join } from 'path'
+
 import { paths } from '@govuk-frontend/config'
 import {
   getComponentsFixtures,
@@ -48,7 +51,17 @@ export default async () => {
   // Feature flags
   const flags = /** @type {FeatureFlags} */ ({
     isDeployedToHeroku: !!process.env.HEROKU_APP,
-    isDevelopment: !['test', 'production'].includes(process.env.NODE_ENV)
+    isDevelopment: !['test', 'production'].includes(process.env.NODE_ENV),
+
+    // Check for JSDoc and SassDoc
+    hasDocs: await stat(join(paths.app, 'dist/docs'))
+      .then(() => true)
+      .catch(() => false),
+
+    // Check for Rollup stats
+    hasStats: await stat(join(paths.stats, 'dist'))
+      .then(() => true)
+      .catch(() => false)
   })
 
   // Set up Express.js
@@ -213,4 +226,6 @@ export default async () => {
  * @typedef {object} FeatureFlags
  * @property {boolean} isDeployedToHeroku - Review app using `HEROKU_APP`
  * @property {boolean} isDevelopment - Review app not using `NODE_ENV` production or test
+ * @property {boolean} hasDocs - JSDoc and SassDoc are available
+ * @property {boolean} hasStats - Rollup stats are available
  */

packages/govuk-frontend-review/src/views/index.njk

@@ -61,6 +61,7 @@
 
   </div>
 
+  {% if flags.hasDocs %}
   <div class="govuk-grid-row">
 
     <div class="govuk-grid-column-two-thirds-from-desktop">
@@ -89,7 +90,9 @@
       </div>
     </div>
   </div>
+  {% endif %}
 
+  {% if flags.hasStats %}
   <div class="govuk-grid-row">
     <div class="govuk-grid-column-two-thirds-from-desktop">
       <hr class="govuk-section-break govuk-section-break--xl govuk-section-break--visible">
@@ -138,4 +141,5 @@
       }) }}
     </div>
   </div>
+  {% endif %}
 {% endblock %}