fix: address PR #455 review comments (#478)

- Pin uv version (0.7.12) and use --locked in CI workflows
- Fix Dockerfile: split uv sync into deps-only + project install
- Remove || true from make lint so pylint failures are not masked
- Move max-line-length to [tool.pylint.format] (canonical section)
- Fix docs: use 'uv sync' instead of 'uv add' for existing deps
- Remove dead initial LLM_PROVIDER/AZURE_FLAG assignments in config

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: gkorland

.github/workflows/playwright.yml

@@ -35,7 +35,7 @@ jobs:
     - name: Install uv
       uses: astral-sh/setup-uv@d4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86 # v5.4.2
       with:
-        version: "latest"
+        version: "0.7.12"
 
     # Setup Node.js
     - uses: actions/setup-node@v6
@@ -49,7 +49,7 @@ jobs:
 
     # Install Python dependencies
     - name: Install Python dependencies
-      run: uv sync
+      run: uv sync --locked
 
     # Install Node dependencies (root - for Playwright)
     - name: Install root dependencies

.github/workflows/tests.yml

@@ -39,11 +39,11 @@ jobs:
     - name: Install uv
       uses: astral-sh/setup-uv@d4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86 # v5.4.2
       with:
-        version: "latest"
+        version: "0.7.12"
 
     - name: Install dependencies
       run: |
-        uv sync
+        uv sync --locked
 
     - name: Install frontend dependencies
       run: |

Dockerfile

@@ -40,8 +40,8 @@ ENV UV_SYSTEM_PYTHON=1
 # Ensure venv binaries are on PATH (uv sync always creates .venv)
 ENV PATH="/app/.venv/bin:$PATH"
 
-# Install Python dependencies from pyproject.toml
-RUN uv sync --frozen --no-dev
+# Install Python dependencies only (project itself installed after COPY)
+RUN uv sync --frozen --no-dev --no-install-project
 
 # Install Node.js (Node 22) so we can build the frontend inside the image.
 # Use NodeSource setup script to get a recent Node version on Debian-based images.
@@ -75,6 +75,9 @@ RUN npm --prefix ./app run build
 # Copy application code 
 COPY . .
 
+# Install the project package now that source code is available
+RUN uv sync --frozen --no-dev
+
 # Copy and make start.sh executable
 COPY start.sh /start.sh
 RUN chmod +x /start.sh

Makefile

@@ -42,7 +42,7 @@ test-e2e-debug: build-dev ## Run E2E tests with debugging enabled
 
 lint: ## Run linting (backend + frontend)
 	@echo "Running backend lint (pylint)"
-	uv run python -m pylint $(shell git ls-files '*.py') || true
+	uv run python -m pylint $(shell git ls-files '*.py')
 	@echo "Running frontend lint (eslint)"
 	make lint-frontend
 

api/config.py

@@ -70,10 +70,6 @@ class Config:
     Configuration class for the text2sql module.
     """
 
-    # Provider flag: "azure", "openai", "gemini", "anthropic", "ollama", "cohere"
-    LLM_PROVIDER = "azure"
-    AZURE_FLAG = True
-
     # User-provided overrides via env vars
     _user_completion = os.getenv("COMPLETION_MODEL", "")
     _user_embedding = os.getenv("EMBEDDING_MODEL", "")

docs/postgres_loader.md

@@ -13,10 +13,10 @@ This loader connects to a PostgreSQL database and extracts the complete schema i
 
 ## Installation
 
-psycopg2-binary is already included in QueryWeaver's dependencies. If you need to add it manually:
+psycopg2-binary is already included in QueryWeaver's dependencies. Install project dependencies with:
 
 ```bash
-uv add psycopg2-binary
+uv sync
 ```
 
 ## Usage

pyproject.toml

@@ -62,9 +62,11 @@ filterwarnings = [
 ]
 
 [tool.pylint.main]
-max-line-length = 120
 disable = [
     "C0114",  # missing-module-docstring
     "C0115",  # missing-class-docstring
     "C0116",  # missing-function-docstring
 ]
+
+[tool.pylint.format]
+max-line-length = 120