Skip to content

Commit 0f1f9ef

Browse files
style: ruff check --fix on Phase C files
CI's QA → Python Isolation → Lint failed on two issues: 1. workspace_files.py:45 had quoted forward refs ('docker.types.Mount') when the underlying type is runtime-imported inside the method — ruff RUF066 prefers unquoted when possible. Now uses 'from docker import types as docker_types' at module level and references the real type without quotes. 2. tests/test_workspace_files.py imported pytest unnecessarily (F401). Tests use plain functions, not pytest fixtures from the import. Auto-fixed via 'uv run ruff check --fix .'. 174 tests still pass.
1 parent 55f2467 commit 0f1f9ef

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

lib/python/agentic_isolation/agentic_isolation/workspace_files.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,14 @@ class WorkspaceFiles:
3535
client — callers pass in whatever client they're already using.
3636
"""
3737

38-
client: "docker.DockerClient"
38+
client: docker.DockerClient
3939

4040
def bind_mount(
4141
self,
4242
host_path: Path,
4343
container_path: str,
4444
read_only: bool = True,
45-
) -> "docker.types.Mount":
45+
) -> docker.types.Mount:
4646
"""Build a Mount descriptor for `containers.create(mounts=[...])`.
4747
4848
Relative host_paths are resolved to absolute paths (Docker rejects

lib/python/agentic_isolation/tests/test_workspace_files.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
from pathlib import Path
77
from unittest.mock import MagicMock
88

9-
import pytest
10-
119

1210
def test_bind_mount_descriptor_shape(tmp_path: Path):
1311
"""bind_mount() returns a docker.types.Mount with the expected

0 commit comments

Comments
 (0)