chore(release): 1.21.2 [skip ci]

## [1.21.2](https://github.com/asepharyana/zesdex/compare/v1.21.1...v1.21.2) (2026-08-28)

### Performance Improvements

* **agent:** symbol index tak pegang mutex global saat rebuild I/O ([3b25e38](https://github.com/asepharyana/zesdex/commit/3b25e3898f9ba2fc1c58b991ad95a8c0bbe98601))

CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.21.2](https://github.com/asepharyana/zesdex/compare/v1.21.1...v1.21.2) (2026-08-28)
+
+
+### Performance Improvements
+
+* **agent:** symbol index tak pegang mutex global saat rebuild I/O ([3b25e38](https://github.com/asepharyana/zesdex/commit/3b25e3898f9ba2fc1c58b991ad95a8c0bbe98601))
+
+## [1.21.1](https://github.com/asepharyana/zesdex/compare/v1.21.0...v1.21.1) (2026-08-28)
+
+
+### Bug Fixes
+
+* **agent:** semantic_search symbol index workspace-aware ([104af3a](https://github.com/asepharyana/zesdex/commit/104af3abb9e626c5d00ea87d523248d606d523ee))
+
 # [1.21.0](https://github.com/asepharyana/zesdex/compare/v1.20.2...v1.21.0) (2026-08-28)
 
 

Cargo.lock

@@ -4862,7 +4862,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-api"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "argon2",
@@ -4885,7 +4885,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-application"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "base64",
@@ -4903,7 +4903,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-bootstrap"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "chrono",
@@ -4920,7 +4920,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-daemon"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "base64",
@@ -4944,7 +4944,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-domain"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "base64",
@@ -4960,7 +4960,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-gateway"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "axum",
@@ -4987,7 +4987,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-grpc"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "axum",
@@ -5004,7 +5004,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-infrastructure"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "argon2",
@@ -5052,7 +5052,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-tui"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "base64",
@@ -5078,7 +5078,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-web"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "axum",
@@ -5098,7 +5098,7 @@ dependencies = [
 
 [[package]]
 name = "zesdex-ws"
-version = "1.20.1"
+version = "1.21.0"
 dependencies = [
  "anyhow",
  "axum",

Cargo.toml

@@ -15,7 +15,7 @@ members = [
 ]
 
 [workspace.package]
-version = "1.21.0"
+version = "1.21.2"
 edition = "2021"
 authors = ["asepharyana <superaseph@gmail.com>"]
 

apps/infrastructure/src/tools/semantic_search.rs

@@ -121,7 +121,66 @@ pub struct CodeSymbol {
 }
 
 /// The in-memory symbol index, shared via a global static.
-static SYMBOL_INDEX: Mutex<Option<SymbolIndex>> = Mutex::new(None);
+///
+/// Keyed by workspace path: each workspace owns its own `SymbolIndex`, so
+/// searching one repo never leaks stale symbols from another, and a rebuild
+/// triggered for workspace A cannot clobber the index of B. The lock is only
+/// ever held briefly (to check/insert/look up) — never across the I/O-heavy
+/// walk in `rebuild`, which runs locally and is swapped in under a short lock.
+static SYMBOL_INDEX: std::sync::OnceLock<Mutex<HashMap<String, SymbolIndex>>> =
+    std::sync::OnceLock::new();
+
+/// Access the (lazily initialised) global per-workspace symbol index map.
+fn symbol_index_map() -> &'static Mutex<HashMap<String, SymbolIndex>> {
+    SYMBOL_INDEX.get_or_init(|| Mutex::new(HashMap::new()))
+}
+
+/// Ensure the per-workspace symbol index is built, returning the symbol count.
+///
+/// If `force` is true, or the workspace has no cached (non-empty) index yet,
+/// the index is rebuilt. The rebuild itself runs OUTSIDE the global lock
+/// (the walk can take seconds on a large repo), then the result is stored
+/// under a short lock so concurrent searches never block on the I/O. Returns
+/// the number of symbols now cached for the workspace.
+fn ensure_symbol_index(workspace: &str, force: bool) -> Result<usize> {
+    let ready = {
+        let map = symbol_index_map()
+            .lock()
+            .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
+        !force && map.get(workspace).is_some_and(|i| !i.is_empty())
+    };
+
+    if !ready {
+        // Rebuild locally, off the global lock (I/O heavy).
+        let mut fresh = SymbolIndex::new();
+        let count = fresh.rebuild(workspace)?;
+        // Swap in under a short lock; keep an existing non-empty index if a
+        // concurrent rebuild already populated this workspace.
+        let mut map = symbol_index_map()
+            .lock()
+            .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
+        if map.get(workspace).is_none_or(|i| i.is_empty()) {
+            map.insert(workspace.to_string(), fresh);
+        }
+        return Ok(count);
+    }
+
+    let map = symbol_index_map()
+        .lock()
+        .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
+    Ok(map.get(workspace).map_or(0, |i| i.len()))
+}
+
+/// Lock and return a borrow to the global per-workspace symbol index map.
+///
+/// The caller must have called [`ensure_symbol_index`] first, then looks up
+/// its workspace key; the lookup is short and in-memory, so holding the guard
+/// for the search is fine.
+fn symbol_index() -> Result<std::sync::MutexGuard<'static, HashMap<String, SymbolIndex>>> {
+    symbol_index_map()
+        .lock()
+        .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))
+}
 
 // ---------------------------------------------------------------------------
 //  Language-specific regexes (lazily compiled)
@@ -270,6 +329,17 @@ impl SymbolIndex {
         self.symbols.is_empty()
     }
 
+    /// Returns true when the cached index must be rebuilt for the given
+    /// workspace — either because nothing has been indexed yet, or because the
+    /// requested workspace differs from the one the index was built for.
+    ///
+    /// Without this, searching a *different* workspace after the first one
+    /// silently returns stale symbols from the previously indexed repo
+    /// (a misleading result for a coding agent).
+    pub fn needs_rebuild(&self, workspace: &str) -> bool {
+        self.is_empty() || self.workspace_path.as_deref() != Some(workspace)
+    }
+
     pub fn len(&self) -> usize {
         self.symbols.len()
     }
@@ -1250,15 +1320,11 @@ impl Tool for SemanticSearch {
             "semantic search"
         );
 
-        let mut guard = SYMBOL_INDEX
-            .lock()
-            .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
-        let index = guard.get_or_insert_with(SymbolIndex::new);
-
-        if rebuild || index.is_empty() {
-            let count = index.rebuild(&workspace)?;
-            debug!(symbol_count = count, "symbol index rebuilt");
-        }
+        // Build (or load) the per-workspace index without holding the global
+        // lock across the I/O-heavy walk.
+        let _count = ensure_symbol_index(&workspace, rebuild)?;
+        let map = symbol_index()?;
+        let index = map.get(&workspace).expect("index should be ensured");
 
         // Map kind filter to enum
         let target_kind = match kind_filter {
@@ -1398,12 +1464,15 @@ impl Tool for RebuildIndex {
 
         info!("rebuilding multi-language symbol index");
 
-        let mut guard = SYMBOL_INDEX
-            .lock()
-            .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
-        let index = guard.get_or_insert_with(SymbolIndex::new);
-        let count = index.rebuild(&workspace)?;
-        let by_lang = index.count_by_language();
+        // Force a rebuild of this workspace's index. The walk runs off the
+        // global lock (via ensure_symbol_index) so it cannot stall concurrent
+        // searches.
+        let count = ensure_symbol_index(&workspace, true)?;
+        let map = symbol_index()?;
+        let by_lang = match map.get(&workspace) {
+            Some(i) => i.count_by_language(),
+            None => Vec::new(),
+        };
 
         let mut out = format!(
             "Symbol index rebuilt successfully. {} symbols indexed.\n\n",
@@ -1497,15 +1566,11 @@ impl Tool for ListSymbols {
             .map(|p| p.to_string_lossy().to_string())
             .unwrap_or_else(|| ".".to_string());
 
-        let mut guard = SYMBOL_INDEX
-            .lock()
-            .map_err(|e| anyhow::anyhow!("index lock failed: {e}"))?;
-        let index = guard.get_or_insert_with(SymbolIndex::new);
-
-        if rebuild || index.is_empty() {
-            let count = index.rebuild(&workspace)?;
-            info!(symbol_count = count, "symbol index rebuilt for list");
-        }
+        // Build (or load) the per-workspace index without holding the global
+        // lock across the I/O-heavy walk.
+        let _count = ensure_symbol_index(&workspace, rebuild)?;
+        let map = symbol_index()?;
+        let index = map.get(&workspace).expect("index should be ensured");
 
         let target_lang = match lang_filter {
             "rust" => Some(Language::Rust),
@@ -1746,4 +1811,80 @@ mod tests {
         let index = SymbolIndex::new();
         assert!(index.search("anything", 10).is_empty());
     }
+
+    #[test]
+    fn test_needs_rebuild_workspace_aware() {
+        let ws_a = std::env::temp_dir().join(format!("ws_a_{}", uuid::Uuid::new_v4()));
+        let ws_b = std::env::temp_dir().join(format!("ws_b_{}", uuid::Uuid::new_v4()));
+        std::fs::create_dir_all(&ws_a).unwrap();
+        std::fs::create_dir_all(&ws_b).unwrap();
+        std::fs::write(ws_a.join("a.rs"), "pub fn fn_in_a() {}\n").unwrap();
+        std::fs::write(ws_b.join("b.rs"), "pub fn fn_in_b() {}\n").unwrap();
+
+        let mut index = SymbolIndex::new();
+        let a = ws_a.to_string_lossy().to_string();
+        let b = ws_b.to_string_lossy().to_string();
+
+        // Fresh index: needs rebuild for any workspace.
+        assert!(index.needs_rebuild(&a));
+
+        // After rebuilding A, searching A needs no rebuild...
+        index.rebuild(&a).unwrap();
+        assert!(!index.needs_rebuild(&a));
+        // ...but searching B DOES (stale index otherwise).
+        assert!(
+            index.needs_rebuild(&b),
+            "workspace switch must trigger rebuild"
+        );
+
+        // Rebuilding B flips the cached workspace.
+        index.rebuild(&b).unwrap();
+        assert!(!index.needs_rebuild(&b));
+        assert!(index.needs_rebuild(&a));
+
+        std::fs::remove_dir_all(&ws_a).ok();
+        std::fs::remove_dir_all(&ws_b).ok();
+    }
+
+    #[test]
+    fn test_ensure_symbol_index_per_workspace_isolation() {
+        let ws_a = std::env::temp_dir().join(format!("iso_a_{}", uuid::Uuid::new_v4()));
+        let ws_b = std::env::temp_dir().join(format!("iso_b_{}", uuid::Uuid::new_v4()));
+        std::fs::create_dir_all(&ws_a).unwrap();
+        std::fs::create_dir_all(&ws_b).unwrap();
+        std::fs::write(ws_a.join("a.rs"), "pub fn only_in_a() {}\n").unwrap();
+        std::fs::write(ws_b.join("b.rs"), "pub fn only_in_b() {}\n").unwrap();
+
+        let a = ws_a.to_string_lossy().to_string();
+        let b = ws_b.to_string_lossy().to_string();
+
+        // Build A and B independently through the shared global helper.
+        let count_a = ensure_symbol_index(&a, false).unwrap();
+        assert!(
+            count_a >= 1,
+            "workspace A should index its fn, got {count_a}"
+        );
+        let count_b = ensure_symbol_index(&b, false).unwrap();
+        assert!(
+            count_b >= 1,
+            "workspace B should index its fn, got {count_b}"
+        );
+
+        // Rebuilding A must not have clobbered B and vice-versa.
+        let count_a_again = ensure_symbol_index(&a, true).unwrap();
+        assert!(count_a_again >= 1);
+
+        // Each workspace's cached index is independently correct.
+        {
+            let map = symbol_index().unwrap();
+            let idx_a = map.get(&a).unwrap();
+            assert!(!idx_a.search("only_in_a", 5).is_empty());
+            assert!(idx_a.search("only_in_b", 5).is_empty());
+            let idx_b = map.get(&b).unwrap();
+            assert!(!idx_b.search("only_in_b", 5).is_empty());
+        }
+
+        std::fs::remove_dir_all(&ws_a).ok();
+        std::fs::remove_dir_all(&ws_b).ok();
+    }
 }