Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

Author: uglide

src/main/java/redis/clients/jedis/RedisClient.java

@@ -10,6 +10,30 @@
 import redis.clients.jedis.util.JedisURIHelper;
 import redis.clients.jedis.util.Pool;
 
+/**
+ * {@code RedisClient} is the recommended client for connecting to standalone Redis deployments.
+ * <p>
+ * This class provides a modern, unified interface for interacting with Redis, supporting
+ * connection pooling, authentication, and configuration via a fluent builder API.
+ * </p>
+ * <p>
+ * {@code RedisClient} supersedes the deprecated {@link JedisPooled} and {@link UnifiedJedis}
+ * classes, offering improved usability and extensibility. For new applications, use
+ * {@code RedisClient} instead of the older classes.
+ * </p>
+ * <p>
+ * Example usage:
+ * <pre>{@code
+ * RedisClient client = RedisClient.builder()
+ *     .host("localhost")
+ *     .port(6379)
+ *     .build();
+ * }</pre>
+ * </p>
+ * <p>
+ * For advanced configuration, see the {@link RedisClient.Builder} class.
+ * </p>
+ */
 public class RedisClient extends UnifiedJedis {
 
   public RedisClient() {
@@ -56,7 +80,7 @@ private RedisClient(CommandExecutor commandExecutor, ConnectionProvider connecti
    * Obtain an instance via {@link #builder()}.
    * </p>
    */
-  static public class Builder extends StandaloneClientBuilder<RedisClient> {
+  public static class Builder extends StandaloneClientBuilder<RedisClient> {
 
     @Override
     protected RedisClient createClient() {

src/main/java/redis/clients/jedis/RedisClusterClient.java

@@ -13,6 +13,32 @@
 import redis.clients.jedis.providers.ConnectionProvider;
 import redis.clients.jedis.util.JedisClusterCRC16;
 
+/**
+ * RedisClusterClient provides a high-level, unified interface for interacting with a Redis Cluster.
+ * <p>
+ * This class is intended as a modern replacement for the deprecated {@code JedisCluster} class.
+ * It supports all cluster operations and is designed to work seamlessly with the {@link UnifiedJedis}
+ * API, allowing for consistent usage patterns across standalone, sentinel, and cluster deployments.
+ * <p>
+ * <b>Usage:</b>
+ * <pre>{@code
+ * Set<HostAndPort> clusterNodes = new HashSet<>();
+ * clusterNodes.add(new HostAndPort("127.0.0.1", 7000));
+ * RedisClusterClient client = new RedisClusterClient(clusterNodes);
+ * client.set("key", "value");
+ * String value = client.get("key");
+ * }</pre>
+ * <p>
+ * <b>Migration:</b>
+ * Users of {@code JedisCluster} are encouraged to migrate to this class for improved API consistency,
+ * better resource management, and enhanced support for future Redis features.
+ * <p>
+ * <b>Thread-safety:</b>
+ * This client is thread-safe and can be shared across multiple threads.
+ * <p>
+ * <b>Configuration:</b>
+ * Various constructors allow for flexible configuration, including authentication and custom timeouts.
+ */
 public class RedisClusterClient extends UnifiedJedis {
 
   public static final String INIT_NO_ERROR_PROPERTY = "jedis.cluster.initNoError";

src/main/java/redis/clients/jedis/RedisSentinelClient.java

@@ -7,8 +7,32 @@
 import redis.clients.jedis.providers.ConnectionProvider;
 import redis.clients.jedis.providers.SentineledConnectionProvider;
 
+/**
+ * A high-level client for interacting with Redis Sentinel-managed Redis deployments.
+ * <p>
+ * {@code RedisSentinelClient} provides robust support for automatic master failover,
+ * connection management, and command execution in environments where Redis Sentinel
+ * is used to monitor and manage Redis servers.
+ * </p>
+ * <p>
+ * Usage:
+ * <pre>
+ *   RedisSentinelClient client = RedisSentinelClient.builder()
+ *       .sentinel("localhost", 26379)
+ *       .masterName("mymaster")
+ *       .build();
+ * </pre>
+ * </p>
+ * <p>
+ * <b>Relationship to {@code JedisSentineled}:</b>
+ * <ul>
+ *   <li>{@code RedisSentinelClient} is the recommended replacement for the deprecated {@code JedisSentineled} class.</li>
+ *   <li>It offers improved API consistency, better failover handling, and a fluent builder for configuration.</li>
+ *   <li>Use {@code RedisSentinelClient} for new codebases and when migrating from {@code JedisSentineled}.</li>
+ * </ul>
+ * </p>
+ */
 public class RedisSentinelClient extends UnifiedJedis {
-
   private RedisSentinelClient(CommandExecutor commandExecutor,
       ConnectionProvider connectionProvider, CommandObjects commandObjects,
       RedisProtocol redisProtocol, Cache cache) {
@@ -21,7 +45,7 @@ private RedisSentinelClient(CommandExecutor commandExecutor,
    * Obtain an instance via {@link #builder()}.
    * </p>
    */
-  static public class Builder extends SentinelClientBuilder<RedisSentinelClient> {
+  public static class Builder extends SentinelClientBuilder<RedisSentinelClient> {
 
     @Override
     protected RedisSentinelClient createClient() {