Skip to content

feat(antd-java): expose new HealthStatus diagnostic fields#41

Merged
Nic-dorman merged 1 commit into
mainfrom
feat/antd-java-health-enrichment
May 6, 2026
Merged

feat(antd-java): expose new HealthStatus diagnostic fields#41
Nic-dorman merged 1 commit into
mainfrom
feat/antd-java-health-enrichment

Conversation

@Nic-dorman
Copy link
Copy Markdown
Collaborator

Summary

Mirrors the daemon-side enrichment shipped in #38: HealthStatus now carries version, evm_network/evmNetwork, uptime_seconds/uptimeSeconds, build_commit/buildCommit, payment_token_address/paymentTokenAddress, payment_vault_address/paymentVaultAddress. All default to "" / 0 so the type stays constructable when talking to a pre-0.4.0 daemon.

Test plan

See commit message for the per-language test command and pass count.

Part of #37.

🤖 Generated with Claude Code

Mirrors antd-go v0.5.0 / antd-py: HealthStatus now carries version,
evmNetwork, uptimeSeconds, buildCommit, paymentTokenAddress, and
paymentVaultAddress. A backward-compatible 2-arg constructor delegates
to the canonical one with empty/zero defaults so pre-0.4.0 callers
keep working.

REST sync + async share AntdClient::parseHealthStatus to map a JSON
response. gRPC routes through GrpcAntdClient::healthStatusFromGrpc to
pull from the regenerated HealthCheckResponse getters. Proto stubs
are regenerated automatically by protobuf-gradle-plugin during the
gradle build.

Mock fixtures in both AntdClientTest and GrpcAntdClientTest populate
all 6 new fields; existing testHealth assertions extended; new
testHealthBackwardCompatConstructor exercises the 2-arg constructor
defaults.

Part of #37.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Nic-dorman Nic-dorman merged commit 3c5fbdd into main May 6, 2026
@Nic-dorman Nic-dorman deleted the feat/antd-java-health-enrichment branch May 6, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant