Skip to content

Conversation

@wenytang-ms
Copy link
Contributor

@wenytang-ms wenytang-ms commented Nov 10, 2025

  1. skip binary jar (may always 3rd lib) java doc
  2. skip more common libs import content

@wenytang-ms wenytang-ms requested a review from Copilot November 10, 2025 05:23
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

This PR optimizes JavaDoc extraction performance by skipping expensive operations for binary types and expands the list of common well-known library packages to reduce unnecessary processing for external dependencies that Copilot already understands.

  • Adds early return in extractRelevantJavaDocContent to skip expensive JavaDoc extraction for binary types (JAR files)
  • Expands SKIP_COMMON_JDK_PACKAGES to include 50+ well-known libraries (Spring, Jakarta EE, testing frameworks, logging, Jackson, Guava, Apache Commons, etc.)
  • Removes redundant binary type handling logic that involved costly operations like JAR I/O, network downloads, and HTML parsing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wenytang-ms wenytang-ms merged commit e6e0d7a into main Nov 10, 2025
10 of 12 checks passed
@wenytang-ms wenytang-ms deleted the wenyt/cc1 branch November 10, 2025 05: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.

3 participants