Merge pull request #25 from redis-applied-ai/init-tweaks

changes for quickstart

Author: abrookins

.gitignore

@@ -22,6 +22,7 @@ share/python-wheels/
 .installed.cfg
 *.egg
 MANIFEST
+.python-version
 
 # Environment files
 .env

Dockerfile

@@ -37,18 +37,29 @@ RUN --mount=type=cache,target=/root/.cache/uv \
     mkdir -p /app/artifacts && \
     # Attempt to build artifacts, but don't fail build if it requires runtime services
     (uv run --no-sync redis-sre-agent pipeline prepare-sources \
-        --source-dir /app/source_documents \
-        --prepare-only \
-        --artifacts-path /app/artifacts || \
+    --source-dir /app/source_documents \
+    --prepare-only \
+    --artifacts-path /app/artifacts || \
     echo "Warning: Could not prepare source documents at build time")
 
 
 # STAGE 2: Runtime
 # This is the final image. It will be much smaller.
 FROM python:3.12-slim
 
-# Copy uv from the official image for runtime use (needed by entrypoint)
-COPY --from=ghcr.io/astral-sh/uv:latest /uv /bin/uv
+# Copy uv binary into the runtime image so entrypoint scripts and docker-compose
+# commands that use `uv run` work correctly.
+COPY --from=builder /bin/uv /bin/uv
+
+# Copy the uv-managed Python installation used by the virtualenv in /app/.venv.
+# Without this, /app/.venv/bin/python points at a non-existent interpreter under
+# /root/.local/share/uv, which causes `uv run` and even direct venv usage to
+# fail with "permission denied" when running as the non-root `app` user.
+COPY --from=builder /root/.local/share/uv /root/.local/share/uv
+
+# Make the uv Python tree readable and traversable by the unprivileged `app`
+# user so that symlinks in /app/.venv/bin/python* can be resolved.
+RUN chmod 755 /root && chmod -R 755 /root/.local/share/uv || true
 
 WORKDIR /app
 

pyproject.toml

@@ -57,6 +57,7 @@ dependencies = [
     "opentelemetry-instrumentation-httpx>=0.57b0",
     "opentelemetry-instrumentation-aiohttp-client>=0.57b0",
     "opentelemetry-instrumentation-openai>=0.47.5",
+    "cryptography>=45.0.0,<46",
     "mcp>=1.23.3",
     "nltk>=3.9.1",
 ]

scripts/docker-entrypoint.sh

@@ -33,8 +33,14 @@ check_knowledge_base() {
     fi
 
     # Check if knowledge base has any documents
+    # grep -c returns 0 and exits with status 1 when there are no matches, so
+    # we ignore its exit code and default empty output to 0.
     local doc_count=$(redis-cli -u "${REDIS_URL:-redis://localhost:6379/0}" \
-        FT._LIST 2>/dev/null | grep -c "sre_knowledge" || echo "0")
+        FT._LIST 2>/dev/null | grep -c "sre_knowledge" || true)
+
+    if [ -z "$doc_count" ]; then
+        doc_count=0
+    fi
 
     if [ "$doc_count" -eq 0 ]; then
         log "Knowledge base is empty"

ui/Dockerfile

@@ -7,7 +7,7 @@ WORKDIR /app
 # Install dependencies for development
 FROM base AS development
 
-# Copy package files
+# Copy package files for both main app and ui-kit
 COPY package*.json ./
 COPY ui-kit/package*.json ./ui-kit/
 
@@ -17,7 +17,7 @@ RUN npm ci --ignore-scripts
 # Install ui-kit dependencies
 RUN npm --prefix ./ui-kit ci --ignore-scripts
 
-# Copy source code
+# Copy remaining source code
 COPY . .
 
 # Build ui-kit now that source files are present
@@ -32,7 +32,7 @@ CMD ["npm", "run", "dev", "--", "--host", "0.0.0.0", "--port", "3000"]
 # Production build stage
 FROM base AS build
 
-# Copy package files
+# Copy package files for both main app and ui-kit
 COPY package*.json ./
 COPY ui-kit/package*.json ./ui-kit/
 
@@ -42,7 +42,7 @@ RUN npm ci --ignore-scripts
 # Install ui-kit dependencies
 RUN npm --prefix ./ui-kit ci --ignore-scripts
 
-# Copy source code
+# Copy remaining source code
 COPY . .
 
 # Build ui-kit first, then the main app

ui/ui-kit/package.json

@@ -46,6 +46,9 @@
     "format:check": "prettier --check \"src/**/*.{ts,tsx}\"",
     "lint": "eslint src --ext .ts,.tsx",
     "lint:fix": "eslint src --ext .ts,.tsx --fix",
+    "pre-commit": "pre-commit run --all-files",
+    "pre-commit:install": "pre-commit install --hook-type pre-commit --hook-type pre-push",
+    "prepare": "echo 'skip pre-commit install in Docker build'",
     "prepublishOnly": "npm run clean && npm run build",
     "storybook": "storybook dev -p 6006",
     "test": "vitest run",
@@ -99,4 +102,4 @@
     "react": ">=18.0.0",
     "react-dom": ">=18.0.0"
   }
-}
+}

ui/vite.config.ts

@@ -1,12 +1,14 @@
-import { defineConfig } from 'vite';
 import react from '@vitejs/plugin-react';
 import { resolve } from 'path';
+import { defineConfig } from 'vite';
 
 export default defineConfig({
   plugins: [react()],
   resolve: {
     alias: {
       '@': resolve(__dirname, 'src'),
+      '@radar/ui-kit': resolve(__dirname, 'ui-kit/src'),
+      '@radar/ui-kit/styles': resolve(__dirname, 'ui-kit/src/styles/index.css'),
     },
   },
   server: {