0.7.0: Making REPO_PATH only mandatory on directory mode

flavienbwk · Sep 8, 2024 · df7e2c5 · df7e2c5
1 parent 6da3373
commit df7e2c5
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 17 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -45,19 +45,9 @@ RUN echo "events { worker_connections 1024; }" > /etc/nginx/nginx.conf && \
     echo "        server_name _;" >> /etc/nginx/nginx.conf && \
     echo "        location / {" >> /etc/nginx/nginx.conf && \
     echo "            proxy_pass http://127.0.0.1:3000;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_http_version 1.1;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Upgrade \$http_upgrade;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Connection 'upgrade';" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Host \$host;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_cache_bypass \$http_upgrade;" >> /etc/nginx/nginx.conf && \
     echo "        }" >> /etc/nginx/nginx.conf && \
     echo "        location /api {" >> /etc/nginx/nginx.conf && \
     echo "            proxy_pass http://127.0.0.1:5328;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_http_version 1.1;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Upgrade \$http_upgrade;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Connection 'upgrade';" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_set_header Host \$host;" >> /etc/nginx/nginx.conf && \
-    echo "            proxy_cache_bypass \$http_upgrade;" >> /etc/nginx/nginx.conf && \
     echo "        }" >> /etc/nginx/nginx.conf && \
     echo "    }" >> /etc/nginx/nginx.conf && \
     echo "}" >> /etc/nginx/nginx.conf

diff --git a/action/index.js b/action/index.js
@@ -86,7 +86,7 @@ try {
         console.log('Retrieved existing namespace:', namespace.id);
       } else {
         console.error('Error creating/retrieving namespace:', error);
-        throw error;
+        core.setFailed(error.message);
       }
     }
 
@@ -130,7 +130,7 @@ try {
           });
           console.log('Container updated:', container.id);
         } else {
-          throw error;
+          core.setFailed(error.message);
         }
       }
 
@@ -173,6 +173,7 @@ try {
       }
 
       // Feed RepoChat with repo data
+      // TODO(flavienbwk): !!
 
       // Set outputs
       core.setOutput('domain', containerEndpoint);

diff --git a/api.docker-compose.yml b/api.docker-compose.yml
@@ -5,7 +5,6 @@ services:
     environment:
       REPO_NAME: ${REPO_NAME}
       REPO_URL: ${REPO_URL}
-      REPO_PATH: ${REPO_PATH}
       OPENAI_API_KEY: ${OPENAI_API_KEY}
       MODEL_TYPE_INFERENCE: ${MODEL_TYPE_INFERENCE}
       MODEL_TYPE_EMBEDDING: ${MODEL_TYPE_EMBEDDING}

diff --git a/api/config.py b/api/config.py
@@ -10,7 +10,7 @@
 
 REPO_NAME = os.environ['REPO_NAME'].strip()
 REPO_URL = os.getenv('REPO_URL', '').strip()
-REPO_PATH = os.environ['REPO_PATH'].strip()
+REPO_PATH = os.getenv('REPO_PATH', '').strip()
 OPEN_AI_API_KEY = os.environ['OPENAI_API_KEY'].strip()
 MODEL_TYPE_INFERENCE = os.environ['MODEL_TYPE_INFERENCE'].strip()
 MODEL_TYPE_EMBEDDING = os.environ['MODEL_TYPE_EMBEDDING'].strip()

diff --git a/api/index.py b/api/index.py
@@ -35,6 +35,8 @@ def get_model():
             logging.info(f"DB directory {PERSIST_DIRECTORY} does not exist. No need to clear.")
 
     if MODE == "directory":
+        if REPO_PATH == "":
+            raise Exception("Repo path is required for directory mode")
         return DirLoader(
             repo_path=REPO_PATH,
             force_reingest=CLEAR_DB_AT_RESTART,

diff --git a/lib/index.js b/lib/index.js
@@ -70229,7 +70229,7 @@ try {
         console.log('Retrieved existing namespace:', namespace.id);
       } else {
         console.error('Error creating/retrieving namespace:', error);
-        throw error;
+        _actions_core__WEBPACK_IMPORTED_MODULE_0__.setFailed(error.message);
       }
     }
 
@@ -70273,7 +70273,7 @@ try {
           });
           console.log('Container updated:', container.id);
         } else {
-          throw error;
+          _actions_core__WEBPACK_IMPORTED_MODULE_0__.setFailed(error.message);
         }
       }
 
@@ -70316,6 +70316,7 @@ try {
       }
 
       // Feed RepoChat with repo data
+      // TODO(flavienbwk): !!
 
       // Set outputs
       _actions_core__WEBPACK_IMPORTED_MODULE_0__.setOutput('domain', containerEndpoint);

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "gh-action-repochat",
-    "version": "0.6.0",
+    "version": "0.7.0",
     "type": "module",
     "description": "GitHub Action to run repochat",
     "main": "index.js",