Remove 'target' directory from cache in CI workflows

Updated CI and publish GitHub Actions workflows to exclude the 'target' directory from cache paths, improving cache efficiency and avoiding potential issues with stale build artifacts. Also refined the version extraction regex in publish.yml for better accuracy.

Author: Tuntii

.github/workflows/ci.yml

@@ -34,7 +34,6 @@ jobs:
           path: |
             ~/.cargo/registry
             ~/.cargo/git
-            target
           key: ${{ runner.os }}-cargo-test-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: |
             ${{ runner.os }}-cargo-test-
@@ -70,7 +69,6 @@ jobs:
           path: |
             ~/.cargo/registry
             ~/.cargo/git
-            target
           key: ${{ runner.os }}-cargo-lint-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: |
             ${{ runner.os }}-cargo-lint-
@@ -104,7 +102,6 @@ jobs:
           path: |
             ~/.cargo/registry
             ~/.cargo/git
-            target
           key: ${{ runner.os }}-cargo-build-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: |
             ${{ runner.os }}-cargo-build-
@@ -141,7 +138,6 @@ jobs:
           path: |
             ~/.cargo/registry
             ~/.cargo/git
-            target
           key: ${{ runner.os }}-cargo-docs-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: |
             ${{ runner.os }}-cargo-docs-

.github/workflows/publish.yml

@@ -30,7 +30,6 @@ jobs:
           path: |
             ~/.cargo/registry
             ~/.cargo/git
-            target
           key: ${{ runner.os }}-cargo-publish-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: |
             ${{ runner.os }}-cargo-publish-
@@ -39,7 +38,7 @@ jobs:
         if: startsWith(github.ref, 'refs/tags/v')
         run: |
           TAG_VERSION=${GITHUB_REF#refs/tags/v}
-          CARGO_VERSION=$(grep '^version' Cargo.toml | head -1 | sed 's/.*"\(.*\)"/\1/')
+          CARGO_VERSION=$(grep '^\s*version\s*=' Cargo.toml | head -n 1 | sed 's/.*"\(.*\)"/\1/')
           if [ "$TAG_VERSION" != "$CARGO_VERSION" ]; then
             echo "Version mismatch: tag=$TAG_VERSION, Cargo.toml=$CARGO_VERSION"
             exit 1

crates/rustapi-extras/src/audit/event.rs

@@ -107,23 +107,18 @@ impl std::fmt::Display for AuditAction {
 }
 
 /// Severity level for audit events.
-#[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Serialize, Deserialize)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Serialize, Deserialize, Default)]
 #[serde(rename_all = "lowercase")]
 pub enum AuditSeverity {
     /// Informational - normal operations
+    #[default]
     Info,
     /// Warning - unusual but not critical
     Warning,
     /// Critical - security or compliance concern
     Critical,
 }
 
-impl Default for AuditSeverity {
-    fn default() -> Self {
-        Self::Info
-    }
-}
-
 /// Compliance-related information for GDPR/SOC2.
 #[derive(Debug, Clone, Default, Serialize, Deserialize)]
 pub struct ComplianceInfo {

crates/rustapi-extras/src/logging.rs

@@ -134,7 +134,7 @@ impl MiddlewareLayer for LoggingLayer {
             let request_id = req
                 .extensions()
                 .get::<String>()
-                .map(|s| s.clone())
+                .cloned()
                 .unwrap_or_else(|| "N/A".to_string());
 
             let start = Instant::now();

crates/rustapi-extras/src/otel/config.rs

@@ -21,9 +21,10 @@ pub enum OtelExporter {
 }
 
 /// Trace sampling strategy
-#[derive(Clone, Debug)]
+#[derive(Clone, Debug, Default)]
 pub enum TraceSampler {
     /// Always sample all traces
+    #[default]
     AlwaysOn,
     /// Never sample traces
     AlwaysOff,
@@ -33,12 +34,6 @@ pub enum TraceSampler {
     ParentBased,
 }
 
-impl Default for TraceSampler {
-    fn default() -> Self {
-        Self::AlwaysOn
-    }
-}
-
 /// OpenTelemetry configuration
 #[derive(Clone, Debug)]
 pub struct OtelConfig {

crates/rustapi-extras/src/otel/propagation.rs

@@ -159,7 +159,7 @@ pub fn extract_trace_context(request: &Request) -> TraceContext {
         .get(TRACEPARENT_HEADER)
         .and_then(|v| v.to_str().ok())
         .and_then(TraceContext::from_traceparent)
-        .unwrap_or_else(TraceContext::new);
+        .unwrap_or_default();
 
     // Extract tracestate if present
     if let Some(state) = headers.get(TRACESTATE_HEADER).and_then(|v| v.to_str().ok()) {

crates/rustapi-extras/src/retry.rs

@@ -171,29 +171,29 @@ impl MiddlewareLayer for RetryLayer {
                 let status = response.status().as_u16();
 
                 // Check if we should retry
-                if attempt < config.max_attempts
-                    && config.retryable_statuses.contains(&status)
-                    && next_req_opt.is_some()
-                {
-                    tracing::warn!(
-                        attempt = attempt + 1,
-                        max_attempts = config.max_attempts,
-                        status = status,
-                        "Request failed, retrying..."
-                    );
-
-                    // Restore request for next attempt
-                    current_req = next_req_opt.unwrap();
-
-                    // Calculate and sleep for backoff duration
-                    let backoff = self_clone.calculate_backoff(attempt);
-                    tracing::debug!(backoff_ms = backoff.as_millis(), "Waiting before retry");
-                    tokio::time::sleep(backoff).await;
-
-                    continue;
-                } else {
-                    // Success or no more retries
-                    if attempt > 0 {
+                if attempt < config.max_attempts && config.retryable_statuses.contains(&status) {
+                    if let Some(req) = next_req_opt {
+                        tracing::warn!(
+                            attempt = attempt + 1,
+                            max_attempts = config.max_attempts,
+                            status = status,
+                            "Request failed, retrying..."
+                        );
+
+                        // Restore request for next attempt
+                        current_req = req;
+
+                        // Calculate and sleep for backoff duration
+                        let backoff = self_clone.calculate_backoff(attempt);
+                        tracing::debug!(backoff_ms = backoff.as_millis(), "Waiting before retry");
+                        tokio::time::sleep(backoff).await;
+
+                        continue;
+                    }
+                }
+
+                // Success or no more retries
+                if attempt > 0 {
                         tracing::info!(
                             attempt = attempt + 1,
                             status = status,

crates/rustapi-extras/src/structured_logging/layer.rs

@@ -148,6 +148,7 @@ impl StructuredLoggingLayer {
 
     /// Build a log entry from request and response data
     #[allow(dead_code)]
+    #[allow(clippy::too_many_arguments)]
     fn build_entry(
         &self,
         message: &str,
@@ -262,8 +263,8 @@ impl MiddlewareLayer for StructuredLoggingLayer {
                 if let Some(ref sid) = span_id {
                     entry = entry.span_id(sid);
                 }
-                if let Some(ref sn) = Some(&service_name) {
-                    entry = entry.service_name(*sn);
+                if let Some(sn) = Some(&service_name) {
+                    entry = entry.service_name(sn);
                 }
 
                 entry.request_headers = request_headers.clone();