feat: CI

.github/workflows/ci.yml

@@ -0,0 +1,62 @@
+name: pyphos-ci
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  backend-tests:
+    runs-on: ubuntu-latest
+
+    # Dummy values, just enough for the app to import and for the test suite
+    # to run. The test suite never touches DATABASE_URL for real - it's
+    # entirely SQLite in-memory (see backend/tests/conftest.py) - this only
+    # needs to exist so backend/utils/database.py doesn't raise at import time.
+    env:
+      DATABASE_URL: sqlite:///./ci-placeholder.db
+      SECRET_KEY: ci-test-secret-key
+      ALGORITHM: HS256
+      ACCESS_TOKEN_EXPIRE_MINUTES: "30"
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v6
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.11"
+
+      - name: Install backend dependencies
+        run: pip install -r backend/requirements-dev.txt
+
+      - name: Run backend tests
+        run: python -m pytest -v
+
+  frontend-build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v6
+
+      - name: Set up Node
+        uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+          cache: "npm"
+          cache-dependency-path: frontend/package-lock.json
+
+      - name: Install frontend dependencies
+        working-directory: frontend
+        run: npm ci
+
+      - name: Lint
+        working-directory: frontend
+        run: npm run lint
+
+      - name: Build
+        working-directory: frontend
+        run: npm run build

backend/requirements-dev.txt

@@ -0,0 +1,3 @@
+-r requirements.txt
+pytest==9.0.3
+httpx==0.28.1

backend/requirements.txt

@@ -23,4 +23,5 @@ argon2-cffi-bindings==25.1.0
 cffi==2.0.0
 pycparser==3.0
 playwright==1.61.0
-starlette==1.0.0
+starlette==1.0.0
+pytest==9.0.3

frontend/eslint.config.js

@@ -17,5 +17,21 @@ export default defineConfig([
       globals: globals.browser,
       parserOptions: { ecmaFeatures: { jsx: true } },
     },
+    rules: {
+      'no-unused-vars': ['error', {
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+        caughtErrorsIgnorePattern: '^_',
+      }],
+    },
+  },
+  {
+    // Vendored shadcn/ui components: many co-export a variants function
+    // (e.g. buttonVariants) alongside their component by design, which
+    // trips this rule. That's expected here, not an app bug.
+    files: ['src/components/ui/**/*.{js,jsx}'],
+    rules: {
+      'react-refresh/only-export-components': 'off',
+    },
   },
 ])

frontend/src/App.jsx

@@ -1,7 +1,8 @@
 import { useContext } from "react"
 import { BrowserRouter as Router, Route, Routes, Navigate } from "react-router"
 
-import { AuthProvider, AuthContext } from './lib/AuthContext';
+import { AuthProvider } from './lib/AuthContext';
+import { AuthContext } from './lib/auth-context';
 import { isTokenValid } from './lib/utils';
 import { Toaster } from '@/components/ui/sonner'
 import Landing from './screens/Landing'

frontend/src/components/AngleSlider.jsx

@@ -2,14 +2,8 @@
 
 import { Label } from "@/components/ui/label"
 import { Slider } from "@/components/ui/slider"
+import { getCompassDirection } from "@/lib/utils"
 
