Skip to content

Remove Gemini hooks setup from entire enable by default #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
peyton-alt merged 1 commit into from
Jan 21, 2026
Merged

Remove Gemini hooks setup from entire enable by default #81

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 0 additions & 44 deletions cmd/entire/cli/setup.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down Expand Up @@ -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 {
Expand Down Expand Up @@ -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 {
Expand Down