System.Linq.AsyncEnumerable ref-pack missing from SDK

[ellahathaway]

Source-build tests uncovered that the System.Linq.AsyncEnumerable ref-pack is missing from the Microsoft SDK:

--- a/msftSdkFiles.txt
+++ b/sbSdkFiles.txt
@@ ------------ @@
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.IO.Pipelines.xml
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.IO.Pipes.AccessControl.xml
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.IO.Pipes.xml
+./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.AsyncEnumerable.xml
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Expressions.xml
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Parallel.xml
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Queryable.xml

System.Linq.AsyncEnumerable was added in #111685. This ref-pack should exist in the Microsoft SDK.

cc @stephentoub @ViktorHofer

[ViktorHofer]

I can't "reproduce" this. I find the file in the last nightly released SDK:

The file is also part of the last nightly released microsoft.netcore.app.ref package:

[ViktorHofer]

Oh I see this is about the Microsoft.AspNetCore.App.Ref pack. I don't see the System.Linq.AsyncEnumerable.dll file there either and it shouldn't be there as the file is already shipping in the base microsoft.netcore.app shared framework.

[ViktorHofer]

@ellahathaway can you please provide more details which component is bringing that file in when building source-only? That looks like a bug in the source-only build of (maybe?) aspnetcore.

[ellahathaway]

it shouldn't be there as the file is already shipping in the base microsoft.netcore.app shared framework

What about other runtime packages such as System.Linq.Parallel? That ships as part of microsoft.netcore.app shared framework, but it also has the IntelliSense file in Microsoft.AspNetCore.App.Ref. Why is System.Linq.AsyncEnumerable different than other packages in the System.Linq namespace?

--- a/msft.diff
+++ b/sb.diff
@@ -1,13 +1,17 @@
 find . -type f -name "System.Linq.Parallel*"
 ./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Parallel.xml
+./packs/runtime.centos.9-x64.Microsoft.DotNet.ILCompiler/x.y.z/framework/System.Linq.Parallel.pdb (KNOWN DIFF)
+./packs/runtime.centos.9-x64.Microsoft.DotNet.ILCompiler/x.y.z/framework/System.Linq.Parallel.dll (KNOWN DIFF)
 ./packs/NETStandard.Library.Ref/x.y.z/ref/netstandard2.1/System.Linq.Parallel.dll
 ./packs/Microsoft.NETCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Parallel.xml
 ./packs/Microsoft.NETCore.App.Ref/x.y.z/ref/netx.y/System.Linq.Parallel.dll
 ./shared/Microsoft.NETCore.App/x.y.z/System.Linq.Parallel.dll
 ./sdk/x.y.z/Microsoft/Microsoft.NET.Build.Extensions/net461/lib/System.Linq.Parallel.dll
-./sdk/x.y.z/TestHostNetFramework/System.Linq.Parallel.dll (KNOWN DIFF)
 
 find . -type f -name "System.Linq.AsyncEnumerable*"
+./packs/Microsoft.AspNetCore.App.Ref/x.y.z/ref/netx.y/System.Linq.AsyncEnumerable.xml
+./packs/runtime.centos.9-x64.Microsoft.DotNet.ILCompiler/x.y.z/framework/System.Linq.AsyncEnumerable.dll (KNOWN DIFF)
+./packs/runtime.centos.9-x64.Microsoft.DotNet.ILCompiler/x.y.z/framework/System.Linq.AsyncEnumerable.pdb (KNOWN DIFF)
 ./packs/Microsoft.NETCore.App.Ref/x.y.z/ref/netx.y/System.Linq.AsyncEnumerable.dll
 ./packs/Microsoft.NETCore.App.Ref/x.y.z/ref/netx.y/System.Linq.AsyncEnumerable.xml
 ./shared/Microsoft.NETCore.App/x.y.z/System.Linq.AsyncEnumerable.dll

[stephentoub]

That ships as part of microsoft.netcore.app shared framework, but it also has the IntelliSense file in Microsoft.AspNetCore.App.Ref.

Why does System.Linq.Parallel ship anything in the aspnetcore app ref?

Why is System.Linq.AsyncEnumerable different than other packages in the System.Linq namespace?

If nothing else, System.Linq.AsyncEnumerable is different from System.Linq and System.Linq.Parallel in that the latter two are only part of the shared framework whereas the former is both part of the shared framework and a nuget package.

[ellahathaway]

After speaking offline with @NikolaMilosavljevic, I've gathered that the IntelliSense files should not exist in the aspnetcore app ref unless they are accompanied by a respective .dll. There are a few of these IntelliSense files that do not fit this criterion. Given this information, plus the content of the comments above, I'm going to close this issue and file a new issue in aspnetcore.