Merge pull request ubiquity-os-marketplace#87 from rndquu/feat/new-priority-msg

feat: new error msg on missing required label

Author: 0x4007

src/handlers/shared/start.ts

@@ -22,11 +22,14 @@ 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 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,
-    });
+    throw logger.error(
+      `This task does not reflect a business priority at the moment. You may start tasks with one of the following labels: ${requiredLabelsToStart.join(", ")}`,
+      {
+        requiredLabelsToStart,
+        issueLabels,
+        issue: issue.html_url,
+      }
+    );
   }
 
   if (!sender) {

tests/main.test.ts

@@ -292,7 +292,9 @@ describe("User start/stop", () => {
     context.adapters = createAdapters(getSupabase(), context);
 
     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." },
+      logMessage: {
+        raw: "This task does not reflect a business priority at the moment. You may start tasks with one of the following labels: Priority: 3 (High), Priority: 4 (Urgent), Priority: 5 (Emergency)",
+      },
     });
   });
 });