docs: remove CLI and Homebrew references — a3s-code is library-only

Replace CLI Installation/Run/CLI Options sections with library dependency
instruction. Remove server/ from project structure diagram. Remove dead
ToolCategory tests (enum already removed). Remove telemetry split from
Phase 10 roadmap.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: ZhiXiao-Lin

README.md

@@ -89,44 +89,14 @@ just test
 
 ## Quick Start
 
-### Installation
+### Add Dependency
 
-```bash
-# Homebrew (macOS / Linux) — prebuilt binary
-brew tap a3s-lab/tap https://github.com/A3S-Lab/homebrew-tap
-brew install a3s-code
-
-# Cargo (builds from source)
-cargo install a3s-code
-
-# From source
-cargo build --release
-```
-
-### Run
-
-```bash
-# Start with default settings
-a3s-code
-
-# With HCL config file
-a3s-code --config agent.hcl
-
-# With JSON config file
-a3s-code --config ~/.a3s/config.json
-
-# Self-update
-a3s-code update
+```toml
+# Cargo.toml
+[dependencies]
+a3s-code-core = "0.1"
 ```
 
-### CLI Options
-
-| Flag | Env Var | Description |
-|------|---------|-------------|
-| `-c, --config` | `A3S_CONFIG` | Path to config file (HCL or JSON, auto-detected by extension) |
-| `--otlp-endpoint` | `OTEL_EXPORTER_OTLP_ENDPOINT` | OpenTelemetry OTLP endpoint |
-| `--json-log` | `A3S_LOG_FORMAT` | Output logs in JSON format |
-
 ## Configuration
 
 A3S Code uses **HCL** (preferred) or **JSON** for configuration. Format is auto-detected by file extension.
@@ -360,12 +330,6 @@ code/                          # Cargo workspace root
 │       ├── context.rs         # Context compaction
 │       ├── session_lane_queue.rs # Priority queue (a3s-lane)
 │       └── telemetry.rs       # Metrics via tracing events
-├── server/                    # a3s-code — CLI binary
-│   ├── Cargo.toml
-│   └── src/
-│       ├── main.rs            # CLI entry point
-│       ├── lib.rs             # Re-exports from core
-│       └── telemetry_init.rs  # OpenTelemetry subscriber setup
 ├── sdk/
 │   ├── node-native/           # Native Node.js addon (napi-rs)
 │   └── python-native/         # Native Python module (PyO3)
@@ -397,7 +361,6 @@ Core agent loop, 11 tools, multi-session management, permission system, HITL, sk
 - [x] Multi-provider LLM config (default model required, multiple providers optional)
 - [x] All 11 tools callable via direct function calls without serialization
 - [x] Native Python bindings (PyO3) and Node.js bindings (napi-rs)
-- [x] Telemetry split: core metrics via tracing events, OTel init in server crate
 - [x] 1,492 unit tests
 
 ### Phase 11: Multi-Model Routing 🚧

core/src/hitl.rs

@@ -378,51 +378,6 @@ impl ConfirmationManager {
 mod tests {
     use super::*;
 
-    // ========================================================================
-    // ToolCategory Tests
-    // ========================================================================
-
-    #[test]
-    fn test_tool_category() {
-        assert_eq!(ToolCategory::from_tool_name("read"), ToolCategory::ReadOnly);
-        assert_eq!(ToolCategory::from_tool_name("glob"), ToolCategory::ReadOnly);
-        assert_eq!(ToolCategory::from_tool_name("bash"), ToolCategory::Mutating);
-        assert_eq!(
-            ToolCategory::from_tool_name("write"),
-            ToolCategory::Mutating
-        );
-        assert_eq!(
-            ToolCategory::from_tool_name("unknown"),
-            ToolCategory::Mutating
-        );
-    }
-
-    #[test]
-    fn test_tool_category_all_readonly() {
-        let readonly_tools = ["read", "glob", "ls", "grep", "list_files", "search"];
-        for tool in readonly_tools {
-            assert_eq!(
-                ToolCategory::from_tool_name(tool),
-                ToolCategory::ReadOnly,
-                "Tool '{}' should be ReadOnly",
-                tool
-            );
-        }
-    }
-
-    #[test]
-    fn test_tool_category_all_mutating() {
-        let mutating_tools = ["bash", "write", "edit", "delete", "move", "copy", "execute"];
-        for tool in mutating_tools {
-            assert_eq!(
-                ToolCategory::from_tool_name(tool),
-                ToolCategory::Mutating,
-                "Tool '{}' should be Mutating",
-                tool
-            );
-        }
-    }
-
     // ========================================================================
     // SessionLane Tests
     // ========================================================================
@@ -497,7 +452,8 @@ mod tests {
     fn test_confirmation_policy_default() {
         let policy = ConfirmationPolicy::default();
         assert!(!policy.enabled);
-        assert!(!policy.requires_confirmation("bash")); // HITL disabled
+        // HITL disabled = everything is YOLO (no confirmation needed)
+        assert!(!policy.requires_confirmation("bash"));
         assert!(!policy.requires_confirmation("write"));
         assert!(!policy.requires_confirmation("read"));
     }
@@ -506,10 +462,11 @@ mod tests {
     fn test_confirmation_policy_enabled() {
         let policy = ConfirmationPolicy::enabled();
         assert!(policy.enabled);
-        assert!(policy.requires_confirmation("bash")); // Mutating tool
-        assert!(policy.requires_confirmation("write")); // Mutating tool
-        assert!(!policy.requires_confirmation("read")); // ReadOnly tool
-        assert!(!policy.requires_confirmation("grep")); // ReadOnly tool
+        // All tools require confirmation when enabled with no YOLO lanes
+        assert!(policy.requires_confirmation("bash"));
+        assert!(policy.requires_confirmation("write"));
+        assert!(policy.requires_confirmation("read"));
+        assert!(policy.requires_confirmation("grep"));
     }
 
     #[test]
@@ -518,7 +475,7 @@ mod tests {
 
         assert!(!policy.requires_confirmation("bash")); // Execute lane in YOLO mode
         assert!(!policy.requires_confirmation("write")); // Execute lane in YOLO mode
-        assert!(!policy.requires_confirmation("read")); // ReadOnly
+        assert!(policy.requires_confirmation("read")); // Query lane NOT in YOLO
     }
 
     #[test]
@@ -533,25 +490,20 @@ mod tests {
     }
 
     #[test]
-    fn test_confirmation_policy_explicit_lists() {
-        let policy = ConfirmationPolicy::enabled()
-            .with_auto_approve_tools(["bash".to_string()])
-            .with_require_confirm_tools(["read".to_string()]);
+    fn test_confirmation_policy_is_yolo() {
+        let policy = ConfirmationPolicy::enabled().with_yolo_lanes([SessionLane::Execute]);
 
-        assert!(!policy.requires_confirmation("bash")); // Explicitly auto-approved
-        assert!(policy.requires_confirmation("read")); // Explicitly required
-        assert!(policy.requires_confirmation("write")); // Default for Mutating
+        assert!(policy.is_yolo("bash")); // Execute lane
+        assert!(policy.is_yolo("write")); // Execute lane
+        assert!(!policy.is_yolo("read")); // Query lane, not YOLO
     }
 
     #[test]
-    fn test_confirmation_policy_explicit_overrides_yolo() {
-        // require_confirm_tools should override YOLO mode
-        let policy = ConfirmationPolicy::enabled()
-            .with_yolo_lanes([SessionLane::Execute])
-            .with_require_confirm_tools(["bash".to_string()]);
-
-        assert!(policy.requires_confirmation("bash")); // Explicitly required, overrides YOLO
-        assert!(!policy.requires_confirmation("write")); // Still in YOLO mode
+    fn test_confirmation_policy_disabled_is_always_yolo() {
+        let policy = ConfirmationPolicy::default(); // disabled
+        assert!(policy.is_yolo("bash"));
+        assert!(policy.is_yolo("read"));
+        assert!(policy.is_yolo("unknown_tool"));
     }
 
     #[test]
@@ -579,8 +531,19 @@ mod tests {
         let (event_tx, _) = broadcast::channel(100);
         let manager = ConfirmationManager::new(ConfirmationPolicy::enabled(), event_tx);
 
+        // All tools require confirmation when HITL enabled with no YOLO lanes
         assert!(manager.requires_confirmation("bash").await);
-        assert!(!manager.requires_confirmation("read").await);
+        assert!(manager.requires_confirmation("read").await);
+    }
+
+    #[tokio::test]
+    async fn test_confirmation_manager_with_yolo() {
+        let (event_tx, _) = broadcast::channel(100);
+        let policy = ConfirmationPolicy::enabled().with_yolo_lanes([SessionLane::Query]);
+        let manager = ConfirmationManager::new(policy, event_tx);
+
+        assert!(manager.requires_confirmation("bash").await); // Execute lane, not YOLO
+        assert!(!manager.requires_confirmation("read").await); // Query lane, YOLO
     }
 
     #[tokio::test]