Devin/1746532369 fix migration revision (#299)

* Fix migration revision ID mismatch for remove_workspace_analysis_tables

Co-Authored-By: Hal Seki <[email protected]>

* Fix isort issues in run_analysis.py

Co-Authored-By: Hal Seki <[email protected]>

* Fix health check endpoint path in docker-compose.test.yml

Co-Authored-By: Hal Seki <[email protected]>

* Update health check to use 127.0.0.1 instead of localhost

Co-Authored-By: Hal Seki <[email protected]>

* Update health check to use CMD-SHELL with explicit error handling

Co-Authored-By: Hal Seki <[email protected]>

* Update health check to use service name instead of localhost

Co-Authored-By: Hal Seki <[email protected]>

* Add start_period to health check to allow more time for application startup

Co-Authored-By: Hal Seki <[email protected]>

* Change health check to use wget instead of curl

Co-Authored-By: Hal Seki <[email protected]>

* Change health check to use Python urllib instead of wget or curl

Co-Authored-By: Hal Seki <[email protected]>

* Update mock API health checks to use Python urllib instead of curl

Co-Authored-By: Hal Seki <[email protected]>

* Update mock API health checks to use Node.js instead of Python

Co-Authored-By: Hal Seki <[email protected]>

* Add data directory creation to mock API Dockerfiles

Co-Authored-By: Hal Seki <[email protected]>

* Preserve node_modules in mock API containers by adding volume mount exclusions

Co-Authored-By: Hal Seki <[email protected]>

* Fix npm dependency installation in mock API containers

Co-Authored-By: Hal Seki <[email protected]>

* Add execute permission to wait-for-services.sh script

Co-Authored-By: Hal Seki <[email protected]>

* Replace curl with wget in wait-for-services.sh script

Co-Authored-By: Hal Seki <[email protected]>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

Author: halsk

integration-tests/mocks/openrouter-api/Dockerfile

@@ -2,9 +2,14 @@ FROM node:18-alpine
 
 WORKDIR /app
 
-COPY package.json ./
+# Copy package files and install dependencies
+COPY package*.json ./
 RUN npm install
 
+# Create data directory
+RUN mkdir -p /app/data
+
+# Copy application code
 COPY . .
 
 # Create data directory if it doesn't exist

integration-tests/mocks/slack-api/Dockerfile

@@ -2,9 +2,14 @@ FROM node:18-alpine
 
 WORKDIR /app
 
-COPY package.json ./
+# Copy package files and install dependencies
+COPY package*.json ./
 RUN npm install
 
+# Create data directory
+RUN mkdir -p /app/data
+
+# Copy application code
 COPY . .
 
 # Create data directory if it doesn't exist

integration-tests/setup/wait-for-services.sh

@@ -3,21 +3,21 @@ set -e
 echo "Waiting for services to be ready..."
 
 echo "Waiting for backend..."
-until curl -s http://test-backend:8000/api/v1/health > /dev/null; do
+until wget -q -O - http://test-backend:8000/health > /dev/null 2>&1 || [ $? -eq 8 ]; do
   echo "Backend is unavailable - sleeping"
   sleep 2
 done
 echo "Backend is up!"
 
 echo "Waiting for mock Slack API..."
-until curl -s http://mock-slack-api:3000/health > /dev/null; do
+until wget -q -O - http://mock-slack-api:3000/health > /dev/null 2>&1 || [ $? -eq 8 ]; do
   echo "Mock Slack API is unavailable - sleeping"
   sleep 2
 done
 echo "Mock Slack API is up!"
 
 echo "Waiting for mock OpenRouter API..."
-until curl -s http://mock-openrouter-api:3000/health > /dev/null; do
+until wget -q -O - http://mock-openrouter-api:3000/health > /dev/null 2>&1 || [ $? -eq 8 ]; do
   echo "Mock OpenRouter API is unavailable - sleeping"
   sleep 2
 done