Skip to content

Fix admin loading when API returns HTML and correct service diagnostics - #42

Draft
ZulTheDev wants to merge 1 commit into
mainfrom
fix/admin-api-json-routing
Draft

ZulTheDev wants to merge 1 commit into
mainfrom
fix/admin-api-json-routing

Conversation

@ZulTheDev

Copy link
Copy Markdown
Owner

The admin content loader parses HTML as JSON when its API URL reaches a frontend or misrouted proxy. Service diagnostics also accept any HTTP 200 as healthy, so the UI can report services online while content loading fails.

This change defaults local development to a loopback Vite /api proxy targeting the private Node server on port 8787, while preserving explicit API URL overrides. It validates JSON responses and the content payload, adds a 10-second content timeout, and loads the bundled portfolio snapshot when the content API is unavailable. Loading no longer deletes existing recovery drafts. Service checks validate the expected service/configuration fields; AI status explicitly reports configuration rather than verified availability.

Setup: existing admin/.env API overrides must be cleared to use the development proxy. Remote private APIs can use ADMIN_PROXY_TARGET. See docs/admin-connection.md for local and production setup. Vercel remains the public-services API and is not used for admin writes.

Validation: admin production build passed; four Node regression tests passed; live Vite proxy checks passed for an unavailable backend (JSON 502) and a mock available backend (JSON forwarding). No credentials or user content changed. Actual private/Tailscale services were not accessible for end-to-end verification. Nothing has been merged or deployed.

@vercel

vercel Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
zul-portfolio-api Ready Ready Preview Oct 9, 2026 7:50pm UTC

This branch was successfully deployed

1 active deployment
Preview — bb52fceb Deployed Oct 9, 2026 by vercel[bot]
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.

1 participant