No IVTs for CodeLenses (#6906)

* Don't expose Legacy MSBuildReferenceResolver

* Move the files boss hoss

* Remove unused opens

* Fix tests

* Remove IVT dependency for CodeLens and put logging behind a flag

* oopsie doopsie

* meep morp

* blub glub

* heepers jeepers

* Update fsi.fs

* Update service.fs

* Update service.fsi

* Update FSharpCheckerProvider.fs

* Update FSharpLanguageServiceTestable.fs

* Update TestLib.LanguageService.fs

* Updates for build in release

* I'm not touching MSBuildReferenceResolver I'm not touching MSBuildReferenceResolver I'm not touching MSBuildReferenceResolver

* Different ignore and adjust ifdefs

Author: cartermp

src/fsharp/service/service.fs

@@ -2103,12 +2103,12 @@ type FSharpCheckFileResults(filename: string, errors: FSharpErrorInfo[], scopeOp
             RequireCompilationThread ctok
             scope.IsRelativeNameResolvableFromSymbol(pos, plid, symbol))
 
-    member info.GetDisplayEnvForPos(pos: pos) : Async<DisplayEnv option> =
+    member info.GetDisplayContextForPos(pos: pos) : Async<FSharpDisplayContext option> =
         let userOpName = "CodeLens"
         reactorOp userOpName "GetDisplayContextAtPos" None (fun ctok scope -> 
             DoesNotRequireCompilerThreadTokenAndCouldPossiblyBeMadeConcurrent ctok
             let (nenv, _), _ = scope.GetBestDisplayEnvForPos pos
-            Some nenv.DisplayEnv)
+            Some(FSharpDisplayContext(fun _ -> nenv.DisplayEnv)))
 
     member info.ImplementationFile =
         if not keepAssemblyContents then invalidOp "The 'keepAssemblyContents' flag must be set to true on the FSharpChecker in order to access the checked contents of assemblies"

src/fsharp/service/service.fsi

@@ -244,8 +244,8 @@ type public FSharpCheckFileResults =
 
     member internal GetVisibleNamespacesAndModulesAtPoint : pos -> Async<Tast.ModuleOrNamespaceRef[]>
 
-    /// Find the most precise display environment for the given line and column.
-    member internal GetDisplayEnvForPos : pos : pos -> Async<DisplayEnv option>
+    /// Find the most precise display context for the given line and column.
+    member GetDisplayContextForPos : pos : pos -> Async<FSharpDisplayContext option>
 
     /// Determines if a long ident is resolvable at a specific point.
     /// <param name="userOpName">An optional string used for tracing compiler operations associated with this request.</param>