fix: resolve typecheck errors by removing duplicate db methods and updating handlers to use models-based API

.dockerignore

@@ -1,9 +1,8 @@
-.git
-node_modules
-.DS_Store
 bin/
+chatd
 data.db
 .env
+.git
 *.test
 *.out
 debug.log

.editorconfig

@@ -1,12 +0,0 @@
-root = true
-
-[*]
-indent_style = space
-indent_size = 4
-end_of_line = lf
-charset = utf-8
-trim_trailing_whitespace = true
-insert_final_newline = true
-
-[Makefile]
-indent_style = tab

.gitea/workflows/check.yml

@@ -1,9 +0,0 @@
-name: check
-on: [push]
-jobs:
-    check:
-        runs-on: ubuntu-latest
-        steps:
-            # actions/checkout v4.2.2, 2026-02-22
-            - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
-            - run: docker build .

.gitignore

@@ -1,28 +1,7 @@
-# OS
-.DS_Store
-Thumbs.db
-
-# Editors
-*.swp
-*.swo
-*~
-*.bak
-.idea/
-.vscode/
-*.sublime-*
-
-# Node
-node_modules/
-
-# Environment / secrets
-.env
-.env.*
-*.pem
-*.key
-
-# Build artifacts
 /chatd
 /bin/
+data.db
+.env
 *.exe
 *.dll
 *.so
@@ -30,9 +9,6 @@ node_modules/
 *.test
 *.out
 vendor/
-
-# Project
-data.db
 debug.log
-/chat-cli
 web/node_modules/
+chat-cli

Dockerfile

@@ -1,7 +1,6 @@
-# golang:1.24-alpine, 2026-02-26
+FROM golang:1.24-alpine AS builder
-FROM golang@sha256:8bee1901f1e530bfb4a7850aa7a479d17ae3a18beb6e09064ed54cfd245b7191 AS builder
 
-RUN apk add --no-cache git build-base
+RUN apk add --no-cache git
 
 WORKDIR /src
 COPY go.mod go.sum ./
@@ -9,15 +8,10 @@ RUN go mod download
 
 COPY . .
 
-# Run all checks — build fails if branch is not green
-RUN CGO_ENABLED=0 go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6
-RUN make check
-
 ARG VERSION=dev
 RUN go build -ldflags "-X main.Version=${VERSION}" -o /chatd ./cmd/chatd
 
-# alpine:3.21, 2026-02-26
+FROM alpine:3.21
-FROM alpine@sha256:c3f8e73fdb79deaebaa2037150150191b9dcbfba68b4a46d70103204c53f4709
 
 RUN apk add --no-cache ca-certificates
 COPY --from=builder /chatd /usr/local/bin/chatd

LICENSE

@@ -1,21 +0,0 @@
-MIT License
-
-Copyright (c) 2025 sneak
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

Makefile

@@ -1,4 +1,4 @@
-.PHONY: all build lint fmt fmt-check test check clean run debug docker hooks
+.PHONY: all build lint fmt test check clean run debug
 
 BINARY := chatd
 VERSION := $(shell git describe --tags --always --dirty 2>/dev/null || echo "dev")
@@ -17,15 +17,18 @@ fmt:
 	gofmt -s -w .
 	goimports -w .
 
-fmt-check:
-	@test -z "$$(gofmt -l .)" || (echo "Files not formatted:" && gofmt -l . && exit 1)
-
 test:
-	go test -timeout 30s -v -race -cover ./...
+	go test -v -race -cover ./...
 
-# check runs all validation without making changes
+# Check runs all validation without making changes
 # Used by CI and Docker build — fails if anything is wrong
-check: test lint fmt-check
+check:
+	@echo "==> Checking formatting..."
+	@test -z "$$(gofmt -l .)" || (echo "Files not formatted:" && gofmt -l . && exit 1)
+	@echo "==> Running linter..."
+	golangci-lint run --config .golangci.yml ./...
+	@echo "==> Running tests..."
+	go test -v -race ./...
 	@echo "==> Building..."
 	go build -ldflags "$(LDFLAGS)" -o /dev/null ./cmd/chatd
 	@echo "==> All checks passed!"
@@ -38,12 +41,3 @@ debug: build
 
 clean:
 	rm -rf bin/ chatd data.db
-
-docker:
-	docker build -t chat .
-
-hooks:
-	@printf '#!/bin/sh\nset -e\n' > .git/hooks/pre-commit
-	@printf 'go mod tidy\ngo fmt ./...\ngit diff --exit-code -- go.mod go.sum || { echo "go mod tidy changed files; please stage and retry"; exit 1; }\n' >> .git/hooks/pre-commit
-	@printf 'make check\n' >> .git/hooks/pre-commit
-	@chmod +x .git/hooks/pre-commit

README.md

@@ -656,8 +656,3 @@ embedded web client.
 ## License
 
 MIT
