From 03f17a6e056e4405f7f7e923da208ea808360e46 Mon Sep 17 00:00:00 2001
From: Kevin Veen-Birkenbach <kevin@veen.world>
Date: Mon, 11 May 2026 00:40:19 +0200
Subject: [PATCH] build(make): run npm-install before docker-compose targets

Add npm-install dependency to up/dev/prod targets so app/static/vendor
and app/node_modules exist on the host before the ./app:/app bind mount
masks the build-time artifacts inside the container.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 Makefile | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index d2a5d60..9aefc94 100644
--- a/Makefile
+++ b/Makefile
@@ -22,7 +22,7 @@ build-no-cache:
 	docker build --no-cache -t application-portfolio .
 
 .PHONY: up
-up:
+up: npm-install
 	# Start the application using docker-compose with build.
 	docker-compose up -d --build --force-recreate
 
@@ -58,12 +58,12 @@ logs:
 	docker logs -f portfolio
 
 .PHONY: dev
-dev:
+dev: npm-install
 	# Start the application in development mode using docker-compose.
 	FLASK_ENV=development docker-compose up -d
 
 .PHONY: prod
-prod:
+prod: npm-install
 	# Start the application in production mode using docker-compose (with build).
 	docker-compose up -d --build