[minor] fix: recursive issue with Draft and JSONTypes

[unional]

fixes #839

I have added test under __tests__/types/type-externals.ts.

There are several things I have observed:

• the default formatter setting in vscode is not set to prettier. I have changed that so I won't mess up your formatting).
• there are some test failure from the previous PR
• the project doesn't build in Windows: yarn build -> 'cp' is not recognized as an internal or external command
• there are some circular dependency issues:

Circular dependency: src\internal.ts -> src\types\types-internal.ts -> src\internal.ts
Circular dependency: src\plugins\patches.ts -> src\immer.ts -> src\plugins\all.ts -> src\plugins\patches.ts
Circular dependency: src\internal.ts -> src\types\types-internal.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\utils\common.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\utils\plugins.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\scope.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\finalize.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\proxy.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\immerClass.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\current.ts -> src\internal.ts
Circular dependency: src\plugins\patches.ts -> src\immer.ts -> src\plugins\patches.ts
Circular dependency: src\plugins\patches.ts -> src\immer.ts -> src\plugins\all.ts -> src\plugins\patches.ts
Circular dependency: src\internal.ts -> src\types\types-internal.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\utils\common.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\utils\plugins.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\scope.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\finalize.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\proxy.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\immerClass.ts -> src\internal.ts
Circular dependency: src\internal.ts -> src\core\current.ts -> src\internal.ts
Circular dependency: src\plugins\patches.ts -> src\immer.ts -> src\plugins\patches.ts

I think these should be fixed.

Here is an video about the details: https://www.youtube.com/watch?v=PCRzWIubAEQ

[netlify]

✅ Deploy Preview for quizzical-lovelace-dcbd6a canceled.

Name	Link
🔨 Latest commit	7e2c3d1
🔍 Latest deploy log	https://app.netlify.com/sites/quizzical-lovelace-dcbd6a/deploys/635ee39eb6a5b700082b76fe

[mweststrate]

Looking good, will merge!

[elijahbenizzy]

Nice work! Thank you!

[github-actions]

🎉 This PR is included in version 9.0.18 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[Methuselah96]

@mweststrate This PR does not properly fix the issue and introduces bugs in other scenarios (#1013). It should probably be reverted.

[mweststrate]

Note that this PR was reverted, see #1013, #1014

[unional]

Thanks for reverting. The fix didn't take account for union type. Will see if I can make a proper fix for that later. Can we reopen #839?