pythinker-code 0.8.0__py3-none-any.whl

pythinker_code/web/store/sessions.py

@@ -0,0 +1,433 @@
+"""Session storage with simple in-memory caching for web UI.
+
+## Design Philosophy
+
+This module uses a simple cache-aside pattern with TTL fallback:
+
+1. **Cache on read**: First read populates cache, subsequent reads hit cache
+2. **Invalidate on write**: API mutations call invalidate_sessions_cache()
+3. **TTL fallback**: Cache expires after CACHE_TTL seconds as safety net
+
+## Applicable Scope
+
+This design works well when:
+- Single worker process (e.g., `uvicorn app:app` without -w flag)
+- All mutations go through the same API
+- Occasional staleness (up to CACHE_TTL) from external changes is acceptable
+"""
+
+from __future__ import annotations
+
+import time
+from dataclasses import dataclass
+from datetime import UTC, datetime
+from pathlib import Path
+from uuid import UUID
+
+from pydantic import ConfigDict, Field
+
+from pythinker_code.metadata import WorkDirMeta, load_metadata
+from pythinker_code.session import Session as PythinkerCLISession
+from pythinker_code.session_state import SessionState, load_session_state, save_session_state
+from pythinker_code.utils.logging import logger
+from pythinker_code.web.models import Session
+from pythinker_code.wire.file import WireFile
+
+# Cache configuration
+CACHE_TTL = 5.0  # seconds - balance between freshness and performance
+
+# Auto-archive configuration
+AUTO_ARCHIVE_DAYS = 15  # Sessions older than this will be auto-archived
+
+_sessions_cache: list[JointSession] | None = None
+_cache_timestamp: float = 0.0
+_sessions_index_cache: list[SessionIndexEntry] | None = None
+_index_cache_timestamp: float = 0.0
+
+
+def invalidate_sessions_cache() -> None:
+    """Clear the sessions cache.
+
+    Call this after any mutation (create/update/delete).
+    This ensures the next read sees fresh data.
+    """
+    global _sessions_cache, _cache_timestamp, _sessions_index_cache, _index_cache_timestamp
+    _sessions_cache = None
+    _cache_timestamp = 0.0
+    _sessions_index_cache = None
+    _index_cache_timestamp = 0.0
+
+
+class JointSession(Session):
+    """Combined session model with both web UI and pythinker-code session data."""
+
+    model_config = ConfigDict(arbitrary_types_allowed=True)
+
+    pythinker_code_session: PythinkerCLISession = Field(exclude=True)
+
+
+@dataclass(slots=True)
+class SessionIndexEntry:
+    session_id: UUID
+    session_dir: Path
+    context_file: Path
+    work_dir: str
+    work_dir_meta: WorkDirMeta
+    last_updated: datetime
+    title: str
+    state: SessionState
+
+
+def _derive_title_from_wire(session_dir: Path) -> str:
+    wire_file = session_dir / "wire.jsonl"
+    if not wire_file.exists():
+        return "Untitled"
+
+    try:
+        import json
+
+        from pythinker_core.message import Message
+
+        from pythinker_code.utils.string import shorten
+
+        with open(wire_file, encoding="utf-8") as f:
+            for line in f:
+                line = line.strip()
+                if not line:
+                    continue
+                try:
+                    record = json.loads(line)
+                    message = record.get("message", {})
+                    if message.get("type") == "TurnBegin":
+                        user_input = message.get("payload", {}).get("user_input")
+                        if user_input:
+                            msg = Message(role="user", content=user_input)
+                            text = msg.extract_text(" ")
+                            return shorten(text, width=300)
+                except json.JSONDecodeError:
+                    continue
+    except Exception:
+        logger.debug("Failed to derive session title from context file", exc_info=True)
+    return "Untitled"
+
+
+def _iter_session_dirs(wd: WorkDirMeta) -> list[tuple[Path, Path]]:
+    session_dirs: list[tuple[Path, Path]] = []
+
+    # Latest sessions
+    for context_file in wd.sessions_dir.glob("*/context.jsonl"):
+        session_dir = context_file.parent
+        session_dirs.append((session_dir, context_file))
+
+    # Legacy sessions
+    for context_file in wd.sessions_dir.glob("*.jsonl"):
+        session_dir = context_file.parent / context_file.stem
+        converted_context_file = session_dir / "context.jsonl"
+        if converted_context_file.exists():
+            continue
+        session_dirs.append((session_dir, context_file))
+
+    return session_dirs
+
+
+def _ensure_title(entry: SessionIndexEntry, *, refresh: bool) -> None:
+    """Ensure session has a title.
+
+    Reads title exclusively from SessionState.custom_title (source of truth).
+    Falls back to deriving from wire.jsonl if custom_title is not set.
+    """
+    if entry.state.custom_title:
+        entry.title = entry.state.custom_title
+        return
+
+    if not refresh:
+        if entry.title and entry.title != "Untitled":
+            return
+        entry.title = "Untitled"
+        return
+
+    # Derive title from wire.jsonl (no caching — title will be persisted
+    # to SessionState by the soul layer or generate-title endpoint)
+    entry.title = _derive_title_from_wire(entry.session_dir)
+
+
+def _build_pythinker_session(entry: SessionIndexEntry) -> PythinkerCLISession:
+    from pythinker_host.path import HostPath
+
+    return PythinkerCLISession(
+        id=str(entry.session_id),
+        work_dir=HostPath.unsafe_from_local_path(Path(entry.work_dir)),
+        work_dir_meta=entry.work_dir_meta,
+        context_file=entry.context_file,
+        wire_file=WireFile(entry.session_dir / "wire.jsonl"),
+        state=entry.state,
+        title=entry.title,
+        updated_at=entry.last_updated.timestamp(),
+    )
+
+
+def _build_joint_session(entry: SessionIndexEntry) -> JointSession:
+    pythinker_session = _build_pythinker_session(entry)
+    return JointSession(
+        session_id=entry.session_id,
+        title=entry.title,
+        last_updated=entry.last_updated,
+        is_running=False,
+        status=None,
+        work_dir=entry.work_dir,
+        session_dir=str(entry.session_dir),
+        pythinker_code_session=pythinker_session,
+        archived=entry.state.archived,
+    )
+
+
+def _should_auto_archive(last_updated: datetime, state: SessionState) -> bool:
+    """Check if a session should be auto-archived based on age and exemption status."""
+    if state.archived:
+        return False
+
+    if state.auto_archive_exempt:
+        return False
+
+    now = datetime.now(tz=UTC)
+    age_days = (now - last_updated).days
+    return age_days >= AUTO_ARCHIVE_DAYS
+
+
+def _build_sessions_index() -> list[SessionIndexEntry]:
+    """Build the sessions index from disk.
+
+    Note: This function only reads data and does NOT perform auto-archive writes.
+    Auto-archive is handled separately by run_auto_archive() to avoid disk writes
+    during read operations.
+    """
+    metadata = load_metadata()
+    entries: list[SessionIndexEntry] = []
+
+    for wd in metadata.work_dirs:
+        for session_dir, context_file in _iter_session_dirs(wd):
+            try:
+                session_id = UUID(session_dir.name)
+            except (ValueError, AttributeError, TypeError):
+                continue
+
+            if not context_file.exists():
+                continue
+
+            last_updated = datetime.fromtimestamp(context_file.stat().st_mtime, tz=UTC)
+            state = load_session_state(session_dir)
+            title = state.custom_title or "Untitled"
+
+            entries.append(
+                SessionIndexEntry(
+                    session_id=session_id,
+                    session_dir=session_dir,
+                    context_file=context_file,
+                    work_dir=wd.path,
+                    work_dir_meta=wd,
+                    last_updated=last_updated,
+                    title=title,
+                    state=state,
+                )
+            )
+
+    entries.sort(key=lambda x: (x.last_updated, str(x.session_id)), reverse=True)
+    return entries
+
+
+# Track when auto-archive was last run to avoid running too frequently
+_last_auto_archive_time: float = 0.0
+AUTO_ARCHIVE_INTERVAL = 300.0  # Run auto-archive at most once every 5 minutes
+
+
+def run_auto_archive() -> int:
+    """Run auto-archive on old sessions.
+
+    This function is designed to be called periodically (e.g., on app startup,
+    or via a background task) rather than on every read operation.
+
+    Returns:
+        Number of sessions that were auto-archived.
+    """
+    global _last_auto_archive_time
+
+    now = time.time()
+    if now - _last_auto_archive_time < AUTO_ARCHIVE_INTERVAL:
+        return 0
+
+    _last_auto_archive_time = now
+    archived_count = 0
+
+    # Load fresh index (bypass cache to get current state)
+    entries = _build_sessions_index()
+
+    for entry in entries:
+        if _should_auto_archive(entry.last_updated, entry.state):
+            if not entry.session_dir.is_dir():
+                continue
+            entry.state.archived = True
+            entry.state.archived_at = time.time()
+            save_session_state(entry.state, entry.session_dir)
+            archived_count += 1
+
+    # Invalidate cache if we archived anything
+    if archived_count > 0:
+        invalidate_sessions_cache()
+
+    return archived_count
+
+
+def _load_sessions_index_cached() -> list[SessionIndexEntry]:
+    global _sessions_index_cache, _index_cache_timestamp
+
+    now = time.time()
+    if _sessions_index_cache is not None and (now - _index_cache_timestamp) < CACHE_TTL:
+        return _sessions_index_cache
+
+    _sessions_index_cache = _build_sessions_index()
+    _index_cache_timestamp = now
+    return _sessions_index_cache
+
+
+def load_all_sessions() -> list[JointSession]:
+    """Load all sessions from all work directories."""
+    entries = _load_sessions_index_cached()
+    sessions: list[JointSession] = []
+
+    for entry in entries:
+        _ensure_title(entry, refresh=False)
+        sessions.append(_build_joint_session(entry))
+
+    sessions.sort(key=lambda x: x.last_updated, reverse=True)
+    return sessions
+
+
+def load_all_sessions_cached() -> list[JointSession]:
+    """Cached version of load_all_sessions().
+
+    Returns cached data if:
+    - Cache exists AND
+    - Cache is younger than CACHE_TTL
+
+    Otherwise, refreshes from disk and updates cache.
+    """
+    global _sessions_cache, _cache_timestamp
+
+    now = time.time()
+    if _sessions_cache is not None and (now - _cache_timestamp) < CACHE_TTL:
+        return _sessions_cache
+
+    _sessions_cache = load_all_sessions()
+    _cache_timestamp = now
+    return _sessions_cache
+
+
+def load_sessions_page(
+    *,
+    limit: int = 100,
+    offset: int = 0,
+    query: str | None = None,
+    archived: bool | None = None,
+) -> list[JointSession]:
+    """Load a paginated list of sessions, optionally filtered by query and archived status.
+
+    Args:
+        limit: Maximum number of sessions to return.
+        offset: Number of sessions to skip.
+        query: Optional search query to filter by title or work_dir.
+        archived: Filter by archived status.
+            - None (default): Only return non-archived sessions.
+            - True: Only return archived sessions.
+            - False: Only return non-archived sessions.
+    """
+    entries = list(_load_sessions_index_cached())
+
+    # Filter by archived status
+    if archived is None or archived is False:
+        entries = [e for e in entries if not e.state.archived]
+    else:
+        entries = [e for e in entries if e.state.archived]
+
+    if query:
+        query_text = query.strip().lower()
+        if query_text:
+            for entry in entries:
+                _ensure_title(entry, refresh=True)
+            entries = [
+                entry
+                for entry in entries
+                if query_text in entry.title.lower() or query_text in (entry.work_dir or "").lower()
+            ]
+
+    if offset < 0:
+        offset = 0
+    if limit <= 0:
+        limit = 100
+
+    page_entries = entries[offset : offset + limit]
+
+    if not query:
+        for entry in page_entries:
+            if not entry.title or entry.title == "Untitled":
+                _ensure_title(entry, refresh=True)
+
+    return [_build_joint_session(entry) for entry in page_entries]
+
+
+def load_session_by_id(id: UUID) -> JointSession | None:
+    """Load a session by ID.
+
+    This function first checks the cache/disk scan, then falls back to
+    directly constructing the session from metadata if not found (handles
+    newly created sessions with empty context files).
+    """
+    global_metadata = load_metadata()
+    session_id_str = str(id)
+
+    for wd in global_metadata.work_dirs:
+        session_dir = wd.sessions_dir / session_id_str
+        context_file = session_dir / "context.jsonl"
+
+        if context_file.exists():
+            last_updated = datetime.fromtimestamp(context_file.stat().st_mtime, tz=UTC)
+            state = load_session_state(session_dir)
+            entry = SessionIndexEntry(
+                session_id=id,
+                session_dir=session_dir,
+                context_file=context_file,
+                work_dir=wd.path,
+                work_dir_meta=wd,
+                last_updated=last_updated,
+                title="Untitled",
+                state=state,
+            )
+            _ensure_title(entry, refresh=True)
+            return _build_joint_session(entry)
+
+        # Legacy sessions: context.jsonl stored directly in sessions_dir
+        legacy_context = wd.sessions_dir / f"{session_id_str}.jsonl"
+        if legacy_context.exists():
+            last_updated = datetime.fromtimestamp(legacy_context.stat().st_mtime, tz=UTC)
+            state = load_session_state(session_dir)
+            entry = SessionIndexEntry(
+                session_id=id,
+                session_dir=session_dir,
+                context_file=legacy_context,
+                work_dir=wd.path,
+                work_dir_meta=wd,
+                last_updated=last_updated,
+                title="Untitled",
+                state=state,
+            )
+            _ensure_title(entry, refresh=True)
+            return _build_joint_session(entry)
+
+    return None
+
+
+if __name__ == "__main__":
+    start_time = time.time()
+    sessions = load_all_sessions()
+    print(f"Found {len(sessions)} Sessions in {time.time() - start_time:.2f} seconds:")
+    for session in sessions:
+        print(session.last_updated, session.session_id, session.title)

pythinker_code/wire/__init__.py

@@ -0,0 +1,148 @@
+from __future__ import annotations
+
+import asyncio
+import contextlib
+import copy
+
+from pythinker_core.message import MergeableMixin
+
+from pythinker_code.utils.aioqueue import Queue, QueueShutDown
+from pythinker_code.utils.broadcast import BroadcastQueue
+from pythinker_code.utils.logging import logger
+from pythinker_code.wire.file import WireFile
+from pythinker_code.wire.types import ContentPart, ToolCallPart, WireMessage, is_wire_message
+
+WireMessageQueue = BroadcastQueue[WireMessage]
+
+
+class Wire:
+    """
+    A spmc channel for communication between the soul and the UI during a soul run.
+    """
+
+    def __init__(self, *, file_backend: WireFile | None = None):
+        self._raw_queue = WireMessageQueue()
+        self._merged_queue = WireMessageQueue()
+
+        self._soul_side = WireSoulSide(self._raw_queue, self._merged_queue)
+
+        if file_backend is not None:
+            # record all complete Wire messages to the file backend
+            self._recorder = _WireRecorder(file_backend, self._merged_queue.subscribe())
+        else:
+            self._recorder = None
+
+    @property
+    def soul_side(self) -> WireSoulSide:
+        return self._soul_side
+
+    def ui_side(self, *, merge: bool) -> WireUISide:
+        """
+        Create a UI side of the `Wire`.
+
+        Args:
+            merge: Whether to merge `Wire` messages as much as possible.
+        """
+        if merge:
+            return WireUISide(self._merged_queue.subscribe())
+        else:
+            return WireUISide(self._raw_queue.subscribe())
+
+    def shutdown(self) -> None:
+        self.soul_side.flush()
+        logger.debug("Shutting down wire")
+        self._raw_queue.shutdown()
+        self._merged_queue.shutdown()
+
+    async def join(self) -> None:
+        if self._recorder is None:
+            return
+        try:
+            await self._recorder.join()
+        except Exception:
+            logger.exception("Wire recorder failed to flush:")
+
+
+class WireSoulSide:
+    """
+    The soul side of a `Wire`.
+    """
+
+    def __init__(self, raw_queue: WireMessageQueue, merged_queue: WireMessageQueue):
+        self._raw_queue = raw_queue
+        self._merged_queue = merged_queue
+        self._merge_buffer: MergeableMixin | None = None
+
+    def send(self, msg: WireMessage) -> None:
+        if not isinstance(msg, ContentPart | ToolCallPart):
+            logger.debug("Sending wire message: {msg}", msg=msg)
+
+        # send raw message
+        try:
+            self._raw_queue.publish_nowait(msg)
+        except QueueShutDown:
+            logger.info("Failed to send raw wire message, queue is shut down: {msg}", msg=msg)
+
+        # merge and send merged message
+        match msg:
+            case MergeableMixin():
+                if self._merge_buffer is None:
+                    self._merge_buffer = copy.deepcopy(msg)
+                elif self._merge_buffer.merge_in_place(msg):
+                    pass
+                else:
+                    self.flush()
+                    self._merge_buffer = copy.deepcopy(msg)
+            case _:
+                self.flush()
+                self._send_merged(msg)
+
+    def flush(self) -> None:
+        buffer = self._merge_buffer
+        if buffer is None:
+            return
+        assert is_wire_message(buffer)
+        self._send_merged(buffer)
+        self._merge_buffer = None
+
+    def _send_merged(self, msg: WireMessage) -> None:
+        try:
+            self._merged_queue.publish_nowait(msg)
+        except QueueShutDown:
+            logger.info("Failed to send merged wire message, queue is shut down: {msg}", msg=msg)
+
+
+class WireUISide:
+    """
+    The UI side of a `Wire`.
+    """
+
+    def __init__(self, queue: Queue[WireMessage]):
+        self._queue = queue
+
+    async def receive(self) -> WireMessage:
+        msg = await self._queue.get()
+        if not isinstance(msg, ContentPart | ToolCallPart):
+            logger.debug("Receiving wire message: {msg}", msg=msg)
+        return msg
+
+
+class _WireRecorder:
+    def __init__(self, wire_file: WireFile, queue: Queue[WireMessage]) -> None:
+        self._wire_file = wire_file
+        self._task = asyncio.create_task(self._consume_loop(queue))
+
+    async def join(self) -> None:
+        with contextlib.suppress(asyncio.CancelledError):
+            await self._task
+
+    async def _consume_loop(self, queue: Queue[WireMessage]) -> None:
+        while True:
+            try:
+                msg = await queue.get()
+                await self._record(msg)
+            except QueueShutDown:
+                break
+
+    async def _record(self, msg: WireMessage) -> None:
+        await self._wire_file.append_message(msg)