Split 'npm install' and source copy (+ 'npm run build') stages (#3371)

Author: stuartleeks

Diff for: src/Azure.Functions.Cli/StaticResources/Dockerfile.node14

@@ -5,7 +5,11 @@ FROM mcr.microsoft.com/azure-functions/node:4-node14
 ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
     AzureFunctionsJobHost__Logging__Console__IsEnabled=true
 
-COPY . /home/site/wwwroot
+WORKDIR /home/site/wwwroot
 
-RUN cd /home/site/wwwroot && \
-    npm install
+# Copy package.json and install packages before copying the rest of the code to enable caching
+COPY package.json package.json
+RUN npm install
+
+# Copy the rest of the code
+COPY . .

Diff for: src/Azure.Functions.Cli/StaticResources/Dockerfile.node16

@@ -5,7 +5,11 @@ FROM mcr.microsoft.com/azure-functions/node:4-node16
 ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
     AzureFunctionsJobHost__Logging__Console__IsEnabled=true
 
-COPY . /home/site/wwwroot
+WORKDIR /home/site/wwwroot
 
-RUN cd /home/site/wwwroot && \
-    npm install
+# Copy package.json and install packages before copying the rest of the code to enable caching
+COPY package.json package.json
+RUN npm install
+
+# Copy the rest of the code
+COPY . .

Diff for: src/Azure.Functions.Cli/StaticResources/Dockerfile.node18

@@ -5,7 +5,11 @@ FROM mcr.microsoft.com/azure-functions/node:4-node18
 ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
     AzureFunctionsJobHost__Logging__Console__IsEnabled=true
 
-COPY . /home/site/wwwroot
+WORKDIR /home/site/wwwroot
 
-RUN cd /home/site/wwwroot && \
-    npm install
+# Copy package.json and install packages before copying the rest of the code to enable caching
+COPY package.json package.json
+RUN npm install
+
+# Copy the rest of the code
+COPY . .

Diff for: src/Azure.Functions.Cli/StaticResources/Dockerfile.typescript

@@ -5,8 +5,12 @@ FROM mcr.microsoft.com/azure-functions/node:4-node16
 ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
     AzureFunctionsJobHost__Logging__Console__IsEnabled=true
 
-COPY . /home/site/wwwroot
+WORKDIR /home/site/wwwroot
 
-RUN cd /home/site/wwwroot && \
-    npm install && \
-    npm run build
+# Copy package.json and install packages before copying the rest of the code to enable caching
+COPY package.json package.json
+RUN npm install
+
+# Copy the rest of the code
+COPY . .
+RUN  npm run build

Diff for: src/Azure.Functions.Cli/StaticResources/Dockerfile.typescript.node18

@@ -5,8 +5,12 @@ FROM mcr.microsoft.com/azure-functions/node:4-node18
 ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
     AzureFunctionsJobHost__Logging__Console__IsEnabled=true
 
-COPY . /home/site/wwwroot
+WORKDIR /home/site/wwwroot
 
-RUN cd /home/site/wwwroot && \
-    npm install && \
-    npm run build
+# Copy package.json and install packages before copying the rest of the code to enable caching
+COPY package.json package.json
+RUN npm install
+
+# Copy the rest of the code
+COPY . .
+RUN  npm run build