From a5dd40a13218fed3606eef38d5fef53d248b8f41 Mon Sep 17 00:00:00 2001 From: hems Date: Sun, 13 Jul 2025 05:59:14 +0100 Subject: [PATCH] revert: use simple solution for fetch content - return empty string MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- .../components/dialogs/permissions/permissions.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/internal/tui/components/dialogs/permissions/permissions.go b/internal/tui/components/dialogs/permissions/permissions.go index 9c07999e25e0b95b3f97735038637c0ecdfd3681..6d9558b6bb3704c4e488f36cc5cebd1fb95d00c9 100644 --- a/internal/tui/components/dialogs/permissions/permissions.go +++ b/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 "" }