Only show verified skills in results

Author: himaniraghav3

src/reports/member/dto/member-search-response.dto.ts

@@ -9,7 +9,7 @@ export class MatchedSkillDto {
 
   @ApiProperty({
     description:
-      "True when the member has win credit and/or at least one platform skill event for this skill (e.g. submission, review); false for self-declared only.",
+      "True for platform-backed skill activity (wins and/or skill events). Matched skills only include these, not self-attested-only skills.",
   })
   isVerified!: boolean;
 

src/reports/member/member-search.service.spec.ts

@@ -142,7 +142,6 @@ describe("MemberSearchService", () => {
     expect(dataSql).not.toContain(
       'EXISTS (SELECT 1 FROM recently_active ra WHERE ra.user_id = m."userId")',
     );
-    expect(dataSql).not.toContain("COALESCE(m.verified, false) = true");
   });
 
   it("adds profileComplete CTE/join only when enabled and keeps count params free of pagination", async () => {
@@ -226,13 +225,11 @@ describe("MemberSearchService", () => {
     expect(validationParams).toEqual([[skillA, skillB]]);
 
     expect(dataSql).toContain("requested_skills AS");
-    expect(dataSql).toContain(
-      "(usd.wins >= rs.min_wins OR usd.submitted > 0)",
-    );
+    expect(dataSql).toContain("FILTER (WHERE usd.wins > 0 OR usd.submitted > 0)");
     expect(dataSql).toContain("INNER JOIN user_match_data umd");
-    expect(dataSql).toContain("THEN COUNT(DISTINCT usd.skill_id) =");
-    expect(dataSql).toContain("ELSE COUNT(DISTINCT usd.skill_id) >= 1");
-    expect(dataSql).not.toContain("usd.wins >= rs.min_wins");
+    expect(dataSql).toContain("THEN COUNT(DISTINCT CASE");
+    expect(dataSql).toContain("ELSE COUNT(DISTINCT CASE");
+    expect(dataSql).toContain("(usd.wins >= rs.min_wins OR usd.submitted > 0)");
     expect(dataParams).toContainEqual([skillA, skillB]);
     expect(dataParams).toContainEqual([5, 0]);
     expect(dataParams).toContain("AND");

src/reports/member/member-search.service.ts

@@ -153,23 +153,29 @@ qualifying_users AS (
 user_match_data AS (
   SELECT
     usd.user_id,
-    SUM(
-      1.0
-      + LEAST(usd.wins::float / 100.0, 0.5)
-      + CASE WHEN usd.submitted > 0
-          THEN (usd.wins::float / usd.submitted::float) * 0.5
-          ELSE 0.0
-        END
+    COALESCE(
+      SUM(
+        1.0
+        + LEAST(usd.wins::float / 100.0, 0.5)
+        + CASE WHEN usd.submitted > 0
+            THEN (usd.wins::float / usd.submitted::float) * 0.5
+            ELSE 0.0
+          END
+      ) FILTER (WHERE usd.wins > 0 OR usd.submitted > 0),
+      0.0
     ) AS total_skill_points,
-    jsonb_agg(
-      jsonb_build_object(
-        'id',         usd.skill_id::text,
-        'name',       usd.skill_name,
-        'isVerified', (usd.wins > 0 OR usd.submitted > 0),
-        'wins',       usd.wins,
-        'submitted',  usd.submitted
-      )
-      ORDER BY usd.skill_name
+    COALESCE(
+      jsonb_agg(
+        jsonb_build_object(
+          'id',         usd.skill_id::text,
+          'name',       usd.skill_name,
+          'isVerified', (usd.wins > 0 OR usd.submitted > 0),
+          'wins',       usd.wins,
+          'submitted',  usd.submitted
+        )
+        ORDER BY usd.skill_name
+      ) FILTER (WHERE usd.wins > 0 OR usd.submitted > 0),
+      '[]'::jsonb
     ) AS matched_skills
   FROM user_skill_data usd
   WHERE usd.user_id IN (SELECT user_id FROM qualifying_users)