feat: Use SDK DI extension methods, migrate to Azure.Connectors.Sdk namespace

[daviburg]

Summary

Replace ~130 lines of manual DI boilerplate in Program.cs with the new SDK extension methods from Azure/Connectors-NET-SDK#117.

Before (Program.cs — 164 lines)

services.AddOptions<ConnectorOptions>()
    .Bind(configuration.GetSection(ConnectorOptions.SectionName))
    .ValidateDataAnnotations()
    .ValidateOnStart();

services.AddSingleton<Office365Client>(serviceProvider =>
{
    var options = serviceProvider.GetRequiredService<IOptions<ConnectorOptions>>().Value;
    return options.Office365.ManagedIdentityClientId != null
        ? new Office365Client(options.Office365.ConnectionRuntimeUrl, ...)
        : new Office365Client(options.Office365.ConnectionRuntimeUrl);
});
// ... repeated 9 more times

After (Program.cs — 33 lines)

var connectors = hostContext.Configuration.GetSection("Connectors");

services
    .AddOffice365Client(connectors.GetSection("Office365"))
    .AddSharepointonlineClient(connectors.GetSection("SharePoint"))
    .AddTeamsClient(connectors.GetSection("Teams"))
    // ...

Changes

• Program.cs: Replace 10 manual factory registrations with 9 one-liner extension calls
• Remove Configuration/ConnectorOptions.cs — no longer needed (was 250 lines of per-connector options classes)
• Remove AzureLogAnalyticsFunctions.cs — connector deprecated in SDK, replaced by azuremonitorlogs
• Namespace migration: Microsoft.Azure.Connectors.DirectClient.* to Azure.Connectors.Sdk.*
• Exception migration: Per-connector exceptions to unified ConnectorException
• Property update: ex.StatusCode to ex.Status (from RequestFailedException)
• Remove IsFatal() guards (made internal in SDK)
• Switch from NuGet package to project reference (pending next SDK release)

Net diff: +184 / -741 lines

Depends on

• feat: Add DI integration extension methods for connector clients Connectors-NET-SDK#117 (SDK DI extension methods)

[Copilot]

Pull request overview

This PR updates the DirectConnector sample to use the new Azure Connectors SDK DI extension methods (reducing DI boilerplate), migrates code to the Azure.Connectors.Sdk.* namespaces, and aligns error handling with the SDK’s unified ConnectorException.

Changes:

• Replaced manual connector client DI registrations with SDK-provided Add*Client(...) extension methods.
• Migrated connector namespaces/exceptions to Azure.Connectors.Sdk.* and ConnectorException (including StatusCode → Status).
• Removed legacy per-connector configuration options class and removed the deprecated Azure Log Analytics sample functions.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
DirectConnector/Program.cs	Switches connector client DI wiring to SDK DI extension methods driven by config sections.
DirectConnector/DirectConnector.csproj	Replaces the SDK NuGet dependency with a project reference (currently pointing outside the repo).
DirectConnector/ConnectorFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/OneDriveFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/Office365UsersFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/MsGraphFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/MqFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/SmtpFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/AzureBlobFunctions.cs	Migrates connector namespaces and exception handling to ConnectorException/Status.
DirectConnector/ConnectorTriggerMetadataAttribute.cs	Updates XML doc references to the new SDK namespace.
DirectConnector/Configuration/ConnectorOptions.cs	Removed (legacy options binding/validation path).
DirectConnector/AzureLogAnalyticsFunctions.cs	Removed (connector deprecated/removed in the SDK).

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

[daviburg]

[Dobby] Converted to draft — this PR depends on Azure/Connectors-NET-SDK#117 being merged and published as \Azure.Connectors.Sdk\ 0.9.0-preview.1 to nuget.org. CI will fail with \NU1101: Unable to find package Azure.Connectors.Sdk\ until then. Mark as ready for review after the SDK release.

[daviburg]

Closing — superseded by #40 which consolidates all SDK v0.9.0-preview.1 breaking changes, namespace migrations, and new features into a single PR.