docs(Common): Refactor transport layer to trait-based abstraction

Update architecture documentation to reflect the shift from concrete transport implementations in Common to a trait-based `TransportStrategy` interface. The transport layer now defines only the trait surface and configuration types, with concrete implementations (`gRPCTransport`, `IPCTransport`, `WASMTransport`, `MistTransport`) delegated to the `Grove` element.

Remove references to the deprecated transport modules (circuit breaker, metrics, dynamic selection) from Common's public API. The `IPCProvider` table entry simplifies to exclude 'establish' as connection lifecycle is now handled by the transport implementations.

Documentation in `Documentation/GitHub/Architecture.md` and `README.md` updated to match the current architectural state. No code changes—purely documentation alignment.

Author: NikolaRHristov

Documentation/GitHub/Architecture.md

@@ -36,7 +36,7 @@ graph TB
         ENV["Environment / Requires<br/>compile-time DI"]
         DTO["Data Transfer Objects<br/>FileStat / InitData /<br/>TerminalOptions"]
         ERR["CommonError<br/>unified error enum"]
-        TRANS["Transport Layer<br/>gRPC / IPC / WASM"]
+        TRANS["Transport Layer<br/>TransportStrategy + Config"]
         TEL["Telemetry<br/>PostHog + OTLP"]
 
         TRAITS --> AE
@@ -123,7 +123,7 @@ pub trait FileSystem: Send + Sync {
 | `Testing/`                 | `TestController`                        | Testing         | run, discover, results                                            |
 | `SourceControlManagement/` | `SourceControlManagementProvider`       | SCM             | status, commit, push, pull, diff                                  |
 | `Synchronization/`         | `SynchronizationProvider`               | Sync            | push, pull, merge, resolve                                        |
-| `IPC/`                     | `IPCProvider`                           | Communication   | establish, send, receive, proxy                                   |
+| `IPC/`                     | `IPCProvider`                           | Communication   | send, receive, proxy                                              |
 | `Webview/`                 | `WebviewProvider`                       | Webviews        | create, sendMessage, dispose                                      |
 | `TreeView/`                | `TreeViewProvider`                      | Tree views      | getChildren, getParent, resolveItem                               |
 | `StatusBar/`               | `StatusBarProvider`                     | Status bar      | setItem, updateItem, removeItem                                   |
@@ -243,7 +243,7 @@ Effect executed with concrete implementation
 | `ConfigurationTarget` | `Configuration`       | Global, Workspace, WorkspaceFolder       | Config ops               |
 | `SearchOptions`       | `Search`              | pattern, include, exclude, maxResults    | Search ops               |
 | `WorkspaceEditDTO`    | `DTO/`                | edits, fileCreates, fileDeletes          | Workspace edits          |
-| `TransportConfig`     | `Transport`           | strategy, timeout, retry config          | IPC configuration        |
+| `TransportConfig`     | `Transport`           | timeout, retry config                    | Transport configuration  |
 
 ---
 
@@ -272,19 +272,17 @@ pub enum CommonError {
 `Common` provides a transport-agnostic communication interface:
 
 ```rust
-pub enum TransportStrategy {
-    Grpc(GrpcConfig),
-    Ipc(IpcConfig),
-    Wasm(WasmConfig),
+pub trait TransportStrategy: Send + Sync {
+    type Error: std::error::Error + Send + Sync + 'static;
+    async fn connect(&self) -> Result<(), Self::Error>;
+    async fn send(&self, request: &[u8]) -> Result<Vec<u8>, Self::Error>;
+    async fn close(&self) -> Result<(), Self::Error>;
+    fn is_connected(&self) -> bool;
+    fn transport_type(&self) -> TransportType;
 }
 ```
 
-| Feature           | Description                                       |
-| ----------------- | ------------------------------------------------- |
-| Circuit breaker   | Automatic failure detection and isolation         |
-| Retry             | Configurable retry with exponential backoff       |
-| Metrics           | Prometheus metrics for transport operations       |
-| Dynamic selection | Runtime transport switching based on availability |
+`Common` defines the trait surface and `TransportConfig`; concrete transports (`gRPCTransport`, `IPCTransport`, `WASMTransport`, `MistTransport`) live in `Grove`.
 
 ---
 

README.md

@@ -72,9 +72,8 @@ The DTO library provides all data structures used for IPC communication with
 across every service domain, making error handling consistent and predictable.
 
 The `Transport` layer offers transport-agnostic communication through a unified
-`TransportStrategy` interface. It supports `gRPC`, `IPC`, and `WASM` with
-built-in circuit breaker, retry logic, metrics collection, and dynamic transport
-selection. The `Telemetry` module provides a dual-pipe (`PostHog` + `OTLP`) emit
+`TransportStrategy` trait. Concrete implementations (`gRPCTransport`, `IPCTransport`,
+`WASMTransport`, `MistTransport`) live in `Grove`. The `Telemetry` module provides a dual-pipe (`PostHog` + `OTLP`) emit
 surface shared across all `Rust` sidecars.
 
 ---
@@ -222,7 +221,7 @@ graph LR
             Errors["CommonError - unified error enum"]:::common
         end
         subgraph INFRA["Infrastructure"]
-            Transport["Transport/ - TransportStrategy\ngRPC · IPC · WASM + circuit breaker"]:::transport
+            Transport["Transport/ - TransportStrategy\ntrait + config types"]:::transport
             Telemetry["Telemetry/ - PostHog + OTLP\ndual-pipe emit surface"]:::common
             Env["Environment/ + Effect/\nApplicationRunTime trait\nDI via Requires / HasEnvironment"]:::common
         end