Comfy-Org / Comfy-Org/ComfyUI_frontend

Implement or remove subgraph IO link connection stubs (throw 'Not implemented')

Open
#11,069 0 comments 0 reactions 0 assignees View on GitHub
area:subgraph audit:half-implemented code-audit effort:medium priority:recommended
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

## Recommendation

### 🔧 FINISH: Complete Subgraph IO Link Connections

Two methods explicitly throw `'Not implemented'`:

- `connectToSubgraphOutput()` in `src/lib/litegraph/src/canvas/ToInputFromIoNodeLink.ts:84`
- `connectToSubgraphInput()` in `src/lib/litegraph/src/canvas/ToOutputFromIoNodeLink.ts:72`

## What Was Found

These are part of the subgraph IO node link system. The methods exist in the class hierarchy but throw at runtime if called. This could cause unexpected errors for users working with subgraphs.

## How to Fix

Either:
1. **Implement** the subgraph IO node link connections (preferred if the feature is needed)
2. **Remove** if the code paths are unreachable and the feature was abandoned

---
**Category:** Half-Implemented | **Priority:** Recommended | **Effort:** Medium
Part of #11022
_Filed by repo-audit skill_

┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-11069-Implement-or-remove-subgraph-IO-link-connection-stubs-throw-Not-implemented-33e6d73d3650814e93bac492277bb379) by [Unito](https://www.unito.io)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.