@sosukesuzuki I don't like what Prettier is doing here 😛

What formatting is expected?

I would have expected it to be

expect(do {
  var bar = "foo";
  bar;
}).toBe("foo");

which is what it does with functions: https://prettier.io/playground/#N4Igxg9gdgLgprEAuEcAeAHOYYAoAmEABMADpRFEBuAhgE5EBG9RAvEaSAGYQScDc5SszqCoAXwCUAOhgQAQnFycefEJLHl0WHLlyS2APhJDqLEWw7deA0yLFTZCpSpvr+IADQgIGGAEtoAGdkUHo6CAB3AAV6BBCUGgAbSJoATxDvRjoaMABrOBgAZQxc-ygAc2QYOgBXOG84AFtGOHx8NoAZGkramgq4ADEIOiaaGADK5BAaWrkvEAALGCakgHVF-3gg0rA4Ivit-yottOmwIMyQcqC4OhhonIqx5C5k2+8AKyC0eRz8wpFGhNOCdcpwV7vBogb5oIrlCpJOAARVqEHgkKSHxApTotzo02YrSSCwwdHKMDW-nwMEWyAAHAAGbxkiC3NY5DDTMlwfFUCHeACOaPgj18CRmQQAtFA4G02gs6HBhf4lY9+i8kG8sdDbk1-NU6rqEUjUeiIVqod4YDRGFSaXSkAAma05fxJBEAYQgTU1qCCAFYFrVbgAVW0JbXYqj1ACSUA6sCKYHJfgAggmijA0kjMbdxOIgA

Currently the tsconfig generator will generate one more item:

{ "path": "../babel-traverse" }

I am not familiar with the references usage here but the type checking and the VSCode type inferences seems to work after I removed babel-traverse, which caused cycling dependencies since this package is also depended by @babel/traverse.

Should we filter out dependencies from introduced only from type imports? In this case we have import type { NodePath } from "@babel/traverse".

/cc @nicolo-ribaudo @zxbodya

The dependency is needed also for type imports: it's needed to tell TS in which order to compile the packages to properly type-check everything.

TS doesn't support circular dependencies yet (microsoft/TypeScript#33685), so we could consider moving to type-checking all the packages as a whole rather than package by package (it will make incremental type-checks a bit slower, but it doesn't have this ordering problem).

I would suggest, if removing reference to @babel/traverse we should also remove type import:

import type { NodePath } from "@babel/traverse";

Might be using any for now.

Otherwise it might result in some weird errors - something like "can not output to file used as input" (basically compiled TS would be imported here, but when compiling @babel/traverse this file would be used as input, which would mean - when compiling @babel/traverse TS will try to use output of it as an import )