Skip to content

Commit 64f316a

Browse files
committed
filter out completed/closed PRs
1 parent ded8c43 commit 64f316a

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

src/github/copilotApi.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -355,6 +355,7 @@ export interface SessionInfo {
355355
premium_requests: number;
356356
error: string | null;
357357
resource_global_id?: string
358+
resource_state: 'open' | 'closed' | 'draft' | 'merged';
358359
}
359360

360361
export interface SessionPullRequestInfo {

src/github/copilotRemoteAgent.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -886,7 +886,7 @@ export class CopilotRemoteAgentManager extends Disposable {
886886
const sessionMap = new Map<string, SessionInfo[]>();
887887

888888
for (const session of sessions) {
889-
if (!session.resource_global_id) {
889+
if (!session.resource_global_id || session.resource_state === 'closed' || session.resource_state === 'merged') {
890890
continue;
891891
}
892892
const existing = sessionMap.get(session.resource_global_id) || [];
@@ -959,6 +959,7 @@ export class CopilotRemoteAgentManager extends Disposable {
959959
} catch (error) {
960960
Logger.error(`Failed to provide coding agents information: ${error}`, CopilotRemoteAgentManager.ID);
961961
} finally {
962+
//todo, previously we set codingAgentPRsPromise to undefined here, but that caused issues with multiple simultaneous calls
962963
}
963964
return [];
964965
}

0 commit comments

Comments
 (0)