Comfy-Org / Comfy-Org/ComfyUI_frontend

[RFC] Consolidate Load3D domain — 19 files across 5 directories

Open
#11,084 0 comments 0 reactions 1 assignee Claimed by @kaili-yang View on GitHub
area:3d developer experience refactor
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

## Problem

Load3D is a self-contained feature domain scattered across 5 directories:

- `src/composables/useLoad3d.ts`, `useLoad3dDrag.ts`, `useLoad3dViewer.ts`
- `src/components/load3d/` (multiple Vue components)
- `src/extensions/core/load3d/` (extension registration + utils)
- `src/extensions/core/load3d.ts`, `load3dLazy.ts`
- `src/services/load3dService.ts`

Working on Load3D requires navigating 5 directory trees.

## Proposed Fix

Consolidate into `platform/load3d/`:

```
platform/load3d/
composables/useLoad3d.ts
composables/useLoad3dDrag.ts
composables/useLoad3dViewer.ts
components/Load3dViewer.vue
services/load3dService.ts
extension.ts
```

## Migration Plan

- Single PR to relocate all files
- Update imports

## Testing Strategy

- All existing Load3D tests pass
- `pnpm typecheck` passes

┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-11084-RFC-Consolidate-Load3D-domain-19-files-across-5-directories-33e6d73d365081919df3e1144b20330b) by [Unito](https://www.unito.io)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.