chore: throw logger

manifest.json

@@ -1,13 +1,7 @@
 {
   "name": "Start | Stop",
   "description": "Assign or un-assign yourself from an issue.",
-  "ubiquity:listeners": [
-    "issue_comment.created",
-    "issues.assigned",
-    "issues.unassigned",
-    "pull_request.opened",
-    "pull_request.edited"
-  ],
+  "ubiquity:listeners": ["issue_comment.created", "issues.assigned", "issues.unassigned", "pull_request.opened", "pull_request.edited"],
   "commands": {
     "start": {
       "ubiquity:example": "/start",
@@ -51,12 +45,7 @@
         "type": "string"
       },
       "rolesWithReviewAuthority": {
-        "default": [
-          "COLLABORATOR",
-          "OWNER",
-          "MEMBER",
-          "ADMIN"
-        ],
+        "default": ["COLLABORATOR", "OWNER", "MEMBER", "ADMIN"],
         "type": "array",
         "items": {
           "type": "string"
@@ -80,4 +69,4 @@
       "requiredLabelsToStart"
     ]
   }
-}
+}

src/handlers/shared/start.ts

@@ -22,7 +22,11 @@ export async function start(
 
   if (requiredLabelsToStart.length && !requiredLabelsToStart.some((label) => issueLabels.includes(label))) {
     // The "Priority" label must reflect a business priority, not a development one.
-    throw new Error("This task does not reflect a business priority at the moment and cannot be started. This will be reassessed in the coming weeks.");
+    throw logger.error("This task does not reflect a business priority at the moment and cannot be started. This will be reassessed in the coming weeks.", {
+      requiredLabelsToStart,
+      issueLabels,
+      issue: issue.html_url,
+    });
   }
 
   if (!sender) {

tests/main.test.ts

@@ -288,9 +288,9 @@ describe("User start/stop", () => {
 
     context.adapters = createAdapters(getSupabase(), context);
 
-    await expect(userStartStop(context)).rejects.toThrow(
-      "This task does not reflect a business priority at the moment and cannot be started. This will be reassessed in the coming weeks."
-    );
+    await expect(userStartStop(context)).rejects.toMatchObject({
+      logMessage: { raw: "This task does not reflect a business priority at the moment and cannot be started. This will be reassessed in the coming weeks." },
+    });
   });
 });