Comfy-Org / Comfy-Org/registry-backend

[Proposal] Let’s Revamp Comfy Registry While Respecting the Community Foundations Comfy-Org Is Built On

Open
#232 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
75
Forks
16
Avg merge
17m
Merged PRs (30d)
1

Description

Comfy Registry/Manager should move away from viewing ordinary plugin and code capabilities as security violations and instead focus on transparent capability declaration, user consent, and **actual malicious behavior**.

Most of the security burden belongs to Comfy-Org and the ComfyUI. Authors cannot provide a reasonable secure extension ABI: process isolation, filesystem sandboxing, network permissions, dependency isolation, or privilege boundaries when ComfyUI itself loads arbitrary Python into the main application trust domain. That has to come from comfy.

Comfy-Org is ofc free to curate its own _service_, but an OSS ecosystem should be especially careful about using the orgs preferred coding policy into a de facto restriction on what legitimate community software is allowed to do. If code performs a lawful, expected function at the user's direction _and transparent discretion_, Manager should describe that capability rather than treating the capability itself as misconduct. Publicly characterizing otherwise legitimate third-party software in that way is not a professional posture for a business to take, particularly when the underlying behavior is ordinary, non-malicious functionality, making the use of that classification in public legally loaded and potentially business/product disparagement.

Registry should distinguish **capability** and **malicious behavior** separately, eg declared:

- Filesystem read/write
- Network access
- Subprocess execution
- External binaries
- Environment access
- Dynamic code execution
- Package/dependency modification
- etc

Users could then filter or restrict on Manager results/settings based on their own risk tolerance. Someone who wants a tightly constrained environment could hide every node requiring filesystem or network access, while power users could allow those capabilities knowingly. "Banned" an especially touchy choice of terminology, should be reserved for actual malicious code and intent.

This would make things a much more useful security layer: `ComfyUI defines and enforces the security boundary, node authors declare what their software needs, Manager communicates that information, and users decide.` or something like that. Just like browsers. or like windows prompts, or mac osx prompts That is more scalable and transparent than expecting every community node author to somehow compensate for the absence of a secure extension ABI in the ComfUi, while their works reputation tarnished by sweeping verdicts that are not transparent to the community.

It is also important to consider user assent and responsibility for third-party driven package changes. A hosted service making automated decisions on a user's behalf should not silently replace a pack with an older one and call that a safe upgrade when the environment isn't even considered before doing so. The risk is being created by the service itself rather than by the node author, or user consent. But more immediately here is the author takes the flack.

Contributor guide

No contributing guide indexed for this repository

Research direction

The proposal names no files, tests, or entry points. First clarify with maintainers whether this issue covers capability declarations, user controls, registry policy, package changes, or a broader redesign; the issue does not define a specific implementation or test. Done would need an agreed scope and acceptance criteria before work can be assessed.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend-api-design
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.