Optimise dependencies

[thompson-tomo]

Confirm this is a feature request for the .NET library and not the underlying OpenAI API

• This is a feature request for the .NET library

Describe the feature or improvement you are requesting

I want to be able to use the library with minimal dependencies. This means for 6 System.Diagnostics.DiagnosticSource is not an explicit dependency.

Additional context

No response

[joseharriaga]

Thank you for reaching out, @thompson-tomo ! Could you please expound on your scenario a little bit for us to understand what is the problem with the System.Diagnostics.DiagnosticSource dependency?

[thompson-tomo]

the dotnet framework natively provides a collection of libraries such as System.Diagnostics.DiagnosticSource which eliminates the need for them to be added explicitly as a dependency unless a newer version that provided in the framework is needed. The issue with explicitly adding framework dependencies when they are not needed, the maintenance effort increases and in my case when using net 8, this package would bring in System.Diagnostics.DiagnosticSource 6.0.2 rather than use what is available in the framework.

[KrzysztofCwalina]

@thompson-tomo, are you sure "this package would bring in System.Diagnostics.DiagnosticSource 6.0.2"? i.e. are you really seeing 6.0.2 in the output folder of a .NET 8 app? I think what you might be seeing is that NuGet says that the OpenAI library has a dependency on System.Diagnostics.DiagnosticSource 6.0.2, but then when the app is built, NuGet should notice that the runtime already contains a newer version and so it should not put 6.0.2 into the output folder.

[thompson-tomo]

yes i have seen it across the board hence have gone and optimised a number of libraries to address this. as far as i have seen nuget doesn't do any filtering as you described, in fact have an issue to add it.

Note you should still have it on the older frameworks ie net standard 2.0 so that it pulls the libary up to the min version you need.

[joseharriaga]

We might be able to fix this with something like this:
🔗 https://github.com/Azure/azure-sdk-for-net/blob/b3d9afded59ecc56b6174f2f3f987fcb689ef9a7/eng/Directory.Build.Common.targets#L107

Also, this issue might be related:
🔗 #297

[thompson-tomo]

Actually it would be even easier to fix. An example to fix it is shown in https://github.com/protobuf-net/protobuf-net/pull/1146/files