Skip to content

Commit a2f1bd1

Browse files
Update build.mjs
1 parent 0738d93 commit a2f1bd1

File tree

1 file changed

+19
-23
lines changed

1 file changed

+19
-23
lines changed

packages/tel-frontend-review/build.mjs

Lines changed: 19 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
1212
const repoRoot = join(__dirname, "../../"); // back to repo root
1313
const reviewSrc = join(__dirname, "src");
1414
const reviewDist = join(__dirname, "dist");
15-
const nhsukDist = join(repoRoot, "node_modules/nhsuk-frontend/dist");
1615

1716
// TEL frontend package paths
1817
const telFrontendDir = join(repoRoot, "packages/tel-frontend");
@@ -25,11 +24,9 @@ const telFrontendSrcScss = join(telFrontendDir, "src/styles.scss");
2524
async function buildTelFrontend() {
2625
console.log("Building TEL Frontend CSS + JS via Gulp...");
2726

28-
// Ensure dist folder exists and is clean
2927
await fse.emptyDir(telFrontendDist);
3028
await fse.ensureDir(telFrontendDist);
3129

32-
// Run Gulp build inside tel-frontend package
3330
try {
3431
execSync("npx gulp build", {
3532
cwd: telFrontendDir,
@@ -43,20 +40,24 @@ async function buildTelFrontend() {
4340
console.log("TEL Frontend CSS/JS built successfully");
4441
}
4542

46-
// Copy built TEL frontend files + NHS frontend assets + review-specific assets
43+
// Copy TEL frontend built files + review-specific assets
4744
async function buildReviewAssets() {
4845
console.log("Copying review site assets...");
4946

5047
await fse.ensureDir(join(reviewDist, "stylesheets"));
5148
await fse.ensureDir(join(reviewDist, "javascripts"));
5249

53-
// Copy NHS.UK frontend dist
54-
await fse.copy(join(nhsukDist, "nhsuk.min.css"), join(reviewDist, "stylesheets/nhsuk.min.css"));
55-
await fse.copy(join(nhsukDist, "nhsuk.min.js"), join(reviewDist, "javascripts/nhsuk.min.js"));
50+
// ✅ No more NHS dist copy — NHS styles now come in via Sass
5651

5752
// Copy TEL frontend built files
58-
await fse.copy(join(telFrontendDist, "tel-frontend.css"), join(reviewDist, "stylesheets/tel-frontend.css"));
59-
await fse.copy(join(telFrontendDist, "tel.min.js"), join(reviewDist, "javascripts/tel.min.js"));
53+
await fse.copy(
54+
join(telFrontendDist, "tel-frontend.css"),
55+
join(reviewDist, "stylesheets/tel-frontend.css")
56+
);
57+
await fse.copy(
58+
join(telFrontendDist, "tel.min.js"),
59+
join(reviewDist, "javascripts/tel.min.js")
60+
);
6061

6162
// Copy static assets for review site (images, etc.)
6263
const reviewAssetsSrc = join(reviewSrc, "assets");
@@ -67,13 +68,13 @@ async function buildReviewAssets() {
6768
console.log("Review site assets copied");
6869
}
6970

70-
// Compile review site SCSS (for review site-specific styles)
71+
// Compile review site SCSS (for review site-specific + NHS styles)
7172
async function buildReviewCSS() {
7273
console.log("Building review site CSS...");
7374

7475
const css = sass.compile(join(reviewSrc, "scss/main.scss"), {
7576
style: "expanded",
76-
loadPaths: ["node_modules"],
77+
loadPaths: ["node_modules"], // allows @use "nhsuk-frontend/..." etc.
7778
});
7879

7980
const outDir = join(reviewDist, "stylesheets");
@@ -87,18 +88,20 @@ async function buildReviewCSS() {
8788
async function buildReviewHtml() {
8889
console.log("Rendering review site HTML...");
8990

90-
const telComponents = join(repoRoot, "packages/tel-frontend/src/tel/components");
91+
const telComponents = join(
92+
repoRoot,
93+
"packages/tel-frontend/src/tel/components"
94+
);
9195

9296
const env = nunjucks.configure(
9397
[
94-
reviewSrc, // review site source
95-
join(repoRoot, "node_modules/nhsuk-frontend"), // NHS macros
96-
telComponents // TEL frontend macros
98+
reviewSrc, // review site source
99+
join(repoRoot, "node_modules/nhsuk-frontend/packages"), // NHS macros live here in v10
100+
telComponents, // TEL frontend macros
97101
],
98102
{ autoescape: true }
99103
);
100104

101-
// Render root-level .njk files
102105
const files = await fse.readdir(reviewSrc);
103106
for (const file of files) {
104107
if (file.endsWith(".njk")) {
@@ -137,16 +140,9 @@ async function build() {
137140
try {
138141
await fse.emptyDir(reviewDist);
139142

140-
// Step 1: Build TEL frontend (CSS + JS) via Gulp
141143
await buildTelFrontend();
142-
143-
// Step 2: Copy assets (TEL + NHS + review site)
144144
await buildReviewAssets();
145-
146-
// Step 3: Compile review site SCSS
147145
await buildReviewCSS();
148-
149-
// Step 4: Render HTML pages
150146
await buildReviewHtml();
151147

152148
console.log("Review site build finished successfully");

0 commit comments

Comments
 (0)