Merge pull request #8 from Bovi-analytics/feat/pre-commit

Feat/pre commit

Author: DouwedeKok

.env.example

@@ -1,3 +1,4 @@
-#for secret/key management 
+#for secret/key management
 milkbot_key = YOUR_MILKBOT_KEY
-uv_publish_token = YOUR_UNV_TOKEN
+uv_publish_token = YOUR_UNV_TOKEN
+pulumi_passphrase = YOUR_PASSPHRASE

.github/workflows/api-tests.yml

@@ -0,0 +1,44 @@
+name: API Tests
+
+on:
+  pull_request:
+    branches:
+      - dev
+      - master
+
+jobs:
+  api-test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v3
+
+      - name: Set up Python 3.12
+        run: uv python install 3.12
+
+      - name: Install app dependencies
+        working-directory: apps/lactation_curves
+        run: uv sync
+
+      - name: Start API server and run tests
+        working-directory: apps/lactation_curves
+        run: |
+          nohup uv run python -m uvicorn main:app --host 0.0.0.0 --port 8000 > /tmp/uvicorn.log 2>&1 &
+          for i in $(seq 1 30); do
+            if curl -sf http://localhost:8000/ > /dev/null 2>&1; then
+              echo "API is ready"
+              break
+            fi
+            echo "Waiting for API... ($i/30)"
+            sleep 1
+          done
+          if ! curl -sf http://localhost:8000/ > /dev/null 2>&1; then
+            echo "API failed to start. Server logs:"
+            cat /tmp/uvicorn.log
+            exit 1
+          fi
+          uv run pytest tests/ -v

.github/workflows/cffconvert.yml

@@ -2,12 +2,11 @@ name: cffconvert
 
 on:
   push:
-    paths:
-      - CITATION.cff
-  
+    branches:
+      - master
   pull_request:
-    paths: 
-      - CITATION.cff
+    branches:
+      - master
 
 jobs:
   validate:

.github/workflows/docs.yml

@@ -4,6 +4,9 @@ on:
   push:
     branches:
       - master
+  pull_request:
+    branches:
+      - master
 
 permissions:
   contents: read
@@ -49,6 +52,7 @@ jobs:
           path: packages/python/lactation/site
 
   deploy:
+    if: github.event_name == 'push'
     needs: build-docs
     runs-on: ubuntu-latest
     environment:

.github/workflows/publish.yml

@@ -0,0 +1,32 @@
+name: Publish (lactationcurve)
+
+on:
+  push:
+    branches:
+      - master
+
+permissions:
+  id-token: write
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    environment: master
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v3
+
+      - name: Set up Python 3.12
+        run: uv python install 3.12
+
+      - name: Build package
+        working-directory: packages/python/lactation
+        run: uv build
+
+      - name: Publish to PyPI
+        working-directory: packages/python/lactation
+        run: uv publish

.github/workflows/pytest.yml

@@ -0,0 +1,29 @@
+name: Unit Tests (lactationcurve)
+
+on:
+  pull_request:
+    branches:
+      - dev
+      - master
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v3
+
+      - name: Set up Python 3.12
+        run: uv python install 3.12
+
+      - name: Install dependencies
+        run: uv sync
+
+      - name: Run tests
+        env:
+          milkbot_key: ${{ secrets.MILKBOT_KEY }}
+        run: uv run pytest

.gitignore

@@ -20,4 +20,5 @@ wheels/
 .env.local
 node_modules
 .next
+*.tsbuildinfo
 tests/lactationcurve/test_dotenv.py

.pre-commit-config.yaml

@@ -0,0 +1,70 @@
+# Pre-commit hooks for lactation_curve_core
+# Install: uv run pre-commit install
+# Run all: uv run pre-commit run --all-files
+
+repos:
+  # ── General checks ──────────────────────────────────────────
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v5.0.0
+    hooks:
+      - id: check-added-large-files
+        args: ['--maxkb=500']
+      - id: check-merge-conflict
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-yaml
+      - id: check-toml
+      - id: check-json
+
+  # ── Secrets detection ───────────────────────────────────────
+  - repo: https://github.com/Yelp/detect-secrets
+    rev: v1.5.0
+    hooks:
+      - id: detect-secrets
+        args: ['--baseline', '.secrets.baseline']
+        exclude: (bun\.lock|uv\.lock|\.tsbuildinfo)$
+
+  # ── Python formatting + linting ─────────────────────────────
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.11.0
+    hooks:
+      - id: ruff-format
+      - id: ruff
+        args: [--fix]
+
+  # ── Python type checking ────────────────────────────────────
+  - repo: local
+    hooks:
+      - id: basedpyright
+        name: basedpyright
+        entry: uv run basedpyright
+        language: system
+        types: [python]
+        pass_filenames: false
+
+  # ── TypeScript formatting (prettier) ────────────────────────
+      - id: prettier
+        name: prettier
+        entry: bash -c 'cd apps/dashboard && bun run format:check'
+        language: system
+        types_or: [ts, tsx, javascript, jsx, json, css]
+        files: ^apps/dashboard/
+        pass_filenames: false
+
+  # ── TypeScript linting (eslint) ─────────────────────────────
+      - id: eslint
+        name: eslint
+        entry: bash -c 'cd apps/dashboard && bun run lint'
+        language: system
+        types_or: [ts, tsx, javascript, jsx]
+        files: ^apps/dashboard/src/
+        pass_filenames: false
+
+  # ── TypeScript type checking (tsc) ──────────────────────────
+      - id: tsc
+        name: tsc
+        entry: bash -c 'cd apps/dashboard && bun run typecheck'
+        language: system
+        types_or: [ts, tsx]
+        files: ^apps/dashboard/
+        pass_filenames: false