Website: Add docker build check

To avoid the mistake that happened earlier

Author: vdavid

AGENTS.md

@@ -53,7 +53,7 @@ Run the smallest set of checks possible for efficiency while maintaining confide
   |cargo-audit|cargo-deny|cargo-udeps|jscpd-rust|cfg-gate|rust-tests|rust-tests-linux (slow)|license-server-prettier
   |license-server-eslint|license-server-typecheck|license-server-tests|gofmt|go-vet|staticcheck|ineffassign|misspell
   |gocyclo|nilaway|govulncheck|deadcode|go-tests|website-prettier|website-eslint|website-typecheck|website-build
-  |website-e2e|pnpm-audit|file-length}` (can use multiple `--check` flags or even a comma-separated list)
+  |website-e2e|docker-build|pnpm-audit|file-length}` (can use multiple `--check` flags or even a comma-separated list)
 - Run all: `./scripts/check.sh`. Runs all tests, linters, and formatters (with auto fixing) for all apps.
 - **E2E testing**: Full guide on Docker E2E, Playwright smoke tests, VNC debugging, fixture system:
   [docs/tooling/e2e-testing-guide.md](docs/tooling/e2e-testing-guide.md)

scripts/check/CLAUDE.md

@@ -81,13 +81,24 @@ check name, duration, result (pass/fail/skip/blocked), and optional counts (tota
 `CheckResult` has `Total`, `Issues`, `Changes` fields (`-1` = N/A, rendered as `N/A` in CSV).
 Disabled by `--no-log` or `--ci`. Implementation in `stats.go`.
 
+## Adding a new check
+
+1. Create `checks/{app}-{name}.go` with a `func RunSomething(ctx *CheckContext) (CheckResult, error)`.
+   Use `website-build.go` or `website-docker.go` as templates — they're the simplest.
+2. Register it in `AllChecks` in `registry.go` (ID, App, Tech, DependsOn, Run).
+3. Return `Success("message")` on pass, `fmt.Errorf(...)` on fail, `Skipped("reason")` to skip.
+4. Add a test file if the check has non-trivial logic (`checks/{app}-{name}_test.go`).
+5. Run `./scripts/check.sh --go` to verify (staticcheck is strict about idiomatic Go).
+6. Update the table below and `AGENTS.md`'s `--check` list.
+
 ## Apps and check counts
 
 | App | Tech | Checks |
 |-----|------|--------|
 | Desktop | Rust | rustfmt, clippy, cargo-audit, cargo-deny, cargo-udeps, jscpd, tests, tests-linux (slow) |
 | Desktop | Svelte | prettier, eslint, stylelint, css-unused, svelte-check, knip, type-drift, tests, smoke, e2e-linux-typecheck, e2e-linux (slow) |
 | Website | Astro | prettier, eslint, typecheck, build, e2e |
+| Website | Docker | docker-build |
 | License server | TS | prettier, eslint, typecheck, tests |
 | Scripts | Go | gofmt, go-vet, staticcheck, ineffassign, misspell, gocyclo, nilaway, govulncheck, deadcode, go-tests |
 | Other | pnpm | pnpm-audit |

scripts/check/checks/registry.go

@@ -214,6 +214,15 @@ var AllChecks = []CheckDefinition{
 		DependsOn:   []string{"website-eslint"},
 		Run:         RunWebsiteTypecheck,
 	},
+	{
+		ID:          "website-docker-build",
+		Nickname:    "docker-build",
+		DisplayName: "docker build",
+		App:         AppWebsite,
+		Tech:        "🐳 Docker",
+		DependsOn:   nil,
+		Run:         RunWebsiteDockerBuild,
+	},
 	{
 		ID:          "website-build",
 		DisplayName: "build",

scripts/check/checks/website-docker.go

@@ -0,0 +1,23 @@
+package checks
+
+import (
+	"fmt"
+	"os/exec"
+)
+
+// RunWebsiteDockerBuild runs the full Docker build for the website.
+// Catches .dockerignore mismatches, missing COPY sources, pnpm install failures,
+// and Astro build errors in the container context — anything that would break the deploy.
+func RunWebsiteDockerBuild(ctx *CheckContext) (CheckResult, error) {
+	if !CommandExists("docker") {
+		return Skipped("docker not installed"), nil
+	}
+
+	cmd := exec.Command("docker", "build", "-f", "apps/website/Dockerfile", "-t", "getcmdr-check", ".")
+	cmd.Dir = ctx.RootDir
+	output, err := RunCommand(cmd, true)
+	if err != nil {
+		return CheckResult{}, fmt.Errorf("docker build failed\n%s", indentOutput(output))
+	}
+	return Success("Docker build passed"), nil
+}