Refactor to eliminate duplication: SessionListFilter extends SessionContext

Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>

Author: Copilot

src/main/java/com/github/copilot/sdk/events/SessionContextChangedEvent.java

@@ -6,6 +6,7 @@
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.github.copilot.sdk.json.SessionContext;
 
 /**
  * Event: session.context_changed
@@ -19,23 +20,18 @@
 public final class SessionContextChangedEvent extends AbstractSessionEvent {
 
     @JsonProperty("data")
-    private SessionContextChangedData data;
+    private SessionContext data;
 
     @Override
     public String getType() {
         return "session.context_changed";
     }
 
-    public SessionContextChangedData getData() {
+    public SessionContext getData() {
         return data;
     }
 
-    public void setData(SessionContextChangedData data) {
+    public void setData(SessionContext data) {
         this.data = data;
     }
-
-    @JsonIgnoreProperties(ignoreUnknown = true)
-    public record SessionContextChangedData(@JsonProperty("cwd") String cwd, @JsonProperty("gitRoot") String gitRoot,
-            @JsonProperty("repository") String repository, @JsonProperty("branch") String branch) {
-    }
 }

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

@@ -44,9 +44,11 @@ public String getCwd() {
      *
      * @param cwd
      *            the current working directory path
+     * @return this instance for method chaining
      */
-    public void setCwd(String cwd) {
+    public SessionContext setCwd(String cwd) {
         this.cwd = cwd;
+        return this;
     }
 
     /**
@@ -63,9 +65,11 @@ public String getGitRoot() {
      *
      * @param gitRoot
      *            the git root path
+     * @return this instance for method chaining
      */
-    public void setGitRoot(String gitRoot) {
+    public SessionContext setGitRoot(String gitRoot) {
         this.gitRoot = gitRoot;
+        return this;
     }
 
     /**
@@ -82,9 +86,11 @@ public String getRepository() {
      *
      * @param repository
      *            the repository in "owner/repo" format
+     * @return this instance for method chaining
      */
-    public void setRepository(String repository) {
+    public SessionContext setRepository(String repository) {
         this.repository = repository;
+        return this;
     }
 
     /**
@@ -101,8 +107,10 @@ public String getBranch() {
      *
      * @param branch
      *            the branch name
+     * @return this instance for method chaining
      */
-    public void setBranch(String branch) {
+    public SessionContext setBranch(String branch) {
         this.branch = branch;
+        return this;
     }
 }

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

@@ -4,14 +4,11 @@
 
 package com.github.copilot.sdk.json;
 
-import com.fasterxml.jackson.annotation.JsonInclude;
-import com.fasterxml.jackson.annotation.JsonProperty;
-
 /**
  * Filter options for listing sessions.
  * <p>
- * Allows filtering sessions by working directory context fields such as cwd,
- * git root, repository, or branch.
+ * Extends {@link SessionContext} to provide filtering capabilities with fluent
+ * setter methods that return the filter instance for method chaining.
  *
  * <h2>Example Usage</h2>
  *
@@ -28,29 +25,7 @@
  * @see com.github.copilot.sdk.CopilotClient#listSessions(SessionListFilter)
  * @since 1.0.0
  */
-@JsonInclude(JsonInclude.Include.NON_NULL)
-public class SessionListFilter {
-
-    @JsonProperty("cwd")
-    private String cwd;
-
-    @JsonProperty("gitRoot")
-    private String gitRoot;
-
-    @JsonProperty("repository")
-    private String repository;
-
-    @JsonProperty("branch")
-    private String branch;
-
-    /**
-     * Gets the current working directory filter.
-     *
-     * @return the cwd filter, or {@code null} if not set
-     */
-    public String getCwd() {
-        return cwd;
-    }
+public class SessionListFilter extends SessionContext {
 
     /**
      * Sets the filter for exact cwd match.
@@ -59,71 +34,48 @@ public String getCwd() {
      *            the current working directory to filter by
      * @return this filter for method chaining
      */
+    @Override
     public SessionListFilter setCwd(String cwd) {
-        this.cwd = cwd;
+        super.setCwd(cwd);
         return this;
     }
 
-    /**
-     * Gets the git root filter.
-     *
-     * @return the git root filter, or {@code null} if not set
-     */
-    public String getGitRoot() {
-        return gitRoot;
-    }
-
     /**
      * Sets the filter for git root directory.
      *
      * @param gitRoot
      *            the git root path to filter by
      * @return this filter for method chaining
      */
+    @Override
     public SessionListFilter setGitRoot(String gitRoot) {
-        this.gitRoot = gitRoot;
+        super.setGitRoot(gitRoot);
         return this;
     }
 
-    /**
-     * Gets the repository filter.
-     *
-     * @return the repository filter, or {@code null} if not set
-     */
-    public String getRepository() {
-        return repository;
-    }
-
     /**
      * Sets the filter for repository (in "owner/repo" format).
      *
      * @param repository
      *            the repository identifier to filter by
      * @return this filter for method chaining
      */
+    @Override
     public SessionListFilter setRepository(String repository) {
-        this.repository = repository;
+        super.setRepository(repository);
         return this;
     }
 
-    /**
-     * Gets the branch filter.
-     *
-     * @return the branch filter, or {@code null} if not set
-     */
-    public String getBranch() {
-        return branch;
-    }
-
     /**
      * Sets the filter for git branch.
      *
      * @param branch
      *            the branch name to filter by
      * @return this filter for method chaining
      */
+    @Override
     public SessionListFilter setBranch(String branch) {
-        this.branch = branch;
+        super.setBranch(branch);
         return this;
     }
 }