synap-git 2.0.0__tar.gz → 2.1.0__tar.gz

{synap_git-2.0.0 → synap_git-2.1.0}/CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.1.0] - 2026-05-29
+
+### Added
+- **CLI `search` command**: New user-facing command for executing hybrid structural searches directly from the terminal.
+- **`embed_provider` configuration**: Added explicit control over the vector embedding provider via `SYNAP_EMBED_PROVIDER` or `config.toml`.
+
+### Fixed
+- **CRITICAL: Metadata Nullability Bug**: Resolved a type mismatch in `CodeSymbol` where `metadata=None` caused background worker crashes. Enforced strict `dict` contract with `default_factory`.
+- **CRITICAL: Exception Swallowing**: Eradicated 26 instances of silent failure (`except Exception: pass`) across all core subsystems (API, Indexer, Storage, Retrieval, MCP, Providers). Replaced with structured, traceback-aware logging.
+- **CRITICAL: SQLite Schema Integrity**: Fixed `NOT NULL` constraint failure in `llm_calls` table by aligning schema with insertion logic and adding `cost_usd` support.
+- **HIGH: Repository-Local Logging**: Redirected daemon logs from a global shared directory to repository-local `.synap/logs` to prevent cross-repository log corruption and lock contention.
+- **HIGH: Rollback Atomicity**: Reordered `synap rollback` operations to ensure Git state is successfully restored before purging index checkpoints.
+- **MEDIUM: Provider Resiliency**: Added explicit handling for `NotImplementedError` in Anthropic and OpenRouter providers, preventing worker panics when embeddings are misconfigured.
+
 ## [2.0.0] - 2026-05-29
 
 ### Added

