datalust / datalust/seqcli

`app run` should resolve native assembly dependencies

Open
#252 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C#
Stars
167
Forks
30
Avg merge
1d 6h
Merged PRs (30d)
9

Description

Some NuGet packages include platform-specific runtime/native dependencies (for example, Microsoft.Data.SqlClient). When these packages are bundled into Seq Apps, the seqcli-based app host does not currently attempt to load the native dependencies, and the packaged app doesn't function.

It's possible to work around this manually to target a single native platform, for example by overwriting the default Microsoft.Data.SqlClient.dll assembly with a platform-specific one, and pulling Microsoft.Data.SqlClient.SNI.dll up from the nested runtime/native dependency folder to the root of the package, but this is tricky to get right, and only supports one platform at a time.

seqcli app run should use the framework's reference resolution logic to automatically pick up and load the correct native dependencies from the default package layout.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the seqcli app run entry point and trace how bundled assemblies are resolved. Compare that behavior with the framework's reference resolution logic and the default package layout, using Microsoft.Data.SqlClient and its nested runtime/native dependency as the example. Done means the correct native dependency is selected automatically for the host platform without manual package changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.