chore: release v1.8.6 (#129 )

fix: make frontend image self-contained for read-only filesystems (#128 )
Revert Dockerfile to use /tmp redirect for envsubst output, so the image works regardless of docker-compose.yml tmpfs configuration. Removes the uid=101,gid=101 requirement from compose that was a breaking change.
2026-02-08 14:06:03 +01:00 · 2026-02-08 14:03:53 +01:00 · 2026-02-08 12:54:19 +00:00 · 2026-02-08 13:53:19 +01:00 · 2026-02-08 13:35:52 +01:00 · 2026-02-08 13:33:40 +01:00
38 changed files with 2436 additions and 486 deletions
@@ -0,0 +1,305 @@
+---
+name: release-manager
+description: Manages the full release lifecycle - from branching and PRs through versioning and GitHub release notes. Use when code changes are complete and ready to ship.
+argument-hint: Describe what was changed, e.g., "fix stock correction bug" or "new refill tracking feature"
+---
+
+# Release Manager Agent
+
+You are the release manager for **MedAssist-ng**. Your job is to guide code from "done" to "released" following the project's strict branch protection, CI pipeline, and semantic versioning rules.
+
+**All output (commits, PR titles, release notes) MUST be in English**, even if the user communicates in German.
+
+## Critical Safety Rules
+
+- **NEVER release, tag, push, or create PRs without explicit user confirmation at each step.** Always present your plan and wait for approval.
+- **NEVER push directly to `main`** — GitHub will reject it (`GH013: Repository rule violations`). All changes go through Pull Requests.
+- **NEVER skip CI checks.** Wait for all status checks to pass before merging.
+
+---
+
+## Task 1: Branch, PR, and Merge Workflow
+
+When code changes (features or bug fixes) are complete and tested locally:
+
+### Step 1: Verify Readiness
+
+1. Check for uncommitted changes: `git status`
+2. Ensure all tests pass locally:
+   ```bash
+   cd backend && CI=true npm test
+   cd frontend && CI=true npm test
+   ```
+3. If tests fail, stop and fix them first.
+
+### Step 2: Create Feature Branch
+
+1. Determine branch name from the change type:
+   - Bug fix: `fix/short-description` (e.g., `fix/stock-correction-consumption`)
+   - Feature: `feat/short-description` (e.g., `feat/refill-tracking`)
+   - Chore: `chore/short-description`
+2. Create and switch to the branch:
+   ```bash
+   git checkout -b feat/short-description
+   ```
+3. Stage and commit changes with a conventional commit message:
+   ```bash
+   git add .
+   git commit -m "fix: short description of what was fixed"
+   ```
+   Commit message prefixes: `feat:`, `fix:`, `chore:`, `refactor:`, `docs:`
+
+### Step 3: Push and Create PR
+
+1. Push the branch:
+   ```bash
+   git push -u origin feat/short-description
+   ```
+2. Create a Pull Request via GitHub CLI:
+   ```bash
+   gh pr create --title "fix: short description" --body "Description of charges"
+   ```
+3. **Present the PR URL to the user and wait for confirmation.**
+
+### Step 4: Wait for CI and Merge
+
+1. Monitor CI status:
+   ```bash
+   gh pr checks <PR_NUMBER> --watch
+   ```
+   Required checks:
+   - ✅ `backend-test` (TypeScript type-check + vitest coverage)
+   - ✅ `frontend-build` (npm build)
+2. If CI fails: analyze the failure, fix it, push again, and re-check.
+3. Once CI is green, **ask the user for merge confirmation**, then:
+   ```bash
+   gh pr merge <PR_NUMBER> --squash --delete-branch
+   ```
+4. Switch back to main and pull:
+   ```bash
+   git checkout main
+   git pull origin main
+   ```
+
+---
+
+## Task 2: Determine Version Number
+
+When the user wants to create a release:
+
+### Step 1: Check Current Version
+
+```bash
+grep '"version"' backend/package.json
+```
+
+Also check the latest git tag:
+```bash
+git tag --sort=-v:refname | head -5
+```
+
+### Step 2: Analyze Changes Since Last Release
+
+```bash
+git log $(git describe --tags --abbrev=0)..HEAD --oneline
+```
+
+Read through the commits to understand what changed.
+
+### Step 3: Select SemVer Level
+
+Apply these rules strictly:
+
+| Change Type | Version Bump | Example |
+|------------|-------------|---------|
+| Bug fixes only, no new features | **patch** | `1.4.2` → `1.4.3` |
+| New features (backward compatible) | **minor** | `1.4.2` → `1.5.0` |
+| Breaking changes (DB schema without migration, removed ENV vars, changed API) | **major** | `1.4.2` → `2.0.0` |
+
+**Guidelines:**
+- When in doubt between patch and minor, prefer **minor** if any user-visible behavior is new.
+- Bug fixes that also introduce small UX improvements = **patch**.
+- Multiple bug fixes in one release = still **patch**.
+- New UI sections, new API endpoints, new settings = **minor**.
+- If a user can run `docker compose pull && docker compose up -d` without changing anything → NOT a breaking change.
+
+**Present your version recommendation to the user with reasoning and wait for confirmation.**
+
+---
+
+## Task 3: Execute Release
+
+Use the release script whenever possible:
+
+```bash
+./scripts/release.sh <patch|minor|major>
+```
+
+This script handles: branch creation → version bump → PR → CI wait → merge → signed tag → push.
+
+### Version Files (MANDATORY)
+
+The version number is displayed in the **About modal** (Settings → About) as a single unified app version. This version is a **clickable link** pointing to the corresponding GitHub release (`https://github.com/DanielVolz/medassist-ng/releases/tag/vX.Y.Z`). The version is read from:
+
+- **`backend/package.json`** → Backend version, returned by `/health` endpoint
+- **`frontend/package.json`** → Frontend version, injected at build time via Vite's `__APP_VERSION__` define and used to construct the release link
+
+**Both files MUST be updated to the new version before tagging a release.** If forgotten:
+- The About modal will show the old version
+- The version link will point to a non-existent GitHub release page
+
+### Manual Release (if script is not available)
+
+1. Create release branch:
+   ```bash
+   git checkout main && git pull origin main
+   git checkout -b chore/release-X.Y.Z
+   ```
+2. Update versions in **both** `backend/package.json` and `frontend/package.json` to `X.Y.Z`
+3. Commit, push, create PR, wait for CI, merge (same as Task 1)
+4. Create signed tag:
+   ```bash
+   git checkout main && git pull origin main
+   git tag -s "vX.Y.Z" -m "Release vX.Y.Z"
+   git push origin "vX.Y.Z"
+   ```
+
+### After Tagging
+
+- The `docker-build.yml` workflow automatically builds and pushes Docker images to GHCR.
+- The `version-bump.yml` workflow automatically updates `package.json` versions if needed.
+- Track progress: `https://github.com/DanielVolz/medassist-ng/actions`
+
+---
+
+## Task 4: Write Release Notes
+
+When the user asks to write release notes (MANDATORY for minor/major releases):
+
+### Step 1: Gather Changes
+
+```bash
+git log vPREVIOUS..vNEW --oneline
+```
+
+Read the actual code changes (not just commit messages) to understand what was added or fixed.
+
+### Step 2: Write Release Notes
+
+**Release title:** Use just `vX.Y.Z` (e.g., `v1.4.1`), NOT "Release vX.Y.Z".
+
+**Required structure:**
+
+1. **"What's New"** (1-2 sentences): Brief intro explaining the main change
+2. **"New Features" / "Bug Fixes" / "Improvements"**: Grouped bullet points with **bold feature names** and descriptions
+3. **"Where to Find It"**: Tell users where they can access the new feature or see the fix
+4. **Breaking Changes Warning** (if applicable): See below
+
+**Style guidelines:**
+- Use `### Heading` for sections
+- Use **bold** for feature names in bullet points
+- Keep descriptions on the same line as the feature name
+- Minimal emoji usage (sparingly, not on every line)
+- Always end with "Where to Find It" section
+- End with: `**Full Changelog**: https://github.com/DanielVolz/medassist-ng/compare/vPREV...vNEW`
+
+**ONLY include user-relevant changes.** DO NOT include:
+- ❌ Technical implementation details (new columns, endpoints, database changes)
+- ❌ Number of tests added
+- ❌ Internal API changes (unless breaking)
+- ❌ Excessive emoji on every bullet point
+- ❌ .gitignore changes or other developer-only file changes
+- ❌ AI/Copilot instruction updates
+- ❌ CI/CD workflow changes (unless affecting users)
+- ❌ Code refactoring without user-visible changes
+
+### Example: Good Release Notes
+
+```markdown
+## What's New
+
+This release introduces a medication refill tracking feature and improves the mobile user experience.
+
+### New Features
+
+- **Medication Refill**: Track when you refill your medications with a single click. Add full packs or individual pills and view complete refill history.
+- **Automatic Stock Updates**: Stock levels are automatically recalculated after each refill.
+- **Refill History**: Each medication shows a complete history of all refills with timestamps.
+
+### Mobile Improvements
+
+- **Centered Tooltips**: Info tooltips now display centered on screen for better readability.
+- **Touch-friendly**: Tooltips close automatically when scrolling on touch devices.
+
+### Where to Find It
+
+The refill button appears in the medication detail modal and in the edit form for each medication.
+
+**Full Changelog**: https://github.com/DanielVolz/medassist-ng/compare/v1.2.3...v1.3.0
+```
+
+### Breaking Changes Warning
+
+If the update breaks existing configurations or stored data, it MUST be prominently warned:
+
+**Breaking Changes include:**
+- Database schema changes without automatic migration
+- Removed or renamed ENV variables
+- Changed API endpoints
+- Incompatible `.env` format changes
+- Loss of stored data after update
+
+**Format:**
+
+```markdown
+## ⚠️ BREAKING CHANGES - Please read before updating!
+
+**Database migration required**: This update changes the database schema.
+Existing installations need to:
+1. Create backup of `data/` folder
+2. Stop containers
+3. Perform update
+4. If issues occur: Rollback using backup
+
+**ENV variables changed**:
+- `OLD_VAR` was renamed to `NEW_VAR`
+- `REMOVED_VAR` is no longer supported
+```
+
+**What is NOT a Breaking Change:**
+- ✅ New optional columns with DEFAULT values
+- ✅ New ENV variables (with sensible defaults)
+- ✅ New features that don't affect existing data
+- ✅ Bug fixes that correct behavior
+
+### Step 3: Publish
+
+Present the release notes to the user. They will copy them to the GitHub release page or ask you to publish via:
+```bash
+gh release create vX.Y.Z --title "vX.Y.Z" --notes "RELEASE_NOTES_HERE"
+```
+
+---
+
+## Complete Workflow Summary
+
+```
+Code complete & tests pass locally
+        ↓
+1. Create feature branch (fix/... or feat/...)
+2. Commit, push, create PR
+3. Wait for CI (backend-test + frontend-build)
+4. Merge PR to main (squash + delete branch)
+        ↓
+Ready for release?
+        ↓
+5. Check current version (git tag + package.json)
+6. Analyze changes → determine SemVer level
+7. Run ./scripts/release.sh <patch|minor|major>
+   (or manually: branch → version bump → PR → CI → merge → tag)
+        ↓
+8. Write release notes (mandatory for minor/major)
+9. Publish GitHub release
+        ↓
+Docker images built automatically via CI
+```
@@ -180,141 +180,6 @@ gh pr merge --squash --delete-branch
 | `.github/workflows/test.yml` | Pull Requests | Run tests, block PR on failures |
 | `.github/workflows/docker-build.yml` | Push to main, Tags | Tests + Build and push Docker images |

-### Adding New Code - Checklist
-1. ✅ Implement feature
-2. ✅ Write tests for the feature
-3. ✅ Run `npm run test:coverage` locally
-4. ✅ Coverage must not decrease
-5. ✅ Create and push feature branch
-6. ✅ Create Pull Request
-7. ✅ Wait until CI is green
-8. ✅ Merge PR (branch is automatically deleted)
-
-## GitHub Releases
-
-> ⚠️ **IMPORTANT**: All GitHub Releases must be written in **English**!
-
-### Release Workflow (MANDATORY for minor/major releases)
-
-The `main` branch is protected - releases are created via GitHub's release UI or API.
-
-**Release Process:**
-1. Create a new release on GitHub with tag `vX.Y.Z`
-2. **Automatic Version Bump**: A GitHub Action (`version-bump.yml`) automatically updates `package.json` versions to match the release tag
-3. User asks AI to write release notes: "Write the release notes for vX.Y.Z"
-4. AI writes descriptive release notes following the style guide below
-5. User publishes the release with the written notes
-
-> ⚠️ **MANDATORY for minor and major releases**: The AI assistant MUST write proper descriptive release notes!
-> Do NOT just publish the auto-generated commit list. Follow the process above.
-
-**AI Assistant Release Notes Workflow:**
-1. When user asks to write release notes for a version:
-   - Check commits since previous tag: `git log vPREV..vNEW --oneline`
-   - Read through the changes to understand what was added/fixed
-   - Write release notes following the style guide below
-   - Present the notes to the user for copying to GitHub
-
-### Creating Release Notes
-
-> ⚠️ **MANDATORY**: GitHub Releases MUST contain a written message!
-> Not just auto-generated commit lists, but a brief descriptive text.
-
-**Release title:** Use just `vX.Y.Z` (e.g., `v1.4.1`), NOT "Release vX.Y.Z".
-
-**Keep it informative but concise.** Users want to know what changed and where to find it.
-
-**Required structure of release notes:**
-
-1. **"What's New"** (1-2 sentences): Brief intro explaining the main change
-2. **"New Features" / "Improvements"**: Grouped bullet points with **bold feature names** and descriptions
-3. **"Where to Find It"**: Tell users where they can access the new feature
-4. **Breaking Changes Warning** (if applicable): See below
-
-**Style guidelines:**
- Use `### Heading` for sections (New Features, Improvements, Security, etc.)
- Use **bold** for feature names in bullet points
- Keep descriptions on the same line as the feature name
- Minimal emoji usage (sparingly, not on every line)
- Always end with "Where to Find It" section
-
-**DO NOT include:**
- ❌ Technical implementation details (new columns, endpoints, database changes)
- ❌ Number of tests added
- ❌ Internal API changes (unless breaking)
- ❌ Excessive emoji on every bullet point
- ❌ .gitignore changes or other developer-only file changes
- ❌ AI/Copilot instruction updates
- ❌ CI/CD workflow changes (unless affecting users)
- ❌ Code refactoring without user-visible changes
-
-**Only include user-relevant changes** - things that affect what users see or experience in the app.
-
-**Example of good release notes:**
-
-```markdown
-## What's New
-
-This release introduces a medication refill tracking feature and improves the mobile user experience.
-
-### New Features
-
- **Medication Refill**: Track when you refill your medications with a single click. Add full packs or individual pills and view complete refill history.
- **Automatic Stock Updates**: Stock levels are automatically recalculated after each refill.
- **Refill History**: Each medication shows a complete history of all refills with timestamps.
-
-### Mobile Improvements
-
- **Centered Tooltips**: Info tooltips now display centered on screen for better readability.
- **Touch-friendly**: Tooltips close automatically when scrolling on touch devices.
-
-### Where to Find It
-
-The refill button appears in the medication detail modal and in the edit form for each medication.
-
-**Full Changelog**: https://github.com/DanielVolz/medassist-ng/compare/v1.2.3...v1.3.0
-```
-
-### Breaking Changes Warning (CRITICAL!)
-
-> ⚠️ **MANDATORY**: If an update breaks existing configurations or stored data, it MUST be prominently warned about in the release notes!
-
-**Breaking Changes include:**
- Database schema changes without automatic migration
- Removed or renamed ENV variables
- Changed API endpoints
- Incompatible `.env` format changes
- Loss of stored data after update
-
-**Format for Breaking Changes:**
-
-```markdown
-## ⚠️ BREAKING CHANGES - Please read before updating!
-
-**Database migration required**: This update changes the database schema. 
-Existing installations need to:
-1. Create backup of `data/` folder
-2. Stop containers
-3. Perform update
-4. If issues occur: Rollback using backup
-
-**ENV variables changed**: 
- `OLD_VAR` was renamed to `NEW_VAR`
- `REMOVED_VAR` is no longer supported
-
-**Medication data**: Intake schedules with only one time entry will be automatically 
-migrated. Please verify all times are correct after update.
-```
-
-**What is NOT a Breaking Change:**
- ✅ New optional columns with DEFAULT values
- ✅ New ENV variables (with sensible defaults)
- ✅ New features that don't affect existing data
- ✅ Bug fixes that correct behavior
-
-**Rule of thumb**: If a user can simply run `docker compose pull && docker compose up -d` 
-without adjusting anything → Not a Breaking Change.
-
 ## Key Patterns

 ### Backend Routes (`backend/src/routes/`)
@@ -3,8 +3,30 @@ name: "CodeQL"
 on:
  push:
    branches: [main]
+    paths:
+      - '**.js'
+      - '**.ts'
+      - '**.tsx'
+      - '**.jsx'
+      - 'backend/package.json'
+      - 'backend/package-lock.json'
+      - 'frontend/package.json'
+      - 'frontend/package-lock.json'
+      - '.github/codeql/**'
+      - '.github/workflows/codeql.yml'
  pull_request:
    branches: [main]
+    paths:
+      - '**.js'
+      - '**.ts'
+      - '**.tsx'
+      - '**.jsx'
+      - 'backend/package.json'
+      - 'backend/package-lock.json'
+      - 'frontend/package.json'
+      - 'frontend/package-lock.json'
+      - '.github/codeql/**'
+      - '.github/workflows/codeql.yml'
  schedule:
    - cron: "0 6 * * 1" # Weekly on Monday at 6am UTC
  workflow_dispatch: # Allow manual trigger
@@ -10,10 +10,38 @@ permissions:

 jobs:
  # =============================================================================
-  # Backend Tests
+  # Detect which paths changed to skip unnecessary jobs
+  # =============================================================================
+  changes:
+    name: Detect Changes
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: read
+    outputs:
+      backend: ${{ steps.filter.outputs.backend }}
+      frontend: ${{ steps.filter.outputs.frontend }}
+    steps:
+      - uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            backend:
+              - 'backend/**'
+              - 'biome.json'
+              - '.github/workflows/test.yml'
+            frontend:
+              - 'frontend/**'
+              - 'biome.json'
+              - '.github/workflows/test.yml'
+
+  # =============================================================================
+  # Backend Tests (skipped if no backend-related files changed)
  # =============================================================================
  backend-test:
    name: Backend Tests
+    needs: changes
+    if: needs.changes.outputs.backend == 'true'
    runs-on: ubuntu-latest
    permissions:
      contents: read
@@ -53,10 +81,12 @@ jobs:
          retention-days: 7

  # =============================================================================
-  # Frontend Build Validation
+  # Frontend Build Validation (skipped if no frontend-related files changed)
  # =============================================================================
  frontend-build:
    name: Frontend Build
+    needs: changes
+    if: needs.changes.outputs.frontend == 'true'
    runs-on: ubuntu-latest
    permissions:
      contents: read
@@ -1,6 +1,7 @@
 name: Update Test Badges

 on:
+  workflow_dispatch:
  push:
    branches: [main]
    paths:
@@ -21,7 +22,7 @@ jobs:
      - name: Checkout repository
        uses: actions/checkout@v4
        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
+          token: ${{ secrets.BADGE_TOKEN || secrets.GITHUB_TOKEN }}

      - name: Setup Node.js
        uses: actions/setup-node@v4
@@ -47,8 +48,9 @@ jobs:
        run: |
          OUTPUT=$(npm run test:run 2>&1) || true
          echo "$OUTPUT"
-          # Extract "Tests  X passed" from output
-          PASSED=$(echo "$OUTPUT" | grep -oP 'Tests\s+\K\d+(?=\s+passed)' | tail -1)
+          # Strip ANSI escape codes, then extract "Tests  X passed" from output
+          CLEAN=$(echo "$OUTPUT" | sed 's/\x1b\[[0-9;]*m//g')
+          PASSED=$(echo "$CLEAN" | grep -oP 'Tests\s+\K\d+(?=\s+passed)' | tail -1)
          echo "count=$PASSED" >> $GITHUB_OUTPUT

      - name: Run frontend tests and capture count
@@ -60,8 +62,9 @@ jobs:
        run: |
          OUTPUT=$(npm run test:run 2>&1) || true
          echo "$OUTPUT"
-          # Extract "Tests  X passed" from output
-          PASSED=$(echo "$OUTPUT" | grep -oP 'Tests\s+\K\d+(?=\s+passed)' | tail -1)
+          # Strip ANSI escape codes, then extract "Tests  X passed" from output
+          CLEAN=$(echo "$OUTPUT" | sed 's/\x1b\[[0-9;]*m//g')
+          PASSED=$(echo "$CLEAN" | grep -oP 'Tests\s+\K\d+(?=\s+passed)' | tail -1)
          echo "count=$PASSED" >> $GITHUB_OUTPUT

      - name: Update README badges
@@ -88,16 +91,14 @@ jobs:
            exit 0
          fi

-      - name: Check for changes
-        id: git-check
+      - name: Commit and push badge updates
        run: |
-          git diff --quiet README.md || echo "changed=true" >> $GITHUB_OUTPUT
-
-      - name: Commit and push if changed
-        if: steps.git-check.outputs.changed == 'true'
-        run: |
-          git config --local user.email "github-actions[bot]@users.noreply.github.com"
-          git config --local user.name "github-actions[bot]"
+          git config user.name "github-actions[bot]"
+          git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
          git add README.md
-          git commit -m "chore: update test count badges [skip ci]"
-          git push
+          if git diff --cached --quiet; then
+            echo "No badge changes to commit"
+          else
+            git commit -m "chore: update test count badges [skip ci]"
+            git push
+          fi
@@ -18,8 +18,8 @@
 </p>

 <p align="center">
-  <img src="https://img.shields.io/badge/Backend_Tests-454%2F454-brightgreen?logo=vitest" alt="Backend Tests 454/454" />
-  <img src="https://img.shields.io/badge/Frontend_Tests-611%2F611-brightgreen?logo=vitest" alt="Frontend Tests 611/611" />
+  <img src="https://img.shields.io/badge/Backend_Tests-494%2F494-brightgreen?logo=vitest" alt="Backend Tests 454/454" />
+  <img src="https://img.shields.io/badge/Frontend_Tests-639%2F639-brightgreen?logo=vitest" alt="Frontend Tests 611/611" />
 </p>

 ### 🤖 AI-Generated Code
@@ -1,12 +1,12 @@
 {
  "name": "medassist-ng-backend",
-  "version": "1.7.1",
+  "version": "1.8.3",
  "lockfileVersion": 3,
  "requires": true,
  "packages": {
    "": {
      "name": "medassist-ng-backend",
-      "version": "1.7.1",
+      "version": "1.8.3",
      "dependencies": {
        "@fastify/cookie": "^10.0.1",
        "@fastify/cors": "^10.0.1",
@@ -1,6 +1,6 @@
 {
  "name": "medassist-ng-backend",
-  "version": "1.7.1",
+  "version": "1.8.6",
  "private": true,
  "type": "module",
  "scripts": {
@@ -1,162 +1,37 @@
-import { accessSync, constants, existsSync, mkdirSync, statSync, writeFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
+import { existsSync, statSync } from "node:fs";
+import { resolve } from "node:path";
 import { type Client, createClient } from "@libsql/client";
 import dotenv from "dotenv";
 import { drizzle } from "drizzle-orm/libsql";
-import { migrate } from "drizzle-orm/libsql/migrator";

-dotenv.config({ path: process.env.DOTENV_PATH || ".env" });
+// Import utilities from db-utils (side-effect-free)
+import {
+	ensureDataDirectory,
+	ensureDefaultUser,
+	getDataDir,
+	getDbPaths,
+	repairOrphanedDoseIds,
+	repairTrailingHyphenDoseIds,
+	runAlterMigrations,
+	runDrizzleMigrations,
+} from "./db-utils.js";

-// Get migrations folder path (relative to this file's location)
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-const migrationsFolder = resolve(__dirname, "../../drizzle");
+// Re-export all utilities so existing imports from client.ts keep working
+export {
+	buildDbUrl,
+	ensureDataDirectory,
+	ensureDefaultUser,
+	getDataDir,
+	getDbPaths,
+	repairOrphanedDoseIds,
+	repairTrailingHyphenDoseIds,
+	runAlterMigrations,
+	runDrizzleMigrations,
+} from "./db-utils.js";

-// =============================================================================
-// Exported utility functions for testing
-// =============================================================================
-
-/** Build the database URL from a path */
-export function buildDbUrl(dbPath: string): string {
-	return `file:${dbPath}`;
-}
-
-/** Get data directory and database path */
-export function getDbPaths(cwd: string = process.cwd()): { dataDir: string; dbPath: string; url: string } {
-	const dataDir = resolve(cwd, "data");
-	const dbPath = resolve(dataDir, "medassist-ng.db");
-	const url = buildDbUrl(dbPath);
-	return { dataDir, dbPath, url };
-}
-
-/** Ensure data directory exists and is writable */
-export function ensureDataDirectory(dataDir: string): { success: boolean; error?: string } {
-	try {
-		if (!existsSync(dataDir)) {
-			mkdirSync(dataDir, { recursive: true });
-		}
-
-		// Check if directory is writable
-		accessSync(dataDir, constants.W_OK);
-
-		// Try to create a test file to verify write access
-		const testFile = resolve(dataDir, ".write-test");
-		writeFileSync(testFile, "test");
-
-		return { success: true };
-	} catch (err: any) {
-		return { success: false, error: err.message };
-	}
-}
-
-/** Run drizzle-kit migrations on the database */
-export async function runDrizzleMigrations(
-	database: ReturnType<typeof drizzle>
-): Promise<{ success: boolean; error?: string; warning?: string }> {
-	try {
-		await migrate(database, { migrationsFolder });
-		return { success: true };
-	} catch (err: any) {
-		// If the error is "duplicate column", it means the schema is already up-to-date
-		// This happens when ALTER migrations in client.ts have already added the columns
-		// We consider this a success with a warning, not a failure
-		if (err.message?.includes("duplicate column")) {
-			return { success: true, warning: `Schema already up-to-date: ${err.message}` };
-		}
-		return { success: false, error: err.message };
-	}
-}
-
-/** Run ALTER TABLE migrations for backward compatibility with older databases */
-export async function runAlterMigrations(client: Client): Promise<{ success: boolean; errors: string[] }> {
-	const errors: string[] = [];
-
-	// These add new columns to existing tables (silently fail if column already exists)
-	const alterMigrations = [
-		// Added in v1.x - repeat reminders and nagging settings
-		`ALTER TABLE user_settings ADD COLUMN skip_reminders_for_taken_doses integer NOT NULL DEFAULT 0`,
-		`ALTER TABLE user_settings ADD COLUMN repeat_reminders_enabled integer NOT NULL DEFAULT 0`,
-		`ALTER TABLE user_settings ADD COLUMN reminder_repeat_interval_minutes integer NOT NULL DEFAULT 30`,
-		`ALTER TABLE user_settings ADD COLUMN max_nagging_reminders integer NOT NULL DEFAULT 5`,
-		// Added in v1.2.3 - dismiss missed doses without deducting stock
-		`ALTER TABLE dose_tracking ADD COLUMN dismissed integer NOT NULL DEFAULT 0`,
-		// Added in v1.3.x - stock calculation mode (automatic/manual)
-		`ALTER TABLE user_settings ADD COLUMN stock_calculation_mode text NOT NULL DEFAULT 'automatic'`,
-		// Added for stock correction - hidden offset that doesn't affect looseTablets
-		`ALTER TABLE medications ADD COLUMN stock_adjustment integer NOT NULL DEFAULT 0`,
-		// Added for stock correction - timestamp to ignore consumed doses before correction
-		`ALTER TABLE medications ADD COLUMN last_stock_correction_at integer`,
-		// Added in v1.5.1 - dismiss past doses until date (robust against timestamp changes)
-		`ALTER TABLE medications ADD COLUMN dismissed_until text`,
-		// Added for more detailed reminder info display
-		`ALTER TABLE user_settings ADD COLUMN last_reminder_med_name text`,
-		`ALTER TABLE user_settings ADD COLUMN last_reminder_taken_by text`,
-		// Added for package type support (blister vs bottle)
-		`ALTER TABLE medications ADD COLUMN package_type text NOT NULL DEFAULT 'blister'`,
-		`ALTER TABLE medications ADD COLUMN total_pills integer`,
-		// Added for dose unit selection (mg, g, mcg, ml, IU, etc.)
-		`ALTER TABLE medications ADD COLUMN dose_unit text DEFAULT 'mg'`,
-		// Added for intake-level takenBy: unified intakes structure
-		`ALTER TABLE medications ADD COLUMN intakes_json text NOT NULL DEFAULT '[]'`,
-	];
-
-	for (const sql of alterMigrations) {
-		try {
-			await client.execute(sql);
-		} catch (e: any) {
-			// Silently ignore "duplicate column" errors - column already exists
-			if (!e.message?.includes("duplicate column")) {
-				errors.push(e.message);
-			}
-		}
-	}
-
-	// Create tables that might be missing (silently fail if already exists)
-	const createTableMigrations = [
-		// Added in v1.3.x - refill history tracking
-		`CREATE TABLE IF NOT EXISTS refill_history (
-      id INTEGER PRIMARY KEY AUTOINCREMENT,
-      medication_id INTEGER NOT NULL REFERENCES medications(id) ON DELETE CASCADE,
-      user_id INTEGER NOT NULL REFERENCES users(id) ON DELETE CASCADE,
-      packs_added INTEGER NOT NULL DEFAULT 0,
-      loose_pills_added INTEGER NOT NULL DEFAULT 0,
-      refill_date INTEGER NOT NULL DEFAULT (strftime('%s','now'))
-    )`,
-	];
-
-	for (const sql of createTableMigrations) {
-		try {
-			await client.execute(sql);
-		} catch (e: any) {
-			// Silently ignore "table already exists" errors
-			if (!e.message?.includes("already exists")) {
-				errors.push(e.message);
-			}
-		}
-	}
-
-	return { success: errors.length === 0, errors };
-}
-
-/** Ensure default user exists for auth-disabled mode */
-export async function ensureDefaultUser(client: Client, authEnabled: boolean): Promise<boolean> {
-	if (authEnabled) {
-		return false; // No default user needed
-	}
-
-	try {
-		const result = await client.execute("SELECT id FROM users WHERE id = 1");
-		if (result.rows.length === 0) {
-			await client.execute("INSERT INTO users (id, username, auth_provider) VALUES (1, 'default', 'local')");
-			return true; // Created
-		}
-		return false; // Already exists
-	} catch (e: any) {
-		console.error(`[DB] Error creating default user:`, e.message);
-		return false;
-	}
-}
+// Load .env: try cwd first, then parent dir (for local dev running from backend/)
+const envPath = process.env.DOTENV_PATH || (existsSync(".env") ? ".env" : "../.env");
+dotenv.config({ path: envPath });

 // =============================================================================
 // Database initialization (runs on import)
@@ -201,7 +76,7 @@ export const db = drizzle(client);
 // Auto-run migrations (self-healing database)
 async function runMigrations() {
 	// Run drizzle-kit generated migrations
-	console.log(`[DB] Running drizzle migrations from: ${migrationsFolder}`);
+	console.log(`[DB] Running drizzle migrations...`);
 	const migrateResult = await runDrizzleMigrations(db);
 	if (!migrateResult.success) {
 		console.error(`[DB] Migration error:`, migrateResult.error);
@@ -218,6 +93,24 @@ async function runMigrations() {
 	}
 	console.log(`[DB] Tables verified/created`);

+	// Repair dose IDs with trailing hyphens (from frontend takenBy bug)
+	const trailingResult = await repairTrailingHyphenDoseIds(client);
+	if (trailingResult.repaired > 0) {
+		console.log(`[DB] Repaired ${trailingResult.repaired} dose IDs with trailing hyphens`);
+	}
+	if (trailingResult.errors.length > 0) {
+		trailingResult.errors.forEach((err) => console.error(`[DB] Trailing-hyphen repair error:`, err));
+	}
+
+	// Repair orphaned dose tracking IDs from past schedule changes
+	const repairResult = await repairOrphanedDoseIds(client);
+	if (repairResult.repaired > 0) {
+		console.log(`[DB] Repaired ${repairResult.repaired} orphaned dose tracking IDs`);
+	}
+	if (repairResult.errors.length > 0) {
+		repairResult.errors.forEach((err) => console.error(`[DB] Dose repair error:`, err));
+	}
+
 	// If auth is disabled, ensure a default user exists (ID=1)
 	const authEnabled = process.env.AUTH_ENABLED === "true";
 	const created = await ensureDefaultUser(client, authEnabled);
@@ -0,0 +1,352 @@
+/**
+ * Pure utility functions for database operations.
+ * Separated from client.ts to allow importing without triggering
+ * top-level database initialization side effects.
+ */
+
+import { accessSync, constants, existsSync, mkdirSync, writeFileSync } from "node:fs";
+import { dirname, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
+import type { Client } from "@libsql/client";
+import type { drizzle } from "drizzle-orm/libsql";
+import { migrate } from "drizzle-orm/libsql/migrator";
+import { parseIntakesJson, parseLocalDateTime } from "../utils/scheduler-utils.js";
+
+// Get migrations folder path (relative to this file's location)
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const migrationsFolder = resolve(__dirname, "../../drizzle");
+
+// =============================================================================
+// Path & Directory utilities
+// =============================================================================
+
+/**
+ * Get the data directory path.
+ *
+ * Resolution order:
+ * 1. DATA_DIR env var (set by docker-compose for containers)
+ * 2. Monorepo detection: if ../docker-compose.yml exists, we're in backend/
+ *    subdirectory → use ../data (project root's data folder)
+ * 3. Fallback: resolve(cwd, "data") (running from project root or standalone)
+ */
+export function getDataDir(cwd: string = process.cwd()): string {
+	// Docker containers set DATA_DIR explicitly
+	if (process.env.DATA_DIR) return resolve(process.env.DATA_DIR);
+
+	// Local dev: detect if we're in backend/ subdirectory of the monorepo
+	if (existsSync(resolve(cwd, "..", "docker-compose.yml"))) {
+		return resolve(cwd, "..", "data");
+	}
+
+	// Default: data/ relative to cwd (running from project root)
+	return resolve(cwd, "data");
+}
+
+/** Build the database URL from a path */
+export function buildDbUrl(dbPath: string): string {
+	return `file:${dbPath}`;
+}
+
+/** Get data directory and database path */
+export function getDbPaths(cwd: string = process.cwd()): { dataDir: string; dbPath: string; url: string } {
+	const dataDir = getDataDir(cwd);
+	const dbPath = resolve(dataDir, "medassist-ng.db");
+	const url = buildDbUrl(dbPath);
+	return { dataDir, dbPath, url };
+}
+
+/** Ensure data directory exists and is writable */
+export function ensureDataDirectory(dataDir: string): { success: boolean; error?: string } {
+	try {
+		if (!existsSync(dataDir)) {
+			mkdirSync(dataDir, { recursive: true });
+		}
+
+		// Check if directory is writable
+		accessSync(dataDir, constants.W_OK);
+
+		// Try to create a test file to verify write access
+		const testFile = resolve(dataDir, ".write-test");
+		writeFileSync(testFile, "test");
+
+		return { success: true };
+	} catch (err: any) {
+		return { success: false, error: err.message };
+	}
+}
+
+// =============================================================================
+// Migration utilities
+// =============================================================================
+
+/** Run drizzle-kit migrations on the database */
+export async function runDrizzleMigrations(
+	database: ReturnType<typeof drizzle>
+): Promise<{ success: boolean; error?: string; warning?: string }> {
+	try {
+		await migrate(database, { migrationsFolder });
+		return { success: true };
+	} catch (err: any) {
+		// If the error is "duplicate column", it means the schema is already up-to-date
+		// This happens when ALTER migrations in client.ts have already added the columns
+		// We consider this a success with a warning, not a failure
+		if (err.message?.includes("duplicate column")) {
+			return { success: true, warning: `Schema already up-to-date: ${err.message}` };
+		}
+		return { success: false, error: err.message };
+	}
+}
+
+/** Run ALTER TABLE migrations for backward compatibility with older databases */
+export async function runAlterMigrations(client: Client): Promise<{ success: boolean; errors: string[] }> {
+	const errors: string[] = [];
+
+	// These add new columns to existing tables (silently fail if column already exists)
+	const alterMigrations = [
+		// Added in v1.x - repeat reminders and nagging settings
+		`ALTER TABLE user_settings ADD COLUMN skip_reminders_for_taken_doses integer NOT NULL DEFAULT 0`,
+		`ALTER TABLE user_settings ADD COLUMN repeat_reminders_enabled integer NOT NULL DEFAULT 0`,
+		`ALTER TABLE user_settings ADD COLUMN reminder_repeat_interval_minutes integer NOT NULL DEFAULT 30`,
+		`ALTER TABLE user_settings ADD COLUMN max_nagging_reminders integer NOT NULL DEFAULT 5`,
+		// Added in v1.2.3 - dismiss missed doses without deducting stock
+		`ALTER TABLE dose_tracking ADD COLUMN dismissed integer NOT NULL DEFAULT 0`,
+		// Added in v1.3.x - stock calculation mode (automatic/manual)
+		`ALTER TABLE user_settings ADD COLUMN stock_calculation_mode text NOT NULL DEFAULT 'automatic'`,
+		// Added for stock correction - hidden offset that doesn't affect looseTablets
+		`ALTER TABLE medications ADD COLUMN stock_adjustment integer NOT NULL DEFAULT 0`,
+		// Added for stock correction - timestamp to ignore consumed doses before correction
+		`ALTER TABLE medications ADD COLUMN last_stock_correction_at integer`,
+		// Added in v1.5.1 - dismiss past doses until date (robust against timestamp changes)
+		`ALTER TABLE medications ADD COLUMN dismissed_until text`,
+		// Added for more detailed reminder info display
+		`ALTER TABLE user_settings ADD COLUMN last_reminder_med_name text`,
+		`ALTER TABLE user_settings ADD COLUMN last_reminder_taken_by text`,
+		// Added for package type support (blister vs bottle)
+		`ALTER TABLE medications ADD COLUMN package_type text NOT NULL DEFAULT 'blister'`,
+		`ALTER TABLE medications ADD COLUMN total_pills integer`,
+		// Added for dose unit selection (mg, g, mcg, ml, IU, etc.)
+		`ALTER TABLE medications ADD COLUMN dose_unit text DEFAULT 'mg'`,
+		// Added for intake-level takenBy: unified intakes structure
+		`ALTER TABLE medications ADD COLUMN intakes_json text NOT NULL DEFAULT '[]'`,
+	];
+
+	for (const sql of alterMigrations) {
+		try {
+			await client.execute(sql);
+		} catch (e: any) {
+			// Silently ignore "duplicate column" errors - column already exists
+			if (!e.message?.includes("duplicate column")) {
+				errors.push(e.message);
+			}
+		}
+	}
+
+	// Create tables that might be missing (silently fail if already exists)
+	const createTableMigrations = [
+		// Added in v1.3.x - refill history tracking
+		`CREATE TABLE IF NOT EXISTS refill_history (
+      id INTEGER PRIMARY KEY AUTOINCREMENT,
+      medication_id INTEGER NOT NULL REFERENCES medications(id) ON DELETE CASCADE,
+      user_id INTEGER NOT NULL REFERENCES users(id) ON DELETE CASCADE,
+      packs_added INTEGER NOT NULL DEFAULT 0,
+      loose_pills_added INTEGER NOT NULL DEFAULT 0,
+      refill_date INTEGER NOT NULL DEFAULT (strftime('%s','now'))
+    )`,
+	];
+
+	for (const sql of createTableMigrations) {
+		try {
+			await client.execute(sql);
+		} catch (e: any) {
+			// Silently ignore "table already exists" errors
+			if (!e.message?.includes("already exists")) {
+				errors.push(e.message);
+			}
+		}
+	}
+
+	return { success: errors.length === 0, errors };
+}
+
+// =============================================================================
+// User utilities
+// =============================================================================
+
+/** Ensure default user exists for auth-disabled mode */
+export async function ensureDefaultUser(client: Client, authEnabled: boolean): Promise<boolean> {
+	if (authEnabled) {
+		return false; // No default user needed
+	}
+
+	try {
+		const result = await client.execute("SELECT id FROM users WHERE id = 1");
+		if (result.rows.length === 0) {
+			await client.execute("INSERT INTO users (id, username, auth_provider) VALUES (1, 'default', 'local')");
+			return true; // Created
+		}
+		return false; // Already exists
+	} catch (e: any) {
+		console.error(`[DB] Error creating default user:`, e.message);
+		return false;
+	}
+}
+
+// =============================================================================
+// Startup repair: fix orphaned dose tracking IDs from past schedule changes
+// =============================================================================
+
+const MS_PER_DAY = 86_400_000;
+
+/**
+ * Repair dose IDs that have a trailing hyphen caused by a frontend bug where
+ * `[].toString()` produced an empty string, resulting in IDs like "5-0-1729123200000-"
+ * instead of "5-0-1729123200000". This strips trailing hyphens from all dose IDs.
+ *
+ * This function is idempotent - safe to run on every startup.
+ */
+export async function repairTrailingHyphenDoseIds(client: Client): Promise<{ repaired: number; errors: string[] }> {
+	const errors: string[] = [];
+	let repaired = 0;
+
+	try {
+		const result = await client.execute(
+			"UPDATE dose_tracking SET dose_id = RTRIM(dose_id, '-') WHERE dose_id LIKE '%-'"
+		);
+		repaired = result.rowsAffected;
+	} catch (e: any) {
+		errors.push(`Trailing-hyphen repair failed: ${e.message}`);
+	}
+
+	return { repaired, errors };
+}
+
+/**
+ * Repair orphaned dose tracking IDs that no longer match the current intake schedule.
+ * This fixes dose IDs that became invalid when a medication's schedule was changed
+ * BEFORE the on-edit migration (PR #103) was introduced.
+ *
+ * For each medication, generates all valid schedule dateOnlyMs values from each intake's
+ * start date up to today, then checks all dose_tracking entries. Any dose whose timestamp
+ * doesn't match a valid schedule date is remapped to the nearest valid date.
+ *
+ * This function is idempotent - safe to run on every startup.
+ */
+export async function repairOrphanedDoseIds(client: Client): Promise<{ repaired: number; errors: string[] }> {
+	const errors: string[] = [];
+	let repaired = 0;
+
+	try {
+		// Get all medications
+		const medsResult = await client.execute(
+			"SELECT id, intakes_json, usage_json, every_json, start_json, intake_reminders_enabled FROM medications"
+		);
+
+		if (medsResult.rows.length === 0) return { repaired, errors };
+
+		// Get all dose tracking entries
+		const dosesResult = await client.execute("SELECT id, dose_id FROM dose_tracking");
+		if (dosesResult.rows.length === 0) return { repaired, errors };
+
+		// Build a map of medId → dose entries for quick lookup
+		const dosesByMed = new Map<number, Array<{ id: number; doseId: string }>>();
+		for (const row of dosesResult.rows) {
+			const doseId = row.dose_id as string;
+			const parts = doseId.split("-");
+			if (parts.length < 3) continue;
+			const medId = parseInt(parts[0], 10);
+			if (Number.isNaN(medId)) continue;
+			if (!dosesByMed.has(medId)) dosesByMed.set(medId, []);
+			dosesByMed.get(medId)!.push({ id: row.id as number, doseId });
+		}
+
+		const now = new Date();
+		const today = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+
+		for (const med of medsResult.rows) {
+			const medId = med.id as number;
+			const medDoses = dosesByMed.get(medId);
+			if (!medDoses || medDoses.length === 0) continue;
+
+			// Parse intakes
+			const intakes = parseIntakesJson(
+				med.intakes_json as string | null,
+				{
+					usageJson: (med.usage_json as string) || "[]",
+					everyJson: (med.every_json as string) || "[]",
+					startJson: (med.start_json as string) || "[]",
+				},
+				(med.intake_reminders_enabled as number) === 1
+			);
+
+			if (intakes.length === 0) continue;
+
+			// For each intake index, build the set of valid dateOnlyMs values
+			const validDatesByIntake = new Map<number, Set<number>>();
+			for (let idx = 0; idx < intakes.length; idx++) {
+				const intake = intakes[idx];
+				const start = parseLocalDateTime(intake.start);
+				const every = intake.every;
+				if (every <= 0 || Number.isNaN(start.getTime())) continue;
+
+				const validDates = new Set<number>();
+				for (let d = new Date(start); d <= today; d.setDate(d.getDate() + every)) {
+					validDates.add(new Date(d.getFullYear(), d.getMonth(), d.getDate()).getTime());
+				}
+				validDatesByIntake.set(idx, validDates);
+			}
+
+			// Check each dose entry
+			for (const dose of medDoses) {
+				const parts = dose.doseId.split("-");
+				if (parts.length < 3) continue;
+
+				const intakeIdx = parseInt(parts[1], 10);
+				const dateOnlyMs = parseInt(parts[2], 10);
+				if (Number.isNaN(intakeIdx) || Number.isNaN(dateOnlyMs)) continue;
+
+				const validDates = validDatesByIntake.get(intakeIdx);
+				if (!validDates) continue; // Unknown intake index - skip
+
+				// Check if this dose's timestamp is valid
+				if (validDates.has(dateOnlyMs)) continue; // Already valid - nothing to do
+
+				// Orphaned dose - find the nearest valid schedule date
+				const intake = intakes[intakeIdx];
+				if (!intake) continue;
+
+				const halfInterval = (intake.every * MS_PER_DAY) / 2;
+				let bestMatch: number | null = null;
+				let bestDist = Infinity;
+
+				for (const validDate of validDates) {
+					const dist = Math.abs(validDate - dateOnlyMs);
+					if (dist < bestDist && dist <= halfInterval) {
+						bestDist = dist;
+						bestMatch = validDate;
+					}
+				}
+
+				if (bestMatch !== null) {
+					// Rebuild dose ID with new timestamp, preserving person suffix
+					const personSuffix = parts.length > 3 ? `-${parts.slice(3).join("-")}` : "";
+					const newDoseId = `${medId}-${intakeIdx}-${bestMatch}${personSuffix}`;
+
+					try {
+						await client.execute({
+							sql: "UPDATE dose_tracking SET dose_id = ? WHERE id = ?",
+							args: [newDoseId, dose.id],
+						});
+						repaired++;
+					} catch (e: any) {
+						errors.push(`Failed to repair dose ${dose.id}: ${e.message}`);
+					}
+				}
+			}
+		}
+	} catch (e: any) {
+		errors.push(`Repair failed: ${e.message}`);
+	}
+
+	return { repaired, errors };
+}
@@ -1,3 +1,4 @@
+import { existsSync } from "node:fs";
 import { dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import { type Client, createClient } from "@libsql/client";
@@ -5,7 +6,9 @@ import dotenv from "dotenv";
 import { drizzle } from "drizzle-orm/libsql";
 import { migrate } from "drizzle-orm/libsql/migrator";

-dotenv.config({ path: process.env.DOTENV_PATH || ".env" });
+// Load .env: try cwd first, then parent dir (for local dev running from backend/)
+const envPath = process.env.DOTENV_PATH || (existsSync(".env") ? ".env" : "../.env");
+dotenv.config({ path: envPath });

 // Get migrations folder path (relative to this file's location)
 const __filename = fileURLToPath(import.meta.url);
@@ -10,6 +10,7 @@ import sensible from "@fastify/sensible";
 import fastifyStatic from "@fastify/static";
 import Fastify, { type FastifyInstance } from "fastify";
 import { migrationsReady } from "./db/client.js";
+import { getDataDir } from "./db/db-utils.js";
 import { env } from "./plugins/env.js";
 import { authRoutes } from "./routes/auth.js";
 import { doseRoutes } from "./routes/doses.js";
@@ -66,7 +67,7 @@ export async function createApp(options?: {
 		accessTtlMinutes: options?.accessTtlMinutes ?? 15,
 		refreshTtlDays: options?.refreshTtlDays ?? 7,
 		isProduction: options?.isProduction ?? false,
-		imagesDir: options?.imagesDir ?? resolve(process.cwd(), "data/images"),
+		imagesDir: options?.imagesDir ?? resolve(getDataDir(), "images"),
 	};

 	const app = Fastify({
@@ -1,7 +1,10 @@
+import { existsSync } from "node:fs";
 import dotenv from "dotenv";
 import { z } from "zod";

-dotenv.config({ path: process.env.DOTENV_PATH || ".env" });
+// Load .env: try cwd first, then parent dir (for local dev running from backend/)
+const envPath = process.env.DOTENV_PATH || (existsSync(".env") ? ".env" : "../.env");
+dotenv.config({ path: envPath });

 const EnvSchema = z.object({
 	NODE_ENV: z.enum(["development", "production", "test"]).default("production"),
@@ -4,6 +4,7 @@ import { eq } from "drizzle-orm";
 import type { FastifyInstance } from "fastify";
 import { z } from "zod";
 import { db } from "../db/client.js";
+import { getDataDir } from "../db/db-utils.js";
 import { refreshTokens, users } from "../db/schema.js";
 import { getAuthState, requireAuth } from "../plugins/auth.js";
 import type { AuthUser } from "../types/fastify.js";
@@ -476,7 +477,7 @@ export async function authRoutes(app: FastifyInstance) {
 			// Save file
 			const fs = await import("node:fs/promises");
 			const path = await import("node:path");
-			const imagesDir = path.join(process.cwd(), "data", "images");
+			const imagesDir = path.join(getDataDir(), "images");
 			await fs.mkdir(imagesDir, { recursive: true });

 			const buffer = await data.toBuffer();
@@ -523,7 +524,7 @@ export async function authRoutes(app: FastifyInstance) {
 			const fs = await import("node:fs/promises");
 			const path = await import("node:path");
 			try {
-				await fs.unlink(path.join(process.cwd(), "data", "images", user.avatarUrl));
+				await fs.unlink(path.join(getDataDir(), "images", user.avatarUrl));
 			} catch {
 				// Ignore if file doesn't exist
 			}
@@ -556,7 +557,7 @@ export async function authRoutes(app: FastifyInstance) {
 				const fs = await import("node:fs/promises");
 				const path = await import("node:path");
 				try {
-					await fs.unlink(path.join(process.cwd(), "data", "images", user.avatarUrl));
+					await fs.unlink(path.join(getDataDir(), "images", user.avatarUrl));
 				} catch {
 					// Ignore if file doesn't exist
 				}
@@ -5,13 +5,14 @@ import { eq } from "drizzle-orm";
 import type { FastifyInstance } from "fastify";
 import { z } from "zod";
 import { db } from "../db/client.js";
+import { getDataDir } from "../db/db-utils.js";
 import { doseTracking, medications, shareTokens, userSettings } from "../db/schema.js";
 import { getAnonymousUserId, requireAuth } from "../plugins/auth.js";
 import { env } from "../plugins/env.js";
 import type { AuthUser } from "../types/fastify.js";
 import { parseIntakesJson, parseTakenByJson } from "../utils/scheduler-utils.js";

-const IMAGES_DIR = resolve(process.cwd(), "data/images");
+const IMAGES_DIR = resolve(getDataDir(), "images");

 // =============================================================================
 // Export Format Version (bump this when format changes)
@@ -5,13 +5,14 @@ import { and, eq, like } from "drizzle-orm";
 import type { FastifyInstance, FastifyReply, FastifyRequest } from "fastify";
 import { z } from "zod";
 import { db } from "../db/client.js";
+import { getDataDir } from "../db/db-utils.js";
 import { doseTracking, medications } from "../db/schema.js";
 import { getAnonymousUserId, requireAuth } from "../plugins/auth.js";
 import { env } from "../plugins/env.js";
 import type { AuthUser } from "../types/fastify.js";
 import { type Intake, parseIntakesJson, parseLocalDateTime, parseTakenByJson } from "../utils/scheduler-utils.js";

-const IMAGES_DIR = resolve(process.cwd(), "data/images");
+const IMAGES_DIR = resolve(getDataDir(), "images");

 // New intake schema with per-intake takenBy
 const intakeSchema = z.object({
@@ -284,6 +285,17 @@ export async function medicationRoutes(app: FastifyInstance) {
 		const startJson = JSON.stringify(intakes.map((s) => s.start));
 		const takenByJson = JSON.stringify(takenBy || []);

+		// If stock-defining fields changed, reset stockAdjustment so the new
+		// base stock reflects actual inventory.  This prevents the old
+		// correction offset from skewing the total after an edit.
+		const stockFieldsChanged =
+			existing.packCount !== packCount ||
+			existing.blistersPerPack !== blistersPerPack ||
+			existing.pillsPerBlister !== pillsPerBlister ||
+			(existing.looseTablets ?? 0) !== (looseTablets ?? 0);
+
+		const stockResetFields = stockFieldsChanged ? { stockAdjustment: 0, lastStockCorrectionAt: new Date() } : {};
+
 		const result = await db
 			.update(medications)
 			.set({
@@ -306,35 +318,129 @@ export async function medicationRoutes(app: FastifyInstance) {
 				everyJson,
 				startJson,
 				updatedAt: new Date(),
+				...stockResetFields,
 			})
 			.where(and(eq(medications.id, idNum), eq(medications.userId, userId)))
 			.returning();

 		if (!result.length) return reply.notFound();

-		// Clean up dose tracking entries that are before the earliest start date
-		// This ensures consistency when the user changes the start date
-		const earliestStart = Math.min(...intakes.map((b) => parseLocalDateTime(b.start).getTime()));
-		if (!Number.isNaN(earliestStart)) {
-			// Get all dose tracking entries for this medication and filter out invalid ones
-			const allDoses = await db
-				.select()
-				.from(doseTracking)
-				.where(and(eq(doseTracking.userId, userId), like(doseTracking.doseId, `${idNum}-%`)));
+		// ---------------------------------------------------------------
+		// Migrate dose tracking IDs when intake schedule changes
+		// ---------------------------------------------------------------
+		// Parse old intakes from the existing medication row
+		const oldIntakes = parseIntakesJson(
+			existing.intakesJson,
+			{ usageJson: existing.usageJson, everyJson: existing.everyJson, startJson: existing.startJson },
+			existing.intakeRemindersEnabled
+		);

-			// Find doses with timestamps before the earliest start date
-			const dosesToDelete = allDoses.filter((dose) => {
-				const parts = dose.doseId.split("-");
-				if (parts.length >= 3) {
-					const timestamp = parseInt(parts[2], 10);
-					return !Number.isNaN(timestamp) && timestamp < earliestStart;
+		// Get all dose tracking entries for this medication
+		const allDoses = await db
+			.select()
+			.from(doseTracking)
+			.where(and(eq(doseTracking.userId, userId), like(doseTracking.doseId, `${idNum}-%`)));
+
+		if (allDoses.length > 0) {
+			// Build migration map: for each intake index, map old dateOnlyMs → new dateOnlyMs
+			const now = new Date();
+			const migrationEnd = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+			const MS_PER_DAY = 86_400_000;
+
+			for (let idx = 0; idx < Math.max(oldIntakes.length, intakes.length); idx++) {
+				const oldIntake = oldIntakes[idx];
+				const newIntake = intakes[idx];
+
+				// Skip if this intake index doesn't exist in both old and new
+				if (!oldIntake || !newIntake) continue;
+
+				const oldStart = parseLocalDateTime(oldIntake.start);
+				const newStart = parseLocalDateTime(newIntake.start);
+				const oldEvery = oldIntake.every;
+				const newEvery = newIntake.every;
+
+				// Check if start date or interval changed (time-of-day changes don't matter for dateOnlyMs)
+				const oldStartDateOnly = new Date(oldStart.getFullYear(), oldStart.getMonth(), oldStart.getDate()).getTime();
+				const newStartDateOnly = new Date(newStart.getFullYear(), newStart.getMonth(), newStart.getDate()).getTime();
+
+				if (oldStartDateOnly === newStartDateOnly && oldEvery === newEvery) {
+					continue; // No schedule change that affects dose IDs
 				}
-				return false;
-			});

-			// Delete invalid doses
-			for (const dose of dosesToDelete) {
-				await db.delete(doseTracking).where(eq(doseTracking.id, dose.id));
+				// Build set of new valid dateOnlyMs values for this intake
+				const newDates = new Set<number>();
+				for (let d = new Date(newStart); d <= migrationEnd; d.setDate(d.getDate() + newEvery)) {
+					newDates.add(new Date(d.getFullYear(), d.getMonth(), d.getDate()).getTime());
+				}
+
+				// Build set of old dateOnlyMs values with mapping to nearest new date
+				const oldToNewMap = new Map<number, number>();
+				for (let d = new Date(oldStart); d <= migrationEnd; d.setDate(d.getDate() + oldEvery)) {
+					const oldDateMs = new Date(d.getFullYear(), d.getMonth(), d.getDate()).getTime();
+					// Find the closest new date within ±(newEvery/2) days
+					const halfInterval = (newEvery * MS_PER_DAY) / 2;
+					let bestMatch: number | null = null;
+					let bestDist = Infinity;
+					for (const newDateMs of newDates) {
+						const dist = Math.abs(newDateMs - oldDateMs);
+						if (dist < bestDist && dist <= halfInterval) {
+							bestDist = dist;
+							bestMatch = newDateMs;
+						}
+					}
+					if (bestMatch !== null && bestMatch !== oldDateMs) {
+						oldToNewMap.set(oldDateMs, bestMatch);
+						// Remove matched new date to prevent double-mapping
+						newDates.delete(bestMatch);
+					}
+				}
+
+				// Apply migrations to dose tracking entries
+				if (oldToNewMap.size > 0) {
+					const prefix = `${idNum}-${idx}-`;
+					const dosesToMigrate = allDoses.filter((d) => d.doseId.startsWith(prefix));
+
+					for (const dose of dosesToMigrate) {
+						const parts = dose.doseId.split("-");
+						if (parts.length >= 3) {
+							const oldTimestamp = parseInt(parts[2], 10);
+							const newTimestamp = oldToNewMap.get(oldTimestamp);
+							if (newTimestamp !== undefined) {
+								// Replace the timestamp in the dose ID, keeping any person suffix
+								const newDoseId = `${idNum}-${idx}-${newTimestamp}${parts.length > 3 ? `-${parts.slice(3).join("-")}` : ""}`;
+								await db.update(doseTracking).set({ doseId: newDoseId }).where(eq(doseTracking.id, dose.id));
+							}
+						}
+					}
+				}
+			}
+
+			// Also clean up dose tracking entries before the earliest new start date
+			const earliestStartDate = intakes.reduce((min, b) => {
+				const d = parseLocalDateTime(b.start);
+				// Use date-only (midnight) to match dose ID format
+				const dateOnly = new Date(d.getFullYear(), d.getMonth(), d.getDate()).getTime();
+				return dateOnly < min ? dateOnly : min;
+			}, Infinity);
+			if (!Number.isNaN(earliestStartDate)) {
+				// Re-fetch after possible migrations
+				const updatedDoses = await db
+					.select()
+					.from(doseTracking)
+					.where(and(eq(doseTracking.userId, userId), like(doseTracking.doseId, `${idNum}-%`)));
+
+				const dosesToDelete = updatedDoses.filter((dose) => {
+					const parts = dose.doseId.split("-");
+					if (parts.length >= 3) {
+						const timestamp = parseInt(parts[2], 10);
+						return !Number.isNaN(timestamp) && timestamp < earliestStartDate;
+					}
+					return false;
+				});
+
+				for (const dose of dosesToDelete) {
+					await db.delete(doseTracking).where(eq(doseTracking.id, dose.id));
+				}
 			}
 		}

@@ -575,10 +681,19 @@ export async function medicationRoutes(app: FastifyInstance) {
 				const blisterStart = parseLocalDateTime(blister.start);
 				if (Number.isNaN(blisterStart.getTime())) return;

-				const effectiveStart = Math.max(blisterStart.getTime(), stockCorrectionCutoff);
+				const period = Math.max(1, blister.every) * msPerDay;
+
+				// After a stock correction, start counting from the NEXT scheduled
+				// dose, because the user's pill count already reflects all
+				// consumption up to the correction time.
+				let effectiveStart: number;
+				if (stockCorrectionCutoff > 0 && stockCorrectionCutoff >= blisterStart.getTime()) {
+					effectiveStart = stockCorrectionCutoff + period;
+				} else {
+					effectiveStart = blisterStart.getTime();
+				}
 				if (effectiveStart > now.getTime()) return;

-				const period = Math.max(1, blister.every) * msPerDay;
 				const occurrences = Math.floor((now.getTime() - effectiveStart) / period) + 1;

 				// Get the people for this intake (from intakes array or medication takenBy)
@@ -3,6 +3,7 @@ import { resolve } from "node:path";
 import { and, eq, gte, lte } from "drizzle-orm";
 import nodemailer from "nodemailer";
 import { db } from "../db/client.js";
+import { getDataDir } from "../db/db-utils.js";
 import { doseTracking, medications } from "../db/schema.js";
 import { getDateLocale, getTranslations, type Language, t } from "../i18n/translations.js";
 import { getAllUserSettings, sendShoutrrrNotification, type UserSettings } from "../routes/settings.js";
@@ -25,7 +26,7 @@ import { updateReminderSentTime, updateUserReminderSentTime } from "./reminder-s
 const REMINDER_MINUTES_BEFORE = parseInt(process.env.REMINDER_MINUTES_BEFORE ?? "15", 10);
 const CHECK_INTERVAL_MS = 60 * 1000; // Check every 1 minute

-const intakeReminderStateFile = resolve(process.cwd(), "data", "intake-reminder-state.json");
+const intakeReminderStateFile = resolve(getDataDir(), "intake-reminder-state.json");

 function loadIntakeReminderState(): IntakeReminderState {
 	try {
@@ -3,6 +3,7 @@ import { resolve } from "node:path";
 import { eq } from "drizzle-orm";
 import nodemailer from "nodemailer";
 import { db } from "../db/client.js";
+import { getDataDir } from "../db/db-utils.js";
 import { medications, userSettings } from "../db/schema.js";
 import { getTranslations, type Language, t } from "../i18n/translations.js";
 import { getAllUserSettings, sendShoutrrrNotification, type UserSettings } from "../routes/settings.js";
@@ -25,7 +26,7 @@ import {

 const REMINDER_HOUR = parseInt(process.env.REMINDER_HOUR ?? "6", 10); // Default 6:00 AM local time

-const reminderStateFile = resolve(process.cwd(), "data", "reminder-state.json");
+const reminderStateFile = resolve(getDataDir(), "reminder-state.json");

 function loadReminderState(): ReminderState {
 	try {
@@ -5,17 +5,20 @@ import { fileURLToPath } from "node:url";
 import { createClient } from "@libsql/client";
 import { drizzle } from "drizzle-orm/libsql";
 import { migrate } from "drizzle-orm/libsql/migrator";
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";

-// Import the exported utility functions from client.ts
+// Import utility functions from db-utils (no side effects, unlike client.ts which initializes the DB)
 import {
 	buildDbUrl,
 	ensureDataDirectory,
 	ensureDefaultUser,
+	getDataDir,
 	getDbPaths,
+	repairOrphanedDoseIds,
+	repairTrailingHyphenDoseIds,
 	runAlterMigrations,
 	runDrizzleMigrations,
-} from "../db/client.js";
+} from "../db/db-utils.js";

 // Import the exported utility functions from migrate.ts
 import { executeMigration, getStatementPreview, splitSQLStatements } from "../db/migrate.js";
@@ -142,15 +145,78 @@ describe("Database Client Utilities", () => {
 		});
 	});

+	describe("getDataDir", () => {
+		const originalDataDir = process.env.DATA_DIR;
+
+		afterEach(() => {
+			if (originalDataDir === undefined) {
+				delete process.env.DATA_DIR;
+			} else {
+				process.env.DATA_DIR = originalDataDir;
+			}
+		});
+
+		it("should use DATA_DIR env var when set (Docker)", () => {
+			process.env.DATA_DIR = "/app/data";
+			expect(getDataDir()).toBe("/app/data");
+		});
+
+		it("should resolve relative DATA_DIR to absolute", () => {
+			process.env.DATA_DIR = "../data";
+			const result = getDataDir();
+			expect(result).not.toContain("..");
+			expect(result).toMatch(/\/data$/);
+		});
+
+		it("should detect monorepo and use ../data when in backend/ subdir", () => {
+			delete process.env.DATA_DIR;
+			// Tests run from backend/ which has ../docker-compose.yml
+			const result = getDataDir();
+			// Should resolve to the project root's data/ folder, not backend/data/
+			expect(result).toMatch(/\/data$/);
+			expect(result).not.toMatch(/backend\/data$/);
+		});
+
+		it("should fall back to cwd/data when not in monorepo", () => {
+			delete process.env.DATA_DIR;
+			// Use a directory that has no ../docker-compose.yml
+			expect(getDataDir("/tmp")).toBe("/tmp/data");
+		});
+
+		it("should prefer DATA_DIR over monorepo detection", () => {
+			process.env.DATA_DIR = "/override/data";
+			expect(getDataDir("/app")).toBe("/override/data");
+		});
+	});
+
 	describe("getDbPaths", () => {
-		it("should return correct paths based on cwd", () => {
+		const originalDataDir = process.env.DATA_DIR;
+
+		afterEach(() => {
+			if (originalDataDir === undefined) {
+				delete process.env.DATA_DIR;
+			} else {
+				process.env.DATA_DIR = originalDataDir;
+			}
+		});
+
+		it("should return correct paths with DATA_DIR set", () => {
+			process.env.DATA_DIR = "/app/data";
 			const paths = getDbPaths("/app");
 			expect(paths.dataDir).toBe("/app/data");
 			expect(paths.dbPath).toBe("/app/data/medassist-ng.db");
 			expect(paths.url).toBe("file:/app/data/medassist-ng.db");
 		});

+		it("should return correct paths without DATA_DIR in non-monorepo dir", () => {
+			delete process.env.DATA_DIR;
+			const paths = getDbPaths("/tmp");
+			expect(paths.dataDir).toBe("/tmp/data");
+			expect(paths.dbPath).toBe("/tmp/data/medassist-ng.db");
+		});
+
 		it("should use process.cwd() by default", () => {
+			delete process.env.DATA_DIR;
 			const paths = getDbPaths();
 			expect(paths.dataDir).toContain("data");
 			expect(paths.dbPath).toContain("medassist-ng.db");
@@ -620,4 +686,373 @@ describe("Database Client", () => {
 			expect(users.rows).toHaveLength(1);
 		});
 	});
+
+	describe("repairOrphanedDoseIds", () => {
+		let client: ReturnType<typeof createClient>;
+
+		beforeEach(async () => {
+			client = createClient({ url: ":memory:" });
+			const db = drizzle(client);
+			await migrate(db, { migrationsFolder });
+			// Create a test user
+			await client.execute("INSERT INTO users (id, username, auth_provider) VALUES (1, 'testuser', 'local')");
+		});
+
+		it("should return 0 repairs when no data exists", async () => {
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(0);
+			expect(result.errors).toHaveLength(0);
+		});
+
+		it("should not modify dose IDs that already match the current schedule", async () => {
+			// Create weekly medication starting Oct 17 (Friday)
+			const intakes = JSON.stringify([
+				{ usage: 1, every: 7, start: "2025-10-17T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Weekly Med', ?, '[1]', '[7]', '["2025-10-17T08:00:00"]')`,
+				args: [intakes],
+			});
+
+			// Insert dose IDs that match the schedule (Fridays)
+			const fri17 = new Date(2025, 9, 17).getTime();
+			const fri24 = new Date(2025, 9, 24).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}`],
+			});
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri24}`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(0);
+
+			// Verify IDs unchanged
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking ORDER BY dose_id");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${fri17}`);
+			expect(doses.rows[1].dose_id).toBe(`1-0-${fri24}`);
+		});
+
+		it("should repair orphaned dose IDs when schedule shifted by 1 day", async () => {
+			// Current schedule: Saturdays (Oct 18)
+			const intakes = JSON.stringify([
+				{ usage: 1, every: 7, start: "2025-10-18T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Weekly Med', ?, '[1]', '[7]', '["2025-10-18T08:00:00"]')`,
+				args: [intakes],
+			});
+
+			// Insert orphaned dose IDs from OLD schedule (Fridays)
+			const fri17 = new Date(2025, 9, 17).getTime();
+			const fri24 = new Date(2025, 9, 24).getTime();
+			const fri31 = new Date(2025, 9, 31).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}`],
+			});
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri24}`],
+			});
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri31}`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(3);
+			expect(result.errors).toHaveLength(0);
+
+			// Verify dose IDs are now Saturdays
+			const sat18 = new Date(2025, 9, 18).getTime();
+			const sat25 = new Date(2025, 9, 25).getTime();
+			const nov1 = new Date(2025, 10, 1).getTime();
+
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking ORDER BY dose_id");
+			const ids = doses.rows.map((r) => r.dose_id);
+			expect(ids).toContain(`1-0-${sat18}`);
+			expect(ids).toContain(`1-0-${sat25}`);
+			expect(ids).toContain(`1-0-${nov1}`);
+		});
+
+		it("should preserve person suffix when repairing dose IDs", async () => {
+			// Current schedule: Saturdays
+			const intakes = JSON.stringify([
+				{ usage: 1, every: 7, start: "2025-10-18T08:00:00", takenBy: "Alice", intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Person Med', ?, '[1]', '[7]', '["2025-10-18T08:00:00"]')`,
+				args: [intakes],
+			});
+
+			// Orphaned dose with person suffix (from old Friday schedule)
+			const fri17 = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}-Alice`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(1);
+
+			// Verify person suffix preserved
+			const sat18 = new Date(2025, 9, 18).getTime();
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${sat18}-Alice`);
+		});
+
+		it("should not repair doses that are too far from any valid schedule date", async () => {
+			// Current schedule: biweekly (every 14 days) starting Oct 18
+			// halfInterval = 7 days, so doses more than 7 days from any valid date won't match
+			const intakes = JSON.stringify([
+				{ usage: 1, every: 14, start: "2025-10-18T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Biweekly Med', ?, '[1]', '[14]', '["2025-10-18T08:00:00"]')`,
+				args: [intakes],
+			});
+
+			// Insert dose on Oct 27 (9 days away from Oct 18, 4 days away from Nov 1)
+			// halfInterval = 7 days. Oct 27 is 9 days from Oct 18 (too far) and 4 days from Nov 1 (within range)
+			// Actually use Oct 26 which is 8 days from both (Oct 18 and Nov 1) - exactly at halfInterval + 1
+			// Wait: biweekly = Oct 18, Nov 1. Oct 26 is 8 days from Oct 18, 6 days from Nov 1 → 6 < 7, matches Nov 1
+			// Use Oct 25: 7 days from Oct 18, 7 days from Nov 1 → exactly at boundary. Use Oct 25 and check.
+			// The condition is dist <= halfInterval, so 7 <= 7 is true. Need dist > 7.
+			// Use a 28-day schedule instead: Oct 18, Nov 15. Midpoint is Nov 1-2. Nov 2 is 15 days from Oct 18, 13 from Nov 15. Both > 14. No match.
+			const intakes28 = JSON.stringify([
+				{ usage: 1, every: 28, start: "2025-10-18T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `UPDATE medications SET intakes_json = ?, every_json = '[28]' WHERE id = 1`,
+				args: [intakes28],
+			});
+
+			// Insert dose on Nov 2 (15 days from Oct 18, 13 days from Nov 15)
+			// halfInterval = 14 days. Both 15 > 14 and 13 < 14, so Nov 2 actually WOULD map to Nov 15.
+			// Use Nov 4: 17 days from Oct 18, 11 days from Nov 15 → 11 < 14, maps to Nov 15.
+			// For a 28-day interval, halfInterval = 14. A date must be > 14 days from ALL schedule dates.
+			// Between Oct 18 and Nov 15 (28 days), the only date > 14 from both is impossible.
+			// So lets use a gap: Oct 18 is the only past date for a monthly schedule.
+			// If we pick a date before Oct 18, like Oct 1 (17 days before Oct 18) → 17 > 14 → no match!
+			const oct1 = new Date(2025, 9, 1).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${oct1}`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(0);
+
+			// Dose should remain unchanged
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${oct1}`);
+		});
+
+		it("should be idempotent - running twice produces same result", async () => {
+			// Current schedule: Saturdays
+			const intakes = JSON.stringify([
+				{ usage: 1, every: 7, start: "2025-10-18T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Weekly Med', ?, '[1]', '[7]', '["2025-10-18T08:00:00"]')`,
+				args: [intakes],
+			});
+
+			// Insert orphaned dose from Friday
+			const fri17 = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}`],
+			});
+
+			// First run
+			const result1 = await repairOrphanedDoseIds(client);
+			expect(result1.repaired).toBe(1);
+
+			// Second run - should find 0 repairs (already fixed)
+			const result2 = await repairOrphanedDoseIds(client);
+			expect(result2.repaired).toBe(0);
+
+			// Verify final state
+			const sat18 = new Date(2025, 9, 18).getTime();
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows).toHaveLength(1);
+			expect(doses.rows[0].dose_id).toBe(`1-0-${sat18}`);
+		});
+
+		it("should handle multiple medications independently", async () => {
+			// Med 1: weekly Saturdays
+			const intakes1 = JSON.stringify([
+				{ usage: 1, every: 7, start: "2025-10-18T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Med 1', ?, '[1]', '[7]', '["2025-10-18T08:00:00"]')`,
+				args: [intakes1],
+			});
+
+			// Med 2: daily starting Oct 20 (valid IDs, no repair needed)
+			const intakes2 = JSON.stringify([
+				{ usage: 1, every: 1, start: "2025-10-20T08:00:00", takenBy: null, intakeRemindersEnabled: false },
+			]);
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (2, 1, 'Med 2', ?, '[1]', '[1]', '["2025-10-20T08:00:00"]')`,
+				args: [intakes2],
+			});
+
+			// Med 1: orphaned Friday dose
+			const fri17 = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}`],
+			});
+
+			// Med 2: valid daily dose
+			const oct20 = new Date(2025, 9, 20).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`2-0-${oct20}`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(1); // Only med 1 dose repaired
+
+			// Med 2 dose should be unchanged
+			const med2Doses = await client.execute("SELECT dose_id FROM dose_tracking WHERE dose_id LIKE '2-%'");
+			expect(med2Doses.rows[0].dose_id).toBe(`2-0-${oct20}`);
+		});
+
+		it("should handle legacy format (no intakes_json, uses usage/every/start arrays)", async () => {
+			// Medication with only legacy fields (intakes_json is '[]')
+			await client.execute({
+				sql: `INSERT INTO medications (id, user_id, name, intakes_json, usage_json, every_json, start_json)
+				      VALUES (1, 1, 'Legacy Med', '[]', '[1]', '[7]', '["2025-10-18T08:00:00"]')`,
+				args: [],
+			});
+
+			// Orphaned Friday dose
+			const fri17 = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${fri17}`],
+			});
+
+			const result = await repairOrphanedDoseIds(client);
+			expect(result.repaired).toBe(1);
+
+			// Verify mapped to Saturday
+			const sat18 = new Date(2025, 9, 18).getTime();
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${sat18}`);
+		});
+	});
+
+	describe("repairTrailingHyphenDoseIds", () => {
+		let client: ReturnType<typeof createClient>;
+
+		beforeEach(async () => {
+			client = createClient({ url: ":memory:" });
+			const db = drizzle(client);
+			await migrate(db, { migrationsFolder });
+			await client.execute("INSERT INTO users (id, username, auth_provider) VALUES (1, 'testuser', 'local')");
+		});
+
+		it("should return 0 repairs when no dose IDs have trailing hyphens", async () => {
+			const ts = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${ts}`],
+			});
+
+			const result = await repairTrailingHyphenDoseIds(client);
+			expect(result.repaired).toBe(0);
+			expect(result.errors).toHaveLength(0);
+		});
+
+		it("should strip trailing hyphen from dose IDs", async () => {
+			const ts = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${ts}-`],
+			});
+
+			const result = await repairTrailingHyphenDoseIds(client);
+			expect(result.repaired).toBe(1);
+			expect(result.errors).toHaveLength(0);
+
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${ts}`);
+		});
+
+		it("should not modify dose IDs with valid person suffixes", async () => {
+			const ts = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${ts}-Alice`],
+			});
+
+			const result = await repairTrailingHyphenDoseIds(client);
+			expect(result.repaired).toBe(0);
+
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${ts}-Alice`);
+		});
+
+		it("should handle multiple trailing hyphens", async () => {
+			const ts = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${ts}--`],
+			});
+
+			const result = await repairTrailingHyphenDoseIds(client);
+			expect(result.repaired).toBe(1);
+
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking");
+			expect(doses.rows[0].dose_id).toBe(`1-0-${ts}`);
+		});
+
+		it("should repair multiple affected rows at once", async () => {
+			const ts1 = new Date(2025, 9, 17).getTime();
+			const ts2 = new Date(2025, 9, 24).getTime();
+			const ts3 = new Date(2025, 9, 31).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?), (1, ?), (1, ?)",
+				args: [`1-0-${ts1}-`, `2-0-${ts2}-`, `1-0-${ts3}`],
+			});
+
+			const result = await repairTrailingHyphenDoseIds(client);
+			expect(result.repaired).toBe(2); // Only 2 had trailing hyphens
+			expect(result.errors).toHaveLength(0);
+
+			const doses = await client.execute("SELECT dose_id FROM dose_tracking ORDER BY dose_id");
+			const ids = doses.rows.map((r) => r.dose_id);
+			expect(ids).toContain(`1-0-${ts1}`);
+			expect(ids).toContain(`2-0-${ts2}`);
+			expect(ids).toContain(`1-0-${ts3}`);
+		});
+
+		it("should be idempotent - running twice has no effect the second time", async () => {
+			const ts = new Date(2025, 9, 17).getTime();
+			await client.execute({
+				sql: "INSERT INTO dose_tracking (user_id, dose_id) VALUES (1, ?)",
+				args: [`1-0-${ts}-`],
+			});
+
+			const result1 = await repairTrailingHyphenDoseIds(client);
+			expect(result1.repaired).toBe(1);
+
+			const result2 = await repairTrailingHyphenDoseIds(client);
+			expect(result2.repaired).toBe(0);
+		});
+	});
 });
@@ -1730,6 +1730,304 @@ describe("E2E Tests with Real Routes", () => {
 		});
 	});

+	// ---------------------------------------------------------------------------
+	// Real Stock Correction (PATCH /medications/:id/stock-adjustment) Tests
+	// ---------------------------------------------------------------------------
+
+	describe("Real /medications/:id/stock-adjustment routes", () => {
+		it("should update stockAdjustment and lastStockCorrectionAt", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Stock Correction Med",
+					packCount: 1,
+					blistersPerPack: 14,
+					pillsPerBlister: 14,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			expect(createResponse.statusCode).toBe(200);
+			const medId = createResponse.json().id;
+
+			// Correct stock: set adjustment to -83 (196 base - 83 = 113 pills)
+			const response = await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -83 },
+			});
+
+			expect(response.statusCode).toBe(200);
+			const data = response.json();
+			expect(data.stockAdjustment).toBe(-83);
+			expect(data.lastStockCorrectionAt).toBeTruthy();
+			expect(data.updatedAt).toBeTruthy();
+		});
+
+		it("should persist stockAdjustment in GET /medications", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Persist Stock Med",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			// Apply stock correction
+			await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -7 },
+			});
+
+			// Verify via GET
+			const getResponse = await app.inject({
+				method: "GET",
+				url: "/medications",
+			});
+
+			expect(getResponse.statusCode).toBe(200);
+			const meds = getResponse.json();
+			const med = meds.find((m: any) => m.id === medId);
+			expect(med).toBeDefined();
+			expect(med.stockAdjustment).toBe(-7);
+			expect(med.lastStockCorrectionAt).toBeTruthy();
+		});
+
+		it("should not reset stockAdjustment when editing medication via PUT", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Keep Adjustment Med",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			// Set stock adjustment
+			await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -5 },
+			});
+
+			// Edit the medication (change name) - should preserve stockAdjustment
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Renamed Med",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+
+			// Verify stockAdjustment is preserved
+			const getResponse = await app.inject({
+				method: "GET",
+				url: "/medications",
+			});
+			const med = getResponse.json().find((m: any) => m.id === medId);
+			expect(med.name).toBe("Renamed Med");
+			expect(med.stockAdjustment).toBe(-5);
+		});
+
+		it("should return 400 for non-numeric stockAdjustment", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Bad Adjustment Med",
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			const response = await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: "not-a-number" },
+			});
+
+			expect(response.statusCode).toBe(400);
+		});
+
+		it("should return 404 for non-existent medication", async () => {
+			const response = await app.inject({
+				method: "PATCH",
+				url: "/medications/99999/stock-adjustment",
+				payload: { stockAdjustment: 5 },
+			});
+
+			expect(response.statusCode).toBe(404);
+		});
+
+		it("should return 400 for invalid medication id", async () => {
+			const response = await app.inject({
+				method: "PATCH",
+				url: "/medications/invalid/stock-adjustment",
+				payload: { stockAdjustment: 5 },
+			});
+
+			expect(response.statusCode).toBe(400);
+		});
+
+		it("should reset stockAdjustment when stock fields change via PUT", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Reset Adj Med",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			// Set stock adjustment to -10
+			await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -10 },
+			});
+
+			// Verify adjustment is set
+			let getMeds = await app.inject({ method: "GET", url: "/medications" });
+			let med = getMeds.json().find((m: any) => m.id === medId);
+			expect(med.stockAdjustment).toBe(-10);
+
+			// Edit medication with CHANGED stock fields (packCount 1 → 2)
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Reset Adj Med",
+					packCount: 2,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+
+			// stockAdjustment should be reset to 0
+			getMeds = await app.inject({ method: "GET", url: "/medications" });
+			med = getMeds.json().find((m: any) => m.id === medId);
+			expect(med.stockAdjustment).toBe(0);
+			expect(med.lastStockCorrectionAt).toBeTruthy();
+		});
+
+		it("should preserve stockAdjustment when only non-stock fields change via PUT", async () => {
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Preserve Adj Med",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			// Set stock adjustment
+			await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -5 },
+			});
+
+			// Edit only non-stock fields (name, notes)
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Renamed Preserve Med",
+					notes: "Updated notes",
+					packCount: 1,
+					blistersPerPack: 1,
+					pillsPerBlister: 30,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: "2025-01-01T08:00:00.000Z" }],
+				},
+			});
+
+			// stockAdjustment should be preserved
+			const getMeds = await app.inject({ method: "GET", url: "/medications" });
+			const med = getMeds.json().find((m: any) => m.id === medId);
+			expect(med.name).toBe("Renamed Preserve Med");
+			expect(med.stockAdjustment).toBe(-5);
+		});
+
+		it("should not count phantom consumption in planner after stock correction", async () => {
+			// Create medication: 1 pack × 14 blisters × 14 pills = 196 pills total
+			// Schedule: 1 pill daily starting far in the past
+			const farPast = new Date("2024-01-01T08:00:00.000Z");
+
+			const createResponse = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Planner Phantom Med",
+					packCount: 1,
+					blistersPerPack: 14,
+					pillsPerBlister: 14,
+					looseTablets: 0,
+					blisters: [{ usage: 1, every: 1, start: farPast.toISOString() }],
+				},
+			});
+			const medId = createResponse.json().id;
+
+			// Correct stock to 113 pills (196 base - 83 = 113)
+			await app.inject({
+				method: "PATCH",
+				url: `/medications/${medId}/stock-adjustment`,
+				payload: { stockAdjustment: -83 },
+			});
+
+			// Query planner immediately - stock should be ~113 (not reduced by phantom dose)
+			const tomorrow = new Date();
+			tomorrow.setDate(tomorrow.getDate() + 1);
+			const nextWeek = new Date();
+			nextWeek.setDate(nextWeek.getDate() + 7);
+
+			const response = await app.inject({
+				method: "POST",
+				url: "/medications/usage",
+				payload: {
+					startDate: tomorrow.toISOString(),
+					endDate: nextWeek.toISOString(),
+				},
+			});
+
+			expect(response.statusCode).toBe(200);
+			const data = response.json();
+			const med = data.find((m: any) => m.medicationId === medId);
+			expect(med).toBeDefined();
+			// Total should be very close to 113 (not 112 or lower from phantom consumption)
+			// Allow up to 1 pill of natural consumption (test runs fast, but at most 1 day could pass)
+			expect(med.totalPills).toBeGreaterThanOrEqual(112);
+			expect(med.totalPills).toBeLessThanOrEqual(113);
+		});
+	});
+
 	// ---------------------------------------------------------------------------
 	// Real Export/Import Routes Tests
 	// ---------------------------------------------------------------------------
