Skip to content

Commit a21650b

Browse files
committed
fix(review): add SessionLocator | None type annotation to _flush session_locator param
The helper's session_locator parameter was untyped, diverging from the flush_session_log signature it wraps.
1 parent 6346130 commit a21650b

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

tests/execution/conftest.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
import pytest
1414

15-
from autoskillit.core import BackendCapabilities, CmdSpec
15+
from autoskillit.core import BackendCapabilities, CmdSpec, SessionLocator
1616
from autoskillit.core.types import SubprocessResult, TerminationReason
1717
from autoskillit.execution.session import ClaudeSessionResult
1818
from autoskillit.execution.session._exit_classification import _CODEX_API_ERROR_PATTERNS
@@ -311,7 +311,11 @@ def _subagent_assistant_ndjson(
311311

312312

313313
def _flush(
314-
tmp_path: Path, *, backend: str = "claude-code", session_locator=None, **overrides
314+
tmp_path: Path,
315+
*,
316+
backend: str = "claude-code",
317+
session_locator: SessionLocator | None = None,
318+
**overrides,
315319
) -> None:
316320
from autoskillit.core.types._type_results import ModelIdentity, ProviderOutcome
317321
from autoskillit.core.types._type_results_execution import (

0 commit comments

Comments
 (0)