Skip to content

[API Nodes] convert StabilityAI to use new API client - #10582

Merged
Kosinkadink merged 1 commit into
Comfy-Org:masterfrom
bigcat88:feat/api-nodes/stabilityai-new-api-client
Nov 1, 2025
Merged

[API Nodes] convert StabilityAI to use new API client#10582
Kosinkadink merged 1 commit into
Comfy-Org:masterfrom
bigcat88:feat/api-nodes/stabilityai-new-api-client

Conversation

@bigcat88

@bigcat88 bigcat88 commented Nov 1, 2025

Copy link
Copy Markdown
Contributor

Just a simple conversion of the StabilityAI API nodes to use the new API client.

Note: Interesting find - when sending "Accept": "*/*" to the Stability endpoint to retrieve the result, it returns raw binary data. I updated the new client to send "Accept": "application/json" by default (as in the old API client), and to send "Accept": "*/*" only if the expect_binary flag is set, which looks like a correct one adjustment.

All nodes were tested after the conversion:

Screenshot From 2025-10-31 15-29-32

@bigcat88
bigcat88 requested a review from Kosinkadink as a code owner November 1, 2025 06:18
@bigcat88

bigcat88 commented Nov 1, 2025

Copy link
Copy Markdown
Contributor Author

+label: Core

@comfy-pr-bot comfy-pr-bot added the Core Core team dependency label Nov 1, 2025

@Kosinkadink Kosinkadink left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good, merging

@Kosinkadink
Kosinkadink merged commit 20182a3 into Comfy-Org:master Nov 1, 2025
12 checks passed
@bigcat88
bigcat88 deleted the feat/api-nodes/stabilityai-new-api-client branch November 1, 2025 19:14
Subarasheese pushed a commit to Subarasheese/ComfyUI that referenced this pull request Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants