docs: reorganize documentation structure for better navigation (#362)

* fix: resolve compilation warnings for secure-storage feature

- Add conditional imports for anyhow::{anyhow, Context} only when secure-storage is enabled
- Mark unused variables with _ when secure-storage is disabled
- Conditionally compile SERVICE_NAME constant only when needed
- Fix error handling in keyring operations

* docs: add missing examples pages to SUMMARY.md

- Add cloud/examples.md to Cloud section
- Add enterprise/examples.md to Enterprise section
- Fixes broken links in overview pages

* docs: reorganize documentation structure for better navigation

- Group commands by functionality instead of flat lists
- Separate Cloud and Enterprise sections with logical subsections
- Move common features to dedicated section
- Add comprehensive examples in getting-started
- Add JMESPath queries and secure storage documentation
- Fix broken internal links throughout
- Temporarily disable mdbook-lint during reorganization

Addresses issue #361

Author: joshrotenberg

crates/redisctl/src/credential_store.rs

@@ -6,13 +6,16 @@
 //! - Plaintext storage (fallback)
 //! - Environment variable override
 
-use anyhow::{Context, Result};
+use anyhow::Result;
+#[cfg(feature = "secure-storage")]
+use anyhow::{Context, anyhow};
 use std::env;
 
 /// Prefix that indicates a value should be retrieved from the keyring
 const KEYRING_PREFIX: &str = "keyring:";
 
 /// Service name for keyring entries
+#[cfg(feature = "secure-storage")]
 const SERVICE_NAME: &str = "redisctl";
 
 /// Storage backend for credentials
@@ -97,6 +100,7 @@ impl CredentialStore {
         #[cfg(not(feature = "secure-storage"))]
         {
             // Without secure-storage feature, always use plaintext
+            let _ = key; // Not used without secure-storage
             Ok(value.to_string())
         }
     }
@@ -150,14 +154,17 @@ impl CredentialStore {
                     match entry.delete_credential() {
                         Ok(()) => Ok(()),
                         Err(keyring::Error::NoEntry) => Ok(()), // Already deleted
-                        Err(e) => Err(e).context("Failed to delete credential from keyring"),
+                        Err(e) => {
+                            Err(anyhow!(e)).context("Failed to delete credential from keyring")
+                        }
                     }
                 }
                 CredentialStorage::Plaintext => Ok(()), // Nothing to delete for plaintext
             }
         }
         #[cfg(not(feature = "secure-storage"))]
         {
+            let _ = key; // Not used without secure-storage
             Ok(()) // Nothing to delete for plaintext
         }
     }

docs/book.toml

@@ -15,6 +15,7 @@ edit-url-template = "https://github.com/joshrotenberg/redisctl/edit/main/docs/{p
 enable = true
 level = 1
 
-[preprocessor.lint]
-command = "mdbook-lint"
-renderer = ["html"]
+# Temporarily disabled for reorganization
+# [preprocessor.lint]
+# command = "mdbook-lint"
+# renderer = ["html"]

docs/src/SUMMARY.md

@@ -5,87 +5,106 @@
 # Getting Started
 
 - [Installation](./getting-started/installation.md)
+- [Docker Quick Start](./getting-started/docker.md)
 - [Configuration](./getting-started/configuration.md)
-- [Quick Start](./getting-started/quickstart.md)
 - [Authentication](./getting-started/authentication.md)
+- [Examples](./getting-started/examples.md)
 - [Shell Completions](./getting-started/shell-completions.md)
-- [Docker Development](./getting-started/docker.md)
+- [Quick Start Guide](./getting-started/quickstart.md)
 
-# Redis Cloud Commands
+# Redis Cloud
 
 - [Overview](./cloud/overview.md)
-- [Subscriptions](./cloud/subscriptions.md)
-- [Databases](./cloud/databases.md)
-- [Network Connectivity](./cloud/connectivity.md)
-- [Access Control](./cloud/acl.md)
-- [Users](./cloud/users.md)
-- [Provider Accounts](./cloud/provider-accounts.md)
-- [Tasks](./cloud/tasks.md)
-- [Workflows](./cloud/workflows.md)
-- [Raw API Access](./cloud/api-access.md)
-
-# Redis Enterprise Commands
+- [Core Resources]()
+  - [Subscriptions](./cloud/core-resources/subscriptions.md)
+  - [Databases](./cloud/core-resources/databases.md)
+- [Access Control]()
+  - [Users](./cloud/access-control/users.md)
+  - [ACLs](./cloud/access-control/acl.md)
+- [Networking]()
+  - [Connectivity (VPC, PSC, TGW)](./cloud/networking/connectivity.md)
+- [Operations]()
+  - [Tasks](./cloud/operations/tasks.md)
+  - [Workflows](./cloud/operations/workflows.md)
+- [Examples](./cloud/examples.md)
+- [Human-Friendly Commands](./cloud/human-commands.md)
+- [All Commands](./cloud/commands.md)
+
+# Redis Enterprise
 
 - [Overview](./enterprise/overview.md)
