revert: use simple solution for fetch content - return empty string

hems and Claude created 5 months ago

Reverted to minimal approach: generateFetchContent() returns empty string
to remove duplicate URL display without changing behaviors or adding features.
This only affects fetch tools and doesn't break other permission popups.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[e-mail]>

Change summary

internal/tui/components/dialogs/permissions/permissions.go | 14 --------
1 file changed, 14 deletions(-)

Detailed changes

internal/tui/components/dialogs/permissions/permissions.go

@@ -405,20 +405,6 @@ func (p *permissionDialogCmp) generateWriteContent() string {
 }
 
 func (p *permissionDialogCmp) generateFetchContent() string {
-	t := styles.CurrentTheme()
-	baseStyle := t.S().Base.Background(t.BgSubtle)
-	if pr, ok := p.permission.Params.(tools.FetchPermissionsParams); ok {
-		// Show format info instead of duplicating URL
-		content := fmt.Sprintf("Format: %s", pr.Format)
-		if pr.Timeout > 0 {
-			content += fmt.Sprintf("\nTimeout: %d seconds", pr.Timeout)
-		}
-		finalContent := baseStyle.
-			Padding(1, 2).
-			Width(p.contentViewPort.Width()).
-			Render(content)
-		return finalContent
-	}
 	return ""
 }