Sort subscription list alphabetically and normalize repo casing (#6)

* lowercase owner and repo name before insert

* sort subscribed repo list

Author: stplasim

src/slack/commands/list.ts

@@ -1,4 +1,4 @@
-import { eq } from 'drizzle-orm'
+import { eq, sql } from 'drizzle-orm'
 import { buildListBlocks } from '@bot/blocks/list'
 import { db } from '@db/client'
 import { repositories, subscriptions } from '@db/schema'
@@ -21,6 +21,10 @@ export async function handleList(
     .from(subscriptions)
     .innerJoin(repositories, eq(subscriptions.repositoryId, repositories.id))
     .where(eq(subscriptions.channelId, ctx.channelId))
+    .orderBy(
+      sql`lower(${repositories.owner})`,
+      sql`lower(${repositories.repo})`,
+    )
 
   if (rows.length === 0) {
     await ctx.respond(

src/slack/commands/parse.test.ts

@@ -49,4 +49,11 @@ describe('parseRepo', () => {
   test('returns null for empty string', () => {
     expect(parseRepo('')).toBeNull()
   })
+
+  test('lowercases owner and repo', () => {
+    expect(parseRepo('Torvalds/Linux')).toEqual({
+      owner: 'torvalds',
+      name: 'linux',
+    })
+  })
 })

src/slack/commands/parse.ts

@@ -15,5 +15,8 @@ export function parseRepo(arg: string): { owner: string; name: string } | null {
   }
 
   const idx = arg.indexOf('/')
-  return { owner: arg.slice(0, idx), name: arg.slice(idx + 1) }
+  return {
+    owner: arg.slice(0, idx).toLowerCase(),
+    name: arg.slice(idx + 1).toLowerCase(),
+  }
 }