Skip to content

feat: Add ComfyUI Template Manager#306

Closed
christian-byrne wants to merge 5 commits into
mainfrom
feature/template-manager
Closed

feat: Add ComfyUI Template Manager#306
christian-byrne wants to merge 5 commits into
mainfrom
feature/template-manager

Conversation

@christian-byrne

@christian-byrne christian-byrne commented Nov 22, 2025

Copy link
Copy Markdown
Contributor

Summary

Complete admin interface for managing ComfyUI workflow templates with modern web technologies and automated workflows.

🎯 Key Features

  • Modern UI: Nuxt 3 + Vue 3 + TypeScript with shadcn-vue components
  • Smart Autocomplete: All inputs show existing patterns from template data
  • Live Preview: Exactly how template will appear in ComfyUI
  • Model Embedding: Automatic embedding into workflow JSON with widget value matching
  • Thumbnail Effects: Hover dissolve, compare slider, zoom, audio support
  • Real-time Validation: Filename uniqueness, format checking, comprehensive validation
  • Backend API: File processing, JSON updates, version bumping
  • GitHub Actions: Automated deployment and CI/CD workflows
  • Testing: Comprehensive test suite with 9 passing tests

🚀 Quick Start

  1. Deploy to Vercel: Set root directory to admin/
  2. Local Development: cd admin && npm install && npm run dev
  3. GitHub Integration: Run npm run setup:github for repository automation

📁 Structure

  • admin/ - Complete Nuxt 3 application
  • .github/workflows/ - CI/CD automation
  • Comprehensive documentation in admin/README.md

🧪 Testing

All tests pass with TypeScript type safety:

  • Model embedding logic
  • Template data parsing
  • File processing utilities
  • Version bumping functions

Test plan

  • Deploy to Vercel for live testing
  • Test complete template creation workflow
  • Verify GitHub integration setup
  • Test model embedding functionality
  • Validate thumbnail effects work correctly

@socket-security

socket-security Bot commented Nov 22, 2025 •
edited
Loading

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addednpm/​class-variance-authority@​0.7.11001006880100
Addednpm/​@​typescript-eslint/​parser@​8.47.01001007198100
Addednpm/​eslint-config-prettier@​9.1.21001007288100
Addednpm/​@​vueuse/​core@​14.0.01001007790100
Addednpm/​clsx@​2.1.11001009680100
Addednpm/​@​typescript-eslint/​eslint-plugin@​8.47.0991008098100
Addednpm/​@​types/​node@​20.19.251001008195100
Addednpm/​@​vueuse/​nuxt@​14.0.01001008395100
Addednpm/​@​vue/​test-utils@​2.4.6991008885100
Addednpm/​autoprefixer@​10.4.221001008986100
Addednpm/​happy-dom@​20.0.10991008895100
Addednpm/​reka-ui@​2.6.0991009689100
Addednpm/​@​nuxtjs/​color-mode@​4.0.0991008989100
Addednpm/​eslint-plugin-prettier@​5.5.410010010091100
Addednpm/​@​vitejs/​plugin-vue@​6.0.210010010092100
Addednpm/​@​nuxt/​devtools@​1.7.0989310099100
Addednpm/​nuxt@​3.17.7979310098100
Addednpm/​shadcn-nuxt@​2.3.39910010094100
Addednpm/​@​tailwindcss/​typography@​0.5.1910010010095100
Addednpm/​jsdom@​27.2.09810010095100
Addednpm/​lucide-vue-next@​0.554.01001009596100
Addednpm/​eslint@​9.39.19710010097100
Addednpm/​prettier@​3.6.29910010098100

View full report

@socket-security

socket-security Bot commented Nov 22, 2025 •
edited
Loading

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Obfuscated code: npm safer-buffer is 94.0% likely obfuscated

Confidence: 0.94

Location: Package overview

From: apps/template-cms/package-lock.json → npm/nuxt@3.17.7 → npm/safer-buffer@2.1.2

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/safer-buffer@2.1.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@christian-byrne
feat: Add ComfyUI Template Manager (clean version)
be8149d 
Complete admin interface for managing ComfyUI workflow templates:
- Modern Nuxt 3 + Vue 3 + TypeScript frontend
- shadcn-vue component system with Tailwind CSS
- API-based template data loading (no file duplication)
- Intelligent autocomplete from existing template data
- Real-time form validation and filename uniqueness checking
- Live template preview showing exact ComfyUI appearance
- Model embedding with widget value matching
- Comprehensive thumbnail effects (hover dissolve, compare slider, zoom, audio)
- Backend API for file processing, JSON updates, and version bumping
- GitHub Actions workflows for automated deployment and CI/CD
- Full test suite with Vitest covering core functionality
- Production-ready with Vercel deployment configuration

Key improvements:
✅ Uses API endpoints to read template data (no duplication)
✅ Smart form with autocomplete suggestions
✅ Model embedding into workflow JSON
✅ File validation and processing
✅ Template preview with thumbnail effects
✅ GitHub integration for automated workflows
✅ Comprehensive testing and TypeScript support
@christian-byrne christian-byrne force-pushed the feature/template-manager branch from c124c70 to be8149d Compare November 22, 2025 22:52
@christian-byrne
fix: Update GitHub Actions to use Node.js 20
d0ca7c8 
- Fix CI failure due to Node.js 18 engine incompatibility
- Nuxt 3.17.7 requires Node.js ^20.9.0 || >=22.0.0
- Update both deploy.yml and template-update.yml workflows
@christian-byrne
Fix TypeScript errors and update dependencies
f2e0d7f 
- Add class-variance-authority dependency for Combobox component
- Fix template data access in admin.vue using getAllTemplateNames()
- Add proper type annotations in test files for workflow nodes and models
- Simplify vitest.config.ts to resolve plugin conflicts
- All tests passing (9/9) and build completes successfully
@christian-byrne
Restructure admin as proper nx app: template-cms
52ff990 
- Create apps/template-cms/ with proper nx project configuration
- Update GitHub workflows to use new app path
- Fix TinaCMS config to build to correct directory
- Update documentation and project structure
- Add old /admin/ directory to gitignore
- All tests passing (9/9) and functionality preserved
@christian-byrne
Complete nx restructure: resolve merge conflicts
fefb811 
- Resolve .gitignore merge conflicts
- Update GitHub workflows to use apps/template-cms paths
- Maintain both old admin ignores and new template-cms structure
@christian-byrne

christian-byrne commented Jan 20, 2026

Copy link
Copy Markdown
Contributor Author

This has been replaced by a better solution at https://comfyui-template-manager.vercel.app/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@christian-byrne