fixing bad term cleanup at exit

[tdaron]

This fixes #2900 by simply saving term info and registering an on-exit handler to put correct term info back at the exit

[tdaron]

CI is failing cause of some RNG, i think this is not related to this PR. Is there anything I should do about it ?

[ahopkins]

Let's not add a dependency here. Let's find a solution without it.

[tdaron]

Hi ! Thanks for your answer ! I think both termios and atexit are part of python's std library, so it should not add any dependency. But as termios is POSIX only, it requires the try except for the import. So my patch will not affect at all non-POSIX systems but AFAIK those are not concerned with the issue.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.683%. Comparing base (b51fa73) to head (875b7fa).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #3088   +/-   ##
=========================================
  Coverage   87.683%   87.683%           
=========================================
  Files          104       104           
  Lines         7778      7778           
  Branches      1236      1236           
=========================================
  Hits          6820      6820           
  Misses         662       662           
  Partials       296       296           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.