fix: address code review findings from #71 and #80#81

Merged

joshsmithxrm merged 17 commits intomainfrom

fix/code-review-71

Jan 2, 2026

Commits on Jan 2, 2026

fix: add thread-safe async locking to ProfileConnectionSource

joshsmithxrm
and
claude
committed
fix: use ConcurrentBag for thread-safe provider tracking

joshsmithxrm
and
claude
committed
fix: properly unregister MSAL cache in InteractiveBrowserCredentialProvider

joshsmithxrm
and
claude
committed
fix: properly unregister MSAL cache in GlobalDiscoveryService

joshsmithxrm
and
claude
committed
fix: remove redundant CancellationTokenSource from Program.cs

joshsmithxrm
and
claude
committed
fix: use ConcurrentDictionary for entity type code cache

joshsmithxrm
and
claude
committed
refactor: extract duplicate ParseBypassPlugins to DataCommandGroup

joshsmithxrm
and
claude
committed
chore: suppress NU1702 warning for cross-framework PPDS.Plugins reference

joshsmithxrm
and
claude
committed
fix: remove NU1903 vulnerability warning suppression

joshsmithxrm
and
claude
committed
fix: use X509CertificateLoader for .NET 9+ certificate loading

joshsmithxrm
and
claude
committed
fix: suppress SYSLIB0014 warning for ServicePointManager

joshsmithxrm
and
claude
committed
fix: use async/await in thread safety tests (xUnit1031)

joshsmithxrm
and
claude
committed
fix: address 11 code review findings from #80

joshsmithxrm
and
claude
committed
fix: add AuthenticationOutput for configurable auth messaging (#80)

joshsmithxrm
and
claude
committed
fix: address remaining issue 71 code review findings (8, 12)

joshsmithxrm
and
claude
committed
refactor: extract phase processors from TieredImporter (#18)

joshsmithxrm
and
claude
committed
fix: address PR review feedback (issues 1, 3, 4)

joshsmithxrm
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: address code review findings from #71 and #80#81

fix: address code review findings from #71 and #80#81
joshsmithxrm merged 17 commits intomainfrom
fix/code-review-71

Commits on Jan 2, 2026

fix: add thread-safe async locking to ProfileConnectionSource

fix: use ConcurrentBag for thread-safe provider tracking

fix: properly unregister MSAL cache in InteractiveBrowserCredentialProvider

fix: properly unregister MSAL cache in GlobalDiscoveryService

fix: remove redundant CancellationTokenSource from Program.cs

fix: use ConcurrentDictionary for entity type code cache

refactor: extract duplicate ParseBypassPlugins to DataCommandGroup

chore: suppress NU1702 warning for cross-framework PPDS.Plugins reference

fix: remove NU1903 vulnerability warning suppression

fix: use X509CertificateLoader for .NET 9+ certificate loading

fix: suppress SYSLIB0014 warning for ServicePointManager

fix: use async/await in thread safety tests (xUnit1031)

fix: address 11 code review findings from #80

fix: add AuthenticationOutput for configurable auth messaging (#80)

fix: address remaining issue 71 code review findings (8, 12)

refactor: extract phase processors from TieredImporter (#18)

fix: address PR review feedback (issues 1, 3, 4)