dernbu/lawnet-scraper
1
0
Fork 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

[3/4] Scaffold web service #14

Merged
dernbu merged 2 commits from feat/1-web-scaffold into main 2026-06-13 20:57:05 +02:00
Conversation 2 Commits 2 Files changed 18 +4918
dernbu commented 2026-06-13 19:44:30 +02:00
Owner
Copy link

Summary

  • add the Node.js 22/pnpm package root and production Docker image
  • add a minimal TanStack Start application
  • add /health reporting process, PostgreSQL, and Flyway readiness
  • add ESLint, Prettier, strict TypeScript, Vitest, and a focused health test

Size

445 handwritten/generated-route lines, plus the required generated 4,544-line pnpm-lock.yaml. The lockfile is the explicit generated-file exception to the review LOC cap.

Verification

  • pnpm install --frozen-lockfile
  • pnpm lint
  • pnpm typecheck
  • pnpm test (1 passed)
  • pnpm build
  • docker build .

Stack 3 of 4. Depends on and targets feat/1-flyway-scaffold; merge the preceding PRs first.

## Summary - add the Node.js 22/pnpm package root and production Docker image - add a minimal TanStack Start application - add `/health` reporting process, PostgreSQL, and Flyway readiness - add ESLint, Prettier, strict TypeScript, Vitest, and a focused health test ## Size 445 handwritten/generated-route lines, plus the required generated 4,544-line `pnpm-lock.yaml`. The lockfile is the explicit generated-file exception to the review LOC cap. ## Verification - `pnpm install --frozen-lockfile` - `pnpm lint` - `pnpm typecheck` - `pnpm test` (1 passed) - `pnpm build` - `docker build .` Stack 3 of 4. Depends on and targets `feat/1-flyway-scaffold`; merge the preceding PRs first.
dernbu force-pushed feat/1-web-scaffold from ce2e41cfb3 to 2f53be70a0 2026-06-13 20:05:47 +02:00 Compare
dernbu changed target branch from feat/1-flyway-scaffold to main 2026-06-13 20:10:55 +02:00
dernbu reviewed 2026-06-13 20:13:03 +02:00
services/web/src/routes/health.ts Outdated
@ -0,0 +1,12 @@
import { createFileRoute } from "@tanstack/react-router";
dernbu commented 2026-06-13 20:12:49 +02:00
Author
Owner
Copy link

/api/health path

/api/health path
dernbu reviewed 2026-06-13 20:14:45 +02:00
services/web/src/server/health.server.ts Outdated
@ -0,0 +7,4 @@
const databaseUrl = process.env.DATABASE_URL;
const pool = databaseUrl ? new Pool({ connectionString: databaseUrl }) : null;
export async function getHealthResponse(): Promise<Response> {
dernbu commented 2026-06-13 20:14:11 +02:00
Author
Owner
Copy link

Lets just return 200 ok first, no query postgres

Lets just return 200 ok first, no query postgres
services/web/src/server/health.ts Outdated
@ -0,0 +7,4 @@
};
}
export function buildHealthReport(
dernbu commented 2026-06-13 20:14:39 +02:00
Author
Owner
Copy link

Same, hardcode 200 ok first for health

Same, hardcode 200 ok first for health
dernbu referenced this pull request from a commit 2026-06-13 20:52:59 +02:00
fix: address PR #14 review comments
dernbu merged commit d3f2b6e653 into main 2026-06-13 20:57:05 +02:00
dernbu deleted branch feat/1-web-scaffold 2026-06-13 20:57:05 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dernbu/lawnet-scraper!14
No description provided.