xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 11 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

fix: 预览标题换行同时匹配半角/和全角／ #1500

Closed
xiaoxia wants to merge 1 commits from fix/title-fullwidth-slash into develop
pull from: fix/title-fullwidth-slash
merge into: xiaoxia:develop
Conversation 3 Commits 1 Files Changed 1
+1 -1
xiaoxia commented 2026-08-25 16:41:57 +08:00
Owner
Copy Link
Copy Source

问题

PreviewVideoPanel.tsx 的标题换行只匹配半角 /，中文输入法下的全角 ／ 不会触发换行。

修复

displayTitle.split("/") → displayTitle.split(/[/／]/)，与 useCanvasPlayer.ts 保持一致。

验收

  • 输入 第一行/第二行（半角）→ 预览换行 ✅
  • 输入 第一行／第二行（全角）→ 预览换行 ✅
## 问题 PreviewVideoPanel.tsx 的标题换行只匹配半角 `/`，中文输入法下的全角 `／` 不会触发换行。 ## 修复 `displayTitle.split("/")` → `displayTitle.split(/[/／]/)`，与 useCanvasPlayer.ts 保持一致。 ## 验收 - 输入 `第一行/第二行`（半角）→ 预览换行 ✅ - 输入 `第一行／第二行`（全角）→ 预览换行 ✅
xiaoxia added 1 commit 2026-08-25 16:41:57 +08:00
fix: 预览标题换行同时匹配半角/和全角／
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 49s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
Details
AI Code Review / AI Code Review (pull_request) Failing after 1m25s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 41s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m41s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m9s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 2m13s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m22s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m28s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m22s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m44s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 30s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 54s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 8m28s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Successful in 12s
Details
a128dcd76d 
PreviewVideoPanel 的 displayTitle.split("/") 改为正则 split(/[/／]/)，
与 useCanvasPlayer.ts 保持一致，中文输入法下的全角斜杠也能触发换行。
auto-approve-bot commented 2026-08-25 16:43:21 +08:00
Collaborator
Copy Link
Copy Source

【阻塞级判定】

  • 是否存在阻塞级问题：是
  • 阻塞级问题数量：1 个

📊 审查概览

  • 整体评价：需修改
  • 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

  1. [apps/web/src/pages/generate/components/PreviewVideoPanel.tsx: 154] 正则表达式语法错误
    • 问题类型：语法错误
    • 问题描述：在 JavaScript/TypeScript 正则表达式字面量 /[/／]/ 中，紧跟在 [ 之后的 / 字符会被解析器误认为是正则表达式的结束定界符，导致代码抛出 SyntaxError: Invalid regular expression: missing / 错误，页面无法正常运行。
    • 修改建议：将字符集中的 / 进行转义，修改为 /[\/％]/（注：此处应为全角和半角斜杠，即 /[\/／]/）。

💡 改进建议（不阻塞合并）

无

✅ 良好实践

  • 使用正则表达式字符集 [] 同时兼容全角（／）和半角（/）斜杠的分割逻辑，提升了对不同输入法场景的兼容性，思路正确。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-25 08:43:21 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 1. **[apps/web/src/pages/generate/components/PreviewVideoPanel.tsx: 154] 正则表达式语法错误** - 问题类型：语法错误 - 问题描述：在 JavaScript/TypeScript 正则表达式字面量 `/[/／]/` 中，紧跟在 `[` 之后的 `/` 字符会被解析器误认为是正则表达式的结束定界符，导致代码抛出 `SyntaxError: Invalid regular expression: missing /` 错误，页面无法正常运行。 - 修改建议：将字符集中的 `/` 进行转义，修改为 `/[\/％]/`（注：此处应为全角和半角斜杠，即 `/[\/／]/`）。 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 - 使用正则表达式字符集 `[]` 同时兼容全角（`／`）和半角（`/`）斜杠的分割逻辑，提升了对不同输入法场景的兼容性，思路正确。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-25 08:43:21 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
gitea-actions bot commented 2026-08-25 16:44:01 +08:00
Copy Link
Copy Source

🚀 预览环境已部署

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

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

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

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

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1500 | | 预览链接 | [https://pr-1500.preview.xiaoxiajianji.com](https://pr-1500.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。
xiaoxia closed this pull request 2026-08-25 16:45:30 +08:00
gitea-actions bot commented 2026-08-25 16:49:42 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1500 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Some checks are pending
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 49s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
Details
AI Code Review / AI Code Review (pull_request) Failing after 1m25s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 41s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m41s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m9s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 2m13s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m22s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m28s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m22s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m44s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 30s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 54s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 8m28s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Successful in 12s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1500
Delete Branch "fix/title-fullwidth-slash"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?