Skip to content

Conversation

@viva-jinyi
Copy link
Member

@viva-jinyi viva-jinyi commented Nov 23, 2025

Summary

  • Fix bug where the "Selected assets count" displayed as 0 in the Imported tab when selecting assets

Root Cause

The getOutputCount function was returning 0 when user_metadata.outputCount was not present.

  • Generated tab: Works correctly because outputCount metadata is set during generation
  • Imported tab: outputCount metadata is never set, so it always returns 0 → selected count shows as 0

Solution

Changed the default return value from 0 to 1 when outputCount metadata is not present, ensuring every asset counts as at least 1.

🤖 Generated with Claude Code

┆Issue is synchronized with this Notion page by Unito

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Nov 23, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 23, 2025

📝 Walkthrough

Walkthrough

The getOutputCount function in AssetsSidebarTab.vue is modified to return a minimum value of 1 instead of 0 when the metadata outputCount is invalid or non-positive, affecting how asset output counts are displayed.

Changes

Cohort / File(s) Change Summary
Output Count Logic
src/components/sidebar/tabs/AssetsSidebarTab.vue
Modified getOutputCount to return 1 as the minimum value when metadata outputCount is not a number or not greater than 0, changing the previous behavior of returning 0
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/imported-tab-selected

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6d41e8b and 9d36ee0.

