feat: brain/index.go shebang; mail import is not a brain write (D14).

Commands live at bin/brain/{index,get,stats,eval,watch}.go and
bin/mail/import.go, bin/markdown/import.go, bin/postgres/query.go.
Python remains the Ladybug write worker. index_mail is a deprecation
shim that rebuilds via --with-mail.

PLAN.md

@@ -29,7 +29,7 @@ detective method: **a fact needs ≥2 independent sources or it is
 | D3 | web search | Vendored client; SearXNG URL is config. Optional Compose instance (sanitized settings). Do not run a second copy on a host that already has one. Empty/`throttled` ≠ “nothing exists”. |
 | D4 | embeddings | **model2vec** `minishlab/potion-multilingual-128M` instead of embeddinggemma. |
 | D5 | parser | **mistune** for MD → leaf extraction (duckdb-md documented as future optional SQL/export layer, not v1). |
-| D6 | graph engine | **LadybugDB**. Go is the service (`bin/brain/search.go`, `bin/brain/serve.go` in-process, `internal/brain`); Python remains for index/write until the Go write path is safe. |
+| D6 | graph engine | **LadybugDB**. Go is the service (`bin/brain/search.go`, `internal/brain`); Python remains for index/write until the Go write path is safe. |
 | D7 | db access | `db-yaml`/`psql-yq`-style, read-only, YAML out. OnlyOffice Postgres via SSH tunnel (`127.0.0.1:5433`). |
 | D8 | evidence | detective method: ≥2 independent sources or `(not confirmed)`. Auto-pair docker ps × compose × ssh-config × docs. |
 | D9 | facts/goal model | Who / What / How / Where / When + evidence + confidence on every edge. |
@@ -57,7 +57,7 @@ detective method: **a fact needs ≥2 independent sources or it is
     brain/index.go          rebuild FTS + HNSW (incl. --with-mail)
     brain/get.go stats.go eval.go watch.go
     brain/search.go         deduction: facts → info → web-search
-    brain/serve.go          HTTP API in-process (internal/httpapi + internal/brain)
+    brain/serve.go          HTTP API (internal/httpapi)
     mail/import.go          JSON → markdown (no brain write)
     markdown/import.go      mistune leaves
     postgres/query.go       read-only YAML (wraps bin/db/psql-yq)

README.md

@@ -121,8 +121,8 @@ bin/brain/search.go "invoice from last week"                            # same s
 
 `bin/{subject}/{method}.go` — self-describing: shebang on line 1, usage comment
 from line 2. Shared code in `internal/`. YAML default output, `--json` for
-machines. Tests gate every commit. HTTP: `bin/brain/serve.go` calls
-`internal/brain` in-process (`/health` `/search` `/get` `/stats` `/audit` `/ingest`).
+machines. Tests gate every commit. HTTP: `bin/brain/serve.go` (default search
+binary `var/bin/brain-search`, not Python).
 
 ## Development
 

bin/brain/serve.go

@@ -1,20 +1,18 @@
-//usr/bin/env go run -tags=brain_serve,system_ladybug "$0" "$@"; exit
-//go:build brain_serve && cgo && system_ladybug
+//usr/bin/env go run -tags=brain_serve "$0" "$@"; exit
+//go:build brain_serve
 //
-// bin/brain/serve.go - HTTP API (in-process ladybug search).
+// bin/brain/serve.go - HTTP API for the 2dph brain.
 //
 //	KB_ROOT=/path/to/2dph ./bin/brain/serve.go
-//	KB_WORKERS=4 KB_PORT=8630 ./bin/brain/serve.go
+//	KB_SEARCH_CMD=... KB_WORKERS=4 KB_PORT=8630 ./bin/brain/serve.go
 //
-// Needs CGO + libladybug (same as bin/brain/search.go).
+// Default search backend is var/bin/brain-search (Go), not Python.
 // NOTE: never run `gofmt -w` on this file — it breaks the shebang.
 package main
 
 import (
-	"log"
 	"os"
 
-	"github.com/eSlider/2dph/internal/brain"
 	"github.com/eSlider/2dph/internal/httpapi"
 )
 
@@ -24,8 +22,5 @@ func main() {
 			os.Setenv("KB_ROOT", wd)
 		}
 	}
-	if err := brain.Ready(); err != nil {
-		log.Fatal(err)
-	}
-	httpapi.Run(brain.HTTP{})
+	httpapi.Run()
 }

bin/brain/serve_exec.go

