Skip to content

Add skill for addressing comments#8599

Merged
alexr00 merged 2 commits intomainfrom
alexr00/military-lamprey
Mar 17, 2026
Merged

Add skill for addressing comments#8599
alexr00 merged 2 commits intomainfrom
alexr00/military-lamprey

Conversation

@alexr00
Copy link
Member

@alexr00 alexr00 commented Mar 17, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 17:08
@alexr00 alexr00 enabled auto-merge (squash) March 17, 2026 17:08
@alexr00 alexr00 self-assigned this Mar 17, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 17, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new language-model “skill” aimed at guiding an agent through addressing review comments on the active pull request, and extends the github-pull-request_activePullRequest tool to optionally refresh PR data and surface an update timestamp.

Changes:

  • Add address-pr-comments skill documentation and register it in package.json.
  • Extend github-pull-request_activePullRequest with an optional refresh input and add lastUpdatedAt to the JSON output.
  • Switch webpack.config.js TypeScript config resolution from invoking yarn to npx.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
webpack.config.js Changes the command used to run tsc --showConfig during webpack config resolution.
src/lm/tools/activePullRequestTool.ts Adds optional refresh behavior and returns an additional timestamp field in the tool result.
src/lm/skills/address-pr-comments/SKILL.md Introduces a new skill playbook for working through PR review comments.
package.json Registers the new skill and declares an inputSchema for the active PR tool (refresh).
Comments suppressed due to low confidence (1)

src/lm/tools/activePullRequestTool.ts:50

  • This tool accepts structured input now; consider typing the invocation options as vscode.LanguageModelToolInvocationOptions<PullRequestToolParams> instead of any to keep the implementation aligned with the inputSchema and avoid unsound casts.
	async invoke(options: vscode.LanguageModelToolInvocationOptions<any>, _token: vscode.CancellationToken): Promise<vscode.ExtendedLanguageModelToolResult | undefined> {
		let pullRequest = this._findActivePullRequest();

You can also share your feedback on Copilot code review. Take the survey.

@alexr00 alexr00 merged commit a135d25 into main Mar 17, 2026
6 checks passed
@alexr00 alexr00 deleted the alexr00/military-lamprey branch March 17, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants