From dfb8c8f821812c9a38ab83a1eafe110fbaf660a6 Mon Sep 17 00:00:00 2001 From: christian-byrne Date: Wed, 2 Sep 2026 00:25:05 +0000 Subject: [PATCH] fix(build): correct billing error code in _report_builder_error docstring MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Docstring cited SUBSCRIPTION_REQUIRED as an example builder error code, but comfy-builder only ever emits PAYMENT_REQUIRED for billing refusals (services/comfy-builder/apiserver/httpkit/billing_gate.go). The actual error-mapping code (_builder_msg) passes the body's error field through verbatim with no hardcoded matching, so behavior was already correct — this fixes the misleading guidance only. Source: Linear BE-11190 --- comfy_cli/command/build.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/comfy_cli/command/build.py b/comfy_cli/command/build.py index a26fa7ed..a73e19ed 100644 --- a/comfy_cli/command/build.py +++ b/comfy_cli/command/build.py @@ -2330,7 +2330,7 @@ def _builder_client(renderer, builder_url: str | None): def _report_builder_error(renderer, e) -> None: """Emit one error envelope for a builder failure. Prefers the limited-beta 403, then the builder's own error body (e.g. `INVALID_DEFINITION: …` or - `SUBSCRIPTION_REQUIRED: …`) over urllib's opaque "HTTP Error 400", then the + `PAYMENT_REQUIRED: …`) over urllib's opaque "HTTP Error 400", then the generic transport error. It carries no Build id, because the old `create` verb's orphan case is gone: