reintroduce module manager, again!

Author: systemshift

internal/memex/commands.go

@@ -10,6 +10,7 @@ import (
 
 	"github.com/systemshift/memex/internal/memex/core"
 	"github.com/systemshift/memex/internal/memex/migration"
+	"github.com/systemshift/memex/internal/memex/modules"
 	"github.com/systemshift/memex/internal/memex/repository"
 )
 
@@ -34,7 +35,7 @@ func ModuleCommand(args ...string) error {
 	switch cmd {
 	case "list":
 		// List installed modules
-		modules := ListModules()
+		modules := modules.ListModules()
 		if len(modules) == 0 {
 			fmt.Println("No modules installed")
 			return nil
@@ -85,17 +86,9 @@ func ModuleCommand(args ...string) error {
 			return fmt.Errorf("install-dev requires module path")
 		}
 		modulePath := args[1]
-		fullPath := fmt.Sprintf("github.com/systemshift/%s", modulePath)
-
-		// Add require directive first
-		cmd := exec.Command("go", "mod", "edit", "-require", fmt.Sprintf("%[email protected]", fullPath))
-		cmd.Dir = "."
-		if err := cmd.Run(); err != nil {
-			return fmt.Errorf("adding require directive: %w", err)
-		}
 
-		// Add replace directive
-		cmd = exec.Command("go", "mod", "edit", "-replace", fmt.Sprintf("%s=./%s", fullPath, modulePath))
+		// Add replace directive to use local module
+		cmd := exec.Command("go", "mod", "edit", "-replace", fmt.Sprintf("github.com/systemshift/%s=./%s", modulePath, modulePath))
 		cmd.Dir = "."
 		if err := cmd.Run(); err != nil {
 			return fmt.Errorf("adding replace directive: %w", err)

internal/memex/module/loader.go

@@ -0,0 +1,26 @@
+package module
+
+import (
+	"fmt"
+	"os/exec"
+)
+
+// LoadModule loads a module from a path
+func LoadModule(modulePath string) error {
+	// Add replace directive to use local module
+	cmd := exec.Command("go", "mod", "edit", "-replace", fmt.Sprintf("github.com/systemshift/%s=./%s", modulePath, modulePath))
+	cmd.Dir = "."
+	if err := cmd.Run(); err != nil {
+		return fmt.Errorf("adding replace directive: %w", err)
+	}
+
+	// Run go mod tidy to download dependencies
+	cmd = exec.Command("go", "mod", "tidy")
+	cmd.Dir = "."
+	if err := cmd.Run(); err != nil {
+		return fmt.Errorf("tidying modules: %w", err)
+	}
+
+	// The module's init() function will register it when imported
+	return nil
+}

internal/memex/modules/registry.go

@@ -0,0 +1,60 @@
+package modules
+
+import (
+	"fmt"
+
+	"github.com/systemshift/memex/internal/memex/core"
+)
+
+// Registry manages registered modules
+type Registry struct {
+	modules map[string]core.Module
+}
+
+// NewRegistry creates a new registry
+func NewRegistry() *Registry {
+	return &Registry{
+		modules: make(map[string]core.Module),
+	}
+}
+
+// RegisterModule registers a module
+func (r *Registry) RegisterModule(m core.Module) error {
+	if _, exists := r.modules[m.ID()]; exists {
+		return fmt.Errorf("module already registered: %s", m.ID())
+	}
+	r.modules[m.ID()] = m
+	return nil
+}
+
+// GetModule returns a module by ID
+func (r *Registry) GetModule(id string) (core.Module, bool) {
+	m, exists := r.modules[id]
+	return m, exists
+}
+
+// ListModules returns all registered modules
+func (r *Registry) ListModules() []core.Module {
+	modules := make([]core.Module, 0, len(r.modules))
+	for _, m := range r.modules {
+		modules = append(modules, m)
+	}
+	return modules
+}
+
+// Global registry instance
+var DefaultRegistry = NewRegistry()
+
+// Helper functions that use the default registry
+
+func RegisterModule(m core.Module) error {
+	return DefaultRegistry.RegisterModule(m)
+}
+
+func GetModule(id string) (core.Module, bool) {
+	return DefaultRegistry.GetModule(id)
+}
+
+func ListModules() []core.Module {
+	return DefaultRegistry.ListModules()
+}

pkg/memex/registry.go

@@ -0,0 +1,29 @@
+// Package memex provides the public API for memex
+package memex
+
+import (
+	"github.com/systemshift/memex/internal/memex/core"
+	"github.com/systemshift/memex/internal/memex/modules"
+)
+
+// Re-export core types
+type (
+	Command    = core.Command
+	Module     = core.Module
+	Repository = core.Repository
+)
+
+// RegisterModule registers a module with memex
+func RegisterModule(m Module) error {
+	return modules.RegisterModule(m)
+}
+
+// GetModule returns a module by ID
+func GetModule(id string) (Module, bool) {
+	return modules.GetModule(id)
+}
+
+// ListModules returns all registered modules
+func ListModules() []Module {
+	return modules.ListModules()
+}

pkg/module/module.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 
 	"github.com/systemshift/memex/internal/memex/core"
+	"github.com/systemshift/memex/internal/memex/modules"
 )
 
 // Re-export core types for module authors
@@ -16,6 +17,11 @@ type (
 	Repository = core.Repository
 )
 
+// Register registers a module with memex
+func Register(m Module) error {
+	return modules.RegisterModule(m)
+}
+
 // Base provides a base implementation of Module interface
 type Base struct {
 	id          string