feat: scaffold adapter interfaces and module boundaries (issue #16)

* feat(rust): introduce domain-adapter-infra module boundaries (issue #16)

* test(docs): add module boundary contract checks and architecture notes (issue #16)

Author: thundermiracle

README.md

@@ -26,13 +26,19 @@ Tauri 2 + React/TypeScript desktop foundation for managing MCP and Skills across
 - `src/`: React frontend application.
 - `src/backend/`: Typed frontend contracts and Tauri invoke client.
 - `src-tauri/`: Rust backend and Tauri application shell.
+- `src-tauri/src/domain/`: Domain layer (adapter interfaces, client profiles, capabilities).
+- `src-tauri/src/adapters/`: Client-specific adapter implementations.
+- `src-tauri/src/infra/`: Infrastructure composition (adapter registry).
+- `src-tauri/src/application/`: Use-case service layer consumed by commands.
 - `docs/spec/`: MVP requirements and contracts.
+- `docs/architecture/`: Module boundary and design documentation.
 - `schemas/`: JSON schema contracts for specs.
 - `tests/`: Node.js unit tests for spec and project contracts.
 
 ## Notes
 
 - The frontend command runner calls typed placeholder backend commands: `detect_clients`, `list_resources`, and `mutate_resource`.
 - Command responses use a shared envelope (`ok`, `data`, `error`, `meta`) with lifecycle and operation metadata.
+- Backend commands are thin boundaries and delegate to `AdapterService` through `AdapterRegistry`.
 - `src-tauri/tauri.conf.json` is configured to use `pnpm` for both dev and build hooks.
 - `scripts/ensure-tauri-icon.mjs` generates the required Tauri icon in clean environments.

docs/architecture/module-boundaries.md

@@ -0,0 +1,23 @@
+# Module Boundaries (Issue #16)
+
+This project uses explicit backend layers to keep client-specific behavior isolated.
+
+## Layer Responsibilities
+
+- `domain/`
+  - Owns adapter interface (`ClientAdapter`) and client profile/capability definitions.
+  - Contains types that describe adapter behavior without binding to concrete client implementations.
+- `adapters/`
+  - Contains concrete adapter implementations for each supported client.
+  - Current scaffold adapters: Claude Code, Codex CLI, Cursor, Codex App.
+- `infra/`
+  - Provides infrastructure wiring (`AdapterRegistry`) that composes adapters.
+  - Exposes registry lookup/iteration used by application services.
+- `application/`
+  - Coordinates use-cases (`AdapterService`) and maps adapter outputs to command contracts.
+- `commands/`
+  - Thin Tauri command boundary; handles envelope metadata and delegates business flow.
+
+## Extension Rule
+
+To add a new client adapter, implement `ClientAdapter` in a new file under `adapters/` and register it in `infra/adapter_registry.rs`. Command and UI layers should remain unchanged.

src-tauri/src/adapters/claude_code.rs

@@ -0,0 +1,34 @@
+use crate::{
+    contracts::{common::ResourceKind, detect::ClientDetection, mutate::MutationAction},
+    domain::{
+        AdapterListResult, AdapterMutationResult, CLAUDE_CODE_PROFILE, ClientAdapter, ClientProfile,
+    },
+};
+
+use super::placeholder::{detect_placeholder, list_placeholder, mutate_placeholder};
+
+pub struct ClaudeCodeAdapter;
+
+impl ClaudeCodeAdapter {
+    pub fn new() -> Self {
+        Self
+    }
+}
+
+impl ClientAdapter for ClaudeCodeAdapter {
+    fn profile(&self) -> &'static ClientProfile {
+        &CLAUDE_CODE_PROFILE
+    }
+
+    fn detect(&self, include_versions: bool) -> ClientDetection {
+        detect_placeholder(self.profile(), include_versions)
+    }
+
+    fn list_resources(&self, resource_kind: ResourceKind) -> AdapterListResult {
+        list_placeholder(self.profile(), resource_kind)
+    }
+
+    fn mutate_resource(&self, action: MutationAction, target_id: &str) -> AdapterMutationResult {
+        mutate_placeholder(self.profile(), action, target_id)
+    }
+}

src-tauri/src/adapters/codex_app.rs

@@ -0,0 +1,34 @@
+use crate::{
+    contracts::{common::ResourceKind, detect::ClientDetection, mutate::MutationAction},
+    domain::{
+        AdapterListResult, AdapterMutationResult, CODEX_APP_PROFILE, ClientAdapter, ClientProfile,
+    },
+};
+
+use super::placeholder::{detect_placeholder, list_placeholder, mutate_placeholder};
+
+pub struct CodexAppAdapter;
+
+impl CodexAppAdapter {
+    pub fn new() -> Self {
+        Self
+    }
+}
+
+impl ClientAdapter for CodexAppAdapter {
+    fn profile(&self) -> &'static ClientProfile {
+        &CODEX_APP_PROFILE
+    }
+
+    fn detect(&self, include_versions: bool) -> ClientDetection {
+        detect_placeholder(self.profile(), include_versions)
+    }
+
+    fn list_resources(&self, resource_kind: ResourceKind) -> AdapterListResult {
+        list_placeholder(self.profile(), resource_kind)
+    }
+
+    fn mutate_resource(&self, action: MutationAction, target_id: &str) -> AdapterMutationResult {
+        mutate_placeholder(self.profile(), action, target_id)
+    }
+}

src-tauri/src/adapters/codex_cli.rs

@@ -0,0 +1,34 @@
+use crate::{
+    contracts::{common::ResourceKind, detect::ClientDetection, mutate::MutationAction},
+    domain::{
+        AdapterListResult, AdapterMutationResult, CODEX_CLI_PROFILE, ClientAdapter, ClientProfile,
+    },
+};
+
+use super::placeholder::{detect_placeholder, list_placeholder, mutate_placeholder};
+
+pub struct CodexCliAdapter;
+
+impl CodexCliAdapter {
+    pub fn new() -> Self {
+        Self
+    }
+}
+
+impl ClientAdapter for CodexCliAdapter {
+    fn profile(&self) -> &'static ClientProfile {
+        &CODEX_CLI_PROFILE
+    }
+
+    fn detect(&self, include_versions: bool) -> ClientDetection {
+        detect_placeholder(self.profile(), include_versions)
+    }
+
+    fn list_resources(&self, resource_kind: ResourceKind) -> AdapterListResult {
+        list_placeholder(self.profile(), resource_kind)
+    }
+
+    fn mutate_resource(&self, action: MutationAction, target_id: &str) -> AdapterMutationResult {
+        mutate_placeholder(self.profile(), action, target_id)
+    }
+}

