mkYarnWorkspace ignores workspace paths which yarn would ignore

default.nix

@@ -167,7 +167,15 @@ in rec {
       else lib.concatMap (child: expandGlobList (base+("/"+child)) rest) matchingChildren;
     # Path -> PathGlob -> [Path]
     expandGlob = base: glob: expandGlobList base (splitGlob glob);
-    packagePaths = lib.concatMap (expandGlob src) packageGlobs;
+    validPackage = path:
+      let
+        packageJSON = path+"/package.json";
+        package = lib.importJSON packageJSON;
+      in
+        builtins.pathExists (path+"/package.json") &&
+        builtins.hasAttr "name" package &&
+        builtins.hasAttr "version" package;
+    packagePaths = builtins.filter validPackage (lib.concatMap (expandGlob src) packageGlobs);
     packages = lib.listToAttrs (map (src:
     let
       packageJSON = src+"/package.json";

tests/default.nix

@@ -21,6 +21,14 @@ in
       '';
     };
   }).package-one;
+  inherit (yarn2nix.mkYarnWorkspace {
+    src = fetchFromGitHub {
+      owner = "facebook";
+      repo = "nuclide";
+      rev = "v0.366.0";
+      sha256 = "1iyy8by8p90dncyqi3pgaj36hmz1khi9w1rkhd5gi9nbkkv5can4";
+    };
+  }) nuclide-node-transpiler;
 } // {
   duplicate-pkgs = import ./duplicate-pkgs { inherit yarn2nix; };
   no-import-from-derivation = import ./no-import-from-derivation {

tests/workspace/package-nameless/package.json

@@ -0,0 +1,4 @@
+{
+  "version": "1.0.0",
+  "license": "MIT"
+}

tests/workspace/package-versionless/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "versionless",
+  "license": "MIT"
+}