fix(events): 响应 PR #9 Copilot CR — N+1 / seed 幂等 / admin seed / 测试 schema

修复 5 个 Copilot 提出的问题：

1) EventInterestRepository 新增 countByEventIds 批量聚合，EventService 暴露
   countInterestByEventIds；EventController 和 EventAdminController 列表接口
   都改用一次 GROUP BY 拿全量 interestCount，避免每条活动单独 COUNT 的 N+1。

2) EventRequest.joinTags 先过滤 null 元素再 trim，避免 JSON 客户端在 tags
   数组里放 null 时 NPE 导致 500。

3) test-schema.sql 的 event_interests 补齐对 events(id) / user_accounts(id) 的
   外键与 ON DELETE CASCADE，和生产 schema 行为对齐；否则 H2 测试无法覆盖
   "删 event 级联清 interest" 这条关键路径。

4) schema.sql 的 events seed 原先用 ON CONFLICT DO NOTHING 但 INSERT 不写 id，
   正常根本不会产生 conflict，等于非幂等。改成 SELECT ... WHERE NOT EXISTS(title)
   的 upsert 风格，重跑 schema 不会再插重复活动。

5) schema.sql 原先的 admin seed 按 GitHub handle（longsizhuo / Mira190 /
   Crokily）插 user_accounts，但 AuthService.loginByGithub 实际创建的 username
   是 "github_{githubId}" 格式，seed 出来的是三个永远没人用的死账号。删掉错误
   seed，改用注释指引：首次 OAuth 登录后按 github_id 执行一次 UPDATE 升
   roles，避免插错数据。

Author: claude

src/main/java/com/involutionhell/backend/events/controller/EventAdminController.java

@@ -17,7 +17,9 @@
 import org.springframework.web.bind.annotation.RestController;
 
 import java.util.List;
+import java.util.Map;
 import java.util.Optional;
+import java.util.stream.Collectors;
 
 /**
  * 活动管理接口（需要 admin 角色）。
@@ -46,8 +48,12 @@ public EventAdminController(EventService eventService) {
     @GetMapping
     public ApiResponse<List<EventView>> list() {
         List<Event> events = eventService.listAllForAdmin();
+        // 批量查 interest count 避免 N+1；admin 列表可能包含大量历史活动，
+        // 单独 COUNT 每条会明显拖慢后台
+        List<Long> ids = events.stream().map(Event::id).collect(Collectors.toList());
+        Map<Long, Long> interestCounts = eventService.countInterestByEventIds(ids);
         List<EventView> views = events.stream()
-                .map(e -> EventView.from(e, eventService.countInterest(e.id())))
+                .map(e -> EventView.from(e, interestCounts.getOrDefault(e.id(), 0L)))
                 .toList();
         return ApiResponse.ok(views);
     }

src/main/java/com/involutionhell/backend/events/controller/EventController.java

@@ -14,6 +14,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
+import java.util.stream.Collectors;
 
 /**
  * 活动公开读接口（匿名可访问）。
@@ -39,8 +40,11 @@ public EventController(EventService eventService) {
     @GetMapping
     public ApiResponse<List<EventView>> list() {
         List<Event> events = eventService.listPublic();
+        // 批量一次查完 interest count，避免每个 event 都单独 COUNT（N+1）
+        List<Long> ids = events.stream().map(Event::id).collect(Collectors.toList());
+        Map<Long, Long> interestCounts = eventService.countInterestByEventIds(ids);
         List<EventView> views = events.stream()
-                .map(e -> EventView.from(e, eventService.countInterest(e.id())))
+                .map(e -> EventView.from(e, interestCounts.getOrDefault(e.id(), 0L)))
                 .toList();
         return ApiResponse.ok(views);
     }

src/main/java/com/involutionhell/backend/events/dto/EventRequest.java

@@ -52,6 +52,13 @@ private static String emptyToNull(String s) {
 
     private static String joinTags(List<String> tags) {
         if (tags == null || tags.isEmpty()) return "";
-        return String.join(",", tags.stream().map(String::trim).filter(s -> !s.isEmpty()).toList());
+        // 先过滤 null 再 trim：JSON 客户端允许数组里放 null，裸调 String::trim 会 NPE → 500
+        return String.join(
+                ",",
+                tags.stream()
+                        .filter(tag -> tag != null)
+                        .map(String::trim)
+                        .filter(s -> !s.isEmpty())
+                        .toList());
     }
 }

src/main/java/com/involutionhell/backend/events/repository/EventInterestRepository.java

@@ -2,9 +2,14 @@
 
 import org.springframework.dao.DuplicateKeyException;
 import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.jdbc.core.namedparam.MapSqlParameterSource;
+import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate;
 import org.springframework.stereotype.Repository;
 
+import java.util.Collection;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 
 /**
  * event_interests 表的数据访问。语义和 FollowService 类似——
@@ -17,9 +22,12 @@
 public class EventInterestRepository {
 
     private final JdbcTemplate jdbc;
+    private final NamedParameterJdbcTemplate namedJdbc;
 
     public EventInterestRepository(JdbcTemplate jdbc) {
         this.jdbc = jdbc;
+        // 批量 count 用 named parameter 的 IN 子句，比自己拼 "?,?,?" 更安全
+        this.namedJdbc = new NamedParameterJdbcTemplate(jdbc);
     }
 
     /** 添加感兴趣记录。幂等：同一 (event, user) 已存在时不报错。 */
