Skip to content

Commit e3bb29c

Browse files
[refactor] Move thumbnail functionality to renderer/core domain (#5586)
Move thumbnail functionality from src/renderer/thumbnail/ to src/renderer/core/thumbnail/ to align with domain-driven design architecture. Thumbnail generation is core rendering infrastructure and belongs alongside other core renderer utilities. Changes: - Move useWorkflowThumbnail.ts and graphThumbnailRenderer.ts to renderer/core/thumbnail/ - Update all import paths in consuming files - Fix relative imports within moved files 🤖 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: Claude <[email protected]>
1 parent ca312fd commit e3bb29c

File tree

6 files changed

+9
-8
lines changed

6 files changed

+9
-8
lines changed

src/components/topbar/WorkflowTab.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ import {
4343
import { useWorkflowService } from '@/platform/workflow/core/services/workflowService'
4444
import { ComfyWorkflow } from '@/platform/workflow/management/stores/workflowStore'
4545
import { useWorkflowStore } from '@/platform/workflow/management/stores/workflowStore'
46-
import { useWorkflowThumbnail } from '@/renderer/thumbnail/useWorkflowThumbnail'
46+
import { useWorkflowThumbnail } from '@/renderer/core/thumbnail/useWorkflowThumbnail'
4747
import { useSettingStore } from '@/stores/settingStore'
4848
import { useWorkspaceStore } from '@/stores/workspaceStore'
4949

src/platform/workflow/core/services/workflowService.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {
88
useWorkflowStore
99
} from '@/platform/workflow/management/stores/workflowStore'
1010
import { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/workflowSchema'
11-
import { useWorkflowThumbnail } from '@/renderer/thumbnail/useWorkflowThumbnail'
11+
import { useWorkflowThumbnail } from '@/renderer/core/thumbnail/useWorkflowThumbnail'
1212
import { app } from '@/scripts/app'
1313
import { blankGraph, defaultGraph } from '@/scripts/defaultGraph'
1414
import { downloadBlob } from '@/scripts/utils'

src/platform/workflow/management/stores/workflowStore.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { t } from '@/i18n'
66
import type { LGraph, Subgraph } from '@/lib/litegraph/src/litegraph'
77
import { ComfyWorkflowJSON } from '@/platform/workflow/validation/schemas/workflowSchema'
88
import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema'
9-
import { useWorkflowThumbnail } from '@/renderer/thumbnail/useWorkflowThumbnail'
9+
import { useWorkflowThumbnail } from '@/renderer/core/thumbnail/useWorkflowThumbnail'
1010
import { api } from '@/scripts/api'
1111
import { app as comfyApp } from '@/scripts/app'
1212
import { ChangeTracker } from '@/scripts/changeTracker'

src/renderer/thumbnail/graphThumbnailRenderer.ts renamed to src/renderer/core/thumbnail/graphThumbnailRenderer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
calculateNodeBounds
77
} from '@/renderer/core/spatial/boundsCalculator'
88

9-
import { renderMinimapToCanvas } from '../extensions/minimap/minimapCanvasRenderer'
9+
import { renderMinimapToCanvas } from '../../extensions/minimap/minimapCanvasRenderer'
1010

1111
/**
1212
* Create a thumbnail of the current canvas's active graph.

src/renderer/thumbnail/useWorkflowThumbnail.ts renamed to src/renderer/core/thumbnail/useWorkflowThumbnail.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
import { ref } from 'vue'
22

33
import { ComfyWorkflow } from '@/platform/workflow/management/stores/workflowStore'
4-
import { createGraphThumbnail } from '@/renderer/thumbnail/graphThumbnailRenderer'
4+
5+
import { createGraphThumbnail } from './graphThumbnailRenderer'
56

67
// Store thumbnails for each workflow
78
const workflowThumbnails = ref<Map<string, string>>(new Map())

tests-ui/tests/renderer/thumbnail/composables/useWorkflowThumbnail.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
useWorkflowStore
77
} from '@/platform/workflow/management/stores/workflowStore'
88

9-
vi.mock('@/renderer/thumbnail/graphThumbnailRenderer', () => ({
9+
vi.mock('@/renderer/core/thumbnail/graphThumbnailRenderer', () => ({
1010
createGraphThumbnail: vi.fn()
1111
}))
1212

@@ -22,10 +22,10 @@ vi.mock('@/scripts/api', () => ({
2222
}))
2323

2424
const { useWorkflowThumbnail } = await import(
25-
'@/renderer/thumbnail/useWorkflowThumbnail'
25+
'@/renderer/core/thumbnail/useWorkflowThumbnail'
2626
)
2727
const { createGraphThumbnail } = await import(
28-
'@/renderer/thumbnail/graphThumbnailRenderer'
28+
'@/renderer/core/thumbnail/graphThumbnailRenderer'
2929
)
3030
const { api } = await import('@/scripts/api')
3131

0 commit comments

Comments
 (0)