Architecture: ApiClient autoload (HTTP + JWT + error contract) #29

Merged
reviewer-bot merged 7 commits from agent/5b3da417b50c into main 2026-08-19 15:45:01 +00:00
Collaborator

Auto-opened by Claude for task 5b3da417b50c (unblocking manually after verification — smoke test now passes end-to-end against a live backend).

Dashboard: http://cloudbison:7070/task/5b3da417b50c

Auto-opened by Claude for task `5b3da417b50c` (unblocking manually after verification — smoke test now passes end-to-end against a live backend). Dashboard: http://cloudbison:7070/task/5b3da417b50c
agent-bot added 7 commits 2026-08-19 14:41:36 +00:00
Two Godot --script-mode-only quirks, neither a bug in ApiClient itself
(verified working against a real backend, health_check returns 200):
1. Bare autoload identifiers don't resolve at compile time in --script/
   SceneTree runs (the node exists at runtime, root.has_node() is true,
   but referencing it as a plain global name fails to compile) -- use
   get_node() instead.
2. A freshly add_child()'d HTTPRequest isn't reliably "inside tree" yet
   at _initialize() time, since --script mode hasn't processed a frame
   yet -- one process_frame await fixes it.

Verified: full smoke test now passes end-to-end against a live backend
(docker compose postgres + alembic migrations + uvicorn), confirming
ApiClient.health_check() genuinely round-trips.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
reviewer-bot merged commit b3e66fc072 into main 2026-08-19 15:45:01 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: daddy/modbison#29