fix: reference projects(guid) not projects(id) in chat_histories FK

.gitea/workflows/ci.yml

@@ -1,41 +0,0 @@
-name: CI
-
-on:
-  push:
-    branches: ['**']
-    tags-ignore: ['v*']
-  pull_request:
-    branches: ['**']
-
-jobs:
-  build-and-test:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: '1.26'
-
-      - name: Cache Go modules
-        uses: actions/cache@v4
-        with:
-          path: |
-            ~/.cache/go-build
-            ~/go/pkg/mod
-          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Run tests
-        run: go test ./...
-
-      - name: Build amcs-server
-        run: go build -o /dev/null ./cmd/amcs-server
-
-      - name: Build amcs-cli
-        run: go build -o /dev/null ./cmd/amcs-cli

.gitea/workflows/release.yml

@@ -1,122 +0,0 @@
-name: Release
-
-on:
-  push:
-    tags:
-      - 'v*.*.*'
-  workflow_dispatch:
-    inputs:
-      tag:
-        description: 'Tag to release (e.g. v1.2.3)'
-        required: true
-
-env:
-  GITEA_SERVER: https://git.warky.dev
-  GITEA_REPO: wdevs/amcs
-
-jobs:
-  release:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: '1.26'
-
-      - name: Cache Go modules
-        uses: actions/cache@v4
-        with:
-          path: |
-            ~/.cache/go-build
-            ~/go/pkg/mod
-          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Set up Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: 'lts/*'
-
-      - name: Install pnpm
-        run: npm install -g pnpm
-
-      - name: Build UI
-        run: |
-          cd ui
-          pnpm install --frozen-lockfile
-          pnpm run build
-
-      - name: Set build vars
-        id: vars
-        run: |
-          TAG="${{ github.event.inputs.tag || github.ref_name }}"
-          echo "VERSION=${TAG}" >> $GITHUB_OUTPUT
-          echo "COMMIT=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
-          echo "BUILD_DATE=$(date -u +%Y-%m-%dT%H:%M:%SZ)" >> $GITHUB_OUTPUT
-
-      - name: Build release binaries
-        run: |
-          VERSION="${{ steps.vars.outputs.VERSION }}"
-          COMMIT="${{ steps.vars.outputs.COMMIT }}"
-          BUILD_DATE="${{ steps.vars.outputs.BUILD_DATE }}"
-          LDFLAGS="-s -w -X git.warky.dev/wdevs/amcs/internal/buildinfo.Version=${VERSION} -X git.warky.dev/wdevs/amcs/internal/buildinfo.TagName=${VERSION} -X git.warky.dev/wdevs/amcs/internal/buildinfo.Commit=${COMMIT} -X git.warky.dev/wdevs/amcs/internal/buildinfo.BuildDate=${BUILD_DATE}"
-          mkdir -p dist
-          for BINARY in amcs-server amcs-cli; do
-            CMD="./cmd/${BINARY}"
-            for PLATFORM in linux/amd64 linux/arm64 darwin/amd64 darwin/arm64 windows/amd64; do
-              OS="${PLATFORM%/*}"
-              ARCH="${PLATFORM#*/}"
-              EXT=""
-              [ "$OS" = "windows" ] && EXT=".exe"
-              OUTPUT="dist/${BINARY}-${OS}-${ARCH}${EXT}"
-              echo "Building ${OUTPUT}..."
-              GOOS=$OS GOARCH=$ARCH go build -ldflags "${LDFLAGS}" -o "${OUTPUT}" "${CMD}"
-            done
-          done
-          cd dist && sha256sum * > checksums.txt && cd ..
-
-      - name: Create Gitea Release
-        id: create_release
-        run: |
-          export VERSION="${{ steps.vars.outputs.VERSION }}"
-          BODY=$(python3 <<'PY'
-          import json, subprocess, os
-          version = os.environ['VERSION']
-          commit = subprocess.check_output(['git', 'rev-parse', '--short', 'HEAD'], text=True).strip()
-          body = f"## {version}\n\nBuilt from commit {commit}.\n\nSee `checksums.txt` to verify downloads."
-          print(json.dumps({
-              'tag_name': version,
-              'name': version,
-              'body': body,
-              'draft': False,
-              'prerelease': False,
-          }))
-          PY
-          )
-          RESPONSE=$(curl -fsS -X POST "${{ env.GITEA_SERVER }}/api/v1/repos/${{ env.GITEA_REPO }}/releases" \
-            -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" \
-            -H "Content-Type: application/json" \
-            -d "$BODY")
-          RELEASE_ID=$(printf '%s' "$RESPONSE" | python3 -c 'import sys,json; print(json.load(sys.stdin)["id"])')
-          echo "RELEASE_ID=${RELEASE_ID}" >> $GITHUB_OUTPUT
-
-      - name: Upload release assets
-        run: |
-          RELEASE_ID="${{ steps.create_release.outputs.RELEASE_ID }}"
-          for f in dist/*; do
-            name=$(basename "$f")
-            echo "Uploading ${name}..."
-            curl -fsS -X POST \
-              "${{ env.GITEA_SERVER }}/api/v1/repos/${{ env.GITEA_REPO }}/releases/${RELEASE_ID}/assets?name=${name}" \
-              -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" \
-              -H "Content-Type: application/octet-stream" \
-              --data-binary @"${f}"
-          done

.gitignore

@@ -31,6 +31,3 @@ cmd/amcs-server/__debug_*
 bin/
 .cache/
 OB1/
-ui/node_modules/
-ui/.svelte-kit/
-internal/app/ui/dist/

Dockerfile

@@ -1,14 +1,3 @@
-FROM node:22-bookworm AS ui-builder
-
-RUN npm install -g pnpm
-WORKDIR /src/ui
-
-COPY ui/package.json ui/pnpm-lock.yaml ./
-RUN pnpm install --frozen-lockfile
-
-COPY ui/ ./
-RUN pnpm run build
-
 FROM golang:1.26.1-bookworm AS builder
 
 WORKDIR /src
@@ -17,7 +6,6 @@ COPY go.mod go.sum ./
 RUN go mod download
 
 COPY . .
-COPY --from=ui-builder /src/internal/app/ui/dist ./internal/app/ui/dist
 
 RUN set -eu; \
     VERSION_TAG="$(git describe --tags --exact-match 2>/dev/null || echo dev)"; \

Makefile

@@ -3,43 +3,25 @@ GO_CACHE_DIR := $(CURDIR)/.cache/go-build
 SERVER_BIN := $(BIN_DIR)/amcs-server
 CMD_SERVER := ./cmd/amcs-server
 BUILDINFO_PKG := git.warky.dev/wdevs/amcs/internal/buildinfo
-UI_DIR := $(CURDIR)/ui
 PATCH_INCREMENT ?= 1
 VERSION_TAG ?= $(shell git describe --tags --exact-match 2>/dev/null || echo dev)
 COMMIT_SHA ?= $(shell git rev-parse --short HEAD 2>/dev/null || echo unknown)
 BUILD_DATE ?= $(shell date -u +%Y-%m-%dT%H:%M:%SZ)
-RELSPEC ?= $(shell command -v relspec 2>/dev/null || echo $(HOME)/go/bin/relspec)
-SCHEMA_FILES := $(sort $(wildcard schema/*.dbml))
-MERGE_TARGET_TMP := $(CURDIR)/.cache/schema.merge-target.dbml
-GENERATED_SCHEMA_MIGRATION := migrations/020_generated_schema.sql
-PNPM ?= pnpm
 LDFLAGS := -s -w \
 	-X $(BUILDINFO_PKG).Version=$(VERSION_TAG) \
 	-X $(BUILDINFO_PKG).TagName=$(VERSION_TAG) \
 	-X $(BUILDINFO_PKG).Commit=$(COMMIT_SHA) \
 	-X $(BUILDINFO_PKG).BuildDate=$(BUILD_DATE)
 
-.PHONY: all build clean migrate release-version test generate-migrations check-schema-drift build-cli ui-install ui-build ui-dev ui-check
+.PHONY: all build clean migrate release-version test
 
 all: build
 
-build: ui-build
+build:
 	@mkdir -p $(BIN_DIR)
 	go build -ldflags "$(LDFLAGS)" -o $(SERVER_BIN) $(CMD_SERVER)
 
-ui-install:
+test:
-	cd $(UI_DIR) && $(PNPM) install --frozen-lockfile
-
-ui-build: ui-install
-	cd $(UI_DIR) && $(PNPM) run build
-
-ui-dev: ui-install
-	cd $(UI_DIR) && $(PNPM) run dev
-
-ui-check: ui-install
-	cd $(UI_DIR) && $(PNPM) run check
-
-test: ui-check
 	@mkdir -p $(GO_CACHE_DIR)
 	GOCACHE=$(GO_CACHE_DIR) go test ./...
 
@@ -61,7 +43,6 @@ release-version:
 		exit 1; \
 	fi; \
 	git tag -a "$$next_tag" -m "Release $$next_tag"; \
-	git push origin "$$next_tag"; \
 	echo "$$next_tag"
 
 migrate:
@@ -69,31 +50,3 @@ migrate:
 
 clean:
 	rm -rf $(BIN_DIR)
-
-generate-migrations:
-	@test -n "$(SCHEMA_FILES)" || (echo "No DBML schema files found in schema/" >&2; exit 1)
-	@command -v $(RELSPEC) >/dev/null 2>&1 || (echo "relspec not found; install git.warky.dev/wdevs/relspecgo/cmd/relspec@latest" >&2; exit 1)
-	@mkdir -p $(dir $(MERGE_TARGET_TMP))
-	@: > $(MERGE_TARGET_TMP)
-	@schema_list=$$(printf '%s\n' $(SCHEMA_FILES) | paste -sd, -); \
-	$(RELSPEC) merge --target dbml --target-path $(MERGE_TARGET_TMP) --source dbml --from-list "$$schema_list" --output pgsql --output-path $(GENERATED_SCHEMA_MIGRATION)
-
-check-schema-drift:
-	@test -f $(GENERATED_SCHEMA_MIGRATION) || (echo "$(GENERATED_SCHEMA_MIGRATION) is missing; run make generate-migrations" >&2; exit 1)
-	@command -v $(RELSPEC) >/dev/null 2>&1 || (echo "relspec not found; install git.warky.dev/wdevs/relspecgo/cmd/relspec@latest" >&2; exit 1)
-	@mkdir -p $(dir $(MERGE_TARGET_TMP))
-	@tmpfile=$$(mktemp); \
-	: > $(MERGE_TARGET_TMP); \
-	schema_list=$$(printf '%s\n' $(SCHEMA_FILES) | paste -sd, -); \
-	$(RELSPEC) merge --target dbml --target-path $(MERGE_TARGET_TMP) --source dbml --from-list "$$schema_list" --output pgsql --output-path $$tmpfile; \
-	if ! cmp -s $$tmpfile $(GENERATED_SCHEMA_MIGRATION); then \
-		echo "Schema drift detected between schema/*.dbml and $(GENERATED_SCHEMA_MIGRATION)" >&2; \
-		diff -u $(GENERATED_SCHEMA_MIGRATION) $$tmpfile || true; \
-		rm -f $$tmpfile; \
-		exit 1; \
-	fi; \
-	rm -f $$tmpfile
-
-build-cli:
-	@mkdir -p $(BIN_DIR)
-	go build -o $(BIN_DIR)/amcs-cli ./cmd/amcs-cli

README.md

@@ -46,55 +46,21 @@ A Go MCP server for capturing and retrieving thoughts, memory, and project conte
 | `load_file` | Retrieve a stored file by ID; returns metadata, base64 content, and an embedded MCP binary resource |
 | `list_files` | Browse stored files by thought, project, or kind |
 | `backfill_embeddings` | Generate missing embeddings for stored thoughts |
-| `reparse_thought_metadata` | Re-extract metadata from thought content |
+| `reparse_thought_metadata` | Re-extract and normalize metadata for stored thoughts |
-| `retry_failed_metadata` | Retry pending/failed metadata extraction |
+| `retry_failed_metadata` | Retry metadata extraction for thoughts still pending or failed |
-| `add_maintenance_task` | Create a recurring or one-time home maintenance task |
+| `add_skill` | Store a reusable agent skill (behavioural instruction or capability prompt) |
-| `log_maintenance` | Log completed maintenance; updates next due date |
-| `get_upcoming_maintenance` | List maintenance tasks due within the next N days |
-| `search_maintenance_history` | Search the maintenance log by task name, category, or date range |
-| `save_chat_history` | Save chat messages with optional title, summary, channel, agent, and project |
-| `get_chat_history` | Fetch chat history by UUID or session_id |
-| `list_chat_histories` | List chat histories; filter by project, channel, agent_id, session_id, or days |
-| `delete_chat_history` | Delete a chat history by id |
-| `add_skill` | Store an agent skill (instruction or capability prompt) |
 | `remove_skill` | Delete an agent skill by id |
 | `list_skills` | List all agent skills, optionally filtered by tag |
-| `add_guardrail` | Store an agent guardrail (constraint or safety rule) |
+| `add_guardrail` | Store a reusable agent guardrail (constraint or safety rule) |
 | `remove_guardrail` | Delete an agent guardrail by id |
 | `list_guardrails` | List all agent guardrails, optionally filtered by tag or severity |
-| `add_project_skill` | Link a skill to a project; pass `project` if client is stateless |
+| `add_project_skill` | Link an agent skill to a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `remove_project_skill` | Unlink a skill from a project; pass `project` if client is stateless |
+| `remove_project_skill` | Unlink an agent skill from a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `list_project_skills` | Skills for a project; pass `project` if client is stateless |
+| `list_project_skills` | List all skills linked to a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `add_project_guardrail` | Link a guardrail to a project; pass `project` if client is stateless |
+| `add_project_guardrail` | Link an agent guardrail to a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `remove_project_guardrail` | Unlink a guardrail from a project; pass `project` if client is stateless |
+| `remove_project_guardrail` | Unlink an agent guardrail from a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `list_project_guardrails` | Guardrails for a project; pass `project` if client is stateless |
+| `list_project_guardrails` | List all guardrails linked to a project; pass `project` explicitly if your client does not preserve MCP sessions |
-| `get_version_info` | Build version, commit, and date |
+| `get_version_info` | Return the server build version information, including version, tag name, commit, and build date |
-| `describe_tools` | List all available MCP tools with names, descriptions, categories, and model-authored usage notes; call this at the start of a session to orient yourself |
-| `annotate_tool` | Persist your own usage notes for a specific tool; notes are returned by `describe_tools` in future sessions |
-
-## Self-Documenting Tools
-
-AMCS includes a built-in tool directory that models can read and annotate.
-
-**`describe_tools`** returns every registered tool with its name, description, category, and any model-written notes. Call it with no arguments to get the full list, or filter by category:
-
-```json
-{ "category": "thoughts" }
-```
-
-Available categories: `system`, `thoughts`, `projects`, `files`, `admin`, `maintenance`, `skills`, `chat`, `meta`.
-
-**`annotate_tool`** lets a model write persistent usage notes against a tool name. Notes survive across sessions and are returned by `describe_tools`:
-
-```json
-{ "tool_name": "capture_thought", "notes": "Always pass project explicitly — session state is not reliable in this client." }
-```
-
-Pass an empty string to clear notes. The intended workflow is:
-
-1. At the start of a session, call `describe_tools` to discover tools and read accumulated notes.
-2. As you learn something non-obvious about a tool — a gotcha, a workflow pattern, a required field ordering — call `annotate_tool` to record it.
-3. Future sessions receive the annotation automatically via `describe_tools`.
 
 ## MCP Error Contract
 
@@ -270,7 +236,7 @@ Alternatively, pass `client_id` and `client_secret` as body parameters instead o
 - `ai.litellm.base_url` and `ai.litellm.api_key` — LiteLLM proxy
 - `ai.ollama.base_url` and `ai.ollama.api_key` — Ollama local or remote server
 
-See `llm/plan.md` for an audited high-level status summary of the original implementation plan, and `llm/todo.md` for the audited backfill/fallback follow-up status.
+See `llm/plan.md` for full architecture and implementation plan.
 
 ## Backfill
 
@@ -533,90 +499,13 @@ Recommended Apache settings:
 - `ProxyTimeout 600` and `ProxyPass ... timeout=600` give Apache enough time to wait for the Go backend.
 - If another proxy or load balancer sits in front of Apache, align its size and timeout settings too.
 
-## CLI
-
-`amcs-cli` is a pre-built CLI client for the AMCS MCP server. Download it from https://git.warky.dev/wdevs/amcs/releases
-
-The primary purpose is to give agents and MCP clients a ready-made bridge to the AMCS server so they do not need to implement their own HTTP MCP client. Configure it once and any stdio-based MCP client can use AMCS immediately.
-
-### Commands
-
-| Command | Purpose |
-|---|---|
-| `amcs-cli tools` | List all tools available on the remote server |
-| `amcs-cli call <tool>` | Call a tool by name with `--arg key=value` flags |
-| `amcs-cli stdio` | Start a stdio MCP bridge backed by the remote server |
-
-`stdio` is the main integration point. It connects to the remote HTTP MCP server, discovers all its tools, and re-exposes them over stdio. Register it as a stdio MCP server in your agent config and it proxies every tool call through to AMCS.
-
-### Configuration
-
-Config file: `~/.config/amcs/config.yaml`
-
-```yaml
-server: https://your-amcs-server
-token: your-bearer-token
-```
-
-Env vars override the config file: `AMCS_URL`, `AMCS_TOKEN`. Flags `--server` and `--token` override env vars.
-
-### stdio MCP client setup
-
-#### Claude Code
-
-```bash
-claude mcp add --transport stdio amcs amcs-cli stdio
-```
-
-With inline credentials (no config file):
-
-```bash
-claude mcp add --transport stdio amcs amcs-cli stdio \
-  --env AMCS_URL=https://your-amcs-server \
-  --env AMCS_TOKEN=your-bearer-token
-```
-
-#### Output format
-
-`call` outputs JSON by default. Pass `--output yaml` for YAML.
-
 ## Development
 
 Run the SQL migrations against a local database with:
 
 `DATABASE_URL=postgres://... make migrate`
 
-### Backend + embedded UI build
+LLM integration instructions are served at `/llm`.
-
-The web UI now lives in the top-level `ui/` module and is embedded into the Go binary at build time with `go:embed`.
-
-**Use `pnpm` for all UI work in this repo.**
-
-- `make build` — runs the real UI build first, then compiles the Go server
-- `make test` — runs `svelte-check` for the frontend and `go test ./...` for the backend
-- `make ui-install` — installs frontend dependencies with `pnpm install --frozen-lockfile`
-- `make ui-build` — builds only the frontend bundle
-- `make ui-dev` — starts the Vite dev server with hot reload on `http://localhost:5173`
-- `make ui-check` — runs the frontend type and Svelte checks
-
-### Local UI workflow
-
-For the normal production-style local flow:
-
-1. Start the backend: `./scripts/run-local.sh configs/dev.yaml`
-2. Open `http://localhost:8080`
-
-For frontend iteration with hot reload and no Go rebuilds:
-
-1. Start the backend once: `go run ./cmd/amcs-server --config configs/dev.yaml`
-2. In another shell start the UI dev server: `make ui-dev`
-3. Open `http://localhost:5173`
-
-The Vite dev server proxies backend routes such as `/api/status`, `/llm`, `/healthz`, `/readyz`, `/files`, `/mcp`, and the OAuth endpoints back to the Go server on `http://127.0.0.1:8080` by default. Override that target with `AMCS_UI_BACKEND` if needed.
-
-The root page (`/`) is now the Svelte frontend. It preserves the existing landing-page content and status information by fetching data from `GET /api/status`.
-
-LLM integration instructions are still served at `/llm`.
 
 ## Containers
 

cmd/amcs-cli/cmd/call.go

@@ -1,98 +0,0 @@
-package cmd
-
-import (
-	"encoding/json"
-	"fmt"
-	"os"
-	"strconv"
-	"strings"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-	"github.com/spf13/cobra"
-	"gopkg.in/yaml.v3"
-)
-
-var argFlags []string
-
-var callCmd = &cobra.Command{
-	Use:   "call <tool>",
-	Short: "Call a remote AMCS tool",
-	Args:  cobra.ExactArgs(1),
-	RunE: func(cmd *cobra.Command, args []string) error {
-		toolName := args[0]
-		toolArgs, err := parseArgs(argFlags)
-		if err != nil {
-			return err
-		}
-
-		session, err := connectRemote(cmd.Context())
-		if err != nil {
-			return err
-		}
-		defer func() { _ = session.Close() }()
-
-		res, err := session.CallTool(cmd.Context(), &mcp.CallToolParams{Name: toolName, Arguments: toolArgs})
-		if err != nil {
-			return fmt.Errorf("call tool %q: %w", toolName, err)
-		}
-		return printOutput(res)
-	},
-}
-
-func init() {
-	callCmd.Flags().StringArrayVar(&argFlags, "arg", nil, "Tool argument in key=value format (repeatable)")
-	rootCmd.AddCommand(callCmd)
-}
-
-func parseArgs(items []string) (map[string]any, error) {
-	result := make(map[string]any, len(items))
-	for _, item := range items {
-		key, value, ok := strings.Cut(item, "=")
-		if !ok || strings.TrimSpace(key) == "" {
-			return nil, fmt.Errorf("invalid --arg %q: want key=value", item)
-		}
-		result[key] = parseScalar(value)
-	}
-	return result, nil
-}
-
-func parseScalar(s string) any {
-	if s == "true" || s == "false" {
-		b, _ := strconv.ParseBool(s)
-		return b
-	}
-	if i, err := strconv.ParseInt(s, 10, 64); err == nil {
-		return i
-	}
-	if f, err := strconv.ParseFloat(s, 64); err == nil && strings.ContainsAny(s, ".eE") {
-		return f
-	}
-	var v any
-	if err := json.Unmarshal([]byte(s), &v); err == nil {
-		switch v.(type) {
-		case map[string]any, []any, float64, bool, nil:
-			return v
-		}
-	}
-	return s
-}
-
-func printOutput(v any) error {
-	switch outputFlag {
-	case "yaml":
-		data, err := yaml.Marshal(v)
-		if err != nil {
-			return fmt.Errorf("marshal yaml: %w", err)
-		}
-		_, err = os.Stdout.Write(data)
-		return err
-	default:
-		data, err := json.MarshalIndent(v, "", "  ")
-		if err != nil {
-			return fmt.Errorf("marshal json: %w", err)
-		}
-		data = append(data, '\n')
-		_, err = os.Stdout.Write(data)
-		return err
-	}
-}

cmd/amcs-cli/cmd/config.go

@@ -1,60 +0,0 @@
-package cmd
-
-import (
-	"errors"
-	"fmt"
-	"os"
-	"path/filepath"
-	"strings"
-
-	"gopkg.in/yaml.v3"
-)
-
-type Config struct {
-	Server string `yaml:"server"`
-	Token  string `yaml:"token"`
-}
-
-func defaultConfigPath() (string, error) {
-	home, err := os.UserHomeDir()
-	if err != nil {
-		return "", fmt.Errorf("resolve home dir: %w", err)
-	}
-	return filepath.Join(home, ".config", "amcs", "config.yaml"), nil
-}
-
-func resolveConfigPath(path string) (string, error) {
-	if strings.TrimSpace(path) != "" {
-		return path, nil
-	}
-	return defaultConfigPath()
-}
-
-func loadConfigFile(path string) (Config, error) {
-	var cfg Config
-	data, err := os.ReadFile(path)
-	if err != nil {
-		if errors.Is(err, os.ErrNotExist) {
-			return cfg, nil
-		}
-		return cfg, fmt.Errorf("read config: %w", err)
-	}
-	if err := yaml.Unmarshal(data, &cfg); err != nil {
-		return cfg, fmt.Errorf("parse config: %w", err)
-	}
-	return cfg, nil
-}
-
-func saveConfigFile(path string, cfg Config) error {
-	if err := os.MkdirAll(filepath.Dir(path), 0o700); err != nil {
-		return fmt.Errorf("create config dir: %w", err)
-	}
-	data, err := yaml.Marshal(cfg)
-	if err != nil {
-		return fmt.Errorf("marshal config: %w", err)
-	}
-	if err := os.WriteFile(path, data, 0o600); err != nil {
-		return fmt.Errorf("write config: %w", err)
-	}
-	return nil
-}

cmd/amcs-cli/cmd/root.go

@@ -1,134 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"net/http"
-	"os"
-	"strings"
-	"time"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-	"github.com/spf13/cobra"
-)
-
-var (
-	cfgFile    string
-	serverFlag string
-	tokenFlag  string
-	outputFlag string
-	cfg        Config
-)
-
-var rootCmd = &cobra.Command{
-	Use:           "amcs-cli",
-	Short:         "CLI for connecting to a remote AMCS MCP server",
-	SilenceUsage:  true,
-	SilenceErrors: true,
-	PersistentPreRunE: func(cmd *cobra.Command, _ []string) error {
-		return loadConfig()
-	},
-}
-
-func Execute() {
-	if err := rootCmd.Execute(); err != nil {
-		fmt.Fprintln(os.Stderr, err)
-		os.Exit(1)
-	}
-}
-
-func init() {
-	rootCmd.PersistentFlags().StringVar(&cfgFile, "config", "", "Path to config file")
-	rootCmd.PersistentFlags().StringVar(&serverFlag, "server", "", "AMCS server URL")
-	rootCmd.PersistentFlags().StringVar(&tokenFlag, "token", "", "AMCS bearer token")
-	rootCmd.PersistentFlags().StringVar(&outputFlag, "output", "json", "Output format: json or yaml")
-}
-
-func loadConfig() error {
-	path, err := resolveConfigPath(cfgFile)
-	if err != nil {
-		return err
-	}
-	loaded, err := loadConfigFile(path)
-	if err != nil {
-		return err
-	}
-	cfg = loaded
-	if v := strings.TrimSpace(os.Getenv("AMCS_URL")); v != "" {
-		cfg.Server = v
-	}
-	if v := strings.TrimSpace(os.Getenv("AMCS_TOKEN")); v != "" {
-		cfg.Token = v
-	}
-	if v := strings.TrimSpace(serverFlag); v != "" {
-		cfg.Server = v
-	}
-	if v := strings.TrimSpace(tokenFlag); v != "" {
-		cfg.Token = v
-	}
-	outputFlag = strings.ToLower(strings.TrimSpace(outputFlag))
-	if outputFlag != "json" && outputFlag != "yaml" {
-		return fmt.Errorf("invalid --output %q: must be json or yaml", outputFlag)
-	}
-	return nil
-}
-
-func requireServer() error {
-	if strings.TrimSpace(cfg.Server) == "" {
-		return fmt.Errorf("server URL is required; set --server, AMCS_URL, or config server")
-	}
-	return nil
-}
-
-func endpointURL() string {
-	base := strings.TrimRight(strings.TrimSpace(cfg.Server), "/")
-	if strings.HasSuffix(base, "/mcp") {
-		return base
-	}
-	return base + "/mcp"
-}
-
-func newHTTPClient() *http.Client {
-	return &http.Client{
-		Timeout: 0,
-		Transport: &bearerTransport{
-			base:  http.DefaultTransport,
-			token: cfg.Token,
-		},
-	}
-}
-
-type bearerTransport struct {
-	base  http.RoundTripper
-	token string
-}
-
-func (t *bearerTransport) RoundTrip(req *http.Request) (*http.Response, error) {
-	base := t.base
-	if base == nil {
-		base = http.DefaultTransport
-	}
-	clone := req.Clone(req.Context())
-	if strings.TrimSpace(t.token) != "" {
-		clone.Header.Set("Authorization", "Bearer "+t.token)
-	}
-	return base.RoundTrip(clone)
-}
-
-func connectRemote(ctx context.Context) (*mcp.ClientSession, error) {
-	if err := requireServer(); err != nil {
-		return nil, err
-	}
-	client := mcp.NewClient(&mcp.Implementation{Name: "amcs-cli", Version: "0.0.1"}, nil)
-	transport := &mcp.StreamableClientTransport{
-		Endpoint:   endpointURL(),
-		HTTPClient: newHTTPClient(),
-	}
-	ctx, cancel := context.WithTimeout(ctx, 30*time.Second)
-	defer cancel()
-	session, err := client.Connect(ctx, transport, nil)
-	if err != nil {
-		return nil, fmt.Errorf("connect to AMCS server: %w", err)
-	}
-	return session, nil
-}

cmd/amcs-cli/cmd/sse.go

@@ -1,86 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strings"
-	"time"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-	"github.com/spf13/cobra"
-)
-
-var sseCmd = &cobra.Command{
-	Use:   "sse",
-	Short: "Run a stdio MCP bridge backed by a remote AMCS server using SSE transport (widely supported by hosted MCP clients)",
-	RunE: func(cmd *cobra.Command, _ []string) error {
-		ctx := cmd.Context()
-
-		if err := requireServer(); err != nil {
-			return err
-		}
-
-		client := mcp.NewClient(&mcp.Implementation{Name: "amcs-cli", Version: "0.0.1"}, nil)
-		transport := &mcp.SSEClientTransport{
-			Endpoint:   sseEndpointURL(),
-			HTTPClient: newHTTPClient(),
-		}
-
-		connectCtx, cancel := context.WithTimeout(ctx, 30*time.Second)
-		defer cancel()
-
-		remote, err := client.Connect(connectCtx, transport, nil)
-		if err != nil {
-			return fmt.Errorf("connect to AMCS SSE endpoint: %w", err)
-		}
-		defer func() { _ = remote.Close() }()
-
-		tools, err := remote.ListTools(ctx, nil)
-		if err != nil {
-			return fmt.Errorf("load remote tools: %w", err)
-		}
-
-		server := mcp.NewServer(&mcp.Implementation{
-			Name:    "amcs-cli",
-			Title:   "AMCS CLI Bridge (SSE)",
-			Version: "0.0.1",
-		}, nil)
-
-		for _, tool := range tools.Tools {
-			remoteTool := tool
-			server.AddTool(&mcp.Tool{
-				Name:         remoteTool.Name,
-				Description:  remoteTool.Description,
-				InputSchema:  remoteTool.InputSchema,
-				OutputSchema: remoteTool.OutputSchema,
-				Annotations:  remoteTool.Annotations,
-			}, func(ctx context.Context, req *mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-				return remote.CallTool(ctx, &mcp.CallToolParams{
-					Name:      req.Params.Name,
-					Arguments: req.Params.Arguments,
-				})
-			})
-		}
-
-		session, err := server.Connect(ctx, &mcp.StdioTransport{}, nil)
-		if err != nil {
-			return fmt.Errorf("start stdio bridge: %w", err)
-		}
-		defer func() { _ = session.Close() }()
-
-		<-ctx.Done()
-		return nil
-	},
-}
-
-func sseEndpointURL() string {
-	base := strings.TrimRight(strings.TrimSpace(cfg.Server), "/")
-	if strings.HasSuffix(base, "/sse") {
-		return base
-	}
-	return base + "/sse"
-}
-
-func init() {
-	rootCmd.AddCommand(sseCmd)
-}

cmd/amcs-cli/cmd/stdio.go

@@ -1,62 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-	"github.com/spf13/cobra"
-)
-
-var stdioCmd = &cobra.Command{
-	Use:   "stdio",
-	Short: "Run a stdio MCP bridge backed by a remote AMCS server",
-	RunE: func(cmd *cobra.Command, _ []string) error {
-		ctx := cmd.Context()
-		remote, err := connectRemote(ctx)
-		if err != nil {
-			return err
-		}
-		defer func() { _ = remote.Close() }()
-
-		tools, err := remote.ListTools(ctx, nil)
-		if err != nil {
-			return fmt.Errorf("load remote tools: %w", err)
-		}
-
-		server := mcp.NewServer(&mcp.Implementation{
-			Name:    "amcs-cli",
-			Title:   "AMCS CLI Bridge",
-			Version: "0.0.1",
-		}, nil)
-
-		for _, tool := range tools.Tools {
-			remoteTool := tool
-			server.AddTool(&mcp.Tool{
-				Name:         remoteTool.Name,
-				Description:  remoteTool.Description,
-				InputSchema:  remoteTool.InputSchema,
-				OutputSchema: remoteTool.OutputSchema,
-				Annotations:  remoteTool.Annotations,
-			}, func(ctx context.Context, req *mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-				return remote.CallTool(ctx, &mcp.CallToolParams{
-					Name:      req.Params.Name,
-					Arguments: req.Params.Arguments,
-				})
-			})
-		}
-
-		session, err := server.Connect(ctx, &mcp.StdioTransport{}, nil)
-		if err != nil {
-			return fmt.Errorf("start stdio bridge: %w", err)
-		}
-		defer func() { _ = session.Close() }()
-
-		<-ctx.Done()
-		return nil
-	},
-}
-
-func init() {
-	rootCmd.AddCommand(stdioCmd)
-}

cmd/amcs-cli/cmd/tools.go

@@ -1,38 +0,0 @@
-package cmd
-
-import (
-	"fmt"
-	"os"
-	"strings"
-	"text/tabwriter"
-
-	"github.com/spf13/cobra"
-)
-
-var toolsCmd = &cobra.Command{
-	Use:   "tools",
-	Short: "List tools available on the remote AMCS server",
-	RunE: func(cmd *cobra.Command, _ []string) error {
-		session, err := connectRemote(cmd.Context())
-		if err != nil {
-			return err
-		}
-		defer func() { _ = session.Close() }()
-
-		res, err := session.ListTools(cmd.Context(), nil)
-		if err != nil {
-			return fmt.Errorf("list tools: %w", err)
-		}
-
-		w := tabwriter.NewWriter(os.Stdout, 0, 0, 2, ' ', 0)
-		fmt.Fprintln(w, "NAME\tDESCRIPTION")
-		for _, tool := range res.Tools {
-			fmt.Fprintf(w, "%s\t%s\n", tool.Name, strings.TrimSpace(tool.Description))
-		}
-		return w.Flush()
-	},
-}
-
-func init() {
-	rootCmd.AddCommand(toolsCmd)
-}

cmd/amcs-cli/main.go

@@ -1,7 +0,0 @@
-package main
-
-import "git.warky.dev/wdevs/amcs/cmd/amcs-cli/cmd"
-
-func main() {
-	cmd.Execute()
-}

configs/config.example.yaml

@@ -9,7 +9,6 @@ server:
 
 mcp:
   path: "/mcp"
-  sse_path: "/sse"
   server_name: "amcs"
   transport: "streamable_http"
   session_timeout: "10m"

configs/dev.yaml

@@ -9,7 +9,6 @@ server:
 
 mcp:
   path: "/mcp"
-  sse_path: "/sse"
   server_name: "amcs"
   transport: "streamable_http"
   session_timeout: "10m"

configs/docker.yaml

@@ -9,7 +9,6 @@ server:
 
 mcp:
   path: "/mcp"
-  sse_path: "/sse"
   server_name: "amcs"
   transport: "streamable_http"
   session_timeout: "10m"

go.mod

@@ -8,13 +8,11 @@ require (
 	github.com/jackc/pgx/v5 v5.9.1
 	github.com/modelcontextprotocol/go-sdk v1.4.1
 	github.com/pgvector/pgvector-go v0.3.0
-	github.com/spf13/cobra v1.10.2
 	golang.org/x/sync v0.17.0
 	gopkg.in/yaml.v3 v3.0.1
 )
 
 require (
-	github.com/inconshreveable/mousetrap v1.1.0 // indirect
 	github.com/jackc/pgpassfile v1.0.0 // indirect
 	github.com/jackc/pgservicefile v0.0.0-20240606120523-5a60cdf6a761 // indirect
 	github.com/jackc/puddle/v2 v2.2.2 // indirect
@@ -22,7 +20,6 @@ require (
 	github.com/rogpeppe/go-internal v1.14.1 // indirect
 	github.com/segmentio/asm v1.1.3 // indirect
 	github.com/segmentio/encoding v0.5.4 // indirect
-	github.com/spf13/pflag v1.0.9 // indirect
 	github.com/x448/float16 v0.8.4 // indirect
 	github.com/yosida95/uritemplate/v3 v3.0.2 // indirect
 	golang.org/x/oauth2 v0.34.0 // indirect

go.sum

@@ -1,6 +1,5 @@
 entgo.io/ent v0.14.3 h1:wokAV/kIlH9TeklJWGGS7AYJdVckr0DloWjIcO9iIIQ=
 entgo.io/ent v0.14.3/go.mod h1:aDPE/OziPEu8+OWbzy4UlvWmD2/kbRuWfK2A40hcxJM=
-github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
 github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
@@ -17,8 +16,6 @@ github.com/google/jsonschema-go v0.4.2 h1:tmrUohrwoLZZS/P3x7ex0WAVknEkBZM46iALbc
 github.com/google/jsonschema-go v0.4.2/go.mod h1:r5quNTdLOYEz95Ru18zA0ydNbBuYoo9tgaYcxEYhJVE=
 github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
-github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
-github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
 github.com/jackc/pgpassfile v1.0.0 h1:/6Hmqy13Ss2zCq62VdNG8tM1wchn8zjSGOBJ6icpsIM=
 github.com/jackc/pgpassfile v1.0.0/go.mod h1:CEx0iS5ambNFdcRtxPj5JhEz+xB6uRky5eyVu/W2HEg=
 github.com/jackc/pgservicefile v0.0.0-20240606120523-5a60cdf6a761 h1:iCEnooe7UlwOQYpKFhBabPMi4aNAfoODPEFNiAnClxo=
@@ -47,15 +44,10 @@ github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZb
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/rogpeppe/go-internal v1.14.1 h1:UQB4HGPB6osV0SQTLymcB4TgvyWu6ZyliaW0tI/otEQ=
 github.com/rogpeppe/go-internal v1.14.1/go.mod h1:MaRKkUm5W0goXpeCfT7UZI6fk/L7L7so1lCWt35ZSgc=
-github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
 github.com/segmentio/asm v1.1.3 h1:WM03sfUOENvvKexOLp+pCqgb/WDjsi7EK8gIsICtzhc=
 github.com/segmentio/asm v1.1.3/go.mod h1:Ld3L4ZXGNcSLRg4JBsZ3//1+f/TjYl0Mzen/DQy1EJg=
 github.com/segmentio/encoding v0.5.4 h1:OW1VRern8Nw6ITAtwSZ7Idrl3MXCFwXHPgqESYfvNt0=
 github.com/segmentio/encoding v0.5.4/go.mod h1:HS1ZKa3kSN32ZHVZ7ZLPLXWvOVIiZtyJnO1gPH1sKt0=
-github.com/spf13/cobra v1.10.2 h1:DMTTonx5m65Ic0GOoRY2c16WCbHxOOw6xxezuLaBpcU=
-github.com/spf13/cobra v1.10.2/go.mod h1:7C1pvHqHw5A4vrJfjNwvOdzYu0Gml16OCs2GRiTUUS4=
-github.com/spf13/pflag v1.0.9 h1:9exaQaMOCwffKiiiYk6/BndUBv+iRViNW+4lEMi0PvY=
-github.com/spf13/pflag v1.0.9/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
 github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
 github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
 github.com/stretchr/testify v1.7.0/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
@@ -81,7 +73,6 @@ github.com/x448/float16 v0.8.4 h1:qLwI1I70+NjRFUR3zs1JPUCgaCXSh3SW62uAKT1mSBM=
 github.com/x448/float16 v0.8.4/go.mod h1:14CWIYCyZA/cWjXOioeEpHeN/83MdbZDRQHoFcYsOfg=
 github.com/yosida95/uritemplate/v3 v3.0.2 h1:Ed3Oyj9yrmi9087+NczuL5BwkIc4wvTb5zIM+UJPGz4=
 github.com/yosida95/uritemplate/v3 v3.0.2/go.mod h1:ILOh0sOhIJR3+L/8afwt/kE++YT040gmv5BQTMR2HP4=
-go.yaml.in/yaml/v3 v3.0.4/go.mod h1:DhzuOOF2ATzADvBadXxruRBLzYTpT36CKvDb3+aBEFg=
 golang.org/x/crypto v0.36.0 h1:AnAEvhDddvBdpY+uR+MyHmuZzzNqXSe/GvuDeob5L34=
 golang.org/x/crypto v0.36.0/go.mod h1:Y4J0ReaxCR1IMaabaSMugxJES1EpwhBHhv2bDHklZvc=
 golang.org/x/oauth2 v0.34.0 h1:hqK/t4AKgbqWkdkcAeI8XLmbK+4m4G5YeQRrmiotGlw=

internal/app/app.go

@@ -158,15 +158,12 @@ func Run(ctx context.Context, configPath string) error {
 
 func routes(logger *slog.Logger, cfg *config.Config, info buildinfo.Info, db *store.DB, provider ai.Provider, keyring *auth.Keyring, oauthRegistry *auth.OAuthRegistry, tokenStore *auth.TokenStore, authCodes *auth.AuthCodeStore, dynClients *auth.DynamicClientStore, activeProjects *session.ActiveProjects) (http.Handler, error) {
 	mux := http.NewServeMux()
-	accessTracker := auth.NewAccessTracker()
+	authMiddleware := auth.Middleware(cfg.Auth, keyring, oauthRegistry, tokenStore, logger)
-	oauthEnabled := oauthRegistry != nil && tokenStore != nil
-	authMiddleware := auth.Middleware(cfg.Auth, keyring, oauthRegistry, tokenStore, accessTracker, logger)
 	filesTool := tools.NewFilesTool(db, activeProjects)
-	enrichmentRetryer := tools.NewEnrichmentRetryer(context.Background(), db, provider, cfg.Capture, cfg.AI.Metadata.Timeout, activeProjects, logger)
+	metadataRetryer := tools.NewMetadataRetryer(context.Background(), db, provider, cfg.Capture, cfg.AI.Metadata.Timeout, activeProjects, logger)
-	backfillTool := tools.NewBackfillTool(db, provider, activeProjects, logger)
 
 	toolSet := mcpserver.ToolSet{
-		Capture:       tools.NewCaptureTool(db, provider, cfg.Capture, cfg.AI.Metadata.Timeout, activeProjects, enrichmentRetryer, backfillTool, logger),
+		Capture:       tools.NewCaptureTool(db, provider, cfg.Capture, cfg.AI.Metadata.Timeout, activeProjects, metadataRetryer, logger),
 		Search:        tools.NewSearchTool(db, provider, cfg.Search, activeProjects),
 		List:          tools.NewListTool(db, cfg.Search, activeProjects),
 		Stats:         tools.NewStatsTool(db),
@@ -181,27 +178,26 @@ func routes(logger *slog.Logger, cfg *config.Config, info buildinfo.Info, db *st
 		Summarize:     tools.NewSummarizeTool(db, provider, cfg.Search, activeProjects),
 		Links:         tools.NewLinksTool(db, provider, cfg.Search),
 		Files:         filesTool,
-		Backfill:      backfillTool,
+		Backfill:      tools.NewBackfillTool(db, provider, activeProjects, logger),
 		Reparse:       tools.NewReparseMetadataTool(db, provider, cfg.Capture, activeProjects, logger),
-		RetryMetadata: tools.NewRetryEnrichmentTool(enrichmentRetryer),
+		RetryMetadata: tools.NewRetryMetadataTool(metadataRetryer),
+		Household:     tools.NewHouseholdTool(db),
 		Maintenance:   tools.NewMaintenanceTool(db),
+		Calendar:      tools.NewCalendarTool(db),
+		Meals:         tools.NewMealsTool(db),
+		CRM:           tools.NewCRMTool(db),
 		Skills:        tools.NewSkillsTool(db, activeProjects),
 		ChatHistory:   tools.NewChatHistoryTool(db, activeProjects),
-		Describe:      tools.NewDescribeTool(db, mcpserver.BuildToolCatalog()),
 	}
 
-	mcpHandlers, err := mcpserver.NewHandlers(cfg.MCP, logger, toolSet, activeProjects.Clear)
+	mcpHandler, err := mcpserver.New(cfg.MCP, logger, toolSet, activeProjects.Clear)
 	if err != nil {
 		return nil, fmt.Errorf("build mcp handler: %w", err)
 	}
-	mux.Handle(cfg.MCP.Path, authMiddleware(mcpHandlers.StreamableHTTP))
+	mux.Handle(cfg.MCP.Path, authMiddleware(mcpHandler))
-	if mcpHandlers.SSE != nil {
-		mux.Handle(cfg.MCP.SSEPath, authMiddleware(mcpHandlers.SSE))
-		logger.Info("SSE transport enabled", slog.String("sse_path", cfg.MCP.SSEPath))
-	}
 	mux.Handle("/files", authMiddleware(fileHandler(filesTool)))
 	mux.Handle("/files/{id}", authMiddleware(fileHandler(filesTool)))
-	if oauthEnabled {
+	if oauthRegistry != nil && tokenStore != nil {
 		mux.HandleFunc("/.well-known/oauth-authorization-server", oauthMetadataHandler())
 		mux.HandleFunc("/oauth-authorization-server", oauthMetadataHandler())
 		mux.HandleFunc("/oauth/register", oauthRegisterHandler(dynClients, logger))
@@ -211,9 +207,7 @@ func routes(logger *slog.Logger, cfg *config.Config, info buildinfo.Info, db *st
 	}
 	mux.HandleFunc("/favicon.ico", serveFavicon)
 	mux.HandleFunc("/images/project.jpg", serveHomeImage)
-	mux.HandleFunc("/images/icon.png", serveIcon)
 	mux.HandleFunc("/llm", serveLLMInstructions)
-	mux.HandleFunc("/api/status", statusAPIHandler(info, accessTracker, oauthEnabled))
 
 	mux.HandleFunc("/healthz", func(w http.ResponseWriter, r *http.Request) {
 		w.WriteHeader(http.StatusOK)
@@ -231,7 +225,59 @@ func routes(logger *slog.Logger, cfg *config.Config, info buildinfo.Info, db *st
 		_, _ = w.Write([]byte("ready"))
 	})
 
-	mux.HandleFunc("/", homeHandler(info, accessTracker, oauthEnabled))
+	mux.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/" {
+			http.NotFound(w, r)
+			return
+		}
+
+		if r.Method != http.MethodGet && r.Method != http.MethodHead {
+			w.Header().Set("Allow", "GET, HEAD")
+			http.Error(w, "method not allowed", http.StatusMethodNotAllowed)
+			return
+		}
+
+		const homePage = `<!doctype html>
+<html lang="en">
+<head>
+  <meta charset="utf-8">
+  <meta name="viewport" content="width=device-width, initial-scale=1">
+  <title>AMCS</title>
+  <style>
+    body { margin: 0; font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif; background: #f5f7fb; color: #172033; }
+    main { max-width: 860px; margin: 48px auto; background: #fff; border-radius: 12px; box-shadow: 0 10px 28px rgba(23, 32, 51, 0.12); overflow: hidden; }
+    .content { padding: 28px; }
+    h1 { margin: 0 0 12px 0; font-size: 2rem; }
+    p { margin: 0; line-height: 1.5; color: #334155; }
+    .actions { margin-top: 18px; }
+    .link { display: inline-block; padding: 10px 14px; border-radius: 8px; background: #172033; color: #fff; text-decoration: none; font-weight: 600; }
+    .link:hover { background: #0f172a; }
+    img { display: block; width: 100%; height: auto; }
+  </style>
+</head>
+<body>
+  <main>
+    <img src="/images/project.jpg" alt="Avelon Memory Crystal project image">
+    <div class="content">
+      <h1>Avelon Memory Crystal Server (AMCS)</h1>
+      <p>AMCS is a memory server that captures, links, and retrieves structured project thoughts for AI assistants using semantic search, summaries, and MCP tools.</p>
+      <div class="actions">
+        <a class="link" href="/llm">LLM Instructions</a>
+        <a class="link" href="/oauth-authorization-server">OAuth Authorization Server</a>
+        <a class="link" href="/healthz">Health Check</a>
+      </div>
+    </div>
+  </main>
+</body>
+</html>`
+		w.Header().Set("Content-Type", "text/html; charset=utf-8")
+		w.WriteHeader(http.StatusOK)
+		if r.Method == http.MethodHead {
+			return
+		}
+
+		_, _ = w.Write([]byte(homePage))
+	})
 
 	return observability.Chain(
 		mux,
@@ -296,26 +342,3 @@ func serveHomeImage(w http.ResponseWriter, r *http.Request) {
 
 	_, _ = w.Write(homeImage)
 }
-
-func serveIcon(w http.ResponseWriter, r *http.Request) {
-	if iconImage == nil {
-		http.NotFound(w, r)
-		return
-	}
-
-	if r.Method != http.MethodGet && r.Method != http.MethodHead {
-		w.Header().Set("Allow", "GET, HEAD")
-		http.Error(w, "method not allowed", http.StatusMethodNotAllowed)
-		return
-	}
-
-	w.Header().Set("Content-Type", "image/png")
-	w.Header().Set("Cache-Control", "public, max-age=31536000, immutable")
-	w.WriteHeader(http.StatusOK)
-
-	if r.Method == http.MethodHead {
-		return
-	}
-
-	_, _ = w.Write(iconImage)
-}

internal/app/static_assets.go

@@ -12,7 +12,6 @@ var (
 
 	faviconICO = mustReadStaticFile("favicon.ico")
 	homeImage  = mustReadStaticFile("avelonmemorycrystal.jpg")
-	iconImage  = tryReadStaticFile("icon.png")
 )
 
 func mustReadStaticFile(name string) []byte {
@@ -23,11 +22,3 @@ func mustReadStaticFile(name string) []byte {
 
 	return data
 }
-
-func tryReadStaticFile(name string) []byte {
-	data, err := fs.ReadFile(staticFiles, "static/"+name)
-	if err != nil {
-		return nil
-	}
-	return data
-}

internal/app/status.go

@@ -1,140 +0,0 @@
-package app
-
-import (
-	"bytes"
-	"encoding/json"
-	"io/fs"
-	"net/http"
-	"path"
-	"strings"
-	"time"
-
-	"git.warky.dev/wdevs/amcs/internal/auth"
-	"git.warky.dev/wdevs/amcs/internal/buildinfo"
-)
-
-const connectedWindow = 10 * time.Minute
-
-type statusAPIResponse struct {
-	Title           string                `json:"title"`
-	Description     string                `json:"description"`
-	Version         string                `json:"version"`
-	BuildDate       string                `json:"build_date"`
-	Commit          string                `json:"commit"`
-	ConnectedCount  int                   `json:"connected_count"`
-	TotalKnown      int                   `json:"total_known"`
-	ConnectedWindow string                `json:"connected_window"`
-	Entries         []auth.AccessSnapshot `json:"entries"`
-	OAuthEnabled    bool                  `json:"oauth_enabled"`
-}
-
-func statusSnapshot(info buildinfo.Info, tracker *auth.AccessTracker, oauthEnabled bool, now time.Time) statusAPIResponse {
-	entries := tracker.Snapshot()
-	return statusAPIResponse{
-		Title:           "Avelon Memory Crystal Server (AMCS)",
-		Description:     "AMCS is a memory server that captures, links, and retrieves structured project thoughts for AI assistants using semantic search, summaries, and MCP tools.",
-		Version:         fallback(info.Version, "dev"),
-		BuildDate:       fallback(info.BuildDate, "unknown"),
-		Commit:          fallback(info.Commit, "unknown"),
-		ConnectedCount:  tracker.ConnectedCount(now, connectedWindow),
-		TotalKnown:      len(entries),
-		ConnectedWindow: "last 10 minutes",
-		Entries:         entries,
-		OAuthEnabled:    oauthEnabled,
-	}
-}
-
-func fallback(value, defaultValue string) string {
-	if strings.TrimSpace(value) == "" {
-		return defaultValue
-	}
-	return value
-}
-
-func statusAPIHandler(info buildinfo.Info, tracker *auth.AccessTracker, oauthEnabled bool) http.HandlerFunc {
-	return func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/api/status" {
-			http.NotFound(w, r)
-			return
-		}
-		if r.Method != http.MethodGet && r.Method != http.MethodHead {
-			w.Header().Set("Allow", "GET, HEAD")
-			http.Error(w, "method not allowed", http.StatusMethodNotAllowed)
-			return
-		}
-
-		w.Header().Set("Content-Type", "application/json; charset=utf-8")
-		w.WriteHeader(http.StatusOK)
-		if r.Method == http.MethodHead {
-			return
-		}
-
-		_ = json.NewEncoder(w).Encode(statusSnapshot(info, tracker, oauthEnabled, time.Now()))
-	}
-}
-
-func homeHandler(_ buildinfo.Info, _ *auth.AccessTracker, _ bool) http.HandlerFunc {
-	return func(w http.ResponseWriter, r *http.Request) {
-		if r.Method != http.MethodGet && r.Method != http.MethodHead {
-			w.Header().Set("Allow", "GET, HEAD")
-			http.Error(w, "method not allowed", http.StatusMethodNotAllowed)
-			return
-		}
-
-		requestPath := strings.TrimPrefix(path.Clean(r.URL.Path), "/")
-		if requestPath == "." {
-			requestPath = ""
-		}
-
-		if requestPath != "" {
-			if serveUIAsset(w, r, requestPath) {
-				return
-			}
-			http.NotFound(w, r)
-			return
-		}
-
-		serveUIIndex(w, r)
-	}
-}
-
-func serveUIAsset(w http.ResponseWriter, r *http.Request, name string) bool {
-	if uiDistFS == nil {
-		return false
-	}
-	if strings.Contains(name, "..") {
-		return false
-	}
-	file, err := uiDistFS.Open(name)
-	if err != nil {
-		return false
-	}
-	defer file.Close()
-
-	info, err := file.Stat()
-	if err != nil || info.IsDir() {
-		return false
-	}
-
-	data, err := fs.ReadFile(uiDistFS, name)
-	if err != nil {
-		return false
-	}
-
-	http.ServeContent(w, r, info.Name(), info.ModTime(), bytes.NewReader(data))
-	return true
-}
-
-func serveUIIndex(w http.ResponseWriter, r *http.Request) {
-	if indexHTML == nil {
-		http.Error(w, "ui assets not built", http.StatusServiceUnavailable)
-		return
-	}
-
-	w.Header().Set("Content-Type", "text/html; charset=utf-8")
-	w.WriteHeader(http.StatusOK)
-	if r.Method == http.MethodHead {
-		return
-	}
-	_, _ = w.Write(indexHTML)
-}

internal/app/status_test.go

@@ -1,133 +0,0 @@
-package app
-
-import (
-	"encoding/json"
-	"io"
-	"log/slog"
-	"net/http"
-	"net/http/httptest"
-	"strings"
-	"testing"
-	"time"
-
-	"git.warky.dev/wdevs/amcs/internal/auth"
-	"git.warky.dev/wdevs/amcs/internal/buildinfo"
-	"git.warky.dev/wdevs/amcs/internal/config"
-)
-
-func TestStatusSnapshotHidesOAuthLinkWhenDisabled(t *testing.T) {
-	tracker := auth.NewAccessTracker()
-	snapshot := statusSnapshot(buildinfo.Info{Version: "v1.2.3", BuildDate: "2026-04-04", Commit: "abc123"}, tracker, false, time.Date(2026, 4, 4, 12, 0, 0, 0, time.UTC))
-
-	if snapshot.OAuthEnabled {
-		t.Fatal("OAuthEnabled = true, want false")
-	}
-	if snapshot.ConnectedCount != 0 {
-		t.Fatalf("ConnectedCount = %d, want 0", snapshot.ConnectedCount)
-	}
-	if snapshot.Title == "" {
-		t.Fatal("Title = empty, want non-empty")
-	}
-}
-
-func TestStatusSnapshotShowsTrackedAccess(t *testing.T) {
-	tracker := auth.NewAccessTracker()
-	now := time.Date(2026, 4, 4, 12, 0, 0, 0, time.UTC)
-	tracker.Record("client-a", "/files", "127.0.0.1:1234", "tester", now)
-
-	snapshot := statusSnapshot(buildinfo.Info{Version: "v1.2.3"}, tracker, true, now)
-
-	if !snapshot.OAuthEnabled {
-		t.Fatal("OAuthEnabled = false, want true")
-	}
-	if snapshot.ConnectedCount != 1 {
-		t.Fatalf("ConnectedCount = %d, want 1", snapshot.ConnectedCount)
-	}
-	if len(snapshot.Entries) != 1 {
-		t.Fatalf("len(Entries) = %d, want 1", len(snapshot.Entries))
-	}
-	if snapshot.Entries[0].KeyID != "client-a" || snapshot.Entries[0].LastPath != "/files" {
-		t.Fatalf("entry = %+v, want keyID client-a and path /files", snapshot.Entries[0])
-	}
-}
-
-func TestStatusAPIHandlerReturnsJSON(t *testing.T) {
-	handler := statusAPIHandler(buildinfo.Info{Version: "v1"}, auth.NewAccessTracker(), true)
-	req := httptest.NewRequest(http.MethodGet, "/api/status", nil)
-	rec := httptest.NewRecorder()
-
-	handler.ServeHTTP(rec, req)
-
-	if rec.Code != http.StatusOK {
-		t.Fatalf("status = %d, want %d", rec.Code, http.StatusOK)
-	}
-	if got := rec.Header().Get("Content-Type"); !strings.Contains(got, "application/json") {
-		t.Fatalf("content-type = %q, want application/json", got)
-	}
-
-	var payload statusAPIResponse
-	if err := json.Unmarshal(rec.Body.Bytes(), &payload); err != nil {
-		t.Fatalf("json.Unmarshal() error = %v", err)
-	}
-	if payload.Version != "v1" {
-		t.Fatalf("version = %q, want %q", payload.Version, "v1")
-	}
-}
-
-func TestHomeHandlerAllowsHead(t *testing.T) {
-	handler := homeHandler(buildinfo.Info{Version: "v1"}, auth.NewAccessTracker(), false)
-	req := httptest.NewRequest(http.MethodHead, "/", nil)
-	rec := httptest.NewRecorder()
-
-	handler.ServeHTTP(rec, req)
-
-	if rec.Code != http.StatusOK {
-		t.Fatalf("status = %d, want %d", rec.Code, http.StatusOK)
-	}
-	if body := rec.Body.String(); body != "" {
-		t.Fatalf("body = %q, want empty for HEAD", body)
-	}
-}
-
-func TestHomeHandlerServesIndex(t *testing.T) {
-	handler := homeHandler(buildinfo.Info{Version: "v1"}, auth.NewAccessTracker(), false)
-	req := httptest.NewRequest(http.MethodGet, "/", nil)
-	rec := httptest.NewRecorder()
-
-	handler.ServeHTTP(rec, req)
-
-	if rec.Code != http.StatusOK {
-		t.Fatalf("status = %d, want %d", rec.Code, http.StatusOK)
-	}
-	if !strings.Contains(rec.Body.String(), "<div id=\"app\"></div>") {
-		t.Fatalf("body = %q, want embedded UI index", rec.Body.String())
-	}
-}
-
-func TestMiddlewareRecordsAuthenticatedAccess(t *testing.T) {
-	keyring, err := auth.NewKeyring([]config.APIKey{{ID: "client-a", Value: "secret"}})
-	if err != nil {
-		t.Fatalf("NewKeyring() error = %v", err)
-	}
-	tracker := auth.NewAccessTracker()
-	logger := slog.New(slog.NewTextHandler(io.Discard, nil))
-	handler := auth.Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, tracker, logger)(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNoContent)
-	}))
-
-	req := httptest.NewRequest(http.MethodGet, "/files", nil)
-	req.Header.Set("x-brain-key", "secret")
-	rec := httptest.NewRecorder()
-	handler.ServeHTTP(rec, req)
-
-	if rec.Code != http.StatusNoContent {
-		t.Fatalf("status = %d, want %d", rec.Code, http.StatusNoContent)
-	}
-	snap := tracker.Snapshot()
-	if len(snap) != 1 {
-		t.Fatalf("len(snapshot) = %d, want 1", len(snap))
-	}
-	if snap[0].KeyID != "client-a" || snap[0].LastPath != "/files" {
-		t.Fatalf("snapshot[0] = %+v, want keyID client-a and path /files", snap[0])
-	}
-}

internal/app/ui_assets.go

@@ -1,22 +0,0 @@
-package app
-
-import (
-	"embed"
-	"io/fs"
-)
-
-var (
-	//go:embed ui/dist
-	uiFiles embed.FS
-	uiDistFS fs.FS
-	indexHTML []byte
-)
-
-func init() {
-	dist, err := fs.Sub(uiFiles, "ui/dist")
-	if err != nil {
-		return
-	}
-	uiDistFS = dist
-	indexHTML, _ = fs.ReadFile(uiDistFS, "index.html")
-}

internal/auth/access_tracker.go

@@ -1,81 +0,0 @@
-package auth
-
-import (
-	"sort"
-	"sync"
-	"time"
-)
-
-type AccessSnapshot struct {
-	KeyID          string    `json:"key_id"`
-	LastPath       string    `json:"last_path"`
-	RemoteAddr     string    `json:"remote_addr"`
-	UserAgent      string    `json:"user_agent"`
-	RequestCount   int       `json:"request_count"`
-	LastAccessedAt time.Time `json:"last_accessed_at"`
-}
-
-type AccessTracker struct {
-	mu      sync.RWMutex
-	entries map[string]AccessSnapshot
-}
-
-func NewAccessTracker() *AccessTracker {
-	return &AccessTracker{entries: make(map[string]AccessSnapshot)}
-}
-
-func (t *AccessTracker) Record(keyID, path, remoteAddr, userAgent string, now time.Time) {
-	if t == nil || keyID == "" {
-		return
-	}
-
-	t.mu.Lock()
-	defer t.mu.Unlock()
-
-	entry := t.entries[keyID]
-	entry.KeyID = keyID
-	entry.LastPath = path
-	entry.RemoteAddr = remoteAddr
-	entry.UserAgent = userAgent
-	entry.LastAccessedAt = now.UTC()
-	entry.RequestCount++
-	t.entries[keyID] = entry
-}
-
-func (t *AccessTracker) Snapshot() []AccessSnapshot {
-	if t == nil {
-		return nil
-	}
-
-	t.mu.RLock()
-	defer t.mu.RUnlock()
-
-	items := make([]AccessSnapshot, 0, len(t.entries))
-	for _, entry := range t.entries {
-		items = append(items, entry)
-	}
-
-	sort.Slice(items, func(i, j int) bool {
-		return items[i].LastAccessedAt.After(items[j].LastAccessedAt)
-	})
-
-	return items
-}
-
-func (t *AccessTracker) ConnectedCount(now time.Time, window time.Duration) int {
-	if t == nil {
-		return 0
-	}
-
-	cutoff := now.UTC().Add(-window)
-	t.mu.RLock()
-	defer t.mu.RUnlock()
-
-	count := 0
-	for _, entry := range t.entries {
-		if !entry.LastAccessedAt.Before(cutoff) {
-			count++
-		}
-	}
-	return count
-}

internal/auth/access_tracker_test.go

@@ -1,45 +0,0 @@
-package auth
-
-import (
-	"testing"
-	"time"
-)
-
-func TestAccessTrackerRecordAndSnapshot(t *testing.T) {
-	tracker := NewAccessTracker()
-	older := time.Date(2026, 4, 4, 10, 0, 0, 0, time.UTC)
-	newer := older.Add(2 * time.Minute)
-
-	tracker.Record("client-a", "/files", "10.0.0.1:1234", "agent-a", older)
-	tracker.Record("client-b", "/mcp", "10.0.0.2:1234", "agent-b", newer)
-	tracker.Record("client-a", "/files/1", "10.0.0.1:1234", "agent-a2", newer.Add(30*time.Second))
-
-	snap := tracker.Snapshot()
-	if len(snap) != 2 {
-		t.Fatalf("len(snapshot) = %d, want 2", len(snap))
-	}
-	if snap[0].KeyID != "client-a" {
-		t.Fatalf("snapshot[0].KeyID = %q, want client-a", snap[0].KeyID)
-	}
-	if snap[0].RequestCount != 2 {
-		t.Fatalf("snapshot[0].RequestCount = %d, want 2", snap[0].RequestCount)
-	}
-	if snap[0].LastPath != "/files/1" {
-		t.Fatalf("snapshot[0].LastPath = %q, want /files/1", snap[0].LastPath)
-	}
-	if snap[0].UserAgent != "agent-a2" {
-		t.Fatalf("snapshot[0].UserAgent = %q, want agent-a2", snap[0].UserAgent)
-	}
-}
-
-func TestAccessTrackerConnectedCount(t *testing.T) {
-	tracker := NewAccessTracker()
-	now := time.Date(2026, 4, 4, 12, 0, 0, 0, time.UTC)
-
-	tracker.Record("recent", "/mcp", "", "", now.Add(-2*time.Minute))
-	tracker.Record("stale", "/mcp", "", "", now.Add(-11*time.Minute))
-
-	if got := tracker.ConnectedCount(now, 10*time.Minute); got != 1 {
-		t.Fatalf("ConnectedCount() = %d, want 1", got)
-	}
-}

internal/auth/keyring_test.go

@@ -39,7 +39,7 @@ func TestMiddlewareAllowsHeaderAuthAndSetsContext(t *testing.T) {
 		t.Fatalf("NewKeyring() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		keyID, ok := KeyIDFromContext(r.Context())
 		if !ok || keyID != "client-a" {
 			t.Fatalf("KeyIDFromContext() = (%q, %v), want (client-a, true)", keyID, ok)
@@ -63,7 +63,7 @@ func TestMiddlewareAllowsBearerAuthAndSetsContext(t *testing.T) {
 		t.Fatalf("NewKeyring() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		keyID, ok := KeyIDFromContext(r.Context())
 		if !ok || keyID != "client-a" {
 			t.Fatalf("KeyIDFromContext() = (%q, %v), want (client-a, true)", keyID, ok)
@@ -90,7 +90,7 @@ func TestMiddlewarePrefersExplicitHeaderOverBearerAuth(t *testing.T) {
 		t.Fatalf("NewKeyring() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		keyID, ok := KeyIDFromContext(r.Context())
 		if !ok || keyID != "client-a" {
 			t.Fatalf("KeyIDFromContext() = (%q, %v), want (client-a, true)", keyID, ok)
@@ -119,7 +119,7 @@ func TestMiddlewareAllowsQueryParamWhenEnabled(t *testing.T) {
 		HeaderName:      "x-brain-key",
 		QueryParam:      "key",
 		AllowQueryParam: true,
-	}, keyring, nil, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	}, keyring, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		w.WriteHeader(http.StatusNoContent)
 	}))
 
@@ -138,7 +138,7 @@ func TestMiddlewareRejectsMissingOrInvalidKey(t *testing.T) {
 		t.Fatalf("NewKeyring() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{HeaderName: "x-brain-key"}, keyring, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		t.Fatal("next handler should not be called")
 	}))
 

internal/auth/middleware.go

@@ -6,7 +6,6 @@ import (
 	"log/slog"
 	"net/http"
 	"strings"
-	"time"
 
 	"git.warky.dev/wdevs/amcs/internal/config"
 )
@@ -15,16 +14,11 @@ type contextKey string
 
 const keyIDContextKey contextKey = "auth.key_id"
 
-func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthRegistry, tokenStore *TokenStore, tracker *AccessTracker, log *slog.Logger) func(http.Handler) http.Handler {
+func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthRegistry, tokenStore *TokenStore, log *slog.Logger) func(http.Handler) http.Handler {
 	headerName := cfg.HeaderName
 	if headerName == "" {
 		headerName = "x-brain-key"
 	}
-	recordAccess := func(r *http.Request, keyID string) {
-		if tracker != nil {
-			tracker.Record(keyID, r.URL.Path, r.RemoteAddr, r.UserAgent(), time.Now())
-		}
-	}
 	return func(next http.Handler) http.Handler {
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			// 1. Custom header → keyring only.
@@ -36,7 +30,6 @@ func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthReg
 						http.Error(w, "invalid API key", http.StatusUnauthorized)
 						return
 					}
-					recordAccess(r, keyID)
 					next.ServeHTTP(w, r.WithContext(context.WithValue(r.Context(), keyIDContextKey, keyID)))
 					return
 				}
@@ -46,14 +39,12 @@ func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthReg
 			if bearer := extractBearer(r); bearer != "" {
 				if tokenStore != nil {
 					if keyID, ok := tokenStore.Lookup(bearer); ok {
-						recordAccess(r, keyID)
 						next.ServeHTTP(w, r.WithContext(context.WithValue(r.Context(), keyIDContextKey, keyID)))
 						return
 					}
 				}
 				if keyring != nil {
 					if keyID, ok := keyring.Lookup(bearer); ok {
-						recordAccess(r, keyID)
 						next.ServeHTTP(w, r.WithContext(context.WithValue(r.Context(), keyIDContextKey, keyID)))
 						return
 					}
@@ -75,7 +66,6 @@ func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthReg
 					http.Error(w, "invalid OAuth client credentials", http.StatusUnauthorized)
 					return
 				}
-				recordAccess(r, keyID)
 				next.ServeHTTP(w, r.WithContext(context.WithValue(r.Context(), keyIDContextKey, keyID)))
 				return
 			}
@@ -89,7 +79,6 @@ func Middleware(cfg config.AuthConfig, keyring *Keyring, oauthRegistry *OAuthReg
 						http.Error(w, "invalid API key", http.StatusUnauthorized)
 						return
 					}
-					recordAccess(r, keyID)
 					next.ServeHTTP(w, r.WithContext(context.WithValue(r.Context(), keyIDContextKey, keyID)))
 					return
 				}

internal/auth/oauth_registry_test.go

@@ -42,7 +42,7 @@ func TestMiddlewareAllowsOAuthBasicAuthAndSetsContext(t *testing.T) {
 		t.Fatalf("NewOAuthRegistry() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{}, nil, oauthRegistry, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{}, nil, oauthRegistry, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		keyID, ok := KeyIDFromContext(r.Context())
 		if !ok || keyID != "oauth-client" {
 			t.Fatalf("KeyIDFromContext() = (%q, %v), want (oauth-client, true)", keyID, ok)
@@ -70,7 +70,7 @@ func TestMiddlewareRejectsOAuthMissingOrInvalidCredentials(t *testing.T) {
 		t.Fatalf("NewOAuthRegistry() error = %v", err)
 	}
 
-	handler := Middleware(config.AuthConfig{}, nil, oauthRegistry, nil, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	handler := Middleware(config.AuthConfig{}, nil, oauthRegistry, nil, testLogger())(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		t.Fatal("next handler should not be called")
 	}))
 

internal/config/config.go

@@ -32,16 +32,10 @@ type ServerConfig struct {
 
 type MCPConfig struct {
 	Path           string        `yaml:"path"`
-	SSEPath        string        `yaml:"sse_path"`
 	ServerName     string        `yaml:"server_name"`
 	Version        string        `yaml:"version"`
 	Transport      string        `yaml:"transport"`
 	SessionTimeout time.Duration `yaml:"session_timeout"`
-	// PublicURL is the externally reachable base URL of this server (e.g. https://amcs.example.com).
-	// When set, it is used to build absolute icon URLs in the MCP server identity.
-	PublicURL string `yaml:"public_url"`
-	// Instructions is set at startup from the embedded memory.md and sent to MCP clients on initialise.
-	Instructions string `yaml:"-"`
 }
 
 type AuthConfig struct {

internal/config/loader.go

@@ -58,7 +58,6 @@ func defaultConfig() Config {
 		},
 		MCP: MCPConfig{
 			Path:           "/mcp",
-			SSEPath:        "/sse",
 			ServerName:     "amcs",
 			Version:        info.Version,
 			Transport:      "streamable_http",
@@ -118,7 +117,6 @@ func defaultConfig() Config {
 
 func applyEnvOverrides(cfg *Config) {
 	overrideString(&cfg.Database.URL, "AMCS_DATABASE_URL")
-	overrideString(&cfg.MCP.PublicURL, "AMCS_PUBLIC_URL")
 	overrideString(&cfg.AI.LiteLLM.BaseURL, "AMCS_LITELLM_BASE_URL")
 	overrideString(&cfg.AI.LiteLLM.APIKey, "AMCS_LITELLM_API_KEY")
 	overrideString(&cfg.AI.Ollama.BaseURL, "AMCS_OLLAMA_BASE_URL")

internal/config/validate.go

@@ -33,14 +33,6 @@ func (c Config) Validate() error {
 	if strings.TrimSpace(c.MCP.Path) == "" {
 		return fmt.Errorf("invalid config: mcp.path is required")
 	}
-	if c.MCP.SSEPath != "" {
-		if strings.TrimSpace(c.MCP.SSEPath) == "" {
-			return fmt.Errorf("invalid config: mcp.sse_path must not be blank whitespace")
-		}
-		if c.MCP.SSEPath == c.MCP.Path {
-			return fmt.Errorf("invalid config: mcp.sse_path %q must differ from mcp.path", c.MCP.SSEPath)
-		}
-	}
 	if c.MCP.SessionTimeout <= 0 {
 		return fmt.Errorf("invalid config: mcp.session_timeout must be greater than zero")
 	}

internal/mcpserver/schema.go

@@ -221,19 +221,12 @@ func formatLogDuration(d time.Duration) string {
 	return fmt.Sprintf("%02d:%02d:%03d", minutes, seconds, milliseconds)
 }
 
-func normalizeObjectSchema(schema *jsonschema.Schema) {
-	if schema != nil && schema.Type == "object" && schema.Properties == nil {
-		schema.Properties = map[string]*jsonschema.Schema{}
-	}
-}
-
 func setToolSchemas[In any, Out any](tool *mcp.Tool) error {
 	if tool.InputSchema == nil {
 		inputSchema, err := jsonschema.For[In](toolSchemaOptions)
 		if err != nil {
 			return fmt.Errorf("infer input schema: %w", err)
 		}
-		normalizeObjectSchema(inputSchema)
 		tool.InputSchema = inputSchema
 	}
 

internal/mcpserver/schema_test.go

@@ -13,24 +13,6 @@ import (
 	"git.warky.dev/wdevs/amcs/internal/tools"
 )
 
-func TestSetToolSchemasAddsEmptyPropertiesForNoArgInput(t *testing.T) {
-	type noArgInput struct{}
-	type anyOutput struct{}
-
-	tool := &mcp.Tool{Name: "no_args"}
-	if err := setToolSchemas[noArgInput, anyOutput](tool); err != nil {
-		t.Fatalf("set tool schemas: %v", err)
-	}
-
-	schema, ok := tool.InputSchema.(*jsonschema.Schema)
-	if !ok {
-		t.Fatalf("input schema type = %T, want *jsonschema.Schema", tool.InputSchema)
-	}
-	if schema.Properties == nil {
-		t.Fatal("input schema missing properties: strict MCP clients require properties:{} on object schemas")
-	}
-}
-
 func TestSetToolSchemasUsesStringUUIDsInListOutput(t *testing.T) {
 	tool := &mcp.Tool{Name: "list_thoughts"}
 

internal/mcpserver/server.go

@@ -3,18 +3,11 @@ package mcpserver
 import (
 	"log/slog"
 	"net/http"
-	"strings"
 
 	"github.com/modelcontextprotocol/go-sdk/mcp"
 
 	"git.warky.dev/wdevs/amcs/internal/config"
 	"git.warky.dev/wdevs/amcs/internal/tools"
-	amcsllm "git.warky.dev/wdevs/amcs/llm"
-)
-
-const (
-	serverTitle      = "Avalon Memory Crystal Server"
-	serverWebsiteURL = "https://git.warky.dev/wdevs/amcs"
 )
 
 type ToolSet struct {
@@ -36,48 +29,20 @@ type ToolSet struct {
 	Backfill      *tools.BackfillTool
 	Reparse       *tools.ReparseMetadataTool
 	RetryMetadata *tools.RetryMetadataTool
+	Household     *tools.HouseholdTool
 	Maintenance   *tools.MaintenanceTool
+	Calendar      *tools.CalendarTool
+	Meals         *tools.MealsTool
+	CRM           *tools.CRMTool
 	Skills        *tools.SkillsTool
 	ChatHistory   *tools.ChatHistoryTool
-	Describe      *tools.DescribeTool
 }
 
-// Handlers groups the HTTP handlers produced for an MCP server instance.
-type Handlers struct {
-	// StreamableHTTP is the primary MCP handler (always non-nil).
-	StreamableHTTP http.Handler
-	// SSE is the SSE transport handler; nil when SSEPath is empty.
-	// SSE is the de facto transport for MCP over the internet and is required by most hosted MCP clients.
-	SSE http.Handler
-}
-
-// New builds the StreamableHTTP MCP handler. It is a convenience wrapper
-// around NewHandlers for callers that only need the primary transport.
 func New(cfg config.MCPConfig, logger *slog.Logger, toolSet ToolSet, onSessionClosed func(string)) (http.Handler, error) {
-	h, err := NewHandlers(cfg, logger, toolSet, onSessionClosed)
-	if err != nil {
-		return nil, err
-	}
-	return h.StreamableHTTP, nil
-}
-
-// NewHandlers builds MCP HTTP handlers for both transports.
-// SSE is nil when cfg.SSEPath is empty.
-func NewHandlers(cfg config.MCPConfig, logger *slog.Logger, toolSet ToolSet, onSessionClosed func(string)) (Handlers, error) {
-	instructions := cfg.Instructions
-	if instructions == "" {
-		instructions = string(amcsllm.MemoryInstructions)
-	}
-
 	server := mcp.NewServer(&mcp.Implementation{
 		Name:    cfg.ServerName,
-		Title:      serverTitle,
 		Version: cfg.Version,
-		WebsiteURL: serverWebsiteURL,
+	}, nil)
-		Icons:      buildServerIcons(cfg.PublicURL),
-	}, &mcp.ServerOptions{
-		Instructions: instructions,
-	})
 
 	for _, register := range []func(*mcp.Server, *slog.Logger, ToolSet) error{
 		registerSystemTools,
@@ -85,12 +50,15 @@ func NewHandlers(cfg config.MCPConfig, logger *slog.Logger, toolSet ToolSet, onS
 		registerProjectTools,
 		registerFileTools,
 		registerMaintenanceTools,
+		registerHouseholdTools,
+		registerCalendarTools,
+		registerMealTools,
+		registerCRMTools,
 		registerSkillTools,
 		registerChatHistoryTools,
-		registerDescribeTools,
 	} {
 		if err := register(server, logger, toolSet); err != nil {
-			return Handlers{}, err
+			return nil, err
 		}
 	}
 
@@ -102,37 +70,15 @@ func NewHandlers(cfg config.MCPConfig, logger *slog.Logger, toolSet ToolSet, onS
 		opts.EventStore = newCleanupEventStore(mcp.NewMemoryEventStore(nil), onSessionClosed)
 	}
 
-	h := Handlers{
+	return mcp.NewStreamableHTTPHandler(func(*http.Request) *mcp.Server {
-		StreamableHTTP: mcp.NewStreamableHTTPHandler(func(*http.Request) *mcp.Server {
 		return server
-		}, opts),
+	}, opts), nil
-	}
-
-	if strings.TrimSpace(cfg.SSEPath) != "" {
-		h.SSE = mcp.NewSSEHandler(func(*http.Request) *mcp.Server {
-			return server
-		}, nil)
-	}
-
-	return h, nil
-}
-
-// buildServerIcons returns icon definitions referencing the server's own /images/icon.png endpoint.
-// Returns nil when publicURL is empty so the icons field is omitted from the MCP identity.
-func buildServerIcons(publicURL string) []mcp.Icon {
-	if strings.TrimSpace(publicURL) == "" {
-		return nil
-	}
-	base := strings.TrimRight(publicURL, "/")
-	return []mcp.Icon{
-		{Source: base + "/images/icon.png", MIMEType: "image/png"},
-	}
 }
 
 func registerSystemTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "get_version_info",
-		Description: "Build version, commit, and date.",
+		Description: "Return the server build version information, including version, tag name, commit, and build date.",
 	}, toolSet.Version.GetInfo); err != nil {
 		return err
 	}
@@ -142,13 +88,13 @@ func registerSystemTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSe
 func registerThoughtTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "capture_thought",
-		Description: "Store a thought; embeddings and metadata extracted async.",
+		Description: "Store a thought with generated embeddings and extracted metadata.",
 	}, toolSet.Capture.Handle); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "search_thoughts",
-		Description: "Semantic search; falls back to full-text if no embeddings.",
+		Description: "Search stored thoughts by semantic similarity.",
 	}, toolSet.Search.Handle); err != nil {
 		return err
 	}
@@ -160,7 +106,7 @@ func registerThoughtTools(server *mcp.Server, logger *slog.Logger, toolSet ToolS
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "thought_stats",
-		Description: "Counts and top metadata buckets for stored thoughts.",
+		Description: "Get counts and top metadata buckets across stored thoughts.",
 	}, toolSet.Stats.Handle); err != nil {
 		return err
 	}
@@ -184,19 +130,19 @@ func registerThoughtTools(server *mcp.Server, logger *slog.Logger, toolSet ToolS
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "archive_thought",
-		Description: "Hide a thought from default search and listing.",
+		Description: "Archive a thought so it is hidden from default search and listing.",
 	}, toolSet.Archive.Handle); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "summarize_thoughts",
-		Description: "LLM summary of a filtered set of thoughts.",
+		Description: "Summarize a filtered or searched set of thoughts.",
 	}, toolSet.Summarize.Handle); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "recall_context",
-		Description: "Semantic + recency context for prompt injection; falls back to full-text.",
+		Description: "Recall semantically relevant and recent context.",
 	}, toolSet.Recall.Handle); err != nil {
 		return err
 	}
@@ -208,7 +154,7 @@ func registerThoughtTools(server *mcp.Server, logger *slog.Logger, toolSet ToolS
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "related_thoughts",
-		Description: "Explicit links and semantic neighbours; falls back to full-text.",
+		Description: "Retrieve explicit links and semantic neighbors for a thought.",
 	}, toolSet.Links.Related); err != nil {
 		return err
 	}
@@ -230,19 +176,19 @@ func registerProjectTools(server *mcp.Server, logger *slog.Logger, toolSet ToolS
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "set_active_project",
-		Description: "Set session's active project. Pass project per call if client is stateless.",
+		Description: "Set the active project for the current MCP session. Requires a stateful MCP client that reuses the same session across calls.",
 	}, toolSet.Projects.SetActive); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "get_active_project",
-		Description: "Return session's active project. Pass project per call if client is stateless.",
+		Description: "Return the active project for the current MCP session. If your client does not preserve MCP sessions, pass project explicitly to project-scoped tools instead.",
 	}, toolSet.Projects.GetActive); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "get_project_context",
-		Description: "Recent and semantic context for a project; falls back to full-text.",
+		Description: "Get recent and semantic context for a project. Uses the explicit project when provided, otherwise the active MCP session project.",
 	}, toolSet.Context.Handle); err != nil {
 		return err
 	}
@@ -258,19 +204,19 @@ func registerFileTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet)
 
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "upload_file",
-		Description: "Stage a file; returns amcs://files/{id}. content_path for large/binary, content_base64 for ≤10 MB. Link now or pass URI to save_file.",
+		Description: "Stage a file and get an amcs://files/{id} resource URI. Provide content_path (absolute server-side path, no size limit) or content_base64 (≤10 MB). Optionally link immediately with thought_id/project, or omit them and pass the returned URI to save_file later.",
 	}, toolSet.Files.Upload); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "save_file",
-		Description: "Store and optionally link a file. content_base64 (≤10 MB) or content_uri from upload_file. >10 MB: use upload_file first.",
+		Description: "Store a file and optionally link it to a thought. Supply either content_base64 (≤10 MB) or content_uri (amcs://files/{id} from a prior upload_file or POST /files call). For files larger than 10 MB, use upload_file with content_path first.",
 	}, toolSet.Files.Save); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "load_file",
-		Description: "Fetch file metadata and content by id (UUID or amcs://files/{id}); includes embedded MCP resource.",
+		Description: "Load a previously stored file by id and return its metadata and base64 content.",
 	}, toolSet.Files.Load); err != nil {
 		return err
 	}
@@ -286,19 +232,19 @@ func registerFileTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet)
 func registerMaintenanceTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "backfill_embeddings",
-		Description: "Generate missing embeddings. Run after model switch or bulk import.",
+		Description: "Generate missing embeddings for stored thoughts using the active embedding model.",
 	}, toolSet.Backfill.Handle); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "reparse_thought_metadata",
-		Description: "Re-extract metadata from thought content.",
+		Description: "Re-extract and normalize metadata for stored thoughts from their content.",
 	}, toolSet.Reparse.Handle); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "retry_failed_metadata",
-		Description: "Retry pending/failed metadata extraction.",
+		Description: "Retry metadata extraction for thoughts still marked pending or failed.",
 	}, toolSet.RetryMetadata.Handle); err != nil {
 		return err
 	}
@@ -310,7 +256,7 @@ func registerMaintenanceTools(server *mcp.Server, logger *slog.Logger, toolSet T
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "log_maintenance",
-		Description: "Log completed maintenance; updates next due date.",
+		Description: "Log completed maintenance work; automatically updates the task's next due date.",
 	}, toolSet.Maintenance.LogWork); err != nil {
 		return err
 	}
@@ -329,10 +275,176 @@ func registerMaintenanceTools(server *mcp.Server, logger *slog.Logger, toolSet T
 	return nil
 }
 
+func registerHouseholdTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_household_item",
+		Description: "Store a household fact (paint color, appliance details, measurement, document, etc.).",
+	}, toolSet.Household.AddItem); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "search_household_items",
+		Description: "Search household items by name, category, or location.",
+	}, toolSet.Household.SearchItems); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_household_item",
+		Description: "Retrieve a household item by id.",
+	}, toolSet.Household.GetItem); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_vendor",
+		Description: "Add a service provider (plumber, electrician, landscaper, etc.).",
+	}, toolSet.Household.AddVendor); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "list_vendors",
+		Description: "List household service vendors, optionally filtered by service type.",
+	}, toolSet.Household.ListVendors); err != nil {
+		return err
+	}
+	return nil
+}
+
+func registerCalendarTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_family_member",
+		Description: "Add a family member to the household.",
+	}, toolSet.Calendar.AddMember); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "list_family_members",
+		Description: "List all family members.",
+	}, toolSet.Calendar.ListMembers); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_activity",
+		Description: "Schedule a one-time or recurring family activity.",
+	}, toolSet.Calendar.AddActivity); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_week_schedule",
+		Description: "Get all activities scheduled for a given week.",
+	}, toolSet.Calendar.GetWeekSchedule); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "search_activities",
+		Description: "Search activities by title, type, or family member.",
+	}, toolSet.Calendar.SearchActivities); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_important_date",
+		Description: "Track a birthday, anniversary, deadline, or other important date.",
+	}, toolSet.Calendar.AddImportantDate); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_upcoming_dates",
+		Description: "Get important dates coming up in the next N days.",
+	}, toolSet.Calendar.GetUpcomingDates); err != nil {
+		return err
+	}
+	return nil
+}
+
+func registerMealTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_recipe",
+		Description: "Save a recipe with ingredients and instructions.",
+	}, toolSet.Meals.AddRecipe); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "search_recipes",
+		Description: "Search recipes by name, cuisine, tags, or ingredient.",
+	}, toolSet.Meals.SearchRecipes); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "update_recipe",
+		Description: "Update an existing recipe.",
+	}, toolSet.Meals.UpdateRecipe); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "create_meal_plan",
+		Description: "Set the meal plan for a week; replaces any existing plan for that week.",
+	}, toolSet.Meals.CreateMealPlan); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_meal_plan",
+		Description: "Get the meal plan for a given week.",
+	}, toolSet.Meals.GetMealPlan); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "generate_shopping_list",
+		Description: "Auto-generate a shopping list from the meal plan for a given week.",
+	}, toolSet.Meals.GenerateShoppingList); err != nil {
+		return err
+	}
+	return nil
+}
+
+func registerCRMTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "add_professional_contact",
+		Description: "Add a professional contact to the CRM.",
+	}, toolSet.CRM.AddContact); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "search_contacts",
+		Description: "Search professional contacts by name, company, title, notes, or tags.",
+	}, toolSet.CRM.SearchContacts); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "log_interaction",
+		Description: "Log an interaction with a professional contact.",
+	}, toolSet.CRM.LogInteraction); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_contact_history",
+		Description: "Get full history (interactions and opportunities) for a contact.",
+	}, toolSet.CRM.GetHistory); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "create_opportunity",
+		Description: "Create a deal, project, or opportunity linked to a contact.",
+	}, toolSet.CRM.CreateOpportunity); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "get_follow_ups_due",
+		Description: "List contacts with a follow-up date due within the next N days.",
+	}, toolSet.CRM.GetFollowUpsDue); err != nil {
+		return err
+	}
+	if err := addTool(server, logger, &mcp.Tool{
+		Name:        "link_thought_to_contact",
+		Description: "Append a stored thought to a contact's notes.",
+	}, toolSet.CRM.LinkThought); err != nil {
+		return err
+	}
+	return nil
+}
+
 func registerSkillTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "add_skill",
-		Description: "Store an agent skill (instruction or capability prompt).",
+		Description: "Store a reusable agent skill (behavioural instruction or capability prompt).",
 	}, toolSet.Skills.AddSkill); err != nil {
 		return err
 	}
@@ -350,7 +462,7 @@ func registerSkillTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "add_guardrail",
-		Description: "Store an agent guardrail (constraint or safety rule).",
+		Description: "Store a reusable agent guardrail (constraint or safety rule).",
 	}, toolSet.Skills.AddGuardrail); err != nil {
 		return err
 	}
@@ -368,37 +480,37 @@ func registerSkillTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "add_project_skill",
-		Description: "Link a skill to a project. Pass project if client is stateless.",
+		Description: "Link an agent skill to a project. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.AddProjectSkill); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "remove_project_skill",
-		Description: "Unlink a skill from a project. Pass project if client is stateless.",
+		Description: "Unlink an agent skill from a project. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.RemoveProjectSkill); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "list_project_skills",
-		Description: "Skills for a project. Load at session start; only add new if none returned. Pass project if stateless.",
+		Description: "List all skills linked to a project. Call this at the start of a project session to load existing agent behaviour instructions before generating new ones. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.ListProjectSkills); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "add_project_guardrail",
-		Description: "Link a guardrail to a project. Pass project if client is stateless.",
+		Description: "Link an agent guardrail to a project. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.AddProjectGuardrail); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "remove_project_guardrail",
-		Description: "Unlink a guardrail from a project. Pass project if client is stateless.",
+		Description: "Unlink an agent guardrail from a project. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.RemoveProjectGuardrail); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "list_project_guardrails",
-		Description: "Guardrails for a project. Load at session start; only add new if none returned. Pass project if stateless.",
+		Description: "List all guardrails linked to a project. Call this at the start of a project session to load existing agent constraints before generating new ones. Pass project explicitly when your client does not preserve MCP sessions.",
 	}, toolSet.Skills.ListProjectGuardrails); err != nil {
 		return err
 	}
@@ -408,114 +520,27 @@ func registerSkillTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet
 func registerChatHistoryTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "save_chat_history",
-		Description: "Save chat messages with optional title, summary, channel, agent, and project.",
+		Description: "Save a chat session's message history for later retrieval. Stores messages with optional title, summary, channel, agent, and project metadata.",
 	}, toolSet.ChatHistory.SaveChatHistory); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "get_chat_history",
-		Description: "Fetch chat history by UUID or session_id.",
+		Description: "Retrieve a saved chat history by its UUID or session_id. Returns the full message list.",
 	}, toolSet.ChatHistory.GetChatHistory); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "list_chat_histories",
-		Description: "List chat histories; filter by project, channel, agent_id, session_id, or days.",
+		Description: "List saved chat histories with optional filters: project, channel, agent_id, session_id, or recent days.",
 	}, toolSet.ChatHistory.ListChatHistories); err != nil {
 		return err
 	}
 	if err := addTool(server, logger, &mcp.Tool{
 		Name:        "delete_chat_history",
-		Description: "Delete a chat history by id.",
+		Description: "Permanently delete a saved chat history by id.",
 	}, toolSet.ChatHistory.DeleteChatHistory); err != nil {
 		return err
 	}
 	return nil
 }
-
-func registerDescribeTools(server *mcp.Server, logger *slog.Logger, toolSet ToolSet) error {
-	if err := addTool(server, logger, &mcp.Tool{
-		Name:        "describe_tools",
-		Description: "Call first each session. All tools with categories and usage notes. Categories: system, thoughts, projects, files, admin, maintenance, skills, chat, meta.",
-	}, toolSet.Describe.Describe); err != nil {
-		return err
-	}
-	if err := addTool(server, logger, &mcp.Tool{
-		Name:        "annotate_tool",
-		Description: "Save usage notes for a tool; returned by describe_tools. Empty string clears.",
-	}, toolSet.Describe.Annotate); err != nil {
-		return err
-	}
-	return nil
-}
-
-// BuildToolCatalog returns the static catalog of all registered MCP tools.
-// Pass this to tools.NewDescribeTool when assembling the ToolSet.
-func BuildToolCatalog() []tools.ToolEntry {
-	return []tools.ToolEntry{
-		// system
-		{Name: "get_version_info", Description: "Return the server build version information, including version, tag name, commit, and build date.", Category: "system"},
-
-		// thoughts
-		{Name: "capture_thought", Description: "Store a thought with generated embeddings and extracted metadata. The thought is saved immediately even if metadata extraction times out; pending thoughts are retried in the background.", Category: "thoughts"},
-		{Name: "search_thoughts", Description: "Search stored thoughts by semantic similarity. Falls back to Postgres full-text search automatically when no embeddings exist for the active model.", Category: "thoughts"},
-		{Name: "list_thoughts", Description: "List recent thoughts with optional metadata filters.", Category: "thoughts"},
-		{Name: "thought_stats", Description: "Get counts and top metadata buckets across stored thoughts.", Category: "thoughts"},
-		{Name: "get_thought", Description: "Retrieve a full thought by id.", Category: "thoughts"},
-		{Name: "update_thought", Description: "Update thought content or merge metadata.", Category: "thoughts"},
-		{Name: "delete_thought", Description: "Hard-delete a thought by id.", Category: "thoughts"},
-		{Name: "archive_thought", Description: "Archive a thought so it is hidden from default search and listing.", Category: "thoughts"},
-		{Name: "summarize_thoughts", Description: "Produce an LLM prose summary of a filtered or searched set of thoughts.", Category: "thoughts"},
-		{Name: "recall_context", Description: "Recall semantically relevant and recent context for prompt injection. Combines vector similarity with recency. Falls back to full-text search when no embeddings exist.", Category: "thoughts"},
-		{Name: "link_thoughts", Description: "Create a typed relationship between two thoughts.", Category: "thoughts"},
-		{Name: "related_thoughts", Description: "Retrieve explicit links and semantic neighbours for a thought. Falls back to full-text search when no embeddings exist.", Category: "thoughts"},
-
-		// projects
-		{Name: "create_project", Description: "Create a named project container for thoughts.", Category: "projects"},
-		{Name: "list_projects", Description: "List projects and their current thought counts.", Category: "projects"},
-		{Name: "set_active_project", Description: "Set the active project for the current MCP session. Requires a stateful MCP client that reuses the same session across calls. If your client does not preserve sessions, pass project explicitly to each tool instead.", Category: "projects"},
-		{Name: "get_active_project", Description: "Return the active project for the current MCP session. If your client does not preserve MCP sessions, pass project explicitly to project-scoped tools instead of relying on this.", Category: "projects"},
-		{Name: "get_project_context", Description: "Get recent and semantic context for a project. Uses the explicit project when provided, otherwise the active MCP session project. Falls back to full-text search when no embeddings exist.", Category: "projects"},
-
-		// files
-		{Name: "upload_file", Description: "Stage a file and get an amcs://files/{id} resource URI. Use content_path (absolute server-side path, no size limit) for large or binary files, or content_base64 (≤10 MB) for small files. Pass thought_id/project to link immediately, or omit and pass the URI to save_file later.", Category: "files"},
-		{Name: "save_file", Description: "Store a file and optionally link it to a thought. Use content_base64 (≤10 MB) for small files, or content_uri (amcs://files/{id} from a prior upload_file) for previously staged files. For files larger than 10 MB, use upload_file with content_path first. If the goal is to retain the artifact, store the file directly instead of reading or summarising it first.", Category: "files"},
-		{Name: "load_file", Description: "Load a stored file by id. Returns metadata, base64 content, and an embedded MCP binary resource at amcs://files/{id}. Prefer the embedded resource when your client supports it. The id field accepts a bare UUID or full amcs://files/{id} URI.", Category: "files"},
-		{Name: "list_files", Description: "List stored files, optionally filtered by thought, project, or kind.", Category: "files"},
-
-		// admin
-		{Name: "backfill_embeddings", Description: "Generate missing embeddings for stored thoughts using the active embedding model. Run this after switching embedding models or importing thoughts that have no vectors.", Category: "admin"},
-		{Name: "reparse_thought_metadata", Description: "Re-extract and normalize metadata for stored thoughts from their content.", Category: "admin"},
-		{Name: "retry_failed_metadata", Description: "Retry metadata extraction for thoughts still marked pending or failed.", Category: "admin"},
-
-		// maintenance
-		{Name: "add_maintenance_task", Description: "Create a recurring or one-time home maintenance task.", Category: "maintenance"},
-		{Name: "log_maintenance", Description: "Log completed maintenance work; automatically updates the task's next due date.", Category: "maintenance"},
-		{Name: "get_upcoming_maintenance", Description: "List maintenance tasks due within the next N days.", Category: "maintenance"},
-		{Name: "search_maintenance_history", Description: "Search the maintenance log by task name, category, or date range.", Category: "maintenance"},
-
-		// skills
-		{Name: "add_skill", Description: "Store a reusable agent skill (behavioural instruction or capability prompt).", Category: "skills"},
-		{Name: "remove_skill", Description: "Delete an agent skill by id.", Category: "skills"},
-		{Name: "list_skills", Description: "List all agent skills, optionally filtered by tag.", Category: "skills"},
-		{Name: "add_guardrail", Description: "Store a reusable agent guardrail (constraint or safety rule).", Category: "skills"},
-		{Name: "remove_guardrail", Description: "Delete an agent guardrail by id.", Category: "skills"},
-		{Name: "list_guardrails", Description: "List all agent guardrails, optionally filtered by tag or severity.", Category: "skills"},
-		{Name: "add_project_skill", Description: "Link an agent skill to a project. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-		{Name: "remove_project_skill", Description: "Unlink an agent skill from a project. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-		{Name: "list_project_skills", Description: "List all skills linked to a project. Call this at the start of every project session to load agent behaviour instructions before generating new ones. Only create new skills if none are returned. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-		{Name: "add_project_guardrail", Description: "Link an agent guardrail to a project. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-		{Name: "remove_project_guardrail", Description: "Unlink an agent guardrail from a project. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-		{Name: "list_project_guardrails", Description: "List all guardrails linked to a project. Call this at the start of every project session to load agent constraints before generating new ones. Only create new guardrails if none are returned. Pass project explicitly when your client does not preserve MCP sessions.", Category: "skills"},
-
-		// chat
-		{Name: "save_chat_history", Description: "Save a chat session's message history for later retrieval. Stores messages with optional title, summary, channel, agent, and project metadata.", Category: "chat"},
-		{Name: "get_chat_history", Description: "Retrieve a saved chat history by its UUID or session_id. Returns the full message list.", Category: "chat"},
-		{Name: "list_chat_histories", Description: "List saved chat histories with optional filters: project, channel, agent_id, session_id, or recent days.", Category: "chat"},
-		{Name: "delete_chat_history", Description: "Permanently delete a saved chat history by id.", Category: "chat"},
-
-		// meta
-		{Name: "describe_tools", Description: "Call this first in every session. Returns all available MCP tools with names, descriptions, categories, and your accumulated usage notes. Filter by category to narrow results. Available categories: system, thoughts, projects, files, admin, household, maintenance, calendar, meals, crm, skills, chat, meta.", Category: "meta"},
-		{Name: "annotate_tool", Description: "Persist usage notes, gotchas, or workflow patterns for a specific tool. Notes survive across sessions and are returned by describe_tools. Call this whenever you discover something non-obvious about a tool's behaviour. Pass an empty string to clear notes.", Category: "meta"},
-	}
-}

internal/mcpserver/server_test.go

@@ -28,27 +28,40 @@ func TestNewListsAllRegisteredTools(t *testing.T) {
 	sort.Strings(got)
 
 	want := []string{
+		"add_activity",
+		"add_family_member",
 		"add_guardrail",
+		"add_household_item",
+		"add_important_date",
 		"add_maintenance_task",
+		"add_professional_contact",
 		"add_project_guardrail",
 		"add_project_skill",
+		"add_recipe",
 		"add_skill",
-		"annotate_tool",
+		"add_vendor",
 		"archive_thought",
 		"backfill_embeddings",
 		"capture_thought",
+		"create_meal_plan",
+		"create_opportunity",
 		"create_project",
-		"delete_chat_history",
 		"delete_thought",
-		"describe_tools",
+		"generate_shopping_list",
 		"get_active_project",
-		"get_chat_history",
+		"get_contact_history",
+		"get_follow_ups_due",
+		"get_household_item",
+		"get_meal_plan",
 		"get_project_context",
 		"get_thought",
+		"get_upcoming_dates",
 		"get_upcoming_maintenance",
 		"get_version_info",
+		"get_week_schedule",
+		"link_thought_to_contact",
 		"link_thoughts",
-		"list_chat_histories",
+		"list_family_members",
 		"list_files",
 		"list_guardrails",
 		"list_project_guardrails",
@@ -56,7 +69,9 @@ func TestNewListsAllRegisteredTools(t *testing.T) {
 		"list_projects",
 		"list_skills",
 		"list_thoughts",
+		"list_vendors",
 		"load_file",
+		"log_interaction",
 		"log_maintenance",
 		"recall_context",
 		"related_thoughts",
@@ -66,13 +81,17 @@ func TestNewListsAllRegisteredTools(t *testing.T) {
 		"remove_skill",
 		"reparse_thought_metadata",
 		"retry_failed_metadata",
-		"save_chat_history",
 		"save_file",
+		"search_activities",
+		"search_contacts",
+		"search_household_items",
 		"search_maintenance_history",
+		"search_recipes",
 		"search_thoughts",
 		"set_active_project",
 		"summarize_thoughts",
 		"thought_stats",
+		"update_recipe",
 		"update_thought",
 		"upload_file",
 	}

internal/mcpserver/sse_test.go

@@ -1,136 +0,0 @@
-package mcpserver
-
-import (
-	"context"
-	"net/http/httptest"
-	"testing"
-	"time"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-
-	"git.warky.dev/wdevs/amcs/internal/config"
-)
-
-func TestNewHandlers_SSEDisabledByDefault(t *testing.T) {
-	h, err := NewHandlers(config.MCPConfig{
-		ServerName:     "test",
-		Version:        "0.0.1",
-		SessionTimeout: time.Minute,
-	}, nil, streamableTestToolSet(), nil)
-	if err != nil {
-		t.Fatalf("NewHandlers() error = %v", err)
-	}
-	if h.StreamableHTTP == nil {
-		t.Fatal("StreamableHTTP handler is nil")
-	}
-	if h.SSE != nil {
-		t.Fatal("SSE handler should be nil when SSEPath is empty")
-	}
-}
-
-func TestNewHandlers_SSEEnabledWhenPathSet(t *testing.T) {
-	h, err := NewHandlers(config.MCPConfig{
-		ServerName:     "test",
-		Version:        "0.0.1",
-		SessionTimeout: time.Minute,
-		SSEPath:        "/sse",
-	}, nil, streamableTestToolSet(), nil)
-	if err != nil {
-		t.Fatalf("NewHandlers() error = %v", err)
-	}
-	if h.StreamableHTTP == nil {
-		t.Fatal("StreamableHTTP handler is nil")
-	}
-	if h.SSE == nil {
-		t.Fatal("SSE handler is nil when SSEPath is set")
-	}
-}
-
-func TestNew_BackwardCompatibility(t *testing.T) {
-	handler, err := New(config.MCPConfig{
-		ServerName:     "test",
-		Version:        "0.0.1",
-		SessionTimeout: time.Minute,
-	}, nil, streamableTestToolSet(), nil)
-	if err != nil {
-		t.Fatalf("New() error = %v", err)
-	}
-	if handler == nil {
-		t.Fatal("New() returned nil handler")
-	}
-}
-
-func TestSSEListTools(t *testing.T) {
-	h, err := NewHandlers(config.MCPConfig{
-		ServerName:     "test",
-		Version:        "0.0.1",
-		SessionTimeout: time.Minute,
-		SSEPath:        "/sse",
-	}, nil, streamableTestToolSet(), nil)
-	if err != nil {
-		t.Fatalf("NewHandlers() error = %v", err)
-	}
-
-	srv := httptest.NewServer(h.SSE)
-	t.Cleanup(srv.Close)
-
-	client := mcp.NewClient(&mcp.Implementation{Name: "client", Version: "0.0.1"}, nil)
-	cs, err := client.Connect(context.Background(), &mcp.SSEClientTransport{Endpoint: srv.URL}, nil)
-	if err != nil {
-		t.Fatalf("connect SSE client: %v", err)
-	}
-	t.Cleanup(func() { _ = cs.Close() })
-
-	result, err := cs.ListTools(context.Background(), nil)
-	if err != nil {
-		t.Fatalf("ListTools() error = %v", err)
-	}
-	if len(result.Tools) == 0 {
-		t.Fatal("ListTools() returned no tools")
-	}
-}
-
-func TestSSEAndStreamableShareTools(t *testing.T) {
-	h, err := NewHandlers(config.MCPConfig{
-		ServerName:     "test",
-		Version:        "0.0.1",
-		SessionTimeout: time.Minute,
-		SSEPath:        "/sse",
-	}, nil, streamableTestToolSet(), nil)
-	if err != nil {
-		t.Fatalf("NewHandlers() error = %v", err)
-	}
-
-	sseSrv := httptest.NewServer(h.SSE)
-	t.Cleanup(sseSrv.Close)
-
-	streamSrv := httptest.NewServer(h.StreamableHTTP)
-	t.Cleanup(streamSrv.Close)
-
-	sseClient := mcp.NewClient(&mcp.Implementation{Name: "sse-client", Version: "0.0.1"}, nil)
-	sseSession, err := sseClient.Connect(context.Background(), &mcp.SSEClientTransport{Endpoint: sseSrv.URL}, nil)
-	if err != nil {
-		t.Fatalf("connect SSE client: %v", err)
-	}
-	t.Cleanup(func() { _ = sseSession.Close() })
-
-	streamClient := mcp.NewClient(&mcp.Implementation{Name: "stream-client", Version: "0.0.1"}, nil)
-	streamSession, err := streamClient.Connect(context.Background(), &mcp.StreamableClientTransport{Endpoint: streamSrv.URL}, nil)
-	if err != nil {
-		t.Fatalf("connect StreamableHTTP client: %v", err)
-	}
-	t.Cleanup(func() { _ = streamSession.Close() })
-
-	sseTools, err := sseSession.ListTools(context.Background(), nil)
-	if err != nil {
-		t.Fatalf("SSE ListTools() error = %v", err)
-	}
-	streamTools, err := streamSession.ListTools(context.Background(), nil)
-	if err != nil {
-		t.Fatalf("StreamableHTTP ListTools() error = %v", err)
-	}
-
-	if len(sseTools.Tools) != len(streamTools.Tools) {
-		t.Fatalf("SSE tool count = %d, StreamableHTTP tool count = %d, want equal", len(sseTools.Tools), len(streamTools.Tools))
-	}
-}

internal/mcpserver/streamable_integration_test.go

@@ -127,7 +127,11 @@ func streamableTestToolSet() ToolSet {
 		Backfill:      new(tools.BackfillTool),
 		Reparse:       new(tools.ReparseMetadataTool),
 		RetryMetadata: new(tools.RetryMetadataTool),
+		Household:     new(tools.HouseholdTool),
 		Maintenance:   new(tools.MaintenanceTool),
+		Calendar:      new(tools.CalendarTool),
+		Meals:         new(tools.MealsTool),
+		CRM:           new(tools.CRMTool),
 		Skills:        new(tools.SkillsTool),
 	}
 }

internal/store/thoughts.go

@@ -58,12 +58,6 @@ func (db *DB) InsertThought(ctx context.Context, thought thoughttypes.Thought, e
 		return thoughttypes.Thought{}, fmt.Errorf("commit thought insert: %w", err)
 	}
 
-	if len(thought.Embedding) > 0 {
-		created.EmbeddingStatus = "done"
-	} else {
-		created.EmbeddingStatus = "pending"
-	}
-
 	return created, nil
 }
 

internal/store/tool_annotations.go

@@ -1,38 +0,0 @@
-package store
-
-import (
-	"context"
-	"fmt"
-)
-
-func (db *DB) UpsertToolAnnotation(ctx context.Context, toolName, notes string) error {
-	_, err := db.pool.Exec(ctx, `
-		insert into tool_annotations (tool_name, notes)
-		values ($1, $2)
-		on conflict (tool_name) do update
-		  set notes      = excluded.notes,
-		      updated_at = now()
-	`, toolName, notes)
-	if err != nil {
-		return fmt.Errorf("upsert tool annotation: %w", err)
-	}
-	return nil
-}
-
-func (db *DB) GetToolAnnotations(ctx context.Context) (map[string]string, error) {
-	rows, err := db.pool.Query(ctx, `select tool_name, notes from tool_annotations`)
-	if err != nil {
-		return nil, fmt.Errorf("get tool annotations: %w", err)
-	}
-	defer rows.Close()
-
-	annotations := make(map[string]string)
-	for rows.Next() {
-		var toolName, notes string
-		if err := rows.Scan(&toolName, &notes); err != nil {
-			return nil, fmt.Errorf("scan tool annotation: %w", err)
-		}
-		annotations[toolName] = notes
-	}
-	return annotations, rows.Err()
-}

internal/tools/backfill.go

@@ -51,30 +51,6 @@ func NewBackfillTool(db *store.DB, provider ai.Provider, sessions *session.Activ
 	return &BackfillTool{store: db, provider: provider, sessions: sessions, logger: logger}
 }
 
-// QueueThought queues a single thought for background embedding generation.
-// It is used by capture when the embedding provider is temporarily unavailable.
-func (t *BackfillTool) QueueThought(ctx context.Context, id uuid.UUID, content string) {
-	go func() {
-		vec, err := t.provider.Embed(ctx, content)
-		if err != nil {
-			t.logger.Warn("background embedding retry failed",
-				slog.String("thought_id", id.String()),
-				slog.String("error", err.Error()),
-			)
-			return
-		}
-		model := t.provider.EmbeddingModel()
-		if err := t.store.UpsertEmbedding(ctx, id, model, vec); err != nil {
-			t.logger.Warn("background embedding upsert failed",
-				slog.String("thought_id", id.String()),
-				slog.String("error", err.Error()),
-			)
-			return
-		}
-		t.logger.Info("background embedding retry succeeded", slog.String("thought_id", id.String()))
-	}()
-}
-
 func (t *BackfillTool) Handle(ctx context.Context, req *mcp.CallToolRequest, in BackfillInput) (*mcp.CallToolResult, BackfillOutput, error) {
 	limit := in.Limit
 	if limit <= 0 {

internal/tools/capture.go

@@ -6,8 +6,8 @@ import (
 	"strings"
 	"time"
 
-	"github.com/google/uuid"
 	"github.com/modelcontextprotocol/go-sdk/mcp"
+	"golang.org/x/sync/errgroup"
 
 	"git.warky.dev/wdevs/amcs/internal/ai"
 	"git.warky.dev/wdevs/amcs/internal/config"
@@ -17,11 +17,6 @@ import (
 	thoughttypes "git.warky.dev/wdevs/amcs/internal/types"
 )
 
-// EmbeddingQueuer queues a thought for background embedding generation.
-type EmbeddingQueuer interface {
-	QueueThought(ctx context.Context, id uuid.UUID, content string)
-}
-
 type CaptureTool struct {
 	store           *store.DB
 	provider        ai.Provider
@@ -29,7 +24,6 @@ type CaptureTool struct {
 	sessions        *session.ActiveProjects
 	metadataTimeout time.Duration
 	retryer         *MetadataRetryer
-	embedRetryer    EmbeddingQueuer
 	log             *slog.Logger
 }
 
@@ -42,8 +36,8 @@ type CaptureOutput struct {
 	Thought thoughttypes.Thought `json:"thought"`
 }
 
-func NewCaptureTool(db *store.DB, provider ai.Provider, capture config.CaptureConfig, metadataTimeout time.Duration, sessions *session.ActiveProjects, retryer *MetadataRetryer, embedRetryer EmbeddingQueuer, log *slog.Logger) *CaptureTool {
+func NewCaptureTool(db *store.DB, provider ai.Provider, capture config.CaptureConfig, metadataTimeout time.Duration, sessions *session.ActiveProjects, retryer *MetadataRetryer, log *slog.Logger) *CaptureTool {
-	return &CaptureTool{store: db, provider: provider, capture: capture, sessions: sessions, metadataTimeout: metadataTimeout, retryer: retryer, embedRetryer: embedRetryer, log: log}
+	return &CaptureTool{store: db, provider: provider, capture: capture, sessions: sessions, metadataTimeout: metadataTimeout, retryer: retryer, log: log}
 }
 
 func (t *CaptureTool) Handle(ctx context.Context, req *mcp.CallToolRequest, in CaptureInput) (*mcp.CallToolResult, CaptureOutput, error) {
@@ -57,10 +51,46 @@ func (t *CaptureTool) Handle(ctx context.Context, req *mcp.CallToolRequest, in C
 		return nil, CaptureOutput{}, err
 	}
 
+	var embedding []float32
 	rawMetadata := metadata.Fallback(t.capture)
+	metadataNeedsRetry := false
+
+	group, groupCtx := errgroup.WithContext(ctx)
+	group.Go(func() error {
+		vector, err := t.provider.Embed(groupCtx, content)
+		if err != nil {
+			return err
+		}
+		embedding = vector
+		return nil
+	})
+	group.Go(func() error {
+		metaCtx := groupCtx
+		attemptedAt := time.Now().UTC()
+		if t.metadataTimeout > 0 {
+			var cancel context.CancelFunc
+			metaCtx, cancel = context.WithTimeout(groupCtx, t.metadataTimeout)
+			defer cancel()
+		}
+		extracted, err := t.provider.ExtractMetadata(metaCtx, content)
+		if err != nil {
+			t.log.Warn("metadata extraction failed, using fallback", slog.String("provider", t.provider.Name()), slog.String("error", err.Error()))
+			rawMetadata = metadata.MarkMetadataPending(rawMetadata, t.capture, attemptedAt, err)
+			metadataNeedsRetry = true
+			return nil
+		}
+		rawMetadata = metadata.MarkMetadataComplete(extracted, t.capture, attemptedAt)
+		return nil
+	})
+
+	if err := group.Wait(); err != nil {
+		return nil, CaptureOutput{}, err
+	}
+
 	thought := thoughttypes.Thought{
 		Content:   content,
-		Metadata: rawMetadata,
+		Embedding: embedding,
+		Metadata:  metadata.Normalize(metadata.SanitizeExtracted(rawMetadata), t.capture),
 	}
 	if project != nil {
 		thought.ProjectID = &project.ID
@@ -73,57 +103,9 @@ func (t *CaptureTool) Handle(ctx context.Context, req *mcp.CallToolRequest, in C
 	if project != nil {
 		_ = t.store.TouchProject(ctx, project.ID)
 	}
-
+	if metadataNeedsRetry && t.retryer != nil {
-	if t.retryer != nil || t.embedRetryer != nil {
+		t.retryer.QueueThought(created.ID)
-		t.launchEnrichment(created.ID, content)
 	}
 
 	return nil, CaptureOutput{Thought: created}, nil
 }
-
-func (t *CaptureTool) launchEnrichment(id uuid.UUID, content string) {
-	go func() {
-		ctx, cancel := context.WithTimeout(context.Background(), 2*time.Minute)
-		defer cancel()
-
-		if t.retryer != nil {
-			attemptedAt := time.Now().UTC()
-			rawMetadata := metadata.Fallback(t.capture)
-			extracted, err := t.provider.ExtractMetadata(ctx, content)
-			if err != nil {
-				failed := metadata.MarkMetadataFailed(rawMetadata, t.capture, attemptedAt, err)
-				if _, updateErr := t.store.UpdateThoughtMetadata(ctx, id, failed); updateErr != nil {
-					t.log.Warn("deferred metadata failure could not be persisted",
-						slog.String("thought_id", id.String()),
-						slog.String("error", updateErr.Error()),
-					)
-				}
-				t.log.Warn("deferred metadata extraction failed",
-					slog.String("thought_id", id.String()),
-					slog.String("provider", t.provider.Name()),
-					slog.String("error", err.Error()),
-				)
-				t.retryer.QueueThought(id)
-			} else {
-				completed := metadata.MarkMetadataComplete(extracted, t.capture, attemptedAt)
-				if _, updateErr := t.store.UpdateThoughtMetadata(ctx, id, completed); updateErr != nil {
-					t.log.Warn("deferred metadata completion could not be persisted",
-						slog.String("thought_id", id.String()),
-						slog.String("error", updateErr.Error()),
-					)
-				}
-			}
-		}
-
-		if t.embedRetryer != nil {
-			if _, err := t.provider.Embed(ctx, content); err != nil {
-				t.log.Warn("deferred embedding failed",
-					slog.String("thought_id", id.String()),
-					slog.String("provider", t.provider.Name()),
-					slog.String("error", err.Error()),
-				)
-			}
-			t.embedRetryer.QueueThought(ctx, id, content)
-		}
-	}()
-}

internal/tools/describe.go

@@ -1,89 +0,0 @@
-package tools
-
-import (
-	"context"
-	"strings"
-
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-
-	"git.warky.dev/wdevs/amcs/internal/store"
-)
-
-// ToolEntry describes a single registered MCP tool.
-type ToolEntry struct {
-	Name        string
-	Description string
-	Category    string
-}
-
-// DescribeTool implements the describe_tools and annotate_tool MCP tools.
-type DescribeTool struct {
-	store   *store.DB
-	catalog []ToolEntry
-}
-
-func NewDescribeTool(db *store.DB, catalog []ToolEntry) *DescribeTool {
-	return &DescribeTool{store: db, catalog: catalog}
-}
-
-// describe_tools
-
-type DescribeToolsInput struct {
-	Category string `json:"category,omitempty" jsonschema:"filter results to a single category (e.g. thoughts, projects, files, skills, chat, meta)"`
-}
-
-type AnnotatedToolEntry struct {
-	Name        string `json:"name"`
-	Description string `json:"description"`
-	Category    string `json:"category"`
-	Notes       string `json:"notes,omitempty"`
-}
-
-type DescribeToolsOutput struct {
-	Tools []AnnotatedToolEntry `json:"tools"`
-}
-
-func (t *DescribeTool) Describe(ctx context.Context, _ *mcp.CallToolRequest, in DescribeToolsInput) (*mcp.CallToolResult, DescribeToolsOutput, error) {
-	annotations, err := t.store.GetToolAnnotations(ctx)
-	if err != nil {
-		return nil, DescribeToolsOutput{}, err
-	}
-
-	cat := strings.TrimSpace(strings.ToLower(in.Category))
-
-	entries := make([]AnnotatedToolEntry, 0, len(t.catalog))
-	for _, e := range t.catalog {
-		if cat != "" && e.Category != cat {
-			continue
-		}
-		entries = append(entries, AnnotatedToolEntry{
-			Name:        e.Name,
-			Description: e.Description,
-			Category:    e.Category,
-			Notes:       annotations[e.Name],
-		})
-	}
-
-	return nil, DescribeToolsOutput{Tools: entries}, nil
-}
-
-// annotate_tool
-
-type AnnotateToolInput struct {
-	ToolName string `json:"tool_name" jsonschema:"the exact name of the tool to annotate"`
-	Notes    string `json:"notes" jsonschema:"your usage notes, reminders, or gotchas for this tool; pass empty string to clear"`
-}
-
-type AnnotateToolOutput struct {
-	ToolName string `json:"tool_name"`
-}
-
-func (t *DescribeTool) Annotate(ctx context.Context, _ *mcp.CallToolRequest, in AnnotateToolInput) (*mcp.CallToolResult, AnnotateToolOutput, error) {
-	if strings.TrimSpace(in.ToolName) == "" {
-		return nil, AnnotateToolOutput{}, errRequiredField("tool_name")
-	}
-	if err := t.store.UpsertToolAnnotation(ctx, in.ToolName, in.Notes); err != nil {
-		return nil, AnnotateToolOutput{}, err
-	}
-	return nil, AnnotateToolOutput{ToolName: in.ToolName}, nil
-}

internal/tools/enrichment_retry.go

@@ -1,209 +0,0 @@
-package tools
-
-import (
-	"context"
-	"log/slog"
-	"sync"
-	"time"
-
-	"github.com/google/uuid"
-	"github.com/modelcontextprotocol/go-sdk/mcp"
-	"golang.org/x/sync/semaphore"
-
-	"git.warky.dev/wdevs/amcs/internal/ai"
-	"git.warky.dev/wdevs/amcs/internal/config"
-	"git.warky.dev/wdevs/amcs/internal/metadata"
-	"git.warky.dev/wdevs/amcs/internal/session"
-	"git.warky.dev/wdevs/amcs/internal/store"
-	thoughttypes "git.warky.dev/wdevs/amcs/internal/types"
-)
-
-const enrichmentRetryConcurrency = 4
-const enrichmentRetryMaxAttempts = 5
-
-var enrichmentRetryBackoff = []time.Duration{
-	30 * time.Second,
-	2 * time.Minute,
-	10 * time.Minute,
-	30 * time.Minute,
-	2 * time.Hour,
-}
-
-type EnrichmentRetryer struct {
-	backgroundCtx   context.Context
-	store           *store.DB
-	provider        ai.Provider
-	capture         config.CaptureConfig
-	sessions        *session.ActiveProjects
-	metadataTimeout time.Duration
-	logger          *slog.Logger
-}
-
-type RetryEnrichmentTool struct {
-	retryer *EnrichmentRetryer
-}
-
-type RetryEnrichmentInput struct {
-	Project         string `json:"project,omitempty" jsonschema:"optional project name or id to scope the retry"`
-	Limit           int    `json:"limit,omitempty" jsonschema:"maximum number of thoughts to process in one call; defaults to 100"`
-	IncludeArchived bool   `json:"include_archived,omitempty" jsonschema:"whether to include archived thoughts; defaults to false"`
-	OlderThanDays   int    `json:"older_than_days,omitempty" jsonschema:"only retry thoughts whose last metadata attempt was at least N days ago; 0 means no restriction"`
-	DryRun          bool   `json:"dry_run,omitempty" jsonschema:"report counts without retrying metadata extraction"`
-}
-
-type RetryEnrichmentFailure struct {
-	ID    string `json:"id"`
-	Error string `json:"error"`
-}
-
-type RetryEnrichmentOutput struct {
-	Scanned  int                      `json:"scanned"`
-	Retried  int                      `json:"retried"`
-	Updated  int                      `json:"updated"`
-	Skipped  int                      `json:"skipped"`
-	Failed   int                      `json:"failed"`
-	DryRun   bool                     `json:"dry_run"`
-	Failures []RetryEnrichmentFailure `json:"failures,omitempty"`
-}
-
-func NewEnrichmentRetryer(backgroundCtx context.Context, db *store.DB, provider ai.Provider, capture config.CaptureConfig, metadataTimeout time.Duration, sessions *session.ActiveProjects, logger *slog.Logger) *EnrichmentRetryer {
-	if backgroundCtx == nil {
-		backgroundCtx = context.Background()
-	}
-	return &EnrichmentRetryer{
-		backgroundCtx:   backgroundCtx,
-		store:           db,
-		provider:        provider,
-		capture:         capture,
-		sessions:        sessions,
-		metadataTimeout: metadataTimeout,
-		logger:          logger,
-	}
-}
-
-func NewRetryEnrichmentTool(retryer *EnrichmentRetryer) *RetryEnrichmentTool {
-	return &RetryEnrichmentTool{retryer: retryer}
-}
-
-func (t *RetryEnrichmentTool) Handle(ctx context.Context, req *mcp.CallToolRequest, in RetryEnrichmentInput) (*mcp.CallToolResult, RetryEnrichmentOutput, error) {
-	return t.retryer.Handle(ctx, req, in)
-}
-
-func (r *EnrichmentRetryer) QueueThought(id uuid.UUID) {
-	go func() {
-		if _, err := r.retryOne(r.backgroundCtx, id); err != nil {
-			r.logger.Warn("background metadata retry failed",
-				slog.String("thought_id", id.String()),
-				slog.String("error", err.Error()),
-			)
-		}
-	}()
-}
-
-func (r *EnrichmentRetryer) Handle(ctx context.Context, req *mcp.CallToolRequest, in RetryEnrichmentInput) (*mcp.CallToolResult, RetryEnrichmentOutput, error) {
-	limit := in.Limit
-	if limit <= 0 {
-		limit = 100
-	}
-
-	project, err := resolveProject(ctx, r.store, r.sessions, req, in.Project, false)
-	if err != nil {
-		return nil, RetryEnrichmentOutput{}, err
-	}
-
-	var projectID *uuid.UUID
-	if project != nil {
-		projectID = &project.ID
-	}
-
-	thoughts, err := r.store.ListThoughtsPendingMetadataRetry(ctx, limit, projectID, in.IncludeArchived, in.OlderThanDays)
-	if err != nil {
-		return nil, RetryEnrichmentOutput{}, err
-	}
-
-	out := RetryEnrichmentOutput{Scanned: len(thoughts), DryRun: in.DryRun}
-	if in.DryRun || len(thoughts) == 0 {
-		return nil, out, nil
-	}
-
-	sem := semaphore.NewWeighted(enrichmentRetryConcurrency)
-	var mu sync.Mutex
-	var wg sync.WaitGroup
-
-	for _, thought := range thoughts {
-		if ctx.Err() != nil {
-			break
-		}
-		if err := sem.Acquire(ctx, 1); err != nil {
-			break
-		}
-
-		wg.Add(1)
-		go func(thought thoughttypes.Thought) {
-			defer wg.Done()
-			defer sem.Release(1)
-
-			mu.Lock()
-			out.Retried++
-			mu.Unlock()
-
-			updated, err := r.retryOne(ctx, thought.ID)
-			if err != nil {
-				mu.Lock()
-				out.Failures = append(out.Failures, RetryEnrichmentFailure{ID: thought.ID.String(), Error: err.Error()})
-				mu.Unlock()
-				return
-			}
-			if updated {
-				mu.Lock()
-				out.Updated++
-				mu.Unlock()
-				return
-			}
-
-			mu.Lock()
-			out.Skipped++
-			mu.Unlock()
-		}(thought)
-	}
-
-	wg.Wait()
-	out.Failed = len(out.Failures)
-
-	return nil, out, nil
-}
-
-func (r *EnrichmentRetryer) retryOne(ctx context.Context, id uuid.UUID) (bool, error) {
-	thought, err := r.store.GetThought(ctx, id)
-	if err != nil {
-		return false, err
-	}
-	if thought.Metadata.MetadataStatus == metadata.MetadataStatusComplete {
-		return false, nil
-	}
-
-	attemptCtx := ctx
-	if r.metadataTimeout > 0 {
-		var cancel context.CancelFunc
-		attemptCtx, cancel = context.WithTimeout(ctx, r.metadataTimeout)
-		defer cancel()
-	}
-
-	attemptedAt := time.Now().UTC()
-	extracted, extractErr := r.provider.ExtractMetadata(attemptCtx, thought.Content)
-	if extractErr != nil {
-		failedMetadata := metadata.MarkMetadataFailed(thought.Metadata, r.capture, attemptedAt, extractErr)
-		if _, updateErr := r.store.UpdateThoughtMetadata(ctx, thought.ID, failedMetadata); updateErr != nil {
-			return false, updateErr
-		}
-		return false, extractErr
-	}
-
-	completedMetadata := metadata.MarkMetadataComplete(metadata.SanitizeExtracted(extracted), r.capture, attemptedAt)
-	completedMetadata.Attachments = thought.Metadata.Attachments
-	if _, updateErr := r.store.UpdateThoughtMetadata(ctx, thought.ID, completedMetadata); updateErr != nil {
-		return false, updateErr
-	}
-
-	return true, nil
-}

internal/tools/helpers.go

@@ -87,7 +87,6 @@ func resolveProject(ctx context.Context, db *store.DB, sessions *session.ActiveP
 					Type:    mcperrors.TypeProjectNotFound,
 					Field:   "project",
 					Project: projectRef,
-					Hint:    fmt.Sprintf("project %q does not exist yet; call create_project with name=%q first, then retry", projectRef, projectRef),
 				},
 			)
 		}

internal/tools/metadata_retry.go

@@ -28,42 +28,12 @@ type MetadataRetryer struct {
 	sessions        *session.ActiveProjects
 	metadataTimeout time.Duration
 	logger          *slog.Logger
-	lock            *RetryLocker
 }
 
 type RetryMetadataTool struct {
 	retryer *MetadataRetryer
 }
 
-type RetryLocker struct {
-	mu    sync.Mutex
-	locks map[uuid.UUID]time.Time
-}
-
-func NewRetryLocker() *RetryLocker {
-	return &RetryLocker{locks: map[uuid.UUID]time.Time{}}
-}
-
-func (l *RetryLocker) Acquire(id uuid.UUID, ttl time.Duration) bool {
-	l.mu.Lock()
-	defer l.mu.Unlock()
-	if l.locks == nil {
-		l.locks = map[uuid.UUID]time.Time{}
-	}
-	now := time.Now()
-	if exp, ok := l.locks[id]; ok && exp.After(now) {
-		return false
-	}
-	l.locks[id] = now.Add(ttl)
-	return true
-}
-
-func (l *RetryLocker) Release(id uuid.UUID) {
-	l.mu.Lock()
-	defer l.mu.Unlock()
-	delete(l.locks, id)
-}
-
 type RetryMetadataInput struct {
 	Project         string `json:"project,omitempty" jsonschema:"optional project name or id to scope the retry"`
 	Limit           int    `json:"limit,omitempty" jsonschema:"maximum number of thoughts to process in one call; defaults to 100"`
@@ -99,7 +69,6 @@ func NewMetadataRetryer(backgroundCtx context.Context, db *store.DB, provider ai
 		sessions:        sessions,
 		metadataTimeout: metadataTimeout,
 		logger:          logger,
-		lock:            NewRetryLocker(),
 	}
 }
 
@@ -113,10 +82,6 @@ func (t *RetryMetadataTool) Handle(ctx context.Context, req *mcp.CallToolRequest
 
 func (r *MetadataRetryer) QueueThought(id uuid.UUID) {
 	go func() {
-		if !r.lock.Acquire(id, 15*time.Minute) {
-			return
-		}
-		defer r.lock.Release(id)
 		if _, err := r.retryOne(r.backgroundCtx, id); err != nil {
 			r.logger.Warn("background metadata retry failed", slog.String("thought_id", id.String()), slog.String("error", err.Error()))
 		}
@@ -173,14 +138,7 @@ func (r *MetadataRetryer) Handle(ctx context.Context, req *mcp.CallToolRequest,
 			out.Retried++
 			mu.Unlock()
 
-			if !r.lock.Acquire(thought.ID, 15*time.Minute) {
-				mu.Lock()
-				out.Skipped++
-				mu.Unlock()
-				return
-			}
 			updated, err := r.retryOne(ctx, thought.ID)
-			r.lock.Release(thought.ID)
 			if err != nil {
 				mu.Lock()
 				out.Failures = append(out.Failures, RetryMetadataFailure{ID: thought.ID.String(), Error: err.Error()})

internal/types/thought.go

@@ -55,7 +55,6 @@ type Thought struct {
 	ID         uuid.UUID       `json:"id"`
 	Content    string          `json:"content"`
 	Embedding  []float32       `json:"embedding,omitempty"`
-	EmbeddingStatus string          `json:"embedding_status,omitempty"`
 	Metadata   ThoughtMetadata `json:"metadata"`
 	ProjectID  *uuid.UUID      `json:"project_id,omitempty"`
 	ArchivedAt *time.Time      `json:"archived_at,omitempty"`

llm/memory.md

@@ -2,12 +2,6 @@
 
 AMCS (Avalon Memory Crystal Server) is an MCP server for capturing and retrieving thoughts, memory, and project context. It is backed by Postgres with pgvector for semantic search.
 
-`amcs-cli` is a pre-built CLI that connects to the AMCS MCP server so agents do not need to implement their own HTTP MCP client. Download it from https://git.warky.dev/wdevs/amcs/releases
-
-The key command is `amcs-cli stdio`, which bridges the remote HTTP MCP server to a local stdio MCP transport. Register it as a stdio MCP server in your agent config and all AMCS tools are available immediately without any custom client code.
-
-Configure with `~/.config/amcs/config.yaml` (`server`, `token`), env vars `AMCS_URL` / `AMCS_TOKEN`, or `--server` / `--token` flags.
-
 You have access to an MCP memory server named AMCS.
 
 Use AMCS as memory with two scopes:
@@ -24,30 +18,15 @@ Use AMCS as memory with two scopes:
 6. If no strong project match exists, you may use global notebook memory with no project.
 7. If multiple projects plausibly match, ask the user before reading or writing project memory.
 
-## Session Startup
-
-At the very start of any session with AMCS:
-
-1. Call `describe_tools` to get the full list of available tools with their categories and any notes you have previously annotated. Read the notes before using a tool — they contain accumulated gotchas, workflow patterns, and field-ordering requirements you have recorded from prior sessions.
-
 ## Project Session Startup
 
-After setting the active project:
+At the start of every project session, after setting the active project:
 
 1. Call `list_project_skills` to load any saved agent behaviour instructions for the project.
 2. Call `list_project_guardrails` to load any saved agent constraints for the project.
 3. Apply all returned skills and guardrails immediately and for the duration of the session.
 4. Only generate or define new skills and guardrails if none are returned. If you do create new ones, store them with `add_skill` or `add_guardrail` and link them to the project with `add_project_skill` or `add_project_guardrail` so they persist for future sessions.
 
-## Project Does Not Exist Yet
-
-If any tool returns a `project_not_found` error:
-
-1. Call `create_project` with the same name you tried to use.
-2. Immediately retry the original tool call with that project.
-
-Do not abandon the project scope or retry without a project. The project simply needs to be created first.
-
 ## Project Memory Rules
 
 - Use project memory for code decisions, architecture, TODOs, debugging findings, and context specific to the current repo or workstream.
@@ -80,16 +59,6 @@ Do not abandon the project scope or retry without a project. The project simply
 - Do not base64-encode a file to pass it to `save_file` if an `amcs://files/{id}` URI is already available from a prior `upload_file` or HTTP upload.
 - When saving, choose the narrowest correct scope: project if project-specific, global if not.
 
-## Tool Annotations
-
-As you learn non-obvious behaviours, gotchas, or workflow patterns for individual tools, persist them with `annotate_tool`:
-
-```json
-{ "tool_name": "capture_thought", "notes": "Always pass project explicitly — session state is unreliable in this client." }
-```
-
-Notes are returned by `describe_tools` in future sessions. Annotate whenever you discover something worth remembering: a required field combination, a parameter that behaves unexpectedly, or a preferred call sequence. Pass an empty string to clear a note.
-
 ## Skills and Guardrails
 
 - **Skills** are reusable agent behaviour instructions (e.g. output formatting rules, reasoning strategies, workflow conventions).
@@ -100,4 +69,4 @@ Notes are returned by `describe_tools` in future sessions. Annotate whenever you
 
 ## Short Operational Form
 
-At the start of every session, call `describe_tools` to read the full tool list and any accumulated usage notes. Use AMCS memory in project scope when the current work matches a known project; if no clear project matches, global notebook memory is allowed for non-project-specific information. At the start of every project session call `list_project_skills` and `list_project_guardrails` and apply what is returned; only create new skills or guardrails if none exist. If your MCP client does not preserve sessions across calls, pass `project` explicitly instead of relying on `set_active_project`. Store durable notes with `capture_thought`. For binary files or files larger than 10 MB, call `upload_file` with `content_path` to stage the file and get an `amcs://files/{id}` URI, then pass that URI to `save_file` as `content_uri` to link it to a thought. For small files, use `save_file` or `upload_file` with `content_base64` directly. Browse stored files with `list_files`, and load them with `load_file` only when their contents are needed. Stored files can also be read as raw binary via MCP resources at `amcs://files/{id}`. Never store project-specific memory globally when a matching project exists, and never store memory in the wrong project. If project matching is ambiguous, ask the user. If a tool returns `project_not_found`, call `create_project` with that name and retry — never drop the project scope. Whenever you discover a non-obvious tool behaviour, gotcha, or workflow pattern, record it with `annotate_tool` so future sessions benefit.
+Use AMCS memory in project scope when the current work matches a known project. If no clear project matches, global notebook memory is allowed for non-project-specific information. At the start of every project session call `list_project_skills` and `list_project_guardrails` and apply what is returned; only create new skills or guardrails if none exist. If your MCP client does not preserve sessions across calls, pass `project` explicitly instead of relying on `set_active_project`. Store durable notes with `capture_thought`. For binary files or files larger than 10 MB, call `upload_file` with `content_path` to stage the file and get an `amcs://files/{id}` URI, then pass that URI to `save_file` as `content_uri` to link it to a thought. For small files, use `save_file` or `upload_file` with `content_base64` directly. Browse stored files with `list_files`, and load them with `load_file` only when their contents are needed. Stored files can also be read as raw binary via MCP resources at `amcs://files/{id}`. Never store project-specific memory globally when a matching project exists, and never store memory in the wrong project. If project matching is ambiguous, ask the user.

llm/options_to_openclaw.md

@@ -1,826 +0,0 @@
-# AMCS → OpenClaw Alternative: Gap Analysis & Roadmap
-
-## Context
-
-AMCS is a **passive** MCP memory server. OpenClaw's key differentiator is that it's an **always-on autonomous agent** — it proactively acts, monitors, and learns without human prompting. AMCS has the data model and search foundation; it's missing the execution engine and channel integrations that make OpenClaw compelling.
-
-OpenClaw's 3 pillars AMCS lacks:
-1. **Autonomous heartbeat** — scheduled jobs that run without user prompts
-2. **Channel integrations** — 25+ messaging platforms (Telegram, Slack, Discord, email, etc.)
-3. **Self-improving memory** — knowledge graph distillation, daily notes, living summary (MEMORY.md)
-
----
-
-## Phase 1: Autonomous Heartbeat Engine (Critical — unlocks everything else)
-
-### 1a. Add `Complete()` to AI Provider
-
-The current `Provider` interface in `internal/ai/provider.go` only supports `Summarize(ctx, systemPrompt, userPrompt)`. An autonomous agent needs a stateful multi-turn call with tool awareness.
-
-**Extend the interface:**
-
-```go
-// internal/ai/provider.go
-
-type CompletionRole string
-
-const (
-    RoleSystem    CompletionRole = "system"
-    RoleUser      CompletionRole = "user"
-    RoleAssistant CompletionRole = "assistant"
-)
-
-type CompletionMessage struct {
-    Role    CompletionRole `json:"role"`
-    Content string         `json:"content"`
-}
-
-type CompletionResult struct {
-    Content    string `json:"content"`
-    StopReason string `json:"stop_reason"` // "stop" | "length" | "error"
-    Model      string `json:"model"`
-}
-
-type Provider interface {
-    Embed(ctx context.Context, input string) ([]float32, error)
-    ExtractMetadata(ctx context.Context, input string) (thoughttypes.ThoughtMetadata, error)
-    Summarize(ctx context.Context, systemPrompt, userPrompt string) (string, error)
-    Complete(ctx context.Context, messages []CompletionMessage) (CompletionResult, error)
-    Name() string
-    EmbeddingModel() string
-}
-```
-
-**Implement in `internal/ai/compat/client.go`:**
-
-`Complete` is a simplification of the existing `extractMetadataWithModel` path — same OpenAI-compatible `/chat/completions` endpoint, same auth headers, no JSON schema coercion. Add a `chatCompletionsRequest` type (reuse or extend the existing unexported struct) and a `Complete` method on `*Client` that:
-1. Builds the request body from `[]CompletionMessage`
-2. POSTs to `c.baseURL + "/chat/completions"` with `c.metadataModel`
-3. Reads the first choice's `message.content`
-4. Returns `CompletionResult{Content, StopReason, Model}`
-
-Error handling mirrors the metadata path: on HTTP 429/503 mark the model unhealthy (`c.modelHealth`), return a wrapped error. No fallback model chain needed for agent calls — callers should retry on next heartbeat tick.
-
----
-
-### 1b. Heartbeat Engine Package
-
-**New package: `internal/agent/`**
-
-#### `internal/agent/job.go`
-
-```go
-package agent
-
-import (
-    "context"
-    "time"
-)
-
-// Job is a single scheduled unit of autonomous work.
-type Job interface {
-    Name() string
-    Interval() time.Duration
-    Run(ctx context.Context) error
-}
-```
-
-#### `internal/agent/engine.go`
-
-The engine manages a set of jobs and fires each on its own ticker. It mirrors the pattern already used for `runBackfillPass` and `runMetadataRetryPass` in `internal/app/app.go`, but generalises it.
-
-```go
-package agent
-
-import (
-    "context"
-    "log/slog"
-    "sync"
-    "time"
-)
-
-type Engine struct {
-    jobs   []Job
-    store  JobStore  // persists agent_job_runs rows
-    logger *slog.Logger
-}
-
-func NewEngine(store JobStore, logger *slog.Logger, jobs ...Job) *Engine {
-    return &Engine{jobs: jobs, store: store, logger: logger}
-}
-
-// Run starts all job tickers and blocks until ctx is cancelled.
-func (e *Engine) Run(ctx context.Context) {
-    var wg sync.WaitGroup
-    for _, job := range e.jobs {
-        wg.Add(1)
-        go func(j Job) {
-            defer wg.Done()
-            e.runLoop(ctx, j)
-        }(job)
-    }
-    wg.Wait()
-}
-
-func (e *Engine) runLoop(ctx context.Context, j Job) {
-    ticker := time.NewTicker(j.Interval())
-    defer ticker.Stop()
-    for {
-        select {
-        case <-ctx.Done():
-            return
-        case <-ticker.C:
-            e.runOnce(ctx, j)
-        }
-    }
-}
-
-func (e *Engine) runOnce(ctx context.Context, j Job) {
-    runID, err := e.store.StartRun(ctx, j.Name())
-    if err != nil {
-        e.logger.Error("agent: failed to start job run record",
-            slog.String("job", j.Name()), slog.String("error", err.Error()))
-        return
-    }
-    if err := j.Run(ctx); err != nil {
-        e.logger.Error("agent: job failed",
-            slog.String("job", j.Name()), slog.String("error", err.Error()))
-        _ = e.store.FinishRun(ctx, runID, "failed", "", err.Error())
-        return
-    }
-    _ = e.store.FinishRun(ctx, runID, "ok", "", "")
-    e.logger.Info("agent: job complete", slog.String("job", j.Name()))
-}
-```
-
-**Deduplication / double-run prevention:** `StartRun` should check for an existing `running` row younger than `2 * j.Interval()` and return `ErrAlreadyRunning` — the caller skips that tick.
-
-#### `internal/agent/distill.go`
-
-```go
-// DistillJob clusters semantically related thoughts and promotes
-// durable insights into knowledge nodes.
-type DistillJob struct {
-    store     store.ThoughtQuerier
-    provider  ai.Provider
-    cfg       AgentDistillConfig
-    projectID *uuid.UUID // nil = all projects
-}
-
-func (j *DistillJob) Name() string             { return "distill" }
-func (j *DistillJob) Interval() time.Duration  { return j.cfg.Interval }
-
-func (j *DistillJob) Run(ctx context.Context) error {
-    // 1. Fetch recent thoughts not yet distilled (metadata.distilled != true)
-    //    using store.ListThoughts with filter Days = cfg.MinAgeHours/24
-    // 2. Group into semantic clusters via SearchSimilarThoughts
-    // 3. For each cluster > MinClusterSize:
-    //    a. Call provider.Summarize with insight extraction prompt
-    //    b. InsertThought with type="insight", metadata.knowledge_node=true
-    //    c. InsertLink from each cluster member to the insight, relation="distilled_from"
-    //    d. UpdateThought on each source to set metadata.distilled=true
-    // 4. Return nil; partial failures are logged but do not abort the run
-}
-```
-
-Prompt used in step 3a:
-```
-System: You extract durable knowledge from a cluster of related notes.
-        Return a single paragraph (2-5 sentences) capturing the core insight.
-        Do not reference the notes themselves. Write in third person.
-User: [concatenated thought content, newest first, max 4000 tokens]
-```
-
-#### `internal/agent/daily_notes.go`
-
-Runs at a configured hour each day (checked by comparing `time.Now().Hour()` against `cfg.Hour` inside the loop — skip if already ran today by querying `agent_job_runs` for a successful `daily_notes` run with `started_at >= today midnight`).
-
-Collects:
-- Thoughts created today (`store.ListThoughts` with `Days=1`)
-- CRM interactions logged today
-- Calendar activities for today
-- Maintenance logs from today
-
-Formats into a structured markdown string and calls `store.InsertThought` with `type=daily_note`.
-
-#### `internal/agent/living_summary.go`
-
-Regenerates `MEMORY.md` from the last N daily notes + all knowledge nodes. Calls `provider.Summarize` and upserts the result via `store.UpsertFile` using a fixed name `MEMORY.md` scoped to the project (or global if no project).
-
----
-
-### 1c. Config Structs
-
-Add to `internal/config/config.go`:
-
-```go
-type Config struct {
-    // ... existing fields ...
-    Agent    AgentConfig    `yaml:"agent"`
-    Channels ChannelsConfig `yaml:"channels"`
-    Shell    ShellConfig    `yaml:"shell"`
-}
-
-type AgentConfig struct {
-    Enabled       bool                  `yaml:"enabled"`
-    Distill       AgentDistillConfig    `yaml:"distill"`
-    DailyNotes    AgentDailyNotesConfig `yaml:"daily_notes"`
-    LivingSummary AgentLivingSummary    `yaml:"living_summary"`
-    Archival      AgentArchivalConfig   `yaml:"archival"`
-    Model         string                `yaml:"model"` // override for agent calls; falls back to AI.Metadata.Model
-}
-
-type AgentDistillConfig struct {
-    Enabled        bool          `yaml:"enabled"`
-    Interval       time.Duration `yaml:"interval"`        // default: 24h
-    BatchSize      int           `yaml:"batch_size"`      // thoughts per run; default: 50
-    MinClusterSize int           `yaml:"min_cluster_size"` // default: 3
-    MinAgeHours    int           `yaml:"min_age_hours"`   // ignore thoughts younger than this; default: 6
-}
-
-type AgentDailyNotesConfig struct {
-    Enabled bool `yaml:"enabled"`
-    Hour    int  `yaml:"hour"` // 0-23 UTC; default: 23
-}
-
-type AgentLivingSummary struct {
-    Enabled  bool          `yaml:"enabled"`
-    Interval time.Duration `yaml:"interval"`   // default: 24h
-    MaxDays  int           `yaml:"max_days"`   // daily notes lookback; default: 30
-}
-
-type AgentArchivalConfig struct {
-    Enabled           bool          `yaml:"enabled"`
-    Interval          time.Duration `yaml:"interval"`             // default: 168h (weekly)
-    ArchiveOlderThan  int           `yaml:"archive_older_than_days"` // default: 90
-}
-```
-
-**Full YAML reference (`configs/dev.yaml` additions):**
-
-```yaml
-agent:
-  enabled: false
-  model: ""                        # leave blank to reuse ai.metadata.model
-  distill:
-    enabled: false
-    interval: 24h
-    batch_size: 50
-    min_cluster_size: 3
-    min_age_hours: 6
-  daily_notes:
-    enabled: false
-    hour: 23                       # UTC hour to generate (0–23)
-  living_summary:
-    enabled: false
-    interval: 24h
-    max_days: 30
-  archival:
-    enabled: false
-    interval: 168h
-    archive_older_than_days: 90
-```
-
----
-
-### 1d. Wire into `internal/app/app.go`
-
-After the existing `MetadataRetry` goroutine block:
-
-```go
-if cfg.Agent.Enabled {
-    jobStore := store.NewJobStore(db)
-    var jobs []agent.Job
-    if cfg.Agent.Distill.Enabled {
-        jobs = append(jobs, agent.NewDistillJob(db, provider, cfg.Agent.Distill, nil))
-    }
-    if cfg.Agent.DailyNotes.Enabled {
-        jobs = append(jobs, agent.NewDailyNotesJob(db, provider, cfg.Agent.DailyNotes))
-    }
-    if cfg.Agent.LivingSummary.Enabled {
-        jobs = append(jobs, agent.NewLivingSummaryJob(db, provider, cfg.Agent.LivingSummary))
-    }
-    if cfg.Agent.Archival.Enabled {
-        jobs = append(jobs, agent.NewArchivalJob(db, cfg.Agent.Archival))
-    }
-    engine := agent.NewEngine(jobStore, logger, jobs...)
-    go engine.Run(ctx)
-}
-```
-
----
-
-### 1e. New MCP Tools — `internal/tools/agent.go`
-
-```go
-// list_agent_jobs
-// Returns all registered jobs with: name, interval, last_run (status, started_at, finished_at), next_run estimate.
-
-// trigger_agent_job
-// Input: { "job": "distill" }
-// Fires the job immediately in a goroutine; returns a run_id for polling.
-
-// get_agent_job_history
-// Input: { "job": "distill", "limit": 20 }
-// Returns rows from agent_job_runs ordered by started_at DESC.
-```
-
-Register in `internal/app/app.go` routes by adding `Agent tools.AgentTool` to `mcpserver.ToolSet` and wiring `tools.NewAgentTool(engine)`.
-
----
-
-### 1f. Migration — `migrations/021_agent_jobs.sql`
-
-```sql
-CREATE TABLE agent_job_runs (
-  id          uuid        PRIMARY KEY DEFAULT gen_random_uuid(),
-  job_name    text        NOT NULL,
-  started_at  timestamptz NOT NULL DEFAULT now(),
-  finished_at timestamptz,
-  status      text        NOT NULL DEFAULT 'running', -- running | ok | failed | skipped
-  output      text,
-  error       text,
-  metadata    jsonb       NOT NULL DEFAULT '{}'
-);
-
-CREATE INDEX idx_agent_job_runs_lookup
-  ON agent_job_runs (job_name, started_at DESC);
-```
-
-**`JobStore` interface (`internal/store/agent.go`):**
-
-```go
-type JobStore interface {
-    StartRun(ctx context.Context, jobName string) (uuid.UUID, error)
-    FinishRun(ctx context.Context, id uuid.UUID, status, output, errMsg string) error
-    LastRun(ctx context.Context, jobName string) (*AgentJobRun, error)
-    ListRuns(ctx context.Context, jobName string, limit int) ([]AgentJobRun, error)
-}
-```
-
----
-
-## Phase 2: Knowledge Graph Distillation
-
-Builds on Phase 1's distillation job. `thought_links` already exists with typed `relation` — the missing piece is a way to mark and query promoted knowledge nodes.
-
-### 2a. Extend `ThoughtMetadata`
-
-In `internal/types/thought.go`, add two fields to `ThoughtMetadata`:
-
-```go
-type ThoughtMetadata struct {
-    // ... existing fields ...
-    KnowledgeNode   bool `json:"knowledge_node,omitempty"`   // true = promoted insight
-    KnowledgeWeight int  `json:"knowledge_weight,omitempty"` // number of source thoughts that fed this node
-    Distilled       bool `json:"distilled,omitempty"`        // true = this thought has been processed by distill job
-}
-```
-
-These are stored in the existing `metadata jsonb` column — no schema migration needed.
-
-### 2b. Store Addition
-
-In `internal/store/thoughts.go` add:
-
-```go
-// ListKnowledgeNodes returns thoughts where metadata->>'knowledge_node' = 'true',
-// ordered by knowledge_weight DESC, then created_at DESC.
-func (db *DB) ListKnowledgeNodes(ctx context.Context, projectID *uuid.UUID, limit int) ([]types.Thought, error)
-```
-
-SQL:
-```sql
-SELECT id, content, metadata, project_id, archived_at, created_at, updated_at
-FROM thoughts
-WHERE (metadata->>'knowledge_node')::boolean = true
-  AND ($1::uuid IS NULL OR project_id = $1)
-  AND archived_at IS NULL
-ORDER BY (metadata->>'knowledge_weight')::int DESC, created_at DESC
-LIMIT $2
-```
-
-### 2c. New MCP Tools — `internal/tools/knowledge.go`
-
-```go
-// get_knowledge_graph
-// Input: { "project_id": "uuid|null", "limit": 50 }
-// Returns: { nodes: [Thought], edges: [ThoughtLink] }
-// Fetches ListKnowledgeNodes + their outgoing/incoming links via store.GetThoughtLinks.
-
-// distill_now
-// Input: { "project_id": "uuid|null", "batch_size": 20 }
-// Triggers the distillation job synchronously (for on-demand use); returns { insights_created: N }
-```
-
----
-
-## Phase 3: Channel Integrations — Telegram First
-
-### 3a. Channel Adapter Interface — `internal/channels/channel.go`
-
-```go
-package channels
-
-import (
-    "context"
-    "time"
-)
-
-type Attachment struct {
-    Name      string
-    MediaType string
-    Data      []byte
-}
-
-type InboundMessage struct {
-    ChannelID   string      // e.g. telegram chat ID as string
-    SenderID    string      // e.g. telegram user ID as string
-    SenderName  string      // display name
-    Text        string
-    Attachments []Attachment
-    Timestamp   time.Time
-    Raw         any         // original platform message for debug/logging
-}
-
-type Channel interface {
-    Name() string
-    Start(ctx context.Context, handler func(InboundMessage)) error
-    Send(ctx context.Context, channelID string, text string) error
-}
-```
-
-### 3b. Telegram Implementation — `internal/channels/telegram/bot.go`
-
-Uses `net/http` only (no external Telegram SDK). Long-polling loop:
-
-```go
-type Bot struct {
-    token      string
-    allowedIDs map[int64]struct{} // empty = all allowed
-    baseURL    string             // https://api.telegram.org/bot{token}
-    client     *http.Client
-    offset     int64
-    logger     *slog.Logger
-}
-
-func (b *Bot) Name() string { return "telegram" }
-
-func (b *Bot) Start(ctx context.Context, handler func(channels.InboundMessage)) error {
-    for {
-        updates, err := b.getUpdates(ctx, b.offset, 30 /*timeout seconds*/)
-        if err != nil {
-            if ctx.Err() != nil { return nil }
-            // transient error: log and back off 5s
-            time.Sleep(5 * time.Second)
-            continue
-        }
-        for _, u := range updates {
-            b.offset = u.UpdateID + 1
-            if u.Message == nil { continue }
-            if !b.isAllowed(u.Message.Chat.ID) { continue }
-            handler(b.toInbound(u.Message))
-        }
-    }
-}
-
-func (b *Bot) Send(ctx context.Context, channelID string, text string) error {
-    // POST /sendMessage with chat_id and text
-    // Splits messages > 4096 chars automatically
-}
-```
-
-**Error handling:**
-- HTTP 401 (bad token): return fatal error, engine stops channel
-- HTTP 429 (rate limit): respect `retry_after` from response body, sleep, retry
-- HTTP 5xx: exponential backoff (5s → 10s → 30s → 60s), max 3 retries then sleep 5 min
-
-### 3c. Channel Router — `internal/channels/router.go`
-
-```go
-type Router struct {
-    store    store.ContactQuerier
-    thoughts store.ThoughtInserter
-    provider ai.Provider
-    channels map[string]channels.Channel
-    cfg      config.ChannelsConfig
-    logger   *slog.Logger
-}
-
-func (r *Router) Handle(msg channels.InboundMessage) {
-    // 1. Resolve sender → CRM contact (by channel_identifiers->>'telegram' = senderID)
-    //    If not found: create a new professional_contact with the sender name + channel identifier
-    // 2. Capture message as thought:
-    //    content = msg.Text
-    //    metadata.source = "telegram"
-    //    metadata.type = "observation"
-    //    metadata.people = [senderName]
-    //    metadata (extra, stored in JSONB): channel="telegram", channel_id=msg.ChannelID, sender_id=msg.SenderID
-    // 3. If cfg.Telegram.Respond:
-    //    a. Load recent context via store.SearchSimilarThoughts(msg.Text, limit=10)
-    //    b. Build []CompletionMessage with system context + recent thoughts + user message
-    //    c. Call provider.Complete(ctx, messages)
-    //    d. Capture response as thought (type="assistant_response", source="telegram")
-    //    e. Send reply via channel.Send(ctx, msg.ChannelID, result.Content)
-    //    f. Save chat history via store.InsertChatHistory
-}
-```
-
-**Agent response system prompt (step 3b):**
-```
-You are a personal assistant with access to the user's memory.
-Relevant context from memory:
-{joined recent thought content}
-
-Respond concisely. If you cannot answer from memory, say so.
-```
-
-### 3d. Config — full YAML reference
-
-```yaml
-channels:
-  telegram:
-    enabled: false
-    bot_token: ""
-    allowed_chat_ids: []           # empty = all chats allowed
-    capture_all: true              # save every inbound message as a thought
-    respond: true                  # send LLM reply back to sender
-    response_model: ""             # blank = uses agent.model or ai.metadata.model
-    poll_timeout_seconds: 30       # Telegram long-poll timeout (max 60)
-    max_message_length: 4096       # split replies longer than this
-  discord:
-    enabled: false
-    bot_token: ""
-    guild_ids: []                  # empty = all guilds
-    capture_all: true
-    respond: true
-  slack:
-    enabled: false
-    bot_token: ""
-    app_token: ""                  # for socket mode
-    capture_all: true
-    respond: true
-  email:
-    enabled: false
-    imap_host: ""
-    imap_port: 993
-    smtp_host: ""
-    smtp_port: 587
-    username: ""
-    password: ""
-    poll_interval: 5m
-    capture_all: true
-    folders: ["INBOX"]
-```
-
-### 3e. Schema Migration — `migrations/022_channel_contacts.sql`
-
-```sql
--- Store per-channel identity handles on CRM contacts
-ALTER TABLE professional_contacts
-  ADD COLUMN IF NOT EXISTS channel_identifiers jsonb NOT NULL DEFAULT '{}';
-
--- e.g. {"telegram": "123456789", "discord": "user#1234", "slack": "U01234567"}
-CREATE INDEX idx_contacts_telegram_id
-  ON professional_contacts ((channel_identifiers->>'telegram'))
-  WHERE channel_identifiers->>'telegram' IS NOT NULL;
-```
-
-### 3f. New MCP Tools — `internal/tools/channels.go`
-
-```go
-// send_channel_message
-// Input: { "channel": "telegram", "channel_id": "123456789", "text": "Hello" }
-// Sends a message on the named channel. Returns { sent: true, channel: "telegram" }
-
-// list_channel_conversations
-// Input: { "channel": "telegram", "limit": 20, "days": 7 }
-// Lists chat histories filtered by channel metadata. Wraps store.ListChatHistories.
-
-// get_channel_status
-// Returns: [{ channel: "telegram", connected: true, uptime_seconds: 3600 }, ...]
-```
-
-### 3g. Future Channel Adapters
-
-Each is a new subdirectory implementing `channels.Channel`. No router or MCP tool changes needed.
-
-| Channel | Package | Approach |
-|---------|---------|----------|
-| Discord | `internal/channels/discord/` | Gateway WebSocket (discord.com/api/gateway); or use `discordgo` lib |
-| Slack | `internal/channels/slack/` | Socket Mode WebSocket (no public URL needed) |
-| Email (IMAP) | `internal/channels/email/` | IMAP IDLE or poll; SMTP for send |
-| Signal | `internal/channels/signal/` | Wrap `signal-cli` JSON-RPC subprocess |
-| WhatsApp | `internal/channels/whatsapp/` | Meta Cloud API webhook (requires public URL) |
-
----
-
-## Phase 4: Shell / Computer Access
-
-### 4a. Shell Tool — `internal/tools/shell.go`
-
-```go
-type ShellInput struct {
-    Command    string `json:"command"`
-    WorkingDir string `json:"working_dir,omitempty"` // override default; must be within allowed prefix
-    Timeout    string `json:"timeout,omitempty"`     // e.g. "30s"; overrides config default
-    CaptureAs  string `json:"capture_as,omitempty"`  // thought type for stored output; default "shell_output"
-    SaveOutput bool   `json:"save_output"`           // store stdout/stderr as a thought
-}
-
-type ShellOutput struct {
-    Stdout   string `json:"stdout"`
-    Stderr   string `json:"stderr"`
-    ExitCode int    `json:"exit_code"`
-    ThoughtID *uuid.UUID `json:"thought_id,omitempty"` // set if save_output=true
-}
-```
-
-**Execution model:**
-1. Validate `command` against `cfg.Shell.AllowedCommands` (if non-empty) and `cfg.Shell.BlockedCommands`
-2. `exec.CommandContext(ctx, "sh", "-c", command)` with `Dir` set to working dir
-3. Capture stdout + stderr into `bytes.Buffer`
-4. On timeout: kill process group (`syscall.Kill(-cmd.Process.Pid, syscall.SIGKILL)`), return exit code -1
-5. If `SaveOutput`: call `store.InsertThought` with content = truncated stdout (max 8KB) + stderr summary
-
-**Security controls:**
-
-```yaml
-shell:
-  enabled: false
-  working_dir: "/tmp/amcs-agent"  # all commands run here unless overridden
-  allowed_working_dirs:           # if set, working_dir overrides must be within one of these
-    - "/tmp/amcs-agent"
-    - "/home/user/projects"
-  timeout: 30s
-  max_output_bytes: 65536         # truncate captured output beyond this
-  allowed_commands: []            # empty = all; non-empty = exact binary name allowlist
-  blocked_commands:               # checked before allowed_commands
-    - "rm"
-    - "sudo"
-    - "su"
-    - "curl"
-    - "wget"
-  save_output_by_default: false
-```
-
-The tool is registered with `mcp.Tool.Annotations` `Destructive: true` so MCP clients prompt for confirmation.
-
-### 4b. File Bridge Tools
-
-Also in `internal/tools/shell.go`:
-
-```go
-// read_file_from_path
-// Input: { "path": "/abs/path/file.txt", "link_to_thought": "uuid|null" }
-// Reads file from server filesystem → stores as AMCS file via store.InsertFile
-// Returns: { file_id: "uuid", size_bytes: N, media_type: "text/plain" }
-
-// write_file_to_path
-// Input: { "file_id": "uuid", "path": "/abs/path/output.txt" }
-// Loads AMCS file → writes to filesystem path
-// Path must be within cfg.Shell.AllowedWorkingDirs if set
-```
-
----
-
-## Phase 5: Self-Improving Memory
-
-### 5a. Skill Discovery Job — `internal/agent/skill_discovery.go`
-
-Runs weekly. Algorithm:
-
-1. Load last 30 days of `chat_histories` via `store.ListChatHistories(days=30)`
-2. Extract assistant message patterns with `provider.Complete`:
-   ```
-   System: Identify reusable behavioural patterns or preferences visible in these conversations.
-           Return a JSON array of { "name": "...", "description": "...", "tags": [...] }.
-           Only include patterns that would be useful across future sessions.
-   User: [last N assistant + user messages, newest first]
-   ```
-3. For each discovered pattern, call `store.InsertSkill` with tag `auto-discovered` and the current date
-4. Link to all projects via `store.LinkSkillToProject`
-
-Deduplication: before inserting, call `store.SearchSkills(pattern.name)` — if similarity > 0.9, skip.
-
-### 5b. Thought Archival Job — `internal/agent/archival.go`
-
-```go
-func (j *ArchivalJob) Run(ctx context.Context) error {
-    // 1. ListThoughts older than cfg.ArchiveOlderThanDays with no knowledge_node link
-    //    SQL: thoughts where created_at < now() - interval '$N days'
-    //         AND metadata->>'knowledge_node' IS DISTINCT FROM 'true'
-    //         AND archived_at IS NULL
-    //         AND id NOT IN (SELECT thought_id FROM thought_links WHERE relation = 'distilled_from')
-    // 2. For each batch: store.ArchiveThought(ctx, id)
-    // 3. Log count
-}
-```
-
-Uses the existing `ArchiveThought` store method — no new SQL needed.
-
----
-
-## End-to-End Agent Loop Flow
-
-```
-Telegram message arrives
-        │
-        ▼
-channels/telegram/bot.go (long-poll goroutine)
-        │  InboundMessage{}
-        ▼
-channels/router.go Handle()
-        ├── Resolve sender → CRM contact (store.SearchContacts by channel_identifiers)
-        ├── store.InsertThought (source="telegram", type="observation")
-        ├── store.SearchSimilarThoughts (semantic context retrieval)
-        ├── ai.Provider.Complete (build messages → LLM call)
-        ├── store.InsertThought (source="telegram", type="assistant_response")
-        ├── store.InsertChatHistory (full turn saved)
-        └── channels.Channel.Send (reply dispatched to Telegram)
-
-Meanwhile, every 24h:
-agent/engine.go ticker fires DistillJob
-        ├── store.ListThoughts (recent, not yet distilled)
-        ├── store.SearchSimilarThoughts (cluster by semantic similarity)
-        ├── ai.Provider.Summarize (insight extraction prompt)
-        ├── store.InsertThought (type="insight", knowledge_node=true)
-        └── store.InsertLink (relation="distilled_from" for each source)
-
-After distill:
-agent/living_summary.go
-        ├── store.ListKnowledgeNodes
-        ├── store.ListThoughts (type="daily_note", last 30 days)
-        ├── ai.Provider.Summarize (MEMORY.md regeneration)
-        └── store.UpsertFile (name="MEMORY.md", linked to project)
-```
-
----
-
-## Error Handling & Retry Strategy
-
-| Scenario | Handling |
-|----------|----------|
-| LLM returns 429 | Mark model unhealthy in `modelHealth` map (existing pattern), return error, engine logs and skips tick |
-| LLM returns 5xx | Same as 429 |
-| Telegram 429 | Read `retry_after` from response, sleep exact duration, retry immediately |
-| Telegram 5xx | Exponential backoff: 5s → 10s → 30s → 60s, reset after success |
-| Telegram disconnects | Long-poll timeout naturally retries; context cancel exits cleanly |
-| Agent job panics | `engine.runOnce` wraps in `recover()`, logs stack trace, marks run `failed` |
-| Agent double-run | `store.StartRun` checks for `running` row < `2 * interval` old → returns `ErrAlreadyRunning`, tick skipped silently |
-| Shell command timeout | `exec.CommandContext` kills process group via SIGKILL, returns exit_code=-1 and partial output |
-| Distillation partial failure | Each cluster processed independently; failure of one cluster logged and skipped, others continue |
-
----
-
-## Critical Files
-
-| File | Change |
-|------|--------|
-| `internal/ai/provider.go` | Add `Complete()`, `CompletionMessage`, `CompletionResult` |
-| `internal/ai/compat/client.go` | Implement `Complete()` on `*Client` |
-| `internal/config/config.go` | Add `AgentConfig`, `ChannelsConfig`, `ShellConfig` |
-| `internal/types/thought.go` | Add `KnowledgeNode`, `KnowledgeWeight`, `Distilled` to `ThoughtMetadata` |
-| `internal/store/thoughts.go` | Add `ListKnowledgeNodes()` |
-| `internal/store/agent.go` | New: `JobStore` interface + implementation |
-| `internal/app/app.go` | Wire agent engine + channel router goroutines |
-| `internal/mcpserver/server.go` | Add `Agent`, `Knowledge`, `Channels`, `Shell` to `ToolSet` |
-| `internal/agent/` | New package: engine, job, distill, daily_notes, living_summary, archival, skill_discovery |
-| `internal/channels/` | New package: channel interface, router, telegram/ |
-| `internal/tools/agent.go` | New: list_agent_jobs, trigger_agent_job, get_agent_job_history |
-| `internal/tools/knowledge.go` | New: get_knowledge_graph, distill_now |
-| `internal/tools/channels.go` | New: send_channel_message, list_channel_conversations, get_channel_status |
-| `internal/tools/shell.go` | New: run_shell_command, read_file_from_path, write_file_to_path |
-| `migrations/021_agent_jobs.sql` | New table: agent_job_runs |
-| `migrations/022_channel_contacts.sql` | ALTER professional_contacts: add channel_identifiers jsonb |
-
----
-
-## Sequence / Parallelism
-
-```
-Phase 1 (Heartbeat Engine) ──► Phase 2 (Knowledge Graph)
-                           └──► Phase 5 (Self-Improving)
-
-Phase 3 (Telegram) ──► Phase 3g (Discord / Slack / Email)
-
-Phase 4 (Shell) [fully independent — no dependencies on other phases]
-```
-
-**Minimum viable OpenClaw competitor = Phase 1 + Phase 3** (autonomous scheduling + Telegram channel).
-
----
-
-## Verification
-
-| Phase | Test |
-|-------|------|
-| 1 — Heartbeat | Set `distill.interval: 1m` in dev config. Capture 5+ related thoughts. Wait 1 min. Query `thought_links` for `relation=distilled_from` rows. Check `agent_job_runs` has a `status=ok` row. |
-| 1 — Daily notes | Set `daily_notes.hour` to current UTC hour. Restart server. Within 1 min, `list_thoughts` should return a `type=daily_note` entry for today. |
-| 2 — Knowledge graph | Call `get_knowledge_graph` MCP tool. Verify `nodes` array contains `type=insight` thoughts with `knowledge_node=true`. Verify edges list `distilled_from` links. |
-| 3 — Telegram inbound | Send a message to the configured bot. Call `search_thoughts` with the message text — should appear with `source=telegram`. |
-| 3 — Telegram response | Send a question to the bot. Verify a reply arrives in Telegram. Call `list_chat_histories` — should contain the turn. |
-| 4 — Shell | Call `run_shell_command` with `{"command": "echo hello", "save_output": true}`. Verify `stdout=hello\n`, `exit_code=0`, and a new thought with `type=shell_output`. |
-| 4 — Blocked command | Call `run_shell_command` with `{"command": "sudo whoami"}`. Verify error returned without execution. |
-| 5 — Skill discovery | Run `trigger_agent_job` with `{"job": "skill_discovery"}`. Verify new rows in `agent_skills` with tag `auto-discovered`. |
-| Full loop | Send Telegram message → agent responds → distill job runs → knowledge node created from conversation → MEMORY.md regenerated with new insight. |

llm/todo.md

@@ -1,172 +1,450 @@
 # AMCS TODO
+## Auto Embedding Backfill Tool
 
-## Future Plugin: Lifestyle Tools (calendar, meals, household, CRM)
+## Objective
 
-The following tool groups have been removed from the core server and are candidates for a separate optional plugin or extension server. The store/tool implementations remain in the codebase but are no longer registered.
+Add an MCP tool that automatically backfills missing embeddings for existing thoughts so semantic search keeps working after:
 
-### calendar
+* embedding model changes
-- `add_family_member` — Add a family member to the household.
+* earlier capture or update failures
-- `list_family_members` — List all family members.
+* import or migration of raw thoughts without vectors
-- `add_activity` — Schedule a one-time or recurring family activity.
-- `get_week_schedule` — Get all activities scheduled for a given week.
-- `search_activities` — Search activities by title, type, or family member.
-- `add_important_date` — Track a birthday, anniversary, deadline, or other important date.
-- `get_upcoming_dates` — Get important dates coming up in the next N days.
 
-### meals
+The tool should be safe to run repeatedly, should not duplicate work, and should make it easy to restore semantic coverage without rewriting existing thoughts.
-- `add_recipe` — Save a recipe with ingredients and instructions.
-- `search_recipes` — Search recipes by name, cuisine, tags, or ingredient.
-- `update_recipe` — Update an existing recipe.
-- `create_meal_plan` — Set the weekly meal plan; replaces existing.
-- `get_meal_plan` — Get the meal plan for a given week.
-- `generate_shopping_list` — Generate shopping list from the weekly meal plan.
-
-### household
-- `add_household_item` — Store a household fact (paint, appliance, measurement, etc.).
-- `search_household_items` — Search household items by name, category, or location.
-- `get_household_item` — Retrieve a household item by id.
-- `add_vendor` — Add a service provider (plumber, electrician, landscaper, etc.).
-- `list_vendors` — List household service vendors, optionally filtered by service type.
-
-### crm
-- `add_professional_contact` — Add a professional contact to the CRM.
-- `search_contacts` — Search professional contacts by name, company, title, notes, or tags.
-- `log_interaction` — Log an interaction with a professional contact.
-- `get_contact_history` — Get full history (interactions and opportunities) for a contact.
-- `create_opportunity` — Create a deal, project, or opportunity linked to a contact.
-- `get_follow_ups_due` — List contacts with a follow-up date due within the next N days.
-- `link_thought_to_contact` — Append a stored thought to a contact's notes.
-
-**Implementation notes:**
-- Store implementations: `internal/tools/calendar.go`, `internal/tools/meals.go`, `internal/tools/household.go`, `internal/tools/crm.go`
-- DB store layers: `internal/store/calendar.go`, `internal/store/meals.go`, `internal/store/household.go`, `internal/store/crm.go`
-- Re-register via `mcpserver.ToolSet` fields: `Household`, `Calendar`, `Meals`, `CRM`
-- Re-add `registerHouseholdTools`, `registerCalendarTools`, `registerMealTools`, `registerCRMTools` to the register slice in `NewHandlers`
-- Add catalog entries back in `BuildToolCatalog`
-
----
-## Embedding Backfill and Text-Search Fallback Audit
-
-This file originally described the planned `backfill_embeddings` work and semantic-to-text fallback behavior. Most of that work is now implemented. This document now tracks what landed, what still needs verification, and what follow-up work remains.
-
-For current operator-facing behavior, prefer `README.md`.
 
 ---
 
-## Status summary
+## Desired outcome
 
-### Implemented
+After this work:
 
-The main work described in this file is already present in the repo:
+* raw thought text remains the source of truth
-
+* embeddings are treated as derived data per model
-- `backfill_embeddings` MCP tool exists
+* search continues to query only embeddings from the active embedding model
-- missing-embedding selection helpers exist in the store layer
+* when no embeddings exist for the active model and scope, search falls back to Postgres text search
-- embedding upsert helpers exist in the store layer
+* operators or MCP clients can trigger a backfill for the current model
-- semantic retrieval falls back to Postgres full-text search when the active model has no embeddings in scope
+* AMCS can optionally auto-run a limited backfill pass on startup or on a schedule later
-- fallback behavior is wired into the main query-driven tools
-- a full-text index migration exists
-- optional automatic backfill runner exists in config/startup flow
-- retry and reparse maintenance tooling also exists around metadata quality
-
-### Still worth checking or improving
-
-The broad feature is done, but some implementation-depth items are still worth tracking:
-
-- test coverage around fallback/backfill behavior
-- whether configured backfill batching is used consistently end-to-end
-- observability depth beyond logs
-- response visibility into which retrieval mode was used
 
 ---
 
-## What is already implemented
+## Why this is needed
 
-### Backfill tool
+Current search behavior is model-specific:
 
-Implemented:
+* query text is embedded with the configured provider model
+* results are filtered by `embeddings.model`
+* thoughts with no embedding for that model are invisible to semantic search
 
-- `backfill_embeddings`
+This means a model switch leaves old thoughts searchable only by listing and metadata filters until new embeddings are generated.
-- project scoping
-- archived-thought filtering
-- age filtering
-- dry-run mode
-- bounded concurrency
-- best-effort per-item failure handling
-- idempotent embedding upsert behavior
 
-### Search fallback
+To avoid that dead zone, AMCS should also support a lexical fallback path backed by native Postgres text-search indexing.
-
-Implemented:
-
-- full-text fallback when no embeddings exist for the active model in scope
-- fallback helper shared by query-based tools
-- full-text index migration on thought content
-
-### Tools using fallback
-
-Implemented fallback coverage for:
-
-- `search_thoughts`
-- `recall_context`
-- `get_project_context` when a query is provided
-- `summarize_thoughts` when a query is provided
-- semantic neighbors in `related_thoughts`
-
-### Optional automatic behavior
-
-Implemented:
-
-- config-gated startup backfill pass
-- config-gated periodic backfill loop
 
 ---
 
-## Remaining follow-ups
+## Tool proposal
 
-### 1. Expose retrieval mode in responses
+### New MCP tool
 
-Still outstanding.
+`backfill_embeddings`
 
-Why it matters:
+Purpose:
-- callers currently benefit from fallback automatically
-- but debugging is easier if responses explicitly say whether retrieval was `semantic` or `text`
 
-Suggested shape:
+* find thoughts missing an embedding for the active model
-- add a machine-readable field such as `retrieval_mode: semantic|text`
+* generate embeddings in batches
-- keep it consistent across all query-based tools that use shared retrieval logic
+* write embeddings with upsert semantics
+* report counts for scanned, embedded, skipped, and failed thoughts
 
-### 2. Verify and improve tests
+### Input
 
-Still worth auditing.
+```json
+{
+  "project": "optional project name or id",
+  "limit": 100,
+  "batch_size": 20,
+  "include_archived": false,
+  "older_than_days": 0,
+  "dry_run": false
+}
+```
 
-Recommended checks:
+Notes:
-- no-embedding scope falls back to text search
-- project-scoped fallback only searches within project scope
-- archived thoughts remain excluded by default
-- `related_thoughts` falls back correctly when semantic vectors are unavailable
-- backfill creates embeddings that later restore semantic search
 
-### 3. Re-embedding / migration ergonomics
+* `project` scopes the backfill to a project when desired
+* `limit` caps total thoughts processed in one tool call
+* `batch_size` controls provider load
+* `include_archived` defaults to `false`
+* `older_than_days` is optional and mainly useful to avoid racing with fresh writes
+* `dry_run` returns counts and sample IDs without calling the embedding provider
 
-Still optional future work.
+### Output
 
-Potential additions:
+```json
-- count missing embeddings by project
+{
-- add `missing_embeddings` stats to `thought_stats`
+  "model": "openai/text-embedding-3-small",
-- add a controlled re-embed or reindex flow for model migrations
+  "scanned": 100,
+  "embedded": 87,
+  "skipped": 13,
+  "failed": 0,
+  "dry_run": false,
+  "failures": []
+}
+```
+
+Optional:
+
+* include a short `next_cursor` later if we add cursor-based paging
 
 ---
 
-## Notes for maintainers
+## Backfill behavior
 
-Do not read this file as an untouched future roadmap item anymore. The repo has already implemented the core work described here.
+### Core rules
 
-If more backfill/fallback work is planned, append it as concrete follow-ups against the current codebase rather than preserving the old speculative rollout order.
+* Backfill only when a thought is missing an embedding row for the active model.
+* Do not recompute embeddings that already exist for that model unless an explicit future `force` flag is added.
+* Keep embeddings per model side by side in the existing `embeddings` table.
+* Use `insert ... on conflict (thought_id, model) do update` so retries stay idempotent.
+
+### Selection query
+
+Add a store query that returns thoughts where no embedding exists for the requested model.
+
+Shape:
+
+* from `thoughts t`
+* left join `embeddings e on e.thought_id = t.guid and e.model = $model`
+* filter `e.id is null`
+* optional filters for project, archived state, age
+* order by `t.created_at asc`
+* limit by requested batch
+
+Ordering oldest first is useful because it steadily restores long-tail recall instead of repeatedly revisiting recent writes.
+
+### Processing loop
+
+For each selected thought:
+
+1. read `content`
+2. call `provider.Embed(content)`
+3. upsert embedding row for `thought_id + model`
+4. continue on per-item failure and collect errors
+
+Use bounded concurrency instead of fully serial processing so large backfills complete in reasonable time without overwhelming the provider.
+
+Recommended first pass:
+
+* one tool invocation handles batches internally
+* concurrency defaults to a small fixed number like `4`
+* `batch_size` and concurrency are kept server-side defaults at first, even if only `limit` is exposed in MCP input
 
 ---
 
-## Historical note
+## Search fallback behavior
 
-The original long-form proposal was replaced during the repo audit because it described work that is now largely complete and was causing issue/document drift.
+### Goal
 
-If needed, recover the older version from git history.
+If semantic retrieval cannot run because no embeddings exist for the active model in the selected scope, AMCS should fall back to Postgres text search instead of returning empty semantic results by default.
+
+### Fallback rules
+
+* If embeddings exist for the active model, keep using vector search as the primary path.
+* If no embeddings exist for the active model in scope, run Postgres text search against raw thought content.
+* Fallback should apply to:
+
+  * `search_thoughts`
+  * `recall_context`
+  * `get_project_context` when `query` is provided
+  * `summarize_thoughts` when `query` is provided
+  * semantic neighbors in `related_thoughts`
+
+* Fallback should not mutate data. It is retrieval-only.
+* Backfill remains the long-term fix; text search is the immediate safety net.
+
+### Postgres search approach
+
+Add a native full-text index on thought content and query it with a matching text-search configuration.
+
+Recommended first pass:
+
+* add a migration creating a GIN index on `to_tsvector('simple', content)`
+* use `websearch_to_tsquery('simple', $query)` for user-entered text
+* rank results with `ts_rank_cd(...)`
+* continue excluding archived thoughts by default
+* continue honoring project scope
+
+Using the `simple` configuration is a safer default for mixed prose, identifiers, and code-ish text than a language-specific stemmer.
+
+### Store additions for fallback
+
+Add store methods such as:
+
+* `HasEmbeddingsForModel(ctx, model string, projectID *uuid.UUID) (bool, error)`
+* `SearchThoughtsText(ctx, query string, limit int, projectID *uuid.UUID, excludeID *uuid.UUID) ([]SearchResult, error)`
+
+These should be used by a shared retrieval helper in `internal/tools` so semantic callers degrade consistently.
+
+### Notes on ranking
+
+Text-search scores will not be directly comparable to vector similarity scores.
+
+That is acceptable in v1 because:
+
+* each request will use one retrieval mode at a time
+* fallback is only used when semantic search is unavailable
+* response payloads can continue to return `similarity` as a generic relevance score
+
+---
+
+## Auto behavior
+
+The user asked for an auto backfill tool, so define two layers:
+
+### Layer 1: explicit MCP tool
+
+Ship `backfill_embeddings` first.
+
+This is the lowest-risk path because:
+
+* it is observable
+* it is rate-limited by the caller
+* it avoids surprise provider cost on startup
+
+### Layer 2: optional automatic runner
+
+Add a config-gated background runner after the tool exists and is proven stable.
+
+Config sketch:
+
+```yaml
+backfill:
+  enabled: false
+  run_on_startup: false
+  interval: "15m"
+  batch_size: 20
+  max_per_run: 100
+  include_archived: false
+```
+
+Behavior:
+
+* on startup, if enabled and `run_on_startup=true`, run a small bounded backfill pass
+* if `interval` is set, periodically backfill missing embeddings for the active configured model
+* log counts and failures, but never block server startup on backfill failure
+
+This keeps the first implementation simple while still giving us a clean path to true automation.
+
+---
+
+## Store changes
+
+Add store methods focused on missing-model coverage.
+
+### New methods
+
+* `ListThoughtsMissingEmbedding(ctx, model string, limit int, projectID *uuid.UUID, includeArchived bool, olderThanDays int) ([]Thought, error)`
+* `UpsertEmbedding(ctx, thoughtID uuid.UUID, model string, embedding []float32) error`
+
+### Optional later methods
+
+* `CountThoughtsMissingEmbedding(ctx, model string, projectID *uuid.UUID, includeArchived bool) (int, error)`
+* `ListThoughtIDsMissingEmbeddingPage(...)` for cursor-based paging on large datasets
+
+### Why separate `UpsertEmbedding`
+
+`InsertThought` and `UpdateThought` already contain embedding upsert logic, but a dedicated helper will:
+
+* reduce duplication
+* let backfill avoid full thought updates
+* make future re-embedding jobs cleaner
+
+---
+
+## Tooling changes
+
+### New file
+
+`internal/tools/backfill.go`
+
+Responsibilities:
+
+* parse input
+* resolve project if provided
+* select missing thoughts
+* run bounded embedding generation
+* record per-item failures without aborting the whole batch
+* return summary counts
+
+### MCP registration
+
+Add the tool to:
+
+* `internal/mcpserver/server.go`
+* `internal/mcpserver/schema.go` and tests if needed
+* `internal/app/app.go` wiring
+
+Suggested tool description:
+
+* `Generate missing embeddings for stored thoughts using the active embedding model.`
+
+---
+
+## Config changes
+
+No config is required for the first manual tool beyond the existing embedding provider settings.
+
+For the later automatic runner, add:
+
+* `backfill.enabled`
+* `backfill.run_on_startup`
+* `backfill.interval`
+* `backfill.batch_size`
+* `backfill.max_per_run`
+* `backfill.include_archived`
+
+Validation rules:
+
+* `batch_size > 0`
+* `max_per_run >= batch_size`
+* `interval` must parse when provided
+
+---
+
+## Failure handling
+
+The backfill tool should be best-effort, not all-or-nothing.
+
+Rules:
+
+* one thought failure does not abort the full run
+* provider errors are captured and counted
+* database upsert failures are captured and counted
+* final tool response includes truncated failure details
+* full details go to logs
+
+Failure payloads should avoid returning raw thought content to the caller if that would create noisy or sensitive responses. Prefer thought IDs plus short error strings.
+
+---
+
+## Observability
+
+Add structured logs for:
+
+* selected model
+* project scope
+* scan count
+* success count
+* failure count
+* duration
+
+Later, metrics can include:
+
+* `amcs_backfill_runs_total`
+* `amcs_backfill_embeddings_total`
+* `amcs_backfill_failures_total`
+* `amcs_thoughts_missing_embeddings`
+
+---
+
+## Concurrency and rate limiting
+
+Keep the first version conservative.
+
+Plan:
+
+* use a worker pool with a small fixed concurrency
+* keep batch sizes small by default
+* stop fetching new work once `limit` is reached
+* respect `ctx` cancellation so long backfills can be interrupted cleanly
+
+Do not add provider-specific rate-limit logic in v1 unless real failures show it is needed.
+
+---
+
+## Security and safety
+
+* Reuse existing MCP auth.
+* Do not expose a broad `force=true` option in v1.
+* Default to non-archived thoughts only.
+* Do not mutate raw thought text or metadata during backfill.
+* Treat embeddings as derived data that may be regenerated safely.
+
+---
+
+## Testing plan
+
+### Store tests
+
+Add tests for:
+
+* listing thoughts missing embeddings for a model
+* project-scoped missing-embedding queries
+* archived thought filtering
+* idempotent upsert behavior
+
+### Tool tests
+
+Add tests for:
+
+* dry-run mode
+* successful batch embedding
+* partial provider failures
+* empty result set
+* project resolution
+* context cancellation
+
+### Integration tests
+
+Add a flow covering:
+
+1. create thoughts without embeddings for a target model
+2. run `backfill_embeddings`
+3. confirm rows exist in `embeddings`
+4. confirm `search_thoughts` can now retrieve them when using that model
+
+### Fallback search tests
+
+Add coverage for:
+
+* no embeddings for model -> `search_thoughts` uses Postgres text search
+* project-scoped queries only search matching project thoughts
+* archived thoughts stay excluded by default
+* `related_thoughts` falls back to text search neighbors when semantic vectors are unavailable
+* once embeddings exist, semantic search remains the primary path
+
+---
+
+## Rollout order
+
+1. Add store helpers for missing-embedding selection and embedding upsert.
+2. Add Postgres full-text index migration and text-search store helpers.
+3. Add shared semantic-or-text fallback retrieval logic for query-based tools.
+4. Add `backfill_embeddings` MCP tool and wire it into the server.
+5. Add unit and integration tests.
+6. Document usage in `README.md`.
+7. Add optional background auto-runner behind config.
+8. Consider a future `force` or `reindex_model` path only after v1 is stable.
+
+---
+
+## Open questions
+
+* Should the tool expose `batch_size` to clients, or should batching stay internal?
+* Should the first version support only the active model, or allow a `model` override for admins?
+* Should archived thoughts be backfilled by default during startup jobs but not MCP calls?
+* Do we want a separate CLI/admin command for large one-time reindex jobs outside MCP?
+
+Recommended answers for v1:
+
+* keep batching mostly internal
+* use only the active configured model
+* exclude archived thoughts by default everywhere
+* postpone a dedicated CLI until volume justifies it
+
+---
+
+## Nice follow-ups
+
+* add a `missing_embeddings` stat to `thought_stats`
+* expose a read-only tool for counting missing embeddings by project
+* add a re-embed path for migrating from one model to another in controlled waves
+* add metadata extraction backfill as a separate job if imported content often lacks metadata
+* expose the retrieval mode in responses for easier debugging of semantic vs text fallback

migrations/019_tool_annotations.sql

@@ -1,14 +0,0 @@
--- Migration: 019_tool_annotations
--- Adds a table for model-authored usage notes per tool.
-
-create table if not exists tool_annotations (
-  id         bigserial   primary key,
-  tool_name  text        not null,
-  notes      text        not null default '',
-  created_at timestamptz not null default now(),
-  updated_at timestamptz not null default now(),
-  constraint tool_annotations_tool_name_unique unique (tool_name)
-);
-
-grant all on table public.tool_annotations to amcs;
-grant usage, select on sequence tool_annotations_id_seq to amcs;

schema/README.md

@@ -1,35 +0,0 @@
-# Schema workflow
-
-The `schema/*.dbml` files are the database schema source of truth.
-
-## Generate SQL migrations
-
-Run:
-
-```bash
-make generate-migrations
-```
-
-This uses `relspec` to convert the DBML files into PostgreSQL SQL and writes the generated schema migration to:
-
-- `migrations/020_generated_schema.sql`
-
-## Check schema drift
-
-Run:
-
-```bash
-make check-schema-drift
-```
-
-This regenerates the SQL from `schema/*.dbml` and compares it with `migrations/020_generated_schema.sql`.
-If the generated output differs, the command fails so CI can catch schema drift.
-
-## Workflow
-
-1. Update the DBML files in `schema/`
-2. Run `make generate-migrations`
-3. Review the generated SQL
-4. Commit both the DBML changes and the generated migration
-
-Existing handwritten migrations stay in place. Going forward, update the DBML first and regenerate the SQL from there.

schema/calendar.dbml

@@ -1,44 +0,0 @@
-Table family_members {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  relationship text
-  birth_date date
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-}
-
-Table activities {
-  id uuid [pk, default: `gen_random_uuid()`]
-  family_member_id uuid [ref: > family_members.id]
-  title text [not null]
-  activity_type text
-  day_of_week text
-  start_time time
-  end_time time
-  start_date date
-  end_date date
-  location text
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    day_of_week
-    family_member_id
-    (start_date, end_date)
-  }
-}
-
-Table important_dates {
-  id uuid [pk, default: `gen_random_uuid()`]
-  family_member_id uuid [ref: > family_members.id]
-  title text [not null]
-  date_value date [not null]
-  recurring_yearly boolean [not null, default: false]
-  reminder_days_before int [not null, default: 7]
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    date_value
-  }
-}

schema/core.dbml

@@ -1,48 +0,0 @@
-Table thoughts {
-  id bigserial [pk]
-  guid uuid [unique, not null, default: `gen_random_uuid()`]
-  content text [not null]
-  metadata jsonb [default: `'{}'::jsonb`]
-  created_at timestamptz [default: `now()`]
-  updated_at timestamptz [default: `now()`]
-  project_id uuid [ref: > projects.guid]
-  archived_at timestamptz
-}
-
-Table projects {
-  id bigserial [pk]
-  guid uuid [unique, not null, default: `gen_random_uuid()`]
-  name text [unique, not null]
-  description text
-  created_at timestamptz [default: `now()`]
-  last_active_at timestamptz [default: `now()`]
-}
-
-Table thought_links {
-  from_id bigint [not null, ref: > thoughts.id]
-  to_id bigint [not null, ref: > thoughts.id]
-  relation text [not null]
-  created_at timestamptz [default: `now()`]
-
-  indexes {
-    (from_id, to_id, relation) [pk]
-    from_id
-    to_id
-  }
-}
-
-Table embeddings {
-  id bigserial [pk]
-  guid uuid [unique, not null, default: `gen_random_uuid()`]
-  thought_id uuid [not null, ref: > thoughts.guid]
-  model text [not null]
-  dim int [not null]
-  embedding vector [not null]
-  created_at timestamptz [default: `now()`]
-  updated_at timestamptz [default: `now()`]
-
-  indexes {
-    (thought_id, model) [unique]
-    thought_id
-  }
-}

schema/crm.dbml

@@ -1,53 +0,0 @@
-Table professional_contacts {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  company text
-  title text
-  email text
-  phone text
-  linkedin_url text
-  how_we_met text
-  tags "text[]" [not null, default: `'{}'`]
-  notes text
-  last_contacted timestamptz
-  follow_up_date date
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    last_contacted
-    follow_up_date
-  }
-}
-
-Table contact_interactions {
-  id uuid [pk, default: `gen_random_uuid()`]
-  contact_id uuid [not null, ref: > professional_contacts.id]
-  interaction_type text [not null]
-  occurred_at timestamptz [not null, default: `now()`]
-  summary text [not null]
-  follow_up_needed boolean [not null, default: false]
-  follow_up_notes text
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    (contact_id, occurred_at)
-  }
-}
-
-Table opportunities {
-  id uuid [pk, default: `gen_random_uuid()`]
-  contact_id uuid [ref: > professional_contacts.id]
-  title text [not null]
-  description text
-  stage text [not null, default: 'identified']
-  value "decimal(12,2)"
-  expected_close_date date
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    stage
-  }
-}

schema/files.dbml

@@ -1,25 +0,0 @@
-Table stored_files {
-  id bigserial [pk]
-  guid uuid [unique, not null, default: `gen_random_uuid()`]
-  thought_id uuid [ref: > thoughts.guid]
-  project_id uuid [ref: > projects.guid]
-  name text [not null]
-  media_type text [not null]
-  kind text [not null, default: 'file']
-  encoding text [not null, default: 'base64']
-  size_bytes bigint [not null]
-  sha256 text [not null]
-  content bytea [not null]
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    thought_id
-    project_id
-    sha256
-  }
-}
-
-// Cross-file refs (for relspecgo merge)
-Ref: stored_files.thought_id > thoughts.guid [delete: set null]
-Ref: stored_files.project_id > projects.guid [delete: set null]

schema/household.dbml

@@ -1,31 +0,0 @@
-Table household_items {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  category text
-  location text
-  details jsonb [not null, default: `'{}'`]
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    category
-  }
-}
-
-Table household_vendors {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  service_type text
-  phone text
-  email text
-  website text
-  notes text
-  rating int
-  last_used date
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    service_type
-  }
-}

schema/maintenance.dbml

@@ -1,30 +0,0 @@
-Table maintenance_tasks {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  category text
-  frequency_days int
-  last_completed timestamptz
-  next_due timestamptz
-  priority text [not null, default: 'medium']
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    next_due
-  }
-}
-
-Table maintenance_logs {
-  id uuid [pk, default: `gen_random_uuid()`]
-  task_id uuid [not null, ref: > maintenance_tasks.id]
-  completed_at timestamptz [not null, default: `now()`]
-  performed_by text
-  cost "decimal(10,2)"
-  notes text
-  next_action text
-
-  indexes {
-    (task_id, completed_at)
-  }
-}

schema/meals.dbml

@@ -1,49 +0,0 @@
-Table recipes {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [not null]
-  cuisine text
-  prep_time_minutes int
-  cook_time_minutes int
-  servings int
-  ingredients jsonb [not null, default: `'[]'`]
-  instructions jsonb [not null, default: `'[]'`]
-  tags "text[]" [not null, default: `'{}'`]
-  rating int
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    cuisine
-    tags
-  }
-}
-
-Table meal_plans {
-  id uuid [pk, default: `gen_random_uuid()`]
-  week_start date [not null]
-  day_of_week text [not null]
-  meal_type text [not null]
-  recipe_id uuid [ref: > recipes.id]
-  custom_meal text
-  servings int
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    week_start
-  }
-}
-
-Table shopping_lists {
-  id uuid [pk, default: `gen_random_uuid()`]
-  week_start date [unique, not null]
-  items jsonb [not null, default: `'[]'`]
-  notes text
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    week_start
-  }
-}

schema/meta.dbml

@@ -1,32 +0,0 @@
-Table chat_histories {
-  id uuid [pk, default: `gen_random_uuid()`]
-  session_id text [not null]
-  title text
-  channel text
-  agent_id text
-  project_id uuid [ref: > projects.guid]
-  messages jsonb [not null, default: `'[]'`]
-  summary text
-  metadata jsonb [not null, default: `'{}'`]
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    session_id
-    project_id
-    channel
-    agent_id
-    created_at
-  }
-}
-
-Table tool_annotations {
-  id bigserial [pk]
-  tool_name text [unique, not null]
-  notes text [not null, default: '']
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-}
-
-// Cross-file refs (for relspecgo merge)
-Ref: chat_histories.project_id > projects.guid [delete: set null]

schema/skills.dbml

@@ -1,46 +0,0 @@
-Table agent_skills {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [unique, not null]
-  description text [not null, default: '']
-  content text [not null]
-  tags "text[]" [not null, default: `'{}'`]
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-}
-
-Table agent_guardrails {
-  id uuid [pk, default: `gen_random_uuid()`]
-  name text [unique, not null]
-  description text [not null, default: '']
-  content text [not null]
-  severity text [not null, default: 'medium']
-  tags "text[]" [not null, default: `'{}'`]
-  created_at timestamptz [not null, default: `now()`]
-  updated_at timestamptz [not null, default: `now()`]
-}
-
-Table project_skills {
-  project_id uuid [not null, ref: > projects.guid]
-  skill_id uuid [not null, ref: > agent_skills.id]
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    (project_id, skill_id) [pk]
-    project_id
-  }
-}
-
-Table project_guardrails {
-  project_id uuid [not null, ref: > projects.guid]
-  guardrail_id uuid [not null, ref: > agent_guardrails.id]
-  created_at timestamptz [not null, default: `now()`]
-
-  indexes {
-    (project_id, guardrail_id) [pk]
-    project_id
-  }
-}
-
-// Cross-file refs (for relspecgo merge)
-Ref: project_skills.project_id > projects.guid [delete: cascade]
-Ref: project_guardrails.project_id > projects.guid [delete: cascade]

scripts/run-local.sh

@@ -2,11 +2,4 @@
 
 set -euo pipefail
 
-CONFIG_PATH="${1:-configs/dev.yaml}"
+go run ./cmd/amcs-server --config "${1:-configs/dev.yaml}"
-
-if [[ ! -f internal/app/ui/dist/index.html ]]; then
-  echo "UI build not found; building frontend first..."
-  make ui-build
-fi
-
-go run ./cmd/amcs-server --config "$CONFIG_PATH"

ui/index.html

@@ -1,16 +0,0 @@
-<!doctype html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <title>AMCS</title>
-    <meta
-      name="description"
-      content="AMCS is a memory server that captures, links, and retrieves structured project thoughts for AI assistants using semantic search, summaries, and MCP tools."
-    />
-  </head>
-  <body class="bg-slate-950">
-    <div id="app"></div>
-    <script type="module" src="/src/main.ts"></script>
-  </body>
-</html>

ui/package.json

@@ -1,23 +0,0 @@
-{
-  "name": "amcs-ui",
-  "private": true,
-  "version": "0.0.0",
-  "packageManager": "pnpm@10.33.0",
-  "type": "module",
-  "scripts": {
-    "dev": "vite",
-    "build": "vite build",
-    "check": "svelte-check --tsconfig ./tsconfig.json",
-    "preview": "vite preview"
-  },
-  "devDependencies": {
-    "@sveltejs/vite-plugin-svelte": "^5.0.3",
-    "@tailwindcss/vite": "^4.1.4",
-    "@types/node": "^24.5.2",
-    "svelte": "^5.28.2",
-    "svelte-check": "^4.1.6",
-    "tailwindcss": "^4.1.4",
-    "typescript": "^5.8.3",
-    "vite": "^6.3.2"
-  }
-}

ui/src/App.svelte

@@ -1,262 +0,0 @@
-<script lang="ts">
-  import { onMount } from "svelte";
-
-  type AccessEntry = {
-    key_id: string;
-    last_accessed_at: string;
-    last_path: string;
-    user_agent: string;
-    request_count: number;
-  };
-
-  type StatusResponse = {
-    title: string;
-    description: string;
-    version: string;
-    build_date: string;
-    commit: string;
-    connected_count: number;
-    total_known: number;
-    connected_window: string;
-    oauth_enabled: boolean;
-    entries: AccessEntry[];
-  };
-
-  let data: StatusResponse | null = null;
-  let loading = true;
-  let error = "";
-
-  const quickLinks = [
-    { href: "/llm", label: "LLM Instructions" },
-    { href: "/healthz", label: "Health Check" },
-    { href: "/readyz", label: "Readiness Check" },
-  ];
-
-  async function loadStatus() {
-    loading = true;
-    error = "";
-
-    try {
-      const response = await fetch("/api/status");
-      if (!response.ok) {
-        throw new Error(`Status request failed with ${response.status}`);
-      }
-      data = (await response.json()) as StatusResponse;
-    } catch (err) {
-      error = err instanceof Error ? err.message : "Failed to load status";
-    } finally {
-      loading = false;
-    }
-  }
-
-  function formatDate(value: string) {
-    return new Date(value).toLocaleString();
-  }
-
-  onMount(loadStatus);
-</script>
-
-<svelte:head>
-  <title>AMCS</title>
-</svelte:head>
-
-<div class="min-h-screen bg-slate-950 text-slate-100">
-  <main
-    class="mx-auto flex min-h-screen max-w-7xl flex-col px-4 py-6 sm:px-6 lg:px-8"
-  >
-    <section
-      class="overflow-hidden rounded-3xl border border-white/10 bg-slate-900 shadow-2xl shadow-slate-950/40"
-    >
-      <img
-        src="/images/project.jpg"
-        alt="Avelon Memory Crystal"
-        class="h-64 w-full object-cover object-center sm:h-80"
-      />
-
-      <div class="grid gap-8 p-6 sm:p-8 lg:grid-cols-[1.6fr_1fr] lg:p-10">
-        <div class="space-y-6">
-          <div class="space-y-4">
-            <div
-              class="inline-flex items-center gap-2 rounded-full border border-cyan-400/20 bg-cyan-400/10 px-3 py-1 text-sm font-medium text-cyan-200"
-            >
-              <span class="h-2 w-2 rounded-full bg-emerald-400"></span>
-              Avalon Memory Crystal Server
-            </div>
-            <div>
-              <h1
-                class="text-3xl font-semibold tracking-tight text-white sm:text-4xl"
-              >
-                Avelon Memory Crystal Server (AMCS)
-              </h1>
-              <p
-                class="mt-3 max-w-3xl text-base leading-7 text-slate-300 sm:text-lg"
-              >
-                {data?.description ??
-                  "AMCS is a memory server that captures, links, and retrieves structured project thoughts for AI assistants using semantic search, summaries, and MCP tools."}
-              </p>
-            </div>
-          </div>
-
-          <div class="flex flex-wrap gap-3">
-            {#each quickLinks as link}
-              <a
-                class="inline-flex items-center justify-center rounded-xl border border-cyan-300/20 bg-cyan-400/10 px-4 py-2 text-sm font-semibold text-cyan-100 transition hover:border-cyan-300/40 hover:bg-cyan-400/20"
-                href={link.href}>{link.label}</a
-              >
-            {/each}
-            {#if data?.oauth_enabled}
-              <a
-                class="inline-flex items-center justify-center rounded-xl border border-violet-300/20 bg-violet-400/10 px-4 py-2 text-sm font-semibold text-violet-100 transition hover:border-violet-300/40 hover:bg-violet-400/20"
-                href="/oauth-authorization-server">OAuth Authorization Server</a
-              >
-            {/if}
-          </div>
-
-          <div class="grid gap-4 sm:grid-cols-3">
-            <div class="rounded-2xl border border-white/10 bg-white/5 p-5">
-              <p class="text-sm uppercase tracking-[0.2em] text-slate-400">
-                Connected users
-              </p>
-              <p class="mt-2 text-3xl font-semibold text-white">
-                {data?.connected_count ?? "—"}
-              </p>
-            </div>
-            <div class="rounded-2xl border border-white/10 bg-white/5 p-5">
-              <p class="text-sm uppercase tracking-[0.2em] text-slate-400">
-                Known principals
-              </p>
-              <p class="mt-2 text-3xl font-semibold text-white">
-                {data?.total_known ?? "—"}
-              </p>
-            </div>
-            <div class="rounded-2xl border border-white/10 bg-white/5 p-5">
-              <p class="text-sm uppercase tracking-[0.2em] text-slate-400">
-                Version
-              </p>
-              <p class="mt-2 break-all text-2xl font-semibold text-white">
-                {data?.version ?? "—"}
-              </p>
-            </div>
-          </div>
-        </div>
-
-        <aside
-          class="space-y-4 rounded-2xl border border-white/10 bg-slate-950/50 p-5"
-        >
-          <div>
-            <h2 class="text-lg font-semibold text-white">Build details</h2>
-            <p class="mt-1 text-sm text-slate-400">The same status info.</p>
-          </div>
-          <dl class="space-y-3 text-sm text-slate-300">
-            <div>
-              <dt class="text-slate-500">Build date</dt>
-              <dd class="mt-1 font-medium text-white">
-                {data?.build_date ?? "unknown"}
-              </dd>
-            </div>
-            <div>
-              <dt class="text-slate-500">Commit</dt>
-              <dd
-                class="mt-1 break-all rounded-lg bg-white/5 px-3 py-2 font-mono text-xs text-cyan-100"
-              >
-                {data?.commit ?? "unknown"}
-              </dd>
-            </div>
-            <div>
-              <dt class="text-slate-500">Connected window</dt>
-              <dd class="mt-1 font-medium text-white">
-                {data?.connected_window ?? "last 10 minutes"}
-              </dd>
-            </div>
-          </dl>
-        </aside>
-      </div>
-    </section>
-
-    <section
-      class="mt-6 rounded-3xl border border-white/10 bg-slate-900/80 p-6 shadow-xl shadow-slate-950/20 sm:p-8"
-    >
-      <div
-        class="flex flex-col gap-3 sm:flex-row sm:items-end sm:justify-between"
-      >
-        <div>
-          <h2 class="text-2xl font-semibold text-white">Recent access</h2>
-          <p class="mt-1 text-sm text-slate-400">
-            Authenticated principals AMCS has seen recently.
-          </p>
-        </div>
-        <button
-          class="inline-flex items-center justify-center rounded-xl border border-white/10 bg-white/5 px-4 py-2 text-sm font-medium text-slate-200 transition hover:bg-white/10"
-          on:click={loadStatus}
-        >
-          Refresh
-        </button>
-      </div>
-
-      {#if loading}
-        <div
-          class="mt-6 rounded-2xl border border-dashed border-white/10 bg-slate-950/40 px-4 py-10 text-center text-slate-400"
-        >
-          Loading status…
-        </div>
-      {:else if error}
-        <div
-          class="mt-6 rounded-2xl border border-rose-400/30 bg-rose-400/10 px-4 py-6 text-sm text-rose-100"
-        >
-          <p class="font-semibold">Couldn’t load the status snapshot.</p>
-          <p class="mt-1 text-rose-100/80">{error}</p>
-        </div>
-      {:else if data && data.entries.length === 0}
-        <div
-          class="mt-6 rounded-2xl border border-dashed border-white/10 bg-slate-950/40 px-4 py-10 text-center text-slate-400"
-        >
-          No authenticated access recorded yet.
-        </div>
-      {:else if data}
-        <div class="mt-6 overflow-hidden rounded-2xl border border-white/10">
-          <div class="overflow-x-auto">
-            <table
-              class="min-w-full divide-y divide-white/10 text-left text-sm text-slate-300"
-            >
-              <thead
-                class="bg-white/5 text-xs uppercase tracking-[0.2em] text-slate-500"
-              >
-                <tr>
-                  <th class="px-4 py-3 font-medium">Principal</th>
-                  <th class="px-4 py-3 font-medium">Last accessed</th>
-                  <th class="px-4 py-3 font-medium">Last path</th>
-                  <th class="px-4 py-3 font-medium">Agent</th>
-                  <th class="px-4 py-3 font-medium">Requests</th>
-                </tr>
-              </thead>
-              <tbody class="divide-y divide-white/5 bg-slate-950/30">
-                {#each data.entries as entry}
-                  <tr class="hover:bg-white/[0.03]">
-                    <td class="px-4 py-3 align-top"
-                      ><code
-                        class="rounded bg-white/5 px-2 py-1 font-mono text-xs text-cyan-100"
-                        >{entry.key_id}</code
-                      ></td
-                    >
-                    <td class="px-4 py-3 align-top text-slate-200"
-                      >{formatDate(entry.last_accessed_at)}</td
-                    >
-                    <td class="px-4 py-3 align-top"
-                      ><code class="text-slate-100">{entry.last_path}</code></td
-                    >
-                    <td class="px-4 py-3 align-top text-slate-400 text-xs max-w-[16rem] truncate"
-                      >{entry.user_agent ?? "—"}</td
-                    >
-                    <td class="px-4 py-3 align-top font-semibold text-white"
-                      >{entry.request_count}</td
-                    >
-                  </tr>
-                {/each}
-              </tbody>
-            </table>
-          </div>
-        </div>
-      {/if}
-    </section>
-  </main>
-</div>

ui/src/app.css

@@ -1,16 +0,0 @@
-@import 'tailwindcss';
-
-:root {
-  color-scheme: dark;
-  font-family: Inter, system-ui, sans-serif;
-}
-
-html,
-body,
-#app {
-  min-height: 100%;
-}
-
-body {
-  margin: 0;
-}

ui/src/main.ts

@@ -1,9 +0,0 @@
-import './app.css';
-import App from './App.svelte';
-import { mount } from 'svelte';
-
-const app = mount(App, {
-  target: document.getElementById('app')!
-});
-
-export default app;

ui/svelte.config.js

@@ -1,5 +0,0 @@
-export default {
-  compilerOptions: {
-    dev: process.env.NODE_ENV !== 'production'
-  }
-};

ui/tsconfig.json

@@ -1,15 +0,0 @@
-{
-  "extends": "./tsconfig.node.json",
-  "compilerOptions": {
-    "module": "ESNext",
-    "moduleResolution": "Bundler",
-    "target": "ES2022",
-    "lib": ["ESNext", "DOM"],
-    "verbatimModuleSyntax": true,
-    "strict": true,
-    "allowJs": true,
-    "checkJs": false,
-    "types": ["svelte", "node"]
-  },
-  "include": ["src/**/*.ts", "src/**/*.js", "src/**/*.svelte", "vite.config.ts"]
-}

ui/tsconfig.node.json

@@ -1,8 +0,0 @@
-{
-  "compilerOptions": {
-    "composite": true,
-    "module": "ESNext",
-    "moduleResolution": "Bundler",
-    "allowSyntheticDefaultImports": true
-  }
-}

ui/vite.config.ts

@@ -1,31 +0,0 @@
-import { defineConfig } from 'vite';
-import { svelte } from '@sveltejs/vite-plugin-svelte';
-import tailwindcss from '@tailwindcss/vite';
-
-const backendTarget = process.env.AMCS_UI_BACKEND ?? 'http://127.0.0.1:8080';
-
-export default defineConfig({
-  plugins: [svelte(), tailwindcss()],
-  server: {
-    host: '0.0.0.0',
-    port: 5173,
-    proxy: {
-      '/api': backendTarget,
-      '/healthz': backendTarget,
-      '/readyz': backendTarget,
-      '/llm': backendTarget,
-      '/images': backendTarget,
-      '/favicon.ico': backendTarget,
-      '/mcp': backendTarget,
-      '/files': backendTarget,
-      '/oauth-authorization-server': backendTarget,
-      '/authorize': backendTarget,
-      '/oauth': backendTarget,
-      '/.well-known': backendTarget
-    }
-  },
-  build: {
-    outDir: '../internal/app/ui/dist',
-    emptyOutDir: true
-  }
-});