fix: map cancelled check runs to failure, preserve raw conclusion

Cancelled runs (manual cancel or timeout) now map to "failure"
instead of "error" so they block merge like real failures.

The description field now stores the raw GitHub conclusion
(cancelled, failure, timed_out, etc.) instead of the mapped
state, so dashboards can differentiate between true failures
and cancelled runs. Updated both the webhook and refresh paths
for consistency.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mlahargou

controllers/githubHooks.js

@@ -155,16 +155,15 @@ const HooksController = {
         dbUpdated = dbManager.updateComment(comment);
       }
     } else if (event === "check_run") {
-      const state = utils.mapCheckToStatus(
-        body.check_run.conclusion || body.check_run.status
-      );
+      const conclusion = body.check_run.conclusion || body.check_run.status;
+      const state = utils.mapCheckToStatus(conclusion);
 
       const status = new Status({
         repo: body.repository.full_name,
         sha: body.check_run.head_sha,
         state: state,
         context: body.check_run.name,
-        description: state,
+        description: conclusion,
         target_url: body.check_run.html_url,
         started_at: body.check_run.started_at,
         completed_at: body.check_run.completed_at,

lib/git-manager.js

@@ -255,8 +255,9 @@ export default {
       });
 
       let checks = jobRuns.map(function (jobRun) {
-        let state = utils.mapCheckToStatus(jobRun.conclusion || jobRun.status);
-        let desc = state;
+        let conclusion = jobRun.conclusion || jobRun.status;
+        let state = utils.mapCheckToStatus(conclusion);
+        let desc = conclusion;
         let url = jobRun.html_url;
         let context = jobRun.name;
 

lib/utils.js

@@ -77,6 +77,7 @@ export default {
       case "skipped":
         return "success";
       case "failure":
+      case "cancelled":
         return "failure";
       default:
         return "error";