Address Copilot review: use SqlcmdEol and fix comment

Author: dlevy-msft-sql

pkg/sqlcmd/commands.go

@@ -210,7 +210,7 @@ func (c Commands) SetBatchTerminator(terminator string) error {
 
 // isExitParenBalanced checks if the parentheses in an EXIT command argument are balanced.
 // It tracks quotes to avoid counting parens inside string literals.
-// It handles SQL Server's quote escaping: ” inside strings and ]] inside bracket identifiers.
+// It handles SQL Server's quote escaping: ” inside single-quoted strings, "" inside double-quoted strings, and ]] inside bracket identifiers.
 func isExitParenBalanced(s string) bool {
 	depth := 0
 	var quote rune

pkg/sqlcmd/commands_test.go

@@ -516,7 +516,7 @@ func TestReadExitContinuation(t *testing.T) {
 
 		result, err := readExitContinuation(s, "(select 1")
 		assert.NoError(t, err)
-		assert.Equal(t, "(select 1\r\n+ 2)", result)
+		assert.Equal(t, "(select 1"+SqlcmdEol+"+ 2)", result)
 
 		// Verify prompt was set
 		tc := s.lineIo.(*testConsole)
@@ -560,7 +560,7 @@ func TestReadExitContinuation(t *testing.T) {
 
 		result, err := readExitContinuation(s, "(select 1")
 		assert.NoError(t, err)
-		assert.Equal(t, "(select 1\r\n+ 2\r\n+ 3\r\n)", result)
+		assert.Equal(t, "(select 1"+SqlcmdEol+"+ 2"+SqlcmdEol+"+ 3"+SqlcmdEol+")", result)
 	})
 
 	t.Run("returns immediately if already balanced", func(t *testing.T) {