fix(hindsight): correct client constructor args and add configurable api_url

[nils010485]

Summary

Fixes the Hindsight cloud client constructor and adds support for self-hosted instances.

Bug: broken constructor arguments

_make_client() passed api_key as the first positional argument to Hindsight(), but the constructor signature is Hindsight(base_url, api_key=None, timeout=300.0). Cloud mode never worked.

# Before (broken) — api_key lands in base_url parameter
return Hindsight(api_key=self._api_key, timeout=30.0)

# After (fixed) — named parameters, correct order
return Hindsight(base_url=base_url, api_key=self._api_key, timeout=120.0)

Feature: configurable api_url for self-hosted instances

Adds api_url configuration field so users running Hindsight via self-hosted Docker can point to their local instance instead of the default cloud endpoint.

Resolution order: config.json → HINDSIGHT_API_URL env var → default (https://api.hindsight.vectorize.io)

Changes

plugins/memory/hindsight/__init__.py (9 insertions, 1 deletion):

• _load_config(): add apiUrl from HINDSIGHT_API_URL env var
• get_config_schema(): add api_url field with env var hint and default
• _make_client(): resolve base_url from config → env → default; pass as named arg; timeout 120s

Context

Related to PR #4762 (closed by maintainer). The sequential dispatch fix from that PR was already merged separately as PR #4803. This PR covers the remaining fix: the constructor argument order and self-hosted URL support.

Testing

• ✅ 124 memory-related tests pass (0 failures)
• ✅ File compiles without errors
• ✅ Tested locally with self-hosted Docker Hindsight (localhost:8888) — retain/recall/reflect all functional

[nils010485]

Reviewing with external agents before finalizing