抽取充值模态框 #87

2026-07-16T16:32:39+08:00

k88936 commented

2026-07-16 16:32:39 +08:00

No description provided.

k88936 requested review from kdletters 2026-07-16 16:33:43 +08:00

kdletters commented

2026-07-16 17:30:01 +08:00

Review 结论

当前 PR head 为 f975dfe8，相对 master 仅领先 1 个提交。现在不建议合并，有 3 个 blocker。

功能/契约风险

[P1] shared 根入口无法在 Node 环境加载。
packages/shared/src/index.ts 同时导出了带 .png 的余额组件和带 .css 的充值组件。执行 shared 根入口导入直接报：
ERR_UNKNOWN_FILE_EXTENSION: Unknown file extension ".png"。
应保留纯契约根入口，把 UI 放到独立 browser/component 子入口。
[P1] “未来 Tauri 复用”目前无法成立。
组件布局仍依赖 Tailwind utilities，例如 index.tsx，样式又依赖未兜底的主题变量，例如 index.css。目标 AI Game Creator Tauri 前端的 vite.config.ts 没有 Tailwind 插件，styles.css 也没有这些主题变量。直接复用后遮罩定位、布局、尺寸和颜色都会缺失。

仓库规范符合度

[P1] 改动文件 ESLint 未通过。
simple-import-sort/imports 在以下三个文件报错：
ImageCanvasEditorView.tsx、usePlatformProfileCenterController.ts、RpgEntryHomeView.tsx。

另外有两个非 blocker 偏差：新组件重新实现了 modal shell、action button、async state 等已有公共组件，与弹窗收口文档相悖。

现有主站调用链本身没有发现行为回归：117 项充值/资金测试通过，typecheck、build:raw、编码检查和 git diff --check 均通过。

**Review 结论** 当前 PR head 为 `f975dfe8`，相对 `master` 仅领先 1 个提交。现在**不建议合并**，有 3 个 blocker。 **功能/契约风险** 1. **[P1] shared 根入口无法在 Node 环境加载。** [packages/shared/src/index.ts](/tmp/genarrative-pr87-review/packages/shared/src/index.ts:1) 同时导出了带 `.png` 的余额组件和带 `.css` 的充值组件。执行 shared 根入口导入直接报： `ERR_UNKNOWN_FILE_EXTENSION: Unknown file extension ".png"`。应保留纯契约根入口，把 UI 放到独立 browser/component 子入口。 2. **[P1] “未来 Tauri 复用”目前无法成立。** 组件布局仍依赖 Tailwind utilities，例如 [index.tsx](/tmp/genarrative-pr87-review/packages/shared/src/components/PlatformProfileRechargeModal/index.tsx:74)，样式又依赖未兜底的主题变量，例如 [index.css](/tmp/genarrative-pr87-review/packages/shared/src/components/PlatformProfileRechargeModal/index.css:1)。目标 AI Game Creator Tauri 前端的 [vite.config.ts](/data/dsk/ca/apps/ai-game-creator-shell/vite.config.ts:41) 没有 Tailwind 插件，[styles.css](/data/dsk/ca/apps/ai-game-creator-shell/src/styles.css:1) 也没有这些主题变量。直接复用后遮罩定位、布局、尺寸和颜色都会缺失。 **仓库规范符合度** 3. **[P1] 改动文件 ESLint 未通过。** `simple-import-sort/imports` 在以下三个文件报错： [ImageCanvasEditorView.tsx](/tmp/genarrative-pr87-review/src/components/image-editor/ImageCanvasEditorView.tsx:27)、[usePlatformProfileCenterController.ts](/tmp/genarrative-pr87-review/src/components/platform-entry/usePlatformProfileCenterController.ts:13)、[RpgEntryHomeView.tsx](/tmp/genarrative-pr87-review/src/components/rpg-entry/RpgEntryHomeView.tsx:135)。另外有两个非 blocker 偏差：新组件重新实现了 modal shell、action button、async state 等已有公共组件，与弹窗收口文档相悖。现有主站调用链本身没有发现行为回归：117 项充值/资金测试通过，`typecheck`、`build:raw`、编码检查和 `git diff --check` 均通过。

k88936 changed target branch from master to editor-agent-refactored

2026-07-16 19:42:46 +08:00

k88936 added 1 commit 2026-07-16 19:42:46 +08:00

extract recharge model f975dfe817

k88936 commented

2026-07-16 19:46:17 +08:00

will be integrate into #86

k88936 closed this pull request

2026-07-16 19:46:17 +08:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: GenarrativeAI/Genarrative#87