-// Azimuth convention used across the app: South is 0°, West is positive, East is negative.
-const COMPASS_DIRECTIONS = ["S", "SW", "W", "NW", "N", "NE", "E", "SE"]
-
-export function getCompassDirection(azimuth) {
-  const normalized = ((azimuth % 360) + 360) % 360
-  return COMPASS_DIRECTIONS[Math.round(normalized / 45) % 8]
-}
 
 export function AngleSlider({ name, min, max, value, onValueChange, tag }) {
   return (

frontend/src/components/EditProjectDialog.jsx

@@ -1,4 +1,4 @@
-import { useEffect, useState } from "react"
+import { useState } from "react"
 import { useNavigate, useParams } from "react-router"
 
 import {
@@ -28,7 +28,7 @@ import LocationCombobox from "./LocationCombobox"
 export default function EditProjectDialog({ onSaved }) {
     const { projectId } = useParams();
     const navigate = useNavigate()
-    const { data: project, loading: projLoading, error: projError } = useProject(projectId)
+    const { data: project } = useProject(projectId)
     const updateProject = useUpdateProject()
     const deleteProject = useDeleteProject()
 
@@ -42,25 +42,25 @@ export default function EditProjectDialog({ onSaved }) {
     const [touched, setTouched] = useState({})
     const [deleting, setDeleting] = useState(false)
     const [deleteError, setDeleteError] = useState(null)
+    const [loadedProjectId, setLoadedProjectId] = useState(null)
 
     const nameCheck = validateName(projectName, "Project name")
     const locationCheck = validateRequired(locationQuery, "Location")
     const isFormValid = nameCheck.valid && locationCheck.valid
 
-    useEffect(() => {
-        if (project) {
-            setProjectName(project.name ?? "")
-            setLocation({
-                name: project.location,
-                country_code: project.country_code,
-                admin1: null,
-                lat: project.lat,
-                lon: project.lon,
-            })
-            setLocationQuery(project.location ?? "")
-            setLocationChanged(false)
-        }
-    }, [project])
+    if (project && project.id !== loadedProjectId) {
+        setLoadedProjectId(project.id)
+        setProjectName(project.name ?? "")
+        setLocation({
+            name: project.location,
+            country_code: project.country_code,
+            admin1: null,
+            lat: project.lat,
+            lon: project.lon,
+        })
+        setLocationQuery(project.location ?? "")
+        setLocationChanged(false)
+    }
 
     function markTouched(field) {
         setTouched((t) => ({ ...t, [field]: true }))

frontend/src/components/Header.jsx

@@ -1,11 +1,9 @@
-import { AuthContext } from '@/lib/AuthContext'
+import { AuthContext } from '@/lib/auth-context'
 import { useContext } from "react"
 import { useNavigate } from 'react-router'
 
 import {
   SettingsIcon,
-  BellIcon,
-  CreditCardIcon,
   LogOutIcon,
 } from "lucide-react"
 import {

frontend/src/components/LocationCombobox.jsx

@@ -9,7 +9,7 @@
     ComboboxList,
 } from "@/components/ui/combobox"
 
-import { AuthContext } from "../lib/AuthContext"
+import { AuthContext } from "../lib/auth-context"
 import { searchLocations } from "../lib/api"
 
 const DEBOUNCE_MS = 300
@@ -37,28 +37,36 @@
     }
 
     useEffect(() => {
-        setQuery(formatLocation(value))
+        function syncQuery() {
+            setQuery(formatLocation(value))
+        }
+
+        syncQuery()
     }, [value])
 
     useEffect(() => {
-        if (!query.trim() || query === formatLocation(value)) {
-            setLoading(false)
-            return
-        }
-
-        setLoading(true)
-        const timeout = setTimeout(async () => {
-            try {
-                const candidates = await searchLocations(token, query)
-                setResults(candidates ?? [])
-            } catch {
-                setResults([])
-            } finally {
+        function search() {
+            if (!query.trim() || query === formatLocation(value)) {
                 setLoading(false)
+                return
             }
-        }, DEBOUNCE_MS)
 
-        return () => clearTimeout(timeout)
+            setLoading(true)
+            const timeout = setTimeout(async () => {
+                try {
+                    const candidates = await searchLocations(token, query)
+                    setResults(candidates ?? [])
+                } catch {
+                    setResults([])
+                } finally {
+                    setLoading(false)
+                }
+            }, DEBOUNCE_MS)
+
+            return () => clearTimeout(timeout)
+        }
+
+        return search()
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [query, token])
 

frontend/src/components/ui/alert-dialog.jsx

@@ -1,6 +1,5 @@
 "use client"
 
-import * as React from "react"
 import { AlertDialog as AlertDialogPrimitive } from "@base-ui/react/alert-dialog"
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/avatar.jsx

@@ -1,4 +1,3 @@
-import * as React from "react"
 import { Avatar as AvatarPrimitive } from "@base-ui/react/avatar"
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/breadcrumb.jsx

@@ -1,4 +1,3 @@
-import * as React from "react"
 import { mergeProps } from "@base-ui/react/merge-props"
 import { useRender } from "@base-ui/react/use-render"
 

frontend/src/components/ui/card.jsx

@@ -1,5 +1,3 @@
-import * as React from "react"
-
 import { cn } from "@/lib/utils"
 
 function Card({

frontend/src/components/ui/dialog.jsx

@@ -1,4 +1,3 @@
-import * as React from "react"
 import { Dialog as DialogPrimitive } from "@base-ui/react/dialog"
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/dropdown-menu.jsx

@@ -1,4 +1,3 @@
-import * as React from "react"
 import { Menu as MenuPrimitive } from "@base-ui/react/menu"
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/input-group.jsx

@@ -1,6 +1,5 @@
 "use client"
 
-import * as React from "react"
 import { cva } from "class-variance-authority";
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/input.jsx

@@ -1,4 +1,3 @@
-import * as React from "react"
 import { Input as InputPrimitive } from "@base-ui/react/input"
 
 import { cn } from "@/lib/utils"

frontend/src/components/ui/label.jsx

@@ -1,7 +1,5 @@
 "use client"
 
-import * as React from "react"
-
 import { cn } from "@/lib/utils"
 
 function Label({

frontend/src/components/ui/table.jsx

@@ -1,7 +1,5 @@
 "use client"
 
-import * as React from "react"
-
 import { cn } from "@/lib/utils"
 
 function Table({

frontend/src/components/ui/textarea.jsx

@@ -1,5 +1,3 @@
-import * as React from "react"
-
 import { cn } from "@/lib/utils"
 
 function Textarea({

frontend/src/lib/AuthContext.jsx

@@ -1,6 +1,5 @@
-import React, { createContext, useState, useEffect } from 'react';
-
-export const AuthContext = createContext();
+import { useState, useEffect } from 'react';
+import { AuthContext } from './auth-context';
 
 async function fetchCurrentUser(token) {
   const response = await fetch('/api/users/me', {
@@ -39,18 +38,15 @@ export const AuthProvider = ({ children }) => {
   };
 
   useEffect(() => {
-    const storedToken = localStorage.getItem('token');
-    if (storedToken) {
-      setToken(storedToken);
+    async function syncUser() {
+      if (!token || isGuest) {
+        setUser(null);
+        return;
+      }
+      setUser(await fetchCurrentUser(token));
     }
-  }, []);
 
-  useEffect(() => {
-    if (!token || isGuest) {
-      setUser(null);
-      return;
-    }
-    fetchCurrentUser(token).then(setUser);
+    syncUser();
   }, [token, isGuest]);
 
   return (

frontend/src/lib/api.js

@@ -1,5 +1,5 @@
 import { useState, useEffect, useContext } from "react"
-import { AuthContext } from "./AuthContext"
+import { AuthContext } from "./auth-context"
 
 const BASE_URL = "/api"
 
@@ -12,11 +12,11 @@ export function useApi(endpoint) {
     useEffect(() => {
         if (!endpoint) return
 
-        setLoading(true)
-        setData(null)
-        setError(null)
-
         const fetchContent = async () => {
+            setLoading(true)
+            setData(null)
+            setError(null)
+
             try {
                 const response = await fetch(endpoint, {
                     headers: token ? { Authorization: `Bearer ${token}` } : {}

frontend/src/lib/auth-context.js

@@ -0,0 +1,3 @@
+import { createContext } from 'react';
+
+export const AuthContext = createContext();

frontend/src/lib/useData.js

@@ -1,5 +1,5 @@
 import { useContext, useMemo } from 'react'
-import { AuthContext } from './AuthContext'
+import { AuthContext } from './auth-context'
 import {
     useApi,
     createProject as createProjectApi,