Enable symlinks (resolver.unstable_enableSymlinks) by default (face…

…book#1068)

Summary:
Pull Request resolved: facebook#1068

Flip the default for `unstable_enableSymlinks` to true. The option was made available in React Native 0.72 and it's been trialled by the community to the extent that we can have good confidence in it (though there are still devx gaps to guide users towards a suitable configuration).

This is also an effective prerequisite for disabling Haste/global package resolution by default in OSS, which currently allows workspaces to mostly-work in the absence of symlink support.

Closes facebook#1

Changelog:
```
 - **[Feature]**: Enable resolution through symlinks (previously experimental `unstable_enableSymlinks`)
```

Reviewed By: motiz88

Differential Revision: D48777855

fbshipit-source-id: a11664bac127db931c03d0e8446b6fc66e3c8a2b

docs/Configuration.md

@@ -403,7 +403,7 @@ Enable experimental support for projects containing symbolic links (symlinks).
 
 When enabled, Metro traverses symlinks during module and asset [resolution](./Resolution.md), instead of ignoring symlinks. Note that, as with any other file Metro needs to resolve, the symlink target *must be within configured [watched folders](#watchfolders)* and not otherwise excluded.
 
-Defaults to `false`.
+Defaults to `true` since Metro v0.79.0.
 
 :::info
 
@@ -413,7 +413,7 @@ For example, if you have a Metro project within a [Yarn workspace](https://class
 
 :::note
 
-In a future release of Metro, this option will become `true` by default.
+In a future release of Metro, this option will be removed (symlink support will be always-on).
 
 :::
 

packages/metro-config/src/__tests__/__snapshots__/loadConfig-test.js.snap

@@ -84,7 +84,7 @@ Object {
       ],
     },
     "unstable_enablePackageExports": false,
-    "unstable_enableSymlinks": false,
+    "unstable_enableSymlinks": true,
     "useWatchman": true,
   },
   "serializer": Object {
@@ -262,7 +262,7 @@ Object {
       ],
     },
     "unstable_enablePackageExports": false,
-    "unstable_enableSymlinks": false,
+    "unstable_enableSymlinks": true,
     "useWatchman": true,
   },
   "serializer": Object {
@@ -440,7 +440,7 @@ Object {
       ],
     },
     "unstable_enablePackageExports": false,
-    "unstable_enableSymlinks": false,
+    "unstable_enableSymlinks": true,
     "useWatchman": true,
   },
   "serializer": Object {
@@ -618,7 +618,7 @@ Object {
       ],
     },
     "unstable_enablePackageExports": false,
-    "unstable_enableSymlinks": false,
+    "unstable_enableSymlinks": true,
     "useWatchman": true,
   },
   "serializer": Object {

packages/metro-config/src/defaults/index.js

@@ -40,7 +40,7 @@ const getDefaultValues = (projectRoot: ?string): ConfigT => ({
     blockList: exclusionList(),
     dependencyExtractor: undefined,
     disableHierarchicalLookup: false,
-    unstable_enableSymlinks: false,
+    unstable_enableSymlinks: true,
     emptyModulePath: require.resolve(
       'metro-runtime/src/modules/empty-module.js',
     ),