📒 Files selected for processing (1)
  • src/components/sidebar/tabs/AssetsSidebarTab.vue (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: setup
  • GitHub Check: test
  • GitHub Check: lint-and-format
  • GitHub Check: collect
🔇 Additional comments (1)
src/components/sidebar/tabs/AssetsSidebarTab.vue (1)

215-218: LGTM! The fix correctly addresses the selection count issue.

The change from returning 0 to 1 as the default value is semantically correct—each existing asset should count as at least one item. This ensures that imported assets (which lack outputCount metadata) properly contribute to the total selection count displayed in the footer (line 273), while generated assets with explicit outputCount metadata continue to work as expected.

The logic handles all edge cases appropriately (undefined, null, zero, negative numbers, non-numbers), and does not affect the output count badge display logic (line 224) which correctly checks count > 1.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Nov 23, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 11/23/2025, 08:28:42 AM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@viva-jinyi viva-jinyi changed the title Fix: Imported 탭에서 선택된 에셋 카운트가 표시되지 않는 문제 수정 Fix: Selected assets count not updating in Imported tab Nov 23, 2025
@github-actions
Copy link

github-actions bot commented Nov 23, 2025

🎭 Playwright Test Results

⚠️ Tests passed with flaky tests

⏰ Completed at: 11/23/2025, 08:39:21 AM UTC

📈 Summary

  • Total Tests: 496
  • Passed: 484 ✅
  • Failed: 0
  • Flaky: 3 ⚠️
  • Skipped: 9 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 475 / ❌ 0 / ⚠️ 3 / ⏭️ 9
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 6 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

@viva-jinyi viva-jinyi requested a review from a team November 23, 2025 08:28
@viva-jinyi viva-jinyi enabled auto-merge (squash) November 23, 2025 08:28
@github-actions
Copy link

Bundle Size Report

Summary

  • Raw size: 14.2 MB baseline 14.2 MB — ⚪ 0 B
  • Gzip: 2.84 MB baseline 2.84 MB — 🟢 -32 B
  • Brotli: 2.22 MB baseline 2.22 MB — 🟢 -124 B
  • Bundles: 92 current • 92 baseline • 38 added / 38 removed

Category Glance
Vendor & Third-Party ⚪ 0 B (5.7 MB) · Other ⚪ 0 B (3.87 MB) · App Entry Points ⚪ 0 B (3.18 MB) · Graph Workspace ⚪ 0 B (940 kB) · Panels & Settings ⚪ 0 B (306 kB) · UI Components ⚪ 0 B (141 kB) · + 3 more

Per-category breakdown
App Entry Points — 3.18 MB (baseline 3.18 MB) • ⚪ 0 B

Main entry bundles and manifests

File Before After Δ Raw Δ Gzip Δ Brotli
assets/index-Bd0sn9CR.js (new) 2.95 MB 🔴 +2.95 MB 🔴 +615 kB 🔴 +467 kB
assets/index-cPvwn2_N.js (removed) 2.95 MB 🟢 -2.95 MB 🟢 -615 kB 🟢 -467 kB
assets/index-DyQGte9E.js (removed) 228 kB 🟢 -228 kB 🟢 -48.9 kB 🟢 -40.3 kB
assets/index-o7RiQwMs.js (new) 228 kB 🔴 +228 kB 🔴 +48.9 kB 🔴 +40.3 kB
assets/index-BSuZz4v8.js (removed) 345 B 🟢 -345 B 🟢 -246 B 🟢 -203 B
assets/index-Ro_2KccU.js (new) 345 B 🔴 +345 B 🔴 +243 B 🔴 +211 B

Status: 3 added / 3 removed

Graph Workspace — 940 kB (baseline 940 kB) • ⚪ 0 B

Graph editor runtime, canvas, workflow orchestration

File Before After Δ Raw Δ Gzip Δ Brotli
assets/GraphView-eVDVc-zA.js (removed) 940 kB 🟢 -940 kB 🟢 -182 kB 🟢 -140 kB
assets/GraphView-vlvmOTIs.js (new) 940 kB 🔴 +940 kB 🔴 +182 kB 🔴 +139 kB

Status: 1 added / 1 removed

Views & Navigation — 7.97 kB (baseline 7.97 kB) • ⚪ 0 B

Top-level views, pages, and routed surfaces

File Before After Δ Raw Δ Gzip Δ Brotli
assets/UserSelectView-BdtWKgqD.js (removed) 7.97 kB 🟢 -7.97 kB 🟢 -2.43 kB 🟢 -2.14 kB
assets/UserSelectView-C4D7Cj6K.js (new) 7.97 kB 🔴 +7.97 kB 🔴 +2.43 kB 🔴 +2.13 kB

Status: 1 added / 1 removed

Panels & Settings — 306 kB (baseline 306 kB) • ⚪ 0 B

Configuration panels, inspectors, and settings screens

File Before After Δ Raw Δ Gzip Δ Brotli
assets/CreditsPanel-D_fPr7AS.js (removed) 22.9 kB 🟢 -22.9 kB 🟢 -5.46 kB 🟢 -4.78 kB
assets/CreditsPanel-DHtdABZo.js (new) 22.9 kB 🔴 +22.9 kB 🔴 +5.46 kB 🔴 +4.78 kB
assets/KeybindingPanel-BMxOYjJM.js (removed) 15.1 kB 🟢 -15.1 kB 🟢 -3.73 kB 🟢 -3.28 kB
assets/KeybindingPanel-Dd8J5frv.js (new) 15.1 kB 🔴 +15.1 kB 🔴 +3.73 kB 🔴 +3.29 kB
assets/ExtensionPanel-BPCedo0z.js (new) 11.9 kB 🔴 +11.9 kB 🔴 +2.79 kB 🔴 +2.45 kB
assets/ExtensionPanel-D7t-VKDm.js (removed) 11.9 kB 🟢 -11.9 kB 🟢 -2.79 kB 🟢 -2.45 kB
assets/AboutPanel-ChI1pusx.js (removed) 10.1 kB 🟢 -10.1 kB 🟢 -2.62 kB 🟢 -2.31 kB
assets/AboutPanel-CIL78AlY.js (new) 10.1 kB 🔴 +10.1 kB 🔴 +2.62 kB 🔴 +2.31 kB
assets/ServerConfigPanel-BxCOJFVl.js (new) 8.02 kB 🔴 +8.02 kB 🔴 +2.12 kB 🔴 +1.88 kB
assets/ServerConfigPanel-C8qGeCT6.js (removed) 8.02 kB 🟢 -8.02 kB 🟢 -2.12 kB 🟢 -1.87 kB
assets/UserPanel-CcNNTvyZ.js (removed) 7.74 kB 🟢 -7.74 kB 🟢 -2.02 kB 🟢 -1.77 kB
assets/UserPanel-QQdosHq_.js (new) 7.74 kB 🔴 +7.74 kB 🔴 +2.02 kB 🔴 +1.77 kB
assets/settings-BXTtSH4O.js 33.3 kB 33.3 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-C9Pzn-NG.js 25.2 kB 25.2 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-CCy2fA_h.js 27.3 kB 27.3 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-CQpqEFfl.js 26.6 kB 26.6 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-DHcnxypw.js 21.7 kB 21.7 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-DhFTK9fY.js 25.1 kB 25.1 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-DlT4t_ui.js 25.9 kB 25.9 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-DRgSrIdD.js 24.2 kB 24.2 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/settings-tjkeqiZq.js 21.1 kB 21.1 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B

Status: 6 added / 6 removed

UI Components — 141 kB (baseline 141 kB) • ⚪ 0 B

Reusable component library chunks

File Before After Δ Raw Δ Gzip Δ Brotli
assets/Load3D.vue_vue_type_script_setup_true_lang-BffsekHu.js (removed) 53.9 kB 🟢 -53.9 kB 🟢 -8.43 kB 🟢 -7.23 kB
assets/Load3D.vue_vue_type_script_setup_true_lang-BsCdr6sd.js (new) 53.9 kB 🔴 +53.9 kB 🔴 +8.43 kB 🔴 +7.23 kB
assets/WidgetSelect.vue_vue_type_script_setup_true_lang-ChryjY9F.js (new) 48 kB 🔴 +48 kB 🔴 +10.3 kB 🔴 +8.92 kB
assets/WidgetSelect.vue_vue_type_script_setup_true_lang-EK1K-3JJ.js (removed) 48 kB 🟢 -48 kB 🟢 -10.3 kB 🟢 -8.9 kB
assets/WidgetInputNumber.vue_vue_type_script_setup_true_lang--XfL2uWj.js (removed) 12.7 kB 🟢 -12.7 kB 🟢 -3.31 kB 🟢 -2.94 kB
assets/WidgetInputNumber.vue_vue_type_script_setup_true_lang-CFenuWxY.js (new) 12.7 kB 🔴 +12.7 kB 🔴 +3.3 kB 🔴 +2.92 kB
assets/ComfyQueueButton-D4oQnsu0.js (new) 9.22 kB 🔴 +9.22 kB 🔴 +2.5 kB 🔴 +2.2 kB
assets/ComfyQueueButton-DuVh5Hwz.js (removed) 9.22 kB 🟢 -9.22 kB 🟢 -2.5 kB 🟢 -2.2 kB
assets/WidgetLayoutField.vue_vue_type_script_setup_true_lang-BSZtEu7r.js (new) 2.14 kB 🔴 +2.14 kB 🔴 +794 B 🔴 +690 B
assets/WidgetLayoutField.vue_vue_type_script_setup_true_lang-jka8Va8z.js (removed) 2.14 kB 🟢 -2.14 kB 🟢 -794 B 🟢 -692 B
assets/MediaTitle.vue_vue_type_script_setup_true_lang-DMUwUjO0.js (new) 848 B 🔴 +848 B 🔴 +474 B 🔴 +414 B
assets/MediaTitle.vue_vue_type_script_setup_true_lang-KtZ1LbXw.js (removed) 848 B 🟢 -848 B 🟢 -475 B 🟢 -414 B
assets/LazyImage.vue_vue_type_script_setup_true_lang-Wi-CcgaU.js 10.7 kB 10.7 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/UserAvatar.vue_vue_type_script_setup_true_lang-D2s8tnS2.js 1.26 kB 1.26 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/WidgetButton-CtnTx0SP.js 1.72 kB 1.72 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B

Status: 6 added / 6 removed

Data & Services — 12.5 kB (baseline 12.5 kB) • ⚪ 0 B

Stores, services, APIs, and repositories

File Before After Δ Raw Δ Gzip Δ Brotli
assets/keybindingService-D-QtMjMo.js (removed) 7.51 kB 🟢 -7.51 kB 🟢 -1.83 kB 🟢 -1.58 kB
assets/keybindingService-tbqRCKbk.js (new) 7.51 kB 🔴 +7.51 kB 🔴 +1.83 kB 🔴 +1.57 kB
assets/audioService-BHi7g98j.js (removed) 2.2 kB 🟢 -2.2 kB 🟢 -963 B 🟢 -823 B
assets/audioService-hx9o60im.js (new) 2.2 kB 🔴 +2.2 kB 🔴 +962 B 🔴 +823 B
assets/serverConfigStore-D2N-YjtJ.js 2.79 kB 2.79 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B

Status: 2 added / 2 removed

Utilities & Hooks — 2.94 kB (baseline 2.94 kB) • ⚪ 0 B

Helpers, composables, and utility bundles

File Before After Δ Raw Δ Gzip Δ Brotli
assets/audioUtils-B--3HfU0.js (new) 1.41 kB 🔴 +1.41 kB 🔴 +649 B 🔴 +547 B
assets/audioUtils-d-DVkEWf.js (removed) 1.41 kB 🟢 -1.41 kB 🟢 -651 B 🟢 -546 B
assets/mathUtil-CTARWQ-l.js 1.07 kB 1.07 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeFilterUtil-CXKCRJ-m.js 460 B 460 B ⚪ 0 B ⚪ 0 B ⚪ 0 B

Status: 1 added / 1 removed

Vendor & Third-Party — 5.7 MB (baseline 5.7 MB) • ⚪ 0 B

External libraries and shared vendor chunks

File Before After Δ Raw Δ Gzip Δ Brotli
assets/vendor-other-OCBGbKU2.js 3.59 MB 3.59 MB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/vendor-primevue-PESgPnbc.js 517 B 517 B ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/vendor-three-aR6ntw5X.js 1.37 MB 1.37 MB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/vendor-tiptap-CmuD0u4q.js 232 kB 232 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/vendor-vue-DhKsTD0i.js 92.6 kB 92.6 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/vendor-xterm-BZLod3g9.js 407 kB 407 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
Other — 3.87 MB (baseline 3.87 MB) • ⚪ 0 B

Bundles that do not match a named category

File Before After Δ Raw Δ Gzip Δ Brotli
assets/WidgetRecordAudio-CCbowyli.js (new) 21.9 kB 🔴 +21.9 kB 🔴 +5.51 kB 🔴 +4.87 kB
assets/WidgetRecordAudio-D91-hpR5.js (removed) 21.9 kB 🟢 -21.9 kB 🟢 -5.52 kB 🟢 -4.87 kB
assets/AudioPreviewPlayer-Dcw7VG0W.js (removed) 14.9 kB 🟢 -14.9 kB 🟢 -3.69 kB 🟢 -3.3 kB
assets/AudioPreviewPlayer-UoiW3Znh.js (new) 14.9 kB 🔴 +14.9 kB 🔴 +3.69 kB 🔴 +3.3 kB
assets/WidgetGalleria-BvcA_mbC.js (removed) 5.56 kB 🟢 -5.56 kB 🟢 -1.74 kB 🟢 -1.54 kB
assets/WidgetGalleria-y6cKJqYq.js (new) 5.56 kB 🔴 +5.56 kB 🔴 +1.74 kB 🔴 +1.54 kB
assets/WidgetColorPicker-aoCfyR0E.js (removed) 4.87 kB 🟢 -4.87 kB 🟢 -1.69 kB 🟢 -1.48 kB
assets/WidgetColorPicker-DgPIuR6B.js (new) 4.87 kB 🔴 +4.87 kB 🔴 +1.69 kB 🔴 +1.48 kB
assets/WidgetMarkdown-Dpn67csf.js (new) 4.64 kB 🔴 +4.64 kB 🔴 +1.62 kB 🔴 +1.41 kB
assets/WidgetMarkdown-TOAWTHLe.js (removed) 4.64 kB 🟢 -4.64 kB 🟢 -1.62 kB 🟢 -1.41 kB
assets/WidgetAudioUI-MjyNob98.js (removed) 4.33 kB 🟢 -4.33 kB 🟢 -1.44 kB 🟢 -1.29 kB
assets/WidgetAudioUI-qWrnOHro.js (new) 4.33 kB 🔴 +4.33 kB 🔴 +1.44 kB 🔴 +1.29 kB
assets/WidgetTextarea-BImtrKdb.js (new) 4.03 kB 🔴 +4.03 kB 🔴 +1.34 kB 🔴 +1.19 kB
assets/WidgetTextarea-C0lPx5H4.js (removed) 4.03 kB 🟢 -4.03 kB 🟢 -1.34 kB 🟢 -1.19 kB
assets/WidgetInputText-DCEqpzkD.js (removed) 3.45 kB 🟢 -3.45 kB 🟢 -1.24 kB 🟢 -1.09 kB
assets/WidgetInputText-EcQfsyGi.js (new) 3.45 kB 🔴 +3.45 kB 🔴 +1.23 kB 🔴 +1.08 kB
assets/WidgetToggleSwitch-D6gWcy9T.js (new) 3.23 kB 🔴 +3.23 kB 🔴 +1.14 kB 🔴 +1.01 kB
assets/WidgetToggleSwitch-H8_xMbu-.js (removed) 3.23 kB 🟢 -3.23 kB 🟢 -1.14 kB 🟢 -1.01 kB
assets/MediaImageBottom-CBDRvTd8.js (removed) 3.05 kB 🟢 -3.05 kB 🟢 -1.05 kB 🟢 -924 B
assets/MediaImageBottom-YPLxGasD.js (new) 3.05 kB 🔴 +3.05 kB 🔴 +1.05 kB 🔴 +908 B
assets/MediaAudioBottom-BdJ8N2j4.js (new) 3 kB 🔴 +3 kB 🔴 +1.05 kB 🔴 +917 B
assets/MediaAudioBottom-Ctrx-_qr.js (removed) 3 kB 🟢 -3 kB 🟢 -1.05 kB 🟢 -921 B
assets/Media3DTop-DmTYRX9b.js (removed) 3 kB 🟢 -3 kB 🟢 -1.08 kB 🟢 -929 B
assets/Media3DTop-Dyi0_-XI.js (new) 3 kB 🔴 +3 kB 🔴 +1.08 kB 🔴 +912 B
assets/MediaVideoBottom-C_cN_01Y.js (new) 3 kB 🔴 +3 kB 🔴 +1.05 kB 🔴 +926 B
assets/MediaVideoBottom-naMkZQrt.js (removed) 3 kB 🟢 -3 kB 🟢 -1.05 kB 🟢 -913 B
assets/Media3DBottom-AdxArhNH.js (new) 2.98 kB 🔴 +2.98 kB 🔴 +1.04 kB 🔴 +928 B
assets/Media3DBottom-N0Rnb2mb.js (removed) 2.98 kB 🟢 -2.98 kB 🟢 -1.04 kB 🟢 -919 B
assets/WidgetSelect-B3oIAuUc.js (removed) 2.17 kB 🟢 -2.17 kB 🟢 -676 B 🟢 -570 B
assets/WidgetSelect-DB7Mo68b.js (new) 2.17 kB 🔴 +2.17 kB 🔴 +675 B 🔴 +575 B
assets/WidgetInputNumber-BhuZFljm.js (new) 2.12 kB 🔴 +2.12 kB 🔴 +662 B 🔴 +556 B
assets/WidgetInputNumber-C3KBD-zQ.js (removed) 2.12 kB 🟢 -2.12 kB 🟢 -664 B 🟢 -556 B
assets/Load3D-B00P3P9T.js (new) 1.94 kB 🔴 +1.94 kB 🔴 +596 B 🔴 +498 B
assets/Load3D-TBaXqF1h.js (removed) 1.94 kB 🟢 -1.94 kB 🟢 -597 B 🟢 -531 B
assets/WidgetLegacy-BudwySMW.js (removed) 1.88 kB 🟢 -1.88 kB 🟢 -561 B 🟢 -477 B
assets/WidgetLegacy-Dp3WsDdD.js (new) 1.88 kB 🔴 +1.88 kB 🔴 +560 B 🔴 +478 B
assets/commands-_s-RvhJR.js 13.6 kB 13.6 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-BuUILW6P.js 13 kB 13 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-BV4R6fLx.js 14.9 kB 14.9 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-CLwPdnT6.js 14.2 kB 14.2 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-CWMchBmd.js 15.9 kB 15.9 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-DazTQhtc.js 12.9 kB 12.9 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-DmWrOe93.js 13.7 kB 13.7 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-DwiH7Kr6.js 13.8 kB 13.8 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/commands-mS3LCNPn.js 14.5 kB 14.5 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-5lOBdqcC.js 84.5 kB 84.5 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-BOCuaVpE.js 73.4 kB 73.4 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-ClrEFGUz.js 72.4 kB 72.4 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-CyNU0iQX.js 99.3 kB 99.3 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-D7gwLxft.js 114 kB 114 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-DC8o4BCt.js 86.8 kB 86.8 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-DKiesCV4.js 94.3 kB 94.3 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-Hq2q-OtB.js 83.6 kB 83.6 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/main-USAlAlnj.js 82 kB 82 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/MediaAudioTop-RTI8pWy9.js 1.42 kB 1.42 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/MediaImageTop-DlKHfEAA.js 1.68 kB 1.68 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/MediaVideoTop-BB0lT7C5.js 2.7 kB 2.7 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-_Px5dSNW.js 306 kB 306 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-7z21KPoS.js 285 kB 285 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-BWKZzBPK.js 346 kB 346 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-CGbgH4Yl.js 320 kB 320 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-CjjjdWkV.js 313 kB 313 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-CVrNtxvj.js 288 kB 288 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-DLRSA0IK.js 309 kB 309 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-DQV2gnwA.js 372 kB 372 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/nodeDefs-ofqLG5vz.js 310 kB 310 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/WidgetChart-rDmYEWg5.js 2.39 kB 2.39 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/WidgetImageCompare-Ds3K3ULR.js 2.15 kB 2.15 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B
assets/widgetPropFilter-BIbGSUAt.js 1.28 kB 1.28 kB ⚪ 0 B ⚪ 0 B ⚪ 0 B

Status: 18 added / 18 removed

@viva-jinyi viva-jinyi merged commit 09c888e into main Nov 23, 2025
32 checks passed
@viva-jinyi viva-jinyi deleted the hotfix/imported-tab-selected branch November 23, 2025 08:57
@christian-byrne christian-byrne added needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch 1.32 cloud/1.32 labels Nov 23, 2025
github-actions bot pushed a commit that referenced this pull request Nov 23, 2025
## Summary
- Fix bug where the "Selected assets count" displayed as 0 in the
Imported tab when selecting assets

## Root Cause
The `getOutputCount` function was returning `0` when
`user_metadata.outputCount` was not present.

- **Generated tab**: Works correctly because `outputCount` metadata is
set during generation
- **Imported tab**: `outputCount` metadata is never set, so it always
returns `0` → selected count shows as 0

## Solution
Changed the default return value from `0` to `1` when `outputCount`
metadata is not present, ensuring every asset counts as at least 1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
github-actions bot pushed a commit that referenced this pull request Nov 23, 2025
## Summary
- Fix bug where the "Selected assets count" displayed as 0 in the
Imported tab when selecting assets

## Root Cause
The `getOutputCount` function was returning `0` when
`user_metadata.outputCount` was not present.

- **Generated tab**: Works correctly because `outputCount` metadata is
set during generation
- **Imported tab**: `outputCount` metadata is never set, so it always
returns `0` → selected count shows as 0

## Solution
Changed the default return value from `0` to `1` when `outputCount`
metadata is not present, ensuring every asset counts as at least 1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@comfy-pr-bot
Copy link
Member

@viva-jinyi Backport branch created but PR creation failed for core/1.32. Please create the PR manually from branch backport-6842-to-core-1.32

@comfy-pr-bot
Copy link
Member

@viva-jinyi Successfully backported to #6873

@github-actions github-actions bot removed the needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch label Nov 23, 2025
christian-byrne pushed a commit that referenced this pull request Nov 23, 2025
…rted tab (#6873)

Backport of #6842 to `cloud/1.32`

Automatically created by backport workflow.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6873-backport-cloud-1-32-Fix-Selected-assets-count-not-updating-in-Imported-tab-2b46d73d365081a0992bed6de971d81e)
by [Unito](https://www.unito.io)

Co-authored-by: Jin Yi <[email protected]>
@christian-byrne christian-byrne added needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch 1.32 and removed 1.32 cloud/1.32 labels Nov 25, 2025
github-actions bot pushed a commit that referenced this pull request Nov 25, 2025
## Summary
- Fix bug where the "Selected assets count" displayed as 0 in the
Imported tab when selecting assets

## Root Cause
The `getOutputCount` function was returning `0` when
`user_metadata.outputCount` was not present.

- **Generated tab**: Works correctly because `outputCount` metadata is
set during generation
- **Imported tab**: `outputCount` metadata is never set, so it always
returns `0` → selected count shows as 0

## Solution
Changed the default return value from `0` to `1` when `outputCount`
metadata is not present, ensuring every asset counts as at least 1.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@comfy-pr-bot
Copy link
Member

@viva-jinyi Successfully backported to #6931

@github-actions github-actions bot removed the needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch label Nov 25, 2025
christian-byrne pushed a commit that referenced this pull request Nov 25, 2025
…ted tab (#6931)

Backport of #6842 to `core/1.32`

Automatically created by backport workflow.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6931-backport-core-1-32-Fix-Selected-assets-count-not-updating-in-Imported-tab-2b66d73d365081edb16ce02c2b55ada0)
by [Unito](https://www.unito.io)

Co-authored-by: Jin Yi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.32 size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants