Clarify fresh cohort scope and coordinated request timeouts

Author: findolor

crates/common/ARCHITECTURE.md

@@ -160,8 +160,9 @@ Target gating is used extensively:
   enforced. Cold network loops catch up, analyze their applied data and
   establish readiness; the cohort does not run an additional `ANALYZE`.
   Import/manifest failures and unsupported coordination fall back to per-network
-  provisioning; stale-target refresh retains its existing behavior. Remote waits
-  are bounded to two minutes per request and scheduler stop cancels
+  provisioning; stale-target refresh retains its existing behavior. Coordinated
+  manifest and seed requests are bounded to two minutes each; per-network runner
+  requests keep their existing behavior. Scheduler stop cancels
   coordination/import cleanup.
 - `LocalDbState` encapsulates the local DB handle, scheduler, `SyncReadiness`
   (tracks which chains have completed a sync cycle), and the set of configured

crates/common/src/raindex_client/local_db/pipeline/runner/bootstrap.rs

@@ -33,8 +33,9 @@ pub(super) async fn has_persisted_targets<DB: LocalDbQueryExecutor + ?Sized>(
     Ok(!rows.is_empty())
 }
 
-/// The caller holds cross-tab bootstrap ownership through preflight and commit.
-/// Existing targets, including stale-target refresh, retain their runner policy.
+/// The caller starts with an empty target_watermarks table and holds cross-tab
+/// bootstrap ownership through preflight and commit. Per-target watermark
+/// checks defensively skip targets committed after that global preflight.
 pub(super) async fn provision_missing_targets<DB, F>(
     db: &DB,
     settings: &ParsedRunnerSettings,
@@ -363,7 +364,7 @@ local-db-sync:
     }
 
     #[wasm_bindgen_test]
-    async fn warm_targets_are_not_downloaded_or_imported() {
+    async fn defensive_preflight_skips_persisted_targets() {
         let (settings, _) = settings_and_manifest();
         let manifests: ManifestFetcher =
             Arc::new(|_| panic!("warm targets must not fetch manifests"));
@@ -380,7 +381,7 @@ local-db-sync:
     }
 
     #[wasm_bindgen_test]
-    async fn adding_a_missing_network_keeps_existing_target_data() {
+    async fn defensive_preflight_filters_already_persisted_target() {
         let (settings, manifests) = settings_and_manifest();
         let manifests: ManifestFetcher = Arc::new(move |raindexes| {
             assert_eq!(raindexes.len(), 1);