fix(#1677): 移除预览网格截断+空指针防御+排序显式Number（AI Review 第二轮） #1713

Merged

auto-approve-bot merged 1 commits from fix/1677-grid-truncation into develop 2026-09-05 13:27:47 +08:00

Conversation 5 Commits 1 Files Changed 3

+7 -6

xiaoxia commented 2026-09-05 13:22:48 +08:00

Owner

Copy Link

Copy Source

背景

PR #1712 合入了 AI Review 第一轮修复，但第二轮修复（AI Review 指出的渲染截断逻辑错误）推送时 PR 已自动合并，本 PR 补上。

修复内容（AI Code Review 第二轮阻塞+建议）

1. 阻塞：移除 CanvasPreviewGrid 的 Math.min(count, MAX_PREVIEW_COUNT) 截断——数量上限已在源头 PreviewCountModal（1~10）clamp，网格完整渲染所有变体，保证每个变体都有勾选/预览入口，UI 与数据不脱节
2. 建议1：variant0Title 加 previewTitles?.[0] 可选链防御
3. 建议2：BatchGenerationGrid 排序显式 Number() 转换，防字符串类型导致 NaN

本地 tsc/eslint 全绿。

## 背景 PR #1712 合入了 AI Review 第一轮修复，但第二轮修复（AI Review 指出的渲染截断逻辑错误）推送时 PR 已自动合并，本 PR 补上。 ## 修复内容（AI Code Review 第二轮阻塞+建议） 1. **阻塞**：移除 `CanvasPreviewGrid` 的 `Math.min(count, MAX_PREVIEW_COUNT)` 截断——数量上限已在源头 `PreviewCountModal`（1~10）clamp，网格完整渲染所有变体，保证每个变体都有勾选/预览入口，UI 与数据不脱节 2. **建议1**：`variant0Title` 加 `previewTitles?.[0]` 可选链防御 3. **建议2**：`BatchGenerationGrid` 排序显式 `Number()` 转换，防字符串类型导致 NaN 本地 tsc/eslint 全绿。

xiaoxia added 1 commit 2026-09-05 13:22:49 +08:00

fix(#1677): AI Review 第二轮——移除预览网格截断+空指针防御+排序显式Number ... 616f404ed4

- CanvasPreviewGrid 移除 Math.min(count, MAX) 截断：数量上限已在源头
  PreviewCountModal clamp（1~10），网格完整渲染保证每个变体都有勾选/预览入口，
  UI 与数据不脱节
- variant0Title 加可选链 previewTitles?.[0] 防御
- BatchGenerationGrid 排序显式 Number() 转换防字符串 NaN

gitea-actions bot commented 2026-09-05 13:24:08 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1713
预览链接	https://pr-1713.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1713 | | 预览链接 | [https://pr-1713.preview.xiaoxiajianji.com](https://pr-1713.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot approved these changes 2026-09-05 13:24:38 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-09-05 13:24:38 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot commented 2026-09-05 13:24:43 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1713

⚠️ 问题（0个需要修改）

无

💡 建议（1个可选）

1. CanvasPreviewGrid.tsx 第44行：移除了 MAX_PREVIEW_COUNT 的硬上限保护，虽然注释说明 count 已在源头（PreviewCountModal）被限制，但这降低了组件的健壮性。如果未来组件被复用或上游逻辑变更导致传入过大的 count（如 100+），可能会导致浏览器因渲染过多 DOM 节点或媒体元素而卡顿崩溃。建议保留 Math.min(count, MAX_PREVIEW_COUNT) 作为防御性编程的最后防线，或者使用 TypeScript 将 count 类型限定为最大值。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-05 05:24:43 | 模型:}

## 代码审查结果 - PR #1713 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（1个可选） 1. **CanvasPreviewGrid.tsx 第44行**：移除了 `MAX_PREVIEW_COUNT` 的硬上限保护，虽然注释说明 `count` 已在源头（`PreviewCountModal`）被限制，但这降低了组件的健壮性。如果未来组件被复用或上游逻辑变更导致传入过大的 `count`（如 100+），可能会导致浏览器因渲染过多 DOM 节点或媒体元素而卡顿崩溃。建议保留 `Math.min(count, MAX_PREVIEW_COUNT)` 作为防御性编程的最后防线，或者使用 TypeScript 将 `count` 类型限定为最大值。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-05 05:24:43 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit 04b4fab131 into develop 2026-09-05 13:27:47 +08:00

auto-approve-bot deleted branch fix/1677-grid-truncation 2026-09-05 13:27:47 +08:00

gitea-actions bot commented 2026-09-05 13:28:03 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1713 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1713 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1713