Skip to content

feat(assets): refine character asset workspace - #612

Merged
huyanxius merged 6 commits into
1024XEngineer:mainfrom
huyanxius:feat/604-asset-detail-workbench
Aug 25, 2026
Merged

feat(assets): refine character asset workspace#612
huyanxius merged 6 commits into
1024XEngineer:mainfrom
huyanxius:feat/604-asset-detail-workbench

Conversation

@huyanxius

Copy link
Copy Markdown
Collaborator

统一角色资产库与详情页的视觉层级,并增加动作帧完美像素化工作台,使高帧数角色资产在桌面端可稳定浏览与处理。

Why

原有资产卡片与项目中心规格不一致,动作和帧数量增加后容易出现横向溢出或浏览效率下降;同时已有完美像素化接口缺少面向动作帧的前端入口。

Changes

  • 统一资产库卡片列数、间距和标题层级。
  • 重构角色详情页头部与动作卡片,并在原页面内展开完整帧序列。
  • 帧序列改为自适应列数和纵向滚动。
  • 增加单动作和全部动作的完美像素化工作台。
  • 保留导出、预览台和新增动作业务路径。

Implementation

  • 增加带认证恢复和二进制响应校验的完美像素化前端客户端。
  • 工作台按动作和帧维护处理状态、结果对象 URL 与失败信息。
  • 动作卡片使用连续换行牌组,展开内容保持在选中卡片内部。
  • 与最新主线的既有 Workflow Editor / Quick Start 路由保持一致。

Verification

  • node_modules/.bin/vitest run src/entities/pixel-perfect/api.test.ts src/pages/character-detail/index.test.tsx:13 tests passed
  • npm run typecheck:通过
  • node_modules/.bin/oxfmt --check ...:通过
  • git diff --check upstream/main...HEAD:通过
  • 真实浏览器:页面无横向溢出,动作卡片可在原页面展开

Scope

Related Issues

Closes #604
Refs #550
Refs #566

The asset workspace needs authenticated access to the binary pixel-perfect tool.

Add a typed client that downloads source media and validates PNG responses and metadata.

Frontend features can process existing asset images without duplicating transport logic.
Binary image transport needs regression coverage for authentication and response validation.

Cover source downloads, token recovery, business failures, metadata checks, and missing media.

The frontend client now has focused evidence for its API boundary.
Character assets need a focused surface for processing existing action frames.

Add single-action and all-action processing with comparison, progress, failure, and download states.

Users can inspect pixel-perfect results without replacing stored assets.
Asset cards and dense action sequences need a consistent responsive hierarchy.

Align library cards, restyle character actions, expand frames inline, and connect pixel processing.

The asset workspace now scales across desktop widths without changing existing workflows.
The refined asset detail must preserve existing routes while changing its visual structure.

Cover action controls, inline frame expansion, vertical scrolling, and empty states.

The page behavior remains reviewable independently from its presentation changes.
@vercel

vercel Bot commented Aug 24, 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 25, 2026 4:53am

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 40.74074% with 144 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...features/pixel-perfect/pixel-perfect-workbench.tsx 0.00% 115 Missing ⚠️
frontend/src/entities/pixel-perfect/api.ts 73.41% 9 Missing and 12 partials ⚠️
frontend/src/pages/character-detail/index.tsx 82.97% 2 Missing and 6 partials ⚠️

📢 Thoughts on this report? Let us know!

@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.

The asset-detail and workbench UI changes are internally consistent, but the new processing path is not deployable against the fixed PR base: the frontend calls /tools/pixel-perfect, while that route is not present in this PR or its base. The focused tests and static inspection cover the client contract, but runtime verification is blocked because frontend dependencies are not installed in the mounted checkout.

Comment thread frontend/src/entities/pixel-perfect/api.ts
xyh202131
xyh202131 previously approved these changes Aug 25, 2026

@xyh202131 xyh202131 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.

当前与 main 冲突,且前端入口依赖暂缓的 #496 接口;先不合并。

三处冲突的取舍:

- export-panel:合并两条按钮形态。iconOnly 走 main 的图标+tooltip
  分支,普通形态走本分支的 icon + label 排版,两个 prop 都保留。
- character-detail:布局以本分支为准,删掉 main 的 OutfitMaster
  外壳(本分支已改成标题旁悬浮预览 + PixelPerfectWorkbench),
  但把它的八向母版网格单独拆成 CharacterTemplateDirections 保留下来。
- 帧序列面板:main 的方向切换搬进本分支的卡内 ActionFramePanel,
  方向状态下沉到面板自身;main 那段卡外的帧序列 section 删除。
@huyanxius
huyanxius merged commit 935b7c0 into 1024XEngineer:main Aug 25, 2026
7 checks passed
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.

[Feature]: 重构角色资产详情与动作帧工作台

2 participants