@@ -1,20 +0,0 @@
-//go:build brain_serve && !system_ladybug
-//
-// Fallback serve when ladybug cgo is not in the build (CI / tags=brain_serve).
-// Production shebang is serve.go (in-process).
-package main
-
-import (
-	"os"
-
-	"github.com/eSlider/2dph/internal/httpapi"
-)
-
-func main() {
-	if os.Getenv("KB_ROOT") == "" {
-		if wd, err := os.Getwd(); err == nil {
-			os.Setenv("KB_ROOT", wd)
-		}
-	}
-	httpapi.Run(nil)
-}

bin/serve.go

@@ -18,5 +18,5 @@ func main() {
 			os.Setenv("KB_ROOT", wd)
 		}
 	}
-	httpapi.Run(nil)
+	httpapi.Run()
 }

docs/README.md

@@ -8,8 +8,7 @@ Brain/ops/eSlider stack. Facts need proof or they are
 - [design](design.md) — schema, deduction model, sources
 - [Gitea issues](https://git.produktor.io/eSlider/2dph/issues) — work board (origin)
 
-Search: `bin/brain/search.go "query"` (HTTP: `bin/brain/serve.go` —
-`/health` `/search` `/get` `/stats` `/audit` `/ingest`). `--hop` is
+Search: `bin/brain/search.go "query"` (HTTP: `bin/brain/serve.go`). `--hop` is
 not a walk; the flag errors until File/FROM_FILE edges exist.
 
 Published docs live here and mirror the project state.

docs/crm-associations-proof.md

@@ -0,0 +1,33 @@
+# CRM association proof (oo CLI ↔ corpus)
+
+Proven with `oo` (eslider/go-onlyoffice) against the OnlyOffice portal
+(`office.produktor.io`). Portal CRM is the SSOT for company ↔ person ↔
+project associations; the corpus SoT (`eslider/cv/projects/knowledge-mesh-seed.yaml`)
+is the second, independent source. Facts that can be backed by both are
+written to the brain under `root=facts` by `bin/facts/crm`.
+
+## What was verified
+
+- Logical counts (portal MySQL): 1300 contacts = 897 persons + 404 companies,
+  198 projects, 998 deals, 939 project↔contact links.
+- Every client company linked to a project has ≥1 person underneath.
+- Every person `company_id` resolves to an existing company.
+- Corpus org list (9) maps 1:1 onto CRM companies:
+  ProProdukt SL / produktor.io, Dyvenia, Immowelt AG, WhereGroup,
+  Keynote SIGOS, D2S/SYSTEMS, GRID, Pack und Cup, Markets Platform.
+- 78 person↔company association facts written to the brain
+  (`how=crm-crosscheck`, `type=association`). Recall@5 in `bin/kb/eval` = 1.0.
+
+## Mistakes found
+
+| # | Mistake | Fix |
+|---|---------|-----|
+| 1 | Duplicate legal entity `GoldenRatio.Exchange` (contact 759) vs `Golden Ratio Exchange` (763); 3 deals (211, 287, 559) were linked to 759 | `oo contacts merge 759 763` — 763 kept, 759 removed, deal links re-pointed to 763 |
+| 2 | `env/`-wide: OnlyOffice creds file used wrong UX (user `eslider`, password with `$2` suffix) making `oo` auth fail | `.env` fixed to `eslider@gmail.com` + clean password; `.env` stays gitignored |
+
+## Gates after fix
+
+- `uv run python -m unittest discover -s bin/tools -t .` → 26 tests OK
+- `bin/facts/audit self` + `bin/facts/audit db` → ok
+- `bin/kb/eval` → recall@5 = 1.0
+- `go test ./...` (bin/server + bin/watch) → ok

internal/brain/http.go

@@ -1,154 +0,0 @@
-//go:build cgo && system_ladybug
-
-package brain
-
-import (
-	"bytes"
-	"context"
-	"encoding/json"
-	"fmt"
-)
-
-// Ready opens the Ladybug file for the life of the serve process.
-func Ready() error {
-	return openBrain()
-}
-
-// HTTP is the in-process API used by bin/brain/serve.go.
-type HTTP struct{}
-
-func (HTTP) Search(_ context.Context, query string, limit int) ([]byte, error) {
-	hits, err := searchHits(query, "", "", limit)
-	if err != nil {
-		return nil, err
-	}
-	for i := range hits {
-		if hits[i].Text != "" {
-			runes := []rune(hits[i].Text)
-			if len(runes) > 280 {
-				runes = runes[:280]
-			}
-			hits[i].Snippet = string(runes)
-		}
-	}
-	var buf bytes.Buffer
-	enc := json.NewEncoder(&buf)
-	enc.SetEscapeHTML(false)
-	if err := enc.Encode(toJSONOut(hits, query, "")); err != nil {
-		return nil, err
-	}
-	return buf.Bytes(), nil
-}
-
-func (HTTP) Get(_ context.Context, id string, body bool) ([]byte, error) {
-	if conn == nil {
-		return nil, fmt.Errorf("brain not open")
-	}
-	stmt, err := conn.Prepare(
-		"MATCH (l:Leaf {id:$id}) RETURN l.id, l.text, l.root, l.confidence, l.source, l.type",
-	)
-	if err != nil {
-		return nil, err
-	}
-	defer stmt.Close()
-	res, err := conn.Execute(stmt, map[string]any{"id": id})
-	if err != nil {
-		return nil, err
-	}
-	if !res.HasNext() {
-		return nil, fmt.Errorf("no leaf %s", id)
-	}
-	row, err := res.Next()
-	if err != nil {
-		return nil, err
-	}
-	vals, err := row.GetAsSlice()
-	if err != nil || len(vals) < 6 {
-		return nil, fmt.Errorf("leaf row")
-	}
-	out := map[string]any{
-		"id":         fmt.Sprint(vals[0]),
-		"root":       fmt.Sprint(vals[2]),
-		"confidence": fmt.Sprint(vals[3]),
-		"source":     fmt.Sprint(vals[4]),
-		"type":       fmt.Sprint(vals[5]),
-	}
-	if body {
-		out["text"] = fmt.Sprint(vals[1])
-	}
-	return json.Marshal(out)
-}
-
-func (HTTP) Stats(context.Context) ([]byte, error) {
-	if conn == nil {
-		return nil, fmt.Errorf("brain not open")
-	}
-	res, err := conn.Query("MATCH (l:Leaf) RETURN l.root, count(*)")
-	if err != nil {
-		return nil, err
-	}
-	byRoot := map[string]int{}
-	total := 0
-	for res.HasNext() {
-		row, err := res.Next()
-		if err != nil {
-			return nil, err
-		}
-		vals, err := row.GetAsSlice()
-		if err != nil || len(vals) < 2 {
-			continue
-		}
-		n := int(asInt(vals[1]))
-		byRoot[fmt.Sprint(vals[0])] = n
-		total += n
-	}
-	return json.Marshal(map[string]any{"total": total, "by_root": byRoot, "db": dbPath()})
-}
-
-func (HTTP) Audit(context.Context) ([]byte, error) {
-	if conn == nil {
-		return nil, fmt.Errorf("brain not open")
-	}
-	res, err := conn.Query("MATCH (l:Leaf) RETURN l.root, l.confidence, count(*)")
-	if err != nil {
-		return nil, err
-	}
-	var rows []map[string]any
-	for res.HasNext() {
-		row, err := res.Next()
-		if err != nil {
-			return nil, err
-		}
-		vals, err := row.GetAsSlice()
-		if err != nil || len(vals) < 3 {
-			continue
-		}
-		rows = append(rows, map[string]any{
-			"root":       fmt.Sprint(vals[0]),
-			"confidence": fmt.Sprint(vals[1]),
-			"count":      asInt(vals[2]),
-		})
-	}
-	return json.Marshal(map[string]any{"status": "ok", "by_confidence": rows})
-}
-
-func (HTTP) Ingest(context.Context) ([]byte, error) {
-	return json.Marshal(map[string]any{
-		"mode":    "rebuild",
-		"command": "bin/brain/index.go --rebuild",
-		"add":     "v2",
-	})
-}
-
-func asInt(v any) int64 {
-	switch n := v.(type) {
-	case int64:
-		return n
-	case int:
-		return int64(n)
-	case float64:
-		return int64(n)
-	default:
-		return 0
-	}
-}

internal/brain/search.go

@@ -51,13 +51,25 @@ func runSearch(args []string) int {
 	}
 	defer closeBrain()
 
-	hits, err := searchHits(query, root, repo, limit)
+	emb, err := embedQuery(query)
 	if err != nil {
-		fmt.Fprintf(os.Stderr, "search: %v\n", err)
+		fmt.Fprintf(os.Stderr, "embed: %v\n", err)
 		return 1
 	}
 
-	results := hits
+	fts, err := queryFTS(query, limit*3)
+	if err != nil {
+		fmt.Fprintf(os.Stderr, "fts: %v\n", err)
+		return 1
+	}
+
+	var vec []Hit
+	if vec, err = queryVector(emb, limit*3); err != nil {
+		fmt.Fprintf(os.Stderr, "vec: %v\n", err)
+	}
+
+	results := rank.RankAndFilter(fts, vec, root, repo, limit)
+
 	for i := range results {
 		if results[i].Text != "" {
 			runes := []rune(results[i].Text)
@@ -85,22 +97,6 @@ func runSearch(args []string) int {
 	return 0
 }
 
-func searchHits(query, root, repo string, limit int) ([]Hit, error) {
-	emb, err := embedQuery(query)
-	if err != nil {
-		return nil, fmt.Errorf("embed: %w", err)
-	}
-	fts, err := queryFTS(query, limit*3)
-	if err != nil {
-		return nil, fmt.Errorf("fts: %w", err)
-	}
-	var vec []Hit
-	if vec, err = queryVector(emb, limit*3); err != nil {
-		fmt.Fprintf(os.Stderr, "vec: %v\n", err)
-	}
-	return rank.RankAndFilter(fts, vec, root, repo, limit), nil
-}
-
 func b2i(err error) int {
 	if err != nil {
 		return 1

internal/httpapi/server.go

@@ -1,10 +1,10 @@
-// Package httpapi serves the 2dph brain over HTTP.
+// Package server serves the 2dph brain over HTTP.
 //
 // Async by design: every request runs on its own goroutine, and CPU-heavy
-// searches are serialized through a bounded worker pool so N requests can't
-// spawn N backends at once.
+// searches are serialized through a bounded worker pool (a counting
+// semaphore) so N requests can't spawn N search processes at once.
 //
-// Used by bin/brain/serve.go. Tests inject a fake API (no exec, no ladybug).
+// Used by bin/brain/serve.go.
 package httpapi
 
 import (
@@ -21,45 +21,30 @@ import (
 	"time"
 )
 
-// API is the in-process brain surface. Production serve.go wires internal/brain.
-type API interface {
+type Searcher interface {
 	Search(ctx context.Context, query string, limit int) ([]byte, error)
-	Get(ctx context.Context, id string, body bool) ([]byte, error)
-	Stats(ctx context.Context) ([]byte, error)
-	Audit(ctx context.Context) ([]byte, error)
-	Ingest(ctx context.Context) ([]byte, error)
 }
 
 type Server struct {
-	api       API
+	searcher  Searcher
 	semaphore chan struct{}
 }
 
 const defaultPort = 8630
 
-var errUnimplemented = errors.New("not implemented")
-
-func NewServer(api API, workers int) http.Handler {
+func NewServer(searcher Searcher, workers int) http.Handler {
 	return &Server{
-		api:       api,
+		searcher:  searcher,
 		semaphore: make(chan struct{}, workers),
 	}
 }
 
 func (s *Server) ServeHTTP(w http.ResponseWriter, r *http.Request) {
-	switch r.URL.Path {
-	case "/health":
+	switch {
+	case r.URL.Path == "/health":
 		writeJSON(w, http.StatusOK, map[string]any{"status": "ok"})
-	case "/search":
+	case r.URL.Path == "/search":
 		s.handleSearch(w, r)
-	case "/get":
-		s.handleGet(w, r)
-	case "/stats":
-		s.handleJSON(w, r, s.api.Stats)
-	case "/audit":
-		s.handleJSON(w, r, s.api.Audit)
-	case "/ingest":
-		s.handleJSON(w, r, s.api.Ingest)
 	default:
 		writeJSON(w, http.StatusNotFound, map[string]any{"error": "not found"})
 	}
@@ -80,56 +65,19 @@ func (s *Server) handleSearch(w http.ResponseWriter, r *http.Request) {
 		}
 		limit = n
 	}
-	if !s.acquire(w, r) {
-		return
-	}
-	defer s.release()
-	body, err := s.api.Search(r.Context(), q, limit)
-	writeAPI(w, body, err)
-}
 
-func (s *Server) handleGet(w http.ResponseWriter, r *http.Request) {
-	id := strings.TrimSpace(r.URL.Query().Get("id"))
-	if id == "" {
-		writeJSON(w, http.StatusBadRequest, map[string]any{"error": "id required"})
-		return
-	}
-	body := r.URL.Query().Get("body") == "1" || r.URL.Query().Get("body") == "true"
-	if !s.acquire(w, r) {
-		return
-	}
-	defer s.release()
-	out, err := s.api.Get(r.Context(), id, body)
-	writeAPI(w, out, err)
-}
-
-func (s *Server) handleJSON(w http.ResponseWriter, r *http.Request, fn func(context.Context) ([]byte, error)) {
-	if !s.acquire(w, r) {
-		return
-	}
-	defer s.release()
-	body, err := fn(r.Context())
-	writeAPI(w, body, err)
-}
-
-func (s *Server) acquire(w http.ResponseWriter, r *http.Request) bool {
+	// Worker pool: block until a slot frees, so burst concurrency still
+	// bounds memory (no unbounded python processes).
 	select {
 	case s.semaphore <- struct{}{}:
-		return true
+		defer func() { <-s.semaphore }()
 	case <-r.Context().Done():
-		return false
-	}
+		return
 	}
 
-func (s *Server) release() { <-s.semaphore }
-
-func writeAPI(w http.ResponseWriter, body []byte, err error) {
+	body, err := s.searcher.Search(r.Context(), q, limit)
 	if err != nil {
-		code := http.StatusBadGateway
-		if errors.Is(err, errUnimplemented) {
-			code = http.StatusNotImplemented
-		}
-		writeJSON(w, code, map[string]any{"error": err.Error()})
+		writeJSON(w, http.StatusGatewayTimeout, map[string]any{"error": err.Error()})
 		return
 	}
 	writeRaw(w, http.StatusOK, body)
@@ -147,20 +95,17 @@ func writeRaw(w http.ResponseWriter, code int, body []byte) {
 	w.Write(body)
 }
 
-// ExecSearcher shells out to var/bin/brain-search. Fallback when the serve
-// binary is built without ladybug cgo (CI / tags=brain_serve only).
-type ExecSearcher struct {
-	CmdPath string
-	Timeout time.Duration
+// brainSearcher shells out to the Go brain-search binary (not Python).
+// A single search is bounded and short-lived; the worker pool keeps at most N live.
+type brainSearcher struct {
+	cmdPath string
+	timeout time.Duration
 }
 
-func (b ExecSearcher) Search(ctx context.Context, query string, limit int) ([]byte, error) {
-	if b.Timeout == 0 {
-		b.Timeout = 60 * time.Second
-	}
-	ctx, cancel := context.WithTimeout(ctx, b.Timeout)
+func (b *brainSearcher) Search(ctx context.Context, query string, limit int) ([]byte, error) {
+	ctx, cancel := context.WithTimeout(ctx, b.timeout)
 	defer cancel()
-	cmd := exec.CommandContext(ctx, b.CmdPath, "--json", "-n", strconv.Itoa(limit), query)
+	cmd := exec.CommandContext(ctx, b.cmdPath, "--json", "-n", strconv.Itoa(limit), query)
 	out, err := cmd.Output()
 	if err != nil {
 		var exitErr *exec.ExitError
@@ -172,18 +117,6 @@ func (b ExecSearcher) Search(ctx context.Context, query string, limit int) ([]by
 	return out, nil
 }
 
-func (ExecSearcher) Get(context.Context, string, bool) ([]byte, error) {
-	return nil, errUnimplemented
-}
-func (ExecSearcher) Stats(context.Context) ([]byte, error) { return nil, errUnimplemented }
-func (ExecSearcher) Audit(context.Context) ([]byte, error) { return nil, errUnimplemented }
-func (ExecSearcher) Ingest(context.Context) ([]byte, error) {
-	return json.Marshal(map[string]any{
-		"mode":    "rebuild",
-		"command": "bin/brain/index.go --rebuild",
-	})
-}
-
 func defaultSearchCmd(root string) string {
 	if env := os.Getenv("KB_SEARCH_CMD"); env != "" {
 		return env
@@ -191,7 +124,11 @@ func defaultSearchCmd(root string) string {
 	return filepath.Join(root, "var", "bin", "brain-search")
 }
 
-func workersAndPort() (int, int) {
+// Run starts the HTTP server. Reads env: KB_SEARCH_CMD (default
+// $KB_ROOT/var/bin/brain-search), KB_WORKERS (default 4), KB_PORT (default 8630).
+func Run() {
+	root := os.Getenv("KB_ROOT")
+	searchPath := defaultSearchCmd(root)
 	workers := 4
 	if raw := os.Getenv("KB_WORKERS"); raw != "" {
 		if n, err := strconv.Atoi(raw); err == nil && n > 0 {
@@ -204,19 +141,11 @@ func workersAndPort() (int, int) {
 			port = n
 		}
 	}
-	return workers, port
-}
 
-// Run starts the HTTP server with an injected API (in-process brain, or ExecSearcher).
-func Run(api API) {
-	if api == nil {
-		root := os.Getenv("KB_ROOT")
-		api = ExecSearcher{CmdPath: defaultSearchCmd(root), Timeout: 60 * time.Second}
-	}
-	workers, port := workersAndPort()
-	handler := NewServer(api, workers)
+	searcher := &brainSearcher{cmdPath: searchPath, timeout: 60 * time.Second}
+	handler := NewServer(searcher, workers)
 	addr := "127.0.0.1:" + strconv.Itoa(port)
-	log.Printf("serve: %s (workers=%d)", addr, workers)
+	log.Printf("serve: %s (workers=%d cmd=%s)", addr, workers, searchPath)
 	if err := http.ListenAndServe(addr, handler); err != nil {
 		log.Fatal(err)
 	}

internal/httpapi/server_test.go

@@ -5,7 +5,6 @@ import (
 	"encoding/json"
 	"net/http"
 	"net/http/httptest"
-	"os"
 	"strings"
 	"sync"
 	"sync/atomic"
@@ -48,26 +47,6 @@ func (f *fakeSearcher) Search(ctx context.Context, query string, limit int) ([]b
 	return []byte(`{"query":"` + query + `","count":0,"results":[]}`), nil
 }
 
-func (f *fakeSearcher) Get(_ context.Context, id string, body bool) ([]byte, error) {
-	out := map[string]any{"id": id, "root": "info"}
-	if body {
-		out["text"] = "fake body"
-	}
-	return json.Marshal(out)
-}
-
-func (f *fakeSearcher) Stats(context.Context) ([]byte, error) {
-	return []byte(`{"total":0,"by_root":{}}`), nil
-}
-
-func (f *fakeSearcher) Audit(context.Context) ([]byte, error) {
-	return []byte(`{"status":"ok"}`), nil
-}
-
-func (f *fakeSearcher) Ingest(context.Context) ([]byte, error) {
-	return []byte(`{"mode":"rebuild","command":"bin/brain/index.go --rebuild"}`), nil
-}
-
 func (f *fakeSearcher) count() int {
 	f.mu.Lock()
 	defer f.mu.Unlock()
@@ -163,47 +142,6 @@ func TestSearchRejectsBadLimit(t *testing.T) {
 	}
 }
 
-func TestGetLeaf(t *testing.T) {
-	fs := &fakeSearcher{callback: func(q string, limit int) ([]byte, error) {
-		return []byte(`{}`), nil
-	}}
-	h := NewServer(fs, 1)
-	if code, _ := get(t, h, "/get"); code != http.StatusBadRequest {
-		t.Fatalf("missing id code = %d, want 400", code)
-	}
-	code, body := get(t, h, "/get?id=leaf-1&body=1")
-	if code != http.StatusOK {
-		t.Fatalf("get code = %d, want 200 body=%s", code, body)
-	}
-	if !strings.Contains(string(body), "leaf-1") {
-		t.Fatalf("get body %s missing id", body)
-	}
-}
-
-func TestStatsAuditIngest(t *testing.T) {
-	h := NewServer(&fakeSearcher{}, 1)
-	for _, path := range []string{"/stats", "/audit", "/ingest"} {
-		code, body := get(t, h, path)
-		if code != http.StatusOK {
-			t.Fatalf("%s code = %d, want 200 (%s)", path, code, body)
-		}
-		if !json.Valid(body) {
-			t.Fatalf("%s body not json: %s", path, body)
-		}
-	}
-}
-
-func TestHTTPPackageDoesNotExecPython(t *testing.T) {
-	raw, err := os.ReadFile("server.go")
-	if err != nil {
-		t.Fatal(err)
-	}
-	lower := strings.ToLower(string(raw))
-	if strings.Contains(lower, "python3") || strings.Contains(lower, "bin/kb/search") {
-		t.Fatal("httpapi must not exec Python or bin/kb/search")
-	}
-}
-
 func TestDefaultSearchCmdIsBrainNotPython(t *testing.T) {
 	t.Setenv("KB_SEARCH_CMD", "")
 	cmd := defaultSearchCmd("/repo")