stacklok / toolhive / 24349073778

66%

Build Type

push

github

Committed by web-flow

Commit Message

Rename remoteURL and externalURL JSON tags to standard camelCase (#4775)

* Rename remoteURL and externalURL JSON tags to camelCase

The JSON tags on MCPRemoteProxySpec.RemoteURL and
MCPRemoteProxyStatus.ExternalURL used all-caps URL which is
inconsistent with the rest of the codebase (issuerUrl, jwksUrl,
tokenUrl, endpointUrl, resourceUrl) and Kubernetes API conventions.

Fixes #4581

- Change json:"remoteURL" to json:"remoteUrl" on MCPRemoteProxySpec
  and MCPServerEntrySpec
- Change json:"externalURL" to json:"externalUrl" on
  MCPRemoteProxyStatus
- Regenerate CRD manifests and API reference docs
- Update all example YAMLs, architecture docs, error messages, and
  test assertions to use the new key names

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Revert remoteURL in RunConfig docs to match Go field name

The architecture docs describing RunConfig fields use lowerCamelCase
Go field names (not CRD JSON tags). The RunConfig struct field is
RemoteURL (Go) / remote_url (JSON), so the doc convention is
remoteURL, not remoteUrl. The previous commit incorrectly renamed
these along with the CRD JSON tag changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

2 of 2 new or added lines in 2 files covered. (100.0%)

12 existing lines in 4 files now uncovered.

56972 of 86851 relevant lines covered (65.6%)

63.01 hits per line