fix: address review council findings

- Dockerfile.dev: add COPY README.md (uv_build needs it for metadata),
  split uv sync with --no-install-project for proper Docker layer caching
- pyproject.toml: remove redundant data key (uv_build includes
  VCS-tracked files automatically; data key installs to data scheme)
- .pre-commit-config.yaml: pin uv==0.7.6 in additional_dependencies
- test.yml: add explicit uv sync steps for better CI diagnostics

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: viraatc

.github/workflows/test.yml

@@ -17,8 +17,11 @@ jobs:
         with:
           python-version-file: .python-version
 
+      - name: Install dependencies
+        run: uv sync --frozen --extra test
+
       - name: Run tests
-        run: uv run --frozen --extra test pytest -xv -m "not slow and not performance" --cov=src --cov-report=xml --cov-report=html
+        run: uv run pytest -xv -m "not slow and not performance" --cov=src --cov-report=xml --cov-report=html
 
       - name: Upload coverage report
         uses: actions/upload-artifact@v4
@@ -38,5 +41,8 @@ jobs:
         with:
           python-version-file: .python-version
 
+      - name: Install dependencies
+        run: uv sync --frozen --extra dev --extra test --extra performance
+
       - name: Audit dependencies for known vulnerabilities
-        run: uv run --frozen --extra dev --extra test --extra performance pip-audit
+        run: uv run pip-audit

.pre-commit-config.yaml

@@ -67,6 +67,6 @@ repos:
         name: Check uv.lock is up-to-date
         entry: uv lock --check
         language: python
-        additional_dependencies: [uv]
+        additional_dependencies: ["uv==0.7.6"]
         pass_filenames: false
         files: ^pyproject\.toml$

pyproject.toml

@@ -13,7 +13,6 @@ environments = [
 
 [tool.uv-build]
 module-root = "src"
-data = {"inference_endpoint" = ["config/templates/*.yaml"]}
 exclude = ["inference_endpoint/evaluation/livecodebench/_server.py"]
 
 [project]

scripts/Dockerfile.dev

@@ -23,9 +23,8 @@ RUN apt-get update && \
 RUN mkdir /mnt/inference-endpoint
 WORKDIR /mnt/inference-endpoint
 
-# Copy lockfile + project metadata first for Docker layer caching
-COPY pyproject.toml uv.lock .python-version ./
-COPY src/ ./src/
+# Copy lockfile + project metadata for dependency layer caching
+COPY pyproject.toml uv.lock .python-version README.md ./
 
 # Create a non-root user for security
 # Accept UID and GID as build arguments to match host user
@@ -40,4 +39,9 @@ RUN if ! getent group ${GROUP_ID}; then \
 USER appuser
 ENV PATH="/opt/venv/bin:/home/appuser/.local/bin:$PATH"
 
+# Install dependencies first (cached unless pyproject.toml/uv.lock change)
+RUN uv sync --frozen --no-install-project --extra dev --extra test
+
+# Copy source and install project
+COPY src/ ./src/
 RUN uv sync --frozen --extra dev --extra test