Watch is production-ready. The watch feature flag adds complexity with no benefit: fallback stubs return unimplemented, all real deployments enable it, and optional Arc<WatchRegistry> spreads null-checks across the codebase.
Scope:
- Remove
watch feature from all Cargo.toml files
- Collapse
Option<Arc<WatchRegistry>> → Arc<WatchRegistry> (always present)
- Delete
#[cfg(not(feature = "watch"))] stubs in grpc_raft_service.rs and embedded.rs
- Remove
enabled: bool from WatchConfig
- Remove
#[cfg(feature = "watch")] gates from all test files
- Update CHANGELOG
Dependency: Merge after #300.
Watch is production-ready. The
watchfeature flag adds complexity with no benefit: fallback stubs returnunimplemented, all real deployments enable it, and optionalArc<WatchRegistry>spreads null-checks across the codebase.Scope:
watchfeature from allCargo.tomlfilesOption<Arc<WatchRegistry>>→Arc<WatchRegistry>(always present)#[cfg(not(feature = "watch"))]stubs ingrpc_raft_service.rsandembedded.rsenabled: boolfromWatchConfig#[cfg(feature = "watch")]gates from all test filesDependency: Merge after #300.