Skip to content

Javadoc fixes#238

Merged
edburns merged 1 commit into
mainfrom
edburns/copilot-sdk-pr-1486-review-comments
May 28, 2026
Merged

Javadoc fixes#238
edburns merged 1 commit into
mainfrom
edburns/copilot-sdk-pr-1486-review-comments

Conversation

@edburns
Copy link
Copy Markdown
Collaborator

@edburns edburns commented May 28, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 23:20
Copy link
Copy Markdown
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

Cleans up and fixes Javadoc references in RPC configuration/docs so generated Javadocs resolve correct types and read cleanly.

Changes:

  • Reflowed Javadoc text for MCP OAuth token storage defaulting in SessionConfig and ResumeSessionConfig.
  • Fixed a broken/ambiguous Javadoc @link in SectionOverride by fully qualifying SystemMessageMode.
Show a summary per file
File Description
src/main/java/com/github/copilot/rpc/SessionConfig.java Reflows Javadoc and corrects the CopilotClientMode link to resolve within com.github.copilot.rpc.
src/main/java/com/github/copilot/rpc/SectionOverride.java Updates Javadoc @link to com.github.copilot.SystemMessageMode#CUSTOMIZE so the reference resolves.
src/main/java/com/github/copilot/rpc/ResumeSessionConfig.java Mirrors the SessionConfig Javadoc reflow/link correction for resumed sessions.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

@edburns edburns merged commit 028decd into main May 28, 2026
12 checks passed
@edburns edburns deleted the edburns/copilot-sdk-pr-1486-review-comments branch May 28, 2026 23:27
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.

2 participants