Skip to content

feat(antd-kotlin): expose new HealthStatus diagnostic fields#43

Merged
Nic-dorman merged 2 commits into
mainfrom
feat/antd-kotlin-health-enrichment
May 6, 2026
Merged

feat(antd-kotlin): expose new HealthStatus diagnostic fields#43
Nic-dorman merged 2 commits into
mainfrom
feat/antd-kotlin-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

Nic-dorman and others added 2 commits May 5, 2026 10:10
Mirrors antd-go v0.5.0 / antd-py: HealthStatus now carries version,
evmNetwork, uptimeSeconds, buildCommit, paymentTokenAddress, and
paymentVaultAddress. All have data-class default values ("" / 0) so
the type stays constructable with just (ok, network) and pre-0.4.0
daemons keep deserializing.

REST routes through HealthResponseDto.toHealthStatus() — the DTO
gains six nullable serial-named properties so kotlinx.serialization
tolerates either old or new daemon shapes. gRPC pulls each field from
the regenerated HealthCheckResponse proto. Proto stubs regenerate
automatically via the protobuf-gradle-plugin during gradle build.

RestClientTest mock fixture extended to populate all 6 fields;
existing assertion expanded; new HealthStatus-defaults regression
test exercises the (ok, network)-only constructor.

Part of #37.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Nic-dorman Nic-dorman merged commit 881db7f into main May 6, 2026
@Nic-dorman Nic-dorman deleted the feat/antd-kotlin-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