• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tari-project / tari / 17394556887
62%

Build:
DEFAULT BRANCH: development
Ran 02 Sep 2025 07:34AM UTC
Jobs 1
Files 699
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

02 Sep 2025 05:51AM UTC coverage: 61.161% (+0.08%) from 61.079%
17394556887

push

github

web-flow
feat!: remove comms from wallet (#7432)

Description
---
- Removed comms from wallet.
- Wired in missing information in the wallet's base node connectivity.

Motivation and Context
---
Comms is not used in the wallet anymore.

How Has This Been Tested?
---
- System-level testing on mainnet sending transactions between two
wallets.

What process can a PR reviewer use to test or verify this change?
---

<!-- Checklist -->
<!-- 1. Is the title of your PR in the form that would make nice release
notes? The title, excluding the conventional commit
tag, will be included exactly as is in the CHANGELOG, so please think
about it carefully. -->


Breaking Changes
---

- [ ] None
- [ ] Requires data directory on base node to be deleted
- [ ] Requires hard fork
- [x] Other - Please specify

<!-- Does this include a breaking change? If so, include this line as a
footer -->
BREAKING CHANGE: 
- Wallet FFI
  - SAF and DHT callbacks have been removed.
- A new type, `TariWalletDbConfig`, now replaces `TariCommsConfig` -
naming for `comms_config_create` and `comms_config_destroy` have been
retained.
  - These functions have been removed:
    - `wallet_get_seed_peers`
  - These functions have changes to their interfaces:
    - `wallet_create`
    - `wallet_get_last_version`
    - `wallet_get_last_network`
    - `comms_config_destroy`
    - `comms_config_create`
- gRPC
- `ListConnectedPeers` now returns a new response,
`ListConnectedHttpPeersResponse`, instead of
`ListConnectedPeersResponse`. The connected base node's public_key and
node_id are also returned for convenience, along with the connection
URL, latency and online status.
  - `GetNetworkStatus` now includes a degraded status as well
- Cli method `discover_peer` has been removed
- All previous wallet comms config settings were deprecated (no-op), but
config files were kept intact, with notices about deprecated settings.




<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by C... (continued)

0 of 43 new or added lines in 4 files covered. (0.0%)

8 existing lines in 4 files now uncovered.

72696 of 118861 relevant lines covered (61.16%)

229440.76 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
0.0
0.0% base_layer/wallet/src/transaction_service/service.rs
5
0.0
0.0% base_layer/wallet/src/config.rs
6
0.0
0.0% base_layer/wallet/src/wallet.rs
31
0.0
0.0% base_layer/wallet/src/connectivity_service/handle.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
0.0
0.0% base_layer/wallet/src/connectivity_service/handle.rs
1
50.19
-0.04% base_layer/wallet/src/transaction_service/storage/sqlite_db.rs
2
60.48
0.13% comms/dht/src/connectivity/mod.rs
4
55.7
-1.69% base_layer/transaction_key_manager/src/storage/sqlite_db/mod.rs
Jobs
ID Job ID Ran Files Coverage
1 17394556887.1 02 Sep 2025 07:34AM UTC 699
61.16
GitHub Action Run
Source Files on build 17394556887
  • Tree
  • List 699
  • Changed 19
  • Source Changed 9
  • Coverage Changed 19
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #17394556887
  • cf89f147 on github
  • Prev Build on development (#17377200040)
  • Next Build on development (#17402739223)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc