Skip to content

feat(antd-cpp): gRPC WalletService parity with REST#189

Merged
Nic-dorman merged 1 commit into
mainfrom
nic/v2-286-antd-cpp-grpc-walletservice
May 28, 2026
Merged

feat(antd-cpp): gRPC WalletService parity with REST#189
Nic-dorman merged 1 commit into
mainfrom
nic/v2-286-antd-cpp-grpc-walletservice

Conversation

@Nic-dorman
Copy link
Copy Markdown
Collaborator

Summary

Adds 3 new methods on antd::GrpcClient mirroring REST antd::Client:

  • WalletAddress wallet_address()
  • WalletBalance wallet_balance()
  • bool wallet_approve()

Depends on #154.

Implementation notes

  • New antd::v1::WalletService::Stub on the existing pimpl, alongside the existing health/data/chunk/file stubs.
  • A missing daemon wallet emits gRPC FAILED_PRECONDITION; the existing check_status helper maps it to antd::PaymentError.

Tests

tests/test_grpc_wallet.cpp (new):

  • Spins up a real grpc++ server on 127.0.0.1:0 with MockWalletService + UnconfiguredWalletService implementations.
  • 4 tests: address, balance, approve, plus the unconfigured-wallet PaymentError path.
  • Gated on -DANTD_BUILD_GRPC=ON, same as the V2-284 cpp PR's antd_grpc_external_signer_tests pattern.

Gates passed on dev2

cmake -B build -DANTD_BUILD_GRPC=ON -DProtobuf_PROTOC_EXECUTABLE=/usr/bin/protoc
cmake --build build
ctest --test-dir build
  • 4/4 tests passing (antd_tests, antd_async_tests, antd_grpc_tests isolated, antd_grpc_wallet_tests V2-286)

Supersedes #167 (force-push after daemon merge auto-closed the original PR).

@Nic-dorman Nic-dorman force-pushed the nic/v2-286-antd-cpp-grpc-walletservice branch from 5c2dd94 to 4153747 Compare May 28, 2026 18:08
@Nic-dorman Nic-dorman merged commit 0716de6 into main May 28, 2026
Nic-dorman added a commit that referenced this pull request May 28, 2026
Closes the gRPC parity gap with REST: external-signer (UploadService +
chunks PrepareChunk/FinalizeChunk) and WalletService are now reachable
over gRPC, plus stream-download stubs return honest UNIMPLEMENTED/501
instead of pretending. All 11 gRPC-capable SDKs follow with matching
fan-out.

## Additive (antd daemon)

- gRPC UploadService + chunks prepare/finalize — external-signer two-phase
  upload + single-chunk publish over gRPC, mirroring REST 1:1 (#140)
- gRPC WalletService — GetAddress / GetBalance / Approve, parity with
  REST /v1/wallet/* (#154)
- /v1/data/prepare now honors visibility for public DataMap chunk
  bundling via data_prepare_upload_with_visibility (#138)

## Fixes (antd daemon)

- Stream-download stubs (DataServiceStub.StreamDownload, REST
  /v1/data/stream/*) now return UNIMPLEMENTED / 501 honestly rather
  than empty success (#155)

## SDK fan-out — external-signer (gRPC prepare/finalize)

All 11 gRPC-capable SDKs replace their stub raises with real gRPC calls
to the new UploadService + ChunkService PrepareChunk/FinalizeChunk RPCs:

- antd-rust (#169), antd-go (#170), antd-py (#171), antd-java (#172),
  antd-kotlin (#173), antd-csharp (#174), antd-ruby (#175),
  antd-dart (#176), antd-swift (#177), antd-cpp (#178),
  antd-elixir (#179)

## SDK fan-out — WalletService

Same 11 SDKs gain walletAddress / walletBalance / walletApprove over gRPC:

- antd-rust (#180), antd-go (#181), antd-py (#182), antd-java (#183),
  antd-kotlin (#184), antd-csharp (#185), antd-ruby (#186),
  antd-dart (#187), antd-swift (#188), antd-cpp (#189),
  antd-elixir (#190)

## SDK examples + build fixes

- antd-ruby: payForQuotes tuple args as Hash for eth-0.5.13
  compatibility (#131)
- antd-go + ant-dev: add 03-chunks + 06-private-data examples;
  renumber 03-files to 04-files (#132)
- antd-java + antd-cpp: close async client surface gaps (#133)
- ant-dev: pull antd[rest,grpc] extras so 08_grpc example runs out
  of the box (#134)
- antd-js + antd-py + antd-go READMEs: surface external-signer
  methods + 07 example (#135)
- antd-elixir: 07_external_signer example via cast shell-out (#136)
- antd-swift: 07_external_signer example + fix FinalizeUploadDTO
  optional address (#137)
- antd-php: empty tx_hashes serializes as JSON object on finalize
  (#139)

## FFI

- ant-ffi refresh against current ant-core API + Swift xcframework
  build pipeline (#150)
- Android AAR build pipeline (#156)

## Infra

- CI: auto-tag Go submodules on umbrella release tag push (#129)
- Deploy: publish multi-arch withautonomi/antd image from ant-sdk
  release (#130)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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