fix(ci): apply ruff format and suppress vulture containment_path
CI / push-validation (pull_request) Successful in 22s
CI / helm (pull_request) Successful in 31s
CI / build (pull_request) Successful in 44s
CI / quality (pull_request) Successful in 50s
CI / lint (pull_request) Successful in 55s
CI / security (pull_request) Successful in 1m4s
CI / typecheck (pull_request) Successful in 1m11s
CI / unit_tests (pull_request) Successful in 6m6s
CI / coverage (pull_request) Failing after 1m5s
CI / docker (pull_request) Successful in 1m25s
CI / integration_tests (pull_request) Failing after 31m41s
CI / status-check (pull_request) Has been cancelled

Reformat features/steps/resource_handler_project_access_1444_steps.py
and src/cleveragents/resource/handlers/_base.py to satisfy ruff format
check (complex slice spacing, collapsing single-f-string assert parens).

Add containment_path to vulture_whitelist.py — the parameter is
required by the ResourceHandler.project_access interface (spec
§21691-21698) but is intentionally unused in stub implementations
that raise NotImplementedError.
This commit is contained in:
2026-05-30 00:12:22 -04:00
committed by drew
parent 6d0b748722
commit 4c53b52678
3 changed files with 10 additions and 14 deletions
@@ -200,28 +200,21 @@ def step_tdd1444_instance_crosses_sandbox(context: Context, expected: str) -> No
def step_tdd1444_instance_read_richness(context: Context, expected: int) -> None:
assert context.tdd1444_result is not None, "No AccessProjection instance"
assert context.tdd1444_result.read_richness == expected, (
f"Expected read_richness={expected}, "
f"got {context.tdd1444_result.read_richness}"
f"Expected read_richness={expected}, got {context.tdd1444_result.read_richness}"
)
@then("tdd1444 the result is an AccessProjection or None")
def step_tdd1444_result_is_projection_or_none(context: Context) -> None:
assert context.tdd1444_error is None, (
f"Unexpected error: {context.tdd1444_error}"
)
assert context.tdd1444_error is None, f"Unexpected error: {context.tdd1444_error}"
assert context.tdd1444_result is None or isinstance(
context.tdd1444_result, AccessProjection
), (
f"Expected AccessProjection or None, got {type(context.tdd1444_result)}"
)
), f"Expected AccessProjection or None, got {type(context.tdd1444_result)}"
@then('tdd1444 the result is an AccessProjection with protocol "{expected}"')
def step_tdd1444_result_protocol(context: Context, expected: str) -> None:
assert context.tdd1444_error is None, (
f"Unexpected error: {context.tdd1444_error}"
)
assert context.tdd1444_error is None, f"Unexpected error: {context.tdd1444_error}"
assert context.tdd1444_result is not None, "Expected AccessProjection, got None"
assert isinstance(context.tdd1444_result, AccessProjection), (
f"Expected AccessProjection, got {type(context.tdd1444_result)}"
@@ -235,6 +228,5 @@ def step_tdd1444_result_protocol(context: Context, expected: str) -> None:
def step_tdd1444_result_read_richness(context: Context, expected: int) -> None:
assert context.tdd1444_result is not None, "Expected AccessProjection, got None"
assert context.tdd1444_result.read_richness == expected, (
f"Expected read_richness={expected}, "
f"got {context.tdd1444_result.read_richness}"
f"Expected read_richness={expected}, got {context.tdd1444_result.read_richness}"
)
+1 -1
View File
@@ -388,7 +388,7 @@ class BaseResourceHandler:
binding_loc = binding_resource.location or ""
target_loc = target_resource.location
if binding_loc and target_loc.startswith(binding_loc):
access_path = target_loc[len(binding_loc):].lstrip("/")
access_path = target_loc[len(binding_loc) :].lstrip("/")
else:
access_path = target_loc
crosses = sandbox is not None
+4
View File
@@ -1244,3 +1244,7 @@ generate_session_id # noqa: B018, F821
# Pydantic @classmethod validator parameter required by decorator protocol
cls # noqa: B018, F821
# ResourceHandler.project_access interface parameter — required by spec (§21691-21698)
# but not referenced in stub/base implementations that raise NotImplementedError
containment_path # noqa: B018, F821