From 0fe61454355d7f21bc01439879e6f492ab857840 Mon Sep 17 00:00:00 2001
From: Sumesh Pradhan <sumesh@appsmith.com>
Date: Wed, 16 Aug 2023 16:39:11 +0530
Subject: [PATCH] fix: moved jest to devdependencies for appsmithctl and
 updated dockerfile to exclude it during install (#26350)

Update:
- Mobed jest to` devDependenies` of package manifest.
- Updated Dockerfile to install utils with `--only=prod` arg for npm
install cmds.

Tested manually
---
 Dockerfile                       | 2 +-
 deploy/docker/utils/package.json | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index a29c4745dd..8abf6b9352 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -84,7 +84,7 @@ COPY ./deploy/docker/entrypoint.sh ./deploy/docker/scripts/* info.*json ./
 
 # Add util tools
 COPY ./deploy/docker/utils ./utils
-RUN cd ./utils && npm install && npm install -g .
+RUN cd ./utils && npm install --only=prod && npm install --only=prod -g .
 
 # Add process config to be run by supervisord
 COPY ./deploy/docker/templates/supervisord.conf /etc/supervisor/supervisord.conf
diff --git a/deploy/docker/utils/package.json b/deploy/docker/utils/package.json
index fa8f13ca84..d3916cec97 100644
--- a/deploy/docker/utils/package.json
+++ b/deploy/docker/utils/package.json
@@ -14,12 +14,14 @@
   "dependencies": {
     "cli-progress": "^3.11.2",
     "dotenv": "10.0.0",
-    "jest": "^29.1.2",
     "mongodb": "^5.7.0",
     "nodemailer": "6.7.5",
     "readline-sync": "1.4.10",
     "shelljs": "0.8.5"
   },
+  "devDependencies": {
+    "jest": "^29.1.2"
+  },
   "bin": {
     "appsmithctl": "./bin/index.js"
   },