@smilintux/skcapstone 0.4.7 → 0.5.1

package/src/skcapstone/emotion_tracker.py

@@ -392,8 +392,8 @@ class EmotionTracker:
                 if item.get("timestamp", "") >= cutoff:
                     try:
                         entries.append(EmotionEntry(**item))
-                    except Exception:
-                        pass
+                    except Exception as exc:
+                        logger.warning("Failed to parse emotion log entry: %s", exc)
             return entries
         except Exception as exc:
             logger.debug("Failed to load emotion log: %s", exc)

package/src/skcapstone/export.py

@@ -423,8 +423,8 @@ def _import_conversations(
                 existing_data = json.loads(peer_file.read_text(encoding="utf-8"))
                 if isinstance(existing_data, list):
                     existing_messages = existing_data
-            except Exception:
-                pass
+            except Exception as exc:
+                logger.warning("Failed to read existing conversation for peer %s: %s", peer, exc)
 
         # Deduplicate by (role, content, timestamp) tuple
         existing_keys = {

package/src/skcapstone/itil.py

@@ -1100,5 +1100,5 @@ class ITILManager:
         try:
             from . import activity
             activity.push(topic, payload)
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to push ITIL event %s to activity bus: %s", topic, exc)

package/src/skcapstone/launchd.py

@@ -420,7 +420,7 @@ def service_logs(suffix: str = "daemon", lines: int = 50) -> str:
                 )
                 if r.stdout.strip():
                     output_parts.append(f"--- {log_path.name} ---\n{r.stdout}")
-            except Exception:
-                pass
+            except Exception as exc:
+                logger.warning("Failed to read launchd log %s: %s", log_path, exc)
 
     return "\n".join(output_parts) if output_parts else f"No logs found in {logs_dir}"

package/src/skcapstone/mcp_server.py

@@ -142,8 +142,8 @@ def _get_agent_name(home: Path) -> str:
         try:
             data = json.loads(identity_path.read_text(encoding="utf-8"))
             return data.get("name", "anonymous")
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to read agent name from identity.json: %s", exc)
     return "anonymous"
 
 
@@ -2654,6 +2654,39 @@ async def list_tools() -> list[Tool]:
                 "required": ["query"],
             },
         ),
+        # Brain-First Protocol
+        Tool(
+            name="brain_first_check",
+            description=(
+                "Brain-First Protocol: consult the agent's memory before "
+                "acting on a task. Extracts keywords from the given context, "
+                "searches memory for relevant prior knowledge, and returns "
+                "any matching memories. Use this before starting new work to "
+                "avoid duplicating effort or missing prior decisions."
+            ),
+            inputSchema={
+                "type": "object",
+                "properties": {
+                    "context": {
+                        "type": "string",
+                        "description": (
+                            "The task description, prompt, or action context "
+                            "to search memory for"
+                        ),
+                    },
+                    "tags": {
+                        "type": "array",
+                        "items": {"type": "string"},
+                        "description": "Optional tag filter for the memory search",
+                    },
+                    "max_results": {
+                        "type": "integer",
+                        "description": "Max memories to return (default: from config, usually 5)",
+                    },
+                },
+                "required": ["context"],
+            },
+        ),
     ]
 
 
@@ -2813,6 +2846,8 @@ async def call_tool(name: str, arguments: dict) -> list[TextContent]:
         "itil_cab_vote": _handle_itil_cab_vote,
         "itil_status": _handle_itil_status,
         "itil_kedb_search": _handle_itil_kedb_search,
+        # Brain-First Protocol
+        "brain_first_check": _handle_brain_first_check,
     }
     handler = handlers.get(name)
     if handler is None:
