Merge pull request #7 from ississippi/decisions-testing

Decisions testing

Author: ississippi

.gitignore

@@ -0,0 +1,5 @@
+################################################################################
+# This .gitignore file was automatically created by Microsoft(R) Visual Studio.
+################################################################################
+
+/favicon_io

NotificationsService/NotificationsService/Controllers/PrController.cs

@@ -43,11 +43,11 @@ public IActionResult ListRoutes()
         [HttpGet("openprs")]
         public async Task<IActionResult> GetOpenPrs()
         {
-            _logger.LogInformation("Received request to get all open PRs.");
+            _logger.LogDebug("Received request to get all open PRs.");
             try
             {
                 var dynamoService = new DynamoService(_logger);
-                _logger.LogInformation("Received request to get PR Details.");
+                _logger.LogDebug("Received request to get PR Details.");
                 var allReviews = await dynamoService.GetAllReviewsAsync();
                 // Create a list to hold the processed reviews
                 var processedReviews = new List<object>();
@@ -92,9 +92,9 @@ public async Task<IActionResult> GetDetails([FromQuery] int id, string repo)
             try
             {
                 var dynamoService = new DynamoService(_logger);
-                _logger.LogInformation("Received request to get PR Details.");
+                _logger.LogDebug("Received request to get PR Details.");
                 var prItem = await dynamoService.GetReviewByIdAsync(id, repo);
-                //_logger.LogInformation($"prItem from DDB: {prItem}");
+                _logger.LogDebug($"prItem from DDB: {prItem}");
                 if (prItem != null)
                 {
                     var metadataMap = prItem["metadata"].M; // Step 1: get the map
@@ -126,7 +126,7 @@ public async Task<IActionResult> GetDetails([FromQuery] int id, string repo)
         [HttpGet("review")]
         public IActionResult GetReview([FromQuery] int prNumber)
         {
-            _logger.LogInformation("Received request to get Review.");
+            _logger.LogDebug("Received request to get Review.");
             // TODO: Replace with real PR review generation
             string mockReview = $"This is an autogenerated review for PR #{prNumber}. It looks good overall but could use minor improvements.";
 
@@ -137,7 +137,7 @@ public IActionResult GetReview([FromQuery] int prNumber)
         [HttpPost("feedback")]
         public IActionResult PostFeedback([FromBody] FeedbackRequest request)
         {
-            _logger.LogInformation($"Received feedback: PR#{request.prNumber}, Vote: {request.vote}");
+            _logger.LogDebug($"Received feedback: PR#{request.prNumber}, Vote: {request.vote}");
             //Console.WriteLine($"Received feedback: PR#{request.PrNumber}, Vote: {request.Vote}");
 
             return Ok(new { message = "Feedback received" });
@@ -147,7 +147,7 @@ public IActionResult PostFeedback([FromBody] FeedbackRequest request)
         [HttpPost("decision")]
         public IActionResult PostReview([FromBody] DecisionRequest request)
         {
-            _logger.LogInformation($"Received decision: PR#{request.prNumber}, Decision: {request.decision}");
+            _logger.LogDebug($"Received decision: PR#{request.prNumber}, Decision: {request.decision}");
             // call the git service to submit the decision to Git
             return Ok(new { message = "Decision received" });
         }

NotificationsService/NotificationsService/Controllers/SnsController.cs

@@ -24,8 +24,7 @@ public async Task<IActionResult> Receive()
         {
             using var reader = new StreamReader(Request.Body);
             var body = await reader.ReadToEndAsync();
-            //_logger.LogInformation("Receive: SNS message received: {0}", body);
-            _logger.LogInformation("Receive: SNS message received.");
+            _logger.LogDebug("Receive: SNS message received: {0}", body);
 
             try
             {
@@ -50,7 +49,7 @@ public async Task<IActionResult> Receive()
                     {
                         try
                         {
-                            _logger.LogInformation($"Pr_Number: {reviewNotification.metadata.pr_number}");
+                            //_logger.LogInformation($"Pr_Number: {reviewNotification.metadata.pr_number}");
                             //_logger.LogInformation($"ReviewTitle: {reviewNotification.reviewTitle}");
                             //_logger.LogInformation($"User_Login: {reviewNotification.metadata.user_login}");
                             //_logger.LogInformation($"Created_At: {reviewNotification.metadata.created_at}");
@@ -66,7 +65,7 @@ public async Task<IActionResult> Receive()
                                 //review = reviewNotification.review
                             };
                             await _prService.BroadcastNewPrAsync(prItem);
-                            _logger.LogInformation($"Review for PR #{prItem.id} sent to BroadcastNewPrAsync().");
+                            //_logger.LogInformation($"Review for PR #{prItem.id} sent to BroadcastNewPrAsync().");
                         }
                         catch
                         {

NotificationsService/NotificationsService/Services/PrService.cs

@@ -516,8 +516,7 @@ await removedClient.Socket.CloseAsync(
                         }
                     }
 
-                    _logger.LogInformation("Completed broadcasting PR #{PrId}. Active clients: {Count}",
-                        newPr.id, _clients.Count);
+                    _logger.LogDebug("Completed broadcasting PR #{PrId}. Active clients: {Count}", newPr.id, _clients.Count);
                 }
                 finally
                 {

NotificationsService/NotificationsService/wwwroot/index.html

@@ -93,6 +93,10 @@ <h6>Action:</h6>
 function renderPrList() {
     const list = document.getElementById('pr-list');
     list.innerHTML = '';
+
+    // Sort PRs by date (most recent first)
+    prList.sort((a, b) => new Date(b.date) - new Date(a.date));
+
     prList.forEach(pr => {
         const item = document.createElement('button');
         item.className = 'list-group-item list-group-item-action';
@@ -127,6 +131,7 @@ <h6>Action:</h6>
 
     document.getElementById('pr-list').classList.add('d-none');
     document.getElementById('pr-detail').classList.remove('d-none');
+    document.getElementById("approve").checked = true;
 
     // Convert Review markdown from the model to html for showdown ------------------------------
     // Create converter instance with appropriate options