Disallow dependency cycles (#13442)

.github/workflows/ci.yml

@@ -36,6 +36,9 @@ jobs:
           YARN_NODE_LINKER: pnp # use pnp linker for better linking performance: it's meant to update yarn cache only
         run: |
           yarn install --immutable --skip-builds
+      - name: Check for dependency cycles
+        run: |
+          yarn release-tool check-cycles
 
   test-coverage:
     name: Test on Node.js Latest

packages/babel-helper-hoist-variables/package.json

@@ -13,11 +13,13 @@
     "access": "public"
   },
   "main": "./lib/index.js",
-  "TODO": "The @babel/traverse dependency is only needed for the NodePath TS type. After converting @babel/core to TS we can import NodePath from there.",
   "dependencies": {
-    "@babel/traverse": "workspace:^7.13.15",
     "@babel/types": "workspace:^7.13.16"
   },
+  "TODO": "The @babel/traverse dependency is only needed for the NodePath TS type. We can consider exporting it from @babel/core.",
+  "devDependencies": {
+    "@babel/traverse": "workspace:*"
+  },
   "engines": {
     "node": ">=6.9.0"
   },

yarn.lock

@@ -596,7 +596,7 @@ __metadata:
   version: 0.0.0-use.local
   resolution: "@babel/helper-hoist-variables@workspace:packages/babel-helper-hoist-variables"
   dependencies:
-    "@babel/traverse": "workspace:^7.13.15"
+    "@babel/traverse": "workspace:*"
     "@babel/types": "workspace:^7.13.16"
   languageName: unknown
   linkType: soft
@@ -3567,7 +3567,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@babel/traverse@workspace:*, @babel/traverse@workspace:^7.13.0, @babel/traverse@workspace:^7.13.15, @babel/traverse@workspace:^7.14.0, @babel/traverse@workspace:^7.14.2, @babel/traverse@workspace:packages/babel-traverse":
+"@babel/traverse@workspace:*, @babel/traverse@workspace:^7.13.0, @babel/traverse@workspace:^7.14.0, @babel/traverse@workspace:^7.14.2, @babel/traverse@workspace:packages/babel-traverse":
   version: 0.0.0-use.local
   resolution: "@babel/traverse@workspace:packages/babel-traverse"
   dependencies: