From 84cec467efb1020194e2e8913a5d4d349fbdd465 Mon Sep 17 00:00:00 2001 From: Peyton Montei Date: Tue, 20 Jan 2026 19:56:49 -0800 Subject: [PATCH] Remove Gemini hooks setup from entire enable by default, except with --agent --- cmd/entire/cli/setup.go | 44 ----------------------------------------- 1 file changed, 44 deletions(-) diff --git a/cmd/entire/cli/setup.go b/cmd/entire/cli/setup.go index 0bce47bfb..f2a223a60 100644 --- a/cmd/entire/cli/setup.go +++ b/cmd/entire/cli/setup.go @@ -158,17 +158,6 @@ func runEnableWithStrategy(w io.Writer, selectedStrategy string, localDev, _, us fmt.Fprintln(w, "✓ Claude Code hooks verified") } - // Setup Gemini CLI hooks - geminiHooksInstalled, err := setupGeminiCLIHook(localDev, forceHooks) - if err != nil { - return fmt.Errorf("failed to setup Gemini CLI hooks: %w", err) - } - if geminiHooksInstalled > 0 { - fmt.Fprintln(w, "✓ Gemini CLI hooks installed - This is a work in progress") - } else { - fmt.Fprintln(w, "✓ Gemini CLI hooks verified - This is a work in progress") - } - // Setup .entire directory dirCreated, err := setupEntireDirectory() if err != nil { @@ -312,17 +301,6 @@ func runEnableInteractive(w io.Writer, localDev, _, useLocalSettings, useProject fmt.Fprintln(w, "✓ Claude Code hooks verified") } - // Setup Gemini CLI hooks - geminiHooksInstalled, err := setupGeminiCLIHook(localDev, forceHooks) - if err != nil { - return fmt.Errorf("failed to setup Gemini CLI hooks: %w", err) - } - if geminiHooksInstalled > 0 { - fmt.Fprintln(w, "✓ Gemini CLI hooks installed - This is a work in progress") - } else { - fmt.Fprintln(w, "✓ Gemini CLI hooks verified - This is a work in progress") - } - // Setup .entire directory dirCreated, err := setupEntireDirectory() if err != nil { @@ -592,28 +570,6 @@ func setupClaudeCodeHook(localDev, forceHooks bool) (int, error) { return count, nil } -// setupGeminiCLIHook sets up Gemini CLI hooks. -// This is a convenience wrapper that uses the agent package. -// Returns the number of hooks installed (0 if already installed). -func setupGeminiCLIHook(localDev, forceHooks bool) (int, error) { - ag, err := agent.Get(agent.AgentNameGemini) - if err != nil { - return 0, fmt.Errorf("failed to get gemini agent: %w", err) - } - - hookAgent, ok := ag.(agent.HookSupport) - if !ok { - return 0, errors.New("gemini agent does not support hooks") - } - - count, err := hookAgent.InstallHooks(localDev, forceHooks) - if err != nil { - return 0, fmt.Errorf("failed to install gemini hooks: %w", err) - } - - return count, nil -} - // setupAgentHooksNonInteractive sets up hooks for a specific agent non-interactively. // If strategyName is provided, it sets the strategy; otherwise uses default. func setupAgentHooksNonInteractive(agentName, strategyName string, localDev, forceHooks, skipPushSessions, noTelemetry, disableMultisessionWarning bool) error {