docs(dashmate): correct state sync retries=0 semantics to unlimited

Tenderdash's SyncAny only returns errNoSnapshots (the block sync fallback trigger) when retries > 0, so retries=0 repeats snapshot discovery indefinitely rather than disabling retries - Tenderdash's own sample config comment is wrong about this. Describe 0 as retry-indefinitely in the schema, rendered template, docs, and test. Also annotate the grpc-concurrency block: Tenderdash drops the concurrency map for the routed transport (NewRoutedClientWithAddr passes address and transport only), so the entries, including the pre-existing check_tx one, are declarative until that propagates.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: PastaPastaPasta

packages/dashmate/docs/config/tenderdash.md

@@ -87,7 +87,7 @@ State sync bootstraps a fresh node from a recent state snapshot fetched from pee
 | Option | Description | Default | Example |
 |--------|-------------|---------|---------|
 | `platform.drive.tenderdash.stateSync.enabled` | Bootstrap a fresh node from a snapshot | `true` | `false` |
-| `platform.drive.tenderdash.stateSync.retries` | Retries before falling back to block sync, `0` disables retries | `3` | `5` |
+| `platform.drive.tenderdash.stateSync.retries` | Retries before falling back to block sync, `0` retries indefinitely (never falls back) | `3` | `5` |
 | `platform.drive.tenderdash.stateSync.chunkRequestTimeout` | Timeout before re-requesting a snapshot chunk, at least `5s` | `15s` | `30s` |
 | `platform.drive.tenderdash.stateSync.fetchersCount` | Concurrent chunk fetchers, 1 to 64 | `4` | `8` |
 

packages/dashmate/src/config/configJsonSchema.js

@@ -1377,7 +1377,7 @@ export default {
                       type: 'integer',
                       minimum: 0,
                       description: 'How many times to retry state sync before falling back to block sync.'
-                        + ' 0 disables retries',
+                        + ' 0 means retry indefinitely and never fall back',
                     },
                     chunkRequestTimeout: {
                       description: 'Timeout before re-requesting a snapshot chunk. Tenderdash requires at least 5s',

packages/dashmate/templates/platform/drive/tenderdash/config.toml.dot

@@ -99,6 +99,13 @@ grpc-concurrency = [
   { "check_tx" = {{= it.platform.drive.tenderdash.mempool.maxConcurrentCheckTx }} },
   # Snapshot serving: discovery is one request per peer, chunk downloads run
   # several concurrent fetchers per syncing peer.
+  #
+  # NOTE: Tenderdash currently applies grpc-concurrency only to a direct
+  # `transport = "grpc"` client; the routed transport used here drops the map
+  # when it builds the nested per-method clients (NewRoutedClientWithAddr
+  # passes address and transport only), so these limits — including the
+  # pre-existing check_tx one — are declarative until Tenderdash propagates
+  # them to routed clients.
   { "list_snapshots" = 10 },
   { "load_snapshot_chunk" = 100 },
 ]
@@ -439,8 +446,9 @@ rpc-servers = ""
 discovery-time = "15s"
 
 # The number of times to retry state sync. When retries are exhausted, the node falls back
-# to block sync. Set to 0 to disable retries. In the pessimistic case it takes at least
-# discovery-time * retries before falling back.
+# to block sync. 0 means retry indefinitely: the node keeps repeating snapshot discovery
+# and never falls back. In the pessimistic case it takes at least discovery-time * retries
+# before falling back.
 retries = {{= it.platform.drive.tenderdash.stateSync.retries }}
 
 # Temporary directory for state sync snapshot chunks, defaults to os.TempDir().

packages/dashmate/test/unit/config/stateSyncOptions.spec.js

@@ -46,7 +46,9 @@ describe('state sync options', () => {
       config = getBaseConfig();
     });
 
-    it('should accept retries of 0 (disable retries) but not negative', () => {
+    // Tenderdash treats 0 as an unlimited retry count: SyncAny only returns
+    // errNoSnapshots (the block sync fallback trigger) when retries > 0.
+    it('should accept retries of 0 (retry indefinitely) but not negative', () => {
       config.set('platform.drive.tenderdash.stateSync.retries', 0);
 
       expect(() => config.set('platform.drive.tenderdash.stateSync.retries', -1))