fix(server): use substring matching for person name search (#26903)

server/src/queries/person.repository.sql

@@ -200,13 +200,10 @@ select
 from
   "person"
 where
-  (
+  "person"."ownerId" = $1
-    "person"."ownerId" = $1
+  and f_unaccent ("person"."name") %>> f_unaccent ($2)
-    and (
+order by
-      lower("person"."name") like $2
+  f_unaccent ("person"."name") <->>> f_unaccent ($3)
-      or lower("person"."name") like $3
-    )
-  )
 limit
   $4
 

server/src/repositories/person.repository.ts

@@ -312,16 +312,10 @@ export class PersonRepository {
     return this.db
       .selectFrom('person')
       .selectAll('person')
-      .where((eb) =>
+      .where('person.ownerId', '=', userId)
-        eb.and([
+      .where(() => sql`f_unaccent("person"."name") %>> f_unaccent(${personName})`)
-          eb('person.ownerId', '=', userId),
+      .orderBy(sql`f_unaccent("person"."name") <->>> f_unaccent(${personName})`)
-          eb.or([
+      .limit(100)
-            eb(eb.fn('lower', ['person.name']), 'like', `${personName.toLowerCase()}%`),
-            eb(eb.fn('lower', ['person.name']), 'like', `% ${personName.toLowerCase()}%`),
-          ]),
-        ]),
-      )
-      .limit(1000)
       .$if(!withHidden, (qb) => qb.where('person.isHidden', '=', false))
       .execute();
   }

server/src/schema/migrations/1773846750001-AddPersonNameTrigramIndex.ts

@@ -0,0 +1,11 @@
+import { Kysely, sql } from 'kysely';
+
+export async function up(db: Kysely<any>): Promise<void> {
+  await sql`CREATE INDEX "idx_person_name_trigram" ON "person" USING gin (f_unaccent("name") gin_trgm_ops);`.execute(db);
+  await sql`INSERT INTO "migration_overrides" ("name", "value") VALUES ('index_idx_person_name_trigram', '{"type":"index","name":"idx_person_name_trigram","sql":"CREATE INDEX \\"idx_person_name_trigram\\" ON \\"person\\" USING gin (f_unaccent(\\"name\\") gin_trgm_ops);"}'::jsonb);`.execute(db);
+}
+
+export async function down(db: Kysely<any>): Promise<void> {
+  await sql`DROP INDEX "idx_person_name_trigram";`.execute(db);
+  await sql`DELETE FROM "migration_overrides" WHERE "name" = 'index_idx_person_name_trigram';`.execute(db);
+}

server/src/schema/tables/person.table.ts

@@ -5,6 +5,7 @@ import {
   CreateDateColumn,
   ForeignKeyColumn,
   Generated,
+  Index,
   PrimaryGeneratedColumn,
   Table,
   Timestamp,
@@ -16,6 +17,11 @@ import { AssetFaceTable } from 'src/schema/tables/asset-face.table';
 import { UserTable } from 'src/schema/tables/user.table';
 
 @Table('person')
+@Index({
+  name: 'idx_person_name_trigram',
+  using: 'gin',
+  expression: 'f_unaccent("name") gin_trgm_ops',
+})
 @UpdatedAtTrigger('person_updatedAt')
 @AfterDeleteTrigger({
   scope: 'statement',