refactor: drop restate-the-code comments from open/clipboard files

Author: dlevy-msft-sql

cmd/modern/root/open/clipboard.go

@@ -11,23 +11,20 @@ import (
 	"github.com/microsoft/go-sqlcmd/internal/pal"
 )
 
-// copyPasswordToClipboard copies the password for the current context to the clipboard
-// if the user is using SQL authentication. Returns true if a password was copied.
+// copyPasswordToClipboard copies the current context's password to the clipboard for SQL auth.
+// Returns true when a password was actually copied.
 func copyPasswordToClipboard(user *sqlconfig.User, out *output.Output) bool {
 	if out == nil || user == nil || user.AuthenticationType != "basic" || user.BasicAuth == nil {
 		return false
 	}
 
-	// Get the decrypted password from the current context
 	_, _, password := config.GetCurrentContextInfo()
-
 	if password == "" {
 		return false
 	}
 
-	err := pal.CopyToClipboard(password)
-	if err != nil {
-		// Don't fail the command if clipboard copy fails, just warn the user
+	if err := pal.CopyToClipboard(password); err != nil {
+		// Don't fail the launch over a clipboard hiccup; warn and continue.
 		out.Warn(localizer.Sprintf("Could not copy password to clipboard: %s", err.Error()))
 		return false
 	}

cmd/modern/root/open/ssms_windows.go

@@ -7,7 +7,6 @@ import (
 	"github.com/microsoft/go-sqlcmd/internal/localizer"
 )
 
-// On Windows, display info before launching
 func (c *SSMS) displayPreLaunchInfo() {
 	output := c.Output()
 	output.Info(localizer.Sprintf("Launching SQL Server Management Studio..."))

cmd/modern/root/open/vscode.go

@@ -55,9 +55,6 @@ func (c *VSCode) DefineCommand(...cmdparser.CommandOptions) {
 	})
 }
 
-// Launch VS Code and configure connection profile for the current context.
-// The connection profile will be added to VS Code's user settings to work
-// with the MSSQL extension.
 func (c *VSCode) run() {
 	if config.CurrentContextName() == "" {
 		c.Output().FatalWithHintExamples([][]string{
@@ -156,7 +153,6 @@ func (c *VSCode) launchVSCode(t tool.Tool, endpoint sqlconfig.Endpoint, user *sq
 	c.CheckErr(err)
 }
 
-// createConnectionProfile creates or updates a connection profile in VS Code's user settings
 func (c *VSCode) createConnectionProfile(build string, endpoint sqlconfig.Endpoint, user *sqlconfig.User, isLocalConnection bool) {
 	output := c.Output()
 

internal/pal/clipboard.go

@@ -3,8 +3,7 @@
 
 package pal
 
-// CopyToClipboard copies the given text to the system clipboard.
-// Returns an error if the clipboard operation fails.
+// CopyToClipboard copies text to the system clipboard.
 func CopyToClipboard(text string) error {
 	return copyToClipboard(text)
 }

internal/pal/clipboard_linux.go

@@ -10,45 +10,31 @@ import (
 )
 
 func copyToClipboard(text string) error {
-	// Try xclip first, then xsel, then wl-copy as fallbacks.
-	// These are common clipboard utilities on Linux.
-
 	var attempts []string
-
-	// Helper to try a single command and record any errors.
 	tryCmd := func(name string, args ...string) bool {
 		if _, err := exec.LookPath(name); err != nil {
 			attempts = append(attempts, fmt.Sprintf("%s not found", name))
 			return false
 		}
-
 		cmd := exec.Command(name, args...)
 		cmd.Stdin = strings.NewReader(text)
 		if err := cmd.Run(); err != nil {
 			attempts = append(attempts, fmt.Sprintf("%s failed: %v", name, err))
 			return false
 		}
-
 		return true
 	}
 
-	// Try xclip
 	if tryCmd("xclip", "-selection", "clipboard") {
 		return nil
 	}
-
-	// Try xsel as fallback
 	if tryCmd("xsel", "--clipboard", "--input") {
 		return nil
 	}
-
-	// Try wl-copy for Wayland
 	if tryCmd("wl-copy") {
 		return nil
 	}
 
-	// All attempts failed - return combined error message. Plain fmt.Errorf
-	// because gotext only scans cmd/modern, cmd/sqlcmd, and pkg/sqlcmd; calling
-	// localizer.Errorf from internal/pal would never make it into the catalog.
+	// gotext only scans cmd/modern, cmd/sqlcmd, and pkg/sqlcmd, so this stays plain fmt.Errorf.
 	return fmt.Errorf("failed to copy to clipboard; tried xclip, xsel, wl-copy: %s", strings.Join(attempts, "; "))
 }

internal/pal/clipboard_windows.go

@@ -10,8 +10,6 @@ import (
 	"strings"
 )
 
-// copyToClipboard copies text to the Windows clipboard using the built-in clip.exe command.
-// This is simpler and safer than using Win32 API calls directly.
 func copyToClipboard(text string) error {
 	cmd := exec.Command(clipExePath())
 	cmd.Stdin = strings.NewReader(text)

internal/tools/tool/tool.go

@@ -48,7 +48,6 @@ func (t *tool) IsInstalled() bool {
 	}
 
 	t.installed = new(bool)
-	// Handle case where tool wasn't found during Init (exeName is empty)
 	if t.exeName != "" && file.Exists(t.exeName) {
 		*t.installed = true
 	} else {