Reduce bot message frequency to prevent spam

csharp/Platform.Bot/Program.cs

@@ -71,7 +71,7 @@ private static async Task<int> Main(string[] args)
             var minimumInteractionIntervalOption = new Option<int>(
                 name: "--minimum-interaction-interval",
                 description: "Minimum interaction interval in seconds.",
-                getDefaultValue: () => 60);
+                getDefaultValue: () => 300);
 
             var rootCommand = new RootCommand("Sample app for System.CommandLine")
             {

csharp/Platform.Bot/Trackers/IssueTracker.cs

@@ -31,6 +31,14 @@ public class IssueTracker : ITracker<Issue>
         /// </summary>
         private IList<ITrigger<Issue>> _triggers { get; }
 
+        /// <summary>
+        /// <para>
+        /// Tracks processed issues to prevent duplicate actions.
+        /// </para>
+        /// <para></para>
+        /// </summary>
+        private HashSet<string> _processedIssues { get; } = new HashSet<string>();
+
         /// <summary>
         /// <para>
         /// Initializes a new <see cref="IssueTracker"/> instance.
@@ -66,6 +74,15 @@ public async Task Start(CancellationToken cancellationToken)
             var allIssues = _storage.GetIssues();
             foreach (var issue in allIssues)
             {
+                // Create a unique key for each issue-trigger combination to prevent duplicate processing
+                var issueKey = $"{issue.Repository.FullName}#{issue.Number}";
+
+                // Skip if this issue has already been processed
+                if (_processedIssues.Contains(issueKey))
+                {
+                    continue;
+                }
+
                 foreach (var trigger in _triggers)
                 {
                     if (cancellationToken.IsCancellationRequested)
@@ -75,6 +92,9 @@ public async Task Start(CancellationToken cancellationToken)
                     if (await trigger.Condition(issue))
                     {
                         await trigger.Action(issue);
+                        // Mark issue as processed after successful action
+                        _processedIssues.Add(issueKey);
+                        break; // Only process one trigger per issue to prevent conflicts
                     }
                 }
             }

csharp/Platform.Bot/Trackers/PullRequestTracker.cs

@@ -33,6 +33,14 @@ public class PullRequestTracker : ITracker<PullRequest>
         /// </summary>
         private IList<ITrigger<PullRequest>> _triggers;
 
+        /// <summary>
+        /// <para>
+        /// Tracks processed pull requests to prevent duplicate actions.
+        /// </para>
+        /// <para></para>
+        /// </summary>
+        private HashSet<string> _processedPullRequests = new HashSet<string>();
+
         /// <summary>
         /// <para>
         /// Initializes a new <see cref="IssueTracker"/> instance.
@@ -75,10 +83,22 @@ public async Task Start(CancellationToken cancellationToken)
                         {
                             return;
                         }
+
+                        // Create a unique key for each pull request to prevent duplicate processing
+                        var pullRequestKey = $"{repository.FullName}#{pullRequest.Number}";
+
+                        // Skip if this pull request has already been processed
+                        if (_processedPullRequests.Contains(pullRequestKey))
+                        {
+                            continue;
+                        }
+
                         var detailedPullRequest = _storage.Client.PullRequest.Get(repository.Id, pullRequest.Number).AwaitResult();
                         if (await trigger.Condition(detailedPullRequest))
                         {
                             await trigger.Action(detailedPullRequest);
+                            // Mark pull request as processed after successful action
+                            _processedPullRequests.Add(pullRequestKey);
                         }
                     }
                 }

csharp/Storage/RemoteStorage/GitHubStorage.cs

@@ -103,12 +103,21 @@ public IReadOnlyList<Issue> GetIssues()
                 Since = lastIssue
             };
             var issues = Client.Issue.GetAllForCurrent(request).Result;
+
+            // Always update lastIssue to current time to prevent reprocessing the same timeframe
+            // even when there are no new issues
+            var now = DateTimeOffset.Now;
             if (issues.Count != 0)
             {
-                lastIssue = issues.Max(x => x.CreatedAt);
-                return issues;
+                var maxIssueTime = issues.Max(x => x.CreatedAt);
+                lastIssue = maxIssueTime > now ? maxIssueTime : now;
             }
-            return new List<Issue>();
+            else
+            {
+                lastIssue = now;
+            }
+
+            return issues;
         }
 
         public Task<IReadOnlyList<GitHubCommit>> GetCommits(long repositoryId, CommitRequest commitRequest) => Client.Repository.Commit.GetAll(repositoryId, commitRequest);