Game creator extract home view #80

2026-07-14T13:33:47+08:00

k88936 commented

2026-07-14 13:33:47 +08:00

抽取homeview方便后续修改

k88936 added 2 commits 2026-07-14 13:33:48 +08:00

extract home view from App.tsx , rewrite style with tailwindcss 95c16a4168

reorganize prop and state d6f6bfc8b4

kdletters commented

2026-07-14 15:21:47 +08:00

审查结论
暂不建议合并，有 1 个阻断项和 4 个明确回归。

[P1] 必跑测试已撞红。 文件 input 删除了 .launcher-file-input，现有测试因此获取到 null。实跑 appSurface.test.ts：231 passed / 1 failed，错误为 Unable to fire a "change" event。index.tsx
[P2] 创建成功后没有清空首页草稿。 返回首页后，上一个项目的 prompt 和附件 File 仍在，可能被再次导入新项目。基线原本会成功后清空。App.tsx
[P2] 首页状态拆分后反馈丢失。 HomeView 展示自己的 homeStatus，父组件仍更新另一份 status。非空目录弹窗点取消后，首页仍显示旧提示；确认后创建失败还会产生未处理 Promise，错误不会回显。index.tsx
[P2] 移动断点从 760px 漂成了 720px。 在 721–760px 区间，外壳已进入移动布局，首页却仍显示 5 列项目和 6 列灵感。740px 浏览器实测已明显挤压。index.tsx

[P2] Tailwind Preflight 全局影响了首页之外的页面。 它会清除按钮默认边框、背景和 padding；项目组、Agent 聊天、项目开发等没有显式按钮样式的操作会退化成裸文本。styles.css

仓库规范符合度
目标分支和 merge-base 正确，无冲突。类型检查、AGC Vite 生产构建、编码检查和 git diff --check 通过。但 PR 没有更新任何测试或对应方案文档，两个提交标题也都是英文，不符合仓库提交规范。

