Skip to content

fix: Bundle Vue for ComfyUI frontend v1.33.9+ compatibility (v0.0.8)#7

Merged
akatz-ai merged 2 commits intomainfrom
dev
Dec 9, 2025
Merged

fix: Bundle Vue for ComfyUI frontend v1.33.9+ compatibility (v0.0.8)#7
akatz-ai merged 2 commits intomainfrom
dev

Conversation

@akatz-ai
Copy link
Copy Markdown
Member

@akatz-ai akatz-ai commented Dec 9, 2025

Summary

ComfyUI frontend v1.33.9 removed the import map plugin (PR #6899) that previously exposed Vue and related dependencies to extensions. This broke our Vue-based panel with the error:

Failed to resolve module specifier "vue"

Per official guidance, extensions must now bundle Vue directly.

Changes

  • Bundle Vue and dependencies - Remove vue, vue-i18n, pinia, primevue from Vite externals so they get bundled
  • Define process.env.NODE_ENV - Fix browser runtime error from Vue's Node.js environment checks
  • Fix version check script - Look for quoted version strings to avoid matching Vue's internal version
  • Version bump - 0.0.7 → 0.0.8

Impact

Metric Before After
Bundle size (gzip) ~100 KB ~176 KB
Works on v1.32.x
Works on v1.33.9+

Test Plan

  • Tested on ComfyUI with frontend v1.33.10 (latest master)
  • Verified ComfyGit panel and commit buttons appear
  • Verified backwards compatibility with older frontend versions

ComfyUI frontend v1.33.9 removed the import map plugin (PR #6899) that
previously exposed Vue and related dependencies to extensions. Extensions
using `rollupOptions.external: ['vue', ...]` now fail with:

  "Failed to resolve module specifier 'vue'"

Per official guidance (Issue #7267), extensions must now bundle Vue
directly. This removes Vue, vue-i18n, pinia, and primevue from the
external list so they get bundled into comfygit-panel.js.

Also:
- Define process.env.NODE_ENV='production' to fix browser runtime error
- Fix version check script to avoid matching Vue's version number

Bundle size increases from ~100KB to ~176KB (gzipped) but now works on
both old (v1.32.x) and new (v1.33.9+) ComfyUI versions.

Context: ctx://46f65107e266912e/c582c011-b93d-451a-94b2-8bb32f70fc1b
@akatz-ai akatz-ai merged commit 9626a59 into main Dec 9, 2025
2 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.

1 participant