fix(clippy): resolve all PR #71 review issues — unused imports, operator precedence, and_then→map

- C1: Wire GovernancePolicy, AuditRecord, MaintenanceAction into harness
  lifecycle — LifecycleOutcome now carries policy, audit trail, and
  maintenance action instead of dropping them as unused imports
- C2: Prefix unused mermaid_results variable with underscore
- C3: Add parentheses around left_edge + self.gap in Kasuari constraint expression
- C4: Replace and_then(|x| Ok(y)) with map(|x| y) in provider.rs
- C5: Remove unused arc_kit_au::EdgeType import from ledgerr-mcp/lib.rs
- Also remove unused AuditRecord import from b00t-iface/core/machine.rs
- Also prefix unused issue_count variable in ledger-core/validation.rs

Author: Claude Sonnet (coordinator)

crates/b00t-iface/src/core/machine.rs

@@ -35,7 +35,7 @@
 //!                         Terminated
 //! ```
 
-use super::surface::{AuditRecord, MaintenanceAction};
+use super::surface::MaintenanceAction;
 use std::fmt;
 use std::time::Duration;
 

crates/b00t-iface/src/exec/harness.rs

@@ -16,6 +16,9 @@ pub struct LifecycleOutcome<H: Clone> {
     pub surface: String,
     pub chain: PromiseChain<H>,
     pub governance_violations: Vec<String>,
+    pub audit: Option<AuditRecord>,
+    pub policy: GovernancePolicy,
+    pub maintenance_action: MaintenanceAction,
 }
 
 /// The executive harness — wraps a surface with governance + machine.
@@ -43,8 +46,8 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
     where
         S::Handle: Clone,
     {
-        let governance = &self.capability.governance;
-        if let Err(v) = governance.validate() {
+        let policy = self.capability.governance.clone();
+        if let Err(v) = self.capability.governance.validate() {
             self.governance_violations.push(v);
         }
 
@@ -81,7 +84,7 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
             .apply_maintenance(&maintain_action, now.elapsed())
         {
             Ok(()) => {
-                LifecyclePromise::fulfilled(PromiseOp::Maintain, now.elapsed(), maintain_action)
+                LifecyclePromise::fulfilled(PromiseOp::Maintain, now.elapsed(), maintain_action.clone())
             }
             Err(e) => LifecyclePromise::rejected(PromiseOp::Maintain, now.elapsed(), e),
         };
@@ -103,6 +106,9 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
                         ),
                     },
                     governance_violations: self.governance_violations.clone(),
+                    policy,
+                    maintenance_action: MaintenanceAction::NoOp,
+                    audit: None,
                 };
             }
         };
@@ -120,6 +126,17 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
             }
         };
 
+        let audit = match &terminate_promise.value {
+            crate::core::PromiseValue::Fulfilled(_) => Some(AuditRecord {
+                surface_name: self.capability.name.to_owned(),
+                uptime: now.elapsed(),
+                exit_reason: "terminated".into(),
+                crash_count: self.machine.crash_count,
+                bytes_logged: 0,
+            }),
+            _ => None,
+        };
+
         LifecycleOutcome {
             surface: self.capability.name.to_owned(),
             chain: PromiseChain {
@@ -129,6 +146,9 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
                 terminate: terminate_promise,
             },
             governance_violations: self.governance_violations.clone(),
+            policy,
+            maintenance_action: maintain_action,
+            audit,
         }
     }
 
@@ -158,6 +178,9 @@ impl<S: ProcessSurface> SurfaceHarness<S> {
                 ),
             },
             governance_violations: self.governance_violations.clone(),
+            policy: self.capability.governance.clone(),
+            maintenance_action: MaintenanceAction::Terminate,
+            audit: None,
         }
     }
 }

crates/b00t-iface/src/sarif/mod.rs

@@ -636,7 +636,7 @@ mod tests {
     fn valid_mermaid_passes_lint() {
         let content = "```rhai\nfn foo() -> bar\n```\n\n```mermaid\nflowchart TD\nfoo[bar]\n```";
         let report = check_l3dg3rr_standards(&[("good.md", content)]);
-        let mermaid_results: Vec<_> = report.runs[0]
+        let _mermaid_results: Vec<_> = report.runs[0]
             .results
             .iter()
             .filter(|r| r.rule_id.contains("mermaid-parse"))

crates/ledger-core/src/validation.rs

@@ -182,7 +182,7 @@ where
 {
     let next = f(current.meta.clone());
     let issues = next.issues.clone();
-    let issue_count = issues.len();
+    let _issue_count = issues.len();
     let meta = next.meta.advance(stage, next.confidence, &issues);
     let result = StageResult {
         data: next.data,

crates/ledger-core/src/visualize.rs

@@ -471,7 +471,7 @@ pub mod layout {
 
                 let left_edge = left_x + left_w;
                 solver
-                    .add_constraint(left_edge + self.gap | LE(Strength::REQUIRED) | right_x)
+                    .add_constraint((left_edge + self.gap) | LE(Strength::REQUIRED) | right_x)
                     .ok();
             }
 

crates/ledgerr-mcp-core/src/provider.rs

@@ -372,7 +372,7 @@ pub mod mock {
             self.call_result
                 .as_ref()
                 .map_err(|e| e.clone())
-                .and_then(|v| Ok(v.clone()))
+                .map(|v| v.clone())
         }
 
         fn shutdown(&self) {}

crates/ledgerr-mcp/src/lib.rs

@@ -1121,7 +1121,6 @@ impl TurboLedgerService {
 
     fn emit_ingest_evidence(&self, row: &TransactionInput, tx_id: &str) -> Result<(), ToolError> {
         use arc_kit_au::node::{ExtractedRow, SourceDoc, Transaction};
-        use arc_kit_au::EdgeType;
         use chrono::Utc;
 
         let mut evidence = self