@@ -365,6 +365,196 @@ describe("Integration Tests", () => {
 		});
 	});

+	// ---------------------------------------------------------------------------
+	// Dose ID Migration on Schedule Changes
+	// ---------------------------------------------------------------------------
+
+	describe("Dose ID migration when schedule changes", () => {
+		it("should migrate dose IDs when weekly start day changes", async () => {
+			// Create a weekly medication starting Friday Oct 17
+			const createRes = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Weekly Med",
+					blisters: [{ usage: 1, every: 7, start: "2025-10-17T08:00:00" }],
+				},
+			});
+			const medId = createRes.json().id;
+
+			// Mark doses for Fridays (Oct 17, Oct 24, Oct 31)
+			const fri17 = new Date(2025, 9, 17).getTime(); // Oct 17
+			const fri24 = new Date(2025, 9, 24).getTime(); // Oct 24
+			const fri31 = new Date(2025, 9, 31).getTime(); // Oct 31
+
+			for (const ts of [fri17, fri24, fri31]) {
+				await app.inject({
+					method: "POST",
+					url: "/doses/taken",
+					payload: { doseId: `${medId}-0-${ts}` },
+				});
+			}
+
+			// Verify 3 doses exist
+			const before = await testClient.execute({
+				sql: `SELECT COUNT(*) as count FROM dose_tracking WHERE dose_id LIKE ?`,
+				args: [`${medId}-%`],
+			});
+			expect(before.rows[0].count).toBe(3);
+
+			// Change start to Saturday Oct 18 (shifts all future and past IDs)
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Weekly Med",
+					blisters: [{ usage: 1, every: 7, start: "2025-10-18T08:00:00" }],
+				},
+			});
+
+			// Doses should be migrated to Saturday dates
+			const sat18 = new Date(2025, 9, 18).getTime(); // Oct 18
+			const sat25 = new Date(2025, 9, 25).getTime(); // Oct 25
+			const nov1 = new Date(2025, 10, 1).getTime(); // Nov 1
+
+			const after = await testClient.execute({
+				sql: `SELECT dose_id FROM dose_tracking WHERE dose_id LIKE ? ORDER BY dose_id`,
+				args: [`${medId}-%`],
+			});
+			expect(after.rows.length).toBe(3);
+			const ids = after.rows.map((r: { dose_id: string }) => r.dose_id);
+			expect(ids).toContain(`${medId}-0-${sat18}`);
+			expect(ids).toContain(`${medId}-0-${sat25}`);
+			expect(ids).toContain(`${medId}-0-${nov1}`);
+		});
+
+		it("should migrate dose IDs with person suffix when schedule changes", async () => {
+			// Create weekly medication with takenBy person
+			const createRes = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Person Med",
+					intakes: [{ usage: 1, every: 7, start: "2025-10-17T08:00:00", takenBy: "Alice" }],
+				},
+			});
+			const medId = createRes.json().id;
+
+			// Mark dose with person suffix
+			const fri17 = new Date(2025, 9, 17).getTime();
+			await app.inject({
+				method: "POST",
+				url: "/doses/taken",
+				payload: { doseId: `${medId}-0-${fri17}-Alice` },
+			});
+
+			// Change start day
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Person Med",
+					intakes: [{ usage: 1, every: 7, start: "2025-10-18T08:00:00", takenBy: "Alice" }],
+				},
+			});
+
+			// Dose should be migrated with person suffix preserved
+			const sat18 = new Date(2025, 9, 18).getTime();
+			const after = await testClient.execute({
+				sql: `SELECT dose_id FROM dose_tracking WHERE dose_id LIKE ?`,
+				args: [`${medId}-%`],
+			});
+			expect(after.rows.length).toBe(1);
+			expect(after.rows[0].dose_id).toBe(`${medId}-0-${sat18}-Alice`);
+		});
+
+		it("should not migrate dose IDs when only time-of-day changes", async () => {
+			// Create daily medication at 08:00
+			const createRes = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Daily Med",
+					blisters: [{ usage: 1, every: 1, start: "2025-10-17T08:00:00" }],
+				},
+			});
+			const medId = createRes.json().id;
+
+			// Mark dose
+			const oct17 = new Date(2025, 9, 17).getTime();
+			await app.inject({
+				method: "POST",
+				url: "/doses/taken",
+				payload: { doseId: `${medId}-0-${oct17}` },
+			});
+
+			// Change only time from 08:00 to 20:00 (same date)
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Daily Med",
+					blisters: [{ usage: 1, every: 1, start: "2025-10-17T20:00:00" }],
+				},
+			});
+
+			// Dose ID should remain unchanged (dateOnlyMs is the same)
+			const after = await testClient.execute({
+				sql: `SELECT dose_id FROM dose_tracking WHERE dose_id LIKE ?`,
+				args: [`${medId}-%`],
+			});
+			expect(after.rows.length).toBe(1);
+			expect(after.rows[0].dose_id).toBe(`${medId}-0-${oct17}`);
+		});
+
+		it("should migrate dose IDs when interval changes from daily to every-other-day", async () => {
+			// Create daily medication starting Oct 17
+			const createRes = await app.inject({
+				method: "POST",
+				url: "/medications",
+				payload: {
+					name: "Interval Med",
+					blisters: [{ usage: 1, every: 1, start: "2025-10-17T08:00:00" }],
+				},
+			});
+			const medId = createRes.json().id;
+
+			// Mark doses for Oct 17, 18, 19
+			const oct17 = new Date(2025, 9, 17).getTime();
+			const oct18 = new Date(2025, 9, 18).getTime();
+			const oct19 = new Date(2025, 9, 19).getTime();
+
+			for (const ts of [oct17, oct18, oct19]) {
+				await app.inject({
+					method: "POST",
+					url: "/doses/taken",
+					payload: { doseId: `${medId}-0-${ts}` },
+				});
+			}
+
+			// Change to every 2 days (Oct 17, 19, 21, ...)
+			await app.inject({
+				method: "PUT",
+				url: `/medications/${medId}`,
+				payload: {
+					name: "Interval Med",
+					blisters: [{ usage: 1, every: 2, start: "2025-10-17T08:00:00" }],
+				},
+			});
+
+			// Oct 17 stays (matches), Oct 18 → Oct 19 (nearest), Oct 19 → no match (already used)
+			// Actually: Oct 17 is exact match (no migration needed), Oct 18 maps to Oct 19 (within 1 day = half of 2),
+			// Oct 19 was the original schedule date but the new schedule also has Oct 19,
+			// which was already taken by Oct 18's migration
+			const after = await testClient.execute({
+				sql: `SELECT dose_id FROM dose_tracking WHERE dose_id LIKE ? ORDER BY dose_id`,
+				args: [`${medId}-%`],
+			});
+			// We should have at least the doses that could be mapped
+			expect(after.rows.length).toBeGreaterThanOrEqual(2);
+		});
+	});
+
 	// ---------------------------------------------------------------------------
 	// Share Link + Dose Tracking Integration
 	// ---------------------------------------------------------------------------
