addressed copilot feedback

Author: bachuv

client/src/test/java/com/microsoft/durabletask/TaskEntityTest.java

@@ -163,6 +163,8 @@ public void add(int amount) {
 
         public void add(String label) {
             // overloaded — should trigger ambiguous match error
+            // 'label' intentionally unused; method exists to create ambiguous overload
+
         }
 
         @Override

endtoendtests/src/main/java/com/functions/EntityFunctions.java

@@ -145,7 +145,14 @@ public HttpResponseMessage startCallEntityOrchestration(
             @DurableClientInput(name = "durableContext") DurableClientContext durableContext,
             final ExecutionContext context) {
         String key = request.getQueryParameters().getOrDefault("key", "e2e-call-" + System.currentTimeMillis());
-        int value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "5"));
+        int value;
+        try {
+            value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "5"));
+        } catch (NumberFormatException e) {
+            return request.createResponseBuilder(HttpStatus.BAD_REQUEST)
+                    .body("The 'value' parameter must be a valid integer.")
+                    .build();
+        }
 
         DurableTaskClient client = durableContext.getClient();
         EntityPayload payload = new EntityPayload(key, value);
@@ -164,7 +171,14 @@ public HttpResponseMessage startSignalThenCallEntityOrchestration(
             @DurableClientInput(name = "durableContext") DurableClientContext durableContext,
             final ExecutionContext context) {
         String key = request.getQueryParameters().getOrDefault("key", "e2e-signal-" + System.currentTimeMillis());
-        int value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "10"));
+        int value;
+        try {
+            value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "10"));
+        } catch (NumberFormatException e) {
+            return request.createResponseBuilder(HttpStatus.BAD_REQUEST)
+                    .body("The 'value' parameter must be a valid integer.")
+                    .build();
+        }
 
         DurableTaskClient client = durableContext.getClient();
         EntityPayload payload = new EntityPayload(key, value);
@@ -217,7 +231,14 @@ public HttpResponseMessage startCallEntityTwiceOrchestration(
             @DurableClientInput(name = "durableContext") DurableClientContext durableContext,
             final ExecutionContext context) {
         String key = request.getQueryParameters().getOrDefault("key", "e2e-twice-" + System.currentTimeMillis());
-        int value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "3"));
+        int value;
+        try {
+            value = Integer.parseInt(request.getQueryParameters().getOrDefault("value", "3"));
+        } catch (NumberFormatException e) {
+            return request.createResponseBuilder(HttpStatus.BAD_REQUEST)
+                    .body("The 'value' parameter must be a valid integer.")
+                    .build();
+        }
 
         DurableTaskClient client = durableContext.getClient();
         EntityPayload payload = new EntityPayload(key, value);

endtoendtests/src/test/java/com/functions/EndToEndTests.java

@@ -564,7 +564,13 @@ private int getEntityStateValue(String entityName, String entityKey) {
         Response response = get("/api/GetEntityState?name=" + entityName + "&key=" + entityKey);
         assertEquals(200, response.getStatusCode(),
                 "GetEntityState should return 200, got: " + response.getStatusCode() + " body: " + response.getBody().asString());
-        return Integer.parseInt(response.getBody().asString().trim());
+        String body = response.getBody().asString().trim();
+        try {
+            return Integer.parseInt(body);
+        } catch (NumberFormatException e) {
+            fail("Expected integer entity state but got: " + body);
+            throw e; // unreachable, but satisfies compiler
+        }
     }
 
     private boolean pollingCheck(String statusQueryGetUri,

samples-azure-functions/src/main/java/com/functions/entities/UserFunctions.java

@@ -87,8 +87,7 @@ public HttpResponseMessage putUser(
                     route = "users/{id}",
                     authLevel = AuthorizationLevel.ANONYMOUS) HttpRequestMessage<Optional<String>> request,
             @DurableClientInput(name = "durableContext") DurableClientContext durableContext,
-            @BindingName("id") String id,
-            final ExecutionContext context) {
+            @BindingName("id") String id) {
         String name = request.getQueryParameters().get("name");
         String ageStr = request.getQueryParameters().get("age");
 
@@ -130,8 +129,7 @@ public HttpResponseMessage patchUser(
                     route = "users/{id}",
                     authLevel = AuthorizationLevel.ANONYMOUS) HttpRequestMessage<Optional<String>> request,
             @DurableClientInput(name = "durableContext") DurableClientContext durableContext,
-            @BindingName("id") String id,
-            final ExecutionContext context) {
+            @BindingName("id") String id) {
         String name = request.getQueryParameters().get("name");
         String ageStr = request.getQueryParameters().get("age");