Refactor Dapr sidecar configuration to simplify AppId handling and remove redundant checks

Author: FullStackChef

src/CommunityToolkit.Aspire.Hosting.Dapr/IDistributedApplicationComponentBuilderExtensions.cs

@@ -63,38 +63,9 @@ public static IResourceBuilder<T> WithDaprSidecar<T>(this IResourceBuilder<T> bu
                                                        });
 
 
-        var serviceId = builder.Resource.Name;
 
         configureSidecar(sidecarBuilder);
 
-        if(sidecarBuilder.Resource.TryGetAnnotationsOfType<DaprSidecarOptionsAnnotation>(out var optionsAnnotations) &&
-           optionsAnnotations.LastOrDefault() is { } optionsAnnotation)
-        {
-            // If no AppId was specified, use the resource name.
-            if (string.IsNullOrEmpty(optionsAnnotation.Options.AppId))
-            {
-                sidecarBuilder.WithOptions(new DaprSidecarOptions
-                {
-                    AppId = serviceId,
-                    Config = optionsAnnotation.Options.Config,
-                    EnableApiLogging = optionsAnnotation.Options.EnableApiLogging,
-                    LogLevel = optionsAnnotation.Options.LogLevel,
-                    EnableProfiling = optionsAnnotation.Options.EnableProfiling,
-                    MetricsPort = optionsAnnotation.Options.MetricsPort,
-                    DaprHttpPort = optionsAnnotation.Options.DaprHttpPort,
-                    DaprGrpcPort = optionsAnnotation.Options.DaprGrpcPort,
-                });
-            }
-        }
-        else
-        {
-            // No options were specified, so set a default AppId.
-            sidecarBuilder.WithOptions(new DaprSidecarOptions
-            {
-                AppId = serviceId,
-            });
-        }
-
 
         return builder.WithAnnotation(new DaprSidecarAnnotation(sidecarBuilder.Resource));
     }