krons 0.2.1__tar.gz → 0.2.2__tar.gz

{krons-0.2.1 → krons-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: krons
-Version: 0.2.1
+Version: 0.2.2
 Summary: Spec-based composable framework for building type-safe systems
 Project-URL: Homepage, https://github.com/khive-ai/krons
 Project-URL: Repository, https://github.com/khive-ai/krons

{krons-0.2.1 → krons-0.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "krons"
-version = "0.2.1"
+version = "0.2.2"
 description = "Spec-based composable framework for building type-safe systems"
 readme = "README.md"
 requires-python = ">=3.11"

{krons-0.2.1 → krons-0.2.2}/src/krons/core/__init__.py

@@ -21,6 +21,7 @@ _LAZY_IMPORTS: dict[str, tuple[str, str]] = {
     "PERSISTABLE_NODE_REGISTRY": ("krons.core.base", "PERSISTABLE_NODE_REGISTRY"),
     # Classes
     "Broadcaster": ("krons.core.base", "Broadcaster"),
+    "DataLoggerConfig": ("krons.core.base", "DataLoggerConfig"),
     "Edge": ("krons.core.base", "Edge"),
     "EdgeCondition": ("krons.core.base", "EdgeCondition"),
     "Element": ("krons.core.base", "Element"),
@@ -75,6 +76,7 @@ if TYPE_CHECKING:
         NODE_REGISTRY,
         PERSISTABLE_NODE_REGISTRY,
         Broadcaster,
+        DataLoggerConfig,
         Edge,
         EdgeCondition,
         Element,
@@ -103,6 +105,7 @@ __all__ = [
     "PERSISTABLE_NODE_REGISTRY",
     # classes
     "Broadcaster",
+    "DataLoggerConfig",
     "Edge",
     "EdgeCondition",
     "Element",

{krons-0.2.1 → krons-0.2.2}/src/krons/core/base/__init__.py

@@ -26,6 +26,8 @@ _LAZY_IMPORTS: dict[str, tuple[str, str]] = {
     "Edge": ("krons.core.base.graph", "Edge"),
     "EdgeCondition": ("krons.core.base.graph", "EdgeCondition"),
     "Graph": ("krons.core.base.graph", "Graph"),
+    # log
+    "DataLoggerConfig": ("krons.core.base.log", "DataLoggerConfig"),
     # node
     "NODE_REGISTRY": ("krons.core.base.node", "NODE_REGISTRY"),
     "PERSISTABLE_NODE_REGISTRY": ("krons.core.base.node", "PERSISTABLE_NODE_REGISTRY"),
@@ -77,6 +79,7 @@ if TYPE_CHECKING:
     from .eventbus import EventBus, Handler
     from .flow import Flow
     from .graph import Edge, EdgeCondition, Graph
+    from .log import DataLoggerConfig
     from .node import (
         NODE_REGISTRY,
         PERSISTABLE_NODE_REGISTRY,
@@ -97,6 +100,7 @@ __all__ = [
     "PERSISTABLE_NODE_REGISTRY",
     # classes
     "Broadcaster",
+    "DataLoggerConfig",
     "Edge",
     "EdgeCondition",
     "Element",

krons-0.2.2/src/krons/core/base/log.py

@@ -0,0 +1,32 @@
+# Copyright (c) 2025 - 2026, HaiyangLi <quantocean.li at gmail dot com>
+# SPDX-License-Identifier: Apache-2.0
+
+"""Logging configuration for Session message persistence.
+
+Provides DataLoggerConfig for configuring automatic message dumps.
+"""
+
+from __future__ import annotations
+
+from pathlib import Path
+from typing import Literal
+
+from pydantic import Field
+
+from krons.core.types import HashableModel
+
+__all__ = ("DataLoggerConfig",)
+
+
+class DataLoggerConfig(HashableModel):
+    """Configuration for Session message persistence.
+
+    Attributes:
+        persist_dir: Directory for dump files.
+        extension: Output format (.json array or .jsonl newline-delimited).
+        auto_save_on_exit: Register atexit handler on Session creation.
+    """
+
+    persist_dir: str | Path = Field(default="./logs")
+    extension: Literal[".json", ".jsonl"] = Field(default=".jsonl")
+    auto_save_on_exit: bool = Field(default=True)

{krons-0.2.1 → krons-0.2.2}/src/krons/session/session.py

@@ -9,12 +9,16 @@ Branch is a named message progression with capability/resource access control.
 
 from __future__ import annotations
 
+import atexit
 import contextlib
 from collections.abc import AsyncGenerator, Iterable
+from pathlib import Path
 from typing import Any, Literal
+
+from krons.core.base.log import DataLoggerConfig
 from uuid import UUID
 
-from pydantic import Field, model_validator
+from pydantic import Field, PrivateAttr, model_validator
 
 from krons.core import Element, Flow, Pile, Progression
 from krons.core.types import HashableModel, Unset, UnsetType, not_sentinel
@@ -59,6 +63,10 @@ class SessionConfig(HashableModel):
     default_gen_model: str | None = None
     default_parse_model: str | None = None
     auto_create_default_branch: bool = True
+    log_config: DataLoggerConfig | None = Field(
+        default=None,
+        description="DataLoggerConfig for auto-logging (None = disabled)",
+    )
 
 
 class Session(Element):
@@ -71,6 +79,9 @@ class Session(Element):
     config: SessionConfig = Field(default_factory=SessionConfig)
     default_branch_id: UUID | None = None
 
+    _registered_atexit: bool = PrivateAttr(default=False)
+    _dump_count: int = PrivateAttr(default=0)
+
     @model_validator(mode="after")
     def _validate_default_branch(self) -> Session:
         """Auto-create default branch and register built-in operations."""
@@ -83,6 +94,15 @@ class Session(Element):
             )
             self.set_default_branch(default_branch_name)
 
+        # Register atexit handler if configured
+        if (
+            self.config.log_config is not None
+            and self.config.log_config.auto_save_on_exit
+            and not self._registered_atexit
+        ):
+            atexit.register(self._save_at_exit)
+            self._registered_atexit = True
+
         # Register built-in operations (lazy import avoids circular)
         from krons.agent.operations import (
             generate,
@@ -386,6 +406,99 @@ class Session(Element):
         async for result in handler(params, ctx):
             yield result
 
+    def dump_messages(self, clear: bool = False) -> Path | None:
+        """Sync dump all session messages to file.
+
+        Args:
+            clear: Clear messages after dump (default False).
+
+        Returns:
+            Path to written file, or None if no log_config or no messages.
+        """
+        from krons.utils import create_path, json_dumpb, json_lines_iter
+        from krons.utils.concurrency import run_async
+
+        if self.config.log_config is None or len(self.messages) == 0:
+            return None
+
+        cfg = self.config.log_config
+        self._dump_count += 1
+
+        filepath = run_async(
+            create_path(
+                directory=cfg.persist_dir,
+                filename=f"session_{str(self.id)[:8]}_{self._dump_count}",
+                extension=cfg.extension,
+                timestamp=True,
+                file_exist_ok=True,
+            )
+        )
+
+        items = [msg.to_dict(mode="json") for msg in self.messages]
+
+        std_path = Path(filepath)
+        if cfg.extension == ".jsonl":
+            with std_path.open("wb") as f:
+                for chunk in json_lines_iter(items, safe_fallback=True):
+                    f.write(chunk)
+        else:
+            data = json_dumpb(items, safe_fallback=True)
+            std_path.write_bytes(data)
+
+        if clear:
+            self.communications.items.clear()
+
+        return std_path
+
+    async def adump_messages(self, clear: bool = False) -> Path | None:
+        """Async dump all session messages to file with lock protection.
+
+        Args:
+            clear: Clear messages after dump (default False).
+
+        Returns:
+            Path to written file, or None if no log_config or no messages.
+        """
+        from krons.utils import create_path, json_dumpb, json_lines_iter
+
+        if self.config.log_config is None or len(self.messages) == 0:
+            return None
+
+        cfg = self.config.log_config
+
+        async with self.messages:
+            self._dump_count += 1
+
+            filepath = await create_path(
+                directory=cfg.persist_dir,
+                filename=f"session_{str(self.id)[:8]}_{self._dump_count}",
+                extension=cfg.extension,
+                timestamp=True,
+                file_exist_ok=True,
+            )
+
+            items = [msg.to_dict(mode="json") for msg in self.messages]
+
+            if cfg.extension == ".jsonl":
+                content = b"".join(json_lines_iter(items, safe_fallback=True))
+                await filepath.write_bytes(content)
+            else:
+                data = json_dumpb(items, safe_fallback=True)
+                await filepath.write_bytes(data)
+
+            if clear:
+                self.communications.items.clear()
+
+        return Path(filepath)
+
+    def _save_at_exit(self) -> None:
+        """atexit callback. Dumps messages synchronously. Errors are suppressed."""
+        if len(self.messages) > 0:
+            try:
+                self.dump_messages(clear=False)
+            except Exception:
+                pass  # Silent failure during interpreter shutdown
+
     def _resolve_branch(self, branch: Branch | UUID | str | None) -> Branch:
         """Resolve to Branch, falling back to default. Raises if neither available."""
         if branch is not None:

krons-0.2.2/tests/core/test_log.py

@@ -0,0 +1,44 @@
+# Copyright (c) 2025 - 2026, HaiyangLi <quantocean.li at gmail dot com>
+# SPDX-License-Identifier: Apache-2.0
+
+"""Tests for krons.core.base.log - DataLoggerConfig."""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+
+from krons.core.base.log import DataLoggerConfig
+
+
+class TestDataLoggerConfig:
+    """Tests for DataLoggerConfig."""
+
+    def test_defaults(self):
+        """Config should have sensible defaults."""
+        config = DataLoggerConfig()
+        assert config.persist_dir == "./logs"
+        assert config.extension == ".jsonl"
+        assert config.auto_save_on_exit is True
+
+    def test_custom_values(self):
+        """Config should accept custom values."""
+        config = DataLoggerConfig(
+            persist_dir="/tmp/custom_logs",
+            extension=".json",
+            auto_save_on_exit=False,
+        )
+        assert config.persist_dir == "/tmp/custom_logs"
+        assert config.extension == ".json"
+        assert config.auto_save_on_exit is False
+
+    def test_extension_validation(self):
+        """Config should only accept .json or .jsonl."""
+        with pytest.raises(Exception):
+            DataLoggerConfig(extension=".csv")
+
+    def test_persist_dir_as_path(self):
+        """Config should accept Path object for persist_dir."""
+        config = DataLoggerConfig(persist_dir=Path("/tmp/logs"))
+        assert config.persist_dir == Path("/tmp/logs")

krons-0.2.2/tests/session/test_session_logging.py

@@ -0,0 +1,337 @@
+# Copyright (c) 2025 - 2026, HaiyangLi <quantocean.li at gmail dot com>
+# SPDX-License-Identifier: Apache-2.0
+
+"""Tests for Session message persistence."""
+
+from __future__ import annotations
+
+import json
+
+import pytest
+
+from krons.core.base.log import DataLoggerConfig
+from krons.session import Session, SessionConfig
+from krons.session.message import Message
+
+
+class TestSessionMessageDump:
+    """Tests for Session.dump_messages and adump_messages."""
+
+    def test_dump_messages_no_config(self):
+        """dump_messages should return None when log_config is None."""
+        session = Session()
+        assert session.dump_messages() is None
+
+    def test_dump_messages_no_messages(self, tmp_path):
+        """dump_messages should return None when no messages."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+        # Clear default branch messages if any
+        session.communications.items.clear()
+        assert session.dump_messages() is None
+
+    def test_dump_messages_jsonl(self, tmp_path):
+        """dump_messages should write JSONL file."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                extension=".jsonl",
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg1 = Message(content={"role": "user", "text": "Hello"})
+        msg2 = Message(content={"role": "assistant", "text": "Hi there"})
+        session.add_message(msg1)
+        session.add_message(msg2)
+
+        filepath = session.dump_messages()
+
+        assert filepath is not None
+        assert filepath.exists()
+        assert filepath.suffix == ".jsonl"
+
+        lines = filepath.read_text().strip().split("\n")
+        assert len(lines) == 2
+
+        first = json.loads(lines[0])
+        assert first["content"]["role"] == "user"
+
+    def test_dump_messages_json(self, tmp_path):
+        """dump_messages should write JSON array file."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                extension=".json",
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"test": "data"})
+        session.add_message(msg)
+
+        filepath = session.dump_messages()
+
+        assert filepath is not None
+        assert filepath.suffix == ".json"
+
+        data = json.loads(filepath.read_bytes())
+        assert isinstance(data, list)
+        assert len(data) == 1
+        assert data[0]["content"]["test"] == "data"
+
+    def test_dump_messages_no_clear_by_default(self, tmp_path):
+        """dump_messages should NOT clear messages by default."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"keep": "me"})
+        session.add_message(msg)
+
+        session.dump_messages()
+        assert len(session.messages) == 1
+
+    def test_dump_messages_with_clear(self, tmp_path):
+        """dump_messages(clear=True) should clear messages."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"delete": "me"})
+        session.add_message(msg)
+
+        session.dump_messages(clear=True)
+        assert len(session.messages) == 0
+
+    def test_dump_creates_directory(self, tmp_path):
+        """dump_messages should create persist_dir if needed."""
+        nested = tmp_path / "sub" / "dir"
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(nested),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"test": 1})
+        session.add_message(msg)
+
+        filepath = session.dump_messages()
+        assert filepath is not None
+        assert nested.exists()
+
+    def test_atexit_registration(self):
+        """Session should register atexit when configured."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(auto_save_on_exit=True)
+        )
+        session = Session(config=config)
+        assert session._registered_atexit is True
+
+    def test_no_atexit_when_disabled(self):
+        """Session should not register atexit when disabled."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(auto_save_on_exit=False)
+        )
+        session = Session(config=config)
+        assert session._registered_atexit is False
+
+    def test_no_atexit_when_no_config(self):
+        """Session should not register atexit when no log_config."""
+        session = Session()
+        assert session._registered_atexit is False
+
+    def test_multiple_dumps(self, tmp_path):
+        """Multiple dumps should create separate files."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg1 = Message(content={"batch": 1})
+        session.add_message(msg1)
+        p1 = session.dump_messages()
+
+        msg2 = Message(content={"batch": 2})
+        session.add_message(msg2)
+        p2 = session.dump_messages()
+
+        assert p1 != p2
+        assert p1.exists()
+        assert p2.exists()
+
+    def test_save_at_exit_with_messages(self, tmp_path):
+        """_save_at_exit should dump messages when present."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"atexit": "test"})
+        session.add_message(msg)
+
+        # Call directly to test the method
+        session._save_at_exit()
+
+        files = list(tmp_path.glob("*.jsonl"))
+        assert len(files) == 1
+
+    def test_save_at_exit_no_messages(self, tmp_path):
+        """_save_at_exit should not create file when no messages."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+        session.communications.items.clear()
+
+        session._save_at_exit()
+
+        files = list(tmp_path.glob("*"))
+        assert len(files) == 0
+
+    def test_save_at_exit_suppresses_errors(self, tmp_path):
+        """_save_at_exit should suppress exceptions silently."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir="/nonexistent/readonly/path",
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"test": 1})
+        session.add_message(msg)
+
+        # Should not raise even with invalid path
+        session._save_at_exit()  # No exception = pass
+
+
+class TestSessionMessageDumpAsync:
+    """Tests for Session.adump_messages."""
+
+    @pytest.mark.anyio
+    async def test_adump_messages_jsonl(self, tmp_path):
+        """adump_messages should write JSONL file."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                extension=".jsonl",
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"async": True})
+        session.add_message(msg)
+
+        filepath = await session.adump_messages()
+
+        assert filepath is not None
+        assert filepath.exists()
+
+        lines = filepath.read_text().strip().split("\n")
+        first = json.loads(lines[0])
+        assert first["content"]["async"] is True
+
+    @pytest.mark.anyio
+    async def test_adump_messages_json(self, tmp_path):
+        """adump_messages should write JSON array file."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                extension=".json",
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"format": "json"})
+        session.add_message(msg)
+
+        filepath = await session.adump_messages()
+
+        assert filepath is not None
+        data = json.loads(filepath.read_bytes())
+        assert data[0]["content"]["format"] == "json"
+
+    @pytest.mark.anyio
+    async def test_adump_no_config(self):
+        """adump_messages should return None when no log_config."""
+        session = Session()
+        assert await session.adump_messages() is None
+
+    @pytest.mark.anyio
+    async def test_adump_no_messages(self, tmp_path):
+        """adump_messages should return None when no messages."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+        session.communications.items.clear()
+
+        assert await session.adump_messages() is None
+
+    @pytest.mark.anyio
+    async def test_adump_with_clear(self, tmp_path):
+        """adump_messages(clear=True) should clear messages."""
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(tmp_path),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"clear": "me"})
+        session.add_message(msg)
+
+        await session.adump_messages(clear=True)
+        assert len(session.messages) == 0
+
+    @pytest.mark.anyio
+    async def test_adump_creates_directory(self, tmp_path):
+        """adump_messages should create persist_dir if needed."""
+        nested = tmp_path / "async" / "logs"
+        config = SessionConfig(
+            log_config=DataLoggerConfig(
+                persist_dir=str(nested),
+                auto_save_on_exit=False,
+            )
+        )
+        session = Session(config=config)
+
+        msg = Message(content={"test": 1})
+        session.add_message(msg)
+
+        filepath = await session.adump_messages()
+        assert filepath is not None
+        assert nested.exists()