Split out directDependencies and devDependencies in pub deps

lib/src/command/deps.dart

@@ -126,7 +126,17 @@ class DepsCommand extends PubCommand {
           'version': currentPackage.version.toString(),
           'kind': kind,
           'source': source,
-          'dependencies': next,
+          // This field is kept for backwards compatibility with dart 3.5 and
+          // before. Clients should opt to consume directDependencies and
+          // devDependencies separately instead.
+          'dependencies': (isRoot
+                  ? currentPackage.dependencies
+                  : currentPackage.immediateDependencies)
+              .keys
+              .toList(),
+          'directDependencies': currentPackage.dependencies.keys.toList(),
+          if (isRoot)
+            'devDependencies': currentPackage.devDependencies.keys.toList(),
         });
         toVisit.addAll(next);
       }

pubspec.lock

@@ -5,23 +5,23 @@ packages:
     dependency: transitive
     description:
       name: _fe_analyzer_shared
-      sha256: f6dbf021f4b214d85c79822912c5fcd142a2c4869f01222ad371bc51f9f1c356
+      sha256: "45cfa8471b89fb6643fe9bf51bd7931a76b8f5ec2d65de4fb176dba8d4f22c77"
       url: "https://pub.dev"
     source: hosted
-    version: "74.0.0"
+    version: "73.0.0"
   _macros:
     dependency: transitive
     description: dart
     source: sdk
-    version: "0.3.3"
+    version: "0.3.2"
   analyzer:
     dependency: "direct main"
     description:
       name: analyzer
-      sha256: f7e8caf82f2d3190881d81012606effdf8a38e6c1ab9e30947149733065f817c
+      sha256: "4959fec185fe70cce007c57e9ab6983101dbe593d2bf8bbfb4453aaec0cf470a"
       url: "https://pub.dev"
     source: hosted
-    version: "6.9.0"
+    version: "6.8.0"
   args:
     dependency: "direct main"
     description:
@@ -194,10 +194,10 @@ packages:
     dependency: transitive
     description:
       name: macros
-      sha256: "1d9e801cd66f7ea3663c45fc708450db1fa57f988142c64289142c9b7ee80656"
+      sha256: "0acaed5d6b7eab89f63350bccd82119e6c602df0f391260d0e32b5e23db79536"
       url: "https://pub.dev"
     source: hosted
-    version: "0.1.3-main.0"
+    version: "0.1.2-main.4"
   matcher:
     dependency: transitive
     description:

test/deps_test.dart

