Refactor hook handlers and improve code formatting for readability

Author: brunoborges

src/main/java/com/github/copilot/sdk/CopilotSession.java

@@ -549,8 +549,8 @@ CompletableFuture<Object> handleHooksInvoke(String hookType, JsonNode input) {
                     break;
                 case "userPromptSubmitted" :
                     if (hooks.getOnUserPromptSubmitted() != null) {
-                        UserPromptSubmittedHookInput promptInput =
-                                MAPPER.treeToValue(input, UserPromptSubmittedHookInput.class);
+                        UserPromptSubmittedHookInput promptInput = MAPPER.treeToValue(input,
+                                UserPromptSubmittedHookInput.class);
                         return hooks.getOnUserPromptSubmitted().handle(promptInput, invocation)
                                 .thenApply(output -> (Object) output);
                     }

src/main/java/com/github/copilot/sdk/json/SessionEndHandler.java

@@ -16,11 +16,9 @@
  *
  * <pre>{@code
  * SessionEndHandler handler = (input, invocation) -> {
- *     System.out.println("Session ended: " + input.getReason());
- *     return CompletableFuture.completedFuture(
- *         new SessionEndHookOutput()
- *             .setSessionSummary("Session completed successfully")
- *     );
+ * 	System.out.println("Session ended: " + input.getReason());
+ * 	return CompletableFuture
+ * 			.completedFuture(new SessionEndHookOutput().setSessionSummary("Session completed successfully"));
  * };
  * }</pre>
  *

src/main/java/com/github/copilot/sdk/json/SessionEndHookInput.java

@@ -78,8 +78,7 @@ public SessionEndHookInput setCwd(String cwd) {
     /**
      * Gets the reason for session end.
      *
-     * @return the reason: "complete", "error", "abort", "timeout", or
-     *         "user_exit"
+     * @return the reason: "complete", "error", "abort", "timeout", or "user_exit"
      */
     public String getReason() {
         return reason;

src/main/java/com/github/copilot/sdk/json/SessionHooks.java

@@ -160,7 +160,7 @@ public SessionHooks setOnSessionEnd(SessionEndHandler onSessionEnd) {
      * @return {@code true} if at least one hook handler is set
      */
     public boolean hasHooks() {
-        return onPreToolUse != null || onPostToolUse != null || onUserPromptSubmitted != null
-                || onSessionStart != null || onSessionEnd != null;
+        return onPreToolUse != null || onPostToolUse != null || onUserPromptSubmitted != null || onSessionStart != null
+                || onSessionEnd != null;
     }
 }

src/main/java/com/github/copilot/sdk/json/SessionStartHandler.java

@@ -16,11 +16,9 @@
  *
  * <pre>{@code
  * SessionStartHandler handler = (input, invocation) -> {
- *     System.out.println("Session started from: " + input.getSource());
- *     return CompletableFuture.completedFuture(
- *         new SessionStartHookOutput()
- *             .setAdditionalContext("Custom initialization context")
- *     );
+ * 	System.out.println("Session started from: " + input.getSource());
+ * 	return CompletableFuture
+ * 			.completedFuture(new SessionStartHookOutput().setAdditionalContext("Custom initialization context"));
  * };
  * }</pre>
  *

src/main/java/com/github/copilot/sdk/json/UserPromptSubmittedHandler.java

@@ -16,12 +16,10 @@
  *
  * <pre>{@code
  * UserPromptSubmittedHandler handler = (input, invocation) -> {
- *     System.out.println("User submitted: " + input.getPrompt());
- *     // Optionally modify the prompt
- *     return CompletableFuture.completedFuture(
- *         new UserPromptSubmittedHookOutput()
- *             .setModifiedPrompt(input.getPrompt() + " (enhanced)")
- *     );
+ * 	System.out.println("User submitted: " + input.getPrompt());
+ * 	// Optionally modify the prompt
+ * 	return CompletableFuture.completedFuture(
+ * 			new UserPromptSubmittedHookOutput().setModifiedPrompt(input.getPrompt() + " (enhanced)"));
  * };
  * }</pre>
  *

src/test/java/com/github/copilot/sdk/HooksTest.java

@@ -30,8 +30,8 @@
 import com.github.copilot.sdk.json.UserPromptSubmittedHookInput;
 
 /**
- * Tests for hooks functionality (pre-tool-use, post-tool-use, user-prompt-submitted,
- * session-start, and session-end hooks).
+ * Tests for hooks functionality (pre-tool-use, post-tool-use,
+ * user-prompt-submitted, session-start, and session-end hooks).
  *
  * <p>
  * These tests use the shared CapiProxy infrastructure for deterministic API
@@ -205,8 +205,8 @@ void testUserPromptSubmittedHookInvokedWhenUserSendsMessage() throws Exception {
         List<UserPromptSubmittedHookInput> promptInputs = new ArrayList<>();
         final String[] sessionIdHolder = new String[1];
 
-        SessionConfig config =
-                new SessionConfig().setHooks(new SessionHooks().setOnUserPromptSubmitted((input, invocation) -> {
+        SessionConfig config = new SessionConfig()
+                .setHooks(new SessionHooks().setOnUserPromptSubmitted((input, invocation) -> {
                     promptInputs.add(input);
                     assertEquals(sessionIdHolder[0], invocation.getSessionId());
                     return CompletableFuture.completedFuture(null);
@@ -233,8 +233,8 @@ void testSessionStartHookInvokedWhenSessionCreated() throws Exception {
 
         List<SessionStartHookInput> startInputs = new ArrayList<>();
 
-        SessionConfig config =
-                new SessionConfig().setHooks(new SessionHooks().setOnSessionStart((input, invocation) -> {
+        SessionConfig config = new SessionConfig()
+                .setHooks(new SessionHooks().setOnSessionStart((input, invocation) -> {
                     startInputs.add(input);
                     return CompletableFuture.completedFuture(null);
                 }));