Temporarily default to enabled

microsoft · Feb 3, 2022 · cba8f31 · cba8f31
1 parent e9dd907
commit cba8f31
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 12 deletions.
diff --git a/src/services/types.ts b/src/services/types.ts
@@ -226,18 +226,6 @@ namespace ts {
         On,
     }
 
-    /* @internal */
-    export function getAutoImportPreferences(preferences: UserPreferences) {
-        const includePackageJson =
-            preferences.includePackageJsonAutoImports === "off" ? PackageJsonAutoImportPreference.Off :
-            preferences.includePackageJsonAutoImports === "on" ? PackageJsonAutoImportPreference.On :
-            PackageJsonAutoImportPreference.Auto;
-        const includeProjectReferences =
-            preferences.includeProjectReferenceAutoImports === "on" ? ProjectReferenceAutoImportPreference.On :
-            ProjectReferenceAutoImportPreference.Off;
-        return { includePackageJson, includeProjectReferences };
-    }
-
     export interface PerformanceEvent {
         kind: "UpdateGraph" | "CreatePackageJsonAutoImportProvider";
         durationMs: number;

diff --git a/src/services/utilities.ts b/src/services/utilities.ts
@@ -3329,5 +3329,18 @@ namespace ts {
         return memoizeOne((source: AutoImportSource) => getProgramForAutoImport(source, program, host));
     }
 
+    export function getAutoImportPreferences(preferences: UserPreferences) {
+        const includePackageJson =
+            preferences.includePackageJsonAutoImports === "off" ? PackageJsonAutoImportPreference.Off :
+            preferences.includePackageJsonAutoImports === "on" ? PackageJsonAutoImportPreference.On :
+            PackageJsonAutoImportPreference.Auto;
+        // TODO: flip default back to "off" if shipping as an opt-in experiment.
+        //       Needs to default to "on" until VS Code is able to set it.
+        const includeProjectReferences =
+            preferences.includeProjectReferenceAutoImports === "off" ? ProjectReferenceAutoImportPreference.Off :
+            ProjectReferenceAutoImportPreference.On;
+        return { includePackageJson, includeProjectReferences };
+    }
+
     // #endregion
 }