chore(build): 修改lombok依赖作用域为provided#198
Conversation
zhangyunan1994
commented
Mar 18, 2026
- 将lombok依赖的scope从compile改为provided
- 避免将lombok打包至最终产物中
- 确保编译时注解处理器正常工作
- 优化构建时依赖管理方式
- 将lombok依赖的scope从compile改为provided - 避免将lombok打包至最终产物中 - 确保编译时注解处理器正常工作 - 优化构建时依赖管理方式
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 此拉取请求旨在优化项目的构建时依赖管理。通过调整 Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
| @@ -59,7 +59,7 @@ | |||
| <groupId>org.projectlombok</groupId> | |||
| <artifactId>lombok</artifactId> | |||
| <version>1.18.26</version> | |||
There was a problem hiding this comment.
为了提高可维护性并方便版本管理,建议将依赖项的版本号提取到 <properties> 标签中进行统一管理。这有助于确保版本一致性,并简化未来的升级过程。
例如,您可以在 <properties> 中定义 lombok.version,然后在此处引用它:
<properties>
<maven.compiler.source>8</maven.compiler.source>
<maven.compiler.target>8</maven.compiler.target>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<lombok.version>1.18.26</lombok.version>
...
</properties>然后,此处的版本标签可以更新为使用该属性。对其他依赖项(如 junit、slf4j、okhttp 等)也推荐采用此方式。
| <version>1.18.26</version> | |
| <version>${lombok.version}</version> |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
感谢提交!已将此改动合入 v2.22.21 版本(commit 2ac8507)。关闭此 PR。 |
* Fix VideoSynthesis ignoring custom baseUrl for asyncCall/call requests (#199) The baseUrl parameter was stored but never passed to ApiServiceOption, causing task creation to always use the default mainland China endpoint. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add code and message fields to ImageGenerationOutput (#187) Error messages from wan2.6 image generation were silently dropped because the output class lacked these fields. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix Recognition NPE that silently kills the ASR stream (#161) Two null-safety issues could cause NPE during stream processing: 1. RecognitionResult.fromDashScopeResult: output could be null when the WebSocket message has no payload, causing NPE on cast and terminating the entire RxJava stream — all subsequent ASR results would be silently dropped. 2. Recognition.call onEvent: getSentence() could be null but was dereferenced without null check before calling isHeartbeat(). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix SSE EventSource connection leak on stream disposal (#37) The Flowable-based streamOut created an EventSource without storing a reference or setting a cancellable on the emitter. When subscribers disposed the Flowable, the EventSource kept running and the HTTP connection was never released. Now the EventSource is stored, set as cancellable on the emitter, and the close() method properly cancels it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add cache_creation_input_tokens to MultiModalConversationTokensDetails (#214) The field was missing, causing cache token usage from context caching to be silently dropped for multimodal conversations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add json_schema support to ResponseFormat (#172) ResponseFormat now supports type "json_schema" with a nested JsonSchemaFormat containing name, strict, and schema fields, consistent with the OpenAI-compatible structured output API. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add HappyHorse video generation model constant (#210) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Update README quickstart examples to use Generation API (#166) Replace deprecated Conversation/QWenConversationParam with Generation/GenerationParam. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix rxjava version incorrectly changed from 2.2.21 to 2.22.21 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add parentNodeId to WorkflowMessage and update version to 2.22.21 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix code formatting in RecognitionResult.java Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix thread-safety issue with concurrent SSE streams in OkHttpHttpClient (#37) Replace single activeEventSource with ConcurrentHashMap-backed Set to support multiple concurrent streaming requests safely. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove samples from jar packaging (#226) Remove build-helper-maven-plugin config that added samples/ as a source directory, causing package-less sample classes to pollute classpath root. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Change lombok dependency scope from compile to provided (#198) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Change slf4j-simple dependency scope to test (#173) Libraries should only depend on slf4j-api, not bind a specific logging implementation. Keep slf4j-simple available for tests only. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix race condition, task shadowing, and redundant annotation - OkHttpHttpClient: use AtomicBoolean to prevent EventSource leak when onFailure/onClosed fires before add() - VideoSynthesis: assign task parameter to this.task in constructor - ResponseFormat: remove redundant @Builder.Default on Boolean strict Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove unused Builder import from ResponseFormat Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Revert parentNodeId field from WorkflowMessage Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: zhansheng.lzs <zhansheng.lzs@alibaba-inc.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>