job51-gitlab-cr-node-jt-1 2.5.5 → 2.5.6

package/index.js

@@ -81,7 +81,7 @@ class GitLabCodeReviewer {
         debugLog(`========== Diff Block ${blockIndex} 开始 ==========`);
         debugLog(`文件路径：${diffObject.new_path || diffObject.old_path}`);
         debugLog(`line_info: old_start=${diffObject.line_info?.old_start}, old_count=${diffObject.line_info?.old_count}, new_start=${diffObject.line_info?.new_start}, new_count=${diffObject.line_info?.new_count}`);
-        debugLog(`diff 内容：${diffObject.diff}`);
+        debugLog(`diff 内容预览（前 500 字符）：${diffObject.diff?.substring(0, 500)}`);
         debugLog(`========== Diff Block ${blockIndex} 结束 ==========`);
 
         // 构造包含元数据的 diff 内容
@@ -210,7 +210,7 @@ ${diffObject.diff}`;
 
         // 打印报告内容前 500 字符（避免过长）
         const reportPreview = claudeResult?.length > 500 ? claudeResult.substring(0, 500) + '...' : claudeResult;
-        debugLog(`AI 审核报告内容预览：${claudeResult}`);
+        debugLog(`AI 审核报告内容预览：${reportPreview}`);
 
         // 使用正则提取 LINE_INFO 内容（支持换行）
         const lineInfoMatch = claudeResult?.match(/<LINE_INFO>\s*\[([^\]]*)\]\s*<\/LINE_INFO>/);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "2.5.5",
+  "version": "2.5.6",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {

package/diff

@@ -1,636 +0,0 @@
-diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
-index e3829e9ad..92d19ba89 100644
---- a/.gitlab-ci.yml
-+++ b/.gitlab-ci.yml
-@@ -1,4 +1,30 @@
--include:
--  - project: 'linton.cao/ai-public-templates'
--    file: '/code-review/gitlab-ci.yml'
--    ref: main
-
-+stages:
-+  - ai
-+ai:
-+  stage: ai
-+  # 调整规则以适应您想要触发作业的方式：
-+  # - 手动运行
-+  # - 合并请求事件
-+  # - 当评论包含 '@claude' 时的 web/API 触发
-+  rules:
-+    - if: '$CI_PIPELINE_SOURCE == "web"'
-+    - if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
-+  variables:
-+    GIT_STRATEGY: fetch
-+  script:
-+    - echo "Running AI code review for merge request $CI_MERGE_REQUEST_IID"
-+    - echo "Project ID $CI_PROJECT_ID"
-+    - echo "MR IID $CI_MERGE_REQUEST_IID"
-+    - echo "API URL $CI_API_V4_URL"
-+    # 构建 API URL
-+    - API_URL="$CI_API_V4_URL/projects/$CI_PROJECT_ID/merge_requests/$CI_MERGE_REQUEST_IID/changes"
-+    - >
-+      curl --header "PRIVATE-TOKEN: glpat-gYMwUhqyCcH8UUZ8xykA"
-+      "$API_URL" > mr_changes.json
-+    - cat mr_changes.json
-+    - >
-+      claude
-+      -p "请分析以下合并请求的代码变更并提供代码审查反馈。特别关注潜在的错误、代码质量、最佳实践和安全问题。变更详情: $(cat mr_changes.json)"
-+      --permission-mode acceptEdits
-+      --allowedTools "Bash(*) Read(*) Edit(*) Write(*) mcp__gitlab"
-+      --debug
-
-diff --git a/51job-dev-boot-bbs/pom.xml b/51job-dev-boot-bbs/pom.xml
-index c63beac38..6be1ef1e0 100644
---- a/51job-dev-boot-bbs/pom.xml
-+++ b/51job-dev-boot-bbs/pom.xml
-@@ -103,7 +103,7 @@
-         <dependency>
-             <groupId>org.jsoup</groupId>
-             <artifactId>jsoup</artifactId>
--            <version>1.14.6</version>
-+            <version>1.14.3</version>
-         </dependency>
-
-         <!-- 引入es      -->
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/client/feign/UserTaskClient.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/client/feign/UserTaskClient.java
-deleted file mode 100644
-index ced593c06..000000000
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/client/feign/UserTaskClient.java
-+++ /dev/null
-@@ -1,49 +0,0 @@
--package com.job51.dev.bbs.client.feign;
--
--import com.job51.dev.bbs.entity.dto.UserTaskPop;
--import com.job51.dev.common.entity.dto.Result;
--import org.springframework.cloud.openfeign.FeignClient;
--import org.springframework.web.bind.annotation.GetMapping;
--import org.springframework.web.bind.annotation.PathVariable;
--import org.springframework.web.bind.annotation.PostMapping;
--import org.springframework.web.bind.annotation.RequestBody;
--import org.springframework.web.bind.annotation.RequestParam;
--
--import java.util.List;
--
--/**
-- * 用户任务API
-- */
--@FeignClient(value = "51job-user-task-v1", contextId = "51job-user-task")
--public interface UserTaskClient {
--
--    /**
--     * 获取用户是否展示任务模块
--     *
--     * @param accountId 用户id
--     * @return 是否展示：true-展示，false-不展示，默认false
--     */
--    @GetMapping("/inner/user-task" + "/user-task/module-control/{accountId}")
--    Result<Boolean> userTaskModuleControl(@PathVariable Long accountId);
--
--
--    /**
--     * 获取第一个 未完成的 有弹窗的 任务信息
--     *
--     * @param accountId 用户id
--     * @return 用户弹窗信息
--     */
--    @GetMapping("/inner/user-task" + "/user-task/pop/{accountId}")
--    Result<UserTaskPop> getUserTaskPop(@PathVariable Long accountId);
--
--    /**
--     * 判断当前用户是否为对应任务目标人群
--     *
--     * @param accountId  用户id
--     * @param actionType 任务类型
--     * @return Boolean
--     */
--    @GetMapping("/inner/user-task" + "/user-task/target-people")
--    Result<Boolean> isTargetPeople(@RequestParam Long accountId, @RequestParam String actionType);
--
--}
-
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/TestMultiIssuesController.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/TestMultiIssuesController.java
-deleted file mode 100644
-index 9b31b0d75..000000000
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/TestMultiIssuesController.java
-+++ /dev/null
-@@ -1,72 +0,0 @@
--package com.job51.dev.bbs.controller;
--
--import com.job51.dev.bbs.service.ForumService;
--import com.job51.dev.bbs.service.PostService;
--import com.job51.dev.bbs.service.UserService;
--import org.springframework.beans.factory.annotation.Autowired;
--import org.springframework.web.bind.annotation.*;
--
--/**
-- * 测试多问题 diff 场景的 Controller
-- * 用于测试 AI 代码审查在同一个 diff 块中检测多个问题的场景
-- */
--@RestController
--@RequestMapping("/api/test")
--public class TestMultiIssuesController {
--
--    @Autowired
--    private ForumService forumService;
--
--    @Autowired
--    private PostService postService;
--
--    @Autowired
--    private UserService userService;
--
--    /**
--     * 测试接口 - 包含多个空指针风险问题
--     * 这些问题都在同一个 diff 块中，用于测试多问题评论发布
--     */
--    @GetMapping("/forum/data")
--    public Object getForumData(@RequestParam String forumId,
--                               @RequestParam String userId,
--                               @RequestParam String postId) {
--        // 问题 1: forumService 可能为 null（未判空）
--        Object forumInfo = forumService.getForumInfo(forumId);
--
--        // 问题 2: postService 可能为 null（未判空）
--        Object postDetail = postService.getPostDetail(postId);
--
--        // 问题 3: userService 可能为 null（未判空）
--        Object userInfo = userService.getUserInfo(userId);
--
--        // 问题 4: 链式调用未判空 - forumInfo 可能为 null
--        String forumName = forumInfo.toString();
--
--        // 问题 5: 链式调用未判空 - postDetail 可能为 null
--        Integer postCount = postDetail.hashCode();
--
--        return new Object() {
--            public String forumName = forumName;
--            public Integer postCount = postCount;
--            public Object userInfo = userInfo;
--        };
--    }
--
--    /**
--     * 另一个测试接口 - 更多空指针场景
--     */
--    @PostMapping("/post/create")
--    public Object createPost(@RequestBody Object postData) {
--        // 问题 6: postData 未判空
--        String title = postData.toString();
--
--        // 问题 7: forumService 可能为 null
--        Object result = forumService.createPost(title);
--
--        // 问题 8: result 可能为 null
--        Boolean success = result.equals(true);
--
--        return success;
--    }
--}
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/open/UserController.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/open/UserController.java
-index b3b20455c..b1d8c318c 100644
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/open/UserController.java
-+++ b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/controller/open/UserController.java
-@@ -2,11 +2,7 @@ package com.job51.dev.bbs.controller.open;
-
- import com.job51.dev.bbs.entity.Base64DecodeMultipartFile;
- import com.job51.dev.bbs.entity.dto.UserDTO;
--import com.job51.dev.bbs.entity.query.*;
--import com.job51.dev.bbs.entity.vo.PagesListVO;
--import com.job51.dev.bbs.entity.vo.UserDetailInfoVO;
--import com.job51.dev.bbs.entity.vo.UserInfoVO;
--import com.job51.dev.bbs.entity.vo.VestUserVO;
-+import com.job51.dev.bbs.entity.query.UserQuery;
- import com.job51.dev.bbs.entity.vo.UserVO;
- import com.job51.dev.bbs.enums.DefaultAvatarEnum;
- import com.job51.dev.bbs.exception.BbsBusinessException;
-@@ -46,7 +42,6 @@ import static com.job51.dev.bbs.constant.BbsConstant.*;
- public class UserController {
-     private static final String API_PREFIX = "/open/bbs/users";
-     private static final String API_NOAUTH_PREFIX = "/open/noauth/bbs/users";
--    private static final String ADMIN_API_PREFIX = "/admin/bbs/users";
-
-     @Resource
-     private WordFilterUtils wordFilterUtils;
-@@ -202,161 +197,6 @@ public class UserController {
-         }
-     }
-
--    /**
--     * 网关登录
--     * @param loginQuery 登录参数
--     * @return 用户信息
--     */
--    @PostMapping(API_NOAUTH_PREFIX + "/login")
--    public Result<UserDTO> login(@RequestBody LoginQuery loginQuery) {
--        return Result.success(userService.login(loginQuery));
--    }
--
--    /**
--     * 根据用户 ID 获取用户信息
--     * @param userId 用户 id
--     * @return 用户信息
--     */
--    @GetMapping(API_NOAUTH_PREFIX + "/{userId}/info")
--    public Result<UserDTO> getUserInfo(@PathVariable Long userId) {
--        return Result.success(userService.getUserInfo(userId));
--    }
--
--    /**
--     * 上传头像，并返回头像的访问链接
--     * @param image 头像
--     * @return 头像的访问链接
--     */
--    @PostMapping(value = API_PREFIX + "/avatar/upload", consumes = MediaType.MULTIPART_FORM_DATA_VALUE)
--    public Result<String> uploadAvatar(@RequestPart("image") MultipartFile image) {
--        return Result.success(userService.uploadAvatar(image));
--    }
--
--    /**
--     * 同步用户简历信息
--     * @param userId 用户 id
--     * @return 用户信息
--     */
--    @PostMapping(API_PREFIX + "/sync/{userId}")
--    public Result<UserDTO> syncUserInfo(@PathVariable Long userId) {
--        return Result.success(userService.syncUserInfo(userId));
--    }
--
--    /**
--     * 注销 bbs 账号
--     * @param foreignId 其他系统的用户 id
--     * @param fromDomain 来源
--     * @return 是否成功
--     */
--    @PostMapping(API_PREFIX + "/logoff")
--    public Result<Boolean> logoff(@RequestParam Long foreignId, @RequestParam String fromDomain) {
--        return Result.success(userService.logoff(foreignId, fromDomain));
--    }
--
--    /**
--     * 用户信息全量导入 redis
--     * @return 结果
--     */
--    @PostMapping(ADMIN_API_PREFIX + "/redis/push")
--    public Result<String> pushAllInfoToRedis() {
--        return Result.success(userService.pushAllInfoToRedis());
--    }
--
--    /**
--     * 用户信息增量导入 redis
--     * @return 结果
--     */
--    @PostMapping(ADMIN_API_PREFIX + "/redis/increment")
--    public Result<String> incrementPushInfoToRedis() {
--        return Result.success(userService.incrementPushInfoToRedis());
--    }
--
--    // ==================== 后台管理接口 ====================
--
--    /**
--     * 创建马甲号
--     * @param query 入参
--     * @param adminId 管理员 ID
--     * @return 是否成功
--     */
--    @PostMapping(ADMIN_API_PREFIX + "/vest")
--    public Result<Boolean> addVest(@RequestBody VestAddQuery query, @RequestParam String adminId) {
--        return Result.success(userService.addVest(query, adminId));
--    }
--
--    /**
--     * 马甲号列表
--     * @param query 查询参数
--     * @return 马甲号列表
--     */
--    @GetMapping(ADMIN_API_PREFIX + "/vest")
--    public Result<PagesListVO<VestUserVO>> indexVest(VestUserQuery query) {
--        return Result.success(userService.indexVest(query));
--    }
--
--    /**
--     * 马甲号详情
--     * @param id 马甲号 ID
--     * @return 马甲号详情
--     */
--    @GetMapping(ADMIN_API_PREFIX + "/vest/{id}")
--    public Result<VestUserVO> showVest(@PathVariable Long id) {
--        return Result.success(userService.showVest(id));
--    }
--
--    /**
--     * 编辑马甲号
--     * @param query 入参
--     * @param adminId 管理员 ID
--     * @return 是否成功
--     */
--    @PutMapping(ADMIN_API_PREFIX + "/vest")
--    public Result<Boolean> editVest(@RequestBody VestEditQuery query, @RequestParam String adminId) {
--        return Result.success(userService.editVest(query, adminId));
--    }
--
--    /**
--     * 用户管理列表
--     * @param query 查询参数
--     * @return 用户列表
--     */
--    @GetMapping(ADMIN_API_PREFIX)
--    public Result<PagesListVO<UserInfoVO>> indexUser(UserInfoQuery query) {
--        return Result.success(userService.indexUser(query));
--    }
--
--    /**
--     * 后台 - 用户详情
--     * @param id 用户 ID
--     * @return 用户详情
--     */
--    @GetMapping(ADMIN_API_PREFIX + "/{id}")
--    public Result<UserDetailInfoVO> show(@PathVariable Long id) {
--        return Result.success(userService.show(id));
--    }
--
--    /**
--     * 内部调用，查询用户详情
--     * @param foreignId bbs 外部 id
--     * @param fromDomain 来源
--     * @return 用户详情
--     */
--    @GetMapping(ADMIN_API_PREFIX + "/detail")
--    public Result<UserDTO> show(@RequestParam String foreignId, @RequestParam String fromDomain) {
--        return Result.success(userService.show(foreignId, fromDomain));
--    }
--
--    /**
--     * 用户封禁
--     * @param adminId 管理员邮箱
--     * @param query 入参
--     * @return 是否成功
--     */
--    @PostMapping(ADMIN_API_PREFIX + "/ban")
--    public Result<Boolean> banUser(@RequestParam String adminId, @RequestBody BanUserQuery query) {
--        return Result.success(userService.banUser(adminId, query));
--    }
--
-     private Boolean isDefaultNickname(String nickname, Integer foreignType) {
-         String regex;
-         if (FOREIGN_TYPE_YJS.contains(foreignType)) {
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/entity/dto/UserTaskPop.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/entity/dto/UserTaskPop.java
-deleted file mode 100644
-index 0884e34f8..000000000
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/entity/dto/UserTaskPop.java
-+++ /dev/null
-@@ -1,48 +0,0 @@
--package com.job51.dev.bbs.entity.dto;
--
--import lombok.AllArgsConstructor;
--import lombok.Builder;
--import lombok.Data;
--import lombok.NoArgsConstructor;
--
--import java.io.Serializable;
--
--/**
-- * @author jingtao
-- * @version V1.0
-- * @date 2026/3/25
-- **/
--@Data
--@Builder
--@NoArgsConstructor
--@AllArgsConstructor
--public class UserTaskPop implements Serializable {
--    /**
--     * 任务ID
--     */
--    private Long taskId;
--    /**
--     * 任务动作类型
--     */
--    private String actionType;
--    /**
--     * 任务code
--     *
--     * @link com.job51.boot.user.task.biz.model.vo.UserTaskVO#code
--     */
--    private String contentId;
--    /**
--     * 任务名
--     *
--     * @link com.job51.boot.user.task.biz.model.vo.UserTaskVO#name
--     */
--    private String popupContent;
--    /**
--     * 跳转连接
--     */
--    private String jumpUrl;
--    /**
--     * （弹窗）图片连接
--     */
--    private String imgUrl;
--}
-
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/sdk/UserTaskSDK.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/sdk/UserTaskSDK.java
-deleted file mode 100644
-index 731ba220a..000000000
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/sdk/UserTaskSDK.java
-+++ /dev/null
-@@ -1,29 +0,0 @@
--package com.job51.dev.bbs.sdk;
--
--import com.job51.dev.bbs.client.feign.UserTaskClient;
--import com.job51.dev.bbs.entity.dto.UserTaskPop;
--import com.job51.dev.common.entity.dto.Result;
--import org.springframework.stereotype.Component;
--
--import javax.annotation.Resource;
--
--/**
-- * @author jingtao
-- * @version V1.0
-- * @date 2026/3/25
-- **/
--@Component
--public class UserTaskSDK {
--
--
--    @Resource
--    private UserTaskClient userTaskClient;
--
--    public UserTaskPop userTaskModuleControl(Long accountId) {
--        Result<UserTaskPop> userTaskPop = userTaskClient.getUserTaskPop(accountId);
--        if (userTaskPop.requestSucceeded() && userTaskPop.getResultbody() != null) {
--            return userTaskPop.getResultbody();
--        }
--        return null;
--    }
--}
-
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java
-index a4ac26619..e64cd4e8d 100644
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java
-+++ b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java
-@@ -4,11 +4,8 @@ import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
- import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
- import com.job51.dev.bbs.entity.bo.ForumCategory;
- import com.job51.dev.bbs.entity.bo.ForumCategoryForum;
--import com.job51.dev.bbs.entity.dto.UserTaskPop;
- import com.job51.dev.bbs.mapper.ForumCategoryForumMapper;
--import com.job51.dev.bbs.sdk.UserTaskSDK;
- import com.job51.dev.bbs.service.IForumCategoryForumService;
--import lombok.extern.slf4j.Slf4j;
- import org.springframework.stereotype.Service;
- import org.springframework.util.CollectionUtils;
-
-@@ -19,16 +16,12 @@ import java.util.List;
-  * @author pengtao.lv
-  * @since 2022/11/23
-  */
--@Slf4j
- @Service
- public class ForumCategoryForumServiceImpl extends ServiceImpl<ForumCategoryForumMapper, ForumCategoryForum>
-         implements IForumCategoryForumService {
-     @Resource
-     private ForumCategoryForumMapper forumCategoryForumMapper;
-
--    @Resource
--    private UserTaskSDK userTaskSDK;
--
-     @Override
-     public List<ForumCategory> getForumCategoryListByForumId(Long forumId) {
-         return forumCategoryForumMapper.getForumCategoryListByForumId(forumId);
-@@ -36,10 +29,6 @@ public class ForumCategoryForumServiceImpl extends ServiceImpl<ForumCategoryForu
-
-     @Override
-     public Long getNewForumCategoryByForumId(Long forumId) {
--        UserTaskPop userTaskPop = userTaskSDK.userTaskModuleControl(forumId);
--        if (userTaskPop != null && userTaskPop.getContentId() != null){
--            return Long.parseLong(userTaskPop.getContentId());
--        }
-         LambdaQueryWrapper<ForumCategoryForum> queryWrapper = new LambdaQueryWrapper<>();
-         queryWrapper.select(ForumCategoryForum::getForumCategoryId);
-         queryWrapper.eq(ForumCategoryForum::getForumId, forumId);
-diff --git a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/TestMultiIssuesService.java b/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/TestMultiIssuesService.java
-deleted file mode 100644
-index 2d5b09bfb..000000000
---- a/51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/TestMultiIssuesService.java
-+++ /dev/null
-@@ -1,75 +0,0 @@
--package com.job51.dev.bbs.service.impl;
--
--import com.job51.dev.bbs.mapper.ForumMapper;
--import com.job51.dev.bbs.mapper.PostMapper;
--import com.job51.dev.bbs.mapper.UserMapper;
--import org.springframework.beans.factory.annotation.Autowired;
--import org.springframework.stereotype.Service;
--
--import java.util.List;
--import java.util.Map;
--
--/**
-- * 测试多问题 diff 场景的 Service
-- * 用于测试 AI 代码审查在同一个 diff 块中检测多个问题的场景
-- */
--@Service
--public class TestMultiIssuesService {
--
--    @Autowired
--    private ForumMapper forumMapper;
--
--    @Autowired
--    private PostMapper postMapper;
--
--    @Autowired
--    private UserMapper userMapper;
--
--    /**
--     * 测试方法 - 包含多个空指针风险问题
--     */
--    public Map<String, Object> getTestData(String forumId, String userId) {
--        // 问题 1: forumMapper 可能为 null
--        Object forum = forumMapper.selectById(forumId);
--
--        // 问题 2: 链式调用 - forum 可能为 null
--        String forumName = forum.toString();
--
--        // 问题 3: userMapper 可能为 null
--        Object user = userMapper.selectById(userId);
--
--        // 问题 4: 链式调用 - user 可能为 null
--        String userName = user.toString();
--
--        // 问题 5: postMapper 可能为 null
--        List<Object> posts = postMapper.selectByForumId(forumId);
--
--        // 问题 6: 集合未判空直接访问
--        Object firstPost = posts.get(0);
--
--        return new java.util.HashMap<String, Object>() {{
--            put("forumName", forumName);
--            put("userName", userName);
--            put("firstPost", firstPost);
--        }};
--    }
--
--    /**
--     * 测试方法 2 - 更多空指针场景
--     */
--    public Object processData(Object inputData) {
--        // 问题 7: inputData 未判空
--        String dataStr = inputData.toString();
--
--        // 问题 8: 链式调用未判空
--        Integer dataLen = dataStr.length();
--
--        // 问题 9: forumMapper 可能为 null
--        Integer count = forumMapper.countAll();
--
--        // 问题 10: 自动拆箱可能 NPE
--        boolean isPositive = count > 0;
--
--        return isPositive ? dataLen : -1;
--    }
--}
-diff --git a/51job-dev-boot-subscribe/src/main/java/com/job51/dev/subscribe/service/impl/UserSubscribeInfoServiceImpl.java b/51job-dev-boot-subscribe/src/main/java/com/job51/dev/subscribe/service/impl/UserSubscribeInfoServiceImpl.java
-index b68722c12..99c04c5f0 100644
---- a/51job-dev-boot-subscribe/src/main/java/com/job51/dev/subscribe/service/impl/UserSubscribeInfoServiceImpl.java
-+++ b/51job-dev-boot-subscribe/src/main/java/com/job51/dev/subscribe/service/impl/UserSubscribeInfoServiceImpl.java
-@@ -167,6 +167,7 @@ public class UserSubscribeInfoServiceImpl extends ServiceImpl<UserSubscribeInfoM
-         List<UserSubscribeInfo> addList = new ArrayList<>();
-
-         for (SubjectSubscribeAddQuery item : query.getItems()) {
-+            UserSubscribeInfo subscribeInfo = new UserSubscribeInfo();
-
-             //验证一遍专区id
-             List<ActivityZoneDTO> activityZoneList = activityZoneService.getActivityZoneDataByIdList(Collections.singletonList(item.getSubjectId()));
-@@ -186,7 +187,7 @@ public class UserSubscribeInfoServiceImpl extends ServiceImpl<UserSubscribeInfoM
-                 continue;
-             }
-
--            UserSubscribeInfo subscribeInfo = new UserSubscribeInfo();
-+
-             subscribeInfo.setAccountId(accountId);
-             subscribeInfo.setSubscribeType(SubscribeEnum.SUBJECT_SUBSCRIBE.getSubscribeType());
-             subscribeInfo.setSubjectId(item.getSubjectId());
-(END)
-             //验证一遍专区id
-             List<ActivityZoneDTO> activityZoneList = activityZoneService.getActivityZoneDataByIdList(Collections.singletonList(item.getSubjectId()));
-@@ -186,7 +187,7 @@ public class UserSubscribeInfoServiceImpl extends ServiceImpl<UserSubscribeInfoM
-                 continue;
-             }
-
--            UserSubscribeInfo subscribeInfo = new UserSubscribeInfo();
-+
-             subscribeInfo.setAccountId(accountId);
-             subscribeInfo.setSubscribeType(SubscribeEnum.SUBJECT_SUBSCRIBE.getSubscribeType());
-             subscribeInfo.setSubjectId(item.getSubjectId());
-(END)
-             //验证一遍专区id
-             List<ActivityZoneDTO> activityZoneList = activityZoneService.getActivityZoneDataByIdList(Collections.singletonList(item.getSubjectId()));
-@@ -186,7 +187,7 @@ public class UserSubscribeInfoServiceImpl extends ServiceImpl<UserSubscribeInfoM
-                 continue;
-             }
-
--            UserSubscribeInfo subscribeInfo = new UserSubscribeInfo();
-+
-             subscribeInfo.setAccountId(accountId);
-             subscribeInfo.setSubscribeType(SubscribeEnum.SUBJECT_SUBSCRIBE.getSubscribeType());
-             subscribeInfo.setSubjectId(item.getSubjectId());