fix(integrations): prevent duplicate hooks during initialization

Author: leefowlercu

cmd/initialize/initialize.go

@@ -604,6 +604,13 @@ func setupIntegrations(integrationNames []string) ([]string, error) {
 			continue
 		}
 
+		// Skip if already enabled (prevents duplicate hooks)
+		if enabled, _ := integration.IsEnabled(); enabled {
+			fmt.Printf("  Integration %s already configured (skipped)\n", name)
+			enabledIntegrations = append(enabledIntegrations, name)
+			continue
+		}
+
 		if err := integration.Setup(binaryPath); err != nil {
 			fmt.Printf("  Warning: Failed to setup %s: %v\n", name, err)
 			continue

internal/integrations/adapters/claude/hook_adapter.go

@@ -292,10 +292,10 @@ func addOrUpdateHook(events []HookEvent, matcher, command string) []HookEvent {
 	}
 
 	if matcherIdx >= 0 {
-		// Update existing matcher
+		// Update existing matcher - look for both old and new binary names
 		hookExists := false
 		for i, hook := range events[matcherIdx].Hooks {
-			if strings.Contains(hook.Command, "agentic-memorizer") {
+			if strings.Contains(hook.Command, "memorizer") {
 				events[matcherIdx].Hooks[i] = newHook
 				hookExists = true
 				break

internal/tui/initialize/components/checkbox.go

@@ -69,11 +69,12 @@ func (c *CheckboxList) CheckedItems() []CheckboxItem {
 	return checked
 }
 
-// CheckedValues returns the values of checked items
+// CheckedValues returns the values of checked AND enabled items
+// Disabled items are excluded even if checked (they represent pre-existing state)
 func (c *CheckboxList) CheckedValues() []any {
 	var values []any
 	for _, item := range c.items {
-		if item.Checked {
+		if item.Checked && item.Enabled {
 			values = append(values, item.Value)
 		}
 	}