@@ -6,6 +6,7 @@
 import { existsSync, mkdirSync } from "node:fs";
 import { resolve } from "node:path";
 import type { CookieSerializeOptions } from "@fastify/cookie";
+import { getDataDir } from "../db/db-utils.js";

 /**
 * Parse comma-separated CORS origins string
@@ -81,8 +82,7 @@ export function buildAppConfig(options: AppConfigOptions): AppConfig {
 * Ensure images directory exists
 */
 export function ensureImagesDirectory(cwd?: string): string {
-	const basePath = cwd || process.cwd();
-	const imagesDir = resolve(basePath, "data/images");
+	const imagesDir = resolve(getDataDir(cwd), "images");
 	if (!existsSync(imagesDir)) {
 		mkdirSync(imagesDir, { recursive: true });
 	}
@@ -9,6 +9,8 @@ services:
      - ./data:/app/data
    env_file:
      - .env
+    environment:
+      - DATA_DIR=/app/data
    ports:
      - "3000:3000"
    security_opt:
@@ -7,6 +7,7 @@ services:
    environment:
      - PUID=${PUID:-1000}
      - PGID=${PGID:-1000}
+      - DATA_DIR=/app/data
    volumes:
      - ./data:/app/data
    ports:
@@ -34,6 +35,8 @@ services:
  frontend:
    image: ghcr.io/danielvolz/medassist-ng-frontend:latest
    container_name: medassist-ng-frontend
+    environment:
+      - BACKEND_URL=backend:3000
    ports:
      - "4174:8080"
    networks:
@@ -49,6 +52,7 @@ services:
      - /tmp:noexec,nosuid,size=64m
      - /var/cache/nginx:noexec,nosuid,size=64m
      - /var/run:noexec,nosuid,size=64m
+      - /etc/nginx/conf.d:noexec,nosuid,size=1m
    cap_drop:
      - ALL

@@ -32,8 +32,14 @@ RUN npm run build
 # -----------------------------------------------------------------------------
 FROM nginxinc/nginx-unprivileged:1.27-alpine AS runner

-# Copy custom nginx config (must listen on 8080, not 80)
-COPY nginx.conf /etc/nginx/conf.d/default.conf
+# Redirect envsubst output to /tmp (writable under read_only: true)
+# and update nginx main config to include from there instead of /etc/nginx/conf.d/
+ENV NGINX_ENVSUBST_OUTPUT_DIR=/tmp
+RUN sed -i 's|include /etc/nginx/conf.d/\*.conf;|include /tmp/default.conf;|' /etc/nginx/nginx.conf
+
+# Copy custom nginx config as template for envsubst processing
+# nginx-unprivileged automatically substitutes env vars in .template files
+COPY nginx.conf /etc/nginx/templates/default.conf.template

 # Copy built static files with correct ownership (nginx user = uid 101)
 COPY --from=builder --chown=101:101 /app/dist /usr/share/nginx/html
@@ -44,5 +50,5 @@ EXPOSE 8080
 # Already runs as non-root (nginx user, uid 101)
 USER nginx

-# Start nginx
+# Start nginx (entrypoint processes templates automatically)
 CMD ["nginx", "-g", "daemon off;"]
@@ -20,7 +20,14 @@ server {
    }

    location /api/ {
-        proxy_pass http://medassist-ng-backend:3000/;
+        # Use variable for runtime DNS resolution (nginx resolves at startup by default)
+        # Docker embedded DNS (127.0.0.11) with 10s cache
+        resolver 127.0.0.11 valid=10s ipv6=off;
+        set $backend_upstream ${BACKEND_URL};
+
+        # Strip /api prefix (nginx doesn't auto-rewrite when using variables in proxy_pass)
+        rewrite ^/api/(.*)$ /$1 break;
+        proxy_pass http://$backend_upstream;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
@@ -1,12 +1,12 @@
 {
  "name": "medassist-ng-frontend",
-  "version": "1.7.1",
+  "version": "1.8.3",
  "lockfileVersion": 3,
  "requires": true,
  "packages": {
    "": {
      "name": "medassist-ng-frontend",
-      "version": "1.7.1",
+      "version": "1.8.3",
      "dependencies": {
        "i18next": "^24.2.2",
        "i18next-browser-languagedetector": "^8.0.4",
@@ -133,7 +133,6 @@
      "integrity": "sha512-e7jT4DxYvIDLk1ZHmU/m/mB19rex9sv0c2ftBtjSBv+kVM/902eh0fINUzD7UwLLNR+jU585GxUJ8/EBfAM5fw==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "@babel/code-frame": "^7.27.1",
        "@babel/generator": "^7.28.5",
@@ -655,7 +654,6 @@
        }
      ],
      "license": "MIT",
-      "peer": true,
      "engines": {
        "node": ">=18"
      },
@@ -699,7 +697,6 @@
        }
      ],
      "license": "MIT",
-      "peer": true,
      "engines": {
        "node": ">=18"
      }
@@ -1647,7 +1644,8 @@
      "resolved": "https://registry.npmjs.org/@types/aria-query/-/aria-query-5.0.4.tgz",
      "integrity": "sha512-rfT93uj5s0PRL7EzccGMs3brplhcrghnDoV26NqKhCAS1hVo+WdNsPvE/yb6ilfr5hi2MEk6d5EWJTKdxg8jVw==",
      "dev": true,
-      "license": "MIT"
+      "license": "MIT",
+      "peer": true
    },
    "node_modules/@types/babel__core": {
      "version": "7.20.5",
@@ -1739,7 +1737,6 @@
      "integrity": "sha512-cisd7gxkzjBKU2GgdYrTdtQx1SORymWyaAFhaxQPK9bYO9ot3Y5OikQRvY0VYQtvwjeQnizCINJAenh/V7MK2w==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "@types/prop-types": "*",
        "csstype": "^3.2.2"
@@ -1751,7 +1748,6 @@
      "integrity": "sha512-MEe3UeoENYVFXzoXEWsvcpg6ZvlrFNlOQ7EOsvhI3CfAXwzPfO8Qwuxd40nepsYKqyyVQnTdEfv68q91yLcKrQ==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "peerDependencies": {
        "@types/react": "^18.0.0"
      }
@@ -1958,6 +1954,7 @@
      "integrity": "sha512-quJQXlTSUGL2LH9SUXo8VwsY4soanhgo6LNSm84E1LBcE8s3O0wpdiRzyR9z/ZZJMlMWv37qOOb9pdJlMUEKFQ==",
      "dev": true,
      "license": "MIT",
+      "peer": true,
      "engines": {
        "node": ">=8"
      }
@@ -1968,6 +1965,7 @@
      "integrity": "sha512-Cxwpt2SfTzTtXcfOlzGEee8O+c+MmUgGrNiBcXnuWxuFJHe6a5Hz7qwhwe5OgaSYI0IJvkLqWX1ASG+cJOkEiA==",
      "dev": true,
      "license": "MIT",
+      "peer": true,
      "engines": {
        "node": ">=10"
      },
@@ -2054,7 +2052,6 @@
        }
      ],
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "baseline-browser-mapping": "^2.9.0",
        "caniuse-lite": "^1.0.30001759",
