Collect all invalid entries in summary

Author: dus7

org/allPRs.ts

@@ -301,6 +301,7 @@ export const featureFlagAsanaLink = async () => {
     if (changedFiles.length === 0) return;
 
     const asanaTaskUrlRegex = /^\/\/\/\s*https:\/\/app\.asana\.com\/1\/137249556945\/project\/1211834678943996\/task\/\d+(\?\S*)?\s*$/;
+    const casesWithInvalidLinks: { file: string; caseName: string }[] = [];
 
     for (const file of changedFiles) {
         const structuredDiff = await danger.git.structuredDiffForFile(file);
@@ -347,7 +348,8 @@ export const featureFlagAsanaLink = async () => {
 
                 // Only check added lines for new case declarations
                 if (change.type !== "add") continue;
-                if (!/^\s*case\s+\w+/.test(content)) continue;
+                const caseMatch = content.match(/^\s*case\s+(\w+)/);
+                if (!caseMatch) continue;
 
                 // Found an added case line – walk upward through added comment lines looking for the Asana URL
                 let foundAsanaLink = false;
@@ -363,12 +365,16 @@ export const featureFlagAsanaLink = async () => {
                 }
 
                 if (!foundAsanaLink) {
-                    warn(`New FeatureFlag case in \`${file}\` is missing a valid Feature Flag link in the comment.\nAdd a task in https://app.asana.com/1/137249556945/project/1211834678943996/list/1211838475578067 and use it in the comment.`);
-                    return;
+                    casesWithInvalidLinks.push({ file, caseName: caseMatch[1] });
                 }
             }
         }
     }
+
+    if (casesWithInvalidLinks.length > 0) {
+        const caseList = casesWithInvalidLinks.map(c => `- \`${c.caseName}\` in \`${c.file}\``).join("\n");
+        warn(`New FeatureFlag cases are missing a valid Feature Flag link in the comment:\n${caseList}\nAdd a task in https://app.asana.com/1/137249556945/project/1211834678943996/list/1211838475578067 and use it in the comment.`);
+    }
 }
 
 // Default run

tests/featureFlagAsanaLink.allPRs.test.ts

@@ -268,4 +268,19 @@ describe("Feature flag Asana link checks", () => {
         await featureFlagAsanaLink()
         expect(dm.warn).toHaveBeenCalled()
     })
+
+    it("reports all cases missing links in a single warning", async () => {
+        dm.rawDiff = `@@ -10,6 +10,10 @@ enum FeatureFlag {
++    case firstFeature
++    /// https://app.asana.com/1/137249556945/project/1211834678943996/task/123456789
++    case validFeature
++    case secondFeature`
+
+        await featureFlagAsanaLink()
+        expect(dm.warn).toHaveBeenCalledTimes(1)
+        const message = dm.warn.mock.calls[0][0]
+        expect(message).toContain("firstFeature")
+        expect(message).toContain("secondFeature")
+        expect(message).not.toContain("validFeature")
+    })
 })