feat: provide feedback on re-review with no changes (fixes #103)

src/core/domain/codeBase/contracts/CommentManagerService.contract.ts

@@ -144,4 +144,13 @@ export interface ICommentManagerService {
         pullRequestMessagesConfig?: IPullRequestMessages,
         dryRun?: CodeReviewPipelineContext['dryRun'],
     ): Promise<void>;
+
+    createNoChangesComment(
+        organizationAndTeamData: OrganizationAndTeamData,
+        prNumber: number,
+        repository: { name: string; id: string },
+        platformType: string,
+        language: string,
+        dryRun?: CodeReviewPipelineContext['dryRun'],
+    ): Promise<void>;
 }

src/core/infrastructure/adapters/services/codeBase/codeReviewPipeline/stages/fetch-changed-files.stage.ts

@@ -4,6 +4,10 @@ import {
     IPullRequestManagerService,
     PULL_REQUEST_MANAGER_SERVICE_TOKEN,
 } from '@/core/domain/codeBase/contracts/PullRequestManagerService.contract';
+import {
+    ICommentManagerService,
+    COMMENT_MANAGER_SERVICE_TOKEN,
+} from '@/core/domain/codeBase/contracts/CommentManagerService.contract';
 import { CodeReviewPipelineContext } from '../context/code-review-pipeline.context';
 import { PinoLoggerService } from '../../../logger/pino.service';
 import {
@@ -24,8 +28,10 @@ export class FetchChangedFilesStage extends BasePipelineStage<CodeReviewPipeline
 
     constructor(
         @Inject(PULL_REQUEST_MANAGER_SERVICE_TOKEN)
-        private pullRequestHandlerService: IPullRequestManagerService,
-        private logger: PinoLoggerService,
+        private readonly pullRequestHandlerService: IPullRequestManagerService,
+        @Inject(COMMENT_MANAGER_SERVICE_TOKEN)
+        private readonly commentManagerService: ICommentManagerService,
+        private readonly logger: PinoLoggerService,
     ) {
         super();
     }
@@ -64,6 +70,17 @@ export class FetchChangedFilesStage extends BasePipelineStage<CodeReviewPipeline
                 ? AutomationMessage.NO_FILES_AFTER_IGNORE
                 : AutomationMessage.TOO_MANY_FILES;
 
+            if (!files?.length && context.lastExecution) {
+                await this.commentManagerService.createNoChangesComment(
+                    context.organizationAndTeamData,
+                    context.pullRequest.number,
+                    context.repository,
+                    context.platformType,
+                    context.codeReviewConfig?.languageResultPrompt ?? 'en-US',
+                    context.dryRun,
+                );
+            }
+
             this.logger.warn({
                 message: `Skipping code review for PR#${context.pullRequest.number} - ${msg}`,
                 context: FetchChangedFilesStage.name,

src/core/infrastructure/adapters/services/codeBase/commentManager.service.ts

@@ -135,7 +135,7 @@ export class CommentManagerService implements ICommentManagerService {
                     !isCommitRun &&
                     updatedPR?.body &&
                     summaryConfig?.behaviourForExistingDescription ===
-                        BehaviourForExistingDescription.COMPLEMENT
+                    BehaviourForExistingDescription.COMPLEMENT
                 ) {
                     promptBase += `\n\n**Additional Instructions**:
                     - Focus on generating new insights and relevant information based on the code changes
@@ -199,7 +199,7 @@ export class CommentManagerService implements ICommentManagerService {
                 if (
                     isCommitRun &&
                     summaryConfig?.behaviourForNewCommits ===
-                        BehaviourForNewCommits.REPLACE
+                    BehaviourForNewCommits.REPLACE
                 ) {
                     userPrompt = `
                     This is the updated pull request summary:
@@ -348,7 +348,7 @@ export class CommentManagerService implements ICommentManagerService {
                     if (
                         updatedPR?.body &&
                         summaryConfig?.behaviourForExistingDescription ===
-                            BehaviourForExistingDescription.CONCATENATE
+                        BehaviourForExistingDescription.CONCATENATE
                     ) {
                         // Log for debugging
                         this.logger.log({
@@ -1287,7 +1287,7 @@ ${reviewOptions}
                 (s) =>
                     s.clusteringInformation?.type === ClusteringType.RELATED &&
                     s.clusteringInformation?.parentSuggestionId ===
-                        suggestion.id,
+                    suggestion.id,
             );
 
             const occurrences = [
@@ -1318,11 +1318,11 @@ ${reviewOptions}
     ): string {
         return platformType === PlatformType.BITBUCKET
             ? markdown
-                  .replace(
-                      /(<\/?details>)|(<\/?summary>)|(<!-- kody-codereview -->(\n|\\n)?&#8203;)/g,
-                      '',
-                  )
-                  .trim()
+                .replace(
+                    /(<\/?details>)|(<\/?summary>)|(<!-- kody-codereview -->(\n|\\n)?&#8203;)/g,
+                    '',
+                )
+                .trim()
             : markdown;
     }
 
@@ -1706,4 +1706,56 @@ ${reviewOptions}
             platformType,
         };
     }
+
+    async createNoChangesComment(
+        organizationAndTeamData: OrganizationAndTeamData,
+        prNumber: number,
+        repository: { name: string; id: string },
+        platformType: string,
+        language: string,
+        dryRun?: CodeReviewPipelineContext['dryRun'],
+    ): Promise<void> {
+        try {
+            const noChangesMessage = getTranslationsForLanguageByCategory(
+                language as LanguageValue,
+                TranslationsCategory.PullRequestFinishSummaryMarkdown,
+            ).noChangesDetected;
+
+            const commentBody = this.sanitizeBitbucketMarkdown(
+                noChangesMessage,
+                platformType as PlatformType,
+            );
+
+            await this.codeManagementService.createIssueComment(
+                {
+                    organizationAndTeamData,
+                    prNumber,
+                    repository: {
+                        name: repository.name,
+                        id: repository.id,
+                    },
+                    body: commentBody,
+                    dryRun,
+                },
+                dryRun?.enabled ? PlatformType.INTERNAL : undefined,
+            );
+
+            this.logger.log({
+                message: `Created no changes comment for PR#${prNumber}`,
+                context: CommentManagerService.name,
+                metadata: { prNumber },
+            });
+        } catch (error) {
+            this.logger.error({
+                message: `Failed to create no changes comment for PR#${prNumber}`,
+                context: CommentManagerService.name,
+                error: error.message,
+                metadata: {
+                    organizationAndTeamData,
+                    prNumber,
+                    repository,
+                },
+            });
+        }
+    }
 }

src/shared/utils/translations/dictionaries/en-US.json

@@ -5,7 +5,8 @@
     },
     "pullRequestFinishSummaryMarkdown": {
         "withComments": "## Code Review Completed! 🔥\n\nThe code review was successfully completed based on your current configurations.\n\n",
-        "withoutComments": "# Kody Review Complete\n**Great news!** 🎉\nNo issues were found that match your current review configurations.\n\nKeep up the excellent work! 🚀"
+        "withoutComments": "# Kody Review Complete\n**Great news!** 🎉\nNo issues were found that match your current review configurations.\n\nKeep up the excellent work! 🚀",
+        "noChangesDetected": "# Kody Review Complete ✅\n**Info:** This PR has already been reviewed with no new changes detected.\n\nIf you've made new changes, ensure they're committed and pushed."
     },
     "pullRequestSummaryMarkdown": {
         "title": "PR Summary (Comment created by [Kody](https://kodus.io) 🤖)",
@@ -102,4 +103,4 @@
         "manualTitle": "Manual Review",
         "manualDesc": "Kody reviews the first push automatically, then only when you request with @kody start-review."
     }
-}
+}

src/shared/utils/translations/translations.ts

@@ -47,6 +47,7 @@ interface ReviewComment {
 interface PullRequestFinishSummaryMarkdown {
     withComments: string;
     withoutComments: string;
+    noChangesDetected: string;
 }
 
 interface PullRequestSummaryMarkdown {