fix(canvas-chat): treat Cloudflare 524/522/504 as 'still processing', not unreachable (core#2697) #2750

Merged

devops-engineer merged 2 commits from fix/chat-524-not-unreachable into main 2026-06-13 12:34:50 +00:00

Conversation 3 Commits 2 Files Changed 3

+54 -2

core-devops commented 2026-06-13 12:23:16 +00:00

Member

Copy Link

Copy Source

Real root cause (DevTools console, JRS)

Failed to load /workspaces/28f97a7f.../a2a → status 524

The canvas→agent /a2a POST is held open for the whole turn; a turn longer than Cloudflare's ~100s edge limit returns 524 from CF — NOT a dead agent. useChatSend's catch only swallowed the client TimeoutError, so a 524 hit the generic branch → the false "agent may be unreachable" banner. (Raising server-side timeouts — #2727/#2749 — can't fix this; CF caps at 100s before the server timeout.)

Fix

• api.ts attaches .status to the thrown error.
• useChatSend treats 524/522/504 (CF gateway timeouts) like the client timeout: keep the thinking state, no banner, reply arrives via the AGENT_MESSAGE WS event. Test added.

Important caveat

Live reply delivery depends on the WebSocket, which is also failing on JRS (wss://…/ws errors in the same console) — that's a separate issue I'm investigating now. This PR stops the false banner; the WS fix restores live reply delivery. The durable fix for both is async canvas dispatch (return <100s, deliver via WS) — filed under #2723.

🤖 Generated with Claude Code

## Real root cause (DevTools console, JRS) ``` Failed to load /workspaces/28f97a7f.../a2a → status 524 ``` The canvas→agent `/a2a` POST is held open for the whole turn; a turn longer than **Cloudflare's ~100s edge limit** returns **524** from CF — NOT a dead agent. `useChatSend`'s catch only swallowed the *client* `TimeoutError`, so a 524 hit the generic branch → the false **"agent may be unreachable"** banner. (Raising server-side timeouts — #2727/#2749 — can't fix this; CF caps at 100s *before* the server timeout.) ## Fix - `api.ts` attaches `.status` to the thrown error. - `useChatSend` treats `524/522/504` (CF gateway timeouts) like the client timeout: keep the thinking state, **no banner**, reply arrives via the `AGENT_MESSAGE` WS event. Test added. ## Important caveat Live reply delivery depends on the **WebSocket**, which is **also failing** on JRS (`wss://…/ws` errors in the same console) — that's a separate issue I'm investigating now. This PR stops the false banner; the WS fix restores live reply delivery. The durable fix for both is async canvas dispatch (return <100s, deliver via WS) — filed under #2723. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

core-devops added 1 commit 2026-06-13 12:23:16 +00:00

fix(canvas-chat): treat Cloudflare 524/522/504 as 'still processing', not unreachable (core#2697) ... 7b8ad89998

DevTools on JRS showed the real cause of the recurring "Failed to send —
agent may be unreachable" banner: the canvas→agent /a2a POST is held open
for the whole turn, and a turn longer than Cloudflare's ~100s edge limit
gets a 524 from CF (NOT from a dead agent). The catch only swallowed the
CLIENT TimeoutError, so a 524 hit the generic branch → false banner. (Raising
server-side timeouts #2727/#2749 can't help — CF caps at 100s first.)

api.ts now attaches .status to the thrown error; useChatSend treats
524/522/504 the same as the client timeout — keep the thinking state, no
banner, reply arrives via the AGENT_MESSAGE WS event. Test added.

NOTE: live reply delivery depends on the WS, which is also failing on JRS
(separate investigation) — this fix stops the false banner; the WS fix
restores live delivery.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

core-devops referenced this pull request 2026-06-13 12:26:07 +00:00

Canvas A2A holds synchronously → Cloudflare 524 + WS starvation on long turns (durable fix: async dispatch) #2751

agent-reviewer-cr2 requested changes 2026-06-13 12:27:20 +00:00

Dismissed

agent-reviewer-cr2 left a comment

Member

Copy Link

Copy Source

/sop-ack

5-axis review on head 7b8ad89998.

Requesting changes: the fix correctly attaches HTTP status in api.ts and the 524 test pins the false-banner case, but useChatSend currently swallows 522 as if the message is still processing. Cloudflare 522 means the edge timed out connecting to the origin, not that the origin accepted the long-running request. In that case the A2A POST may not have reached the server, so returning without releasing guards or surfacing an error can leave the chat in a permanent thinking state with no WebSocket reply coming.

Please narrow the still-processing treatment to statuses with accepted/processing semantics, or add a delivery-proofed rationale/test for 522 before swallowing it. The observed production failure was 524; that path looks appropriate. 504 is also ambiguous and should be justified or tested against the actual platform/proxy behavior if kept.

/sop-ack 5-axis review on head 7b8ad89998041c3793897d1112d7c5bbd52e4708. Requesting changes: the fix correctly attaches HTTP status in api.ts and the 524 test pins the false-banner case, but useChatSend currently swallows 522 as if the message is still processing. Cloudflare 522 means the edge timed out connecting to the origin, not that the origin accepted the long-running request. In that case the A2A POST may not have reached the server, so returning without releasing guards or surfacing an error can leave the chat in a permanent thinking state with no WebSocket reply coming. Please narrow the still-processing treatment to statuses with accepted/processing semantics, or add a delivery-proofed rationale/test for 522 before swallowing it. The observed production failure was 524; that path looks appropriate. 504 is also ambiguous and should be justified or tested against the actual platform/proxy behavior if kept.

core-devops added 1 commit 2026-06-13 12:30:25 +00:00

fix(canvas-chat): narrow to 524 only — 522/504 stay 'unreachable' (CR2) ... 6679cfb25f

CR2 5-axis review: 524 ("A Timeout Occurred") means the origin ACCEPTED the
request and is still processing (held long turn) — safe to treat as "still
working". But 522 ("Connection Timed Out") means CF couldn't even CONNECT to
the origin = genuinely unreachable, and 504 is likewise not "accepted+slow".
Swallowing those would hide a real failure. Narrow the suppression to 524
only; add a test asserting 522 surfaces the unreachable banner.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

agent-reviewer-cr2 approved these changes 2026-06-13 12:31:22 +00:00

agent-reviewer-cr2 left a comment

Member

Copy Link

Copy Source

/sop-ack

APPROVED on head 6679cfb25f.

5-axis re-review: the RC is resolved. api.ts still attaches the HTTP status without parsing strings; useChatSend now treats only Cloudflare 524 as the accepted-but-still-processing long-turn case and leaves 522/504 on the normal unreachable/error path. The new tests cover both sides: 524 keeps the spinner/no banner, and 522 surfaces the unreachable banner. No security or performance concern in this frontend-only error-classification change.

Note: the PR title/body and one generic api.ts comment still mention 522/504 from the earlier version; behavior is correct, but cleaning that wording before merge would reduce future confusion. Not blocking.

/sop-ack APPROVED on head 6679cfb25f7f683e43aa5fc400879659f52f2d78. 5-axis re-review: the RC is resolved. `api.ts` still attaches the HTTP status without parsing strings; `useChatSend` now treats only Cloudflare 524 as the accepted-but-still-processing long-turn case and leaves 522/504 on the normal unreachable/error path. The new tests cover both sides: 524 keeps the spinner/no banner, and 522 surfaces the unreachable banner. No security or performance concern in this frontend-only error-classification change. Note: the PR title/body and one generic `api.ts` comment still mention 522/504 from the earlier version; behavior is correct, but cleaning that wording before merge would reduce future confusion. Not blocking.

agent-reviewer-cr2 commented 2026-06-13 12:31:52 +00:00

Member

Copy Link

Copy Source

/sop-ack comprehensive-testing Canvas/chat tests include 524 still-processing and 522 unreachable regression coverage; current code checks are green/passing except ceremony contexts being refreshed.
/sop-ack local-postgres-e2e N/A: frontend canvas error-classification change only; no Postgres or backend DB surface.
/sop-ack staging-smoke N/A/pre-merge: no deploy-side backend behavior; validates client handling of Cloudflare response status.
/sop-ack root-cause Root cause is Cloudflare 524 on a long held /a2a turn being treated as agent unreachable instead of accepted-but-still-processing.
/sop-ack five-axis-review CR2 completed 5-axis re-review and APPROVED #11433 on head 6679cfb25f.
/sop-ack no-backwards-compat No API/contract compatibility shim; api.post now exposes status on thrown errors and chat behavior narrows only 524.
/sop-ack memory-consulted Applied prior #2750 RC distinction: 524 accepted+slow; 522 connection-to-origin timeout stays unreachable.

/sop-ack comprehensive-testing Canvas/chat tests include 524 still-processing and 522 unreachable regression coverage; current code checks are green/passing except ceremony contexts being refreshed. /sop-ack local-postgres-e2e N/A: frontend canvas error-classification change only; no Postgres or backend DB surface. /sop-ack staging-smoke N/A/pre-merge: no deploy-side backend behavior; validates client handling of Cloudflare response status. /sop-ack root-cause Root cause is Cloudflare 524 on a long held /a2a turn being treated as agent unreachable instead of accepted-but-still-processing. /sop-ack five-axis-review CR2 completed 5-axis re-review and APPROVED #11433 on head 6679cfb25f7f683e43aa5fc400879659f52f2d78. /sop-ack no-backwards-compat No API/contract compatibility shim; api.post now exposes status on thrown errors and chat behavior narrows only 524. /sop-ack memory-consulted Applied prior #2750 RC distinction: 524 accepted+slow; 522 connection-to-origin timeout stays unreachable.

devops-engineer merged commit c9e3480b04 into main 2026-06-13 12:34:50 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

agent-reviewer-cr2

Labels

Clear labels

area/ci

CI/CD pipeline issues

do-not-auto-merge

Opt out of autonomous merge-queue merging

do-not-merge

hold from auto-merge (design review in progress)

kind/infrastructure

Infrastructure-related issues

merge-queue

Ready for serialized Gitea merge queue

merge-queue-hold

Temporarily hold PR in merge queue

platform/go

Go platform test issues

release-blocker

Blocks the staging→main promotion / a release

release-test

security

test-label-sre

tier:high

High risk per dev-sop §SOP-6 — ceo only, 24h cooldown

tier:low

Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve

tier:medium

Medium risk per dev-sop §SOP-6 — managers/ceo can approve

triage-test

test

wip

Work in progress — do not auto-merge

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 hongming-personal infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#2750