fix: re-enable OPOST after setraw to fix terminal rendering

Author: V2arK

centml/cli/shell.py

@@ -143,6 +143,12 @@ async def _interactive_session(ws_url, token):
     old_settings = termios.tcgetattr(fd)
     try:
         tty.setraw(fd)
+        # setraw disables OPOST, which means \n won't be translated to \r\n.
+        # The remote PTY may send bare \n; re-enable output post-processing
+        # so the local terminal handles the translation (like xterm.js does).
+        mode = termios.tcgetattr(fd)
+        mode[1] = mode[1] | termios.OPOST
+        termios.tcsetattr(fd, termios.TCSANOW, mode)
         rows, cols = shutil.get_terminal_size()
 
         headers = {"Authorization": f"Bearer {token}"}

tests/test_shell.py

@@ -306,7 +306,13 @@ async def test_restores_terminal_on_exception(self):
         ), patch("centml.cli.shell.websockets") as mock_ws_mod:
 
             mock_sys.stdin.fileno.return_value = 0
-            mock_termios.tcgetattr.return_value = ["old_settings"]
+            # tcgetattr is called twice: once to save, once after setraw for OPOST fix.
+            # Return a realistic 7-element termios attrs list.
+            old_attrs = [0, 0, 0, 0, 0, 0, [0] * 32]
+            mock_termios.tcgetattr.return_value = old_attrs
+            mock_termios.OPOST = 0x1
+            mock_termios.TCSANOW = 0
+            mock_termios.TCSADRAIN = 1
 
             mock_ws_mod.connect = MagicMock(
                 return_value=AsyncMock(
@@ -318,9 +324,9 @@ async def test_restores_terminal_on_exception(self):
             with pytest.raises(ConnectionRefusedError):
                 await _interactive_session("wss://test/ws", "fake-token")
 
-            mock_termios.tcsetattr.assert_called_once()
+            mock_termios.tcsetattr.assert_called()
             restore_call = mock_termios.tcsetattr.call_args
-            assert restore_call[0][2] == ["old_settings"]
+            assert restore_call[0][2] == old_attrs
 
 
 # ===========================================================================