Skip to content

fix(project): 项目名称改为可重复展示名 - #912

Open
xyh202131 wants to merge 1 commit into
1024XEngineer:mainfrom
xyh202131:fix/project-display-name-duplicates
Open

fix(project): 项目名称改为可重复展示名#912
xyh202131 wants to merge 1 commit into
1024XEngineer:mainfrom
xyh202131:fix/project-display-name-duplicates

Conversation

@xyh202131

Copy link
Copy Markdown
Contributor

Summary

  • 项目名称只作为展示字段,稳定身份继续使用项目 ID
  • 创建和重命名项目时允许同名,不再自动追加序号
  • 增加迁移以移除用户与项目名称的唯一约束

Verification

  • 后端项目与 Schema 测试(35 passed)
  • 前端项目创建测试(13 passed)
  • Ruff、TypeScript、lint、生产构建通过

Refs #783

Treat project names as display labels while keeping project IDs as the stable identity.

Co-authored-by: Codex <codex@openai.com>
@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
windup Ready Ready Preview Aug 28, 2026 7:47pm

@github-actions

Copy link
Copy Markdown

⚠️ @xyh202131,此 PR 尚未关联 issue。请在 PR 描述中使用 Closes #123 等关闭关键字;更新描述后,此提醒将自动标记为已解决。

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #912      +/-   ##
==========================================
- Coverage   86.50%   86.21%   -0.29%     
==========================================
  Files         304      305       +1     
  Lines       23830    23787      -43     
  Branches     3597     3601       +4     
==========================================
- Hits        20615    20509     -106     
- Misses       2249     2318      +69     
+ Partials      966      960       -6     
Flag Coverage Δ
backend 91.26% <100.00%> (+<0.01%) ⬆️
frontend 80.01% <ø> (-0.63%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ges/app/src/windup_app/server/project/interface.py 95.00% <ø> (-0.46%) ⬇️
...ackages/app/src/windup_app/server/project/model.py 100.00% <100.00%> (ø)
...ckages/app/src/windup_app/server/project/naming.py 100.00% <ø> (ø)
...kages/app/src/windup_app/server/project/service.py 97.18% <ø> (-0.12%) ⬇️
...end/packages/app/src/windup_app/web/api/project.py 94.36% <100.00%> (+1.62%) ⬆️
frontend/src/pages/project-create/index.tsx 96.15% <ø> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审阅了项目展示名改为可重复后的完整变更,覆盖 ORM 约束、创建/重命名 API、迁移脚本、前端文案及相关回归测试。项目 ID 仍作为详情、列表和关联路径的稳定标识,创建与重命名的同名行为与新的数据库约束保持一致;未发现达到报告阈值的正确性、性能或安全问题。

验证:PR 的 lint-and-testFrontend checksgate 和 Vercel 检查均通过;本地静态检查(git diff --check、Python 编译)通过。当前环境未安装 uvpytest,因此未重复运行 Python 测试。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant