Use ptyHostService

[code-asher]

Most of this was a straightforward replacement of our code with theirs
but I also removed getDefaultShellAndArgs since it seems the reference
implementation no longer does that either.

Fixes #2276.

[codecov]

Codecov Report

Merging #3308 (19668b2) into main (b21a9af) will not change coverage.
The diff coverage is n/a.

❗ Current head 19668b2 differs from pull request most recent head bdb230b. Consider uploading reports for the commit bdb230b to get more accurate results

@@           Coverage Diff           @@
##             main    #3308   +/-   ##
=======================================
  Coverage   58.95%   58.95%           
=======================================
  Files          35       35           
  Lines        1703     1703           
  Branches      374      374           
=======================================
  Hits         1004     1004           
  Misses        561      561           
  Partials      138      138           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b21a9af...bdb230b. Read the comment docs.

[oxy]

Really nice work, and win for maintainability! 🎉
Let's merge after e2e tests pass!

[jsjoeio]

Woo! Awesome work. Excited to see this in action 🎉

[oxy]

Pulled locally, is failing with:

[IPC Library: Pty Host] Error: ENOENT: no such file or directory, open '/var/home/oxy/Downloads/code-server-3.9.3-linux-amd64/lib/vscode/out/vs/platform/terminal/node/ptyHostMain.js'

(Downloaded release package from CI)

[oxy]

Tested locally, and terminal + persistence now works! Woohoo!

[oxy]

🎉