Skip to content

chore: bump ComfyUI to 0.16.4#1644

Merged
benceruleanlu merged 2 commits intomainfrom
chore/bump-comfyui-0.16.4
Mar 7, 2026
Merged

chore: bump ComfyUI to 0.16.4#1644
benceruleanlu merged 2 commits intomainfrom
chore/bump-comfyui-0.16.4

Conversation

@benceruleanlu
Copy link
Member

@benceruleanlu benceruleanlu commented Mar 7, 2026

Summary

  • Bump bundled ComfyUI from 0.16.3 to 0.16.4.
  • Update scripts/core-requirements.patch to match upstream requirements.txt context for v0.16.4.
  • Refresh assets/requirements/*.compiled by rerunning yarn make:assets and recompile_from_headers.sh.

Testing

  • yarn format
  • yarn lint
  • yarn typecheck
  • yarn test:unit

Follow-up

Summary by CodeRabbit

  • Chores
    • Updated ComfyUI to 0.16.4.
    • Bumped multiple workflow-related packages to new patch versions.
    • Added simpleeval 1.0.3 as a new pinned dependency.
    • Removed one legacy frontend package entry and adjusted core requirements for platform builds.

@benceruleanlu benceruleanlu requested a review from a team as a code owner March 7, 2026 22:04
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. dependencies labels Mar 7, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 7, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c1fbc2aa-6dbe-48c1-9380-9bb85812c001

📥 Commits

Reviewing files that changed from the base of the PR and between 41bbccd and 168b50d.

📒 Files selected for processing (1)
  • assets/requirements/macos.compiled

📝 Walkthrough

Walkthrough

Bumped multiple comfyui-workflow-templates packages across compiled requirements, added explicit dependency simpleeval==1.0.3, removed comfyui-frontend-package==1.39.19 in a patch, and updated package.json's comfyUI config to 0.16.4.

Changes

Cohort / File(s) Summary
Compiled Requirements
assets/requirements/macos.compiled, assets/requirements/windows_amd.compiled, assets/requirements/windows_cpu.compiled, assets/requirements/windows_nvidia.compiled
Bumped comfyui-workflow-templates packages (main, core, media-api, media-image, media-other, media-video) to new patch versions (e.g., 0.9.10→0.9.11 / 0.3.158→0.3.159). Added simpleeval==1.0.3 entries.
Package Configuration
package.json
Updated config.comfyUI version from 0.16.3 to 0.16.4.
Patch File
scripts/core-requirements.patch
Removed comfyui-frontend-package==1.39.19 and replaced comfyui-workflow-templates==0.9.10 with 0.9.11.

Possibly related PRs

Suggested reviewers

  • christian-byrne
  • luke-mino-altherr

Poem

🐇 I hopped through pins and version trees,

Pushing tiny bumps with nimble ease,
simpleeval tucked in my satchel light,
ComfyUI polished, tidy and bright,
A rabbit's cheer for builds that compile right.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: bump ComfyUI to 0.16.4' clearly and concisely summarizes the main change—bumping the ComfyUI version from 0.16.3 to 0.16.4 across all relevant files (package.json and compiled requirements).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/bump-comfyui-0.16.4

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@benceruleanlu benceruleanlu merged commit 0522da8 into main Mar 7, 2026
6 checks passed
@benceruleanlu benceruleanlu deleted the chore/bump-comfyui-0.16.4 branch March 7, 2026 22:07
benceruleanlu added a commit that referenced this pull request Mar 7, 2026
## Summary
- Bump desktop package version to `0.8.17` on top of ComfyUI bump stack.

## Testing
- `yarn format`
- `yarn lint`
- `yarn typecheck`

## Merge after
- Merge after: #1644
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 41bbccd14c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

# torchvision
# from https://pypi.org/simple
torchaudio==2.10.0
torchaudio==2.9.1

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Keep macOS torchaudio pin in sync with torch

The macOS compiled lock now pins torch==2.10.0 but torchaudio==2.9.1, and this file is installed directly by installRequirements() (src/virtualEnvironment.ts) during normal setup, so users can end up with a mixed PyTorch stack. In practice, torch ecosystem packages are version-coupled enough that cross-minor combinations frequently fail at install or import time (especially when ComfyUI loads audio nodes), so this should be pinned to a matching torchaudio release (or torch downgraded consistently).

Useful? React with 👍 / 👎.

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

Labels

dependencies size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant