bazaar-compute-node 0.1.10__tar.gz → 0.1.12__tar.gz

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bazaar-compute-node
-Version: 0.1.10
+Version: 0.1.12
 Summary: Runtime-agnostic computer node daemon for agents and channels.
 License-Expression: AGPL-3.0-only
 License-File: LICENSE
@@ -67,7 +67,7 @@ bcn stop
 | --- | --- |
 | ✅ | Reminders / 定时器 |
 | ✅ | Attachments / 附件 |
-| 🚧 | Multi-Agents / 多 Agent |
+| ✅ | Multi-Agents / 多 Agent |
 | 🚧 | Teams / 团队协作 |
 
 ### Channels / 渠道

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/README.md

@@ -53,7 +53,7 @@ bcn stop
 | --- | --- |
 | ✅ | Reminders / 定时器 |
 | ✅ | Attachments / 附件 |
-| 🚧 | Multi-Agents / 多 Agent |
+| ✅ | Multi-Agents / 多 Agent |
 | 🚧 | Teams / 团队协作 |
 
 ### Channels / 渠道

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "bazaar-compute-node"
-version = "0.1.10"
+version = "0.1.12"
 description = "Runtime-agnostic computer node daemon for agents and channels."
 readme = "README.md"
 license = "AGPL-3.0-only"
@@ -35,6 +35,7 @@ logging = "bazaar_compute_node.contrib.logging.plugin:create_audit"
 [dependency-groups]
 dev = [
     "bcn-test-support",
+    "pyright==1.1.408",
     "pytest>=8.3.0",
     "pytest-asyncio>=0.24.0",
     "ruff>=0.9.0",

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "bazaar-compute-node"
-version = "0.1.10"
+version = "0.1.12"
 description = "Runtime-agnostic computer node daemon for agents and channels."
 readme = "README.md"
 license = "AGPL-3.0-only"
@@ -35,6 +35,7 @@ logging = "bazaar_compute_node.contrib.logging.plugin:create_audit"
 [dependency-groups]
 dev = [
     "bcn-test-support",
+    "pyright==1.1.408",
     "pytest>=8.3.0",
     "pytest-asyncio>=0.24.0",
     "ruff>=0.9.0",

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/app/agent.py

@@ -101,6 +101,7 @@ class AgentApplication:
                 attachments=self._attachment_materializer,
                 options=dict(configuration.channel.options),
                 workspace=self.workspace_path,
+                translator=self.translator,
             )
         )
         self.channel: IChannel = AgentScopedChannel(self.agent_id, provider_channel)
@@ -443,8 +444,8 @@ class AgentApplication:
 
     def _error_feedback_detail(
         self,
-        session_id: str,
-        error_message: str,
+        session_id: object,
+        error_message: object,
     ) -> str:
         if not isinstance(session_id, str) or not session_id:
             raise ValueError("session_id must be a non-empty string")
@@ -490,7 +491,7 @@ class AgentApplication:
         )
         input_data = body.encode() if body is not None else None
         try:
-            _stdout, stderr = await process.communicate(input=input_data)
+            _, stderr = await process.communicate(input=input_data)
         except asyncio.CancelledError:
             if process.returncode is None:
                 process.terminate()

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/app/transport.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 import asyncio
 import json
 import os
+import platform
 import secrets
 import sys
 import tempfile
@@ -223,7 +224,7 @@ class LocalCommandClient:
         parsed = urlsplit(endpoint)
         request = dict(payload)
         if parsed.scheme == "pipe":
-            if sys.platform != "win32":
+            if platform.system() != "Windows":
                 raise ValueError(
                     "Windows named pipe endpoints are not supported on this platform"
                 )

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/app/wrapper.py

@@ -32,11 +32,15 @@ def install_bcc_wrapper(bin_dir: Path, *, agent_id: str) -> Path:
             command_path.write_text(
                 "@echo off\n"
                 f'set "BCN_AGENT_ID={agent_id}"\n'
+                'set "PYTHONUTF8=1"\n'
+                'set "PYTHONIOENCODING=utf-8"\n'
                 f'"{python_executable}" -m bazaar_compute_node.bcc %*\n',
                 encoding="utf-8",
             )
             (bin_dir / "bcc.ps1").write_text(
                 f"$env:BCN_AGENT_ID = '{agent_id}'\n"
+                "$env:PYTHONUTF8 = '1'\n"
+                "$env:PYTHONIOENCODING = 'utf-8'\n"
                 f'& "{python_executable}" -m bazaar_compute_node.bcc @args\n'
                 "exit $LASTEXITCODE\n",
                 encoding="utf-8",

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/cli.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 import argparse
 import asyncio
 import os
+import platform
 import subprocess
 import sys
 from collections.abc import Mapping, Sequence
@@ -334,7 +335,9 @@ async def _start_daemon(
     await asyncio.to_thread(data_dir.mkdir, parents=True, exist_ok=True)
     endpoint_path = _endpoint_path(args, data_dir)
     endpoint = local_endpoint_for_path(endpoint_path)
-    if os.name == "nt" and await _endpoint_is_reachable(endpoint, timeout=0.5):
+    if platform.system() == "Windows" and await _endpoint_is_reachable(
+        endpoint, timeout=0.5
+    ):
         parser.error("bcn is already running")
     if os.name != "nt" and await asyncio.to_thread(endpoint_path.exists):
         parser.error(f"bcn endpoint already exists: {endpoint_path}")

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/codex/client.py

@@ -63,7 +63,7 @@ def build_initialize_params(
     }
 
 
-def build_fs_watch_params(path: Path, watch_id: str) -> dict[str, object]:
+def build_fs_watch_params(path: object, watch_id: str) -> dict[str, object]:
     if not isinstance(path, Path):
         raise TypeError("path must be a Path")
     if not path.is_absolute():
@@ -78,7 +78,7 @@ def build_thread_start_params(
     model: str | None = None,
     approval_policy: str | None = None,
     cwd: Path | None = None,
-    ephemeral: bool | None = None,
+    ephemeral: object = None,
 ) -> dict[str, object]:
     """Build provider-local parameters for a Codex ``thread/start`` request."""
 
@@ -509,7 +509,7 @@ def _optional_text(value: object, field_name: str) -> str | None:
     return value
 
 
-def _path_text(value: Path, field_name: str) -> str:
+def _path_text(value: object, field_name: str) -> str:
     if not isinstance(value, Path):
         raise TypeError(f"{field_name} must be a Path")
     return str(value)

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/codex/process.py

@@ -253,8 +253,8 @@ class JsonlProcessSupervisor:
         self,
         request_id: JsonlRequestId,
         *,
-        result: Mapping[str, object] | None = None,
-        error: Mapping[str, object] | None = None,
+        result: object = None,
+        error: object = None,
         timeout: float,
     ) -> None:
         """Write one response to a provider-initiated JSON-RPC request."""

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/codex/protocol.py

@@ -76,7 +76,7 @@ class AppServerProtocolError(ValueError):
     """The Codex App Server response does not match its provider contract."""
 
 
-def validate_message(payload: Mapping[str, object]) -> JsonlMessage:
+def validate_message(payload: object) -> JsonlMessage:
     """Copy a mapping into the transport's provider-local message shape."""
 
     if not isinstance(payload, Mapping):

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/codex/runtime.py

@@ -120,7 +120,7 @@ class Runtime(IRuntime, IAsyncLifecycle):
         self._stopping = True
         connections = tuple(self._connections.items())
         self._connections.clear()
-        for _session_id, connection in connections:
+        for _, connection in connections:
             try:
                 await connection.supervisor.stop(timeout=timeout)
             except asyncio.CancelledError:

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/sqlite/codec.py

@@ -22,7 +22,7 @@ from ...core.models import (
 )
 
 
-def _channel_session_from_row(row: aiosqlite.Row) -> ChannelSession:
+def channel_session_from_row(row: aiosqlite.Row) -> ChannelSession:
     following = row["following"]
     if (
         isinstance(following, bool)
@@ -54,7 +54,7 @@ def _channel_session_from_row(row: aiosqlite.Row) -> ChannelSession:
     )
 
 
-def _bcn_session_from_row(row: aiosqlite.Row) -> BcnSession:
+def bcn_session_from_row(row: aiosqlite.Row) -> BcnSession:
     return BcnSession(
         id=_required_text(row["id"], "id"),
         channel_session_id=_required_text(
@@ -70,7 +70,7 @@ def _bcn_session_from_row(row: aiosqlite.Row) -> BcnSession:
     )
 
 
-def _runtime_attempt_from_row(row: aiosqlite.Row) -> RuntimeAttempt:
+def runtime_attempt_from_row(row: aiosqlite.Row) -> RuntimeAttempt:
     return RuntimeAttempt(
         turn_id=_required_text(row["turn_id"], "turn_id"),
         session_id=_required_text(row["session_id"], "session_id"),
@@ -81,7 +81,7 @@ def _runtime_attempt_from_row(row: aiosqlite.Row) -> RuntimeAttempt:
     )
 
 
-def _inbound_message_from_row(
+def inbound_message_from_row(
     row: aiosqlite.Row,
     attachments: tuple[InboundAttachment, ...] = (),
 ) -> InboundMessage:
@@ -133,7 +133,7 @@ def _inbound_message_from_row(
     )
 
 
-def _inbound_attachment_from_row(row: aiosqlite.Row) -> InboundAttachment:
+def inbound_attachment_from_row(row: aiosqlite.Row) -> InboundAttachment:
     return InboundAttachment(
         attachment_id=_required_text(row["attachment_id"], "attachment_id"),
         name=_required_text(row["name"], "attachment.name"),
@@ -148,7 +148,7 @@ def _inbound_attachment_from_row(row: aiosqlite.Row) -> InboundAttachment:
     )
 
 
-def _outbound_message_from_row(row: aiosqlite.Row) -> OutboundMessage:
+def outbound_message_from_row(row: aiosqlite.Row) -> OutboundMessage:
     raw_attachments = row["attachments_json"]
     if not isinstance(raw_attachments, str):
         raise TypeError("attachments_json must be text")
@@ -231,7 +231,7 @@ def _outbound_message_from_row(row: aiosqlite.Row) -> OutboundMessage:
     )
 
 
-def _consumer_cursor_from_row(row: aiosqlite.Row) -> ConsumerCursor:
+def consumer_cursor_from_row(row: aiosqlite.Row) -> ConsumerCursor:
     return ConsumerCursor(
         session_id=_required_text(row["session_id"], "session_id"),
         delivered_through_seq=_required_non_negative_int(
@@ -258,12 +258,12 @@ def _consumer_cursor_from_row(row: aiosqlite.Row) -> ConsumerCursor:
     )
 
 
-def _validate_inbound_message_input(message: InboundMessage) -> None:
+def validate_inbound_message_input(message: object) -> None:
     if not isinstance(message, InboundMessage):
         raise TypeError("message must be an InboundMessage")
 
 
-def _validate_outbound_message_input(message: OutboundMessage) -> None:
+def validate_outbound_message_input(message: object) -> None:
     if not isinstance(message, OutboundMessage):
         raise TypeError("message must be an OutboundMessage")
     if not isinstance(message.state, OutboundDeliveryState):
@@ -369,7 +369,7 @@ def _validate_outbound_message_input(message: OutboundMessage) -> None:
         raise ValueError("delivered outbound message requires a provider receipt")
 
 
-def _validate_outbound_insert(message: OutboundMessage) -> None:
+def validate_outbound_insert(message: OutboundMessage) -> None:
     if message.state is not OutboundDeliveryState.DRAFT:
         raise ValueError("a new outbound message must start in draft state")
     if message.fresh_check_state is not FreshCheckState.REQUIRED:
@@ -387,7 +387,7 @@ def _validate_outbound_insert(message: OutboundMessage) -> None:
         raise ValueError("a new outbound draft cannot contain delivery timestamps")
 
 
-def _validate_outbound_update(
+def validate_outbound_update(
     existing: OutboundMessage,
     incoming: OutboundMessage,
 ) -> OutboundMessage:
@@ -496,7 +496,7 @@ def _validate_optional_input_text(value: object, field_name: str) -> None:
         raise ValueError(f"{field_name} must be a non-empty string when present")
 
 
-def _validate_consumer_cursor_input(cursor: ConsumerCursor) -> None:
+def validate_consumer_cursor_input(cursor: object) -> None:
     if not isinstance(cursor, ConsumerCursor):
         raise TypeError("cursor must be a ConsumerCursor")
     source = cursor.inbox_snapshot_source
@@ -511,14 +511,14 @@ def _validate_consumer_cursor_input(cursor: ConsumerCursor) -> None:
         raise ValueError("inbox snapshot sequence requires a snapshot time")
 
 
-def _validate_cursor_bounds(cursor: ConsumerCursor, latest_seq: int) -> None:
+def validate_cursor_bounds(cursor: ConsumerCursor, latest_seq: int) -> None:
     if cursor.delivered_through_seq > latest_seq:
         raise ValueError("delivered cursor cannot exceed the latest inbound sequence")
     if cursor.inbox_snapshot_seq is not None and cursor.inbox_snapshot_seq > latest_seq:
         raise ValueError("inbox snapshot cannot exceed the latest inbound sequence")
 
 
-def _validate_consumer_cursor_update(
+def validate_consumer_cursor_update(
     existing: ConsumerCursor,
     incoming: ConsumerCursor,
 ) -> None:
@@ -553,12 +553,12 @@ def _validate_consumer_cursor_update(
             raise ValueError(f"{field_name} cannot move backwards")
 
 
-def _validate_channel_session_input(session: ChannelSession) -> None:
+def validate_channel_session_input(session: ChannelSession) -> None:
     if not isinstance(session.following, bool):
         raise TypeError("channel session following must be a boolean")
 
 
-def _validate_channel_session_update(
+def validate_channel_session_update(
     existing: ChannelSession,
     incoming: ChannelSession,
 ) -> ChannelSession:
@@ -580,7 +580,7 @@ def _validate_channel_session_update(
     )
 
 
-def _validate_bcn_session_update(
+def validate_bcn_session_update(
     existing: BcnSession,
     incoming: BcnSession,
 ) -> BcnSession:
@@ -608,15 +608,15 @@ def _required_text(value: object, field_name: str) -> str:
     return _string_value(value, field_name, allow_empty=False)
 
 
-def _validate_non_empty_text(value: object, field_name: str) -> None:
+def validate_non_empty_text(value: object, field_name: str) -> None:
     _required_text(value, field_name)
 
 
-def _validate_non_negative_int(value: object, field_name: str) -> None:
+def validate_non_negative_int(value: object, field_name: str) -> None:
     _required_non_negative_int(value, field_name)
 
 
-def _validate_positive_int(value: object, field_name: str) -> None:
+def validate_positive_int(value: object, field_name: str) -> None:
     _required_positive_int(value, field_name)
 
 
@@ -669,7 +669,7 @@ def _optional_non_negative_int(value: object, field_name: str) -> int | None:
     return _required_non_negative_int(value, field_name)
 
 
-def _encode_metadata(metadata: Mapping[str, object]) -> str:
+def encode_metadata(metadata: object) -> str:
     if not isinstance(metadata, Mapping):
         raise TypeError("metadata must be a mapping")
     if any(not isinstance(key, str) for key in metadata):

{bazaar_compute_node-0.1.10 → bazaar_compute_node-0.1.12}/src/bazaar_compute_node/contrib/sqlite/reminder_repository.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 from dataclasses import replace
+from typing import cast
 from uuid import uuid7
 
 from ...core.models import (
@@ -14,12 +15,12 @@ from ...core.models import (
 )
 from ...core.reminder import canonical_id_reference
 from .codec import (
-    _inbound_message_from_row,
     _required_non_negative_int,
     _required_text,
-    _validate_non_empty_text,
-    _validate_non_negative_int,
-    _validate_positive_int,
+    inbound_message_from_row,
+    validate_non_empty_text,
+    validate_non_negative_int,
+    validate_positive_int,
 )
 from .reminder_codec import reminder_from_row, reminder_occurrence_from_row
 from .repository import SqliteTransaction
@@ -50,7 +51,7 @@ class ReminderTransaction(SqliteTransaction):
         session_id: str,
         message_id_or_prefix: str,
     ) -> InboundMessage | None:
-        _validate_non_empty_text(session_id, "session_id")
+        validate_non_empty_text(session_id, "session_id")
         reference = canonical_id_reference(message_id_or_prefix)
         if len(reference) == 8:
             rows = await self.fetchall(
@@ -69,16 +70,14 @@ class ReminderTransaction(SqliteTransaction):
             )
         if row is None:
             return None
-        return _inbound_message_from_row(
-            row, await self._attachments(row["message_id"])
-        )
+        return inbound_message_from_row(row, await self._attachments(row["message_id"]))
 
     async def get_reminder(
         self,
         owner_session_id: str,
         reminder_id_or_prefix: str,
     ) -> Reminder | None:
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
         reference = canonical_id_reference(reminder_id_or_prefix)
         if len(reference) == 8:
             rows = await self.fetchall(
@@ -103,7 +102,7 @@ class ReminderTransaction(SqliteTransaction):
         owner_session_id: str,
         statuses: frozenset[ReminderState],
     ) -> tuple[Reminder, ...]:
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
         if not isinstance(statuses, frozenset) or not statuses:
             raise ValueError("statuses must be a non-empty frozenset")
         if not all(isinstance(status, ReminderState) for status in statuses):
@@ -118,7 +117,7 @@ class ReminderTransaction(SqliteTransaction):
         )
         return tuple(reminder_from_row(row) for row in rows)
 
-    async def save_new_reminder(self, reminder: Reminder) -> Reminder:
+    async def save_new_reminder(self, reminder: object) -> Reminder:
         if not isinstance(reminder, Reminder):
             raise TypeError("reminder must be a Reminder")
         if reminder.state is not ReminderState.SCHEDULED:
@@ -166,9 +165,9 @@ class ReminderTransaction(SqliteTransaction):
     async def save_reminder_transition(
         self,
         expected_revision: int,
-        reminder: Reminder,
+        reminder: object,
     ) -> Reminder:
-        _validate_positive_int(expected_revision, "expected_revision")
+        validate_positive_int(expected_revision, "expected_revision")
         if not isinstance(reminder, Reminder):
             raise TypeError("reminder must be a Reminder")
         existing = await self.get_reminder(
@@ -216,8 +215,8 @@ class ReminderTransaction(SqliteTransaction):
         *,
         limit: int,
     ) -> tuple[Reminder, ...]:
-        _validate_non_negative_int(now_ms, "now_ms")
-        _validate_positive_int(limit, "limit")
+        validate_non_negative_int(now_ms, "now_ms")
+        validate_positive_int(limit, "limit")
         rows = await self.fetchall(
             f"SELECT {_REMINDER_COLUMNS} FROM reminders "
             "WHERE state = ? AND next_fire_at_ms <= ? "
@@ -232,9 +231,9 @@ class ReminderTransaction(SqliteTransaction):
         owner_session_id: str,
         reminder_id: str,
     ) -> OwnedReminder | None:
-        _validate_non_empty_text(agent_id, "agent_id")
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
-        _validate_non_empty_text(reminder_id, "reminder_id")
+        validate_non_empty_text(agent_id, "agent_id")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_non_empty_text(reminder_id, "reminder_id")
         bound_agent_id = self._bound_agent_id()
         if bound_agent_id is not None and bound_agent_id != agent_id:
             return None
@@ -267,8 +266,8 @@ class ReminderTransaction(SqliteTransaction):
         *,
         limit: int,
     ) -> tuple[OwnedReminder, ...]:
-        _validate_non_negative_int(now_ms, "now_ms")
-        _validate_positive_int(limit, "limit")
+        validate_non_negative_int(now_ms, "now_ms")
+        validate_positive_int(limit, "limit")
         predicates = ["state = ?", "next_fire_at_ms <= ?"]
         parameters: list[object] = [ReminderState.SCHEDULED.value, now_ms]
         bound_agent_id = self._bound_agent_id()
@@ -286,10 +285,10 @@ class ReminderTransaction(SqliteTransaction):
     async def save_owned_fired_occurrence(
         self,
         expected_revision: int,
-        reminder: OwnedReminder,
-        occurrence: OwnedReminderOccurrence,
+        reminder: object,
+        occurrence: object,
     ) -> OwnedReminderOccurrence:
-        _validate_positive_int(expected_revision, "expected_revision")
+        validate_positive_int(expected_revision, "expected_revision")
         if not isinstance(reminder, OwnedReminder):
             raise TypeError("reminder must be an OwnedReminder")
         if not isinstance(occurrence, OwnedReminderOccurrence):
@@ -434,10 +433,10 @@ class ReminderTransaction(SqliteTransaction):
     async def save_fired_occurrence(
         self,
         expected_revision: int,
-        reminder: Reminder,
-        occurrence: ReminderOccurrence,
+        reminder: object,
+        occurrence: object,
     ) -> ReminderOccurrence:
-        _validate_positive_int(expected_revision, "expected_revision")
+        validate_positive_int(expected_revision, "expected_revision")
         if not isinstance(reminder, Reminder):
             raise TypeError("reminder must be a Reminder")
         if not isinstance(occurrence, ReminderOccurrence):
@@ -516,8 +515,8 @@ class ReminderTransaction(SqliteTransaction):
         *,
         limit: int,
     ) -> tuple[ReminderOccurrence, ...]:
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
-        _validate_positive_int(limit, "limit")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_positive_int(limit, "limit")
         rows = await self.fetchall(
             f"SELECT {_OCCURRENCE_COLUMNS} FROM reminder_occurrences "
             "WHERE owner_session_id = ? AND read_at_ms IS NULL "
@@ -527,7 +526,7 @@ class ReminderTransaction(SqliteTransaction):
         return tuple(reminder_occurrence_from_row(row) for row in rows)
 
     async def count_pending_reminder_occurrences(self, owner_session_id: str) -> int:
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
         row = await self.fetchone(
             "SELECT COUNT(*) AS pending_count FROM reminder_occurrences "
             "WHERE owner_session_id = ? AND read_at_ms IS NULL",
@@ -540,14 +539,15 @@ class ReminderTransaction(SqliteTransaction):
     async def mark_reminder_occurrences_read(
         self,
         owner_session_id: str,
-        occurrence_ids: tuple[str, ...],
+        occurrence_ids: object,
         *,
         read_at_ms: int,
     ) -> tuple[ReminderOccurrence, ...]:
-        _validate_non_empty_text(owner_session_id, "owner_session_id")
-        _validate_non_negative_int(read_at_ms, "read_at_ms")
+        validate_non_empty_text(owner_session_id, "owner_session_id")
+        validate_non_negative_int(read_at_ms, "read_at_ms")
         if not isinstance(occurrence_ids, tuple):
             raise TypeError("occurrence_ids must be a tuple")
+        occurrence_ids = cast(tuple[str, ...], occurrence_ids)
         if not occurrence_ids:
             return ()
         if len(set(occurrence_ids)) != len(occurrence_ids):