feat: 拆分微信小店独立模块 - #4117
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
This pull request is too large for Augment to review. The PR exceeds the maximum size limit of 75000 tokens (approximately 300000 characters) for automated code review. Please consider breaking this PR into smaller, more focused changes. |
There was a problem hiding this comment.
🟡 Changes recommended
BOM 目前未对本 PR 新增的 Store Starter / Solon 插件 artifact 做版本托管,可能影响依赖 BOM 的用户集成与升级体验。
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
该 PR 将“微信小店(Store)”能力从 weixin-java-channel 中拆分为独立的 weixin-java-store SDK,并新增对应的 Spring Boot / Solon 集成模块;同时对 weixin-java-channel 中保留的小店相关 API/模型进行 @Deprecated 标记,以引导用户迁移且保持既有行为不变。
Changes:
- 新增
weixin-java-store模块,并在根聚合构建中注册该模块。 - 新增 Store 的 Spring Boot Starter(单账号/多账号)与 Solon 插件(单账号/多账号)的自动装配/插件声明资源。
- 在
weixin-java-channel中对小店相关接口与大量请求/响应/模型类增加@Deprecated标记。
File summaries
| File | Description |
|---|---|
| pom.xml | 在根聚合模块中加入 weixin-java-store,纳入统一构建/发布链路。 |
| wx-java-bom/pom.xml | BOM 增加 weixin-java-store 的版本托管入口(但目前未覆盖 Store Starter/Solon 插件,见评论)。 |
| spring-boot-starters/pom.xml | 注册 Store 的 Spring Boot Starter 两个子模块。 |
| spring-boot-starters/wx-java-store-spring-boot-starter/src/main/resources/META-INF/spring/org.springframework.boot.autoconfigure.AutoConfiguration.imports | 声明 Store 单账号 Starter 的 AutoConfiguration 导入。 |
| spring-boot-starters/wx-java-store-spring-boot-starter/src/main/resources/META-INF/spring.factories | 兼容 Spring Boot 旧机制的自动装配声明。 |
| spring-boot-starters/wx-java-store-multi-spring-boot-starter/src/main/resources/META-INF/spring/org.springframework.boot.autoconfigure.AutoConfiguration.imports | 声明 Store 多账号 Starter 的 AutoConfiguration 导入。 |
| spring-boot-starters/wx-java-store-multi-spring-boot-starter/src/main/resources/META-INF/spring.factories | 兼容 Spring Boot 旧机制的自动装配声明。 |
| solon-plugins/pom.xml | 注册 Store 的 Solon 插件两个子模块。 |
| solon-plugins/wx-java-store-solon-plugin/src/main/resources/META-INF/solon/wx-java-store-solon-plugin.properties | 声明 Solon 插件入口与优先级。 |
| solon-plugins/wx-java-store-solon-plugin/src/test/resources/app.yml | Store Solon 插件测试资源(当前为空文件)。 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/WxChannelKfService.java | 将 Channel 内小店客服服务接口标记为 @Deprecated(建议补充迁移指引,见评论)。 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/WxChannelAfterSaleService.java | 将 Channel 内小店售后服务接口标记为 @Deprecated。 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/WxChannelProductService.java | 将 Channel 内小店商品服务接口标记为 @Deprecated。 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/** | 大量小店相关 Bean/Param/Response 增加 @Deprecated,用于迁移引导与边界划分。 |
Review details
- Files reviewed: 299/1242 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: da31b5dd85
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@copilot 再次review最新代码 |
以下是最新代码的 Review 报告: BOM 完整性 ✅
发现问题🔴 P0 — HTTP Response Body 双读 Bug(功能缺陷)文件: 两个 executor 在 try-with-resources 中同时声明 修复建议:先检查 Content-Type,若为 JSON 则只用 🟠 P1 — OkHttp
|
变更内容
weixin-java-storeSDK,Java 包名为com.binarywang.wxjava.store.*,不依赖weixin-java-channel。wx.store。weixin-java-channel的小店服务和模型,行为不变,并统一标记为@Deprecated。验证
mvn -pl weixin-java-channel,weixin-java-store,spring-boot-starters/wx-java-store-spring-boot-starter,spring-boot-starters/wx-java-store-multi-spring-boot-starter,solon-plugins/wx-java-store-solon-plugin,solon-plugins/wx-java-store-multi-solon-plugin -am test -Dmaven.test.skip=falsegit diff --check