fix(hub): 12 处 list 端点把越界 pageSize 塌回默认 50 而非 clamp 到上限（统一 config.ClampPageSize）

[DeliciousBuding]

结论先说

12 个 list 入口用同一个错误形态归一化越界 pageSize：

if pageSize <= 0 || pageSize > 200 { pageSize = defaultXPageSize }   // 50

上分支是错的：请求 201 行（api/openapi.yaml 的共享 PageSize 声明 maximum: 200，handler 层还先 clamp 到 config.MaxPageLimit=500）会静默得到 50 行 + HTTP 200。

关联：#2154（Popper 后端架构探索批 P1-1；严重度分级与"裸数组"范围经主机侧逐端点核实后修正，见下）

严重度按分页方式分两档（逐个核实，非照抄报告）

① cursor 分页 — 契约违背 + 4× 往返，不丢数据
ListWorkspaces / ListSkills / ListPublicSkills / ListAgentProfiles / ListPublicProfiles / ListExecutionTargets / ListProviderBindings / ListMCPServers / ListPublicMCPServers / ListAuditEvents
nextCursor 由实际返回行推导，所以客户端仍能翻完，只是每页只有请求量的 1/4；同时端点实际强制的上限（50）与声明的契约（200）不一致。

② offset 分页 / 无游标 — 真截断

入口	形状	后果	定性
ListNotifications	返回裸数组（app/shared/src/hub/hubClient.ts:166 = request<HubNotification[]>，无 hasMore/nextCursor），limit 塌回而 offset 照给	按「请求的 limit」推进 offset 的消费者会跳行丢数据	latent：全 app 树 grep 未发现 listNotifications 的 UI 调用点
service/workspace.ListThreadMessages（handler/workspace.go:206，handler 先 clamp 到 500）	固定 offset 0、无游标	线程面板请求最多 500 条却静默只渲染 50 条，用户无法翻到其余部分	活体可见截断

对探索报告的两处更正（已核实）：报告称「10 个在产端点静默截断、裸数组无 hasMore」。实测 cursor 端点的响应含 page.hasMore（handler/workspace.go:103-105），不会让客户端误判到底；裸数组只出现在上表两处。故本 PR 只声明上表的影响面。

"是抄错而非策略"的证据：同仓已有两处写对的形态（clamp 到上限）——repository/message.go:48（GetMessagesIncrement）与 repository/agent_team_assignments.go:160。

改法

• config.ClampPageSize(requested, max, def) 单点实现：<=0 → def、> max → **max**、其余原样。
• 裸 200 字面量（10 处重复）提为 config.MaxListPageSize，并在测试里钉住 == 200：改它就是改契约，必须显式。
• 13 处站点改为调用它（12 处 repository + service/workspace.ListThreadMessages）。
• max/def 做成参数而非硬编码常量：仓里合法存在两个上限（通用 list = 200，message 族 = MaxMessagePageLimit 100，document 族 = MaxPageLimit 500），由调用方传自己端点声明的那个。

测试（行为级：种真实行数、数返回行数，不读代码）

测试	断言
config/paging_test.go	三分支表驱动（0/-7/1/137/200/201/500/2^20）+ max 由调用方传入 + MaxListPageSize == 200 契约钉
repository/pagination_clamp_test.go	12 个入口 × 3 个方向 = 36 例

• 红证据（修复前）：12/12 子测试 expected: 200 / actual: 50（notifications 与 messages 为 100 → 50）。
• 绿：36/36。
• 另两个方向防止"修成新的忽略客户端意图"：in-range 30 → 正好 30 行；非正数 0/-5 → 仍是默认 50 行（旧行为里正确的那一半必须保留）。
• 共享夹具缺 6 张表（workspaces/skills/execution_targets/provider_bindings/mcp_servers/audit_events），用 AutoMigrate 在测试内局部补齐，不改共享夹具。

门禁（全部跑在本 HEAD）

门禁	结果
hub go build ./... / go vet ./...	干净
go test ./internal/repository/... ./internal/config/... ./internal/service/... ./internal/handler/... -short -count=1	30 包 ok（无既有测试依赖旧塌回行为）
golangci-lint run ./...（清缓存后）	0 issues
gosec -fmt=json ./... | verify-gosec-gates.sh	CLEAN
verify-openapi-contract	rc=0（violations 0）
verify-doc-ssot / verify-conventions / verify-hub-layering / verify-hub-pure-packages / verify-quality-debt-ratchet	全部 rc=0
git diff --check	干净

证据等级：L0（sqlite 行为测试 + 静态门禁）。clamp 是纯 Go 整数分支、与 DB 引擎无关，故未加 L1；未做真实客户端翻页 E2E。

不做的事（附完整清点，已登记 #2154）

不删 handler 侧的 13 处 clamp。 对本 PR 修的端点它们确实已成死代码（handler clamp 到 500，repository clamp 到 200/100，前者永不决定性），但：

• market.go:49 / session.go:346 / agent.go:374 / agent_team_runs.go:128 四处的下游走 service 层，要证明"repository 侧确有等价约束"需逐个穿透；
• handler/message.go:105 是活的——它把 limit 压到 100，恰好掩盖了 repository 的塌回（这也是消息列表没暴露本 bug 的原因）；
• 在未逐点证明前删掉某个端点的唯一约束点，风险大于留着死代码。

故拆为后续切片：把边界收敛为「repository 单一属主」，逐端点证明后再删 handler clamp（清点已在 #2154）。同时不改 openapi（PageSize maximum: 200 现在才真的成立；documents/custom-agents 用 MaxPageLimit=500 与共享 PageSize 的三方不一致属报告 P1-2，另片处理）。

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: ae4fdb95-3213-4365-a15b-c03ce97c8a16

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}