feat(docker): enhance .dockerignore for better file management; update Dockerfile for improved package verification and debugging; refine entrypoint script checks

.dockerignore

@@ -1,7 +1,40 @@
-# Ignore everything by default
-*
+# Version control
+.git/
+.gitignore
+.gitattributes
 
-# Explicitly allow required files
+# Cache and temporary files
+__pycache__/
+*.py[cod]
+*$py.class
+*.so
+.Python
+*.egg
+*.egg-info/
+.installed.cfg
+.pytest_cache/
+.coverage
+htmlcov/
+
+# Environment and IDE
+venv/
+env/
+.env
+.idea/
+.vscode/
+*.swp
+*.swo
+
+# Build artifacts
+build/
+dist/
+wheels/
+
+# Project specific
+rules_backup/
+
+# Keep these files
+!src/
 !src/simpleguardhome/
 !src/simpleguardhome/**
 !requirements.txt
@@ -11,14 +44,3 @@
 !LICENSE
 !README.md
 !docker-entrypoint.sh
-
-# Still exclude these even if in allowed directories
-**/__pycache__
-**/*.pyc
-**/*.pyo
-**/*.pyd
-**/.Python
-**/*.so
-**/*.egg
-**/*.egg-info
-**/*.egg-info/

Dockerfile

@@ -1,4 +1,4 @@
-# Stage 1: Build dependencies
+# Stage 1: Build dependencies and package
 FROM python:3.11-slim-bullseye as builder
 
 # Set working directory
