Skip to content

Conversation

@wenytang-ms
Copy link
Contributor

@wenytang-ms wenytang-ms commented Oct 16, 2025

this feature does two things:

  1. Adjust the class info into a hummable language to make copilot easy to handle
  2. Add class javadoc into context, and remove the html tags from javadoc
file url java-deps cost item num
Configuration.java 13.78 ms 15 items
PathTypeAdapter.java 4.34ms 0 items
VersionChecker.java 9.6 ms 3 items
RuntimeDecompiler.java 15.57 ms 6 items
ModLauncherClassTracker.java 16.5 ms 3items
CookieReceiveEvent.java 14.42 ms 6 items
ResultEvent.java 4.57 ms 0 items

@wenytang-ms
Copy link
Contributor Author

wenytang-ms commented Oct 16, 2025

Before

file_path: /c:/Users/xxxxx/test/javacase/internal-api-priority-demo2/acme-commons/src/main/java/com/acme/common/Result.java
class:com.acme.common.Result|modifiers:public|publicMethods:success(QT;),failure(QString;),isSuccess(),getMessage(),getData()

file_path: /c:/Users/xxxxx/test/javacase/internal-api-priority-demo2/acme-core/src/main/java/com/acme/user/UserService.java
class:com.acme.user.UserService|modifiers:public|publicMethods:findById(QString;),deactivate(QString;),findByEmail(QString;)

Now

file_path: /c:/Users/xxxxx/test/javacase/internal-api-priority-demo2/acme-commons/src/main/java/com/acme/common/Result.java
Class: com.acme.common.Result
Signature: public class Result<T:>

Methods:
  - public static <T:> Result<T> success(T data)
  - public static <T:> Result<T> failure(String message)
  - public boolean isSuccess()
  - public String getMessage()
  - public T getData()

file_path: /c:/Users/xxxxx/test/javacase/internal-api-priority-demo2/acme-core/src/main/java/com/acme/user/UserService.java
Class: com.acme.user.UserService
Signature: public class UserService

JavaDoc:
``java
Result<User> r = userService.findById(id);
if (r.isSuccess()) log.info("user={}", r.getData().getId());
``

Methods:
  - // Find user by ID, cache first
      public Result<User> findById(String id)
  - // Deactivate user (idempotent)
      public Result<Void> deactivate(String id)
  - // Find user by email
      public Result<User> findByEmail(String email)

@wenytang-ms wenytang-ms marked this pull request as ready for review October 17, 2025 01:03
chagong
chagong previously approved these changes Oct 17, 2025
@wenytang-ms wenytang-ms merged commit c424e64 into main Oct 17, 2025
4 of 6 checks passed
@wenytang-ms wenytang-ms deleted the wenyt/cp2 branch October 17, 2025 03:38
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