Terminal overwrite text on overflow #5092

Activity

1. Vrajs16

   added

   needs-triage🔍

   on Aug 15, 2026
2. linear-code commented on Aug 15, 2026

   

   linear-codeBot

   on Aug 15, 2026 – with Linear Code

   More actions

   DOK-642
3. dosubot

   added

   bugSomething isn't workingSomething isn't working

   on Aug 15, 2026
4. dosubot commented on Aug 15, 2026

   

   dosubotBot

   on Aug 15, 2026 – with DosuBot

   More actions

   Hi @Vrajs16! This is a terminal sizing issue with xterm.js — the FitAddon calculates the terminal's column count once at mount time via addonFit.fit() [1], but there's no ResizeObserver or window resize handler to recalculate when the container dimensions change. This means the cols value goes stale, so the cursor position wraps incorrectly and new characters overwrite existing text instead of advancing.

   A closed PR (#1135) addressed exactly this — it added a ResizeObserver to re-call addonFit.fit() on container resize and synchronized the updated cols/rows to the backend PTY via WebSocket [2]. It was never merged, but it serves as a good reference.

   To fix this in docker-terminal.tsx, you'd need to:

   1. Add a ResizeObserver that calls addonFit.fit() when the container size changes.
   2. Send the new dimensions to the server via the WebSocket so the backend PTY resizes to match.

   A minimal client-side addition would look like:

   const resizeObserver = new ResizeObserver(() => {
     addonFit.fit();
   });
   resizeObserver.observe(termRef.current!);
   
   return () => {
     resizeObserver.disconnect();
     ws.readyState === WebSocket.OPEN && ws.close();
     term.dispose();
   };

   The server-side (docker-container-terminal.ts) would also need to handle a resize message to call pty.resize(cols, rows). PR #1135's diff has the full implementation you can use as a starting point [2].

   To reply, just mention @dosu.

   ---

   Share context across your team and agents. Try Dosu.

   
5. Siumauricio

   added a commit that references this issue on Aug 17, 2026

   Merge pull request #5104 from Dokploy/fix/issue-5092-terminal-resize-…

   ...

   This issue will close once commit b53ffac is merged into the 'canary' branch.

   b53ffac
6. Siumauricio

   closed this as completedin #5104on Aug 17, 2026
7. Siumauricio

   removed

   needs-triage🔍

   on Aug 17, 2026
8. pull

   added 2 commits that reference this issue on Aug 17, 2026

   fix: sync terminal size with backend PTY, avoid connecting with place…

   ...

   This issue will close once commit ff275fe is merged into the 'canary' branch.

   ff275fe

   Merge pull request Dokploy#5104 from Dokploy/fix/issue-5092-terminal-…

   ...

   This issue will close once commit c0265b4 is merged into the 'canary' branch.

   c0265b4
9. Abhash-Chakraborty

   added a commit that references this issue on Aug 23, 2026

   Merge upstream v0.30.2 into main

   ...

   925e77a
10. github-actions

    added a commit that references this issue on Oct 12, 2026

    chore(release): 0.42.0 [skip ci]

    ...

    63ce614