::code-comment{title="[P1] 首页上传测试失效" body="文件 input 删除了现有测试依赖的 .launcher-file-input，且没有提供稳定的可访问名称，导致必跑 appSurface 套件 232 项中 1 项失败。请恢复契约，或补 aria-label 并同步测试。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=217 end=220 priority=1}
::code-comment{title="[P2] 成功后未清空草稿" body="成功进入项目开发后直接返回，没有重置全局首页 store。返回首页会恢复上一个项目的文字和 File 附件，可能误提交到下一项目。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/App.tsx" start=2424 end=2437 priority=2}
::code-comment{title="[P2] 首页状态源发生分叉" body="HomeView 改为展示局部 homeStatus，但非空目录取消、确认失败和最近项目打开失败仍只更新父层 status。请统一状态源，并捕获继续新建路径抛出的异常。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=83 end=86 priority=2}
::code-comment{title="[P2] 响应式断点不一致" body="首页在 720px 才切单列，外层壳仍在 760px 切换。740px 实测项目卡仍为五列并严重挤压，应继续与壳层使用同一断点。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=267 end=267 priority=2}
::code-comment{title="[P2] Preflight 影响非首页控件" body="全局引入 Tailwind Preflight 会清除未显式样式化按钮的默认边框、背景和 padding，影响项目组、Agent 聊天和项目开发页面。请限制影响范围或补齐这些控件样式。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/styles.css" start=1 end=1 priority=2}

**审查结论** 暂不建议合并，有 1 个阻断项和 4 个明确回归。 1. **[P1] 必跑测试已撞红。** 文件 input 删除了 `.launcher-file-input`，现有测试因此获取到 `null`。实跑 `appSurface.test.ts`：`231 passed / 1 failed`，错误为 `Unable to fire a "change" event`。[index.tsx](/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx:217) 2. **[P2] 创建成功后没有清空首页草稿。** 返回首页后，上一个项目的 prompt 和附件 `File` 仍在，可能被再次导入新项目。基线原本会成功后清空。[App.tsx](/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/App.tsx:2437) 3. **[P2] 首页状态拆分后反馈丢失。** `HomeView` 展示自己的 `homeStatus`，父组件仍更新另一份 `status`。非空目录弹窗点取消后，首页仍显示旧提示；确认后创建失败还会产生未处理 Promise，错误不会回显。[index.tsx](/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx:84) 4. **[P2] 移动断点从 `760px` 漂成了 `720px`。** 在 `721–760px` 区间，外壳已进入移动布局，首页却仍显示 5 列项目和 6 列灵感。`740px` 浏览器实测已明显挤压。[index.tsx](/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx:267) ![image.png](/attachments/31ac3091-7835-4dad-9f79-12011b2af4d2) 5. **[P2] Tailwind Preflight 全局影响了首页之外的页面。** 它会清除按钮默认边框、背景和 padding；项目组、Agent 聊天、项目开发等没有显式按钮样式的操作会退化成裸文本。[styles.css](/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/styles.css:1) **仓库规范符合度** 目标分支和 merge-base 正确，无冲突。类型检查、AGC Vite 生产构建、编码检查和 `git diff --check` 通过。但 PR 没有更新任何测试或对应方案文档，两个提交标题也都是英文，不符合仓库提交规范。 ::code-comment{title="[P1] 首页上传测试失效" body="文件 input 删除了现有测试依赖的 .launcher-file-input，且没有提供稳定的可访问名称，导致必跑 appSurface 套件 232 项中 1 项失败。请恢复契约，或补 aria-label 并同步测试。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=217 end=220 priority=1} ::code-comment{title="[P2] 成功后未清空草稿" body="成功进入项目开发后直接返回，没有重置全局首页 store。返回首页会恢复上一个项目的文字和 File 附件，可能误提交到下一项目。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/App.tsx" start=2424 end=2437 priority=2} ::code-comment{title="[P2] 首页状态源发生分叉" body="HomeView 改为展示局部 homeStatus，但非空目录取消、确认失败和最近项目打开失败仍只更新父层 status。请统一状态源，并捕获继续新建路径抛出的异常。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=83 end=86 priority=2} ::code-comment{title="[P2] 响应式断点不一致" body="首页在 720px 才切单列，外层壳仍在 760px 切换。740px 实测项目卡仍为五列并严重挤压，应继续与壳层使用同一断点。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/view/home/index.tsx" start=267 end=267 priority=2} ::code-comment{title="[P2] Preflight 影响非首页控件" body="全局引入 Tailwind Preflight 会清除未显式样式化按钮的默认边框、背景和 padding，影响项目组、Agent 聊天和项目开发页面。请限制影响范围或补齐这些控件样式。" file="/tmp/genarrative-pr80-review/apps/ai-game-creator-shell/src/styles.css" start=1 end=1 priority=2}

image.png

63 KiB

k88936 added 2 commits 2026-07-15 12:26:47 +08:00

removed home status, adjust style, reset draft on submitted chat, give error echo back dd8eb60e05

adjusted project card height to be dynamic in home view cc0b289df6

k88936 commented

2026-07-15 12:27:29 +08:00

fixed 1 2 3 4

5 后来打算复用样式表的时候再调整

fixed 1 2 3 4 5 后来打算复用样式表的时候再调整 ![shotmd-1784087227.jpg](/attachments/335b9fab-d478-442d-a809-ccd610ccbe57) ![shotmd-1784087241.jpg](/attachments/d3d14fb5-9a5b-4bc5-9c69-0448ef7ba62c)

shotmd-1784087241.jpg

48 KiB

shotmd-1784087227.jpg

39 KiB

🚀 1

k88936 added 1 commit 2026-07-15 13:35:32 +08:00

update doc 427203471e

k88936 requested review from kdletters 2026-07-16 10:29:40 +08:00

kdletters added 1 commit 2026-07-16 14:24:10 +08:00

修复游戏创作页头按钮样式 c524e4573e

恢复 Tailwind Preflight 后的页头操作按钮边框与间距
补充页头按钮样式与禁用态回归测试

kdletters merged commit ee0b4b5d8a into codex/ai-game-creator-app

2026-07-16 14:33:58 +08:00

kdletters deleted branch game-creator-extract-home-view

2026-07-16 14:33:58 +08:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: GenarrativeAI/Genarrative#80