@@ -49,6 +57,26 @@ public long countByEvent(long eventId) {
         return cnt != null ? cnt : 0L;
     }
 
+    /**
+     * 批量统计多场活动的兴趣人数，避免列表接口 N+1 查询。
+     *
+     * 一次 GROUP BY 查完返回 map；没出现在结果里的 event id（即兴趣人数为 0）调用方
+     * 自己 getOrDefault(id, 0L) 兜底。传入空集合直接返回空 map，不打 DB。
+     */
+    public Map<Long, Long> countByEventIds(Collection<Long> eventIds) {
+        if (eventIds == null || eventIds.isEmpty()) return Map.of();
+        Map<Long, Long> result = new HashMap<>();
+        MapSqlParameterSource params = new MapSqlParameterSource("ids", eventIds);
+        namedJdbc.query(
+                "SELECT event_id, COUNT(*) AS cnt FROM event_interests "
+                        + "WHERE event_id IN (:ids) GROUP BY event_id",
+                params,
+                rs -> {
+                    result.put(rs.getLong("event_id"), rs.getLong("cnt"));
+                });
+        return result;
+    }
+
     /** 当前登录用户是否对某活动感兴趣。匿名调用方需自己短路 false，不要调这个。 */
     public boolean isInterested(long eventId, long userId) {
         Integer cnt = jdbc.queryForObject(

src/main/java/com/involutionhell/backend/events/service/EventService.java

@@ -5,7 +5,9 @@
 import com.involutionhell.backend.events.repository.EventRepository;
 import org.springframework.stereotype.Service;
 
+import java.util.Collection;
 import java.util.List;
+import java.util.Map;
 import java.util.Optional;
 
 /**
@@ -63,6 +65,14 @@ public long countInterest(long eventId) {
         return interestRepository.countByEvent(eventId);
     }
 
+    /**
+     * 批量拿多场活动的兴趣人数。列表接口用，避免 N+1。
+     * 返回 map 中不存在的 key 表示该活动兴趣人数为 0，调用方自己 getOrDefault 兜底。
+     */
+    public Map<Long, Long> countInterestByEventIds(Collection<Long> eventIds) {
+        return interestRepository.countByEventIds(eventIds);
+    }
+
     /** 当前登录用户是否对某活动感兴趣。匿名调用方需短路传 false，不要调这个。 */
     public boolean isInterested(long eventId, long userId) {
         return interestRepository.isInterested(eventId, userId);

src/main/resources/schema.sql

@@ -27,16 +27,16 @@ VALUES ('admin',   'ad89b64d66caa8e30e5d5ce4a9763f4ecc205814c412175f3e2c50027471
        ('auditor', 'ccabaaba054fb98905b5b9ee47174f57cb6088e04b1526f08b872dc06eaa6bb9', 'Auditor', TRUE, 'auditor', 'user:profile:read,user:center:read')
 ON CONFLICT (username) DO NOTHING;
 
--- 站点维护者（GitHub OAuth 登录后由 sync 服务补 github_id；此处按 username 打 admin role）
--- 生产 Neon 上这些账号是 GitHub OAuth 登录后由 AuthService 自动创建的，所以用
--- ON CONFLICT DO UPDATE 幂等升级角色，避免漏 seed。
-INSERT INTO user_accounts (username, password_hash, display_name, enabled, roles, permissions)
-VALUES ('longsizhuo', '', 'Siz Long', TRUE, 'admin,user', 'user:profile:read,user:center:read,user:center:manage'),
-       ('Mira190',    '', 'Mira',     TRUE, 'admin,user', 'user:profile:read,user:center:read,user:center:manage'),
-       ('Crokily',    '', 'Crokily',  TRUE, 'admin,user', 'user:profile:read,user:center:read,user:center:manage')
-ON CONFLICT (username) DO UPDATE
-    SET roles       = 'admin,user',
-        permissions = 'user:profile:read,user:center:read,user:center:manage';
+-- 站点维护者 admin 升级：原先尝试按 GitHub handle（longsizhuo / Mira190 / Crokily）
+-- 做 seed，但 AuthService.loginByGithub 实际创建的 username 是 "github_{githubId}" 格式
+-- （见 AuthService.java），按 handle seed 只会插一批永远没人用的本地管理员账号。
+-- 正确做法：维护者首次 GitHub OAuth 登录后，手动（或由 admin 管理界面）按 github_id
+-- 升 roles：
+--   UPDATE user_accounts
+--   SET roles = 'admin,user',
+--       permissions = 'user:profile:read,user:center:read,user:center:manage'
+--   WHERE github_id IN (114939201, ...);
+-- 本文件不再插入 admin seed 账号。
 
 -- =============================================================================
 -- Events（活动）相关表
@@ -76,24 +76,33 @@ CREATE TABLE IF NOT EXISTS event_interests (
 
 CREATE INDEX IF NOT EXISTS idx_event_interests_user_id ON event_interests(user_id);
 
--- 种子：原来 data/event.json 里的 4 条活动（startTime 不填，先只保留元信息；
--- 管理员登录后在 /admin/events 里补时间再 publish）
+-- 种子：原来 data/event.json 里的 4 条活动。幂等策略：
+--   events 表主键是 BIGSERIAL id，INSERT 不带 id 所以不会冲突；原先用 ON CONFLICT
+--   DO NOTHING 实际上不防重复。这里改用 WHERE NOT EXISTS(title) 做幂等——
+--   每次 schema 重跑时，title 已存在的就跳过。
+-- startTime / endTime 先不填，管理员登录后在 /admin/events 里补时间再 publish。
 INSERT INTO events (title, description, cover_url, discord_link, playback_url, tags, status)
-VALUES
-    ('Mock Interview',   '模拟面试专场：匹配面试官 1v1，结束即反馈，积累真实面试体感。', '/event/mockInterview.webp',
-     'https://discord.gg/QHsjqezfC?event=1430500169299922965',
-     'https://involutionhell.com/docs/jobs/event-keynote/event-takeway',
-     'interview,mock', 'archived'),
-    ('Coffee Chat',      '邀请业界嘉宾小范围交流，聊 career path、求职反思、日常 dev 体感。', '/event/coffeeChat.webp',
-     'https://discord.com/invite/8AQZj7sa?event=1432010537402761348',
-     'https://involutionhell.com/docs/jobs/event-keynote/coffee-chat',
-     'career,chat', 'archived'),
-    ('Career Journey',   '资深从业者分享完整职业路径 + 关键决策点。', '/event/careerJourney.webp',
-     'https://discord.com/invite/8AQZj7sa?event=1432010537402761348',
-     'https://involutionhell.com/docs/jobs/event-keynote/event-takeway',
-     'career,sharing', 'archived'),
-    ('Open.Onion',       '持续进行中的开源 / 内部项目协作节奏，参与即获得 contributor 标签。', '/event/openOnion.webp',
-     'https://discord.gg/kJZFMr5chU?event=1477581193582088304',
-     NULL,
-     'project,open-source', 'published')
-ON CONFLICT DO NOTHING;
+SELECT seed.title, seed.description, seed.cover_url, seed.discord_link,
+       seed.playback_url, seed.tags, seed.status
+FROM (
+    VALUES
+        ('Mock Interview',   '模拟面试专场：匹配面试官 1v1，结束即反馈，积累真实面试体感。', '/event/mockInterview.webp',
+         'https://discord.gg/QHsjqezfC?event=1430500169299922965',
+         'https://involutionhell.com/docs/jobs/event-keynote/event-takeway',
+         'interview,mock', 'archived'),
+        ('Coffee Chat',      '邀请业界嘉宾小范围交流，聊 career path、求职反思、日常 dev 体感。', '/event/coffeeChat.webp',
+         'https://discord.com/invite/8AQZj7sa?event=1432010537402761348',
+         'https://involutionhell.com/docs/jobs/event-keynote/coffee-chat',
+         'career,chat', 'archived'),
+        ('Career Journey',   '资深从业者分享完整职业路径 + 关键决策点。', '/event/careerJourney.webp',
+         'https://discord.com/invite/8AQZj7sa?event=1432010537402761348',
+         'https://involutionhell.com/docs/jobs/event-keynote/event-takeway',
+         'career,sharing', 'archived'),
+        ('Open.Onion',       '持续进行中的开源 / 内部项目协作节奏，参与即获得 contributor 标签。', '/event/openOnion.webp',
+         'https://discord.gg/kJZFMr5chU?event=1477581193582088304',
+         NULL,
+         'project,open-source', 'published')
+) AS seed(title, description, cover_url, discord_link, playback_url, tags, status)
+WHERE NOT EXISTS (
+    SELECT 1 FROM events e WHERE e.title = seed.title
+);

src/test/resources/test-schema.sql

@@ -46,5 +46,9 @@ CREATE TABLE IF NOT EXISTS event_interests (
     event_id   BIGINT      NOT NULL,
     user_id    BIGINT      NOT NULL,
     created_at TIMESTAMP   NOT NULL DEFAULT CURRENT_TIMESTAMP,
-    PRIMARY KEY (event_id, user_id)
+    PRIMARY KEY (event_id, user_id),
+    -- 补齐外键与 ON DELETE CASCADE 和生产 schema 对齐，否则 H2 测试既不能覆盖
+    -- "删 event 级联清 interest" 这个关键路径，也可能插入不存在的 event/user 产生脏数据
+    FOREIGN KEY (event_id) REFERENCES events(id)        ON DELETE CASCADE,
+    FOREIGN KEY (user_id)  REFERENCES user_accounts(id) ON DELETE CASCADE
 );