fixed copilot comments

Author: bachuv

client/src/main/java/com/microsoft/durabletask/ReplaySafeLogger.java

@@ -18,6 +18,13 @@
  *
  * <p>This mirrors the {@code ReplaySafeLogger} nested class in the modern .NET
  * {@code TaskOrchestrationContext}.
+ *
+ * <p><b>Note on overloads:</b> The {@code (String, Object, Object)} and {@code (String, Object...)}
+ * overloads at each log level are both defined by the {@link Logger} interface. SLF4J provides the
+ * explicit 2-arg overload to avoid varargs array allocation in the common case. Java's overload
+ * resolution (JLS §15.12.2) always prefers the non-varargs form, so dispatch is unambiguous despite
+ * the apparent overlap. This wrapper delegates each overload to the identical overload on the inner
+ * logger.
  */
 final class ReplaySafeLogger implements Logger {
 

client/src/main/java/com/microsoft/durabletask/ReplaySafeLoggerFactory.java

@@ -31,6 +31,7 @@ ILoggerFactory underlying() {
 
     @Override
     public Logger getLogger(String name) {
+        Helpers.throwIfArgumentNullOrWhiteSpace(name, "name");
         return new ReplaySafeLogger(context, ReplaySafeLoggers.unwrap(context).getLogger(name));
     }
 }

client/src/main/java/com/microsoft/durabletask/ReplaySafeLoggers.java

@@ -33,6 +33,11 @@ private ReplaySafeLoggers() {
     static ILoggerFactory unwrap(TaskOrchestrationContext context) {
         Helpers.throwIfArgumentNull(context, "context");
         ILoggerFactory factory = context.getLoggerFactory();
+        if (factory == null) {
+            throw new IllegalStateException(
+                "getLoggerFactory() returned null. Ensure the context's getLoggerFactory() " +
+                "implementation returns a non-null ILoggerFactory.");
+        }
         int depth = 0;
         while (factory instanceof ReplaySafeLoggerFactory) {
             if (++depth > MAX_UNWRAP_DEPTH) {

client/src/main/java/com/microsoft/durabletask/TaskOrchestrationContext.java

@@ -892,18 +892,20 @@ default ILoggerFactory getLoggerFactory() {
      * @return a replay-safe SLF4J {@code Logger}
      */
     default Logger createReplaySafeLogger(String name) {
+        Helpers.throwIfArgumentNullOrWhiteSpace(name, "name");
         return new ReplaySafeLogger(this, ReplaySafeLoggers.unwrap(this).getLogger(name));
     }
 
     /**
      * Returns an SLF4J {@link Logger} that is replay-safe, using the fully-qualified class name
      * as the logger category.
      *
-     * @param clazz the class whose name to use as the logger category
+     * @param class the class whose name to use as the logger category
      * @return a replay-safe SLF4J {@code Logger}
      * @see #createReplaySafeLogger(String)
      */
-    default Logger createReplaySafeLogger(Class<?> clazz) {
+    default Logger createReplaySafeLogger(Class<?> class) {
+        Helpers.throwIfArgumentNull(class, "class");
         return new ReplaySafeLogger(this, ReplaySafeLoggers.unwrap(this).getLogger(clazz.getName()));
     }