4 1 45 DEV 1-aeb02df8 8-10 Craft release script in rultor configuration. We should create release script that will tag and release Java application into GitHub releases page. .rultor.yml @h1alexbel aliaksei.bialiauski@hey.com 5 1 45 DEV 1-c5e36512 411-414 Set COVEREDRATIO back Previous values was: 0.65, 0.65, 0.65, 0.65, 0.65. Let's keep the MISSEDCOUNT with value of `10`. Don't forget to remove this puzzle. pom.xml @h1alexbel aliaksei.bialiauski@hey.com 6 1 25 DEV 1-e94249ce 37-40 Remove EntryTest#foo test. This test was required only to pass the first build, since we need at least one test to run the `install` maven phase. Don't forget to remove this puzzle. src/test/java/git/tracehub/codereview/action/EntryTest.java @h1alexbel aliaksei.bialiauski@hey.com 22 2 25 DEV 2-b2790fa0 52-57 Implement GhIdentity. For now we do create RtGithub right here, in the main method. Let's implement some sort of "smart" identity that based on INPUT_GITHUBTOKEN variable presence inside the environment will create corresponding object, either RtGithub (real github instance) or MkGithub (mocked github server in memory xml). src/main/java/git/tracehub/codereview/action/Entry.java @h1alexbel aliaksei.bialiauski@hey.com 35 22 90 DEV 22-be91e75e 37-41 Pull GhIdentity from /tracehub repo. We have the same object in tracehubpm/tracehub repo, take a look <a href="https://github.com/tracehubpm/tracehub/blob/master/src/main/java/git/tracehub/identity/GhIdentity.java">here</a>. Let's push this object into separate repository and make it more reusable. Don't forget to remove this puzzle. src/main/java/git/tracehub/codereview/action/github/GhIdentity.java @h1alexbel hialexbel@gmail.com 23 2 30 DEV 2-15ec98f3 58-61 Develop a prompt to the language model. After information is collected (pull request itself, its files, and reviews) we can feed it into the model asking what is the quality of the following code review. src/main/java/git/tracehub/codereview/action/Entry.java @h1alexbel aliaksei.bialiauski@hey.com 24 2 30 DEV 2-a5971890 62-64 Fetch all the info about pull request we are working with. We should fetch all useful information about pull request that we are dealing with: title, files (changes), and its author. Let's present this data in JSON/XML format. src/main/java/git/tracehub/codereview/action/Entry.java @h1alexbel aliaksei.bialiauski@hey.com 25 2 30 DEV 2-b6929245 65-68 Formulate action stoppers. We should formulate some action stoppers that would not "go further" into processing if: pull request is too small (we need a specific number), or some other alerts that would be reasonable. src/main/java/git/tracehub/codereview/action/Entry.java @h1alexbel aliaksei.bialiauski@hey.com 36 2 25 DEV 2-a383e1db 60-62 Fetch pull request approver to post suggestions. To post suggestion to the reviewer, we need to fetch the GitHub login of person who approved pull request. Don't forget to remove this puzzle. src/main/java/git/tracehub/codereview/action/Entry.java @h1alexbel hialexbel@gmail.com 53 51 45 DEV 51-24b09c2d 55-59 Box SkipIfTooSmall.java into more major routine. SkipIfTooSmall.java is a good routine, but it should be encapsulated by something that logically is bigger than that. For now it can be not clear why at the end we run SkipIfTooSmall.java, not something like Analyze.java or similar. src/main/java/git/tracehub/codereview/action/Entry.java @rultor me@rultor.com 54 51 30 DEV 51-c87481d8 41-44 This test fails since there is no injected INPUT_MIN_LINES variable. During test execution we should set `INPUT_MIN_LINES` into system environment, so `MinLines.java` will fetch it and should return expected number of lines. Don't forget to remove this puzzle. src/test/java/git/tracehub/codereview/action/MinLinesTest.java @rultor me@rultor.com 63 60 30 DEV 60-3f8063c1 50-54 Create integration test case for AnalysisRoutine. We should create integration test case for whole analysis routine. Profiles in maven are ready and packed with all necessary env variables. Let's create it and run on reasonable pull request in h1alexbel/test repo. Don't forget to remove this puzzle. src/main/java/git/tracehub/codereview/action/AnalysisRoutine.java @h1alexbel hialexbel@gmail.com 101 85 15 DEV 85-d5fc9715 25-29 apply SkipIfMentioned too skip pull request. SkipIfMentioned should be applied on incoming pull request to check author on exclusion. Let's integrate this field with action.yml inputs, so defined values will be injected into mentions let's call this input skip_authors. src/main/java/git/tracehub/codereview/action/SkipIfMentioned.java @h1alexbel aliaksei.bialiauski@hey.com 109 101 45 DEV 101-c06e9d1d 25-27 Add integration test for pr. SkipIfMentioned should be applied on incoming pull request to check author on exclusion. It should be an integration test. src/main/java/git/tracehub/codereview/action/SkipIfMentioned.java @h1alexbel aliaksei.bialiauski@hey.com 110 101 15 DEV 101-46f592d2 30-33 Create test repo for integration tests. We need to create a repository for integration testing. It should be placed in `tracehubpm/test`. So other contributors can create new integration tests. src/main/java/git/tracehub/codereview/action/SkipIfMentioned.java @h1alexbel aliaksei.bialiauski@hey.com 126 88 45 DEV 88-7fa4f496 75-78 Invalid target release 21 on rultor build. Let's fix 'invalid target release 21' in rultor builds. Probably we need top update the rultor version. Don't forget to remove this puzzle. pom.xml @h1alexbel aliaksei.bialiauski@hey.com 127 88 90 DEV 88-f785f812 53-56 Create integration tests with OpenAIModel. We should create an integration test cases for OpenAI integration objects. Let's do them the similar way as DeepModelITCase.java. Don't forget to remove this puzzle. src/main/java/git/tracehub/codereview/action/AnalysisRoutine.java @h1alexbel aliaksei.bialiauski@hey.com