Skip to content

Wake Render backend before embedded login - #22

Merged
xrpbanks merged 4 commits into
mainfrom
fix-embedded-backend-wakeup
Aug 29, 2026
Merged

Wake Render backend before embedded login#22
xrpbanks merged 4 commits into
mainfrom
fix-embedded-backend-wakeup

Conversation

@xrpbanks

Copy link
Copy Markdown
Contributor

Summary

  • wake the spun-down Render backend with a GET health probe before creating embedded login state
  • show a clear cold-start message while WordPress prepares Xaman in parallel
  • add behavioral coverage proving readiness precedes login-state creation

Verification

  • Node embedded-login behavior tests pass
  • frontend lint passes
  • production build passes

Add prepareEmbeddedLogin function to handle login preparation and backend readiness.
Copilot AI lite review requested due to automatic review settings August 29, 2026 21:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the embedded Xaman login flow by proactively waking a potentially cold Render backend (via a health/readiness probe) before attempting to create the login state, and adds a regression test to ensure the readiness step happens first.

Changes:

  • Add prepareEmbeddedLogin() to run waitForBackendReady() before startLoginWithRetry() for embedded login.
  • Update the embedded login path in XamanLoginPanel to use the new preparation function and show progress messaging during cold start.
  • Add a Node behavioral test that verifies backend readiness occurs before login-state creation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tools/tests/xaman_login_start_retry.test.mjs Adds coverage asserting the embedded flow wakes the backend before calling login/start.
frontend/components/XamanLoginPanel.tsx Introduces prepareEmbeddedLogin() and routes embedded login through it with a new “waking-up” progress phase.
Suppressed comments (1)

frontend/components/XamanLoginPanel.tsx:623

  • During the new "waking-up" phase, the UI message claims "Xaman is ready", but at this point we are only warming the backend (WordPress/Xaman preparation may still be in progress). This conflicts with the intent to show a cold-start message while WordPress prepares Xaman in parallel and can mislead users.
              reason === "waking-up"
                ? "Xaman is ready. Starting the secure CalorieApp service. This can take about a minute. Keep this page open."

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread frontend/components/XamanLoginPanel.tsx
xrpbanks and others added 2 commits August 29, 2026 23:06
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@xrpbanks
xrpbanks merged commit 258454f into main Aug 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants