mcp: depend on SlashCommandSyncer interface instead of concrete type (task 410)

2 files changed, 5 insertions, 6 deletions

diff --git a/internal/hexaimcp/run.go b/internal/hexaimcp/run.go
index 23faa08..b6fed8c 100644
--- a/internal/hexaimcp/run.go
+++ b/internal/hexaimcp/run.go
@@ -26,11 +26,11 @@ type ServerFactory func(
 	w io.Writer,
 	logger *log.Logger,
 	store promptstore.PromptStore,
-	syncer *slashcommands.Syncer,
+	syncer mcp.SlashCommandSyncer,
 ) ServerRunner
 
 // defaultServerFactory is the production server factory.
-func defaultServerFactory(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer *slashcommands.Syncer) ServerRunner {
+func defaultServerFactory(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer mcp.SlashCommandSyncer) ServerRunner {
 	return mcp.NewServer(r, w, logger, store, syncer)
 }
 
diff --git a/internal/hexaimcp/run_test.go b/internal/hexaimcp/run_test.go
index 794fa1f..3c3f9d8 100644
--- a/internal/hexaimcp/run_test.go
+++ b/internal/hexaimcp/run_test.go
@@ -15,7 +15,6 @@ import (
 	"codeberg.org/snonux/hexai/internal/appconfig"
 	"codeberg.org/snonux/hexai/internal/mcp"
 	"codeberg.org/snonux/hexai/internal/promptstore"
-	"codeberg.org/snonux/hexai/internal/slashcommands"
 )
 
 // mockServerRunner implements ServerRunner for testing
@@ -35,7 +34,7 @@ func TestFullProtocolFlow(t *testing.T) {
 	tmpDir := t.TempDir()
 
 	// Create test server factory
-	serverFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer *slashcommands.Syncer) ServerRunner {
+	serverFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer mcp.SlashCommandSyncer) ServerRunner {
 		return mcp.NewServer(r, w, logger, store, syncer)
 	}
 
@@ -283,7 +282,7 @@ func TestRun(t *testing.T) {
 	logPath := filepath.Join(tmpDir, "test.log")
 
 	// Create a mock server factory that returns immediately
-	mockFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer *slashcommands.Syncer) ServerRunner {
+	mockFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer mcp.SlashCommandSyncer) ServerRunner {
 		return &mockServerRunner{
 			runFunc: func() error {
 				return nil // Exit immediately
@@ -316,7 +315,7 @@ func TestRunWithFactory_ServerError(t *testing.T) {
 	logPath := filepath.Join(tmpDir, "test.log")
 
 	// Create a mock server factory that returns an error
-	mockFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer *slashcommands.Syncer) ServerRunner {
+	mockFactory := func(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore, syncer mcp.SlashCommandSyncer) ServerRunner {
 		return &mockServerRunner{
 			runFunc: func() error {
 				return fmt.Errorf("mock server error")