ICD-10 microservice, Docker stack, and CI improvements

.claude/skills/build/SKILL.md

@@ -0,0 +1,41 @@
+---
+name: build
+description: Build the DataProvider .NET solution or specific projects. Use when asked to build, compile, or check for compilation errors.
+disable-model-invocation: true
+allowed-tools: Bash(dotnet build *)
+---
+
+# Build
+
+Build the entire solution or a specific project.
+
+## Full solution
+
+```bash
+dotnet build /Users/christianfindlay/Documents/Code/DataProvider/DataProvider.sln
+```
+
+## Specific project
+
+If `$ARGUMENTS` names a component, build only that project:
+
+| Argument | Project path |
+|----------|-------------|
+| dataprovider | DataProvider/DataProvider/DataProvider.csproj |
+| dataprovider-sqlite | DataProvider/DataProvider.SQLite/DataProvider.SQLite.csproj |
+| sqlite-cli | DataProvider/DataProvider.SQLite.Cli/DataProvider.SQLite.Cli.csproj |
+| migration | Migration/Migration.Cli/Migration.Cli.csproj |
+| lql | Lql/Lql/Lql.csproj |
+| sync | Sync/Sync/Sync.csproj |
+| gatekeeper | Gatekeeper/Gatekeeper.Api/Gatekeeper.Api.csproj |
+| clinical | Samples/Clinical/Clinical.Api/Clinical.Api.csproj |
+| scheduling | Samples/Scheduling/Scheduling.Api/Scheduling.Api.csproj |
+| icd10 | Samples/ICD10/ICD10.Api/ICD10.Api.csproj |
+
+If no argument is provided, build the full solution.
+
+## Notes
+
+- Generated `.g.cs` files are in `.gitignore` and must be generated at build time
+- MSBuild targets in sample projects handle code generation automatically
+- If stale `Generated/` folders cause issues, delete them to force regeneration

.claude/skills/container-logs/SKILL.md

@@ -0,0 +1,48 @@
+---
+name: container-logs
+description: View Docker container logs for the Healthcare Samples stack. Use when asked to check logs, debug container issues, or see service output.
+disable-model-invocation: true
+allowed-tools: Bash(docker compose *), Bash(docker logs *)
+argument-hint: "[container-name] [--tail N]"
+---
+
+# Container Logs
+
+View logs from the Healthcare Samples Docker stack.
+
+## Usage
+
+`/container-logs` - show recent logs from all containers
+`/container-logs app` - show logs from the app container
+`/container-logs db` - show logs from the Postgres container
+
+## Commands
+
+All logs (last 100 lines):
+```bash
+docker compose -f /Users/christianfindlay/Documents/Code/DataProvider/Samples/docker/docker-compose.yml logs --tail 100
+```
+
+Specific container:
+```bash
+docker compose -f /Users/christianfindlay/Documents/Code/DataProvider/Samples/docker/docker-compose.yml logs --tail 100 $ARGUMENTS
+```
+
+Follow logs in real-time (use timeout to avoid hanging):
+```bash
+timeout 10 docker compose -f /Users/christianfindlay/Documents/Code/DataProvider/Samples/docker/docker-compose.yml logs -f $ARGUMENTS
+```
+
+## Container names
+
+| Name | Service |
+|------|---------|
+| app | All .NET APIs + embedding service |
+| db | Postgres 16 + pgvector |
+| dashboard | nginx serving static files |
+
+## Check container status
+
+```bash
+docker compose -f /Users/christianfindlay/Documents/Code/DataProvider/Samples/docker/docker-compose.yml ps
+```

.claude/skills/format/SKILL.md

@@ -0,0 +1,20 @@
+---
+name: format
+description: Format C# code with CSharpier. Use when asked to format code, fix formatting, or before committing changes.
+disable-model-invocation: true
+allowed-tools: Bash(dotnet csharpier *)
+---
+
+# Format
+
+Run CSharpier to format all C# code in the repository.
+
+```bash
+dotnet csharpier /Users/christianfindlay/Documents/Code/DataProvider
+```
+
+If formatting a specific directory:
+
+```bash
+dotnet csharpier /Users/christianfindlay/Documents/Code/DataProvider/$ARGUMENTS
+```

.claude/skills/migrate/SKILL.md

@@ -0,0 +1,38 @@
+---
+name: migrate
+description: Run database migrations using the Migration CLI with YAML schemas. Use when asked to create databases, run migrations, or set up schema.
+disable-model-invocation: true
+allowed-tools: Bash(dotnet run --project *Migration*)
+argument-hint: "[schema.yaml] [output.db] [provider]"
+---
+
+# Migrate
+
+Run the Migration CLI to create or update databases from YAML schema files.
+
+## Usage
+
+`/migrate` - show help
+`/migrate icd10` - create ICD10 SQLite database from its schema
+
+## Shortcuts
+
+| Argument | Schema | Output | Provider |
+|----------|--------|--------|----------|
+| icd10 | Samples/ICD10/ICD10.Api/icd10-schema.yaml | Samples/ICD10/ICD10.Api/icd10.db | sqlite |
+
+## Manual usage
+
+```bash
+dotnet run --project /Users/christianfindlay/Documents/Code/DataProvider/Migration/Migration.Cli -- \
+    --schema <path-to-schema.yaml> \
+    --output <output-path> \
+    --provider <sqlite|postgres>
+```
+
+## Notes
+
+- YAML schemas are the ONLY valid way to define database schema (raw SQL DDL is ILLEGAL)
+- Schema files live alongside their API projects
+- Supported providers: `sqlite`, `postgres`
+- The Migration CLI converts YAML to SQL DDL and applies it