-- [Cluster](./enterprise/cluster.md)
-- [Bootstrap](./enterprise/bootstrap.md)
-- [CM Settings](./enterprise/cm-settings.md)
-- [Databases](./enterprise/databases.md)
-- [Database Groups](./enterprise/bdb-groups.md)
-- [Shards](./enterprise/shards.md)
-- [Nodes](./enterprise/nodes.md)
-- [Proxy Management](./enterprise/proxy.md)
-- [Users & RBAC](./enterprise/users.md)
-- [Statistics](./enterprise/stats.md)
-- [Usage Reports](./enterprise/usage-report.md)
-- [Modules](./enterprise/modules.md)
-- [Logs](./enterprise/logs.md)
-- [Database Migration](./enterprise/migration.md)
-- [Active-Active (CRDB)](./enterprise/crdb.md)
-- [CRDB Tasks](./enterprise/crdb-tasks.md)
-- [Actions (Tasks)](./enterprise/actions.md)
-- [Alerts](./enterprise/alerts.md)
-- [Debug Info](./enterprise/debuginfo.md)
-- [Diagnostics](./enterprise/diagnostics.md)
-- [Endpoints](./enterprise/endpoints.md)
-- [Job Scheduler](./enterprise/job-scheduler.md)
-- [JSON Schema](./enterprise/jsonschema.md)
-- [LDAP Integration](./enterprise/ldap.md)
-- [License Management](./enterprise/license.md)
-- [OCSP Validation](./enterprise/ocsp.md)
-- [Service Management](./enterprise/services.md)
-- [DNS Suffixes](./enterprise/suffix.md)
-- [Workflows](./enterprise/workflows.md)
-- [Raw API Access](./enterprise/api-access.md)
-
-# Core Features
-
-- [Async Operations](./features/async-operations.md)
-- [Output Formats](./features/output-formats.md)
-- [Profile Management](./features/profiles.md)
-- [Workflows](./features/workflows.md)
+- [Core Resources]()
+  - [Cluster](./enterprise/core-resources/cluster.md)
+  - [Databases](./enterprise/core-resources/databases.md)
+  - [Nodes](./enterprise/core-resources/nodes.md)
+  - [Shards](./enterprise/core-resources/shards.md)
+  - [Modules](./enterprise/core-resources/modules.md)
+  - [Database Groups](./enterprise/core-resources/bdb-groups.md)
+- [Access Control]()
+  - [Users & Roles](./enterprise/access-control/users.md)
+  - [LDAP Integration](./enterprise/access-control/ldap.md)
+- [Active-Active]()
+  - [CRDB Operations](./enterprise/active-active/crdb.md)
+  - [CRDB Tasks](./enterprise/active-active/crdb-tasks.md)
+- [Monitoring]()
+  - [Statistics](./enterprise/monitoring/stats.md)
+  - [Logs](./enterprise/monitoring/logs.md)
+  - [Alerts](./enterprise/monitoring/alerts.md)
+  - [Usage Reports](./enterprise/monitoring/usage-report.md)
+- [Operations]()
+  - [Support Package](./enterprise/operations/support-package.md)
+  - [Debug Info](./enterprise/operations/debuginfo.md)
+  - [Diagnostics](./enterprise/operations/diagnostics.md)
+  - [License Management](./enterprise/operations/license.md)
+  - [Migrations](./enterprise/operations/migration.md)
+- [Advanced]()
+  - [Actions](./enterprise/advanced/actions.md)
+  - [Workflows](./enterprise/advanced/workflows.md)
+  - [Job Scheduler](./enterprise/advanced/job-scheduler.md)
+  - [Bootstrap](./enterprise/advanced/bootstrap.md)
+  - [Proxies](./enterprise/advanced/proxy.md)
+  - [Endpoints](./enterprise/advanced/endpoints.md)
+  - [Services](./enterprise/advanced/services.md)
+  - [CM Settings](./enterprise/advanced/cm-settings.md)
+  - [OCSP](./enterprise/advanced/ocsp.md)
+  - [DNS Suffix](./enterprise/advanced/suffix.md)
+  - [JSON Schema](./enterprise/advanced/jsonschema.md)
+- [Examples](./enterprise/examples.md)
+- [Human-Friendly Commands](./enterprise/human-commands.md)
+- [All Commands](./enterprise/commands.md)
+- [API Access](./enterprise/api-access.md)
+
+# Common Features
+
+- [Output Formats](./common-features/output-formats.md)
+- [JMESPath Queries](./common-features/jmespath-queries.md)
+- [Async Operations](./common-features/async-operations.md)
+- [Raw API Access](./common-features/raw-api.md)
+- [Secure Storage](./common-features/secure-storage.md)
 
 # Tutorials
 
+- [CI/CD Integration](./tutorials/cicd.md)
 - [Common Recipes](./tutorials/common-recipes.md)
-- [Managing Production Databases](./tutorials/production-databases.md)
-- [Setting Up Monitoring](./tutorials/monitoring.md)
 - [Disaster Recovery](./tutorials/disaster-recovery.md)
+- [Monitoring Setup](./tutorials/monitoring.md)
 - [Network Security](./tutorials/network-security.md)
-- [CI/CD Integration](./tutorials/cicd.md)
+- [Production Databases](./tutorials/production-databases.md)
 
 # Reference
 
 - [Environment Variables](./reference/environment-variables.md)
 - [Configuration File](./reference/config-file.md)
-- [Security Best Practices](./reference/security.md)
-- [Troubleshooting](./reference/troubleshooting.md)
+- [Security](./reference/security.md)
 - [Best Practices](./reference/best-practices.md)
-- [API Reference](./reference/api.md)
+- [Troubleshooting](./reference/troubleshooting.md)
+
+# API Reference
+
+- [API Overview](./api-reference/api.md)
 
-# Developer Guide
+# Developer
 
 - [Architecture](./developer/architecture.md)
-- [Using the Libraries](./developer/libraries.md)
-- [Contributing](./developer/contributing.md)
-- [Adding Commands](./developer/adding-commands.md)
+- [Libraries](./developer/libraries.md)
+- [Contributing](./developer/contributing.md)