src-tauri/src/adapters/cursor.rs

@@ -0,0 +1,34 @@
+use crate::{
+    contracts::{common::ResourceKind, detect::ClientDetection, mutate::MutationAction},
+    domain::{
+        AdapterListResult, AdapterMutationResult, CURSOR_PROFILE, ClientAdapter, ClientProfile,
+    },
+};
+
+use super::placeholder::{detect_placeholder, list_placeholder, mutate_placeholder};
+
+pub struct CursorAdapter;
+
+impl CursorAdapter {
+    pub fn new() -> Self {
+        Self
+    }
+}
+
+impl ClientAdapter for CursorAdapter {
+    fn profile(&self) -> &'static ClientProfile {
+        &CURSOR_PROFILE
+    }
+
+    fn detect(&self, include_versions: bool) -> ClientDetection {
+        detect_placeholder(self.profile(), include_versions)
+    }
+
+    fn list_resources(&self, resource_kind: ResourceKind) -> AdapterListResult {
+        list_placeholder(self.profile(), resource_kind)
+    }
+
+    fn mutate_resource(&self, action: MutationAction, target_id: &str) -> AdapterMutationResult {
+        mutate_placeholder(self.profile(), action, target_id)
+    }
+}

src-tauri/src/adapters/mod.rs

@@ -0,0 +1,10 @@
+mod claude_code;
+mod codex_app;
+mod codex_cli;
+mod cursor;
+mod placeholder;
+
+pub use claude_code::ClaudeCodeAdapter;
+pub use codex_app::CodexAppAdapter;
+pub use codex_cli::CodexCliAdapter;
+pub use cursor::CursorAdapter;

src-tauri/src/adapters/placeholder.rs

@@ -0,0 +1,57 @@
+use crate::{
+    contracts::{
+        common::ResourceKind,
+        detect::{ClientDetection, DetectionEvidence, DetectionStatus},
+        mutate::MutationAction,
+    },
+    domain::{AdapterListResult, AdapterMutationResult, ClientProfile},
+};
+
+pub fn detect_placeholder(
+    profile: &'static ClientProfile,
+    include_versions: bool,
+) -> ClientDetection {
+    ClientDetection {
+        client: profile.kind,
+        status: DetectionStatus::Absent,
+        evidence: DetectionEvidence {
+            binary_path: None,
+            config_path: None,
+            version: include_versions.then_some("not_collected".to_string()),
+        },
+        note: format!(
+            "{} adapter scaffold is ready. Detection implementation will be added in issue #19/#20.",
+            profile.display_name
+        ),
+    }
+}
+
+pub fn list_placeholder(
+    profile: &'static ClientProfile,
+    resource_kind: ResourceKind,
+) -> AdapterListResult {
+    AdapterListResult {
+        items: Vec::new(),
+        warning: Some(format!(
+            "{} listing for '{}' is not implemented yet.",
+            resource_kind.as_str(),
+            profile.key
+        )),
+    }
+}
+
+pub fn mutate_placeholder(
+    profile: &'static ClientProfile,
+    action: MutationAction,
+    target_id: &str,
+) -> AdapterMutationResult {
+    AdapterMutationResult {
+        accepted: false,
+        message: format!(
+            "Mutation '{}' for '{}' on '{}' is not implemented yet.",
+            action.as_str(),
+            target_id,
+            profile.key
+        ),
+    }
+}