#copilotCodingAgent tool adds follow-ups if active PR is open (#7222)

* checkpoint

* Initial plan

* Add public getStateForPR method and fix encapsulation violation

Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>

---------

Co-authored-by: Josh Spicer <23246594+joshspicer@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Author: Copilot

src/github/copilotRemoteAgent.ts

@@ -5,7 +5,7 @@
 
 import vscode from 'vscode';
 import { Repository } from '../api/api';
-import { COPILOT_LOGINS } from '../common/copilot';
+import { COPILOT_LOGINS, CopilotPRStatus } from '../common/copilot';
 import { commands } from '../common/executeCommands';
 import { Disposable } from '../common/lifecycle';
 import Logger from '../common/logger';
@@ -663,4 +663,8 @@ export class CopilotRemoteAgentManager extends Disposable {
 		const key = this._stateModel.makeKey(owner, repo, pullRequestNumber);
 		return this._stateModel.notifications.has(key);
 	}
+
+	public getStateForPR(owner: string, repo: string, prNumber: number): CopilotPRStatus {
+		return this._stateModel.get(owner, repo, prNumber);
+	}
 }

src/lm/tools/copilotRemoteAgentTool.ts

@@ -70,6 +70,7 @@ export class CopilotRemoteAgentTool implements vscode.LanguageModelTool<CopilotR
 			]);
 		}
 
+
 		/* __GDPR__
 			"remoteAgent.tool.invoke" : {
 				"hasExistingPR" : { "classification": "SystemMetaData", "purpose": "FeatureInsight" },
@@ -81,13 +82,23 @@ export class CopilotRemoteAgentTool implements vscode.LanguageModelTool<CopilotR
 			hasBody: body ? 'true' : 'false'
 		});
 
+		let pullRequestNumber: number | undefined;
 		if (existingPullRequest) {
-			const pullRequestNumber = parseInt(existingPullRequest, 10);
+			pullRequestNumber = parseInt(existingPullRequest, 10);
 			if (isNaN(pullRequestNumber)) {
 				return new vscode.LanguageModelToolResult([
 					new vscode.LanguageModelTextPart(vscode.l10n.t('Invalid pull request number: {0}', existingPullRequest))
 				]);
 			}
+		} else {
+			const { repo, owner } = targetRepo;
+			const activePR = targetRepo.fm.activePullRequest;
+			if (activePR && this.manager.getStateForPR(owner, repo, activePR.number)) {
+				pullRequestNumber = activePR.number;
+			}
+		}
+
+		if (pullRequestNumber) {
 			await this.manager.addFollowUpToExistingPR(pullRequestNumber, title, body);
 			return new vscode.LanguageModelToolResult([
 				new vscode.LanguageModelTextPart(vscode.l10n.t('Follow-up added to pull request #{0}.', pullRequestNumber)),