tree d764762ddbbf381c321095a9a7f23eec9fbed13f
parent 1b00c2c3c7819abe09898e6fe0da2e7f1fb65944
author Jacek Centkowski <geminica.programs@gmail.com> 1779432095 +0200
committer Jacek Centkowski <geminica.programs@gmail.com> 1779432095 +0200

Preserve upstream provider detail in OpenRouter errors

OpenRouter wraps the real upstream failure under `error.metadata`
(`provider_name`, `raw`). The previous `extractErrorMessage` only
took `error.message`, leaving users with the generic
`Provider returned error` whenever the upstream model returned a
non-2xx (rate limits, quota exhaustion, model-side failures).

Append the metadata when present so the surfaced message looks
like:

  Provider returned error: provider=Venice, qwen/qwen3-coder:free
  is temporarily rate-limited upstream. Please retry shortly, ...

Drop the `[${code}]` prefix from the formatted message: the HTTP
status is already logged by `AiHttpClientImpl` and exposed
separately on `AiCodeReviewOutput.error.status_code`.

Change-Id: I7e4075a209d1062bc2149b96f8c2c67e3c820738