Skip to content

Commit e5db75f

Browse files
nhortonclaude
andcommitted
style: apply ruff format to satisfy CI format check
CI runs `ruff format --check`, which my local `uv run ruff check` calls did not exercise (check runs the linter, not the formatter). Two lines I had manually broken up were re-joined by ruff format. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent a068d37 commit e5db75f

2 files changed

Lines changed: 5 additions & 15 deletions

File tree

src/deepwork/review/instructions.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,7 @@ def _paths_component(files: list[str]) -> str:
6363
return joined
6464

6565

66-
def _content_hash(
67-
files: list[str], project_root: Path, inline_content: str | None = None
68-
) -> str:
66+
def _content_hash(files: list[str], project_root: Path, inline_content: str | None = None) -> str:
6967
"""SHA-256 content hash (first 12 hex chars) of the task content.
7068
7169
Files are sorted alphabetically before concatenation. Files that

tests/unit/jobs/mcp/test_quality_gate.py

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1220,9 +1220,7 @@ def test_output_level_review_produces_inline_task(self, tmp_path: Path) -> None:
12201220
def test_argument_level_review_produces_inline_task(self, tmp_path: Path) -> None:
12211221
"""A review block on the step_argument (not the output-ref) produces a task."""
12221222
arg_review = ReviewBlock(strategy="matches_together", instructions="Verify the note")
1223-
arg = StepArgument(
1224-
name="note", description="Note", type="string", review=arg_review
1225-
)
1223+
arg = StepArgument(name="note", description="Note", type="string", review=arg_review)
12261224
output_ref = StepOutputRef(argument_name="note", required=True)
12271225
step = WorkflowStep(name="write", outputs={"note": output_ref})
12281226
job, workflow = _make_job(tmp_path, [arg], step)
@@ -1244,18 +1242,12 @@ def test_argument_level_review_produces_inline_task(self, tmp_path: Path) -> Non
12441242

12451243
# THIS TEST VALIDATES A HARD REQUIREMENT (JOBS-REQ-004.8.3).
12461244
# YOU MUST NOT MODIFY THIS TEST UNLESS THE REQUIREMENT CHANGES
1247-
def test_both_output_and_arg_level_reviews_produce_separate_tasks(
1248-
self, tmp_path: Path
1249-
) -> None:
1245+
def test_both_output_and_arg_level_reviews_produce_separate_tasks(self, tmp_path: Path) -> None:
12501246
"""Output-level and arg-level reviews on the same string output both execute."""
12511247
output_review = ReviewBlock(strategy="individual", instructions="Output check")
12521248
arg_review = ReviewBlock(strategy="individual", instructions="Arg check")
1253-
arg = StepArgument(
1254-
name="summary", description="Summary", type="string", review=arg_review
1255-
)
1256-
output_ref = StepOutputRef(
1257-
argument_name="summary", required=True, review=output_review
1258-
)
1249+
arg = StepArgument(name="summary", description="Summary", type="string", review=arg_review)
1250+
output_ref = StepOutputRef(argument_name="summary", required=True, review=output_review)
12591251
step = WorkflowStep(name="write", outputs={"summary": output_ref})
12601252
job, workflow = _make_job(tmp_path, [arg], step)
12611253

0 commit comments

Comments
 (0)