@@ -2238,7 +2235,8 @@
      "resolved": "https://registry.npmjs.org/dom-accessibility-api/-/dom-accessibility-api-0.5.16.tgz",
      "integrity": "sha512-X7BJ2yElsnOJ30pZF4uIIDfBEVgF4XEBxL9Bxhy6dnrm5hkzqmsWHGTiHqRiITNhMyFLyAiWndIJP7Z1NTteDg==",
      "dev": true,
-      "license": "MIT"
+      "license": "MIT",
+      "peer": true
    },
    "node_modules/electron-to-chromium": {
      "version": "1.5.267",
@@ -2468,7 +2466,6 @@
        }
      ],
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "@babel/runtime": "^7.26.10"
      },
@@ -2558,7 +2555,6 @@
      "integrity": "sha512-mjzqwWRD9Y1J1KUi7W97Gja1bwOOM5Ug0EZ6UDK3xS7j7mndrkwozHtSblfomlzyB4NepioNt+B2sOSzczVgtQ==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "@acemir/cssom": "^0.9.28",
        "@asamuzakjp/dom-selector": "^6.7.6",
@@ -2647,6 +2643,7 @@
      "integrity": "sha512-h5bgJWpxJNswbU7qCrV0tIKQCaS3blPDrqKWx+QxzuzL1zGUzij9XCWLrSLsJPu5t+eWA/ycetzYAO5IOMcWAQ==",
      "dev": true,
      "license": "MIT",