@@ -158,17 +158,24 @@ void main() {
       "dependencies": [
         "normal",
         "overridden",
-        "from_path",
-        "unittest",
-        "override_only"
+        "from_path"
+      ],
+      "directDependencies": [
+        "normal",
+        "overridden",
+        "from_path"
+      ],
+      "devDependencies": [
+        "unittest"
       ]
     },
     {
       "name": "override_only",
       "version": "1.2.3",
       "kind": "transitive",
       "source": "hosted",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     },
     {
       "name": "unittest",
@@ -178,14 +185,19 @@ void main() {
       "dependencies": [
         "shared",
         "dev_only"
+      ],
+      "directDependencies": [
+        "shared",
+        "dev_only"
       ]
     },
     {
       "name": "dev_only",
       "version": "1.2.3",
       "kind": "transitive",
       "source": "hosted",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     },
     {
       "name": "shared",
@@ -194,6 +206,9 @@ void main() {
       "source": "hosted",
       "dependencies": [
         "other"
+      ],
+      "directDependencies": [
+        "other"
       ]
     },
     {
@@ -203,21 +218,26 @@ void main() {
       "source": "hosted",
       "dependencies": [
         "myapp"
+      ],
+      "directDependencies": [
+        "myapp"
       ]
     },
     {
       "name": "from_path",
       "version": "1.2.3",
       "kind": "direct",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     },
     {
       "name": "overridden",
       "version": "2.0.0",
       "kind": "direct",
       "source": "hosted",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     },
     {
       "name": "normal",
@@ -227,6 +247,10 @@ void main() {
       "dependencies": [
         "transitive",
         "circular_a"
+      ],
+      "directDependencies": [
+        "transitive",
+        "circular_a"
       ]
     },
     {
@@ -236,6 +260,9 @@ void main() {
       "source": "hosted",
       "dependencies": [
         "circular_b"
+      ],
+      "directDependencies": [
+        "circular_b"
       ]
     },
     {
@@ -245,6 +272,9 @@ void main() {
       "source": "hosted",
       "dependencies": [
         "circular_a"
+      ],
+      "directDependencies": [
+        "circular_a"
       ]
     },
     {
@@ -254,6 +284,9 @@ void main() {
       "source": "hosted",
       "dependencies": [
         "shared"
+      ],
+      "directDependencies": [
+        "shared"
       ]
     }
   ],

test/testdata/goldens/deps/executables_test/applies formatting before printing executables.txt

@@ -48,21 +48,28 @@ $ pub deps --json
       "dependencies": [
         "foo",
         "bar"
-      ]
+      ],
+      "directDependencies": [
+        "foo",
+        "bar"
+      ],
+      "devDependencies": []
     },
     {
       "name": "bar",
       "version": "1.0.0",
       "kind": "direct",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     },
     {
       "name": "foo",
       "version": "1.0.0",
       "kind": "direct",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/dev dependencies.txt

@@ -34,7 +34,9 @@ $ pub deps --json
       "version": "0.0.0",
       "kind": "root",
       "source": "root",
-      "dependencies": [
+      "dependencies": [],
+      "directDependencies": [],
+      "devDependencies": [
         "foo"
       ]
     },
@@ -43,7 +45,8 @@ $ pub deps --json
       "version": "1.0.0",
       "kind": "dev",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/lists Dart executables, without entrypoints.txt

@@ -33,7 +33,9 @@ $ pub deps --json
       "version": "0.0.0",
       "kind": "root",
       "source": "root",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": [],
+      "devDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/lists executables from a dependency.txt

@@ -36,14 +36,19 @@ $ pub deps --json
       "source": "root",
       "dependencies": [
         "foo"
-      ]
+      ],
+      "directDependencies": [
+        "foo"
+      ],
+      "devDependencies": []
     },
     {
       "name": "foo",
       "version": "1.0.0",
       "kind": "direct",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/lists executables only from immediate dependencies.txt

@@ -40,7 +40,11 @@ $ pub deps --json
       "source": "root",
       "dependencies": [
         "foo"
-      ]
+      ],
+      "directDependencies": [
+        "foo"
+      ],
+      "devDependencies": []
     },
     {
       "name": "foo",
@@ -49,14 +53,18 @@ $ pub deps --json
       "source": "path",
       "dependencies": [
         "baz"
+      ],
+      "directDependencies": [
+        "baz"
       ]
     },
     {
       "name": "baz",
       "version": "1.0.0",
       "kind": "transitive",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/overriden dependencies executables.txt

@@ -41,14 +41,19 @@ $ pub deps --json
       "source": "root",
       "dependencies": [
         "foo"
-      ]
+      ],
+      "directDependencies": [
+        "foo"
+      ],
+      "devDependencies": []
     },
     {
       "name": "foo",
       "version": "2.0.0",
       "kind": "direct",
       "source": "path",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/skips executables in sub directories.txt

@@ -34,7 +34,9 @@ $ pub deps --json
       "version": "0.0.0",
       "kind": "root",
       "source": "root",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": [],
+      "devDependencies": []
     }
   ],
   "sdks": [

test/testdata/goldens/deps/executables_test/skips non-Dart executables.txt

@@ -31,7 +31,9 @@ $ pub deps --json
       "version": "0.0.0",
       "kind": "root",
       "source": "root",
-      "dependencies": []
+      "dependencies": [],
+      "directDependencies": [],
+      "devDependencies": []
     }
   ],
   "sdks": [