.claude/skills/run-samples/SKILL.md

@@ -0,0 +1,52 @@
+---
+name: run-samples
+description: Start the Healthcare Samples stack (Postgres, APIs, Dashboard). Use when asked to run, start, or launch the sample applications.
+---
+
+# Run Samples
+
+Start the full Healthcare Samples stack. Decide based on `$ARGUMENTS`:
+
+IMPORTANT: Do NOT run in the background. Run in the foreground so the user can see all output streaming in real-time. Set a long timeout (600000ms).
+
+## Default (no args) - keep existing data
+
+Run with existing database volumes intact:
+
+```bash
+cd /Users/christianfindlay/Documents/Code/DataProvider/Samples/scripts && ./start.sh
+```
+
+## Fresh start - blow away databases
+
+If the user says "fresh", "clean", "reset", or `$ARGUMENTS` contains `--fresh`:
+
+```bash
+cd /Users/christianfindlay/Documents/Code/DataProvider/Samples/scripts && ./start.sh --fresh
+```
+
+## Force rebuild containers
+
+If the user says "rebuild" or `$ARGUMENTS` contains `--build`:
+
+```bash
+cd /Users/christianfindlay/Documents/Code/DataProvider/Samples/scripts && ./start.sh --build
+```
+
+## Both fresh + rebuild
+
+```bash
+cd /Users/christianfindlay/Documents/Code/DataProvider/Samples/scripts && ./start.sh --fresh --build
+```
+
+## Services
+
+| Service | Port |
+|---------|------|
+| Gatekeeper API | 5002 |
+| Clinical API | 5080 |
+| Scheduling API | 5001 |
+| ICD10 API | 5090 |
+| Embedding Service | 8000 |
+| Dashboard | 5173 |
+| Postgres | 5432 |

.claude/skills/submit-pr/SKILL.md

@@ -0,0 +1,54 @@
+---
+name: submit-pr
+description: Submit a pull request following DataProvider project standards
+---
+
+# Submit Pull Request
+
+Create a pull request following project requirements.
+
+## Get Context
+
+Get the diff between main and current branch:
+
+```bash
+git diff main...HEAD
+```
+
+DO NOT include commit messages or branch names in analysis.
+
+Read the PR template:
+
+```bash
+cat .github/PULL_REQUEST_TEMPLATE.md
+```
+
+## Write PR Description
+
+The template has three sections (gh will auto-populate structure):
+
+### TLDR
+- Few lines maximum
+- Bullet points if many changes
+- For people who won't read details
+
+### Brief Details
+- Keep BRIEF
+- May reference code/files
+- What changed and why
+
+### How Do The Tests Prove This Works? (CRITICAL)
+- Point to specific test files/methods
+- Explain WHAT each test verifies
+- Show HOW tests prove correctness, not just "tests added"
+
+## Requirements
+
+- TIGHT - no fluff
+- ACCURATE - based on actual diff
+
+## Submit
+
+```bash
+gh pr create
+```

.claude/skills/test/SKILL.md

@@ -0,0 +1,58 @@
+---
+name: test
+description: Run tests for the DataProvider solution or specific test projects. Use when asked to run tests, verify changes, or check test results.
+disable-model-invocation: true
+allowed-tools: Bash(dotnet test *)
+argument-hint: "[component|project-path]"
+---
+
+# Test
+
+Run tests for a specific component or the full solution.
+
+## Usage
+
+`/test` - run all tests
+`/test dataprovider` - run DataProvider tests
+`/test icd10` - run ICD10 API tests
+
+## Test projects by component
+
+| Argument | Test project path |
+|----------|------------------|
+| dataprovider | DataProvider/DataProvider.Tests |
+| dataprovider-example | DataProvider/DataProvider.Example.Tests |
+| lql | Lql/Lql.Tests |
+| lql-cli | Lql/LqlCli.SQLite.Tests |
+| migration | Migration/Migration.Tests |
+| sync | Sync/Sync.Tests |
+| sync-sqlite | Sync/Sync.SQLite.Tests |
+| sync-postgres | Sync/Sync.Postgres.Tests |
+| sync-http | Sync/Sync.Http.Tests |
+| sync-integration | Sync/Sync.Integration.Tests |
+| gatekeeper | Gatekeeper/Gatekeeper.Api.Tests |
+| clinical | Samples/Clinical/Clinical.Api.Tests |
+| scheduling | Samples/Scheduling/Scheduling.Api.Tests |
+| icd10 | Samples/ICD10/ICD10.Api.Tests |
+| icd10-cli | Samples/ICD10/ICD10.Cli.Tests |
+| dashboard | Samples/Dashboard/Dashboard.Integration.Tests |
+
+## Commands
+
+Run a specific test project:
+```bash
+dotnet test <project-path> --no-restore --verbosity normal
+```
+
+Run all tests in the solution:
+```bash
+dotnet test /Users/christianfindlay/Documents/Code/DataProvider/DataProvider.sln --no-restore --verbosity normal
+```
+
+## Notes
+
+- Tests use xUnit 2.9.2
+- Coverage config: `coverlet.runsettings`
+- Sync and Gatekeeper tests require a running Postgres instance
+- Dashboard tests use Playwright (E2E)
+- NEVER skip tests - failing tests are OK, skipped tests are ILLEGAL

.config/dotnet-tools.json

@@ -10,7 +10,7 @@
       "rollForward": false
     },
     "h5-compiler": {
-      "version": "24.11.53871",
+      "version": "26.3.64893",
       "commands": [
         "h5"
       ],