@@ -3154,8 +3189,8 @@ async def _handle_coord_complete(args: dict) -> list[TextContent]:
                     t.priority.value, ("community", "support_ticket", 50)
                 )
                 break
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to calculate joules for completed task %s: %s", task_id, exc)
 
     return _json_response({
         "completed": True,
@@ -4937,6 +4972,15 @@ async def _handle_itil_kedb_search(args: dict) -> list[TextContent]:
     return await _impl(args)
 
 
+# ── Brain-First Protocol ──────────────────────────────────
+
+
+async def _handle_brain_first_check(args: dict) -> list[TextContent]:
+    """Consult memory before acting on a task."""
+    from .mcp_tools.brain_first_tools import _handle_brain_first_check as _impl
+    return await _impl(args)
+
+
 # ═══════════════════════════════════════════════════════════
 # Entry Point
 # ═══════════════════════════════════════════════════════════

package/src/skcapstone/mcp_tools/__init__.py

@@ -17,6 +17,7 @@ from mcp.types import TextContent, Tool
 from . import (
     agent_tools,
     ansible_tools,
+    brain_first_tools,
     chat_tools,
     comm_tools,
     consciousness_tools,
@@ -49,6 +50,7 @@ from . import (
 # Ordered list of all tool-group modules.
 _MODULES = [
     agent_tools,
+    brain_first_tools,
     memory_tools,
     comm_tools,
     sync_tools,

package/src/skcapstone/mcp_tools/_helpers.py

@@ -46,6 +46,6 @@ def _get_agent_name(home: Path) -> str:
         try:
             data = json.loads(identity_path.read_text(encoding="utf-8"))
             return data.get("name", "anonymous")
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to read agent name from identity.json: %s", exc)
     return "anonymous"

package/src/skcapstone/mcp_tools/ansible_tools.py

@@ -10,6 +10,7 @@ from __future__ import annotations
 
 import asyncio
 import json
+import logging
 import shutil
 import uuid
 from pathlib import Path
@@ -18,6 +19,8 @@ from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="run_ansible_playbook",
@@ -146,8 +149,8 @@ async def _handle_run_ansible_playbook(args: dict) -> list[TextContent]:
                 try:
                     if _activity is not None:
                         _activity.push(event_type, {"run_id": run_id, "line": line})
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to push ansible line event for run %s: %s", run_id, exc)
 
         await asyncio.gather(
             _drain(proc.stdout, stdout_lines, "ansible.playbook.line"),
@@ -184,8 +187,8 @@ async def _handle_run_ansible_playbook(args: dict) -> list[TextContent]:
     try:
         if _activity is not None:
             _activity.push("ansible.playbook.done", summary)
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to push ansible.playbook.done event for run %s: %s", run_id, exc)
 
     # --- store in memory with tag=ansible-run ---
     try:

package/src/skcapstone/mcp_tools/brain_first_tools.py

@@ -0,0 +1,90 @@
+"""Brain-First Protocol MCP tools.
+
+Exposes the brain-first memory consultation as an MCP tool so that
+any MCP client can ask "what do I already know about this?" before
+acting on a task.
+"""
+
+from __future__ import annotations
+
+from mcp.types import TextContent, Tool
+
+from ._helpers import _json_response
+
+TOOLS: list[Tool] = [
+    Tool(
+        name="brain_first_check",
+        description=(
+            "Brain-First Protocol: consult the agent's memory before "
+            "acting on a task. Extracts keywords from the given context, "
+            "searches memory for relevant prior knowledge, and returns "
+            "any matching memories. Use this before starting new work to "
+            "avoid duplicating effort or missing prior decisions."
+        ),
+        inputSchema={
+            "type": "object",
+            "properties": {
+                "context": {
+                    "type": "string",
+                    "description": (
+                        "The task description, prompt, or action context "
+                        "to search memory for"
+                    ),
+                },
+                "tags": {
+                    "type": "array",
+                    "items": {"type": "string"},
+                    "description": "Optional tag filter for the memory search",
+                },
+                "max_results": {
+                    "type": "integer",
+                    "description": "Max memories to return (default: from config, usually 5)",
+                },
+            },
+            "required": ["context"],
+        },
+    ),
+]
+
+
+async def _handle_brain_first_check(args: dict) -> list[TextContent]:
+    """Run a brain-first memory consultation."""
+    from ..brain_first import BrainFirstConfig, brain_first_check, _load_config
+
+    context = args.get("context", "")
+    if not context:
+        return _json_response({"error": "context is required"})
+
+    config = _load_config()
+
+    # Allow per-call override of max_results
+    max_results = args.get("max_results")
+    if max_results is not None:
+        config.max_results = max_results
+
+    result = brain_first_check(
+        context=context,
+        config=config,
+        tags=args.get("tags"),
+    )
+
+    response = {
+        "enabled": result.enabled,
+        "query": result.query,
+        "keywords": result.keywords,
+        "memories_found": len(result.memories),
+        "memories": result.memories,
+    }
+
+    if result.error:
+        response["warning"] = result.error
+
+    if result.has_memories:
+        response["context_block"] = result.as_context()
+
+    return _json_response(response)
+
+
+HANDLERS: dict = {
+    "brain_first_check": _handle_brain_first_check,
+}

package/src/skcapstone/mcp_tools/capauth_tools.py

@@ -8,11 +8,14 @@ Exposes two tools:
 from __future__ import annotations
 
 import json as _json
+import logging
 
 from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="capauth_status",
@@ -82,8 +85,8 @@ async def _handle_capauth_status(_args: dict) -> list[TextContent]:
     if did_key_file.exists():
         try:
             result["did_key_file"] = did_key_file.read_text(encoding="utf-8").strip()
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to read DID key file: %s", exc)
 
     # Check identity file
     identity_file = home / "identity" / "identity.json"
@@ -92,8 +95,8 @@ async def _handle_capauth_status(_args: dict) -> list[TextContent]:
             ident = _json.loads(identity_file.read_text(encoding="utf-8"))
             result["identity_name"] = ident.get("name")
             result["identity_fingerprint"] = ident.get("fingerprint")
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to read identity.json for capauth status: %s", exc)
 
     return _json_response(result)
 

package/src/skcapstone/mcp_tools/coord_tools.py

@@ -2,10 +2,14 @@
 
 from __future__ import annotations
 
+import logging
+
 from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response, _shared_root
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="coord_status",
@@ -149,8 +153,8 @@ async def _handle_coord_claim(args: dict) -> list[TextContent]:
         try:
             from .. import activity
             activity.push("task.claimed", {"task_id": task_id, "agent": agent_name})
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to push task.claimed activity for %s: %s", task_id, exc)
         return _json_response({
             "claimed": True,
             "task_id": task_id,
@@ -175,8 +179,8 @@ async def _handle_coord_complete(args: dict) -> list[TextContent]:
     try:
         from .. import activity
         activity.push("task.completed", {"task_id": task_id, "agent": agent_name})
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to push task.completed activity for %s: %s", task_id, exc)
     return _json_response({
         "completed": True,
         "task_id": task_id,

package/src/skcapstone/mcp_tools/did_tools.py

@@ -10,6 +10,7 @@ Exposes four tools:
 from __future__ import annotations
 
 import json as _json
+import logging
 import os as _os
 import socket as _socket
 from pathlib import Path
@@ -18,6 +19,8 @@ from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="did_show",
@@ -165,8 +168,8 @@ def _load_policy() -> dict:
     if p.exists():
         try:
             return _json.loads(p.read_text(encoding="utf-8"))
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to load DID policy from %s: %s", p, exc)
     return dict(_POLICY_DEFAULT)
 
 
@@ -191,8 +194,8 @@ def _resolve_tailnet(tailnet_hostname: str, tailnet_name: str) -> tuple[str, str
         if not tailnet_hostname:
             try:
                 tailnet_hostname = _socket.gethostname()
-            except Exception:
-                pass
+            except Exception as exc:
+                logger.warning("Failed to resolve hostname for tailnet DID: %s", exc)
     if not tailnet_name:
         tailnet_name = _os.environ.get("SKWORLD_TAILNET", "")
     return tailnet_hostname, tailnet_name
@@ -297,8 +300,8 @@ async def _handle_did_verify_peer(args: dict) -> list[TextContent]:
                 _json.dumps(peer_data, indent=2, default=str),
                 encoding="utf-8",
             )
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.warning("Failed to cache did:key back to peer file %s: %s", peer_file, exc)
 
     return _json_response({
         "name": name,

package/src/skcapstone/mcp_tools/memory_tools.py

@@ -2,10 +2,14 @@
 
 from __future__ import annotations
 
+import logging
+
 from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="memory_store",
@@ -130,8 +134,8 @@ async def _handle_memory_store(args: dict) -> list[TextContent]:
             "importance": entry.importance,
             "tags": entry.tags,
         })
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to push memory.stored activity for %s: %s", entry.memory_id, exc)
     return _json_response({
         "memory_id": entry.memory_id,
         "layer": entry.layer.value,

package/src/skcapstone/mcp_tools/soul_tools.py

@@ -2,10 +2,14 @@
 
 from __future__ import annotations
 
+import logging
+
 from mcp.types import TextContent, Tool
 
 from ._helpers import _error_response, _home, _json_response, _text_response
 
+logger = logging.getLogger(__name__)
+
 TOOLS: list[Tool] = [
     Tool(
         name="soul_list",
@@ -250,8 +254,8 @@ async def _handle_soul_list(args: dict) -> list[TextContent]:
                 "source": "installed",
                 "active": name == state.active_soul,
             })
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to list installed soul blueprints: %s", exc)
 
     # 2) Blueprints repo
     blueprints_repo = Path.home() / "clawd" / "soul-blueprints" / "blueprints"

package/src/skcapstone/mdns_discovery.py

@@ -238,8 +238,8 @@ class MDNSDiscovery:
                         agent_name,
                     )
                     return
-            except Exception:
-                pass
+            except Exception as exc:
+                logger.warning("Failed to read existing mDNS heartbeat for %s: %s", agent_name, exc)
 
         heartbeat = {
             "agent_name": agent_name,

package/src/skcapstone/metrics.py

@@ -358,8 +358,8 @@ class MetricsCollector:
                         1 for t in transports.values()
                         if isinstance(t, dict) and t.get("enabled", True)
                     )
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to parse skcomm transport config: %s", exc)
 
             report.transport = TransportMetrics(
                 available=True,
@@ -474,8 +474,8 @@ class MetricsCollector:
             if state_path.exists():
                 try:
                     state = json.loads(state_path.read_text(encoding="utf-8"))
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to read sync_state.json: %s", exc)
 
             report.sync = SyncMetrics(
                 available=True,
@@ -510,8 +510,8 @@ class MetricsCollector:
                 try:
                     subs = json.loads(subs_file.read_text(encoding="utf-8"))
                     sub_count = len(subs)
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to read pubsub subscriptions.json: %s", exc)
 
             report.pubsub = PubSubMetrics(
                 available=True,
@@ -546,8 +546,8 @@ class MetricsCollector:
                 try:
                     rot_data = json.loads(rot_log.read_text(encoding="utf-8"))
                     rotations = len(rot_data)
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to read KMS rotation log: %s", exc)
 
             report.kms = KmsMetrics(
                 available=True,

package/src/skcapstone/migrate_memories.py

@@ -100,8 +100,8 @@ def migrate(
     try:
         existing = store.list_memories(limit=10000)
         existing_ids = {m.id for m in existing}
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.warning("Failed to load existing memory IDs for deduplication: %s", exc)
 
     for entry in entries:
         if entry.memory_id in existing_ids:

package/src/skcapstone/models.py

@@ -238,6 +238,19 @@ class SyncConfig(BaseModel):
     git_remote: Optional[str] = None
 
 
+class BrainFirstConfig(BaseModel):
+    """Configuration for the brain-first protocol.
+
+    When enabled, agents consult memory before acting on tasks
+    to surface prior knowledge and avoid redundant work.
+    """
+
+    enabled: bool = True
+    max_results: int = 5
+    min_importance: float = 0.3
+    auto_inject: bool = False
+
+
 class AgentConfig(BaseModel):
     """Persistent configuration for the agent runtime."""
 
@@ -249,6 +262,7 @@ class AgentConfig(BaseModel):
     trust_home: Path = Path("~/.cloud9")
     default_connector: Optional[str] = None
     sync: SyncConfig = Field(default_factory=SyncConfig)
+    brain_first: BrainFirstConfig = Field(default_factory=BrainFirstConfig)
     capabilities: list[str] = Field(
         default_factory=lambda: ["consciousness", "code", "chat", "memory"]
     )

package/src/skcapstone/onboard.py

@@ -21,6 +21,7 @@ Steps:
 from __future__ import annotations
 
 import json
+import logging
 import sys
 import time
 from datetime import datetime, timezone
@@ -28,6 +29,8 @@ from pathlib import Path
 from typing import Optional
 
 import click
+
+logger = logging.getLogger(__name__)
 from rich.console import Console
 from rich.panel import Panel
 from rich.prompt import Confirm, Prompt
@@ -503,8 +506,8 @@ def _step_ollama_models(prereqs: dict) -> bool:
         if DEFAULT_MODEL in (r.stdout or ""):
             click.echo(click.style("  ✓ ", fg="green") + f"{DEFAULT_MODEL} already present")
             return True
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.debug("Failed to check ollama model list: %s", exc)
 
     if not click.confirm(f"  Pull default model ({DEFAULT_MODEL}, ~2 GB)?", default=True):
         click.echo(click.style("  ↷ ", fg="bright_black") + f"Skipped — pull later: ollama pull {DEFAULT_MODEL}")
@@ -991,8 +994,8 @@ def run_onboard(home: Optional[str] = None) -> None:
         soul = load_soul()
         if soul and soul.boot_message:
             boot_message = soul.boot_message
-    except Exception:
-        pass
+    except Exception as exc:
+        logger.debug("Failed to load soul boot message, using default: %s", exc)
 
     # -----------------------------------------------------------------------
     # Summary table

package/src/skcapstone/peer_directory.py

@@ -250,8 +250,8 @@ class PeerDirectory:
                         ts = data.get("timestamp", "")
                         if ts:
                             self._entries[agent_name].last_seen = ts
-                    except Exception:
-                        pass
+                    except Exception as exc:
+                        logger.warning("Failed to update last_seen from heartbeat for %s: %s", agent_name, exc)
                     continue
 
                 try:

package/src/skcapstone/providers/docker.py

@@ -307,8 +307,8 @@ class DockerProvider(ProviderBackend):
             old = client.containers.get(container_name)
             logger.warning("Removing stale container: %s", container_name)
             old.remove(force=True)
-        except Exception:
-            pass
+        except Exception as exc:
+            logger.debug("No stale container to remove for %s (expected if first run): %s", container_name, exc)
 
         # Ensure named volume for agent state persistence
         try:

package/src/skcapstone/register.py

@@ -88,7 +88,7 @@ def _build_package_registry(workspace: Optional[Path] = None) -> list[dict]:
             "mcp_cmd": None,
             "mcp_args": None,
             "mcp_env": None,
-            "openclaw_plugin_path": workspace / "pillar-repos" / "cloud9-python" / "openclaw-plugin" / "src" / "index.ts",
+            "openclaw_plugin_path": workspace / "pillar-repos" / "cloud9" / "openclaw-plugin-python" / "src" / "index.ts",
         },
         {
             "name": "sksecurity",
@@ -124,7 +124,7 @@ _PILLAR_DIR_MAP: dict[str, Optional[str]] = {
     "skcomm": "skcomm",
     "skchat": "skchat",
     "capauth": "capauth",
-    "cloud9": "cloud9-python",
+    "cloud9": "cloud9",
     "sksecurity": "sksecurity",
     "skseed": "skseed",
     "skgit": None,  # skill dir only, no pillar repo

package/src/skcapstone/service_health.py

@@ -76,8 +76,8 @@ def _http_check(
                 try:
                     body = json.loads(resp.read().decode("utf-8"))
                     result["version"] = body.get(version_key)
-                except Exception:
-                    pass
+                except Exception as exc:
+                    logger.warning("Failed to parse version from service health response: %s", exc)
     except urllib.error.HTTPError as exc:
         latency = (time.monotonic() - t0) * 1000
         result["latency_ms"] = round(latency, 1)

package/src/skcapstone/sync_watcher.py

@@ -574,8 +574,8 @@ class SyncWatcher:
             try:
                 self._observer.stop()
                 self._observer.join(timeout=5)
-            except Exception:
-                pass
+            except Exception as exc:
+                logger.warning("Error stopping SyncWatcher observer: %s", exc)
             self._observer = None
         logger.info("SyncWatcher stopped.")