lint and update

Author: CTY-git

.github/workflows/test.yml

@@ -1,10 +1,6 @@
 name: Tests
 
 on:
-  pull_request:
-    types:
-      - ready_for_review
-      - review_requested
   push:
     branches-ignore:
       - main
@@ -15,15 +11,12 @@ on:
       - generateunittests-*
       - generatecodeusageexample-*
       - resolveissue-*
-
       - demo*
 
 # Credits to https://blog.maximeheckel.com/posts/building-perfect-github-action-frontend-teams/#you-are-terminated
 concurrency:
   # Here the group is defined by the head_ref of the PR
   group: ${{ github.head_ref || github.ref_name }}
-  # Here we specify that we'll cancel any "in progress" workflow of the same group. Thus if we push, ammend a commit and push
-  # again the previous workflow will be cancelled, thus saving us github action build minutes and avoid any conflicts
   cancel-in-progress: true
 
 jobs:

patchwork/patchflows/PRReview/PRReview.py

@@ -1,20 +1,11 @@
-import json
 from pathlib import Path
 
 import yaml
 
 from patchwork.common.utils.progress_bar import PatchflowProgressBar
 from patchwork.common.utils.step_typing import validate_steps_with_inputs
 from patchwork.step import Step
-from patchwork.steps import (
-    LLM,
-    CallLLM,
-    CreatePRComment,
-    ExtractModelResponse,
-    PreparePR,
-    PreparePrompt,
-    ReadPRDiffs, SimplifiedLLMOnce, JoinList,
-)
+from patchwork.steps import CreatePRComment, ReadPRDiffs, SimplifiedLLMOnce
 
 _DEFAULT_PROMPT_JSON = Path(__file__).parent / "pr_review_prompt.json"
 _DEFAULT_INPUT_FILE = Path(__file__).parent / "defaults.yml"
@@ -54,7 +45,7 @@ def run(self) -> dict:
             llm1_outputs = SimplifiedLLMOnce(
                 dict(
                     prompt_value=diffs,
-                    user_prompt='''\
+                    user_prompt="""\
 Analyze the following code diff against the provided rules:
 
 <CODE_DIFF>
@@ -118,26 +109,26 @@ def run(self) -> dict:
 </REVIEW_FORMAT>
 
 Ensure that you include all rules in your response, even if there\'s no violation. The output should directly reflect the reasoning in your thinking section.
-''',
+""",
                     json_schema={"review": "The markdown text of the reviews"},
-                    **self.inputs
+                    **self.inputs,
                 )
             ).run()
 
             llm2_outputs = SimplifiedLLMOnce(
                 dict(
                     prompt_value=llm1_outputs,
-                    user_prompt='''\
+                    user_prompt="""\
 You are a software manager compiling code reviews from all teams. You are given a list of code reviews. You have to remove code reviews that is either not actionable or useful. Do not change the accepted reviews, return the original review for the response. Do not remove the path from the review.
 
 <code_reviews>
 {{review}}
 <code_reviews>
 
 You should return an empty response if there are no code reviews that is actionable or useful.
-''',
+""",
                     json_schema={"review": "The reviews curated"},
-                    **self.inputs
+                    **self.inputs,
                 )
             ).run()
 

patchwork/steps/SimplifiedLLMOnce/SimplifiedLLMOnce.py

@@ -1,6 +1,9 @@
 from patchwork.step import Step
 from patchwork.steps.SimplifiedLLM.SimplifiedLLM import SimplifiedLLM
-from patchwork.steps.SimplifiedLLMOnce.typed import SimplifiedLLMOnceInputs, SimplifiedLLMOnceOutputs
+from patchwork.steps.SimplifiedLLMOnce.typed import (
+    SimplifiedLLMOnceInputs,
+    SimplifiedLLMOnceOutputs,
+)
 
 
 class SimplifiedLLMOnce(Step, input_class=SimplifiedLLMOnceInputs, output_class=SimplifiedLLMOnceOutputs):

patchwork/steps/SimplifiedLLMOnce/typed.py

@@ -42,4 +42,4 @@ class SimplifiedLLMOnceInputs(__SimplifiedLLMOncePBInputsRequired, total=False):
 
 class SimplifiedLLMOnceOutputs(TypedDict):
     request_tokens: int
-    response_tokens: int
+    response_tokens: int