Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Pull request overview

该 PR 为视频号小店商品能力补充独立的赠品、限时抢购、库存及商品辅助服务，并通过委托保留既有 WxChannelProductService 兼容性。

Changes:

• 新增领域服务接口、实现类及 WxChannelService 访问器。
• 新增限时抢购更新、库存流水及 6 个商品辅助接口的请求/响应 Bean。
• 新增 TestNG 测试并启用模块测试套件；审核策略与额度接口仍未实现。

Reviewed changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 3 comments.

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/WxChannelProductStockService.java:26

• #4002 列出的库存子 API 名称是 updateStockFast、getStock 和 getStockBatch，但这里仍只暴露既有的 updateStock、getSkuStock 和 getSkuStockBatch。拆分出新的公开领域服务后，调用方仍无法按完整 API 名称使用这些接口；建议补充规范命名的方法，并保留当前短名作为兼容委托。

  WxChannelBaseResponse updateStock(String productId, String skuId, Integer diffType, Integer num)

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/impl/WxChannelLimitedDiscountServiceImpl.java:36

• 当前针对新限时抢购服务的测试只覆盖了 updateLimitTask；addLimitTask、listLimitTask、stopLimitTask 和 deleteLimitTask 的新实现没有直接验证。旧的 WxChannelProductServiceImplTest 也未被 testng.xml 纳入本次测试套件，因此这些抽取后的路径或请求格式回归仍可能漏过；请补齐端点级单元测试。

  public LimitTaskAddResponse addLimitTask(LimitTaskParam param) throws WxErrorException {
    String reqJson = JsonUtils.encode(param);
    String resJson = shopService.post(ADD_LIMIT_TASK_URL, reqJson);
    return ResponseUtils.decode(resJson, LimitTaskAddResponse.class);

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/impl/WxChannelProductStockServiceImpl.java:37

• 当前库存服务测试只覆盖了 getStockFlow；本次抽取后的 updateStock、getSkuStock 和 getSkuStockBatch 都没有直接的请求/响应测试。旧的商品服务测试未被当前 testng.xml 套件执行，库存接口的 URL、序列化或响应类型回归可能因此漏过；请补齐这些测试。

  public WxChannelBaseResponse updateStock(String productId, String skuId, Integer diffType, Integer num)
    throws WxErrorException {
    SkuStockParam param = new SkuStockParam(productId, skuId, diffType, num);
    String reqJson = JsonUtils.encode(param);
    String resJson = shopService.post(SPU_UPDATE_STOCK_URL, reqJson);

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/assistant/CategoryPreCheckParam.java:19

• 这里将 cat_id 建模为 Long，与本 PR 所述“标识符保持字符串”不一致；同模块现有 PassCategoryInfo.catId 和 CatsV2.catId 也使用 String。这会把公共 API 的类型和序列化契约固定为数值，无法保留字符串标识符的原始形式；请改为 String，并同步调整调用方测试。

  private Long catId;

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/assistant/ExternalProductMappingNewParam.java:20

• 这里将 cat_id 建模为 Long，与本 PR 所述“标识符保持字符串”及同模块现有类目 ID 类型不一致。该公共请求 Bean 会把标识符序列化为数值，不能保持字符串形式；请改为 String，并同步调整测试中的赋值。

  private Long catId;

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/assistant/ExternalProductMappingParam.java:19

• 这里将 cat_id 建模为 Long，与本 PR 所述“标识符保持字符串”及同模块 PassCategoryInfo.catId、CatsV2.catId 的类型约定不一致。该公共请求 Bean 会因此把标识符序列化为数值而不是字符串；请改为 String，并同步调整测试中的赋值。

  private Long catId;

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/assistant/ProductBrandRecommendParam.java:20

• 这里将 cat_id 建模为 Long，与本 PR 所述“标识符保持字符串”不一致；同模块现有类对类目 ID 使用 String。这会使该公共请求 Bean 只能按数值处理标识符，丢失字符串契约；请改为 String，并同步调整测试中的赋值。

  private Long catId;

weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/assistant/ProductBrandRecommendResponse.java:21

• brand_id 是对外暴露的标识符，但这里使用了 Long；本 PR 明确要求标识符保持字符串，而同模块的 SpuInfo.brandId、BasicBrand.brandId 等也使用 String。如果接口返回带引号、前导零或超出数值语义的标识符，该类型会破坏原始值；请改为 String。

  private Long brandId;

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.