+      "peer": true,
      "bin": {
        "lz-string": "bin/bin.js"
      }
@@ -2796,7 +2793,6 @@
      "integrity": "sha512-5gTmgEY/sqK6gFXLIsQNH19lWb4ebPDLA4SdLP7dsWkIXHWlG66oPuVvXSGFPppYZz8ZDZq0dYYrbHfBCVUb1Q==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "engines": {
        "node": ">=12"
      },
@@ -2886,6 +2882,7 @@
      "integrity": "sha512-Qb1gy5OrP5+zDf2Bvnzdl3jsTf1qXVMazbvCoKhtKqVs4/YK4ozX4gKQJJVyNe+cajNPn0KoC0MC3FUmaHWEmQ==",
      "dev": true,
      "license": "MIT",
+      "peer": true,
      "dependencies": {
        "ansi-regex": "^5.0.1",
        "ansi-styles": "^5.0.0",
@@ -2910,7 +2907,6 @@
      "resolved": "https://registry.npmjs.org/react/-/react-18.3.1.tgz",
      "integrity": "sha512-wS+hAgJShR0KhEvPJArfuPVN1+Hz1t0Y6n5jLrGQbkb4urgPE/0Rve+1kMB1v/oWgHgm4WIcV+i7F2pTVj+2iQ==",
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "loose-envify": "^1.1.0"
      },
