Skip to content

Commit efbd269

Browse files
refactor(db): retire dockerless PostgreSQL support (#1522)
1 parent 2366715 commit efbd269

33 files changed

Lines changed: 62 additions & 1234 deletions

.changeset/solid-socks-smash.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
---
2+
---

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,6 @@ sdist/
190190
server/.env.bak.*
191191
server/logs/
192192
server/postgres-data/
193-
server/postgres-data-local/
194193
server/postgres-data-temp-*
195194
server/postgres-data-temp/
196195
server/postgres-data*.tgz

docs/contributor/dockerless-postgres.mdx

Lines changed: 0 additions & 61 deletions
This file was deleted.

docs/contributor/local-development.mdx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@ Hephaestus supports full-stack development across Java and TypeScript. This guid
1212
Install and configure the following tools before you attempt a local build:
1313

1414
1. **Java JDK 21** – Required for the Spring Boot application server.
15-
2. **Docker Desktop** – Used for PostgreSQL and NATS containers.
15+
2. **Docker and Docker Compose** – Required for PostgreSQL and NATS. Verify both with `docker info`
16+
and `docker compose version`.
1617
3. **Node.js LTS (>= 24)** and **pnpm (>= 11)** (enable via `corepack enable`) – For the React client and root scripts.
1718
4. **Bun** – Runs the agent runtime and precompute tests (`pnpm run test:agents`). `corepack` does not
1819
provide it; install it separately. Match the version the agent image pins — the `ARG BUN_VERSION`
@@ -64,8 +65,7 @@ they are used.
6465

6566
They are **not** Maven profiles — `./mvnw -Plocal` warns that the profile does not exist and changes
6667
nothing. Maven passes the active Spring profile through the `app.profiles` property, so the override
67-
is `-Dapp.profiles=…` (this is what `pnpm dev:server:e2e` does). Maven's own profiles are a separate,
68-
smaller set: `live-tests`, `architecture-tests`, `quick`, `pitest`, `aot` and `local-db-timeout`.
68+
is `-Dapp.profiles=…` (this is what `pnpm dev:server:e2e` does).
6969

7070
Create `server/src/main/resources/application-local.yml` to override defaults. This file is gitignored.
7171

@@ -75,7 +75,7 @@ Never commit `application-local.yml`. It may contain secrets and machine-specifi
7575

7676
### Running the stack
7777

78-
1. Start Docker Desktop.
78+
1. Ensure the Docker daemon is running with `docker info`.
7979
2. From the repo root, start everything with one command:
8080

8181
```bash

docs/sidebars.contributor.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,6 @@ const sidebars: SidebarsConfig = {
1717
'coding-guidelines',
1818
'api-error-handling',
1919
'workspace-context',
20-
'dockerless-postgres',
21-
// How we develop, not what we ship — the "AI Development" category below is the product's
22-
// own agent runtime, which is a different meaning of the word.
2320
'ai-agent-workflow',
2421
],
2522
},
@@ -43,7 +40,7 @@ const sidebars: SidebarsConfig = {
4340
},
4441
{
4542
type: 'category',
46-
label: 'AI Development',
43+
label: 'Agent Runtime',
4744
items: ['unified-pi-runtime', 'agent/agent-workspace-abi', 'llm-cost-vocabulary'],
4845
},
4946
{

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
"scripts": {
88
"changeset:version": "changeset version && node scripts/sync-selfhost-version.ts",
99
"generate:api:application-server:clean": "node -e \"import('node:fs/promises').then(fs => fs.rm('webapp/src/api', {recursive: true, force: true}))\"",
10-
"generate:api:application-server:specs": "cd server && sh -c 'if command -v docker >/dev/null 2>&1 && docker info >/dev/null 2>&1; then ./mvnw verify -DskipTests=true -Dapp.profiles=specs; else SPRING_DOCKER_COMPOSE_ENABLED=false ./mvnw verify -DskipTests=true -Dapp.profiles=specs; fi'",
10+
"generate:api:application-server:specs": "cd server && ./mvnw verify -DskipTests=true -Dapp.profiles=specs",
1111
"generate:api:application-server:client": "pnpm run generate:api:application-server:clean && pnpm --filter webapp run openapi-ts",
1212
"generate:api:application-server": "pnpm run generate:api:application-server:specs && pnpm run generate:api:application-server:client",
1313
"generate:api": "pnpm run generate:api:application-server",

scripts/README.md

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,20 +10,22 @@ Scripts use dependencies from the root `package.json`. Run `pnpm install` at the
1010

1111
### Database Utilities
1212

13-
All database-related commands are accessed via `db-utils.sh` which handles database setup, migrations, and cleanup:
13+
Database documentation commands use `db-utils.sh` and require Docker with the Compose plugin:
1414

1515
```bash
1616
pnpm run db:generate-erd-docs # Generate Mermaid ERD diagram
1717
pnpm run db:draft-changelog # Generate Liquibase changelog diff
1818
```
1919

20-
**ERD Generation Environment Variables:**
20+
**ERD generation environment variables:**
2121

22-
| Variable | Default | Description |
23-
| -------------------- | ----------- | ---------------------------- |
24-
| `POSTGRES_HOST` | `localhost` | PostgreSQL host |
25-
| `POSTGRES_PORT` | `5432` | PostgreSQL port |
26-
| `HEPHAESTUS_DB_MODE` | `docker` | Database mode: docker/local |
22+
| Variable | Default | Description |
23+
| ------------------- | ------------ | ------------------- |
24+
| `POSTGRES_HOST` | `localhost` | PostgreSQL host |
25+
| `POSTGRES_PORT` | `5432` | PostgreSQL port |
26+
| `POSTGRES_DB` | `hephaestus` | PostgreSQL database |
27+
| `POSTGRES_USER` | `root` | PostgreSQL user |
28+
| `POSTGRES_PASSWORD` | `root` | PostgreSQL password |
2729

2830
### Achievement Formatting
2931

scripts/codex-maintenance.sh

Lines changed: 0 additions & 105 deletions
This file was deleted.

0 commit comments

Comments
 (0)