fix: bound polling loop timeouts by remaining deadline time

[Copilot]

---

Before the change?

• Polling loops in testShouldGetLastSessionId and testShouldGetSessionMetadataById used a fixed 30s timeout per Future.get() call inside a loop with a 10s deadline. If the RPC stalled, a single iteration could block 3× longer than the intended overall deadline.

After the change?

• Each iteration computes remaining = Math.max(1, deadline - currentTimeMillis()) and passes it as the per-iteration timeout, ensuring the overall wait is bounded by the 10s deadline:

long remaining = Math.max(1, deadline - System.currentTimeMillis());
lastId = client.getLastSessionId().get(remaining, TimeUnit.MILLISECONDS);

Pull request checklist

• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)
• mvn spotless:apply has been run to format the code
• mvn clean verify passes locally

Does this introduce a breaking change?

• Yes
• No

---

[Copilot]

Pull request overview

This PR updates two polling-based tests in CopilotSessionTest so that per-iteration Future.get() timeouts are bounded by the remaining overall deadline, preventing a single stalled RPC from blocking longer than intended.

Changes:

• Replaced fixed per-iteration Future.get(30s) timeouts with remaining-based timeouts against a 10s deadline.
• Added polling loops (with short sleeps) for getLastSessionId() and getSessionMetadata() to accommodate asynchronous persistence behavior.

Show a summary per file

File	Description
src/test/java/com/github/copilot/sdk/CopilotSessionTest.java	Bounds polling loop get() timeouts by remaining deadline time for last-session-id and session-metadata tests.

Copilot's findings

Comments suppressed due to low confidence (1)

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

• Same issue as the getLastSessionId polling loop: get(remaining, ...) can throw TimeoutException when remaining becomes small (down to 1ms), causing a premature test failure instead of continuing to poll until the 10s deadline. Handle TimeoutException inside the loop (retry until deadline) and consider capping the per-iteration timeout to avoid ultra-short timeouts near the end.

            long deadline = System.currentTimeMillis() + 10_000;
            while (System.currentTimeMillis() < deadline) {
                long remaining = Math.max(1, deadline - System.currentTimeMillis());
                metadata = client.getSessionMetadata(sessionId).get(remaining, TimeUnit.MILLISECONDS);
                if (metadata != null) {
                    break;
                }
                Thread.sleep(50);

• Files reviewed: 1/1 changed files
• Comments generated: 1