fix: update logic for most used languages to match contributor profile (

#3205)

lib/utils/contributor-utils.test.ts

@@ -50,7 +50,7 @@ describe("contributor utilities", () => {
         },
       }) satisfies DbUser;
 
-      expect(getTopContributorLanguages(user)).toEqual(["go", "javascript"]);
+      expect(getTopContributorLanguages(user)).toEqual(["go", "rust"]);
     });
 
     it("should get no languages when there are no languages", () => {

lib/utils/contributor-utils.ts

@@ -1,6 +1,14 @@
 export function getTopContributorLanguages(contributor: DbUser) {
   // Note that it's possible for a user to have no languages
-  const languages = Object.entries(contributor.languages).map(([language]) => language);
+  const languages = Object.entries(contributor.languages).map(([language, percentageUsed]) => ({
+    language,
+    percentageUsed: percentageUsed,
+  }));
+  const langArray = languages.slice().sort((a, b) => b.percentageUsed - a.percentageUsed);
+  const sortedLangArray = langArray
+    .sort((a, b) => (a.percentageUsed < b.percentageUsed ? 1 : -1))
+    .slice(0, 2)
+    .map((lang) => lang.language);
 
-  return languages.sort((a, b) => (a < b ? -1 : 1)).slice(0, 2);
+  return sortedLangArray;
 }

next-types.d.ts

@@ -275,7 +275,7 @@ interface DbUser {
   readonly discord_url: string;
   readonly notification_count: number;
   readonly insights_count: number;
-  readonly languages: { [lang]: number };
+  readonly languages: { [lang: string]: number };
   readonly first_opened_pr_at: string;
   readonly followers_count: number;
   readonly following_count: number;