-
-## Author
-
-[@sneak](https://sneak.berlin)
-

REPO_POLICIES.md

@@ -1,182 +0,0 @@
----
-title: Repository Policies
-last_modified: 2026-02-22
----
-
-This document covers repository structure, tooling, and workflow standards. Code
-style conventions are in separate documents:
-
-- [Code Styleguide](https://git.eeqj.de/sneak/prompts/raw/branch/main/prompts/CODE_STYLEGUIDE.md)
-  (general, bash, Docker)
-- [Go](https://git.eeqj.de/sneak/prompts/raw/branch/main/prompts/CODE_STYLEGUIDE_GO.md)
-- [JavaScript](https://git.eeqj.de/sneak/prompts/raw/branch/main/prompts/CODE_STYLEGUIDE_JS.md)
-- [Python](https://git.eeqj.de/sneak/prompts/raw/branch/main/prompts/CODE_STYLEGUIDE_PYTHON.md)
-- [Go HTTP Server Conventions](https://git.eeqj.de/sneak/prompts/raw/branch/main/prompts/GO_HTTP_SERVER_CONVENTIONS.md)
-
----
-
-- Cross-project documentation (such as this file) must include
-  `last_modified: YYYY-MM-DD` in the YAML front matter so it can be kept in sync
-  with the authoritative source as policies evolve.
-
-- **ALL external references must be pinned by cryptographic hash.** This
-  includes Docker base images, Go modules, npm packages, GitHub Actions, and
-  anything else fetched from a remote source. Version tags (`@v4`, `@latest`,
-  `:3.21`, etc.) are server-mutable and therefore remote code execution
-  vulnerabilities. The ONLY acceptable way to reference an external dependency
-  is by its content hash (Docker `@sha256:...`, Go module hash in `go.sum`, npm
-  integrity hash in lockfile, GitHub Actions `@<commit-sha>`). No exceptions.
-  This also means never `curl | bash` to install tools like pyenv, nvm, rustup,
-  etc. Instead, download a specific release archive from GitHub, verify its hash
-  (hardcoded in the Dockerfile or script), and only then install. Unverified
-  install scripts are arbitrary remote code execution. This is the single most
-  important rule in this document. Double-check every external reference in
-  every file before committing. There are zero exceptions to this rule.
-
-- Every repo with software must have a root `Makefile` with these targets:
-  `make test`, `make lint`, `make fmt` (writes), `make fmt-check` (read-only),
-  `make check` (prereqs: `test`, `lint`, `fmt-check`), `make docker`, and
-  `make hooks` (installs pre-commit hook). A model Makefile is at
-  `https://git.eeqj.de/sneak/prompts/raw/branch/main/Makefile`.
-
-- Always use Makefile targets (`make fmt`, `make test`, `make lint`, etc.)
-  instead of invoking the underlying tools directly. The Makefile is the single
-  source of truth for how these operations are run.
-
-- The Makefile is authoritative documentation for how the repo is used. Beyond
-  the required targets above, it should have targets for every common operation:
-  running a local development server (`make run`, `make dev`), re-initializing
-  or migrating the database (`make db-reset`, `make migrate`), building
-  artifacts (`make build`), generating code, seeding data, or anything else a
-  developer would do regularly. If someone checks out the repo and types
-  `make<tab>`, they should see every meaningful operation available. A new
-  contributor should be able to understand the entire development workflow by
-  reading the Makefile.
-
-- Every repo should have a `Dockerfile`. All Dockerfiles must run `make check`
-  as a build step so the build fails if the branch is not green. For non-server
-  repos, the Dockerfile should bring up a development environment and run
-  `make check`. For server repos, `make check` should run as an early build
-  stage before the final image is assembled.
-
-- Every repo should have a Gitea Actions workflow (`.gitea/workflows/`) that
-  runs `docker build .` on push. Since the Dockerfile already runs `make check`,
-  a successful build implies all checks pass.
-
-- Use platform-standard formatters: `black` for Python, `prettier` for
-  JS/CSS/Markdown/HTML, `go fmt` for Go. Always use default configuration with
-  two exceptions: four-space indents (except Go), and `proseWrap: always` for
-  Markdown (hard-wrap at 80 columns). Documentation and writing repos (Markdown,
-  HTML, CSS) should also have `.prettierrc` and `.prettierignore`.
-
-- Pre-commit hook: `make check` if local testing is possible, otherwise
-  `make lint && make fmt-check`. The Makefile should provide a `make hooks`
-  target to install the pre-commit hook.
-
-- All repos with software must have tests that run via the platform-standard
-  test framework (`go test`, `pytest`, `jest`/`vitest`, etc.). If no meaningful
-  tests exist yet, add the most minimal test possible — e.g. importing the
-  module under test to verify it compiles/parses. There is no excuse for
-  `make test` to be a no-op.
-
-- `make test` must complete in under 20 seconds. Add a 30-second timeout in the
-  Makefile.
-
-- Docker builds must complete in under 5 minutes.
-
-- `make check` must not modify any files in the repo. Tests may use temporary
-  directories.
-
-- `main` must always pass `make check`, no exceptions.
-
-- Never commit secrets. `.env` files, credentials, API keys, and private keys
-  must be in `.gitignore`. No exceptions.
-
-- `.gitignore` should be comprehensive from the start: OS files (`.DS_Store`),
-  editor files (`.swp`, `*~`), language build artifacts, and `node_modules/`.
-  Fetch the standard `.gitignore` from
-  `https://git.eeqj.de/sneak/prompts/raw/branch/main/.gitignore` when setting up
-  a new repo.
-
-- Never use `git add -A` or `git add .`. Always stage files explicitly by name.
-
-- Never force-push to `main`.
-
-- Make all changes on a feature branch. You can do whatever you want on a
-  feature branch.
-
-- `.golangci.yml` is standardized and must _NEVER_ be modified by an agent, only
-  manually by the user. Fetch from
-  `https://git.eeqj.de/sneak/prompts/raw/branch/main/.golangci.yml`.
-
-- When pinning images or packages by hash, add a comment above the reference
-  with the version and date (YYYY-MM-DD).
-
-- Use `yarn`, not `npm`.
-
-- Write all dates as YYYY-MM-DD (ISO 8601).
-
-- Simple projects should be configured with environment variables.
-
-- Dockerized web services listen on port 8080 by default, overridable with
-  `PORT`.
-
-- `README.md` is the primary documentation. Required sections:
-    - **Description**: First line must include the project name, purpose,
-      category (web server, SPA, CLI tool, etc.), license, and author. Example:
-      "µPaaS is an MIT-licensed Go web application by @sneak that receives
-      git-frontend webhooks and deploys applications via Docker in realtime."
-    - **Getting Started**: Copy-pasteable install/usage code block.
-    - **Rationale**: Why does this exist?
-    - **Design**: How is the program structured?
-    - **TODO**: Update meticulously, even between commits. When planning, put
-      the todo list in the README so a new agent can pick up where the last one
-      left off.
-    - **License**: MIT, GPL, or WTFPL. Ask the user for new projects. Include a
-      `LICENSE` file in the repo root and a License section in the README.
-    - **Author**: [@sneak](https://sneak.berlin).
-
-- First commit of a new repo should contain only `README.md`.
-
-- Go module root: `sneak.berlin/go/<name>`. Always run `go mod tidy` before
-  committing.
-
-- Use SemVer.
-
-- Database migrations live in `internal/db/migrations/` and must be embedded in
-  the binary. Pre-1.0.0: modify existing migrations (no installed base assumed).
-  Post-1.0.0: add new migration files.
-
-- All repos should have an `.editorconfig` enforcing the project's indentation
-  settings.
-
-- Avoid putting files in the repo root unless necessary. Root should contain
-  only project-level config files (`README.md`, `Makefile`, `Dockerfile`,
-  `LICENSE`, `.gitignore`, `.editorconfig`, `REPO_POLICIES.md`, and
-  language-specific config). Everything else goes in a subdirectory. Canonical
-  subdirectory names:
-    - `bin/` — executable scripts and tools
-    - `cmd/` — Go command entrypoints
-    - `configs/` — configuration templates and examples
-    - `deploy/` — deployment manifests (k8s, compose, terraform)
-    - `docs/` — documentation and markdown (README.md stays in root)
-    - `internal/` — Go internal packages
-    - `internal/db/migrations/` — database migrations
-    - `pkg/` — Go library packages
-    - `share/` — systemd units, data files
-    - `static/` — static assets (images, fonts, etc.)
-    - `web/` — web frontend source
-
-- When setting up a new repo, files from the `prompts` repo may be used as
-  templates. Fetch them from
-  `https://git.eeqj.de/sneak/prompts/raw/branch/main/<path>`.
-
-- New repos must contain at minimum:
-    - `README.md`, `.git`, `.gitignore`, `.editorconfig`
-    - `LICENSE`, `REPO_POLICIES.md` (copy from the `prompts` repo)
-    - `Makefile`
-    - `Dockerfile`, `.dockerignore`
-    - `.gitea/workflows/check.yml`
-    - Go: `go.mod`, `go.sum`, `.golangci.yml`
-    - JS: `package.json`, `yarn.lock`, `.prettierrc`, `.prettierignore`
-    - Python: `pyproject.toml`

cmd/chat-cli/api/client.go

@@ -1,31 +1,15 @@
-// Package chatapi provides a client for the chat server HTTP API.
+package api
-package chatapi
 
 import (
 	"bytes"
 	"encoding/json"
-	"errors"
 	"fmt"
 	"io"
 	"net/http"
 	"net/url"
-	"strconv"
 	"time"
 )
 
-const (
-	httpTimeout      = 30 * time.Second
-	pollExtraDelay   = 5
-	httpErrThreshold = 400
-)
-
-// ErrHTTP is returned for non-2xx responses.
-var ErrHTTP = errors.New("http error")
-
-// ErrUnexpectedFormat is returned when the response format is
-// not recognised.
-var ErrUnexpectedFormat = errors.New("unexpected format")
-
 // Client wraps HTTP calls to the chat server API.
 type Client struct {
 	BaseURL    string
@@ -38,32 +22,59 @@ func NewClient(baseURL string) *Client {
 	return &Client{
 		BaseURL: baseURL,
 		HTTPClient: &http.Client{
-			Timeout: httpTimeout,
+			Timeout: 30 * time.Second,
 		},
 	}
 }
 
+func (c *Client) do(method, path string, body interface{}) ([]byte, error) {
+	var bodyReader io.Reader
+	if body != nil {
+		data, err := json.Marshal(body)
+		if err != nil {
+			return nil, fmt.Errorf("marshal: %w", err)
+		}
+		bodyReader = bytes.NewReader(data)
+	}
+
+	req, err := http.NewRequest(method, c.BaseURL+path, bodyReader)
+	if err != nil {
+		return nil, fmt.Errorf("request: %w", err)
+	}
+	req.Header.Set("Content-Type", "application/json")
+	if c.Token != "" {
+		req.Header.Set("Authorization", "Bearer "+c.Token)
+	}
+
+	resp, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return nil, fmt.Errorf("http: %w", err)
+	}
+	defer resp.Body.Close()
+
+	data, err := io.ReadAll(resp.Body)
+	if err != nil {
+		return nil, fmt.Errorf("read body: %w", err)
+	}
+
+	if resp.StatusCode >= 400 {
+		return data, fmt.Errorf("HTTP %d: %s", resp.StatusCode, string(data))
+	}
+
+	return data, nil
+}
+
 // CreateSession creates a new session on the server.
-func (c *Client) CreateSession(
+func (c *Client) CreateSession(nick string) (*SessionResponse, error) {
-	nick string,
+	data, err := c.do("POST", "/api/v1/session", &SessionRequest{Nick: nick})
-) (*SessionResponse, error) {
-	data, err := c.do(
-		"POST", "/api/v1/session",
-		&SessionRequest{Nick: nick},
-	)
 	if err != nil {
 		return nil, err
 	}
-
 	var resp SessionResponse
-
+	if err := json.Unmarshal(data, &resp); err != nil {
-	err = json.Unmarshal(data, &resp)
-	if err != nil {
 		return nil, fmt.Errorf("decode session: %w", err)
 	}
-
 	c.Token = resp.Token
-
 	return &resp, nil
 }
 
@@ -73,113 +84,78 @@ func (c *Client) GetState() (*StateResponse, error) {
 	if err != nil {
 		return nil, err
 	}
-
 	var resp StateResponse
-
+	if err := json.Unmarshal(data, &resp); err != nil {
-	err = json.Unmarshal(data, &resp)
-	if err != nil {
 		return nil, fmt.Errorf("decode state: %w", err)
 	}
-
 	return &resp, nil
 }
 
 // SendMessage sends a message (any IRC command).
 func (c *Client) SendMessage(msg *Message) error {
 	_, err := c.do("POST", "/api/v1/messages", msg)
-
 	return err
 }
 
 // PollMessages long-polls for new messages.
-func (c *Client) PollMessages(
+func (c *Client) PollMessages(afterID string, timeout int) ([]Message, error) {
-	afterID string,
+	// Use a longer HTTP timeout than the server long-poll timeout.
-	timeout int,
+	client := &http.Client{Timeout: time.Duration(timeout+5) * time.Second}
-) ([]Message, error) {
-	pollTimeout := time.Duration(
-		timeout+pollExtraDelay,
-	) * time.Second
-
-	client := &http.Client{Timeout: pollTimeout}
 
 	params := url.Values{}
 	if afterID != "" {
 		params.Set("after", afterID)
 	}
-
+	params.Set("timeout", fmt.Sprintf("%d", timeout))
-	params.Set("timeout", strconv.Itoa(timeout))
 
 	path := "/api/v1/messages"
 	if len(params) > 0 {
 		path += "?" + params.Encode()
 	}
 
-	req, err := http.NewRequest( //nolint:noctx // CLI tool
+	req, err := http.NewRequest("GET", c.BaseURL+path, nil)
-		http.MethodGet, c.BaseURL+path, nil,
-	)
 	if err != nil {
 		return nil, err
 	}
-
 	req.Header.Set("Authorization", "Bearer "+c.Token)
 
-	resp, err := client.Do(req) //nolint:gosec // URL from user config
+	resp, err := client.Do(req)
 	if err != nil {
 		return nil, err
 	}
-
+	defer resp.Body.Close()
-	defer func() { _ = resp.Body.Close() }()
 
 	data, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return nil, err
 	}
 
-	if resp.StatusCode >= httpErrThreshold {
+	if resp.StatusCode >= 400 {
-		return nil, fmt.Errorf(
+		return nil, fmt.Errorf("HTTP %d: %s", resp.StatusCode, string(data))
-			"%w: %d: %s",
-			ErrHTTP, resp.StatusCode, string(data),
-		)
 	}
 
-	return decodeMessages(data)
+	// The server may return an array directly or wrapped.
-}
-
-func decodeMessages(data []byte) ([]Message, error) {
 	var msgs []Message
-
+	if err := json.Unmarshal(data, &msgs); err != nil {
-	err := json.Unmarshal(data, &msgs)
+		// Try wrapped format.
-	if err == nil {
+		var wrapped MessagesResponse
-		return msgs, nil
+		if err2 := json.Unmarshal(data, &wrapped); err2 != nil {
+			return nil, fmt.Errorf("decode messages: %w (raw: %s)", err, string(data))
+		}
+		msgs = wrapped.Messages
 	}
 
-	var wrapped MessagesResponse
+	return msgs, nil
-
-	err2 := json.Unmarshal(data, &wrapped)
-	if err2 != nil {
-		return nil, fmt.Errorf(
-			"decode messages: %w (raw: %s)",
-			err, string(data),
-		)
-	}
-
-	return wrapped.Messages, nil
 }
 
-// JoinChannel joins a channel via the unified command
+// JoinChannel joins a channel via the unified command endpoint.
-// endpoint.
 func (c *Client) JoinChannel(channel string) error {
-	return c.SendMessage(
+	return c.SendMessage(&Message{Command: "JOIN", To: channel})
-		&Message{Command: "JOIN", To: channel},
-	)
 }
 
-// PartChannel leaves a channel via the unified command
+// PartChannel leaves a channel via the unified command endpoint.
-// endpoint.
 func (c *Client) PartChannel(channel string) error {
-	return c.SendMessage(
+	return c.SendMessage(&Message{Command: "PART", To: channel})
-		&Message{Command: "PART", To: channel},
-	)
 }
 
 // ListChannels returns all channels on the server.
@@ -188,39 +164,29 @@ func (c *Client) ListChannels() ([]Channel, error) {
 	if err != nil {
 		return nil, err
 	}
-
 	var channels []Channel
-
+	if err := json.Unmarshal(data, &channels); err != nil {
-	err = json.Unmarshal(data, &channels)
-	if err != nil {
 		return nil, err
 	}
-
 	return channels, nil
 }
 
 // GetMembers returns members of a channel.
-func (c *Client) GetMembers(
+func (c *Client) GetMembers(channel string) ([]string, error) {
-	channel string,
+	data, err := c.do("GET", "/api/v1/channels/"+url.PathEscape(channel)+"/members", nil)
-) ([]string, error) {
-	path := "/api/v1/channels/" +
-		url.PathEscape(channel) + "/members"
-
-	data, err := c.do("GET", path, nil)
 	if err != nil {
 		return nil, err
 	}
-
 	var members []string
-
+	if err := json.Unmarshal(data, &members); err != nil {
-	err = json.Unmarshal(data, &members)
+		// Try object format.
-	if err != nil {
+		var obj map[string]interface{}
-		return nil, fmt.Errorf(
+		if err2 := json.Unmarshal(data, &obj); err2 != nil {
-			"%w: members: %s",
+			return nil, err
-			ErrUnexpectedFormat, string(data),
+		}
-		)
+		// Extract member names from whatever format.
+		return nil, fmt.Errorf("unexpected members format: %s", string(data))
 	}
-
 	return members, nil
 }
 
@@ -230,63 +196,9 @@ func (c *Client) GetServerInfo() (*ServerInfo, error) {
 	if err != nil {
 		return nil, err
 	}
-
 	var info ServerInfo
-
+	if err := json.Unmarshal(data, &info); err != nil {
-	err = json.Unmarshal(data, &info)
-	if err != nil {
 		return nil, err
 	}
-
 	return &info, nil
 }
-
-func (c *Client) do(
-	method, path string,
-	body any,
-) ([]byte, error) {
-	var bodyReader io.Reader
-
-	if body != nil {
-		data, err := json.Marshal(body)
-		if err != nil {
-			return nil, fmt.Errorf("marshal: %w", err)
-		}
-
-		bodyReader = bytes.NewReader(data)
-	}
-
-	req, err := http.NewRequest( //nolint:noctx // CLI tool
-		method, c.BaseURL+path, bodyReader,
-	)
-	if err != nil {
-		return nil, fmt.Errorf("request: %w", err)
-	}
-
-	req.Header.Set("Content-Type", "application/json")
-
-	if c.Token != "" {
-		req.Header.Set("Authorization", "Bearer "+c.Token)
-	}
-
-	resp, err := c.HTTPClient.Do(req) //nolint:gosec // URL from user config
-	if err != nil {
-		return nil, fmt.Errorf("http: %w", err)
-	}
-
-	defer func() { _ = resp.Body.Close() }()
-
-	data, err := io.ReadAll(resp.Body)
-	if err != nil {
-		return nil, fmt.Errorf("read body: %w", err)
-	}
-
-	if resp.StatusCode >= httpErrThreshold {
-		return data, fmt.Errorf(
-			"%w: %d: %s",
-			ErrHTTP, resp.StatusCode, string(data),
-		)
-	}
-
-	return data, nil
-}

cmd/chat-cli/api/types.go

@@ -1,4 +1,4 @@
-package chatapi
+package api
 
 import "time"
 
@@ -9,45 +9,42 @@ type SessionRequest struct {
 
 // SessionResponse is the response from POST /api/v1/session.
 type SessionResponse struct {
-	SessionID string `json:"sessionId"`
+	SessionID string `json:"session_id"`
-	ClientID  string `json:"clientId"`
+	ClientID  string `json:"client_id"`
 	Nick      string `json:"nick"`
 	Token     string `json:"token"`
 }
 
 // StateResponse is the response from GET /api/v1/state.
 type StateResponse struct {
-	SessionID string   `json:"sessionId"`
+	SessionID string   `json:"session_id"`
-	ClientID  string   `json:"clientId"`
+	ClientID  string   `json:"client_id"`
 	Nick      string   `json:"nick"`
 	Channels  []string `json:"channels"`
 }
 
 // Message represents a chat message envelope.
 type Message struct {
-	Command string   `json:"command"`
+	Command string      `json:"command"`
-	From    string   `json:"from,omitempty"`
+	From    string      `json:"from,omitempty"`
-	To      string   `json:"to,omitempty"`
+	To      string      `json:"to,omitempty"`
-	Params  []string `json:"params,omitempty"`
+	Params  []string    `json:"params,omitempty"`
-	Body    any      `json:"body,omitempty"`
+	Body    interface{} `json:"body,omitempty"`
-	ID      string   `json:"id,omitempty"`
+	ID      string      `json:"id,omitempty"`
-	TS      string   `json:"ts,omitempty"`
+	TS      string      `json:"ts,omitempty"`
-	Meta    any      `json:"meta,omitempty"`
+	Meta    interface{} `json:"meta,omitempty"`
 }
 
-// BodyLines returns the body as a slice of strings (for text
+// BodyLines returns the body as a slice of strings (for text messages).
-// messages).
 func (m *Message) BodyLines() []string {
 	switch v := m.Body.(type) {
-	case []any:
+	case []interface{}:
 		lines := make([]string, 0, len(v))
-
 		for _, item := range v {
 			if s, ok := item.(string); ok {
 				lines = append(lines, s)
 			}
 		}
-
 		return lines
 	case []string:
 		return v
@@ -61,7 +58,7 @@ type Channel struct {
 	Name      string `json:"name"`
 	Topic     string `json:"topic"`
 	Members   int    `json:"members"`
-	CreatedAt string `json:"createdAt"`
+	CreatedAt string `json:"created_at"`
 }
 
 // ServerInfo is the response from GET /api/v1/server.
@@ -82,6 +79,5 @@ func (m *Message) ParseTS() time.Time {
 	if err != nil {
 		return time.Now()
 	}
-
 	return t
 }

cmd/chat-cli/main.go

@@ -1,21 +1,13 @@
-// Package main implements chat-cli, an IRC-style terminal client.
 package main
 
 import (
 	"fmt"
 	"os"
-	"strconv"
 	"strings"
 	"sync"
 	"time"
 
-	api "git.eeqj.de/sneak/chat/cmd/chat-cli/api"
+	"git.eeqj.de/sneak/chat/cmd/chat-cli/api"
-)
-
-const (
-	pollTimeoutSec = 15
-	retryDelay     = 2 * time.Second
-	maxNickLength  = 32
 )
 
 // App holds the application state.
@@ -40,18 +32,11 @@ func main() {
 	app.ui.OnInput(app.handleInput)
 	app.ui.SetStatus(app.nick, "", "disconnected")
 
-	app.ui.AddStatus(
+	app.ui.AddStatus("Welcome to chat-cli — an IRC-style client")
-		"Welcome to chat-cli \u2014 an IRC-style client",
+	app.ui.AddStatus("Type [yellow]/connect <server-url>[white] to begin, or [yellow]/help[white] for commands")
-	)
-	app.ui.AddStatus(
-		"Type [yellow]/connect <server-url>[white] " +
-			"to begin, or [yellow]/help[white] for commands",
-	)
-
-	err := app.ui.Run()
-	if err != nil {
-		_, _ = fmt.Fprintf(os.Stderr, "Error: %v\n", err)
 
+	if err := app.ui.Run(); err != nil {
+		fmt.Fprintf(os.Stderr, "Error: %v\n", err)
 		os.Exit(1)
 	}
 }
@@ -59,34 +44,21 @@ func main() {
 func (a *App) handleInput(text string) {
 	if strings.HasPrefix(text, "/") {
 		a.handleCommand(text)
-
 		return
 	}
 
-	a.sendPlainText(text)
+	// Plain text → PRIVMSG to current target.
-}
-
-func (a *App) sendPlainText(text string) {
 	a.mu.Lock()
 	target := a.target
 	connected := a.connected
-	nick := a.nick
 	a.mu.Unlock()
 
 	if !connected {
-		a.ui.AddStatus(
+		a.ui.AddStatus("[red]Not connected. Use /connect <url>")
-			"[red]Not connected. Use /connect <url>",
-		)
-
 		return
 	}
-
 	if target == "" {
-		a.ui.AddStatus(
+		a.ui.AddStatus("[red]No target. Use /join #channel or /query nick")
-			"[red]No target. " +
-				"Use /join #channel or /query nick",
-		)
-
 		return
 	}
 
@@ -96,28 +68,21 @@ func (a *App) sendPlainText(text string) {
 		Body:    []string{text},
 	})
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Send error: %v", err))
-			fmt.Sprintf("[red]Send error: %v", err),
-		)
-
 		return
 	}
 
+	// Echo locally.
 	ts := time.Now().Format("15:04")
-
+	a.mu.Lock()
-	a.ui.AddLine(
+	nick := a.nick
-		target,
+	a.mu.Unlock()
-		fmt.Sprintf(
+	a.ui.AddLine(target, fmt.Sprintf("[gray]%s [green]<%s>[white] %s", ts, nick, text))
-			"[gray]%s [green]<%s>[white] %s",
-			ts, nick, text,
-		),
-	)
 }
 
-func (a *App) handleCommand(text string) { //nolint:cyclop // command dispatch
+func (a *App) handleCommand(text string) {
-	parts := strings.SplitN(text, " ", 2) //nolint:mnd // split into cmd+args
+	parts := strings.SplitN(text, " ", 2)
 	cmd := strings.ToLower(parts[0])
-
 	args := ""
 	if len(parts) > 1 {
 		args = parts[1]
@@ -149,41 +114,27 @@ func (a *App) handleCommand(text string) { //nolint:cyclop // command dispatch
 	case "/help":
 		a.cmdHelp()
 	default:
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Unknown command: %s", cmd))
-			"[red]Unknown command: " + cmd,
-		)
 	}
 }
 
 func (a *App) cmdConnect(serverURL string) {
 	if serverURL == "" {
-		a.ui.AddStatus(
+		a.ui.AddStatus("[red]Usage: /connect <server-url>")
-			"[red]Usage: /connect <server-url>",
-		)
-
 		return
 	}
-
 	serverURL = strings.TrimRight(serverURL, "/")
 
-	a.ui.AddStatus(
+	a.ui.AddStatus(fmt.Sprintf("Connecting to %s...", serverURL))
-		fmt.Sprintf("Connecting to %s...", serverURL),
-	)
 
 	a.mu.Lock()
 	nick := a.nick
 	a.mu.Unlock()
 
 	client := api.NewClient(serverURL)
-
 	resp, err := client.CreateSession(nick)
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Connection failed: %v", err))
-			fmt.Sprintf(
-				"[red]Connection failed: %v", err,
-			),
-		)
-
 		return
 	}
 
@@ -194,26 +145,19 @@ func (a *App) cmdConnect(serverURL string) {
 	a.lastMsgID = ""
 	a.mu.Unlock()
 
-	a.ui.AddStatus(
+	a.ui.AddStatus(fmt.Sprintf("[green]Connected! Nick: %s, Session: %s", resp.Nick, resp.SessionID))
-		fmt.Sprintf(
-			"[green]Connected! Nick: %s, Session: %s",
-			resp.Nick, resp.SessionID,
-		),
-	)
 	a.ui.SetStatus(resp.Nick, "", "connected")
 
+	// Start polling.
 	a.stopPoll = make(chan struct{})
-
 	go a.pollLoop()
 }
 
 func (a *App) cmdNick(nick string) {
 	if nick == "" {
 		a.ui.AddStatus("[red]Usage: /nick <name>")
-
 		return
 	}
-
 	a.mu.Lock()
 	connected := a.connected
 	a.mu.Unlock()
@@ -222,14 +166,7 @@ func (a *App) cmdNick(nick string) {
 		a.mu.Lock()
 		a.nick = nick
 		a.mu.Unlock()
-
+		a.ui.AddStatus(fmt.Sprintf("Nick set to %s (will be used on connect)", nick))
-		a.ui.AddStatus(
-			fmt.Sprintf(
-				"Nick set to %s (will be used on connect)",
-				nick,
-			),
-		)
-
 		return
 	}
 
@@ -238,12 +175,7 @@ func (a *App) cmdNick(nick string) {
 		Body:    []string{nick},
 	})
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Nick change failed: %v", err))
-			fmt.Sprintf(
-				"[red]Nick change failed: %v", err,
-			),
-		)
-
 		return
 	}
 
@@ -251,20 +183,15 @@ func (a *App) cmdNick(nick string) {
 	a.nick = nick
 	target := a.target
 	a.mu.Unlock()
-
 	a.ui.SetStatus(nick, target, "connected")
-	a.ui.AddStatus(
+	a.ui.AddStatus(fmt.Sprintf("Nick changed to %s", nick))
-		"Nick changed to " + nick,
-	)
 }
 
 func (a *App) cmdJoin(channel string) {
 	if channel == "" {
 		a.ui.AddStatus("[red]Usage: /join #channel")
-
 		return
 	}
-
 	if !strings.HasPrefix(channel, "#") {
 		channel = "#" + channel
 	}
@@ -272,19 +199,14 @@ func (a *App) cmdJoin(channel string) {
 	a.mu.Lock()
 	connected := a.connected
 	a.mu.Unlock()
-
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
 
 	err := a.client.JoinChannel(channel)
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Join failed: %v", err))
-			fmt.Sprintf("[red]Join failed: %v", err),
-		)
-
 		return
 	}
 
@@ -294,55 +216,39 @@ func (a *App) cmdJoin(channel string) {
 	a.mu.Unlock()
 
 	a.ui.SwitchToBuffer(channel)
-	a.ui.AddLine(
+	a.ui.AddLine(channel, fmt.Sprintf("[yellow]*** Joined %s", channel))
-		channel,
-		"[yellow]*** Joined "+channel,
-	)
 	a.ui.SetStatus(nick, channel, "connected")
 }
 
 func (a *App) cmdPart(channel string) {
 	a.mu.Lock()
-
 	if channel == "" {
 		channel = a.target
 	}
-
 	connected := a.connected
 	a.mu.Unlock()
 
 	if channel == "" || !strings.HasPrefix(channel, "#") {
 		a.ui.AddStatus("[red]No channel to part")
-
 		return
 	}
-
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
 
 	err := a.client.PartChannel(channel)
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Part failed: %v", err))
-			fmt.Sprintf("[red]Part failed: %v", err),
-		)
-
 		return
 	}
 
-	a.ui.AddLine(
+	a.ui.AddLine(channel, fmt.Sprintf("[yellow]*** Left %s", channel))
-		channel,
-		"[yellow]*** Left "+channel,
-	)
 
 	a.mu.Lock()
-
 	if a.target == channel {
 		a.target = ""
 	}
-
 	nick := a.nick
 	a.mu.Unlock()
 
@@ -351,23 +257,19 @@ func (a *App) cmdPart(channel string) {
 }
 
 func (a *App) cmdMsg(args string) {
-	parts := strings.SplitN(args, " ", 2) //nolint:mnd // split into target+text
+	parts := strings.SplitN(args, " ", 2)
-	if len(parts) < 2 {                   //nolint:mnd // min args
+	if len(parts) < 2 {
 		a.ui.AddStatus("[red]Usage: /msg <nick> <text>")
-
 		return
 	}
-
 	target, text := parts[0], parts[1]
 
 	a.mu.Lock()
 	connected := a.connected
 	nick := a.nick
 	a.mu.Unlock()
-
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
 
@@ -377,28 +279,17 @@ func (a *App) cmdMsg(args string) {
 		Body:    []string{text},
 	})
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Send failed: %v", err))
-			fmt.Sprintf("[red]Send failed: %v", err),
-		)
-
 		return
 	}
 
 	ts := time.Now().Format("15:04")
-
+	a.ui.AddLine(target, fmt.Sprintf("[gray]%s [green]<%s>[white] %s", ts, nick, text))
-	a.ui.AddLine(
-		target,
-		fmt.Sprintf(
-			"[gray]%s [green]<%s>[white] %s",
-			ts, nick, text,
-		),
-	)
 }
 
 func (a *App) cmdQuery(nick string) {
 	if nick == "" {
 		a.ui.AddStatus("[red]Usage: /query <nick>")
-
 		return
 	}
 
@@ -419,29 +310,22 @@ func (a *App) cmdTopic(args string) {
 
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
-
 	if !strings.HasPrefix(target, "#") {
 		a.ui.AddStatus("[red]Not in a channel")
-
 		return
 	}
 
 	if args == "" {
+		// Query topic.
 		err := a.client.SendMessage(&api.Message{
 			Command: "TOPIC",
 			To:      target,
 		})
 		if err != nil {
-			a.ui.AddStatus(
+			a.ui.AddStatus(fmt.Sprintf("[red]Topic query failed: %v", err))
-				fmt.Sprintf(
-					"[red]Topic query failed: %v", err,
-				),
-			)
 		}
-
 		return
 	}
 
@@ -451,11 +335,7 @@ func (a *App) cmdTopic(args string) {
 		Body:    []string{args},
 	})
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Topic set failed: %v", err))
-			fmt.Sprintf(
-				"[red]Topic set failed: %v", err,
-			),
-		)
 	}
 }
 
@@ -467,32 +347,20 @@ func (a *App) cmdNames() {
 
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
-
 	if !strings.HasPrefix(target, "#") {
 		a.ui.AddStatus("[red]Not in a channel")
-
 		return
 	}
 
 	members, err := a.client.GetMembers(target)
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]Names failed: %v", err))
-			fmt.Sprintf("[red]Names failed: %v", err),
-		)
-
 		return
 	}
 
-	a.ui.AddLine(
+	a.ui.AddLine(target, fmt.Sprintf("[cyan]*** Members of %s: %s", target, strings.Join(members, " ")))
-		target,
-		fmt.Sprintf(
-			"[cyan]*** Members of %s: %s",
-			target, strings.Join(members, " "),
-		),
-	)
 }
 
 func (a *App) cmdList() {
@@ -502,55 +370,46 @@ func (a *App) cmdList() {
 
 	if !connected {
 		a.ui.AddStatus("[red]Not connected")
-
 		return
 	}
 
 	channels, err := a.client.ListChannels()
 	if err != nil {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("[red]List failed: %v", err))
-			fmt.Sprintf("[red]List failed: %v", err),
-		)
-
 		return
 	}
 
 	a.ui.AddStatus("[cyan]*** Channel list:")
-
 	for _, ch := range channels {
-		a.ui.AddStatus(
+		a.ui.AddStatus(fmt.Sprintf("  %s (%d members) %s", ch.Name, ch.Members, ch.Topic))
-			fmt.Sprintf(
-				"  %s (%d members) %s",
-				ch.Name, ch.Members, ch.Topic,
-			),
-		)
 	}
-
 	a.ui.AddStatus("[cyan]*** End of channel list")
 }
 
 func (a *App) cmdWindow(args string) {
 	if args == "" {
 		a.ui.AddStatus("[red]Usage: /window <number>")
-
 		return
 	}
-
+	n := 0
-	n, _ := strconv.Atoi(args)
+	fmt.Sscanf(args, "%d", &n)
 	a.ui.SwitchBuffer(n)
 
 	a.mu.Lock()
+	if n < a.ui.BufferCount() && n >= 0 {
+		// Update target to the buffer name.
+		// Needs to be done carefully.
+	}
 	nick := a.nick
 	a.mu.Unlock()
 
-	if n >= 0 && n < a.ui.BufferCount() {
+	// Update target based on buffer.
+	if n < a.ui.BufferCount() {
 		buf := a.ui.buffers[n]
-
 		if buf.Name != "(status)" {
 			a.mu.Lock()
 			a.target = buf.Name
 			a.mu.Unlock()
-
 			a.ui.SetStatus(nick, buf.Name, "connected")
 		} else {
 			a.ui.SetStatus(nick, "", "connected")
@@ -560,17 +419,12 @@ func (a *App) cmdWindow(args string) {
 
 func (a *App) cmdQuit() {
 	a.mu.Lock()
-
 	if a.connected && a.client != nil {
-		_ = a.client.SendMessage(
+		_ = a.client.SendMessage(&api.Message{Command: "QUIT"})
-			&api.Message{Command: "QUIT"},
-		)
 	}
-
 	if a.stopPoll != nil {
 		close(a.stopPoll)
 	}
-
 	a.mu.Unlock()
 	a.ui.Stop()
 }
@@ -578,21 +432,20 @@ func (a *App) cmdQuit() {
 func (a *App) cmdHelp() {
 	help := []string{
 		"[cyan]*** chat-cli commands:",
-		"  /connect <url>  \u2014 Connect to server",
+		"  /connect <url>  — Connect to server",
-		"  /nick <name>    \u2014 Change nickname",
+		"  /nick <name>    — Change nickname",
-		"  /join #channel  \u2014 Join channel",
+		"  /join #channel  — Join channel",
-		"  /part [#chan]    \u2014 Leave channel",
+		"  /part [#chan]    — Leave channel",
-		"  /msg <nick> <text> \u2014 Send DM",
+		"  /msg <nick> <text> — Send DM",
-		"  /query <nick>   \u2014 Open DM window",
+		"  /query <nick>   — Open DM window",
-		"  /topic [text]   \u2014 View/set topic",
+		"  /topic [text]   — View/set topic",
-		"  /names          \u2014 List channel members",
+		"  /names          — List channel members",
-		"  /list           \u2014 List channels",
+		"  /list           — List channels",
-		"  /window <n>     \u2014 Switch buffer (Alt+0-9)",
+		"  /window <n>     — Switch buffer (Alt+0-9)",
-		"  /quit           \u2014 Disconnect and exit",
+		"  /quit           — Disconnect and exit",
-		"  /help           \u2014 This help",
+		"  /help           — This help",
 		"  Plain text sends to current target.",
 	}
-
 	for _, line := range help {
 		a.ui.AddStatus(line)
 	}
@@ -616,31 +469,32 @@ func (a *App) pollLoop() {
 			return
 		}
 
-		msgs, err := client.PollMessages(
+		msgs, err := client.PollMessages(lastID, 15)
-			lastID, pollTimeoutSec,
-		)
 		if err != nil {
-			time.Sleep(retryDelay)
+			// Transient error — retry after delay.
-
+			time.Sleep(2 * time.Second)
 			continue
 		}
 
-		for i := range msgs {
+		for _, msg := range msgs {
-			a.handleServerMessage(&msgs[i])
+			a.handleServerMessage(&msg)
-
+			if msg.ID != "" {
-			if msgs[i].ID != "" {
 				a.mu.Lock()
-				a.lastMsgID = msgs[i].ID
+				a.lastMsgID = msg.ID
 				a.mu.Unlock()
 			}
 		}
 	}
 }
 
-func (a *App) handleServerMessage(
+func (a *App) handleServerMessage(msg *api.Message) {
-	msg *api.Message,
+	ts := ""
-) {
+	if msg.TS != "" {
-	ts := a.parseMessageTS(msg)
+		t := msg.ParseTS()
+		ts = t.Local().Format("15:04")
+	} else {
+		ts = time.Now().Format("15:04")
+	}
 
 	a.mu.Lock()
 	myNick := a.nick
@@ -648,203 +502,79 @@ func (a *App) handleServerMessage(
 
 	switch msg.Command {
 	case "PRIVMSG":
-		a.handlePrivmsgMsg(msg, ts, myNick)
+		lines := msg.BodyLines()
+		text := strings.Join(lines, " ")
+		if msg.From == myNick {
+			// Skip our own echoed messages (already displayed locally).
+			return
+		}
+		target := msg.To
+		if !strings.HasPrefix(target, "#") {
+			// DM — use sender's nick as buffer name.
+			target = msg.From
+		}
+		a.ui.AddLine(target, fmt.Sprintf("[gray]%s [green]<%s>[white] %s", ts, msg.From, text))
+
 	case "JOIN":
-		a.handleJoinMsg(msg, ts)
+		target := msg.To
+		if target != "" {
+			a.ui.AddLine(target, fmt.Sprintf("[gray]%s [yellow]*** %s has joined %s", ts, msg.From, target))
+		}
+
 	case "PART":
-		a.handlePartMsg(msg, ts)
+		target := msg.To
+		lines := msg.BodyLines()
+		reason := strings.Join(lines, " ")
+		if target != "" {
+			if reason != "" {
+				a.ui.AddLine(target, fmt.Sprintf("[gray]%s [yellow]*** %s has left %s (%s)", ts, msg.From, target, reason))
+			} else {
+				a.ui.AddLine(target, fmt.Sprintf("[gray]%s [yellow]*** %s has left %s", ts, msg.From, target))
+			}
+		}
+
 	case "QUIT":
-		a.handleQuitMsg(msg, ts)
+		lines := msg.BodyLines()
+		reason := strings.Join(lines, " ")
+		if reason != "" {
+			a.ui.AddStatus(fmt.Sprintf("[gray]%s [yellow]*** %s has quit (%s)", ts, msg.From, reason))
+		} else {
+			a.ui.AddStatus(fmt.Sprintf("[gray]%s [yellow]*** %s has quit", ts, msg.From))
+		}
+
 	case "NICK":
-		a.handleNickMsg(msg, ts, myNick)
+		lines := msg.BodyLines()
+		newNick := ""
+		if len(lines) > 0 {
+			newNick = lines[0]
+		}
+		if msg.From == myNick && newNick != "" {
+			a.mu.Lock()
+			a.nick = newNick
+			target := a.target
+			a.mu.Unlock()
+			a.ui.SetStatus(newNick, target, "connected")
+		}
+		a.ui.AddStatus(fmt.Sprintf("[gray]%s [yellow]*** %s is now known as %s", ts, msg.From, newNick))
+
 	case "NOTICE":
-		a.handleNoticeMsg(msg, ts)
+		lines := msg.BodyLines()
+		text := strings.Join(lines, " ")
+		a.ui.AddStatus(fmt.Sprintf("[gray]%s [magenta]--%s-- %s", ts, msg.From, text))
+
 	case "TOPIC":
-		a.handleTopicMsg(msg, ts)
+		lines := msg.BodyLines()
+		text := strings.Join(lines, " ")
+		if msg.To != "" {
+			a.ui.AddLine(msg.To, fmt.Sprintf("[gray]%s [cyan]*** %s set topic: %s", ts, msg.From, text))
+		}
+
 	default:
-		a.handleDefaultMsg(msg, ts)
+		// Numeric replies and other messages → status window.
+		lines := msg.BodyLines()
+		text := strings.Join(lines, " ")
+		if text != "" {
+			a.ui.AddStatus(fmt.Sprintf("[gray]%s [white][%s] %s", ts, msg.Command, text))
+		}
 	}
 }
-
-func (a *App) parseMessageTS(msg *api.Message) string {
-	if msg.TS != "" {
-		t := msg.ParseTS()
-
-		return t.In(time.Local).Format("15:04") //nolint:gosmopolitan // CLI uses local time
-	}
-
-	return time.Now().Format("15:04")
-}
-
-func (a *App) handlePrivmsgMsg(
-	msg *api.Message,
-	ts, myNick string,
-) {
-	lines := msg.BodyLines()
-	text := strings.Join(lines, " ")
-
-	if msg.From == myNick {
-		return
-	}
-
-	target := msg.To
-	if !strings.HasPrefix(target, "#") {
-		target = msg.From
-	}
-
-	a.ui.AddLine(
-		target,
-		fmt.Sprintf(
-			"[gray]%s [green]<%s>[white] %s",
-			ts, msg.From, text,
-		),
-	)
-}
-
-func (a *App) handleJoinMsg(
-	msg *api.Message, ts string,
-) {
-	target := msg.To
-	if target == "" {
-		return
-	}
-
-	a.ui.AddLine(
-		target,
-		fmt.Sprintf(
-			"[gray]%s [yellow]*** %s has joined %s",
-			ts, msg.From, target,
-		),
-	)
-}
-
-func (a *App) handlePartMsg(
-	msg *api.Message, ts string,
-) {
-	target := msg.To
-	if target == "" {
-		return
-	}
-
-	lines := msg.BodyLines()
-	reason := strings.Join(lines, " ")
-
-	if reason != "" {
-		a.ui.AddLine(
-			target,
-			fmt.Sprintf(
-				"[gray]%s [yellow]*** %s has left %s (%s)",
-				ts, msg.From, target, reason,
-			),
-		)
-	} else {
-		a.ui.AddLine(
-			target,
-			fmt.Sprintf(
-				"[gray]%s [yellow]*** %s has left %s",
-				ts, msg.From, target,
-			),
-		)
-	}
-}
-
-func (a *App) handleQuitMsg(
-	msg *api.Message, ts string,
-) {
-	lines := msg.BodyLines()
-	reason := strings.Join(lines, " ")
-
-	if reason != "" {
-		a.ui.AddStatus(
-			fmt.Sprintf(
-				"[gray]%s [yellow]*** %s has quit (%s)",
-				ts, msg.From, reason,
-			),
-		)
-	} else {
-		a.ui.AddStatus(
-			fmt.Sprintf(
-				"[gray]%s [yellow]*** %s has quit",
-				ts, msg.From,
-			),
-		)
-	}
-}
-
-func (a *App) handleNickMsg(
-	msg *api.Message, ts, myNick string,
-) {
-	lines := msg.BodyLines()
-
-	newNick := ""
-	if len(lines) > 0 {
-		newNick = lines[0]
-	}
-
-	if msg.From == myNick && newNick != "" {
-		a.mu.Lock()
-		a.nick = newNick
-		target := a.target
-		a.mu.Unlock()
-
-		a.ui.SetStatus(newNick, target, "connected")
-	}
-
-	a.ui.AddStatus(
-		fmt.Sprintf(
-			"[gray]%s [yellow]*** %s is now known as %s",
-			ts, msg.From, newNick,
-		),
-	)
-}
-
-func (a *App) handleNoticeMsg(
-	msg *api.Message, ts string,
-) {
-	lines := msg.BodyLines()
-	text := strings.Join(lines, " ")
-
-	a.ui.AddStatus(
-		fmt.Sprintf(
-			"[gray]%s [magenta]--%s-- %s",
-			ts, msg.From, text,
-		),
-	)
-}
-
-func (a *App) handleTopicMsg(
-	msg *api.Message, ts string,
-) {
-	if msg.To == "" {
-		return
-	}
-
-	lines := msg.BodyLines()
-	text := strings.Join(lines, " ")
-
-	a.ui.AddLine(
-		msg.To,
-		fmt.Sprintf(
-			"[gray]%s [cyan]*** %s set topic: %s",
-			ts, msg.From, text,
-		),
-	)
-}
-
-func (a *App) handleDefaultMsg(
-	msg *api.Message, ts string,
-) {
-	lines := msg.BodyLines()
-	text := strings.Join(lines, " ")
-
-	if text == "" {
-		return
-	}
-
-	a.ui.AddStatus(
-		fmt.Sprintf(
-			"[gray]%s [white][%s] %s",
-			ts, msg.Command, text,
-		),
-	)
-}

cmd/chat-cli/ui.go

@@ -31,7 +31,6 @@ type UI struct {
 }
 
 // NewUI creates the tview-based IRC-like UI.
-
 func NewUI() *UI {
 	ui := &UI{
 		app: tview.NewApplication(),
@@ -40,145 +39,7 @@ func NewUI() *UI {
 		},
 	}
 
-	ui.setupMessages()
+	// Message area.
-	ui.setupStatusBar()
-	ui.setupInput()
-	ui.setupKeybindings()
-	ui.setupLayout()
-
-	return ui
-}
-
-// Run starts the UI event loop (blocks).
-
-func (ui *UI) Run() error {
-	return ui.app.Run()
-}
-
-// Stop stops the UI.
-
-func (ui *UI) Stop() {
-	ui.app.Stop()
-}
-
-// OnInput sets the callback for user input.
-
-func (ui *UI) OnInput(fn func(string)) {
-	ui.onInput = fn
-}
-
-// AddLine adds a line to the specified buffer.
-
-func (ui *UI) AddLine(bufferName string, line string) {
-	ui.app.QueueUpdateDraw(func() {
-		buf := ui.getOrCreateBuffer(bufferName)
-		buf.Lines = append(buf.Lines, line)
-
-		// Mark unread if not currently viewing this buffer.
-		if ui.buffers[ui.currentBuffer] != buf {
-			buf.Unread++
-
-			ui.refreshStatus()
-		}
-
-		// If viewing this buffer, append to display.
-		if ui.buffers[ui.currentBuffer] == buf {
-			_, _ = fmt.Fprintln(ui.messages, line)
-		}
-	})
-}
-
-// AddStatus adds a line to the status buffer (buffer 0).
-
-func (ui *UI) AddStatus(line string) {
-	ts := time.Now().Format("15:04")
-
-	ui.AddLine(
-		"(status)",
-		fmt.Sprintf("[gray]%s[white] %s", ts, line),
-	)
-}
-
-// SwitchBuffer switches to the buffer at index n.
-
-func (ui *UI) SwitchBuffer(n int) {
-	ui.app.QueueUpdateDraw(func() {
-		if n < 0 || n >= len(ui.buffers) {
-			return
-		}
-
-		ui.currentBuffer = n
-
-		buf := ui.buffers[n]
-		buf.Unread = 0
-
-		ui.messages.Clear()
-
-		for _, line := range buf.Lines {
-			_, _ = fmt.Fprintln(ui.messages, line)
-		}
-
-		ui.messages.ScrollToEnd()
-		ui.refreshStatus()
-	})
-}
-
-// SwitchToBuffer switches to the named buffer, creating it
-
-func (ui *UI) SwitchToBuffer(name string) {
-	ui.app.QueueUpdateDraw(func() {
-		buf := ui.getOrCreateBuffer(name)
-
-		for i, b := range ui.buffers {
-			if b == buf {
-				ui.currentBuffer = i
-
-				break
-			}
-		}
-
-		buf.Unread = 0
-
-		ui.messages.Clear()
-
-		for _, line := range buf.Lines {
-			_, _ = fmt.Fprintln(ui.messages, line)
-		}
-
-		ui.messages.ScrollToEnd()
-		ui.refreshStatus()
-	})
-}
-
-// SetStatus updates the status bar text.
-
-func (ui *UI) SetStatus(
-	nick, target, connStatus string,
-) {
-	ui.app.QueueUpdateDraw(func() {
-		ui.refreshStatusWith(nick, target, connStatus)
-	})
-}
-
-// BufferCount returns the number of buffers.
-
-func (ui *UI) BufferCount() int {
-	return len(ui.buffers)
-}
-
-// BufferIndex returns the index of a named buffer, or -1.
-
-func (ui *UI) BufferIndex(name string) int {
-	for i, buf := range ui.buffers {
-		if buf.Name == name {
-			return i
-		}
-	}
-
-	return -1
-}
-
-func (ui *UI) setupMessages() {
 	ui.messages = tview.NewTextView().
 		SetDynamicColors(true).
 		SetScrollable(true).
@@ -187,109 +48,162 @@ func (ui *UI) setupMessages() {
 			ui.app.Draw()
 		})
 	ui.messages.SetBorder(false)
-}
 
-func (ui *UI) setupStatusBar() {
+	// Status bar.
 	ui.statusBar = tview.NewTextView().
 		SetDynamicColors(true)
 	ui.statusBar.SetBackgroundColor(tcell.ColorNavy)
 	ui.statusBar.SetTextColor(tcell.ColorWhite)
-}
 
-func (ui *UI) setupInput() {
+	// Input field.
 	ui.input = tview.NewInputField().
 		SetFieldBackgroundColor(tcell.ColorBlack).
 		SetFieldTextColor(tcell.ColorWhite)
-
 	ui.input.SetDoneFunc(func(key tcell.Key) {
-		if key != tcell.KeyEnter {
+		if key == tcell.KeyEnter {
-			return
+			text := ui.input.GetText()
-		}
+			if text == "" {
-
+				return
-		text := ui.input.GetText()
+			}
-		if text == "" {
+			ui.input.SetText("")
-			return
+			if ui.onInput != nil {
-		}
+				ui.onInput(text)
-
+			}
-		ui.input.SetText("")
-
-		if ui.onInput != nil {
-			ui.onInput(text)
 		}
 	})
-}
-
-func (ui *UI) setupKeybindings() {
-	ui.app.SetInputCapture(
-		func(event *tcell.EventKey) *tcell.EventKey {
-			if event.Modifiers()&tcell.ModAlt == 0 {
-				return event
-			}
 
+	// Capture Alt+N for window switching.
+	ui.app.SetInputCapture(func(event *tcell.EventKey) *tcell.EventKey {
+		if event.Modifiers()&tcell.ModAlt != 0 {
 			r := event.Rune()
 			if r >= '0' && r <= '9' {
 				idx := int(r - '0')
 				ui.SwitchBuffer(idx)
-
 				return nil
 			}
+		}
+		return event
+	})
 
-			return event
+	// Layout: messages on top, status bar, input at bottom.
-		},
+	ui.layout = tview.NewFlex().SetDirection(tview.FlexRow).
-	)
-}
-
-func (ui *UI) setupLayout() {
-	ui.layout = tview.NewFlex().
-		SetDirection(tview.FlexRow).
 		AddItem(ui.messages, 0, 1, false).
 		AddItem(ui.statusBar, 1, 0, false).
 		AddItem(ui.input, 1, 0, true)
 
 	ui.app.SetRoot(ui.layout, true)
 	ui.app.SetFocus(ui.input)
+
+	return ui
 }
 
-// if needed.
+// Run starts the UI event loop (blocks).
+func (ui *UI) Run() error {
+	return ui.app.Run()
+}
+
+// Stop stops the UI.
+func (ui *UI) Stop() {
+	ui.app.Stop()
+}
+
+// OnInput sets the callback for user input.
+func (ui *UI) OnInput(fn func(string)) {
+	ui.onInput = fn
+}
+
+// AddLine adds a line to the specified buffer.
+func (ui *UI) AddLine(bufferName string, line string) {
+	ui.app.QueueUpdateDraw(func() {
+		buf := ui.getOrCreateBuffer(bufferName)
+		buf.Lines = append(buf.Lines, line)
+
+		// Mark unread if not currently viewing this buffer.
+		if ui.buffers[ui.currentBuffer] != buf {
+			buf.Unread++
+			ui.refreshStatus()
+		}
+
+		// If viewing this buffer, append to display.
+		if ui.buffers[ui.currentBuffer] == buf {
+			fmt.Fprintln(ui.messages, line)
+		}
+	})
+}
+
+// AddStatus adds a line to the status buffer (buffer 0).
+func (ui *UI) AddStatus(line string) {
+	ts := time.Now().Format("15:04")
+	ui.AddLine("(status)", fmt.Sprintf("[gray]%s[white] %s", ts, line))
+}
+
+// SwitchBuffer switches to the buffer at index n.
+func (ui *UI) SwitchBuffer(n int) {
+	ui.app.QueueUpdateDraw(func() {
+		if n < 0 || n >= len(ui.buffers) {
+			return
+		}
+		ui.currentBuffer = n
+		buf := ui.buffers[n]
+		buf.Unread = 0
+		ui.messages.Clear()
+		for _, line := range buf.Lines {
+			fmt.Fprintln(ui.messages, line)
+		}
+		ui.messages.ScrollToEnd()
+		ui.refreshStatus()
+	})
+}
+
+// SwitchToBuffer switches to the named buffer, creating it if needed.
+func (ui *UI) SwitchToBuffer(name string) {
+	ui.app.QueueUpdateDraw(func() {
+		buf := ui.getOrCreateBuffer(name)
+		for i, b := range ui.buffers {
+			if b == buf {
+				ui.currentBuffer = i
+				break
+			}
+		}
+		buf.Unread = 0
+		ui.messages.Clear()
+		for _, line := range buf.Lines {
+			fmt.Fprintln(ui.messages, line)
+		}
+		ui.messages.ScrollToEnd()
+		ui.refreshStatus()
+	})
+}
+
+// SetStatus updates the status bar text.
+func (ui *UI) SetStatus(nick, target, connStatus string) {
+	ui.app.QueueUpdateDraw(func() {
+		ui.refreshStatusWith(nick, target, connStatus)
+	})
+}
 
 func (ui *UI) refreshStatus() {
-	// Rebuilt from app state by parent QueueUpdateDraw.
+	// Will be called from the main goroutine via QueueUpdateDraw parent.
+	// Rebuild status from app state — caller must provide context.
 }
 
-func (ui *UI) refreshStatusWith(
+func (ui *UI) refreshStatusWith(nick, target, connStatus string) {
-	nick, target, connStatus string,
-) {
 	var unreadParts []string
-
 	for i, buf := range ui.buffers {
 		if buf.Unread > 0 {
-			unreadParts = append(
+			unreadParts = append(unreadParts, fmt.Sprintf("%d:%s(%d)", i, buf.Name, buf.Unread))
-				unreadParts,
-				fmt.Sprintf(
-					"%d:%s(%d)", i, buf.Name, buf.Unread,
-				),
-			)
 		}
 	}
-
 	unread := ""
 	if len(unreadParts) > 0 {
-		unread = " [Act: " +
+		unread = " [Act: " + strings.Join(unreadParts, ",") + "]"
-			strings.Join(unreadParts, ",") + "]"
 	}
 
-	bufInfo := fmt.Sprintf(
+	bufInfo := fmt.Sprintf("[%d:%s]", ui.currentBuffer, ui.buffers[ui.currentBuffer].Name)
-		"[%d:%s]",
-		ui.currentBuffer,
-		ui.buffers[ui.currentBuffer].Name,
-	)
 
 	ui.statusBar.Clear()
-
+	fmt.Fprintf(ui.statusBar, " [%s] %s %s %s%s",
-	_, _ = fmt.Fprintf(
+		connStatus, nick, bufInfo, target, unread)
-		ui.statusBar, " [%s] %s %s %s%s",
-		connStatus, nick, bufInfo, target, unread,
-	)
 }
 
 func (ui *UI) getOrCreateBuffer(name string) *Buffer {
@@ -298,9 +212,22 @@ func (ui *UI) getOrCreateBuffer(name string) *Buffer {
 			return buf
 		}
 	}
-
 	buf := &Buffer{Name: name}
 	ui.buffers = append(ui.buffers, buf)
-
 	return buf
 }
+
+// BufferCount returns the number of buffers.
+func (ui *UI) BufferCount() int {
+	return len(ui.buffers)
+}
+
+// BufferIndex returns the index of a named buffer, or -1.
+func (ui *UI) BufferIndex(name string) int {
+	for i, buf := range ui.buffers {
+		if buf.Name == name {
+			return i
+		}
+	}
+	return -1
+}

internal/db/db.go

@@ -88,10 +88,8 @@ func NewTest(dsn string) (*Database, error) {
 	}
 
 	// Item 9: Enable foreign keys
-	_, err = d.Exec("PRAGMA foreign_keys = ON") //nolint:noctx // no context in sql.Open path
+	if _, err := d.Exec("PRAGMA foreign_keys = ON"); err != nil {
-	if err != nil {
 		_ = d.Close()
-
 		return nil, fmt.Errorf("enable foreign keys: %w", err)
 	}
 
@@ -164,8 +162,8 @@ func (s *Database) GetChannelByID(
 	return c, nil
 }
 
-// GetUserByNickModel looks up a user by their nick.
+// GetUserByNick looks up a user by their nick.
-func (s *Database) GetUserByNickModel(
+func (s *Database) GetUserByNick(
 	ctx context.Context,
 	nick string,
 ) (*models.User, error) {
@@ -187,8 +185,8 @@ func (s *Database) GetUserByNickModel(
 	return u, nil
 }
 
-// GetUserByTokenModel looks up a user by their auth token.
+// GetUserByToken looks up a user by their auth token.
-func (s *Database) GetUserByTokenModel(
+func (s *Database) GetUserByToken(
 	ctx context.Context,
 	token string,
 ) (*models.User, error) {
@@ -221,7 +219,6 @@ func (s *Database) DeleteAuthToken(
 	_, err := s.db.ExecContext(ctx,
 		`DELETE FROM auth_tokens WHERE token = ?`, token,
 	)
-
 	return err
 }
 
@@ -234,12 +231,11 @@ func (s *Database) UpdateUserLastSeen(
 		`UPDATE users SET last_seen_at = CURRENT_TIMESTAMP WHERE id = ?`,
 		userID,
 	)
-
 	return err
 }
 
-// CreateUserModel inserts a new user into the database.
+// CreateUser inserts a new user into the database.
-func (s *Database) CreateUserModel(
+func (s *Database) CreateUser(
 	ctx context.Context,
 	id, nick, passwordHash string,
 ) (*models.User, error) {
@@ -398,7 +394,6 @@ func (s *Database) DequeueMessages(
 	if err != nil {
 		return nil, err
 	}
-
 	defer func() { _ = rows.Close() }()
 
 	entries := []*models.MessageQueueEntry{}
@@ -428,14 +423,14 @@ func (s *Database) AckMessages(
 	}
 
 	placeholders := make([]string, len(entryIDs))
-	args := make([]any, len(entryIDs))
+	args := make([]interface{}, len(entryIDs))
 
 	for i, id := range entryIDs {
 		placeholders[i] = "?"
 		args[i] = id
 	}
 
-	query := fmt.Sprintf( //nolint:gosec // placeholders are ?, not user input
+	query := fmt.Sprintf(
 		"DELETE FROM message_queue WHERE id IN (%s)",
 		strings.Join(placeholders, ","),
 	)
@@ -554,8 +549,7 @@ func (s *Database) connect(ctx context.Context) error {
 	s.log.Info("database connected")
 
 	// Item 9: Enable foreign keys on every connection
-	_, err = s.db.ExecContext(ctx, "PRAGMA foreign_keys = ON")
+	if _, err := s.db.ExecContext(ctx, "PRAGMA foreign_keys = ON"); err != nil {
-	if err != nil {
 		return fmt.Errorf("enable foreign keys: %w", err)
 	}
 
@@ -682,54 +676,41 @@ func (s *Database) applyMigrations(
 			"version", m.version, "name", m.name,
 		)
 
-		err = s.executeMigration(ctx, m)
+		tx, err := s.db.BeginTx(ctx, nil)
 		if err != nil {
-			return err
+			return fmt.Errorf(
+				"begin tx for migration %d: %w", m.version, err,
+			)
+		}
+
+		_, err = tx.ExecContext(ctx, m.sql)
+		if err != nil {
+			_ = tx.Rollback()
+
+			return fmt.Errorf(
+				"apply migration %d (%s): %w",
+				m.version, m.name, err,
+			)
+		}
+
+		_, err = tx.ExecContext(ctx,
+			"INSERT INTO schema_migrations (version) VALUES (?)",
+			m.version,
+		)
+		if err != nil {
+			_ = tx.Rollback()
+
+			return fmt.Errorf(
+				"record migration %d: %w", m.version, err,
+			)
+		}
+
+		if err := tx.Commit(); err != nil {
+			return fmt.Errorf(
+				"commit migration %d: %w", m.version, err,
+			)
 		}
 	}
 
 	return nil
 }
-
-func (s *Database) executeMigration(
-	ctx context.Context,
-	m migration,
-) error {
-	tx, err := s.db.BeginTx(ctx, nil)
-	if err != nil {
-		return fmt.Errorf(
-			"begin tx for migration %d: %w", m.version, err,
-		)
-	}
-
-	_, err = tx.ExecContext(ctx, m.sql)
-	if err != nil {
-		_ = tx.Rollback()
-
-		return fmt.Errorf(
-			"apply migration %d (%s): %w",
-			m.version, m.name, err,
-		)
-	}
-
-	_, err = tx.ExecContext(ctx,
-		"INSERT INTO schema_migrations (version) VALUES (?)",
-		m.version,
-	)
-	if err != nil {
-		_ = tx.Rollback()
-
-		return fmt.Errorf(
-			"record migration %d: %w", m.version, err,
-		)
-	}
-
-	err = tx.Commit()
-	if err != nil {
-		return fmt.Errorf(
-			"commit migration %d: %w", m.version, err,
-		)
-	}
-
-	return nil
-}

internal/db/db_test.go

@@ -43,119 +43,213 @@ func TestCreateUser(t *testing.T) {
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	id, token, err := d.CreateUser(ctx, nickAlice)
+	u, err := d.CreateUser(ctx, "u1", nickAlice, "hash1")
 	if err != nil {
 		t.Fatalf("CreateUser: %v", err)
 	}
 
-	if id <= 0 {
+	if u.ID != "u1" || u.Nick != nickAlice {
-		t.Errorf("expected positive id, got %d", id)
+		t.Errorf("got user %+v", u)
-	}
-
-	if token == "" {
-		t.Error("expected non-empty token")
 	}
 }
 
-func TestGetUserByToken(t *testing.T) {
+func TestCreateAuthToken(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	_, token, _ := d.CreateUser(ctx, nickAlice)
+	_, err := d.CreateUser(ctx, "u1", nickAlice, "h")
-
-	id, nick, err := d.GetUserByToken(ctx, token)
 	if err != nil {
-		t.Fatalf("GetUserByToken: %v", err)
+		t.Fatalf("CreateUser: %v", err)
 	}
 
-	if id <= 0 || nick != nickAlice {
+	tok, err := d.CreateAuthToken(ctx, "tok1", "u1")
-		t.Errorf(
+	if err != nil {
-			"got id=%d nick=%s, want nick=%s",
+		t.Fatalf("CreateAuthToken: %v", err)
-			id, nick, nickAlice,
+	}
-		)
+
+	if tok.Token != "tok1" || tok.UserID != "u1" {
+		t.Errorf("unexpected token: %+v", tok)
+	}
+
+	u, err := tok.User(ctx)
+	if err != nil {
+		t.Fatalf("AuthToken.User: %v", err)
+	}
+
+	if u.ID != "u1" || u.Nick != nickAlice {
+		t.Errorf("AuthToken.User got %+v", u)
 	}
 }
 
-func TestGetUserByNick(t *testing.T) {
+func TestCreateChannel(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	origID, _, _ := d.CreateUser(ctx, nickAlice)
+	ch, err := d.CreateChannel(
-
+		ctx, "c1", "#general", "welcome", "+n",
-	id, err := d.GetUserByNick(ctx, nickAlice)
+	)
 	if err != nil {
-		t.Fatalf("GetUserByNick: %v", err)
+		t.Fatalf("CreateChannel: %v", err)
 	}
 
-	if id != origID {
+	if ch.ID != "c1" || ch.Name != "#general" {
-		t.Errorf("got id %d, want %d", id, origID)
+		t.Errorf("unexpected channel: %+v", ch)
 	}
 }
 
-func TestGetOrCreateChannel(t *testing.T) {
+func TestAddChannelMember(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	id1, err := d.GetOrCreateChannel(ctx, "#general")
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateChannel(ctx, "c1", "#general", "", "")
+
+	cm, err := d.AddChannelMember(ctx, "c1", "u1", "+o")
 	if err != nil {
-		t.Fatalf("GetOrCreateChannel: %v", err)
+		t.Fatalf("AddChannelMember: %v", err)
 	}
 
-	if id1 <= 0 {
+	if cm.ChannelID != "c1" || cm.Modes != "+o" {
-		t.Errorf("expected positive id, got %d", id1)
+		t.Errorf("unexpected member: %+v", cm)
-	}
-
-	// Same channel returns same ID.
-	id2, err := d.GetOrCreateChannel(ctx, "#general")
-	if err != nil {
-		t.Fatalf("GetOrCreateChannel(2): %v", err)
-	}
-
-	if id1 != id2 {
-		t.Errorf("got different ids: %d vs %d", id1, id2)
 	}
 }
 
-func TestJoinAndListChannels(t *testing.T) {
+func TestCreateMessage(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
-	ch1, _ := d.GetOrCreateChannel(ctx, "#alpha")
-	ch2, _ := d.GetOrCreateChannel(ctx, "#beta")
 
-	_ = d.JoinChannel(ctx, ch1, uid)
+	msg, err := d.CreateMessage(
-	_ = d.JoinChannel(ctx, ch2, uid)
+		ctx, "m1", "u1", nickAlice,
-
+		"#general", "message", "hello",
-	channels, err := d.ListChannels(ctx, uid)
+	)
 	if err != nil {
-		t.Fatalf("ListChannels: %v", err)
+		t.Fatalf("CreateMessage: %v", err)
+	}
+
+	if msg.ID != "m1" || msg.Body != "hello" {
+		t.Errorf("unexpected message: %+v", msg)
+	}
+}
+
+func TestQueueMessage(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateUser(ctx, "u2", nickBob, "h")
+	_, _ = d.CreateMessage(
+		ctx, "m1", "u1", nickAlice, "u2", "message", "hi",
+	)
+
+	mq, err := d.QueueMessage(ctx, "u2", "m1")
+	if err != nil {
+		t.Fatalf("QueueMessage: %v", err)
+	}
+
+	if mq.UserID != "u2" || mq.MessageID != "m1" {
+		t.Errorf("unexpected queue entry: %+v", mq)
+	}
+}
+
+func TestCreateSession(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+
+	sess, err := d.CreateSession(ctx, "s1", "u1")
+	if err != nil {
+		t.Fatalf("CreateSession: %v", err)
+	}
+
+	if sess.ID != "s1" || sess.UserID != "u1" {
+		t.Errorf("unexpected session: %+v", sess)
+	}
+
+	u, err := sess.User(ctx)
+	if err != nil {
+		t.Fatalf("Session.User: %v", err)
+	}
+
+	if u.ID != "u1" {
+		t.Errorf("Session.User got %v, want u1", u.ID)
+	}
+}
+
+func TestCreateServerLink(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	sl, err := d.CreateServerLink(
+		ctx, "sl1", "peer1",
+		"https://peer.example.com", "keyhash", true,
+	)
+	if err != nil {
+		t.Fatalf("CreateServerLink: %v", err)
+	}
+
+	if sl.ID != "sl1" || !sl.IsActive {
+		t.Errorf("unexpected server link: %+v", sl)
+	}
+}
+
+func TestUserChannels(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	u, _ := d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateChannel(ctx, "c1", "#alpha", "", "")
+	_, _ = d.CreateChannel(ctx, "c2", "#beta", "", "")
+	_, _ = d.AddChannelMember(ctx, "c1", "u1", "")
+	_, _ = d.AddChannelMember(ctx, "c2", "u1", "")
+
+	channels, err := u.Channels(ctx)
+	if err != nil {
+		t.Fatalf("User.Channels: %v", err)
 	}
 
 	if len(channels) != 2 {
 		t.Fatalf("expected 2 channels, got %d", len(channels))
 	}
+
+	if channels[0].Name != "#alpha" {
+		t.Errorf("first channel: got %s", channels[0].Name)
+	}
+
+	if channels[1].Name != "#beta" {
+		t.Errorf("second channel: got %s", channels[1].Name)
+	}
 }
 
-func TestListChannelsEmpty(t *testing.T) {
+func TestUserChannelsEmpty(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
+	u, _ := d.CreateUser(ctx, "u1", nickAlice, "h")
 
-	channels, err := d.ListChannels(ctx, uid)
+	channels, err := u.Channels(ctx)
 	if err != nil {
-		t.Fatalf("ListChannels: %v", err)
+		t.Fatalf("User.Channels: %v", err)
 	}
 
 	if len(channels) != 0 {
@@ -163,57 +257,60 @@ func TestListChannelsEmpty(t *testing.T) {
 	}
 }
 
-func TestPartChannel(t *testing.T) {
+func TestUserQueuedMessages(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
+	u, _ := d.CreateUser(ctx, "u1", nickAlice, "h")
-	chID, _ := d.GetOrCreateChannel(ctx, "#general")
+	_, _ = d.CreateUser(ctx, "u2", nickBob, "h")
 
-	_ = d.JoinChannel(ctx, chID, uid)
+	for i := range 3 {
-	_ = d.PartChannel(ctx, chID, uid)
+		id := fmt.Sprintf("m%d", i)
 
-	channels, err := d.ListChannels(ctx, uid)
+		_, _ = d.CreateMessage(
-	if err != nil {
+			ctx, id, "u2", nickBob, "u1",
-		t.Fatalf("ListChannels: %v", err)
+			"message", fmt.Sprintf("msg%d", i),
+		)
+
+		time.Sleep(10 * time.Millisecond)
+
+		_, _ = d.QueueMessage(ctx, "u1", id)
 	}
 
-	if len(channels) != 0 {
+	msgs, err := u.QueuedMessages(ctx)
-		t.Errorf("expected 0 after part, got %d", len(channels))
+	if err != nil {
+		t.Fatalf("User.QueuedMessages: %v", err)
+	}
+
+	if len(msgs) != 3 {
+		t.Fatalf("expected 3 messages, got %d", len(msgs))
+	}
+
+	for i, msg := range msgs {
+		want := fmt.Sprintf("msg%d", i)
+		if msg.Body != want {
+			t.Errorf("msg %d: got %q, want %q", i, msg.Body, want)
+		}
 	}
 }
 
-func TestSendAndGetMessages(t *testing.T) {
+func TestUserQueuedMessagesEmpty(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
+	u, _ := d.CreateUser(ctx, "u1", nickAlice, "h")
-	chID, _ := d.GetOrCreateChannel(ctx, "#general")
-	_ = d.JoinChannel(ctx, chID, uid)
 
-	_, err := d.SendMessage(ctx, chID, uid, "hello world")
+	msgs, err := u.QueuedMessages(ctx)
 	if err != nil {
-		t.Fatalf("SendMessage: %v", err)
+		t.Fatalf("User.QueuedMessages: %v", err)
 	}
 
-	msgs, err := d.GetMessages(ctx, chID, 0, 0)
+	if len(msgs) != 0 {
-	if err != nil {
+		t.Errorf("expected 0 messages, got %d", len(msgs))
-		t.Fatalf("GetMessages: %v", err)
-	}
-
-	if len(msgs) != 1 {
-		t.Fatalf("expected 1 message, got %d", len(msgs))
-	}
-
-	if msgs[0].Content != "hello world" {
-		t.Errorf(
-			"got content %q, want %q",
-			msgs[0].Content, "hello world",
-		)
 	}
 }
 
@@ -223,23 +320,35 @@ func TestChannelMembers(t *testing.T) {
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid1, _, _ := d.CreateUser(ctx, nickAlice)
+	ch, _ := d.CreateChannel(ctx, "c1", "#general", "", "")
-	uid2, _, _ := d.CreateUser(ctx, nickBob)
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
-	uid3, _, _ := d.CreateUser(ctx, nickCharlie)
+	_, _ = d.CreateUser(ctx, "u2", nickBob, "h")
-	chID, _ := d.GetOrCreateChannel(ctx, "#general")
+	_, _ = d.CreateUser(ctx, "u3", nickCharlie, "h")
+	_, _ = d.AddChannelMember(ctx, "c1", "u1", "+o")
+	_, _ = d.AddChannelMember(ctx, "c1", "u2", "+v")
+	_, _ = d.AddChannelMember(ctx, "c1", "u3", "")
 
-	_ = d.JoinChannel(ctx, chID, uid1)
+	members, err := ch.Members(ctx)
-	_ = d.JoinChannel(ctx, chID, uid2)
-	_ = d.JoinChannel(ctx, chID, uid3)
-
-	members, err := d.ChannelMembers(ctx, chID)
 	if err != nil {
-		t.Fatalf("ChannelMembers: %v", err)
+		t.Fatalf("Channel.Members: %v", err)
 	}
 
 	if len(members) != 3 {
 		t.Fatalf("expected 3 members, got %d", len(members))
 	}
+
+	nicks := map[string]bool{}
+	for _, m := range members {
+		nicks[m.Nick] = true
+	}
+
+	for _, want := range []string{
+		nickAlice, nickBob, nickCharlie,
+	} {
+		if !nicks[want] {
+			t.Errorf("missing nick %q", want)
+		}
+	}
 }
 
 func TestChannelMembersEmpty(t *testing.T) {
@@ -248,11 +357,11 @@ func TestChannelMembersEmpty(t *testing.T) {
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	chID, _ := d.GetOrCreateChannel(ctx, "#empty")
+	ch, _ := d.CreateChannel(ctx, "c1", "#empty", "", "")
 
-	members, err := d.ChannelMembers(ctx, chID)
+	members, err := ch.Members(ctx)
 	if err != nil {
-		t.Fatalf("ChannelMembers: %v", err)
+		t.Fatalf("Channel.Members: %v", err)
 	}
 
 	if len(members) != 0 {
@@ -260,166 +369,126 @@ func TestChannelMembersEmpty(t *testing.T) {
 	}
 }
 
-func TestSendDM(t *testing.T) {
+func TestChannelRecentMessages(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	uid1, _, _ := d.CreateUser(ctx, nickAlice)
+	ch, _ := d.CreateChannel(ctx, "c1", "#general", "", "")
-	uid2, _, _ := d.CreateUser(ctx, nickBob)
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
-
-	msgID, err := d.SendDM(ctx, uid1, uid2, "hey bob")
-	if err != nil {
-		t.Fatalf("SendDM: %v", err)
-	}
-
-	if msgID <= 0 {
-		t.Errorf("expected positive msgID, got %d", msgID)
-	}
-
-	msgs, err := d.GetDMs(ctx, uid1, uid2, 0, 0)
-	if err != nil {
-		t.Fatalf("GetDMs: %v", err)
-	}
-
-	if len(msgs) != 1 {
-		t.Fatalf("expected 1 DM, got %d", len(msgs))
-	}
-
-	if msgs[0].Content != "hey bob" {
-		t.Errorf("got %q, want %q", msgs[0].Content, "hey bob")
-	}
-}
-
-func TestPollMessages(t *testing.T) {
-	t.Parallel()
-
-	d := setupTestDB(t)
-	ctx := context.Background()
-
-	uid1, _, _ := d.CreateUser(ctx, nickAlice)
-	uid2, _, _ := d.CreateUser(ctx, nickBob)
-	chID, _ := d.GetOrCreateChannel(ctx, "#general")
-
-	_ = d.JoinChannel(ctx, chID, uid1)
-	_ = d.JoinChannel(ctx, chID, uid2)
-
-	_, _ = d.SendMessage(ctx, chID, uid2, "hello")
-	_, _ = d.SendDM(ctx, uid2, uid1, "private")
-
-	time.Sleep(10 * time.Millisecond)
-
-	msgs, err := d.PollMessages(ctx, uid1, 0, 0)
-	if err != nil {
-		t.Fatalf("PollMessages: %v", err)
-	}
-
-	if len(msgs) < 2 {
-		t.Fatalf("expected >=2 messages, got %d", len(msgs))
-	}
-}
-
-func TestChangeNick(t *testing.T) {
-	t.Parallel()
-
-	d := setupTestDB(t)
-	ctx := context.Background()
-
-	_, token, _ := d.CreateUser(ctx, nickAlice)
-
-	err := d.ChangeNick(ctx, 1, "alice2")
-	if err != nil {
-		t.Fatalf("ChangeNick: %v", err)
-	}
-
-	_, nick, err := d.GetUserByToken(ctx, token)
-	if err != nil {
-		t.Fatalf("GetUserByToken: %v", err)
-	}
-
-	if nick != "alice2" {
-		t.Errorf("got nick %q, want alice2", nick)
-	}
-}
-
-func TestSetTopic(t *testing.T) {
-	t.Parallel()
-
-	d := setupTestDB(t)
-	ctx := context.Background()
-
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
-	_, _ = d.GetOrCreateChannel(ctx, "#general")
-
-	err := d.SetTopic(ctx, "#general", uid, "new topic")
-	if err != nil {
-		t.Fatalf("SetTopic: %v", err)
-	}
-
-	channels, err := d.ListAllChannels(ctx)
-	if err != nil {
-		t.Fatalf("ListAllChannels: %v", err)
-	}
-
-	found := false
-
-	for _, ch := range channels {
-		if ch.Name == "#general" && ch.Topic == "new topic" {
-			found = true
-		}
-	}
-
-	if !found {
-		t.Error("topic was not updated")
-	}
-}
-
-func TestGetMessagesBefore(t *testing.T) {
-	t.Parallel()
-
-	d := setupTestDB(t)
-	ctx := context.Background()
-
-	uid, _, _ := d.CreateUser(ctx, nickAlice)
-	chID, _ := d.GetOrCreateChannel(ctx, "#general")
-
-	_ = d.JoinChannel(ctx, chID, uid)
 
 	for i := range 5 {
-		_, _ = d.SendMessage(
+		id := fmt.Sprintf("m%d", i)
-			ctx, chID, uid,
+
-			fmt.Sprintf("msg%d", i),
+		_, _ = d.CreateMessage(
+			ctx, id, "u1", nickAlice, "#general",
+			"message", fmt.Sprintf("msg%d", i),
 		)
 
 		time.Sleep(10 * time.Millisecond)
 	}
 
-	msgs, err := d.GetMessagesBefore(ctx, chID, 0, 3)
+	msgs, err := ch.RecentMessages(ctx, 3)
 	if err != nil {
-		t.Fatalf("GetMessagesBefore: %v", err)
+		t.Fatalf("RecentMessages: %v", err)
 	}
 
 	if len(msgs) != 3 {
 		t.Fatalf("expected 3, got %d", len(msgs))
 	}
+
+	if msgs[0].Body != "msg4" {
+		t.Errorf("first: got %q, want msg4", msgs[0].Body)
+	}
+
+	if msgs[2].Body != "msg2" {
+		t.Errorf("last: got %q, want msg2", msgs[2].Body)
+	}
 }
 
-func TestListAllChannels(t *testing.T) {
+func TestChannelRecentMessagesLargeLimit(t *testing.T) {
 	t.Parallel()
 
 	d := setupTestDB(t)
 	ctx := context.Background()
 
-	_, _ = d.GetOrCreateChannel(ctx, "#alpha")
+	ch, _ := d.CreateChannel(ctx, "c1", "#general", "", "")
-	_, _ = d.GetOrCreateChannel(ctx, "#beta")
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateMessage(
+		ctx, "m1", "u1", nickAlice,
+		"#general", "message", "only",
+	)
 
-	channels, err := d.ListAllChannels(ctx)
+	msgs, err := ch.RecentMessages(ctx, 100)
 	if err != nil {
-		t.Fatalf("ListAllChannels: %v", err)
+		t.Fatalf("RecentMessages: %v", err)
 	}
 
-	if len(channels) != 2 {
+	if len(msgs) != 1 {
-		t.Errorf("expected 2, got %d", len(channels))
+		t.Errorf("expected 1, got %d", len(msgs))
+	}
+}
+
+func TestChannelMemberUser(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateChannel(ctx, "c1", "#general", "", "")
+
+	cm, _ := d.AddChannelMember(ctx, "c1", "u1", "+o")
+
+	u, err := cm.User(ctx)
+	if err != nil {
+		t.Fatalf("ChannelMember.User: %v", err)
+	}
+
+	if u.ID != "u1" || u.Nick != nickAlice {
+		t.Errorf("got %+v", u)
+	}
+}
+
+func TestChannelMemberChannel(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateChannel(ctx, "c1", "#general", "topic", "+n")
+
+	cm, _ := d.AddChannelMember(ctx, "c1", "u1", "")
+
+	ch, err := cm.Channel(ctx)
+	if err != nil {
+		t.Fatalf("ChannelMember.Channel: %v", err)
+	}
+
+	if ch.ID != "c1" || ch.Topic != "topic" {
+		t.Errorf("got %+v", ch)
+	}
+}
+
+func TestDMMessage(t *testing.T) {
+	t.Parallel()
+
+	d := setupTestDB(t)
+	ctx := context.Background()
+
+	_, _ = d.CreateUser(ctx, "u1", nickAlice, "h")
+	_, _ = d.CreateUser(ctx, "u2", nickBob, "h")
+
+	msg, err := d.CreateMessage(
+		ctx, "m1", "u1", nickAlice, "u2", "message", "hey",
+	)
+	if err != nil {
+		t.Fatalf("CreateMessage DM: %v", err)
+	}
+
+	if msg.Target != "u2" {
+		t.Errorf("target: got %q, want u2", msg.Target)
 	}
 }

internal/db/queries.go

@@ -2,140 +2,43 @@ package db
 
 import (
 	"context"
-	"crypto/rand"
-	"database/sql"
-	"encoding/hex"
 	"fmt"
 	"time"
 )
 
-const (
+// GetOrCreateChannel returns the channel id, creating it if needed.
-	defaultMessageLimit = 50
+func (s *Database) GetOrCreateChannel(ctx context.Context, name string) (string, error) {
-	defaultPollLimit    = 100
+	var id string
-	tokenBytes          = 32
-)
 
-func generateToken() string {
+	err := s.db.QueryRowContext(ctx, "SELECT id FROM channels WHERE name = ?", name).Scan(&id)
-	b := make([]byte, tokenBytes)
-	_, _ = rand.Read(b)
-
-	return hex.EncodeToString(b)
-}
-
-// CreateUser registers a new user with the given nick and
-// returns the user with token.
-func (s *Database) CreateUser(
-	ctx context.Context,
-	nick string,
-) (int64, string, error) {
-	token := generateToken()
-	now := time.Now()
-
-	res, err := s.db.ExecContext(ctx,
-		"INSERT INTO users (nick, token, created_at, last_seen) VALUES (?, ?, ?, ?)",
-		nick, token, now, now)
-	if err != nil {
-		return 0, "", fmt.Errorf("create user: %w", err)
-	}
-
-	id, _ := res.LastInsertId()
-
-	return id, token, nil
-}
-
-// GetUserByToken returns user id and nick for a given auth
-// token.
-func (s *Database) GetUserByToken(
-	ctx context.Context,
-	token string,
-) (int64, string, error) {
-	var id int64
-
-	var nick string
-
-	err := s.db.QueryRowContext(
-		ctx,
-		"SELECT id, nick FROM users WHERE token = ?",
-		token,
-	).Scan(&id, &nick)
-	if err != nil {
-		return 0, "", err
-	}
-
-	// Update last_seen
-	_, _ = s.db.ExecContext(
-		ctx,
-		"UPDATE users SET last_seen = ? WHERE id = ?",
-		time.Now(), id,
-	)
-
-	return id, nick, nil
-}
-
-// GetUserByNick returns user id for a given nick.
-func (s *Database) GetUserByNick(
-	ctx context.Context,
-	nick string,
-) (int64, error) {
-	var id int64
-
-	err := s.db.QueryRowContext(
-		ctx,
-		"SELECT id FROM users WHERE nick = ?",
-		nick,
-	).Scan(&id)
-
-	return id, err
-}
-
-// GetOrCreateChannel returns the channel id, creating it if
-// needed.
-func (s *Database) GetOrCreateChannel(
-	ctx context.Context,
-	name string,
-) (int64, error) {
-	var id int64
-
-	err := s.db.QueryRowContext(
-		ctx,
-		"SELECT id FROM channels WHERE name = ?",
-		name,
-	).Scan(&id)
 	if err == nil {
 		return id, nil
 	}
 
 	now := time.Now()
+	id = fmt.Sprintf("ch-%d", now.UnixNano())
 
-	res, err := s.db.ExecContext(ctx,
+	_, err = s.db.ExecContext(ctx,
-		"INSERT INTO channels (name, created_at, updated_at) VALUES (?, ?, ?)",
+		"INSERT INTO channels (id, name, topic, modes, created_at, updated_at) VALUES (?, ?, '', '', ?, ?)",
-		name, now, now)
+		id, name, now, now)
 	if err != nil {
-		return 0, fmt.Errorf("create channel: %w", err)
+		return "", fmt.Errorf("create channel: %w", err)
 	}
 
-	id, _ = res.LastInsertId()
-
 	return id, nil
 }
 
 // JoinChannel adds a user to a channel.
-func (s *Database) JoinChannel(
+func (s *Database) JoinChannel(ctx context.Context, channelID, userID string) error {
-	ctx context.Context,
-	channelID, userID int64,
-) error {
 	_, err := s.db.ExecContext(ctx,
-		"INSERT OR IGNORE INTO channel_members (channel_id, user_id, joined_at) VALUES (?, ?, ?)",
+		"INSERT OR IGNORE INTO channel_members (channel_id, user_id, modes, joined_at) VALUES (?, ?, '', ?)",
 		channelID, userID, time.Now())
 
 	return err
 }
 
 // PartChannel removes a user from a channel.
-func (s *Database) PartChannel(
+func (s *Database) PartChannel(ctx context.Context, channelID, userID string) error {
-	ctx context.Context,
-	channelID, userID int64,
-) error {
 	_, err := s.db.ExecContext(ctx,
 		"DELETE FROM channel_members WHERE channel_id = ? AND user_id = ?",
 		channelID, userID)
@@ -143,18 +46,8 @@ func (s *Database) PartChannel(
 	return err
 }
 
-// ChannelInfo is a lightweight channel representation.
-type ChannelInfo struct {
-	ID    int64  `json:"id"`
-	Name  string `json:"name"`
-	Topic string `json:"topic"`
-}
-
 // ListChannels returns all channels the user has joined.
-func (s *Database) ListChannels(
+func (s *Database) ListChannels(ctx context.Context, userID string) ([]ChannelInfo, error) {
-	ctx context.Context,
-	userID int64,
-) ([]ChannelInfo, error) {
 	rows, err := s.db.QueryContext(ctx,
 		`SELECT c.id, c.name, c.topic FROM channels c
 		 INNER JOIN channel_members cm ON cm.channel_id = c.id
@@ -165,45 +58,31 @@ func (s *Database) ListChannels(
 
 	defer func() { _ = rows.Close() }()
 
-	var channels []ChannelInfo
+	channels := []ChannelInfo{}
 
 	for rows.Next() {
 		var ch ChannelInfo
-
+		if err := rows.Scan(&ch.ID, &ch.Name, &ch.Topic); err != nil {
-		err := rows.Scan(&ch.ID, &ch.Name, &ch.Topic)
-		if err != nil {
 			return nil, err
 		}
 
 		channels = append(channels, ch)
 	}
 
-	err = rows.Err()
+	return channels, rows.Err()
-	if err != nil {
-		return nil, err
-	}
-
-	if channels == nil {
-		channels = []ChannelInfo{}
-	}
-
-	return channels, nil
 }
 
-// MemberInfo represents a channel member.
+// ChannelInfo is a lightweight channel representation.
-type MemberInfo struct {
+type ChannelInfo struct {
-	ID       int64     `json:"id"`
+	ID    string `json:"id"`
-	Nick     string    `json:"nick"`
+	Name  string `json:"name"`
-	LastSeen time.Time `json:"lastSeen"`
+	Topic string `json:"topic"`
 }
 
 // ChannelMembers returns all members of a channel.
-func (s *Database) ChannelMembers(
+func (s *Database) ChannelMembers(ctx context.Context, channelID string) ([]MemberInfo, error) {
-	ctx context.Context,
-	channelID int64,
-) ([]MemberInfo, error) {
 	rows, err := s.db.QueryContext(ctx,
-		`SELECT u.id, u.nick, u.last_seen FROM users u
+		`SELECT u.id, u.nick, u.last_seen_at FROM users u
 		 INNER JOIN channel_members cm ON cm.user_id = u.id
 		 WHERE cm.channel_id = ? ORDER BY u.nick`, channelID)
 	if err != nil {
@@ -212,34 +91,30 @@ func (s *Database) ChannelMembers(
 
 	defer func() { _ = rows.Close() }()
 
-	var members []MemberInfo
+	members := []MemberInfo{}
 
 	for rows.Next() {
 		var m MemberInfo
-
+		if err := rows.Scan(&m.ID, &m.Nick, &m.LastSeen); err != nil {
-		err := rows.Scan(&m.ID, &m.Nick, &m.LastSeen)
-		if err != nil {
 			return nil, err
 		}
 
 		members = append(members, m)
 	}
 
-	err = rows.Err()
+	return members, rows.Err()
-	if err != nil {
+}
-		return nil, err
-	}
 
-	if members == nil {
+// MemberInfo represents a channel member.
-		members = []MemberInfo{}
+type MemberInfo struct {
-	}
+	ID       string     `json:"id"`
-
+	Nick     string     `json:"nick"`
-	return members, nil
+	LastSeen *time.Time `json:"lastSeen"`
 }
 
 // MessageInfo represents a chat message.
 type MessageInfo struct {
-	ID        int64     `json:"id"`
+	ID        string    `json:"id"`
 	Channel   string    `json:"channel,omitempty"`
 	Nick      string    `json:"nick"`
 	Content   string    `json:"content"`
@@ -248,433 +123,212 @@ type MessageInfo struct {
 	CreatedAt time.Time `json:"createdAt"`
 }
 
-// GetMessages returns messages for a channel, optionally
-// after a given ID.
-func (s *Database) GetMessages(
-	ctx context.Context,
-	channelID int64,
-	afterID int64,
-	limit int,
-) ([]MessageInfo, error) {
-	if limit <= 0 {
-		limit = defaultMessageLimit
-	}
-
-	rows, err := s.db.QueryContext(ctx,
-		`SELECT m.id, c.name, u.nick, m.content, m.created_at
-		 FROM messages m
-		 INNER JOIN users u ON u.id = m.user_id
-		 INNER JOIN channels c ON c.id = m.channel_id
-		 WHERE m.channel_id = ? AND m.is_dm = 0 AND m.id > ?
-		 ORDER BY m.id ASC LIMIT ?`, channelID, afterID, limit)
-	if err != nil {
-		return nil, err
-	}
-
-	defer func() { _ = rows.Close() }()
-
-	var msgs []MessageInfo
-
-	for rows.Next() {
-		var m MessageInfo
-
-		err := rows.Scan(
-			&m.ID, &m.Channel, &m.Nick,
-			&m.Content, &m.CreatedAt,
-		)
-		if err != nil {
-			return nil, err
-		}
-
-		msgs = append(msgs, m)
-	}
-
-	err = rows.Err()
-	if err != nil {
-		return nil, err
-	}
-
-	if msgs == nil {
-		msgs = []MessageInfo{}
-	}
-
-	return msgs, nil
-}
-
 // SendMessage inserts a channel message.
-func (s *Database) SendMessage(
+func (s *Database) SendMessage(ctx context.Context, channelID, userID, nick, content string) (string, error) {
-	ctx context.Context,
+	now := time.Now()
-	channelID, userID int64,
+	id := fmt.Sprintf("msg-%d", now.UnixNano())
-	content string,
+
-) (int64, error) {
+	_, err := s.db.ExecContext(ctx,
-	res, err := s.db.ExecContext(ctx,
+		`INSERT INTO messages (id, ts, from_user_id, from_nick, target, type, body, meta, created_at)
-		"INSERT INTO messages (channel_id, user_id, content, is_dm, created_at) VALUES (?, ?, ?, 0, ?)",
+		 VALUES (?, ?, ?, ?, ?, 'message', ?, '{}', ?)`,
-		channelID, userID, content, time.Now())
+		id, now, userID, nick, channelID, content, now)
 	if err != nil {
-		return 0, err
+		return "", err
 	}
 
-	return res.LastInsertId()
+	return id, nil
 }
 
 // SendDM inserts a direct message.
-func (s *Database) SendDM(
+func (s *Database) SendDM(ctx context.Context, fromID, fromNick, toID, content string) (string, error) {
-	ctx context.Context,
+	now := time.Now()
-	fromID, toID int64,
+	id := fmt.Sprintf("msg-%d", now.UnixNano())
-	content string,
+
-) (int64, error) {
+	_, err := s.db.ExecContext(ctx,
-	res, err := s.db.ExecContext(ctx,
+		`INSERT INTO messages (id, ts, from_user_id, from_nick, target, type, body, meta, created_at)
-		"INSERT INTO messages (user_id, content, is_dm, dm_target_id, created_at) VALUES (?, ?, 1, ?, ?)",
+		 VALUES (?, ?, ?, ?, ?, 'message', ?, '{}', ?)`,
-		fromID, content, toID, time.Now())
+		id, now, fromID, fromNick, toID, content, now)
 	if err != nil {
-		return 0, err
+		return "", err
 	}
 
-	return res.LastInsertId()
+	return id, nil
 }
 
-// GetDMs returns direct messages between two users after a
+// PollMessages returns all new messages for a user's joined channels, ordered by timestamp.
-// given ID.
+func (s *Database) PollMessages(ctx context.Context, userID string, afterTS string, limit int) ([]MessageInfo, error) {
-func (s *Database) GetDMs(
-	ctx context.Context,
-	userA, userB int64,
-	afterID int64,
-	limit int,
-) ([]MessageInfo, error) {
 	if limit <= 0 {
-		limit = defaultMessageLimit
+		limit = 100
 	}
 
 	rows, err := s.db.QueryContext(ctx,
-		`SELECT m.id, u.nick, m.content, t.nick, m.created_at
+		`SELECT m.id, m.target, m.from_nick, m.body, m.created_at
 		 FROM messages m
-		 INNER JOIN users u ON u.id = m.user_id
+		 WHERE m.created_at > COALESCE(NULLIF(?, ''), '1970-01-01')
-		 INNER JOIN users t ON t.id = m.dm_target_id
+		 AND (
-		 WHERE m.is_dm = 1 AND m.id > ?
+		   m.target IN (SELECT cm.channel_id FROM channel_members cm WHERE cm.user_id = ?)
-		 AND ((m.user_id = ? AND m.dm_target_id = ?)
+		   OR m.target = ?
-		   OR (m.user_id = ? AND m.dm_target_id = ?))
+		   OR m.from_user_id = ?
-		 ORDER BY m.id ASC LIMIT ?`,
-		afterID, userA, userB, userB, userA, limit)
-	if err != nil {
-		return nil, err
-	}
-
-	defer func() { _ = rows.Close() }()
-
-	var msgs []MessageInfo
-
-	for rows.Next() {
-		var m MessageInfo
-
-		err := rows.Scan(
-			&m.ID, &m.Nick, &m.Content,
-			&m.DMTarget, &m.CreatedAt,
-		)
-		if err != nil {
-			return nil, err
-		}
-
-		m.IsDM = true
-
-		msgs = append(msgs, m)
-	}
-
-	err = rows.Err()
-	if err != nil {
-		return nil, err
-	}
-
-	if msgs == nil {
-		msgs = []MessageInfo{}
-	}
-
-	return msgs, nil
-}
-
-// PollMessages returns all new messages (channel + DM) for
-// a user after a given ID.
-func (s *Database) PollMessages(
-	ctx context.Context,
-	userID int64,
-	afterID int64,
-	limit int,
-) ([]MessageInfo, error) {
-	if limit <= 0 {
-		limit = defaultPollLimit
-	}
-
-	rows, err := s.db.QueryContext(ctx,
-		`SELECT m.id, COALESCE(c.name, ''), u.nick, m.content,
-		        m.is_dm, COALESCE(t.nick, ''), m.created_at
-		 FROM messages m
-		 INNER JOIN users u ON u.id = m.user_id
-		 LEFT JOIN channels c ON c.id = m.channel_id
-		 LEFT JOIN users t ON t.id = m.dm_target_id
-		 WHERE m.id > ? AND (
-		   (m.is_dm = 0 AND m.channel_id IN
-		     (SELECT channel_id FROM channel_members
-		      WHERE user_id = ?))
-		   OR (m.is_dm = 1
-		     AND (m.user_id = ? OR m.dm_target_id = ?))
 		 )
-		 ORDER BY m.id ASC LIMIT ?`,
+		 ORDER BY m.created_at ASC LIMIT ?`,
-		afterID, userID, userID, userID, limit)
+		afterTS, userID, userID, userID, limit)
 	if err != nil {
 		return nil, err
 	}
 
 	defer func() { _ = rows.Close() }()
 
-	msgs := make([]MessageInfo, 0)
+	msgs := []MessageInfo{}
-
-	for rows.Next() {
-		var (
-			m    MessageInfo
-			isDM int
-		)
-
-		err := rows.Scan(
-			&m.ID, &m.Channel, &m.Nick, &m.Content,
-			&isDM, &m.DMTarget, &m.CreatedAt,
-		)
-		if err != nil {
-			return nil, err
-		}
-
-		m.IsDM = isDM == 1
-		msgs = append(msgs, m)
-	}
-
-	err = rows.Err()
-	if err != nil {
-		return nil, err
-	}
-
-	return msgs, nil
-}
-
-func scanChannelMessages(
-	rows *sql.Rows,
-) ([]MessageInfo, error) {
-	var msgs []MessageInfo
 
 	for rows.Next() {
 		var m MessageInfo
-
+		if err := rows.Scan(&m.ID, &m.Channel, &m.Nick, &m.Content, &m.CreatedAt); err != nil {
-		err := rows.Scan(
-			&m.ID, &m.Channel, &m.Nick,
-			&m.Content, &m.CreatedAt,
-		)
-		if err != nil {
 			return nil, err
 		}
 
 		msgs = append(msgs, m)
 	}
 
-	err := rows.Err()
+	return msgs, rows.Err()
+}
+
+// GetMessagesBefore returns channel messages before a given timestamp (for history scrollback).
+func (s *Database) GetMessagesBefore(ctx context.Context, target string, beforeTS string, limit int) ([]MessageInfo, error) {
+	if limit <= 0 {
+		limit = 50
+	}
+
+	var rows interface {
+		Next() bool
+		Scan(dest ...interface{}) error
+		Close() error
+		Err() error
+	}
+
+	var err error
+
+	if beforeTS != "" {
+		rows, err = s.db.QueryContext(ctx,
+			`SELECT m.id, m.target, m.from_nick, m.body, m.created_at
+			 FROM messages m
+			 WHERE m.target = ? AND m.created_at < ?
+			 ORDER BY m.created_at DESC LIMIT ?`,
+			target, beforeTS, limit)
+	} else {
+		rows, err = s.db.QueryContext(ctx,
+			`SELECT m.id, m.target, m.from_nick, m.body, m.created_at
+			 FROM messages m
+			 WHERE m.target = ?
+			 ORDER BY m.created_at DESC LIMIT ?`,
+			target, limit)
+	}
+
 	if err != nil {
 		return nil, err
 	}
 
-	if msgs == nil {
+	defer func() { _ = rows.Close() }()
-		msgs = []MessageInfo{}
-	}
 
-	return msgs, nil
+	msgs := []MessageInfo{}
-}
-
-func scanDMMessages(
-	rows *sql.Rows,
-) ([]MessageInfo, error) {
-	var msgs []MessageInfo
 
 	for rows.Next() {
 		var m MessageInfo
-
+		if err := rows.Scan(&m.ID, &m.Channel, &m.Nick, &m.Content, &m.CreatedAt); err != nil {
-		err := rows.Scan(
-			&m.ID, &m.Nick, &m.Content,
-			&m.DMTarget, &m.CreatedAt,
-		)
-		if err != nil {
 			return nil, err
 		}
 
-		m.IsDM = true
-
 		msgs = append(msgs, m)
 	}
 
-	err := rows.Err()
+	// Reverse to ascending order.
-	if err != nil {
-		return nil, err
-	}
-
-	if msgs == nil {
-		msgs = []MessageInfo{}
-	}
-
-	return msgs, nil
-}
-
-func reverseMessages(msgs []MessageInfo) {
 	for i, j := 0, len(msgs)-1; i < j; i, j = i+1, j-1 {
 		msgs[i], msgs[j] = msgs[j], msgs[i]
 	}
+
+	return msgs, rows.Err()
 }
 
-// GetMessagesBefore returns channel messages before a given
+// GetDMsBefore returns DMs between two users before a given timestamp.
-// ID (for history scrollback).
+func (s *Database) GetDMsBefore(ctx context.Context, userA, userB string, beforeTS string, limit int) ([]MessageInfo, error) {
-func (s *Database) GetMessagesBefore(
-	ctx context.Context,
-	channelID int64,
-	beforeID int64,
-	limit int,
-) ([]MessageInfo, error) {
 	if limit <= 0 {
-		limit = defaultMessageLimit
+		limit = 50
 	}
 
-	var query string
+	var rows interface {
+		Next() bool
+		Scan(dest ...interface{}) error
+		Close() error
+		Err() error
+	}
 
-	var args []any
+	var err error
 
-	if beforeID > 0 {
+	if beforeTS != "" {
-		query = `SELECT m.id, c.name, u.nick, m.content,
+		rows, err = s.db.QueryContext(ctx,
-		                m.created_at
+			`SELECT m.id, m.from_nick, m.body, m.target, m.created_at
-			FROM messages m
+			 FROM messages m
-			INNER JOIN users u ON u.id = m.user_id
+			 WHERE m.created_at < ?
-			INNER JOIN channels c ON c.id = m.channel_id
+			 AND ((m.from_user_id = ? AND m.target = ?) OR (m.from_user_id = ? AND m.target = ?))
-			WHERE m.channel_id = ? AND m.is_dm = 0
+			 ORDER BY m.created_at DESC LIMIT ?`,
-			  AND m.id < ?
+			beforeTS, userA, userB, userB, userA, limit)
-			ORDER BY m.id DESC LIMIT ?`
-		args = []any{channelID, beforeID, limit}
 	} else {
-		query = `SELECT m.id, c.name, u.nick, m.content,
+		rows, err = s.db.QueryContext(ctx,
-		                m.created_at
+			`SELECT m.id, m.from_nick, m.body, m.target, m.created_at
-			FROM messages m
+			 FROM messages m
-			INNER JOIN users u ON u.id = m.user_id
+			 WHERE (m.from_user_id = ? AND m.target = ?) OR (m.from_user_id = ? AND m.target = ?)
-			INNER JOIN channels c ON c.id = m.channel_id
+			 ORDER BY m.created_at DESC LIMIT ?`,
-			WHERE m.channel_id = ? AND m.is_dm = 0
+			userA, userB, userB, userA, limit)
-			ORDER BY m.id DESC LIMIT ?`
-		args = []any{channelID, limit}
 	}
 
-	rows, err := s.db.QueryContext(ctx, query, args...)
 	if err != nil {
 		return nil, err
 	}
 
 	defer func() { _ = rows.Close() }()
 
-	msgs, scanErr := scanChannelMessages(rows)
+	msgs := []MessageInfo{}
-	if scanErr != nil {
-		return nil, scanErr
-	}
 
-	// Reverse to ascending order.
+	for rows.Next() {
-	reverseMessages(msgs)
+		var m MessageInfo
-
+		if err := rows.Scan(&m.ID, &m.Nick, &m.Content, &m.DMTarget, &m.CreatedAt); err != nil {
-	return msgs, nil
+			return nil, err
-}
-
-// GetDMsBefore returns DMs between two users before a given
-// ID (for history scrollback).
-func (s *Database) GetDMsBefore(
-	ctx context.Context,
-	userA, userB int64,
-	beforeID int64,
-	limit int,
-) ([]MessageInfo, error) {
-	if limit <= 0 {
-		limit = defaultMessageLimit
-	}
-
-	var query string
-
-	var args []any
-
-	if beforeID > 0 {
-		query = `SELECT m.id, u.nick, m.content, t.nick,
-		                m.created_at
-			FROM messages m
-			INNER JOIN users u ON u.id = m.user_id
-			INNER JOIN users t ON t.id = m.dm_target_id
-			WHERE m.is_dm = 1 AND m.id < ?
-			AND ((m.user_id = ? AND m.dm_target_id = ?)
-			  OR (m.user_id = ? AND m.dm_target_id = ?))
-			ORDER BY m.id DESC LIMIT ?`
-		args = []any{
-			beforeID, userA, userB, userB, userA, limit,
 		}
-	} else {
-		query = `SELECT m.id, u.nick, m.content, t.nick,
-		                m.created_at
-			FROM messages m
-			INNER JOIN users u ON u.id = m.user_id
-			INNER JOIN users t ON t.id = m.dm_target_id
-			WHERE m.is_dm = 1
-			AND ((m.user_id = ? AND m.dm_target_id = ?)
-			  OR (m.user_id = ? AND m.dm_target_id = ?))
-			ORDER BY m.id DESC LIMIT ?`
-		args = []any{userA, userB, userB, userA, limit}
-	}
 
-	rows, err := s.db.QueryContext(ctx, query, args...)
+		m.IsDM = true
-	if err != nil {
+		msgs = append(msgs, m)
-		return nil, err
-	}
-
-	defer func() { _ = rows.Close() }()
-
-	msgs, scanErr := scanDMMessages(rows)
-	if scanErr != nil {
-		return nil, scanErr
 	}
 
 	// Reverse to ascending order.
-	reverseMessages(msgs)
+	for i, j := 0, len(msgs)-1; i < j; i, j = i+1, j-1 {
+		msgs[i], msgs[j] = msgs[j], msgs[i]
+	}
 
-	return msgs, nil
+	return msgs, rows.Err()
 }
 
 // ChangeNick updates a user's nickname.
-func (s *Database) ChangeNick(
+func (s *Database) ChangeNick(ctx context.Context, userID string, newNick string) error {
-	ctx context.Context,
-	userID int64,
-	newNick string,
-) error {
 	_, err := s.db.ExecContext(ctx,
-		"UPDATE users SET nick = ? WHERE id = ?",
+		"UPDATE users SET nick = ? WHERE id = ?", newNick, userID)
-		newNick, userID)
 
 	return err
 }
 
 // SetTopic sets the topic for a channel.
-func (s *Database) SetTopic(
+func (s *Database) SetTopic(ctx context.Context, channelName string, _ string, topic string) error {
-	ctx context.Context,
-	channelName string,
-	_ int64,
-	topic string,
-) error {
 	_, err := s.db.ExecContext(ctx,
-		"UPDATE channels SET topic = ? WHERE name = ?",
+		"UPDATE channels SET topic = ? WHERE name = ?", topic, channelName)
-		topic, channelName)
 
 	return err
 }
 
-// GetServerName returns the server name (unused, config
+// GetServerName returns the server name (unused, config provides this).
-// provides this).
 func (s *Database) GetServerName() string {
 	return ""
 }
 
 // ListAllChannels returns all channels.
-func (s *Database) ListAllChannels(
+func (s *Database) ListAllChannels(ctx context.Context) ([]ChannelInfo, error) {
-	ctx context.Context,
-) ([]ChannelInfo, error) {
 	rows, err := s.db.QueryContext(ctx,
 		"SELECT id, name, topic FROM channels ORDER BY name")
 	if err != nil {
@@ -683,29 +337,16 @@ func (s *Database) ListAllChannels(
 
 	defer func() { _ = rows.Close() }()
 
-	var channels []ChannelInfo
+	channels := []ChannelInfo{}
 
 	for rows.Next() {
 		var ch ChannelInfo
-
+		if err := rows.Scan(&ch.ID, &ch.Name, &ch.Topic); err != nil {
-		err := rows.Scan(
-			&ch.ID, &ch.Name, &ch.Topic,
-		)
-		if err != nil {
 			return nil, err
 		}
 
 		channels = append(channels, ch)
 	}
 
-	err = rows.Err()
+	return channels, rows.Err()
-	if err != nil {
-		return nil, err
-	}
-
-	if channels == nil {
-		channels = []ChannelInfo{}
-	}
-
-	return channels, nil
 }

internal/db/schema/003_users.sql

@@ -1,18 +1,6 @@
--- Migration 003: Replace UUID-based tables with simple integer-keyed
+PRAGMA foreign_keys = ON;
--- tables for the HTTP API.  Drops the 002 tables and recreates them.
 
-PRAGMA foreign_keys = OFF;
+CREATE TABLE IF NOT EXISTS users (
-
-DROP TABLE IF EXISTS message_queue;
-DROP TABLE IF EXISTS sessions;
-DROP TABLE IF EXISTS server_links;
-DROP TABLE IF EXISTS messages;
-DROP TABLE IF EXISTS channel_members;
-DROP TABLE IF EXISTS auth_tokens;
-DROP TABLE IF EXISTS channels;
-DROP TABLE IF EXISTS users;
-
-CREATE TABLE users (
     id INTEGER PRIMARY KEY AUTOINCREMENT,
     nick TEXT NOT NULL UNIQUE,
     token TEXT NOT NULL UNIQUE,
@@ -20,15 +8,7 @@ CREATE TABLE users (
     last_seen DATETIME DEFAULT CURRENT_TIMESTAMP
 );
 
-CREATE TABLE channels (
+CREATE TABLE IF NOT EXISTS channel_members (
-    id INTEGER PRIMARY KEY AUTOINCREMENT,
-    name TEXT NOT NULL UNIQUE,
-    topic TEXT NOT NULL DEFAULT '',
-    created_at DATETIME DEFAULT CURRENT_TIMESTAMP,
-    updated_at DATETIME DEFAULT CURRENT_TIMESTAMP
-);
-
-CREATE TABLE channel_members (
     id INTEGER PRIMARY KEY AUTOINCREMENT,
     channel_id INTEGER NOT NULL REFERENCES channels(id) ON DELETE CASCADE,
     user_id INTEGER NOT NULL REFERENCES users(id) ON DELETE CASCADE,
@@ -36,7 +16,7 @@ CREATE TABLE channel_members (
     UNIQUE(channel_id, user_id)
 );
 
-CREATE TABLE messages (
+CREATE TABLE IF NOT EXISTS messages (
     id INTEGER PRIMARY KEY AUTOINCREMENT,
     channel_id INTEGER REFERENCES channels(id) ON DELETE CASCADE,
     user_id INTEGER NOT NULL REFERENCES users(id) ON DELETE CASCADE,
@@ -46,8 +26,6 @@ CREATE TABLE messages (
     created_at DATETIME DEFAULT CURRENT_TIMESTAMP
 );
 
-CREATE INDEX idx_messages_channel ON messages(channel_id, created_at);
+CREATE INDEX IF NOT EXISTS idx_messages_channel ON messages(channel_id, created_at);
-CREATE INDEX idx_messages_dm ON messages(user_id, dm_target_id, created_at);
+CREATE INDEX IF NOT EXISTS idx_messages_dm ON messages(user_id, dm_target_id, created_at);
-CREATE INDEX idx_users_token ON users(token);
+CREATE INDEX IF NOT EXISTS idx_users_token ON users(token);
-
-PRAGMA foreign_keys = ON;

internal/models/auth_token.go

@@ -2,6 +2,7 @@ package models
 
 import (
 	"context"
+	"fmt"
 	"time"
 )
 
@@ -22,5 +23,5 @@ func (t *AuthToken) User(ctx context.Context) (*User, error) {
 		return ul.GetUserByID(ctx, t.UserID)
 	}
 
-	return nil, ErrUserLookupNotAvailable
+	return nil, fmt.Errorf("user lookup not available")
 }

internal/models/channel_member.go

@@ -2,6 +2,7 @@ package models
 
 import (
 	"context"
+	"fmt"
 	"time"
 )
 
@@ -22,7 +23,7 @@ func (cm *ChannelMember) User(ctx context.Context) (*User, error) {
 		return ul.GetUserByID(ctx, cm.UserID)
 	}
 
-	return nil, ErrUserLookupNotAvailable
+	return nil, fmt.Errorf("user lookup not available")
 }
 
 // Channel returns the full Channel for this membership.
@@ -31,5 +32,5 @@ func (cm *ChannelMember) Channel(ctx context.Context) (*Channel, error) {
 		return cl.GetChannelByID(ctx, cm.ChannelID)
 	}
 
-	return nil, ErrChannelLookupNotAvailable
+	return nil, fmt.Errorf("channel lookup not available")
 }

internal/models/model.go

@@ -6,7 +6,6 @@ package models
 import (
 	"context"
 	"database/sql"
-	"errors"
 )
 
 // DB is the interface that models use to query the database.
@@ -25,12 +24,6 @@ type ChannelLookup interface {
 	GetChannelByID(ctx context.Context, id string) (*Channel, error)
 }
 
-// Sentinel errors for model lookup methods.
-var (
-	ErrUserLookupNotAvailable    = errors.New("user lookup not available")
-	ErrChannelLookupNotAvailable = errors.New("channel lookup not available")
-)
-
 // Base is embedded in all model structs to provide database access.
 type Base struct {
 	db DB
@@ -47,7 +40,7 @@ func (b *Base) GetDB() *sql.DB {
 }
 
 // GetUserLookup returns the DB as a UserLookup if it implements the interface.
-func (b *Base) GetUserLookup() UserLookup { //nolint:ireturn
+func (b *Base) GetUserLookup() UserLookup {
 	if ul, ok := b.db.(UserLookup); ok {
 		return ul
 	}
@@ -56,7 +49,7 @@ func (b *Base) GetUserLookup() UserLookup { //nolint:ireturn
 }
 
 // GetChannelLookup returns the DB as a ChannelLookup if it implements the interface.
-func (b *Base) GetChannelLookup() ChannelLookup { //nolint:ireturn
+func (b *Base) GetChannelLookup() ChannelLookup {
 	if cl, ok := b.db.(ChannelLookup); ok {
 		return cl
 	}

internal/models/session.go

@@ -2,6 +2,7 @@ package models
 
 import (
 	"context"
+	"fmt"
 	"time"
 )
 
@@ -22,5 +23,5 @@ func (s *Session) User(ctx context.Context) (*User, error) {
 		return ul.GetUserByID(ctx, s.UserID)
 	}
 
-	return nil, ErrUserLookupNotAvailable
+	return nil, fmt.Errorf("user lookup not available")
 }

internal/server/routes.go

@@ -71,37 +71,17 @@ func (s *Server) SetupRoutes() {
 		s.log.Error("failed to get web dist filesystem", "error", err)
 	} else {
 		fileServer := http.FileServer(http.FS(distFS))
-
 		s.router.Get("/*", func(w http.ResponseWriter, r *http.Request) {
-			s.serveSPA(distFS, fileServer, w, r)
+			// Try to serve the file; if not found, serve index.html for SPA routing
+			f, err := distFS.(fs.ReadFileFS).ReadFile(r.URL.Path[1:])
+			if err != nil || len(f) == 0 {
+				indexHTML, _ := distFS.(fs.ReadFileFS).ReadFile("index.html")
+				w.Header().Set("Content-Type", "text/html; charset=utf-8")
+				w.WriteHeader(http.StatusOK)
+				_, _ = w.Write(indexHTML)
+				return
+			}
+			fileServer.ServeHTTP(w, r)
 		})
 	}
 }
-
-func (s *Server) serveSPA(
-	distFS fs.FS,
-	fileServer http.Handler,
-	w http.ResponseWriter,
-	r *http.Request,
-) {
-	readFS, ok := distFS.(fs.ReadFileFS)
-	if !ok {
-		http.Error(w, "filesystem error", http.StatusInternalServerError)
-
-		return
-	}
-
-	// Try to serve the file; fall back to index.html for SPA routing.
-	f, err := readFS.ReadFile(r.URL.Path[1:])
-	if err != nil || len(f) == 0 {
-		indexHTML, _ := readFS.ReadFile("index.html")
-
-		w.Header().Set("Content-Type", "text/html; charset=utf-8")
-		w.WriteHeader(http.StatusOK)
-		_, _ = w.Write(indexHTML)
-
-		return
-	}
-
-	fileServer.ServeHTTP(w, r)
-}