Comfy-Org / Comfy-Org/ComfyUI_frontend

Follow-up: Add broader mutation-path coverage for public-inclusive input cache invalidation

Open
#11,883 0 comments 0 reactions 1 assignee Claimed by @jaeone94 View on GitHub
area:testing
Dominant language
TypeScript
Stars
2k
Forks
699
Avg merge
1d 7h
Merged PRs (30d)
490

Description

## Context

Identified as a follow-up item in PR #11873.

## Problem

The public-inclusive input asset cache introduced in #11873 (`assetService.getInputAssetsIncludingPublic()`) currently has invalidation coverage for a limited set of mutation paths. As the cache gains more consumers, additional mutation paths may need to trigger invalidation to keep the cache consistent.

## Proposed Work

Once the cache has more consumers, review all mutation paths that can change input assets and ensure each triggers `assetService.invalidateInputAssetsIncludingPublic()`. Add test coverage for any newly identified invalidation paths.

## References

- PR: https://github.com/Comfy-Org/ComfyUI_frontend/pull/11873
- Requested by: @jaeone94

┆Issue is synchronized with this [Notion page](https://www.notion.so/Issue-11883-Follow-up-Add-broader-mutation-path-coverage-for-public-inclusive-input-cache-invali-3566d73d365081f091cfda0f207ba497) 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.