@@ -17,14 +17,22 @@ RUN apt-get update && \
     && rm -rf /var/lib/apt/lists/* \
     && python3 -m pip install --no-cache-dir --upgrade "pip>=21.3" setuptools wheel
 
-# Copy requirements and install dependencies
+# Copy package files
+COPY src/ /build/src/
+COPY pyproject.toml setup.py MANIFEST.in README.md LICENSE ./
+
+# Install requirements and build package
 COPY requirements.txt .
-RUN pip install --no-cache-dir -r requirements.txt
+RUN pip install --no-cache-dir -r requirements.txt && \
+    pip install -e .
+
+# Verify package installation in builder
+RUN python3 -c "import simpleguardhome; print(f'Package installed at {simpleguardhome.__file__}')"
 
 # Stage 2: Final image
 FROM python:3.11-slim-bullseye
 
-# Install runtime dependencies and tree for debugging
+# Install runtime dependencies
 RUN apt-get update && \
     DEBIAN_FRONTEND=noninteractive apt-get install -y \
     --no-install-recommends \
@@ -35,44 +43,31 @@ RUN apt-get update && \
 # Set working directory
 WORKDIR /app
 
-# Copy installed dependencies from builder
+# Create source directory
+RUN mkdir -p /app/src/simpleguardhome
+
+# Copy package files from builder
+COPY --from=builder /build/src/simpleguardhome/ /app/src/simpleguardhome/
+COPY --from=builder /build/setup.py /build/pyproject.toml /build/MANIFEST.in /app/
+
+# Copy dependencies from builder
 COPY --from=builder /usr/local/lib/python3.11/site-packages/ /usr/local/lib/python3.11/site-packages/
 COPY --from=builder /usr/local/bin/ /usr/local/bin/
 
-# Debug: Show initial state
-RUN echo "Initial directory structure:" && \
-    tree /app || true
-
-# First copy the package source directory
-COPY src/ /app/src/
-
-# Copy project files
-COPY pyproject.toml setup.py MANIFEST.in README.md LICENSE docker-entrypoint.sh ./
-
-# Debug: Verify directory structure
-RUN echo "After copying files:" && \
-    tree /app && \
-    echo "Verifying package directory:" && \
-    ls -la /app/src/simpleguardhome/
-
-# Set permissions
-RUN chmod -R 755 /app && \
-    chmod +x /app/docker-entrypoint.sh && \
+# Copy and set permissions for entrypoint
+COPY docker-entrypoint.sh /app/
+RUN chmod +x /app/docker-entrypoint.sh && \
     cp /app/docker-entrypoint.sh /usr/local/bin/
 
+# Debug: Show directory structure
+RUN echo "Directory structure:" && \
+    tree /app && \
+    echo "Package contents:" && \
+    ls -la /app/src/simpleguardhome/
+
 # Set PYTHONPATH
 ENV PYTHONPATH=/app/src
 
-# Install the package
-RUN set -ex && \
-    echo "Installing package..." && \
-    pip install -e . && \
-    echo "Verifying installation..." && \
-    python3 -c "import sys; print('Python path:', sys.path)" && \
-    python3 -c "import simpleguardhome; print('Package found at:', simpleguardhome.__file__)" && \
-    python3 -c "from simpleguardhome.main import app; print('App imported successfully')" && \
-    echo "Package installation successful"
-
 # Create rules backup directory
 RUN mkdir -p /app/rules_backup && \
     chmod 777 /app/rules_backup

docker-entrypoint.sh

@@ -21,70 +21,61 @@ log() {
 check_package() {
     log "System information:"
     uname -a
+    
     log "Python version:"
     python3 --version
     
-    # Debug: Show current directory and its contents
-    log "Current directory: $(pwd)"
-    log "Directory contents:"
-    ls -la
-
-    # Debug: Show all Python paths
-    log "Python paths:"
-    python3 -c "import sys; print('\n'.join(sys.path))"
-    
-    # Debug: Show package installation status
-    log "Installed packages:"
-    pip list
+    log "Directory structure:"
+    tree /app
     
     log "Verifying package files..."
     if [ ! -d "/app/src/simpleguardhome" ]; then
-        log "ERROR: Package directory not found at /app/src/simpleguardhome"
-        log "Searching for package directory..."
-        find / -name "simpleguardhome" -type d 2>/dev/null || echo "No simpleguardhome directory found"
+        log "ERROR: Package directory not found at /app/src/simpleguardhome!"
         exit 1
     fi
 
-    log "Checking critical files..."
-    for file in "__init__.py" "main.py" "adguard.py" "config.py"; do
+    # Check critical files
+    required_files=(
+        "__init__.py"
+        "main.py"
+        "adguard.py"
+        "config.py"
+        "templates/index.html"
+        "favicon.ico"
+    )
+
+    for file in "${required_files[@]}"; do
         if [ ! -f "/app/src/simpleguardhome/$file" ]; then
             log "ERROR: Required file $file not found!"
             exit 1
         fi
     done
 
-    log "Package structure:"
-    tree /app/src/simpleguardhome
-
-    log "Environment variables:"
-    echo "PYTHONPATH=$PYTHONPATH"
-    echo "PWD=$(pwd)"
+    log "Environment:"
+    echo "PYTHONPATH: $PYTHONPATH"
+    echo "PWD: $(pwd)"
     
     log "Testing package import..."
-    PYTHONPATH=/app/src python3 -c "
+    if ! python3 -c "
 import sys
-print('Python path:', sys.path)
+print('Python paths:', sys.path)
 import simpleguardhome
-print('Package location:', simpleguardhome.__file__)
+print('Package found at:', simpleguardhome.__file__)
 from simpleguardhome.main import app
 print('Package imported successfully')
-" || {
+"; then
         log "ERROR: Package import failed!"
         exit 1
-    }
+    fi
 }
 
-# Run checks with error handling
-if ! check_package; then
-    log "Package verification failed"
-    exit 1
-fi
+# Run checks
+check_package
 
 log "All checks passed. Starting server..."
 
 # Start the application
-echo "Starting SimpleGuardHome server..."
-exec python3 -c "from simpleguardhome import start; start()"
+exec python3 -m uvicorn simpleguardhome.main:app --host 0.0.0.0 --port 8000
 
 # Store child PID
 child=$!