[server] Fix inconsistent handling of base path in studio #1782

[michaelvlach]

No description provided.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
agdb	Ready	Preview, Comment	May 1, 2026 2:35pm

[Copilot]

Pull request overview

Fixes inconsistent base-path handling by centralizing server URL construction and applying it consistently across Studio asset rewriting, OpenAPI server metadata, logging, and cluster node identification.

Changes:

• Introduce ConfigImpl::server_url() and use it for Studio URL replacement and OpenAPI server configuration.
• Normalize basepath input (ensure leading /) and strip any path component from address (with a warning).
• Initialize tracing earlier with an Info default and reload to configured log level after config is read.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
agdb_server/src/routes/studio.rs	Use config.server_url() when rewriting Studio’s localhost URL.
agdb_server/src/main.rs	Initialize tracing before config load; log server URL via server_url().
agdb_server/src/config.rs	Add server_url(), normalize basepath, strip path from address, and use server URL for cluster node id selection.
agdb_server/src/cluster.rs	Determine local cluster index using config.server_url().
agdb_server/src/app.rs	Use server_url() for OpenAPI server URL (avoid manual concatenation).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.