{synap_git-2.0.0 → synap_git-2.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: synap-git
-Version: 2.0.0
+Version: 2.1.0
 Summary: Persistent structural context infrastructure for AI coding agents.
 Project-URL: Homepage, https://github.com/saahilpal/synap-git
 Project-URL: Repository, https://github.com/saahilpal/synap-git

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/__init__.py

@@ -2,4 +2,4 @@
 
 __all__ = ["__version__"]
 
-__version__ = "2.0.0"
+__version__ = "2.1.0"

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/api/app.py

@@ -101,8 +101,10 @@ def create_app(runtime: SynapRuntime) -> FastAPI:
             target = target[:-3]
         try:
             await asyncio.to_thread(runtime.wiki.ensure_wiki_page, target)
-        except Exception:
-            pass
+        except Exception as e:
+            import structlog
+
+            structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
         wiki_path = runtime.wiki.wiki_dir / f"{target}.md"
         exists = await asyncio.to_thread(wiki_path.exists)

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/cli/main.py

@@ -642,8 +642,10 @@ def start(
                 return
             else:
                 pid_file.unlink()
-        except Exception:
-            pass
+        except Exception as e:
+            import structlog
+
+            structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
     # 2. Spawn detached daemon process
     cmd = [sys.executable, "-m", "synap_git.cli", "daemon-run", abs_path.as_posix()]
@@ -782,8 +784,10 @@ def stop(
                 os.waitpid(pid, os.WNOHANG)
             except (ChildProcessError, OSError):
                 pass
-        except Exception:
-            pass
+        except Exception as e:
+            import structlog
+
+            structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
         success = not _is_process_running(pid)
 
     # Clean up lockfiles
@@ -895,6 +899,7 @@ def status(
 
 @app.command()
 def logs(
+    path: Annotated[str, typer.Argument(help="Repository path.")] = ".",
     tail: Annotated[
         bool, typer.Option("--tail", "-t", help="Stream new log entries in real-time.")
     ] = False,
@@ -906,7 +911,9 @@ def logs(
     """View and tail Synap runtime logs."""
     import time
 
-    log_dir = Path("~/.config/synap/logs").expanduser()
+    settings = _settings(path)
+    log_dir = settings.log_path
+    assert log_dir is not None
     log_file = log_dir / "daemon.log"
 
     if not log_file.exists():
@@ -938,6 +945,32 @@ def logs(
         console.print(f"[red]Error reading logs: {e}[/red]")
 
 
+@app.command()
+def search(
+    query: Annotated[str, typer.Argument(help="Search query.")],
+    path: Annotated[str, typer.Option("--path", "-p", help="Repository path.")] = ".",
+    max_tokens: Annotated[
+        int, typer.Option("--max-tokens", help="Maximum context window tokens.")
+    ] = 4000,
+) -> None:
+    """Execute a hybrid structural search across the repository."""
+    settings = _settings(path)
+    runtime = SynapRuntime(settings)
+
+    with Progress(
+        SpinnerColumn(),
+        TextColumn("[progress.description]{task.description}"),
+        transient=True,
+    ) as progress:
+        progress.add_task(description="Retrieving context...", total=None)
+        answer, context, _ = runtime.query_hybrid(query, max_tokens=max_tokens)
+
+    console.print("\n[bold cyan]Hybrid Retrieval Results[/bold cyan]")
+    console.print("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━")
+    console.print(answer)
+    console.print("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━")
+
+
 @app.command()
 def update() -> None:
     """Check for updates and upgrade the Synap runtime."""
@@ -1179,16 +1212,6 @@ def rollback(
         if not typer.confirm("\nProceed?", default=False):
             raise typer.Abort()
 
-    # Clear current checkpoint
-    try:
-        with runtime.store.connect() as conn:
-            conn.execute(
-                "DELETE FROM checkpoints WHERE branch = ?",
-                (runtime.git.state().effective_branch,),
-            )
-    except Exception:
-        pass
-
     # git checkout
     try:
         subprocess.check_call(  # noqa: S603
@@ -1199,6 +1222,18 @@ def rollback(
         console.print(f"[red]✗ Git checkout failed: {e}[/red]")
         raise typer.Abort()
 
+    # Clear current checkpoint
+    try:
+        with runtime.store.connect() as conn:
+            conn.execute(
+                "DELETE FROM checkpoints WHERE branch = ?",
+                (runtime.git.state().effective_branch,),
+            )
+    except Exception as e:
+        import structlog
+
+        structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
+
     runtime.bootstrap(force=True)
     console.print(f"[green]✓ Rolled back successfully to {selected_commit}[/green]")
 
@@ -1264,8 +1299,10 @@ def repair(
     console.print("[3/3] Regenerating wiki from last known state")
     try:
         runtime.wiki.generate_project_wiki()
-    except Exception:
-        pass
+    except Exception as e:
+        import structlog
+
+        structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
     status_info = runtime.status()
     console.print(f"\n[green]✓ Recovery complete. {status_info.files} files restored.[/green]")

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/config.py

@@ -92,7 +92,9 @@ class SynapSettings(BaseSettings):
                     if line.strip().startswith(f"{key}="):
                         return line.split("=", 1)[1].strip().strip("\"'")
         except Exception:
-            pass
+            import structlog
+
+            structlog.get_logger(__name__).exception("fallback_credential_lookup_failed", key=key)
         return None
 
     @property

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/diagnostics/logger.py

@@ -22,7 +22,6 @@ class TraceResult:
 
 def configure_logging(settings: SynapSettings) -> None:
     from logging.handlers import RotatingFileHandler
-    from pathlib import Path
 
     timestamper = structlog.processors.TimeStamper(fmt="iso", utc=True)
     shared_processors: list[structlog.typing.Processor] = [
@@ -38,7 +37,8 @@ def configure_logging(settings: SynapSettings) -> None:
     else:
         renderer = structlog.dev.ConsoleRenderer(colors=False)
 
-    log_dir = Path("~/.config/synap/logs").expanduser()
+    log_dir = settings.log_path
+    assert log_dir is not None
     log_dir.mkdir(parents=True, exist_ok=True)
     log_file = log_dir / "daemon.log"
 

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/git/state.py

@@ -128,8 +128,10 @@ class GitRepository:
                         break
                 if is_revert:
                     return GitChange(kind=GitChangeKind.REVERT, previous=previous, current=current)
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
             if len(current.commit_parent_hashes) > 1:
                 return GitChange(kind=GitChangeKind.MERGE, previous=previous, current=current)

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/indexer/daemon.py

@@ -244,16 +244,20 @@ class RuntimeDaemon:
         if heartbeat_file.exists():
             try:
                 heartbeat_file.unlink()
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
         # Cleanup process PID lockfile as well
         pid_file = self.settings.repository_path / ".synap" / "daemon.pid"
         if pid_file.exists():
             try:
                 pid_file.unlink()
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
     async def _poll_git_loop(self) -> None:
         while not self._stop_event.is_set():

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/indexer/engine.py

@@ -141,8 +141,10 @@ class SynapRuntime:
                 if change.kind == "revert":
                     self.handle_revert(prev, git_state)
                     return git_state.head_commit
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
         return self.index_repository(git_state=git_state, force=force)
 
@@ -263,11 +265,20 @@ class SynapRuntime:
                     for sym in batch:
                         try:
                             # Avoid re-embedding if unchanged?
-                            text = sym["name"] + " " + sym.get("metadata", {}).get("docstring", "")
+                            text = (
+                                sym["name"] + " " + (sym.get("metadata") or {}).get("docstring", "")
+                            )
                             vector = provider.embed(text)
                             self.store.put_embedding(
                                 sym["symbol_id"], model_name, "1.0", "1.0", vector, content_hash
                             )
+                        except NotImplementedError as e:
+                            logger.warning(
+                                "embedding_not_supported_by_provider",
+                                error=str(e),
+                                suggestion="Configure SYNAP_EMBED_PROVIDER=ollama or openai.",
+                            )
+                            return  # Stop attempting batch if not supported
                         except Exception as e:
                             logger.warning("background_embed_failed_for_symbol", error=str(e))
             except Exception as e:
@@ -676,8 +687,12 @@ class SynapRuntime:
                         candidate_rel = (target_dir / "/".join(parts)).as_posix()
                         for ext in [".py", ".ts", ".go", ".rs"]:
                             candidate_paths.add(f"{candidate_rel}{ext}")
-                    except Exception:
-                        pass
+                    except Exception as e:
+                        import structlog
+
+                        structlog.get_logger().error(
+                            "suppressed_error_caught", error=str(e), exc_info=True
+                        )
                 else:
                     module_keys.add(module_part)
                 fts_names.add(target_name)
@@ -743,8 +758,12 @@ class SynapRuntime:
                                 if cand_path in path_to_file_id:
                                     target_file_id = path_to_file_id[cand_path]
                                     break
-                        except Exception:
-                            pass
+                        except Exception as e:
+                            import structlog
+
+                            structlog.get_logger().error(
+                                "suppressed_error_caught", error=str(e), exc_info=True
+                            )
                     else:
                         target_file_id = module_key_to_file_id.get(module_part)
 
@@ -811,8 +830,12 @@ class SynapRuntime:
                                 if cand_path in path_to_file_id:
                                     target_file_id = path_to_file_id[cand_path]
                                     break
-                        except Exception:
-                            pass
+                        except Exception as e:
+                            import structlog
+
+                            structlog.get_logger().error(
+                                "suppressed_error_caught", error=str(e), exc_info=True
+                            )
                     else:
                         target_file_id = module_key_to_file_id.get(module_part)
 
@@ -888,8 +911,10 @@ class SynapRuntime:
         is_dirty = False
         try:
             is_dirty = self.git.state().is_dirty
-        except Exception:
-            pass
+        except Exception as e:
+            import structlog
+
+            structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
         return self.retrieval_engine.retrieve(query, max_tokens=max_tokens, is_dirty=is_dirty)
 
     def doctor(self, *, auto_recover: bool = False) -> dict[str, Any]:

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/indexer/wiki.py

@@ -132,8 +132,10 @@ class WikiEngine:
                     purpose="wiki",
                     file_path=file_path,
                 )
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
         except Exception as e:
             logger.error("wiki_generation_failed", path=file_path, error=str(e))
             raise
@@ -189,7 +191,9 @@ class WikiEngine:
                     purpose="wiki_project",
                     file_path=clean_name,
                 )
-            except Exception:
-                pass
+            except Exception as e:
+                import structlog
+
+                structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
         except Exception as e:
             logger.error("wiki_project_generation_failed", page=clean_name, error=str(e))

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/mcp/server.py

@@ -138,8 +138,12 @@ class SynapMCPServer:
                                 "dirty_tree": dirty,
                             },
                         )
-                    except Exception:
-                        pass
+                    except Exception as e:
+                        import structlog
+
+                        structlog.get_logger().error(
+                            "suppressed_error_caught", error=str(e), exc_info=True
+                        )
 
                     return json.dumps(
                         {
@@ -176,8 +180,12 @@ class SynapMCPServer:
                                 "trace_id": trace_id,
                             },
                         )
-                    except Exception:
-                        pass
+                    except Exception as e:
+                        import structlog
+
+                        structlog.get_logger().error(
+                            "suppressed_error_caught", error=str(e), exc_info=True
+                        )
 
                     return json.dumps(
                         {

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/parser/registry.py

@@ -1,6 +1,6 @@
 from __future__ import annotations
 
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from pathlib import Path
 from typing import Any
 
@@ -28,7 +28,7 @@ class CodeSymbol:
     start_line: int
     end_line: int
     ast_hash: str
-    metadata: dict[str, Any] | None = None
+    metadata: dict[str, Any] = field(default_factory=dict)
 
 
 @dataclass(frozen=True)

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/provider/anthropic.py

@@ -5,8 +5,12 @@ import urllib.request
 from collections.abc import Iterator
 from typing import Any
 
+import structlog
+
 from synap_git.provider.base import LLMProvider, LLMResponse
 
+logger = structlog.get_logger(__name__)
+
 
 class AnthropicProvider(LLMProvider):
     """Anthropic API provider implementation."""
@@ -109,7 +113,7 @@ class AnthropicProvider(LLMProvider):
                                 if "text" in delta:
                                     yield delta["text"]
                         except Exception:
-                            pass
+                            logger.exception("anthropic_stream_chunk_parse_failed", data=data_part)
         except Exception as exc:
             raise RuntimeError(f"Anthropic generate stream failed: {exc}") from exc
 

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/provider/openai.py

@@ -5,8 +5,12 @@ import urllib.request
 from collections.abc import Iterator
 from typing import Any
 
+import structlog
+
 from synap_git.provider.base import LLMProvider, LLMResponse
 
+logger = structlog.get_logger(__name__)
+
 
 class OpenAIProvider(LLMProvider):
     """OpenAI API provider implementation."""
@@ -101,7 +105,7 @@ class OpenAIProvider(LLMProvider):
                             if "content" in delta:
                                 yield delta["content"]
                         except Exception:
-                            pass
+                            logger.exception("openai_stream_chunk_parse_failed", data=data_part)
         except Exception as exc:
             raise RuntimeError(f"OpenAI generate stream failed: {exc}") from exc
 

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/provider/openrouter.py

@@ -5,9 +5,13 @@ import urllib.request
 from collections.abc import Iterator
 from typing import Any
 
+import structlog
+
 from synap_git.provider.base import LLMResponse
 from synap_git.provider.openai import OpenAIProvider
 
+logger = structlog.get_logger(__name__)
+
 
 class OpenRouterProvider(OpenAIProvider):
     """OpenRouter API provider implementation using OpenAI-compatible endpoints."""
@@ -106,7 +110,7 @@ class OpenRouterProvider(OpenAIProvider):
                             if "content" in delta:
                                 yield delta["content"]
                         except Exception:
-                            pass
+                            logger.exception("openrouter_stream_chunk_parse_failed", data=data_part)
         except Exception as exc:
             raise RuntimeError(f"OpenRouter generate stream failed: {exc}") from exc
 

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/retrieval/engine.py

@@ -284,8 +284,12 @@ class HybridRetrievalEngine:
                         output_tokens=response.completion_tokens,
                         purpose="retrieval",
                     )
-                except Exception:
-                    pass
+                except Exception as e:
+                    import structlog
+
+                    structlog.get_logger().error(
+                        "suppressed_error_caught", error=str(e), exc_info=True
+                    )
             else:
                 err_msg = str(last_err)
                 answer_content = (

{synap_git-2.0.0 → synap_git-2.1.0}/src/synap_git/storage/sqlite.py

@@ -156,6 +156,7 @@ class SynapStore:
                         model TEXT NOT NULL,
                         input_tokens INTEGER NOT NULL,
                         output_tokens INTEGER NOT NULL,
+                        cost_usd REAL NOT NULL,
                         purpose TEXT NOT NULL,
                         file_path TEXT,
                         created_at INTEGER NOT NULL
@@ -244,9 +245,9 @@ class SynapStore:
                     CREATE INDEX IF NOT EXISTS idx_files_content_hash ON files(content_hash);
                     CREATE INDEX IF NOT EXISTS idx_files_git_oid ON files(git_oid);
 
-                    CREATE TRIGGER IF NOT EXISTS tgr_symbols_insert 
+                    CREATE TRIGGER IF NOT EXISTS tgr_symbols_insert
                     AFTER INSERT ON symbols BEGIN
-                        INSERT INTO symbols_fts(symbol_id, name, kind, body) 
+                        INSERT INTO symbols_fts(symbol_id, name, kind, body)
                         VALUES (new.symbol_id, new.name, new.kind, new.metadata_json);
                     END;
 
@@ -682,8 +683,10 @@ class SynapStore:
                 res = conn.execute("PRAGMA quick_check").fetchone()[0]
                 if res == "ok":
                     return False
-        except Exception:
-            pass
+        except Exception as e:
+            import structlog
+
+            structlog.get_logger().error("suppressed_error_caught", error=str(e), exc_info=True)
 
         # If we got here, it's corrupted or unreachable. Wipe it securely.
         for ext in ["", "-wal", "-shm"]:
@@ -702,6 +705,7 @@ class SynapStore:
         input_tokens: int,
         output_tokens: int,
         purpose: str,
+        cost_usd: float = 0.0,
         file_path: str | None = None,
     ) -> None:
         import uuid
@@ -712,8 +716,8 @@ class SynapStore:
         with self.connect() as conn:
             conn.execute(
                 """
-                INSERT INTO llm_calls (call_id, provider, model, input_tokens, output_tokens, purpose, file_path, created_at)
-                VALUES (?, ?, ?, ?, ?, ?, ?, ?)
+                INSERT INTO llm_calls (call_id, provider, model, input_tokens, output_tokens, cost_usd, purpose, file_path, created_at)
+                VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
                 """,
                 (
                     call_id,
@@ -721,6 +725,7 @@ class SynapStore:
                     model or "unknown",
                     input_tokens,
                     output_tokens,
+                    cost_usd,
                     purpose,
                     file_path,
                     now,