code-puppy 0.0.714__tar.gz → 0.0.716__tar.gz

{code_puppy-0.0.714 → code_puppy-0.0.716}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: code-puppy
-Version: 0.0.714
+Version: 0.0.716
 Summary: Code generation agent
 Project-URL: repository, https://github.com/mpfaffenberger/code_puppy
 Project-URL: HomePage, https://github.com/mpfaffenberger/code_puppy

{code_puppy-0.0.714 → code_puppy-0.0.716}/code_puppy/plugins/__init__.py

@@ -2,6 +2,7 @@ import importlib
 import importlib.util
 import logging
 import sys
+from importlib.metadata import entry_points
 import types
 from pathlib import Path
 
@@ -13,6 +14,8 @@ logger = logging.getLogger(__name__)
 # User plugins directory
 USER_PLUGINS_DIR = Path.home() / ".code_puppy" / "plugins"
 
+PLUGIN_ENTRY_POINT_GROUP = "code_puppy.plugins"
+
 # Track if plugins have already been loaded to prevent duplicate registration
 _PLUGINS_LOADED = False
 
@@ -24,21 +27,68 @@ _loaded_plugin_names: dict[str, list[str]] = {"builtin": [], "user": [], "projec
 _project_plugin_status: dict[str, str] = {}
 
 
-def _load_builtin_plugins(plugins_dir: Path) -> list[str]:
-    """Load built-in plugins from the package plugins directory.
+def _load_installed_plugins() -> list[str]:
+    """Load distribution-provided plugins advertised through entry points.
 
-    Returns list of successfully loaded plugin names.
+    Installed plugin bundles are the builtin tier: they load before user and
+    project plugins, but remain physically independent from the core package.
+    Entry points are sorted for deterministic startup and test behavior.
+    """
+    from code_puppy.config import get_safety_permission_level
+
+    loaded: list[str] = []
+    discovered = sorted(
+        entry_points(group=PLUGIN_ENTRY_POINT_GROUP), key=lambda item: item.name
+    )
+    for entry_point in discovered:
+        plugin_name = entry_point.name
+        if plugin_name == "shell_safety" and get_safety_permission_level() not in (
+            "none",
+            "low",
+        ):
+            logger.debug("Skipping shell_safety plugin due to safety permission level")
+            continue
+        try:
+            set_loading_context(plugin_name)
+            entry_point.load()
+            loaded.append(plugin_name)
+        except ImportError as exc:
+            logger.warning("Failed to import installed plugin %s: %s", plugin_name, exc)
+        except Exception as exc:
+            logger.error(
+                "Unexpected error loading installed plugin %s: %s",
+                plugin_name,
+                exc,
+                exc_info=True,
+            )
+        finally:
+            clear_loading_context()
+    return loaded
+
+
+def _load_builtin_plugins(
+    plugins_dir: Path, skip_names: set[str] | None = None
+) -> list[str]:
+    """Load legacy plugins still bundled in the core package.
+
+    ``skip_names`` prevents duplicate registration during the migration when
+    the same plugin is both installed through an entry point and still present
+    in an older core checkout.
     """
     # Import safety permission check for shell_safety plugin
     from code_puppy.config import get_safety_permission_level
 
     loaded = []
+    skip_names = set(skip_names or ())
 
     for item in plugins_dir.iterdir():
         if item.is_dir() and not item.name.startswith("_"):
             plugin_name = item.name
             callbacks_file = item / "register_callbacks.py"
 
+            if plugin_name in skip_names:
+                continue
+
             if callbacks_file.exists():
                 # Skip shell_safety plugin unless safety_permission_level is "low" or "none"
                 if plugin_name == "shell_safety":
@@ -478,7 +528,9 @@ def load_plugin_callbacks() -> dict[str, list[str]]:
             if _trust.is_plugin_trusted(project_root, name, project_plugins_dir / name)
         }
 
-    builtin_loaded = _load_builtin_plugins(plugins_dir)
+    installed_loaded = _load_installed_plugins()
+    legacy_loaded = _load_builtin_plugins(plugins_dir, skip_names=set(installed_loaded))
+    builtin_loaded = installed_loaded + legacy_loaded
     user_skip_names = set(builtin_loaded) | project_plugin_names
     user_loaded = _load_user_plugins(USER_PLUGINS_DIR, skip_names=user_skip_names)
 

{code_puppy-0.0.714 → code_puppy-0.0.716}/code_puppy/plugins/puppy_kennel/README.md

@@ -124,10 +124,11 @@ Plus a **`/kennel`** slash command suite for humans:
 
 ## Enable / disable
 
-The plugin is **enabled by default**. Flip it with the slash commands:
+The plugin is **disabled by default** — running in the background on
+noise is off until you opt in. Flip it with the slash commands:
 
+- ``/kennel enable`` (or ``/kennel on``) — turn memory on
 - ``/kennel disable`` (or ``/kennel off``) — turn memory off
-- ``/kennel enable`` (or ``/kennel on``) — turn memory back on
 
 State is persisted to ``puppy.cfg`` under ``kennel_enabled`` and read on
 every callback, so the toggle is live — no restart needed, and the
@@ -168,7 +169,7 @@ front end can read or write the same value.
 
 | Key | Default | Effect |
 |---|---|---|
-| `kennel_enabled` | unset → enabled | Single source of truth for the on/off toggle. Set to `false`/`0`/`no`/`off` (case-insensitive) to disable. Anything else — missing, blank, or garbage — leaves the kennel on. Flipped live by `/kennel enable` and `/kennel disable`. |
+| `kennel_enabled` | unset → disabled | Single source of truth for the on/off toggle. Set to `true`/`1`/`yes`/`on` (case-insensitive) to enable. Anything else — missing, blank, or garbage — leaves the kennel off. Flipped live by `/kennel enable` and `/kennel disable`. |
 
 ## How tools reach the agent
 

{code_puppy-0.0.714 → code_puppy-0.0.716}/code_puppy/plugins/puppy_kennel/state.py

@@ -5,12 +5,12 @@ Single source of truth for whether the kennel is on. Flipped by the
 ``puppy.cfg`` under the key ``kennel_enabled`` so the front end can read
 and write the same value.
 
-Default is **enabled** -- a missing key, blank value, or garbage value
-all leave the kennel on. Only the explicit-falsy tokens
-``{"false", "0", "no", "off"}`` (case-insensitive) turn it off. That
-asymmetry is on purpose: on a default-on system, a typo like
-``kennel_enabled = noep`` must not silently kill memory. In the face of
-ambiguity, refuse to guess.
+Default is **disabled** -- a missing key, blank value, or garbage value
+all leave the kennel off. Only the explicit-truthy tokens
+``{"true", "1", "yes", "on"}`` (case-insensitive) turn it on. That
+asymmetry is on purpose: on a default-off system, a typo like
+``kennel_enabled = yeas`` must not silently enable memory. In the face
+of ambiguity, refuse to guess.
 
 Reads and writes go through ``code_puppy.config.get_value`` /
 ``set_config_value``, the same helpers every other plugin (statusline,
@@ -34,7 +34,7 @@ from code_puppy.config import get_value, set_config_value
 
 _CFG_KEY = "kennel_enabled"
 _ENV_KEY = "PUPPY_KENNEL_ENABLED"
-_FALSY = frozenset({"false", "0", "no", "off"})
+_TRUTHY = frozenset({"true", "1", "yes", "on"})
 
 DISABLED_TOOL_ERROR = (
     "Puppy Kennel memory is currently disabled. "
@@ -45,8 +45,8 @@ DISABLED_TOOL_ERROR = (
 def is_enabled() -> bool:
     """Return True if the kennel is on.
 
-    Defaults to True when the key is missing, blank, or set to anything
-    that isn't one of the explicit-falsy tokens. Evaluated on every call
+    Defaults to False when the key is missing, blank, or set to anything
+    that isn't one of the explicit-truthy tokens. Evaluated on every call
     so flipping the toggle takes effect immediately -- no relaunch
     required.
     """
@@ -54,8 +54,8 @@ def is_enabled() -> bool:
     if raw is None:
         raw = get_value(_CFG_KEY)
     if raw is None:
-        return True
-    return str(raw).strip().lower() not in _FALSY
+        return False
+    return str(raw).strip().lower() in _TRUTHY
 
 
 def set_enabled(value: bool) -> None:

{code_puppy-0.0.714 → code_puppy-0.0.716}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "code-puppy"
-version = "0.0.714"
+version = "0.0.716"
 description = "Code generation agent"
 readme = "README.md"
 requires-python = ">=3.11,<3.15"