mirror of
https://github.com/Dvorinka/Containr.git
synced 2026-10-28 07:41:27 +00:00
Tomas Dvorak
aef9eeeff8
- CONTRIBUTING.md gains a boot smoke test (live/health/401 checks) and a cmd/migrate verification recipe. - desloppify rerun (ref/ excluded): objective 66.1, strict 16.5 under the new scoring model — the subjective review queue (20 items) is unscored, so strict is not comparable to the February 89.1. Authored By: TDvorak <[email protected]>
70 lines
2.2 KiB
Markdown
70 lines
2.2 KiB
Markdown
# Contributing to Containr
|
|
|
|
Thanks for your interest in contributing. Containr is a self-hosted deployment
|
|
platform — contributions of all sizes are welcome.
|
|
|
|
## Project Layout
|
|
|
|
```
|
|
app/backend/ Go control-plane API (Gin, PostgreSQL, Redis, Docker SDK)
|
|
app/frontend/ React + Vite + TypeScript dashboard
|
|
infra/ Self-hosted compose stack (Traefik, Cloudflare Tunnel)
|
|
docs/ Guides, API spec, and design documents
|
|
templates/ Service template catalog notes
|
|
```
|
|
|
|
## Development Setup
|
|
|
|
1. Copy `.env.example` to `.env` and fill in values (never commit `.env`).
|
|
2. Start Postgres and Redis: `docker compose up -d postgres redis`
|
|
3. Backend: `cd app/backend && go run ./cmd/server`
|
|
4. Frontend: `cd app/frontend && npm install && npm run dev`
|
|
|
|
## Checks Before Submitting
|
|
|
|
- Backend: `cd app/backend && go build ./... && go test ./... && go vet ./...`
|
|
- Frontend: `cd app/frontend && npm run build && npm test`
|
|
- Compose: `docker compose config -q`
|
|
|
|
## Boot Smoke Test
|
|
|
|
Verify the API actually starts and serves against real dependencies:
|
|
|
|
```bash
|
|
# Requires Postgres + Redis reachable; migrations run automatically on boot.
|
|
cd app/backend
|
|
DATABASE_URL="postgres://user:pass@localhost:5432/containr?sslmode=disable" \
|
|
REDIS_URL="redis://localhost:6379" \
|
|
go run ./cmd/server &
|
|
|
|
curl -fsS http://localhost:8080/live # -> 200
|
|
curl -fsS http://localhost:8080/health # -> {"database":"ok","redis":"ok",...}
|
|
curl -s http://localhost:8080/api/v1/services/x # -> 401 (auth required, route exists)
|
|
|
|
kill %1
|
|
```
|
|
|
|
To verify migrations alone against a scratch database:
|
|
|
|
```bash
|
|
cd app/backend
|
|
DATABASE_URL="postgres://user:pass@localhost:5432/scratch?sslmode=disable" \
|
|
go run ./cmd/migrate up
|
|
DATABASE_URL=... go run ./cmd/migrate goose-status
|
|
```
|
|
|
|
## Conventions
|
|
|
|
- Reuse existing utilities before adding dependencies.
|
|
- Keep handlers thin; business logic lives in `internal/` packages.
|
|
- Regenerate API types after schema changes:
|
|
`npm --prefix app/frontend run generate:api`.
|
|
- Never commit secrets, tokens, or environment files.
|
|
- Write tests for non-trivial logic.
|
|
|
|
## Pull Requests
|
|
|
|
- Keep PRs focused — one concern per PR.
|
|
- Describe the "why", not just the "what".
|
|
- Include a test plan for anything touching deployment, auth, or networking.
|