@@ -2923,7 +2919,6 @@
      "resolved": "https://registry.npmjs.org/react-dom/-/react-dom-18.3.1.tgz",
      "integrity": "sha512-5m4nQKp+rZRb09LNH59GM4BxTh9251/ylbKIbpe7TpGxfJ+9kv6BLkLBXIjjspbgbnIBNqlI23tRnTWT0snUIw==",
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "loose-envify": "^1.1.0",
        "scheduler": "^0.23.2"
@@ -2963,7 +2958,8 @@
      "resolved": "https://registry.npmjs.org/react-is/-/react-is-17.0.2.tgz",
      "integrity": "sha512-w2GsyukL62IJnlaff/nRegPQR94C/XXamvMWmSHRJ4y7Ts/4ocGRmTHvOs8PSE6pB3dWOrD/nueuU5sduBsQ4w==",
      "dev": true,
-      "license": "MIT"
+      "license": "MIT",
+      "peer": true
    },
    "node_modules/react-refresh": {
      "version": "0.17.0",
@@ -3277,7 +3273,6 @@
      "integrity": "sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw==",
      "devOptional": true,
      "license": "Apache-2.0",
-      "peer": true,
      "bin": {
        "tsc": "bin/tsc",
        "tsserver": "bin/tsserver"
@@ -3323,7 +3318,6 @@
      "integrity": "sha512-dZwN5L1VlUBewiP6H9s2+B3e3Jg96D0vzN+Ry73sOefebhYr9f94wwkMNN/9ouoU8pV1BqA1d1zGk8928cx0rg==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "esbuild": "^0.27.0",
        "fdir": "^6.5.0",
@@ -3399,7 +3393,6 @@
      "integrity": "sha512-FQMeF0DJdWY0iOnbv466n/0BudNdKj1l5jYgl5JVTwjSsZSlqyXFt/9+1sEyhR6CLowbZpV7O1sCHrzBhucKKg==",
      "dev": true,
      "license": "MIT",
-      "peer": true,
      "dependencies": {
        "@vitest/expect": "4.0.17",
        "@vitest/mocker": "4.0.17",
@@ -1,7 +1,7 @@
 {
  "name": "medassist-ng-frontend",
  "private": true,
-  "version": "1.7.1",
+  "version": "1.8.6",
  "type": "module",
  "scripts": {
    "dev": "vite",
@@ -3,7 +3,7 @@ import { useTranslation } from "react-i18next";
 import { FRONTEND_VERSION, GITHUB_URL } from "../App";

 interface UpdateCheckResult {
-	status: "checking" | "up-to-date" | "update-available" | "error";
+	status: "up-to-date" | "update-available" | "error";
 	latestVersion?: string;
 	lastChecked?: string;
 }
@@ -15,19 +15,13 @@ interface AboutModalProps {

 export default function AboutModal({ isOpen, onClose }: AboutModalProps) {
 	const { t } = useTranslation();
-	const [backendVersion, setBackendVersion] = useState<string | null>(null);
+	const [isChecking, setIsChecking] = useState(false);
 	const [updateCheckResult, setUpdateCheckResult] = useState<UpdateCheckResult | null>(null);

-	// Fetch backend version and cached update result on mount
+	// Load cached update check result on mount
 	useEffect(() => {
 		if (!isOpen) return;

-		// Fetch backend version
-		fetch("/api/health")
-			.then((res) => res.json())
-			.then((data) => setBackendVersion(data.version || "unknown"))
-			.catch(() => setBackendVersion("unknown"));
-
 		// Load cached update check result
 		const cached = sessionStorage.getItem("updateCheckResult");
 		if (cached) {
@@ -43,9 +37,13 @@ export default function AboutModal({ isOpen, onClose }: AboutModalProps) {
 	}, [isOpen]);

 	async function checkForUpdates() {
-		setUpdateCheckResult({ status: "checking" });
+		setIsChecking(true);
+		const minDelay = new Promise((resolve) => setTimeout(resolve, 1000));
 		try {
-			const res = await fetch(`https://api.github.com/repos/DanielVolz/medassist-ng/releases/latest`);
+			const [res] = await Promise.all([
+				fetch(`https://api.github.com/repos/DanielVolz/medassist-ng/releases/latest`),
+				minDelay,
+			]);
 			if (!res.ok) throw new Error("Failed to fetch");
 			const data = await res.json();
 			const latestVersion = (data.tag_name || "").replace(/^v/, "");
@@ -61,6 +59,8 @@ export default function AboutModal({ isOpen, onClose }: AboutModalProps) {
 			sessionStorage.setItem("updateCheckResult", JSON.stringify(result));
 		} catch {
 			setUpdateCheckResult({ status: "error" });
+		} finally {
+			setIsChecking(false);
 		}
 	}

@@ -74,32 +74,27 @@ export default function AboutModal({ isOpen, onClose }: AboutModalProps) {
 				</button>
 				<div className="about-header">
 					<div className="about-logo">
-						<svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5">
-							<path d="M19.5 12c0 4.14-3.36 7.5-7.5 7.5S4.5 16.14 4.5 12 7.86 4.5 12 4.5s7.5 3.36 7.5 7.5z" />
-							<path d="M12 8v4l2.5 2.5" />
-							<path d="M9 2h6M12 2v2" />
-						</svg>
+						<img src="/favicon.svg" alt="MedAssist-ng" />
 					</div>
 					<h2>{t("about.appName", "MedAssist-ng")}</h2>
 					<p className="about-tagline">{t("about.description", "Personal medication tracking and reminder app")}</p>
 				</div>
 				<div className="about-versions">
 					<div className="about-version-row">
-						<span className="about-version-label">{t("about.frontendVersion", "Frontend")}</span>
-						<span className="about-version-value">{FRONTEND_VERSION}</span>
-					</div>
-					<div className="about-version-row">
-						<span className="about-version-label">{t("about.backendVersion", "Backend")}</span>
-						<span className="about-version-value">{backendVersion || "..."}</span>
+						<span className="about-version-label">{t("about.version", "Version")}</span>
+						<a
+							href={`${GITHUB_URL}/releases/tag/v${FRONTEND_VERSION}`}
+							target="_blank"
+							rel="noopener noreferrer"
+							className="about-version-value about-version-link"
+						>
+							{FRONTEND_VERSION}
+						</a>
 					</div>
 				</div>
 				<div className="about-update-section">
-					<button
-						className="about-update-btn"
-						onClick={checkForUpdates}
-						disabled={updateCheckResult?.status === "checking"}
-					>
-						{updateCheckResult?.status === "checking" ? (
+					<button className="about-update-btn" onClick={checkForUpdates} disabled={isChecking}>
+						{isChecking ? (
 							<>
 								<span className="spinner-small"></span>
 								{t("about.checking", "Checking...")}
@@ -116,7 +111,7 @@ export default function AboutModal({ isOpen, onClose }: AboutModalProps) {
 							</>
 						)}
 					</button>
-					{updateCheckResult && updateCheckResult.status !== "checking" && (
+					{updateCheckResult && (
 						<div className={`about-update-result ${updateCheckResult.status}`}>
 							{updateCheckResult.status === "up-to-date" && (
 								<span className="update-status-text">✓ {t("about.upToDate", "You are up to date!")}</span>
@@ -351,38 +351,47 @@ export function AppProvider({ children }: { children: React.ReactNode }) {

 	const groupedSchedule = useMemo(() => {
 		const days = new Map<string, { dateStr: string; date: Date; isPast: boolean; meds: Map<string, DayMedEntry> }>();
-		schedule.events.slice(0, 2000).forEach((event) => {
-			const day = days.get(event.dateStr) ?? {
-				dateStr: event.dateStr,
-				date: new Date(event.when),
-				isPast: event.isPast,
-				meds: new Map(),
-			};
-			const medEntry = day.meds.get(event.medName) ?? {
-				medName: event.medName,
-				total: 0,
-				doses: [],
-				lastWhen: event.when,
-			};
-			medEntry.total += event.usage;
-			medEntry.doses.push({
-				id: event.id,
-				timeStr: event.timeStr,
-				when: event.when,
-				usage: event.usage,
-				takenBy: event.takenBy ? [event.takenBy] : [],
+		// Limit past events to scheduleDays window to avoid overwhelming the UI.
+		// Without this, medications with start dates far in the past generate thousands
+		// of events that fill the display budget and push out today/future events.
+		const pastCutoff = new Date();
+		pastCutoff.setDate(pastCutoff.getDate() - scheduleDays);
+		pastCutoff.setHours(0, 0, 0, 0);
+		const pastCutoffMs = pastCutoff.getTime();
+		schedule.events
+			.filter((e) => !e.isPast || e.when >= pastCutoffMs)
+			.forEach((event) => {
+				const day = days.get(event.dateStr) ?? {
+					dateStr: event.dateStr,
+					date: new Date(event.when),
+					isPast: event.isPast,
+					meds: new Map(),
+				};
+				const medEntry = day.meds.get(event.medName) ?? {
+					medName: event.medName,
+					total: 0,
+					doses: [],
+					lastWhen: event.when,
+				};
+				medEntry.total += event.usage;
+				medEntry.doses.push({
+					id: event.id,
+					timeStr: event.timeStr,
+					when: event.when,
+					usage: event.usage,
+					takenBy: event.takenBy ? [event.takenBy] : [],
+				});
+				medEntry.lastWhen = Math.max(medEntry.lastWhen, event.when);
+				day.meds.set(event.medName, medEntry);
+				days.set(event.dateStr, day);
 			});
-			medEntry.lastWhen = Math.max(medEntry.lastWhen, event.when);
-			day.meds.set(event.medName, medEntry);
-			days.set(event.dateStr, day);
-		});
 		return Array.from(days.values()).map((d) => ({
 			dateStr: d.dateStr,
 			date: d.date,
 			isPast: d.isPast,
 			meds: Array.from(d.meds.values()),
 		}));
-	}, [schedule.events]);
+	}, [schedule.events, scheduleDays]);

 	const pastDays = useMemo(() => groupedSchedule.filter((d) => d.isPast), [groupedSchedule]);

@@ -474,8 +474,6 @@
    "appName": "MedAssist-ng",
    "description": "Open-Source Medikamentenverwaltung und Planungsanwendung für selbst gehostete Umgebungen.",
    "version": "Version",
-    "frontend": "Frontend",
-    "backend": "Backend",
    "checkForUpdates": "Nach Updates suchen",
    "checking": "Prüfe...",
    "upToDate": "Du bist auf dem neuesten Stand!",
@@ -474,8 +474,6 @@
    "appName": "MedAssist-ng",
    "description": "Open-source medication tracking and planning application for self-hosted environments.",
    "version": "Version",
-    "frontend": "Frontend",
-    "backend": "Backend",
    "checkForUpdates": "Check for Updates",
    "checking": "Checking...",
    "upToDate": "You're up to date!",
@@ -4,7 +4,7 @@ import { useAuth } from "../components/Auth";
 import { useAppContext } from "../context";
 import type { Coverage } from "../types";
 import { formatNumber, getExpiryClass, getSystemLocale } from "../utils/formatters";
-import { getStockStatus } from "../utils/schedule";
+import { expandDoseIds, getStockStatus } from "../utils/schedule";

 // Helper for user-specific localStorage keys
 function userStorageKey(userId: number | undefined, key: string): string {
@@ -490,11 +490,7 @@ export function DashboardPage() {
 						{pastDays.length > 0 &&
 							(() => {
 								const missedCount = missedPastDoseIds.length;
-								const totalPastDoses = pastDays.flatMap((d) =>
-									d.meds.flatMap((m) =>
-										m.doses.flatMap((dose) => (dose.takenBy ? [`${dose.id}-${dose.takenBy}`] : [dose.id]))
-									)
-								);
+								const totalPastDoses = pastDays.flatMap((d) => d.meds.flatMap((m) => expandDoseIds(m.doses)));
 								return (
 									<div className="past-days-header">
 										<div
@@ -541,7 +537,10 @@ export function DashboardPage() {
 						{showPastDays &&
 							pastDays.map((day) => {
 								const allDoseIds = day.meds.flatMap((item) =>
-									item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]))
+									item.doses.flatMap((d) => {
+										const takenByArray = Array.isArray(d.takenBy) ? d.takenBy : [];
+										return takenByArray.length > 0 ? takenByArray.map((p) => `${d.id}-${p}`) : [d.id];
+									})
 								);
 								const allDayTaken =
 									allDoseIds.length > 0 && allDoseIds.every((id) => takenDoses.has(id) || dismissedDoses.has(id));
@@ -586,7 +585,7 @@ export function DashboardPage() {
 												const med = meds.find((m) => m.name === item.medName);
 												const medCov = coverageByMed[item.medName];
 												const isEmpty = medCov ? medCov.medsLeft <= 0 : false;
-												const itemDoseIds = item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]));
+												const itemDoseIds = expandDoseIds(item.doses);
 												const allTaken = itemDoseIds.every((id) => takenDoses.has(id));
 												return (
 													<div key={`${day.dateStr}-${item.medName}`} className={`time-row ${allTaken ? "taken" : ""}`}>
@@ -676,9 +675,7 @@ export function DashboardPage() {
 						{todayDay &&
 							(() => {
 								const day = todayDay;
-								const allDoseIds = day.meds.flatMap((item) =>
-									item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]))
-								);
+								const allDoseIds = day.meds.flatMap((item) => expandDoseIds(item.doses));
 								const allDayTaken = allDoseIds.length > 0 && allDoseIds.every((id) => takenDoses.has(id));
 								const takenCount = allDoseIds.filter((id) => takenDoses.has(id)).length;

@@ -737,7 +734,7 @@ export function DashboardPage() {
 													: medCoverage
 														? getStockStatus(medCoverage.daysLeft, medCoverage.medsLeft, stockThresholds)
 														: null;
-												const itemDoseIds = item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]));
+												const itemDoseIds = expandDoseIds(item.doses);
 												const allTaken = itemDoseIds.every((id) => takenDoses.has(id));
 												return (
 													<div key={`${day.dateStr}-${item.medName}`} className={`time-row ${allTaken ? "taken" : ""}`}>
@@ -769,7 +766,7 @@ export function DashboardPage() {
 														<div className="doses-col">
 															{item.doses.map((dose) => {
 																const isOverdue = dose.when < Date.now();
-																const people = dose.takenBy ? [dose.takenBy] : [null];
+																const people = dose.takenBy.length > 0 ? dose.takenBy : [null];
 																const allTaken = people.every((person) => takenDoses.has(getDoseId(dose.id, person)));
 																return (
 																	<div
@@ -866,9 +863,7 @@ export function DashboardPage() {
 						{/* Future days */}
 						{showFutureDays &&
 							futureDays.map((day) => {
-								const allDoseIds = day.meds.flatMap((item) =>
-									item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]))
-								);
+								const allDoseIds = day.meds.flatMap((item) => expandDoseIds(item.doses));
 								const allDayTaken = allDoseIds.length > 0 && allDoseIds.every((id) => takenDoses.has(id));
 								const takenCount = allDoseIds.filter((id) => takenDoses.has(id)).length;

@@ -926,7 +921,7 @@ export function DashboardPage() {
 													: medCoverage
 														? getStockStatus(medCoverage.daysLeft, medCoverage.medsLeft, stockThresholds)
 														: null;
-												const itemDoseIds = item.doses.flatMap((d) => (d.takenBy ? [`${d.id}-${d.takenBy}`] : [d.id]));
+												const itemDoseIds = expandDoseIds(item.doses);
 												const allTaken = itemDoseIds.every((id) => takenDoses.has(id));
 												return (
 													<div key={`${day.dateStr}-${item.medName}`} className={`time-row ${allTaken ? "taken" : ""}`}>
@@ -3,6 +3,7 @@ import { MedicationAvatar } from "../components";
 import { useAuth } from "../components/Auth";
 import { useAppContext } from "../context";
 import type { Coverage } from "../types";
+import { expandDoseIds } from "../utils/schedule";

 // Helper for user-specific localStorage keys
 function userStorageKey(userId: number | undefined, key: string): string {
@@ -125,12 +126,7 @@ export function SchedulePage() {
 					{/* Past days (when expanded) */}
 					{showPastDays &&
 						pastDays.map((day) => {
-							const allDoseIds = day.meds.flatMap((item) =>
-								item.doses.flatMap((d) => {
-									const takenByArray = Array.isArray(d.takenBy) ? d.takenBy : [];
-									return takenByArray.length > 0 ? takenByArray.map((p) => `${d.id}-${p}`) : [d.id];
-								})
-							);
+							const allDoseIds = day.meds.flatMap((item) => expandDoseIds(item.doses));
 							const allDayTaken = allDoseIds.length > 0 && allDoseIds.every((id) => takenDoses.has(id));
 							const takenCount = allDoseIds.filter((id) => takenDoses.has(id)).length;
 							const isManuallyExpanded = manuallyExpandedDays.has(day.dateStr);
@@ -172,10 +168,7 @@ export function SchedulePage() {
 											const med = meds.find((m) => m.name === item.medName);
 											const medCov = coverageByMed[item.medName];
 											const isEmpty = medCov ? medCov.medsLeft <= 0 : false;
-											const itemDoseIds = item.doses.flatMap((d) => {
-												const takenByArray = Array.isArray(d.takenBy) ? d.takenBy : [];
-												return takenByArray.length > 0 ? takenByArray.map((p) => `${d.id}-${p}`) : [d.id];
-											});
+											const itemDoseIds = expandDoseIds(item.doses);
 											const allTaken = itemDoseIds.every((id) => takenDoses.has(id));
 											return (
 												<div key={`${day.dateStr}-${item.medName}`} className={`time-row ${allTaken ? "taken" : ""}`}>
@@ -277,10 +270,7 @@ export function SchedulePage() {
 										: medCoverage
 											? getStockStatus(medCoverage.daysLeft, medCoverage.medsLeft, settings)
 											: null;
-									const itemDoseIds = item.doses.flatMap((d) => {
-										const takenByArray = Array.isArray(d.takenBy) ? d.takenBy : [];
-										return takenByArray.length > 0 ? takenByArray.map((p) => `${d.id}-${p}`) : [d.id];
-									});
+									const itemDoseIds = expandDoseIds(item.doses);
 									const allTaken = itemDoseIds.every((id) => takenDoses.has(id));
 									return (
 										<div key={`${day.dateStr}-${item.medName}`} className={`time-row ${allTaken ? "taken" : ""}`}>
@@ -4551,18 +4551,12 @@ h3 .reminder-icon.info-tooltip {
 	width: 64px;
 	height: 64px;
 	margin: 0 auto 1rem;
-	background: linear-gradient(135deg, var(--accent) 0%, var(--accent-hover) 100%);
-	border-radius: 16px;
-	display: flex;
-	align-items: center;
-	justify-content: center;
-	box-shadow: 0 4px 12px rgba(var(--accent-rgb, 59, 130, 246), 0.25);
 }

-.about-logo svg {
-	width: 36px;
-	height: 36px;
-	stroke: white;
+.about-logo img {
+	width: 64px;
+	height: 64px;
+	border-radius: 16px;
 }

 .about-header h2 {
@@ -4586,15 +4580,12 @@ h3 .reminder-icon.info-tooltip {

 .about-version-row {
 	display: flex;
-	justify-content: space-between;
+	justify-content: center;
 	align-items: center;
+	gap: 0.75rem;
 	padding: 0.5rem 0;
 }

-.about-version-row:not(:last-child) {
-	border-bottom: 1px dashed var(--border-secondary);
-}
-
 .about-version-label {
 	font-size: 0.875rem;
 	color: var(--text-secondary);
@@ -4610,9 +4601,20 @@ h3 .reminder-icon.info-tooltip {
 	border-radius: 4px;
 }

+a.about-version-link {
+	color: var(--accent-primary);
+	text-decoration: none;
+	transition: text-decoration 0.15s ease;
+}
+
+a.about-version-link:hover {
+	text-decoration: underline;
+}
+
 .about-update-section {
 	padding: 1.25rem 1.5rem;
 	border-bottom: 1px solid var(--border-primary);
+	min-height: 148px;
 }

 .about-update-btn {
@@ -16,10 +16,6 @@ describe("AboutModal", () => {

 	beforeEach(() => {
 		vi.clearAllMocks();
-		(global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-			ok: true,
-			json: () => Promise.resolve({ version: "1.0.0" }),
-		});
 	});

 	it("returns null when not open", () => {
@@ -65,8 +61,10 @@ describe("AboutModal", () => {
 		expect(links.length).toBeGreaterThan(0);
 	});

-	it("fetches backend version on open", async () => {
+	it("renders version as link to GitHub release", () => {
 		render(<AboutModal {...defaultProps} />);
-		expect(fetch).toHaveBeenCalledWith("/api/health");
+		const versionLink = screen.getByText("1.0.0").closest("a");
+		expect(versionLink).toHaveAttribute("href", "https://github.com/test/repo/releases/tag/v1.0.0");
+		expect(versionLink).toHaveAttribute("target", "_blank");
 	});
 });
@@ -4,6 +4,7 @@ import {
 	buildSchedulePreview,
 	calculateCoverage,
 	computeMissedPastDoseIds,
+	expandDoseIds,
 	getNextReminderForMed,
 	getReminderStatusText,
 	getStockStatus,
@@ -365,6 +366,123 @@ describe("calculateCoverage", () => {
 		expect(result.all).toHaveLength(1);
 		// Daily rate should be doubled for 2 people
 	});
+
+	it("stock correction immediately reflects new stock without phantom consumption", () => {
+		// BUG: After a stock correction of +1 pill, the coverage calculation
+		// immediately consumed 1 dose (due to the +1 in occurrences formula),
+		// making the correction appear to have no effect.
+		//
+		// Scenario: User has 112 pills, corrects to 113 (+1 pill).
+		// Expected: medsLeft = 113 immediately after correction.
+		// Bug: medsLeft stayed at 112 because 1 dose was counted as consumed.
+		const correctionTime = new Date("2024-03-15T12:00:00Z");
+
+		const meds: Medication[] = [
+			{
+				id: 1,
+				name: "TestMed",
+				packCount: 1,
+				blistersPerPack: 14,
+				pillsPerBlister: 14,
+				looseTablets: 0,
+				stockAdjustment: -83, // 196 - 83 = 113 pills
+				lastStockCorrectionAt: correctionTime.toISOString(),
+				takenBy: [],
+				blisters: [
+					{
+						usage: 1,
+						every: 1,
+						start: "2024-01-01T08:00:00",
+					},
+				],
+				updatedAt: correctionTime.toISOString(),
+			},
+		];
+
+		// Calculate coverage immediately after correction (same second)
+		const result = calculateCoverage(meds, [], "en", 7, "automatic", new Set());
+
+		expect(result.all).toHaveLength(1);
+		// getMedTotal = 1*14*14 + 0 + (-83) = 113
+		// Consumed since correction should be 0 (not 1!)
+		expect(result.all[0].medsLeft).toBe(113);
+	});
+
+	it("stock correction with dose tracking data also reflects correctly", () => {
+		// When the user has dose tracking data, the actualConsumed path is used.
+		// Verify that no phantom dose is generated right after a stock correction.
+		const correctionTime = new Date("2024-03-15T12:00:00Z");
+		const march14 = new Date("2024-03-14T00:00:00").getTime();
+
+		const meds: Medication[] = [
+			{
+				id: 1,
+				name: "TestMed",
+				packCount: 1,
+				blistersPerPack: 1,
+				pillsPerBlister: 30,
+				looseTablets: 0,
+				stockAdjustment: -7, // 30 - 7 = 23 pills
+				lastStockCorrectionAt: correctionTime.toISOString(),
+				takenBy: [],
+				blisters: [
+					{
+						usage: 1,
+						every: 1,
+						start: "2024-03-01T08:00:00",
+					},
+				],
+				updatedAt: correctionTime.toISOString(),
+			},
+		];
+
+		// User has tracked a dose yesterday (before the correction)
+		const takenDoses = new Set([`1-0-${march14}`]);
+
+		const result = calculateCoverage(meds, [], "en", 7, "automatic", takenDoses);
+
+		expect(result.all).toHaveLength(1);
+		// getMedTotal = 30 - 7 = 23.
+		// The taken dose from yesterday should NOT be counted (it's before the correction).
+		// No new doses should exist since the correction just happened.
+		expect(result.all[0].medsLeft).toBe(23);
+	});
+
+	it("stock correction consumption resumes after one full period", () => {
+		// After 1 day (for daily medication), the next dose should be consumed.
+		// Set system time to 1 day + 1 hour after correction.
+		const correctionTime = new Date("2024-03-14T12:00:00Z");
+		vi.setSystemTime(new Date("2024-03-15T13:00:00Z")); // 25 hours after correction
+
+		const meds: Medication[] = [
+			{
+				id: 1,
+				name: "TestMed",
+				packCount: 1,
+				blistersPerPack: 1,
+				pillsPerBlister: 30,
+				looseTablets: 0,
+				stockAdjustment: -7, // 30 - 7 = 23 pills
+				lastStockCorrectionAt: correctionTime.toISOString(),
+				takenBy: [],
+				blisters: [
+					{
+						usage: 1,
+						every: 1,
+						start: "2024-03-01T08:00:00",
+					},
+				],
+				updatedAt: correctionTime.toISOString(),
+			},
+		];
+
+		const result = calculateCoverage(meds, [], "en", 7, "automatic", new Set());
+
+		expect(result.all).toHaveLength(1);
+		// After 1 full period since correction, 1 dose should be consumed.
+		// medsLeft = 23 - 1 = 22
+		expect(result.all[0].medsLeft).toBe(22);
+	});
 });

 describe("getStockStatus", () => {
@@ -1147,3 +1265,300 @@ function groupEventsIntoPastDays(
 		meds: Array.from(medMap.entries()).map(([medName, doses]) => ({ medName, doses })),
 	}));
 }
+
+describe("expandDoseIds", () => {
+	it("returns base IDs when takenBy is empty array", () => {
+		const doses = [
+			{ id: "1-0-1729123200000", takenBy: [] as string[] },
+			{ id: "2-0-1729123200000", takenBy: [] as string[] },
+		];
+		const result = expandDoseIds(doses);
+		expect(result).toEqual(["1-0-1729123200000", "2-0-1729123200000"]);
+	});
+
+	it("returns person-suffixed IDs when takenBy has entries", () => {
+		const doses = [{ id: "1-0-1729123200000", takenBy: ["Alice"] }];
+		const result = expandDoseIds(doses);
+		expect(result).toEqual(["1-0-1729123200000-Alice"]);
+	});
+
+	it("returns multiple IDs for multiple takenBy entries", () => {
+		const doses = [{ id: "1-0-1729123200000", takenBy: ["Alice", "Bob"] }];
+		const result = expandDoseIds(doses);
+		expect(result).toEqual(["1-0-1729123200000-Alice", "1-0-1729123200000-Bob"]);
+	});
+
+	it("handles mix of empty and non-empty takenBy", () => {
+		const doses = [
+			{ id: "1-0-1729123200000", takenBy: ["Alice"] },
+			{ id: "2-0-1729123200000", takenBy: [] as string[] },
+			{ id: "3-0-1729123200000", takenBy: ["Bob", "Carol"] },
+		];
+		const result = expandDoseIds(doses);
+		expect(result).toEqual([
+			"1-0-1729123200000-Alice",
+			"2-0-1729123200000",
+			"3-0-1729123200000-Bob",
+			"3-0-1729123200000-Carol",
+		]);
+	});
+
+	it("returns empty array for empty doses", () => {
+		expect(expandDoseIds([])).toEqual([]);
+	});
+
+	it("handles non-array takenBy gracefully", () => {
+		// In case of unexpected data, treat non-array as empty
+		const doses = [{ id: "1-0-1729123200000", takenBy: null as unknown as string[] }];
+		const result = expandDoseIds(doses);
+		expect(result).toEqual(["1-0-1729123200000"]);
+	});
+});
+
+describe("past schedule windowing", () => {
+	// Reproduces the bug where daily medications with far-past start dates
+	// would fill all 2000 event slots, pushing today/future events out.
+	// The fix replaces slice(0, 2000) with a time-based window filter.
+
+	function buildGroupedScheduleFromEvents(
+		events: Array<{
+			dateStr: string;
+			when: number;
+			isPast: boolean;
+			medName: string;
+			id: string;
+			usage: number;
+			takenBy: string | null;
+		}>,
+		scheduleDays: number
+	) {
+		// Mirror the fixed groupedSchedule logic from AppContext.tsx
+		const pastCutoff = new Date();
+		pastCutoff.setDate(pastCutoff.getDate() - scheduleDays);
+		pastCutoff.setHours(0, 0, 0, 0);
+		const pastCutoffMs = pastCutoff.getTime();
+
+		type DayMedEntry = {
+			medName: string;
+			total: number;
+			doses: Array<{ id: string; takenBy: string[] }>;
+			lastWhen: number;
+		};
+		const days = new Map<string, { dateStr: string; date: Date; isPast: boolean; meds: Map<string, DayMedEntry> }>();
+
+		events
+			.filter((e) => !e.isPast || e.when >= pastCutoffMs)
+			.forEach((event) => {
+				const day = days.get(event.dateStr) ?? {
+					dateStr: event.dateStr,
+					date: new Date(event.when),
+					isPast: event.isPast,
+					meds: new Map(),
+				};
+				const medEntry = day.meds.get(event.medName) ?? {
+					medName: event.medName,
+					total: 0,
+					doses: [],
+					lastWhen: event.when,
+				};
+				medEntry.total += event.usage;
+				medEntry.doses.push({
+					id: event.id,
+					takenBy: event.takenBy ? [event.takenBy] : [],
+				});
+				medEntry.lastWhen = Math.max(medEntry.lastWhen, event.when);
+				day.meds.set(event.medName, medEntry);
+				days.set(event.dateStr, day);
+			});
+
+		return Array.from(days.values()).map((d) => ({
+			dateStr: d.dateStr,
+			date: d.date,
+			isPast: d.isPast,
+			meds: Array.from(d.meds.values()),
+		}));
+	}
+
+	it("includes daily meds within the scheduleDays window, not just weekly meds", () => {
+		const now = new Date();
+		const todayStart = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+		const events: Array<{
+			dateStr: string;
+			when: number;
+			isPast: boolean;
+			medName: string;
+			id: string;
+			usage: number;
+			takenBy: string | null;
+		}> = [];
+
+		// Simulate daily med starting 400 days ago (way beyond any window)
+		for (let i = 400; i >= 1; i--) {
+			const d = new Date(todayStart);
+			d.setDate(d.getDate() - i);
+			events.push({
+				dateStr: d.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+				when: d.getTime(),
+				isPast: true,
+				medName: "DailyMed",
+				id: `1-0-${d.getTime()}`,
+				usage: 1,
+				takenBy: "Daniel",
+			});
+		}
+
+		// Simulate weekly Friday med starting 400 days ago
+		for (let i = 400; i >= 1; i--) {
+			const d = new Date(todayStart);
+			d.setDate(d.getDate() - i);
+			if (d.getDay() !== 5) continue; // Only Fridays
+			events.push({
+				dateStr: d.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+				when: d.getTime(),
+				isPast: true,
+				medName: "WeeklyMed",
+				id: `2-0-${d.getTime()}`,
+				usage: 1,
+				takenBy: null,
+			});
+		}
+
+		// Add today event
+		events.push({
+			dateStr: todayStart.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+			when: todayStart.getTime(),
+			isPast: false,
+			medName: "DailyMed",
+			id: `1-0-${todayStart.getTime()}`,
+			usage: 1,
+			takenBy: "Daniel",
+		});
+
+		events.sort((a, b) => a.when - b.when);
+
+		const grouped = buildGroupedScheduleFromEvents(events, 30);
+		const pastDays = grouped.filter((d) => d.isPast);
+		const todayDays = grouped.filter((d) => !d.isPast);
+
+		// Past days should contain at most 30 days (scheduleDays window)
+		expect(pastDays.length).toBeLessThanOrEqual(30);
+		expect(pastDays.length).toBeGreaterThan(0);
+
+		// Today should be present (not pushed out by past events)
+		expect(todayDays.length).toBeGreaterThanOrEqual(1);
+
+		// Past days should include DailyMed (within the 30-day window)
+		const pastDaysWithDailyMed = pastDays.filter((d) => d.meds.some((m) => m.medName === "DailyMed"));
+		expect(pastDaysWithDailyMed.length).toBeGreaterThan(0);
+
+		// Past days should NOT only be Fridays (the bug)
+		const pastDateDays = pastDays.map((d) => d.date.getDay());
+		const uniqueDaysOfWeek = new Set(pastDateDays);
+		expect(uniqueDaysOfWeek.size).toBeGreaterThan(1); // Multiple days of week, not just Friday
+	});
+
+	it("old slice(0,2000) would have cut off today for large datasets", () => {
+		const now = new Date();
+		const todayStart = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+		const events: Array<{
+			dateStr: string;
+			when: number;
+			isPast: boolean;
+			medName: string;
+			id: string;
+			usage: number;
+			takenBy: string | null;
+		}> = [];
+
+		// Generate 3 daily meds × 2 intakes × 400 days = 2400 past events (> 2000 limit)
+		for (let medIdx = 0; medIdx < 3; medIdx++) {
+			for (let intakeIdx = 0; intakeIdx < 2; intakeIdx++) {
+				for (let i = 400; i >= 1; i--) {
+					const d = new Date(todayStart);
+					d.setDate(d.getDate() - i);
+					events.push({
+						dateStr: d.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+						when: d.getTime() + intakeIdx * 3600000, // offset intakes by 1 hour
+						isPast: true,
+						medName: `Med${medIdx}`,
+						id: `${medIdx}-${intakeIdx}-${d.getTime()}`,
+						usage: 1,
+						takenBy: null,
+					});
+				}
+			}
+			// Today event
+			events.push({
+				dateStr: todayStart.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+				when: todayStart.getTime(),
+				isPast: false,
+				medName: `Med${medIdx}`,
+				id: `${medIdx}-0-${todayStart.getTime()}`,
+				usage: 1,
+				takenBy: null,
+			});
+		}
+
+		events.sort((a, b) => a.when - b.when);
+
+		// OLD behavior: slice(0, 2000) would have cut off today
+		const oldSliced = events.slice(0, 2000);
+		const todayInOld = oldSliced.filter((e) => !e.isPast);
+		expect(todayInOld.length).toBe(0); // Today events are gone!
+
+		// NEW behavior: time-based window keeps today
+		const grouped = buildGroupedScheduleFromEvents(events, 30);
+		const todayDays = grouped.filter((d) => !d.isPast);
+		expect(todayDays.length).toBeGreaterThanOrEqual(1);
+		// All 3 meds should appear in today
+		const todayMeds = todayDays[0]?.meds.map((m) => m.medName).sort();
+		expect(todayMeds).toEqual(["Med0", "Med1", "Med2"]);
+	});
+
+	it("respects scheduleDays parameter for past window size", () => {
+		const now = new Date();
+		const todayStart = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+		const events: Array<{
+			dateStr: string;
+			when: number;
+			isPast: boolean;
+			medName: string;
+			id: string;
+			usage: number;
+			takenBy: string | null;
+		}> = [];
+
+		// Daily med for 200 days
+		for (let i = 200; i >= 1; i--) {
+			const d = new Date(todayStart);
+			d.setDate(d.getDate() - i);
+			events.push({
+				dateStr: d.toLocaleDateString("en", { weekday: "short", day: "2-digit", month: "short" }),
+				when: d.getTime(),
+				isPast: true,
+				medName: "TestMed",
+				id: `1-0-${d.getTime()}`,
+				usage: 1,
+				takenBy: null,
+			});
+		}
+		events.sort((a, b) => a.when - b.when);
+
+		// With 30 days window
+		const grouped30 = buildGroupedScheduleFromEvents(events, 30);
+		const past30 = grouped30.filter((d) => d.isPast);
+		expect(past30.length).toBeLessThanOrEqual(30);
+
+		// With 90 days window
+		const grouped90 = buildGroupedScheduleFromEvents(events, 90);
+		const past90 = grouped90.filter((d) => d.isPast);
+		expect(past90.length).toBeLessThanOrEqual(90);
+		expect(past90.length).toBeGreaterThan(past30.length);
+
+		// With 180 days window
+		const grouped180 = buildGroupedScheduleFromEvents(events, 180);
+		const past180 = grouped180.filter((d) => d.isPast);
+		expect(past180.length).toBeLessThanOrEqual(180);
+		expect(past180.length).toBeGreaterThan(past90.length);
+	});
+});
@@ -126,9 +126,18 @@ export function calculateCoverage(
 			// but also account for manual corrections (doses marked as not taken)
 			blisters.forEach((s, blisterIdx) => {
 				const blisterStart = new Date(s.start).getTime();
-				const effectiveStart = Math.max(blisterStart, stockCorrectionCutoff);
-				if (Number.isNaN(effectiveStart) || effectiveStart > now) return;
 				const period = Math.max(1, s.every) * MS_PER_DAY;
+
+				// After a stock correction, start counting consumption from the NEXT
+				// scheduled dose, because the user's pill count already reflects all
+				// consumption up to the correction time.
+				let effectiveStart: number;
+				if (stockCorrectionCutoff > 0 && stockCorrectionCutoff >= blisterStart) {
+					effectiveStart = stockCorrectionCutoff + period;
+				} else {
+					effectiveStart = blisterStart;
+				}
+				if (Number.isNaN(effectiveStart) || effectiveStart > now) return;
 				const occurrences = Math.floor((now - effectiveStart) / period) + 1;
 				const intake = intakes[blisterIdx];
 				const intakePerson = intake?.takenBy;
@@ -446,6 +455,18 @@ export function isDoseDismissed(doseId: string, dismissedUntilDate: string | und
 * @param dismissedDoses - Set of dose IDs individually dismissed
 * @returns Array of dose IDs that are missed
 */
+/**
+ * Compute the full set of dose IDs for a list of doses, correctly handling
+ * per-intake takenBy arrays. Empty arrays produce base IDs (no suffix),
+ * non-empty arrays produce one ID per person with a `-person` suffix.
+ */
+export function expandDoseIds(doses: ReadonlyArray<{ id: string; takenBy: string[] }>): string[] {
+	return doses.flatMap((d) => {
+		const takenByArray = Array.isArray(d.takenBy) ? d.takenBy : [];
+		return takenByArray.length > 0 ? takenByArray.map((p) => `${d.id}-${p}`) : [d.id];
+	});
+}
+
 export function computeMissedPastDoseIds(
 	pastDays: ReadonlyArray<{
 		meds: ReadonlyArray<{
Author	SHA1	Message	Date
Daniel Volz	e55e415a50	chore: release v1.8.6 (#129 )	2026-02-08 14:06:03 +01:00
Daniel Volz	5253d14af7	fix: make frontend image self-contained for read-only filesystems (#128 ) Revert Dockerfile to use /tmp redirect for envsubst output, so the image works regardless of docker-compose.yml tmpfs configuration. Removes the uid=101,gid=101 requirement from compose that was a breaking change.	2026-02-08 14:03:53 +01:00
github-actions[bot]	4f75d78a2b	chore: update test count badges [skip ci]	2026-02-08 12:54:19 +00:00
Daniel Volz	8f9b65147b	fix: use PAT for badge workflow to bypass branch protection (#127 )	2026-02-08 13:53:19 +01:00
Daniel Volz	571ab00918	chore: release v1.8.5 (#126 )	2026-02-08 13:35:52 +01:00
Daniel Volz	27f5478dad	fix: clean up nginx read-only filesystem approach (#125 ) Remove Dockerfile /tmp workaround hacks (NGINX_ENVSUBST_OUTPUT_DIR and sed). Use tmpfs with uid=101,gid=101 in docker-compose.yml instead, so the nginx user can write to /etc/nginx/conf.d directly under read_only: true.	2026-02-08 13:33:40 +01:00
Daniel Volz	5cd519be50	chore: release v1.8.4 (#124 )	2026-02-08 13:12:58 +01:00
Daniel Volz	e0c5eb4bf3	feat: simplify About modal with single version link to GitHub release (#123 ) - Replace separate Frontend/Backend versions with single app version - Version is now a clickable link to the GitHub release page - Replace stopwatch SVG with actual app logo (favicon.svg) - Fix update check UX: previous result stays visible during re-check - Add 1s minimum delay for update check spinner visibility - Reserve space for update result to prevent modal jumping - Remove unused i18n keys (frontend/backend) - Update release-manager docs with version link info	2026-02-08 13:09:33 +01:00
Daniel Volz	aa92bcd96d	fix: nginx read_only filesystem compatibility for envsubst (#122 ) Redirect NGINX_ENVSUBST_OUTPUT_DIR to /tmp and update nginx.conf include path so envsubst works with read_only: true in docker-compose. Add tmpfs mount for /etc/nginx/conf.d for additional write layer.	2026-02-08 13:07:21 +01:00
Daniel Volz	1798a608bc	fix: badge workflow commits directly instead of creating PRs (#121 ) * fix: badge workflow commits directly instead of creating PRs Replace peter-evans/create-pull-request with direct git push. Removes need for pull-requests:write permission and the repo setting 'Allow GitHub Actions to create pull requests'. Uses [skip ci] in commit message to avoid triggering itself. * chore: trigger CI	2026-02-08 12:25:33 +01:00
Daniel Volz	2ec9db1c13	chore: release v1.8.3 (#120 )	2026-02-08 12:09:52 +01:00
Daniel Volz	042f0cfb29	docs: add version files reminder to release manager agent (#119 ) Document that both backend/package.json and frontend/package.json must be updated before tagging a release, since the About modal reads versions from these files.	2026-02-08 12:06:20 +01:00
Daniel Volz	78a0d3ac8e	fix: use dynamic BACKEND_URL for nginx reverse proxy (#118 ) Fixes #96 - nginx.conf converted to template processed by envsubst at container start - BACKEND_URL env var (default: backend:3000) replaces hardcoded container name - Docker DNS resolver used for dynamic upstream resolution - Dockerfile copies nginx.conf as template to /etc/nginx/templates/ This prevents frontend breakage when users customize container names in their docker-compose.yml.	2026-02-08 12:05:43 +01:00
Daniel Volz	7d6664e684	fix: auto-detect data directory in monorepo without DATA_DIR env var (#117 ) - getDataDir() now detects monorepo by checking for ../docker-compose.yml - DATA_DIR env var removed from .env and .env.example (no longer needed for local dev) - Docker compose files explicitly set DATA_DIR=/app/data for containers - Updated tests for monorepo detection logic	2026-02-08 12:04:09 +01:00
Daniel Volz	2a84a43654	fix: unify data directory for dev and prod environments (#116 ) Add DATA_DIR env var support to configure the data directory path. All hardcoded resolve(cwd, 'data') paths now use a central getDataDir() function from db-utils.ts that checks DATA_DIR first, falling back to resolve(cwd, 'data'). This prevents local dev (cd backend && npm run dev) from creating a separate backend/data/ directory instead of using the root data/ folder. Changes: - Add getDataDir() to db-utils.ts as single source of truth - Update all 8 source files that reference the data directory - Add dotenv fallback to ../.env for local dev from backend/ - Add DATA_DIR documentation to .env.example - Add 7 new tests for getDataDir and getDbPaths with DATA_DIR - 493 tests pass, TypeScript clean	2026-02-08 11:20:55 +01:00
Daniel Volz	99bb9c3931	fix: backend planner phantom consumption + PUT stock reset (#115 ) Two bugs in the backend medications route: 1. Planner /medications/usage had the same +1 phantom consumption bug that was fixed in the frontend (PR #109). After a stock correction, effectiveStart was set to max(blisterStart, correctionCutoff) instead of correctionCutoff + period, causing 1 dose to be immediately counted as consumed. 2. PUT /medications/:id did not reset stockAdjustment when stock fields (packCount, blistersPerPack, pillsPerBlister, looseTablets) changed. If a user edited stock values to correct their inventory, the old stockAdjustment offset was preserved, resulting in wrong totals. Added 4 tests covering both scenarios.	2026-02-08 11:05:56 +01:00
Daniel Volz	6b3a7b4104	fix: prevent tests from creating stale backend/data directory (#112 ) Extract DB utility functions (buildDbUrl, getDbPaths, ensureDataDirectory, runAlterMigrations, etc.) from client.ts into db-utils.ts. client.ts contained top-level initialization code (ensureDataDirectory, createClient) that ran on every import. database.test.ts imported utility functions from client.ts, which triggered the initialization as a side effect — creating backend/data/ with a .write-test file and medassist-ng.db every time tests ran. Now database.test.ts imports from db-utils.ts (side-effect-free), and client.ts re-exports everything for backward compatibility.	2026-02-07 14:14:10 +01:00
Daniel Volz	2d9cd0ad1a	ci: add path filtering to skip unnecessary CI runs (#111 ) test.yml: Use dorny/paths-filter to detect changed paths. Backend tests only run when backend/, biome.json, or the workflow itself changes. Frontend build only runs when frontend/, biome.json, or the workflow changes. Jobs skipped via job-level 'if:' are treated as passed by GitHub required checks. codeql.yml: Only run on push/PR when JS/TS source files, package files, or CodeQL config changes. Weekly schedule and manual dispatch remain unfiltered.	2026-02-07 13:37:13 +01:00
Daniel Volz	098a7655a5	chore: add release-manager agent and move release docs (#110 ) Create dedicated GitHub Copilot agent for release management with 4 tasks: branch/PR workflow, version determination, release execution, and release notes writing. Move release-specific instructions (workflow, release notes format, breaking changes) from copilot-instructions.md to the agent file to keep concerns separated.	2026-02-07 13:32:50 +01:00
Daniel Volz	f73c79c6cf	fix: stock correction no longer neutralized by phantom consumption (#109 ) After correcting medication stock, the coverage calculation immediately counted 1 dose as consumed (due to +1 in occurrences formula), which neutralized small corrections like +1 pill. Fix: start consumption counting from stockCorrectionCutoff + period (the next scheduled dose) instead of from the correction time itself. Added 3 frontend tests for stock correction scenarios and 6 backend e2e tests for the PATCH /medications/:id/stock-adjustment endpoint.	2026-02-07 13:30:44 +01:00
Daniel Volz	06943f5831	fix: add pull-requests write permission to badge workflow (#108 ) peter-evans/create-pull-request requires pull-requests: write permission to create PRs via the GITHUB_TOKEN.	2026-02-07 12:41:26 +01:00
Daniel Volz	73b3eb6686	fix: replace event count limit with time-based window for past schedule (#107 ) The groupedSchedule useMemo used slice(0, 2000) to limit events. With daily medications having start dates far in the past, thousands of past events would fill all 2000 slots, pushing today and future events completely out of the display. This caused the past schedule to only show weekly medications (fewer events) while daily medications appeared missing. Replace the fixed count limit with a time-based window: only past events within the scheduleDays window (30/90/180 days) are included. All today and future events are always included regardless. Coverage calculations are not affected as they use schedule.events directly.	2026-02-07 00:35:14 +01:00
Daniel Volz	a4313afc34	fix: use PR instead of direct push for badge updates (#106 ) Branch protection prevents direct pushes to main. Use peter-evans/create-pull-request action instead.	2026-02-07 00:15:05 +01:00
Daniel Volz	690cb2ff74	fix: correct dose ID generation for empty takenBy arrays (#105 ) The takenBy field is a string[]. Empty arrays [] are truthy in JavaScript, causing d.takenBy ? [...] patterns to generate dose IDs with trailing hyphens (e.g., '5-0-173...-') instead of base IDs ('5-0-173...'). This mismatch between ID generation and computeMissedPastDoseIds (which correctly uses .length > 0) caused doses to always appear as missed. Changes: - Add expandDoseIds() helper function using correct .length > 0 check - Replace 8 buggy inline patterns in DashboardPage.tsx - Refactor SchedulePage.tsx to use shared expandDoseIds() - Add backend startup repair to strip trailing hyphens from existing IDs - Add 12 new tests (6 frontend + 6 backend)	2026-02-07 00:08:58 +01:00
Daniel Volz	21127b38ab	fix: repair orphaned dose tracking IDs on startup (#104 ) Add repairOrphanedDoseIds() function that runs during app startup (after ALTER migrations) to fix dose tracking entries that became invalid when medication schedules were changed before PR #103. The function: - Generates valid schedule dates for each medication's current intakes - Detects dose_tracking entries whose dateOnlyMs doesn't match any valid schedule date - Remaps orphaned doses to the nearest valid schedule date within half the intake interval - Preserves person suffixes in dose IDs - Is idempotent (safe to run on every startup) This complements PR #103 which only migrates dose IDs on future edits. The startup repair fixes existing broken data in production databases. Includes 8 tests covering: valid doses unchanged, 1-day shift repair, person suffix preservation, out-of-range detection, idempotency, multi-medication handling, and legacy format fallback.	2026-02-06 22:59:40 +01:00
Daniel Volz	f5f189e0a4	fix: migrate dose tracking IDs when intake schedule changes (#103 ) When a medication's start date or interval changes, the generated dose IDs shift (dateOnlyMs values change). Previously, doses marked as taken under the old schedule were orphaned — they no longer matched the new schedule's dose IDs, causing them to appear as missed. Now the PUT /medications/:id endpoint: 1. Parses old intakes from the existing medication row 2. Detects which intake indices had schedule changes 3. Maps old dateOnlyMs values to the nearest new dateOnlyMs 4. Updates dose_tracking entries with the migrated IDs 5. Preserves person suffixes (e.g. -Alice) during migration Also fixes the start-date cleanup to use date-only comparison, preventing doses on the start date from being incorrectly deleted when the start time is after midnight. Adds 4 integration tests covering weekly day shift, person suffix preservation, time-only changes, and interval changes.	2026-02-06 22:38:28 +01:00
Daniel Volz	43c5402592	fix: add workflow_dispatch trigger to test badge workflow (#102 ) Allows manual triggering of the badge update workflow, useful when the ANSI fix or other workflow-only changes need to take effect without waiting for source code changes.	2026-02-06 22:27:01 +01:00
Daniel Volz	02bae889b4	fix: strip ANSI escape codes in test badge workflow (#101 ) Vitest 4 outputs ANSI color codes in test results, which caused the grep regex to fail when extracting test counts. The badge workflow silently skipped the update, leaving stale counts in the README. Add a sed pass to strip ANSI escape sequences before parsing.	2026-02-06 22:24:09 +01:00