Make native-keymap and keytar optional dependencies

[GirlBossRush]

This PR addresses a regression introduced in our VS code fork. Previously, we removed two dependencies which required additional native packages, however they were included in the latest release. Rather than remove these and break the desktop build, we now mark them as optional: coder/vscode@07bcbc3

Fixes #4179

[codecov]

Codecov Report

Merging #4192 (b448a66) into main (7925f88) will not change coverage.
The diff coverage is n/a.

❗ Current head b448a66 differs from pull request most recent head 76a2116. Consider uploading reports for the commit 76a2116 to get more accurate results

@@           Coverage Diff           @@
##             main    #4192   +/-   ##
=======================================
  Coverage   64.22%   64.22%           
=======================================
  Files          36       36           
  Lines        1873     1873           
  Branches      379      379           
=======================================
  Hits         1203     1203           
  Misses        569      569           
  Partials      101      101           

---

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 7925f88...76a2116. Read the comment docs.

[github-actions]

✨ Coder.com for PR #4192 deployed! It will be updated on every commit.

• Host: https://codercom-6w2x46gou-codercom.vercel.app/docs/code-server
• Last deploy status: success
• Commit: 76a2116
• Workflow status: https://github.com/cdr/code-server/actions/runs/1243424517

[jsjoeio]

This looks good to me! Curious about @edvincent's comment here coder/vscode@07bcbc3#commitcomment-56613657

[jsjoeio]

🎉

[edvincent]

This looks good to me! Curious about @edvincent's comment here coder/vscode@07bcbc3#commitcomment-56613657

@jsjoeio I'm keeping an eye on the above build, to test with the NPM artifact and report back on that comment - I'm myself curious of exactly what does (or doesn't) happen on installs.

[jsjoeio]

I'm keeping an eye on the above build, to test with the NPM artifact and report back on that comment

Dang, you rock! We really appreciate the help!

[edvincent]

Yup it all seems to work as expected when pulling them as optional dependencies! 🎉

The errors are still there, but are now warnings. All the other installs still go through (instead of the install exiting because of the error). code-server seems usable and doesn't seem to be having any issue from what I can see / clicking around so far.

[jsjoeio]

Yup it all seems to work as expected when pulling them as optional dependencies! 🎉

Hooray! Thanks so much for reporting and helping us test the fix @edvincent! We really appreciate it 🙌