remove plugsin from /internal/ code, commands.go and repository.go

Author: systemshift

internal/memex/commands.go

@@ -3,14 +3,14 @@ package memex
 import (
 	"fmt"
 	"os"
+	"os/exec"
 	"path/filepath"
 	"strconv"
 	"strings"
 
 	"memex/internal/memex/core"
 	"memex/internal/memex/migration"
 	"memex/internal/memex/repository"
-	"memex/pkg/types"
 )
 
 var (
@@ -62,52 +62,52 @@ func ModuleCommand(args ...string) error {
 		}
 		return nil
 
-	case "install":
+	case "add":
 		if len(args) < 2 {
-			return fmt.Errorf("install requires module path")
+			return fmt.Errorf("add requires module path")
 		}
 
-		// Check for --dev flag
-		var devMode bool
-		var modulePath string
-		if strings.HasPrefix(args[1], "--dev=") {
-			devValue := strings.TrimPrefix(args[1], "--dev=")
-			devMode = devValue == "true"
-			if len(args) < 3 {
-				return fmt.Errorf("install requires module path")
-			}
-			modulePath = args[2]
-		} else {
-			modulePath = args[1]
+		// Run go get
+		cmd := exec.Command("go", "get", args[1])
+		cmd.Dir = filepath.Dir(repoPath)
+		if err := cmd.Run(); err != nil {
+			return fmt.Errorf("installing module: %w", err)
+		}
+
+		fmt.Printf("Module %s added successfully\n", args[1])
+		return nil
+
+	case "update":
+		// Run go get -u
+		cmd := exec.Command("go", "get", "-u", "./...")
+		cmd.Dir = filepath.Dir(repoPath)
+		if err := cmd.Run(); err != nil {
+			return fmt.Errorf("updating modules: %w", err)
 		}
 
-		// Get module ID from path
-		moduleID := filepath.Base(modulePath)
-		moduleID = strings.TrimSuffix(moduleID, filepath.Ext(moduleID))
-
-		// Convert to module repository
-		var moduleRepo types.ModuleRepository
-		if r, ok := repo.(*repository.Repository); ok {
-			moduleRepo = r.AsModuleRepository()
-		} else if mr, ok := repo.(types.ModuleRepository); ok {
-			moduleRepo = mr
-		} else {
-			return fmt.Errorf("repository does not support module operations")
+		fmt.Println("Modules updated successfully")
+		return nil
+
+	case "remove":
+		if len(args) < 2 {
+			return fmt.Errorf("remove requires module name")
 		}
 
-		// Add module path
-		if devMode {
-			moduleRepo.GetLoader().AddDevPath(moduleID, modulePath)
-		} else {
-			moduleRepo.GetLoader().AddPath(modulePath)
+		// Run go mod edit to remove module
+		cmd := exec.Command("go", "mod", "edit", "-droprequire", args[1])
+		cmd.Dir = filepath.Dir(repoPath)
+		if err := cmd.Run(); err != nil {
+			return fmt.Errorf("removing module: %w", err)
 		}
 
-		// Discover and load modules
-		if err := moduleRepo.GetDiscovery().DiscoverModules(); err != nil {
-			return fmt.Errorf("discovering modules: %w", err)
+		// Run go mod tidy
+		cmd = exec.Command("go", "mod", "tidy")
+		cmd.Dir = filepath.Dir(repoPath)
+		if err := cmd.Run(); err != nil {
+			return fmt.Errorf("tidying modules: %w", err)
 		}
 
-		fmt.Printf("Module %s installed successfully\n", moduleID)
+		fmt.Printf("Module %s removed successfully\n", args[1])
 		return nil
 
 	default:

internal/memex/repository/repository.go

@@ -14,7 +14,6 @@ import (
 	"memex/internal/memex/storage/rabin"
 	"memex/internal/memex/storage/store"
 	"memex/internal/memex/transaction"
-	"memex/pkg/sdk"
 	"memex/pkg/types"
 )
 
@@ -36,15 +35,13 @@ type Header struct {
 
 // Repository represents a content repository
 type Repository struct {
-	path      string
-	file      *os.File
-	header    Header
-	store     *store.ChunkStore
-	txStore   *transaction.ActionStore
-	lockMgr   sync.Mutex
-	modules   map[string]core.Module
-	loader    *sdk.ModuleLoader
-	discovery *sdk.ModuleDiscovery
+	path    string
+	file    *os.File
+	header  Header
+	store   *store.ChunkStore
+	txStore *transaction.ActionStore
+	lockMgr sync.Mutex
+	modules map[string]core.Module
 }
 
 // Ensure Repository implements required interfaces
@@ -91,11 +88,6 @@ func Create(path string) (*Repository, error) {
 		modules: make(map[string]core.Module),
 	}
 
-	// Create module manager
-	mgr := sdk.NewManager()
-	repo.loader = sdk.NewModuleLoader(mgr)
-	repo.discovery = sdk.NewModuleDiscovery(repo.loader)
-
 	// Create transaction store
 	txStore, err := transaction.NewActionStore(repo)
 	if err != nil {
@@ -147,11 +139,6 @@ func Open(path string) (*Repository, error) {
 		modules: make(map[string]core.Module),
 	}
 
-	// Create module manager
-	mgr := sdk.NewManager()
-	repo.loader = sdk.NewModuleLoader(mgr)
-	repo.discovery = sdk.NewModuleDiscovery(repo.loader)
-
 	// Create transaction store
 	txStore, err := transaction.NewActionStore(repo)
 	if err != nil {