Change Save Image default value

[andreszs]

Set a more useful default filename for the Save Image node. Sample output path would be: output/2025-01-07/190401_00001_.png

Sorting the folders and files by name will also sort them chronologically.

Note that "00001" is a suffix added by ComfyUI itself.

[catboxanon]

The date syntax in the filename is substituted in the frontend, not the backend. I'm not sure this is a safe change to make, especially since this behavior technically comes from a frontend extension that can be disabled by the user (Comfy.SaveImageExtraOutput).
https://github.com/Comfy-Org/ComfyUI_frontend/blob/daee073045e895821536d873083abf071602b3d8/src/scripts/utils.ts#L48

[andreszs]

The date syntax in the filename is substituted in the frontend, not the backend. I'm not sure this is a safe change to make, especially since this behavior technically comes from a frontend extension that can be disabled by the user (Comfy.SaveImageExtraOutput). https://github.com/Comfy-Org/ComfyUI_frontend/blob/daee073045e895821536d873083abf071602b3d8/src/scripts/utils.ts#L48

And that remains unchanged, this PR simply updates the default value of newly created Save Image node from ComfyUI to %date:yyyy-MM-dd%/%date:hhmmss%.

I don't think the filename formatting depends on any extension, it's more likely a core feature.

[catboxanon]

It's problematic in that you are relying on the frontend in use to implement that functionality. The string is processed in the frontend before being sent to the backend. Raw backend requests will do nothing with that special syntax.

It's also not a core feature, it's still an extension. Extensions have been moved to core in the past, which prevents them from being disabled, but this is not one of those. Changing the default of this node would warrant proposing this to be a core feature imo -- but this still doesn't fully address the aforementioned point.

Fundamentally I have nothing against your PR in it's desire to improve the default file output structure but these things should be considered.

[andreszs]

It's problematic in that you are relying on the frontend in use to implement that functionality. The string is processed in the frontend before being sent to the backend. Raw backend requests will do nothing with that special syntax.

But there’s no way to set this default value in the ComfyUI Frontend repository.

By the way, what other frontends exist beyond the default one? And why should we concern ourselves with the implementations of third-party frontends rather than focusing on the official one? In the worst-case scenario, it’s the responsibility of the third-party frontend developer to ensure compliance with the standard frontend behavior—not the other way around.