v0.56.1 — Fix mcpclient tests: build fakeserver from source instead of using pre-compiled binary

The committed fakeserver binary was ELF x86-64 (Linux), causing
"exec format error" on macOS ARM64. Fixed by compiling the fake
MCP server from testdata/main.go on-the-fly at test time.

Changes:
- internal/mcpclient/client_test.go: fakeServerPath now runs go build
- cmd/odek/mcp_e2e_test.go: fakeMCPPath now runs go build
- cmd/odek/subagent_e2e_test.go: fix hardcoded /root/projects/odek paths
- Remove pre-compiled binary (5.8 MB) from git
- Add fakeserver to .gitignore

Author: jkyberneees

.gitignore

@@ -6,3 +6,4 @@ odek-bin
 .plans/
 .hermes/
 notify-channel-proposal.md
+internal/mcpclient/testdata/fakeserver

cmd/odek/mcp_e2e_test.go

@@ -3,7 +3,9 @@ package main
 import (
 	"context"
 	"os"
+	"os/exec"
 	"path/filepath"
+	"runtime"
 	"testing"
 
 	"github.com/BackendStack21/odek/internal/mcpclient"
@@ -15,11 +17,21 @@ import (
 // them, and verify the full ToolAdapter round-trip. Gated by ODEK_E2E=true
 // since they spawn external processes.
 
-// fakeMCPPath returns the path to the pre-compiled fake MCP server binary.
-// The test runs inside cmd/odek/, so we go up two levels to reach the repo root.
+// fakeMCPPath compiles the fake MCP server from source on-the-fly
+// and returns the path to the compiled binary.
 func fakeMCPPath(t *testing.T) string {
 	t.Helper()
-	return filepath.Join("..", "..", "internal", "mcpclient", "testdata", "fakeserver")
+	_, thisFile, _, _ := runtime.Caller(0)
+	// Navigate from cmd/odek/ to the repo root, then to internal/mcpclient/testdata/
+	repoRoot := filepath.Join(filepath.Dir(thisFile), "..", "..")
+	testdataDir := filepath.Join(repoRoot, "internal", "mcpclient", "testdata")
+	exePath := filepath.Join(t.TempDir(), "fakeserver")
+	cmd := exec.Command("go", "build", "-o", exePath, testdataDir)
+	cmd.Dir = repoRoot // run from module root so go.mod is found
+	if out, err := cmd.CombinedOutput(); err != nil {
+		t.Fatalf("build fakeserver: %v\n%s", err, out)
+	}
+	return exePath
 }
 
 func skipIfNoMCPE2E(t *testing.T) {

cmd/odek/subagent_e2e_test.go

@@ -7,6 +7,7 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
@@ -52,12 +53,16 @@ func TestMain(m *testing.M) {
 	e2eBinDir = dir
 	e2eBinary = filepath.Join(dir, ".odek")
 
+	// Resolve repo root from this source file
+	_, thisFile, _, _ := runtime.Caller(0)
+	repoRoot := filepath.Join(filepath.Dir(thisFile), "..", "..")
+
 	cmd := exec.Command("go", "build",
 		"-ldflags", "-X main.version=v0.0.0-e2e",
 		"-o", e2eBinary,
-			"/root/projects/odek/cmd/odek/",
+		filepath.Join(repoRoot, "cmd", "odek"),
 	)
-	cmd.Dir = "/root/projects/odek"
+	cmd.Dir = repoRoot
 	stderr := &bytes.Buffer{}
 	cmd.Stderr = stderr
 	if err := cmd.Run(); err != nil {

internal/mcpclient/client_test.go

@@ -4,21 +4,27 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
-	"os"
+	"os/exec"
 	"path/filepath"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
 )
 
-// fakeServerPath returns the path to the pre-compiled fake MCP server.
+// fakeServerPath compiles the fake MCP server from source on-the-fly
+// and returns the path to the compiled binary. The binary is placed in
+// t.TempDir() and cleaned up automatically when the test completes.
 func fakeServerPath(t *testing.T) string {
 	t.Helper()
-	dir, err := os.Getwd()
-	if err != nil {
-		t.Fatalf("Getwd: %v", err)
-	}
-	return filepath.Join(dir, "testdata", "fakeserver")
+	_, thisFile, _, _ := runtime.Caller(0)
+	testdataDir := filepath.Join(filepath.Dir(thisFile), "testdata")
+	exePath := filepath.Join(t.TempDir(), "fakeserver")
+	cmd := exec.Command("go", "build", "-o", exePath, testdataDir)
+	if out, err := cmd.CombinedOutput(); err != nil {
+		t.Fatalf("build fakeserver: %v\n%s", err, out)
+	}
+	return exePath
 }
 
 func TestNew_NonExistentCommand(t *testing.T) {