fix(config): Strip urls only if fully qualified

[craciunoiuc]

So if you look at the parse it used string parsing. This means that the /v1 suffix stayed in the host variable aka the name of the node.

Or maybe you like the AI explanation better:

URL parser derived name before removing /v1 → client registered as localhost:8080/v1; resource requests select localhost:8080 → missing client.

Closes: TOOL-1140

[Copilot]

Pull request overview

Updates environment-based metro parsing to correctly handle local proxy URLs.

Changes:

• Derives metro names using net/url.
• Adds coverage for localhost:8080/v1.

Reviewed changes

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

File	Description
internal/config/profile.go	Parses metro URL hosts before deriving names.
internal/config/config_test.go	Tests local proxy URL parsing.

Comments suppressed due to low confidence (1)

internal/config/profile.go:124

• The new local-proxy handling still truncates dotted proxy hosts: http://127.0.0.1:8080/v1 registers the client as 127, and http://proxy.example.test/v1 as proxy, while resource selection uses the endpoint authority (the same mismatch described in the PR for localhost:8080). Preserve the full authority for non-Unikraft proxy hosts and only derive the first label for known cloud API hostnames.

		host := strings.TrimPrefix(u.Host, "api.")
		name, _, _ = strings.Cut(host, ".")

---

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

[jedevc]

Instead of doing this then, can't we just reformat the URL?

Or alternatively, just share the /v1 stripping between name + endpoint.

I like the idea of moving away from just string - based parsing, but we should then do it for the endpoint as well.