Skip to content

Commit 648e3f1

Browse files
Copilotedburns
andauthored
Update getSessionMetadata to match final upstream PR #899 state
Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/1dcd7e77-9de7-4ada-81f4-2a0b425919fb Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
1 parent b6b0ab6 commit 648e3f1

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
*
1616
* @see com.github.copilot.sdk.CopilotClient#getSessionMetadata(String)
1717
* @see SessionMetadata
18-
* @since 1.0.0
18+
* @since 1.2.0
1919
*/
2020
@JsonInclude(JsonInclude.Include.NON_NULL)
2121
public record GetSessionMetadataResponse(

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
* }</pre>
3030
*
3131
* @see com.github.copilot.sdk.CopilotClient#listSessions()
32+
* @see com.github.copilot.sdk.CopilotClient#getSessionMetadata(String)
3233
* @see com.github.copilot.sdk.CopilotClient#resumeSession(String,
3334
* ResumeSessionConfig)
3435
* @since 1.0.0

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -698,12 +698,12 @@ void testShouldDeleteSession() throws Exception {
698698
/**
699699
* Verifies that session metadata can be retrieved by ID.
700700
* <p>
701-
* TODO: Re-enable once test harness CAPI proxy supports this test's session
702-
* lifecycle.
701+
* Disabled until {@code .lastmerge} is advanced past upstream PR #899 which
702+
* adds the {@code should_get_session_metadata} snapshot.
703703
*
704704
* @see Snapshot: session/should_get_session_metadata
705705
*/
706-
@Disabled("Needs test harness CAPI proxy support")
706+
@Disabled("Snapshot not available until .lastmerge is advanced past upstream PR #899")
707707
@Test
708708
void testShouldGetSessionMetadata() throws Exception {
709709
ctx.configureForTest("session", "should_get_session_metadata");

0 commit comments

Comments
 (0)