fix(plugin-search): exclude skipped drafts in reindex handler (#14224)

<!--

Thank you for the PR! Please go through the checklist below and make
sure you've completed all the steps.

Please review the
[CONTRIBUTING.md](https://github.com/payloadcms/payload/blob/main/CONTRIBUTING.md)
document in this repository if you haven't already.

The following items will ensure that your PR is handled as smoothly as
possible:

- PR Title must follow conventional commits format. For example, `feat:
my new feature`, `fix(plugin-seo): my fix`.
- Minimal description explained as if explained to someone not
immediately familiar with the code.
- Provide before/after screenshots or code diffs if applicable.
- Link any related issues/discussions from GitHub or Discord.
- Add review comments if necessary to explain to the reviewer the logic
behind a change

### What?

### Why?

### How?

Fixes #

-->
### What?
This PR aims to provide better insight into how many documents were
reindexed. Currently, the reindex handler will ignore document status
and claim all documents were reindexed even when none were due to being
drafts (as per plugin-search default behavior).

This PR takes into consideration `syncDrafts` enabled in the plugin
config and whether the collection of the document being reindexed has
`drafts` enabled. In this way, we can "skip" attempting to reindex
documents that are drafts when syncDrafts is not true _and_ provide a
useful count for the end-user of how many draft docs were skipped during
reindexing.

### Why?
To provide more insight into how many documents/drafts were reindexed,
as well as provide a minor perf improvement when reindexing collections
with `syncDrafts: false`.

### How?
- By tracking document count in collections with drafts enabled via
`totalDocsWithDrafts`.
- Filtering the find in `reindexCollection` based on if `drafts` &
`syncDrafts` are enabled
 
 Before:
 

[Search-Results---Payload--before.webm](https://github.com/user-attachments/assets/bfd7696d-1124-4df5-8f99-d86c282f485c)

 After:
 

[Search-Results---Payload.webm](https://github.com/user-attachments/assets/7b0d5c84-328e-43e9-b0ae-5bcd0a0e9f07)

Author: akhrarovsaid

packages/plugin-search/src/utilities/generateReindexHandler.ts

@@ -1,4 +1,4 @@
-import type { PayloadHandler } from 'payload'
+import type { PayloadHandler, Where } from 'payload'
 
 import {
   addLocalesToRequestFromData,
@@ -82,21 +82,28 @@ export const generateReindexHandler =
     }
 
     const payload = req.payload
-    const batchSize = pluginConfig.reindexBatchSize
+    const { reindexBatchSize: batchSize, syncDrafts } = pluginConfig
 
     const defaultLocalApiProps = {
       overrideAccess: false,
       req,
       user: req.user,
     }
+    const whereStatusPublished: Where = {
+      _status: {
+        equals: 'published',
+      },
+    }
+    let aggregateDocsWithDrafts = 0
     let aggregateErrors = 0
     let aggregateDocs = 0
 
-    const countDocuments = async (collection: string): Promise<number> => {
+    const countDocuments = async (collection: string, drafts?: boolean): Promise<number> => {
       const { totalDocs } = await payload.count({
         collection,
         ...defaultLocalApiProps,
         req: undefined,
+        where: drafts ? undefined : whereStatusPublished,
       })
       return totalDocs
     }
@@ -112,8 +119,16 @@ export const generateReindexHandler =
     }
 
     const reindexCollection = async (collection: string) => {
-      const totalDocs = await countDocuments(collection)
+      const draftsEnabled = Boolean(payload.collections[collection]?.config.versions?.drafts)
+
+      const totalDocsWithDrafts = await countDocuments(collection, true)
+      const totalDocs =
+        syncDrafts || !draftsEnabled
+          ? totalDocsWithDrafts
+          : await countDocuments(collection, !draftsEnabled)
       const totalBatches = Math.ceil(totalDocs / batchSize)
+
+      aggregateDocsWithDrafts += totalDocsWithDrafts
       aggregateDocs += totalDocs
 
       for (let j = 0; j < reindexLocales.length; j++) {
@@ -128,6 +143,7 @@ export const generateReindexHandler =
             limit: batchSize,
             locale: localeToSync,
             page: i + 1,
+            where: syncDrafts || !draftsEnabled ? undefined : whereStatusPublished,
             ...defaultLocalApiProps,
           })
 
@@ -171,7 +187,8 @@ export const generateReindexHandler =
     const message = t('general:successfullyReindexed', {
       collections: collections.join(', '),
       count: aggregateDocs - aggregateErrors,
-      total: aggregateDocs,
+      skips: syncDrafts ? 0 : aggregateDocsWithDrafts - aggregateDocs,
+      total: aggregateDocsWithDrafts,
     })
 
     if (shouldCommit) {

packages/translations/src/languages/ar.ts

@@ -406,7 +406,7 @@ export const arTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} تم إنشاؤها بنجاح.',
     successfullyDuplicated: '{{label}} تم استنساخها بنجاح.',
     successfullyReindexed:
-      'تم إعادة فهرسة {{count}} من أصل {{total}} مستندات من {{collections}} مجموعات بنجاح.',
+      'تمت إعادة فهرسة {{count}} من أصل {{total}} مستند من {{collections}} وتخطي {{skips}} مسودة.',
     takeOver: 'تولي',
     thisLanguage: 'العربية',
     time: 'الوقت',

packages/translations/src/languages/az.ts

@@ -418,7 +418,7 @@ export const azTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} uğurla yaradıldı.',
     successfullyDuplicated: '{{label}} uğurla dublikatlandı.',
     successfullyReindexed:
-      '{{collections}} kolleksiyalarından {{total}} sənəddən {{count}} sənəd uğurla yenidən indeksləndi.',
+      '"{{collections}}" kolleksiyalarından {{total}} sənəddən {{count}} sənəd uğurla yenidən indeksləndi və {{skips}} qaralama keçildi.',
     takeOver: 'Əvvəl',
     thisLanguage: 'Azərbaycan dili',
     time: 'Vaxt',

packages/translations/src/languages/bg.ts

@@ -415,7 +415,7 @@ export const bgTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} успешно създаден.',
     successfullyDuplicated: '{{label}} успешно дупликиран.',
     successfullyReindexed:
-      'Успешно преиндексирани {{count}} от {{total}} документа от {{collections}} колекции.',
+      'Успешно преиндексирани {{count}} от общо {{total}} документа от {{collections}} и пропуснати {{skips}} чернови.',
     takeOver: 'Поемане',
     thisLanguage: 'Български',
     time: 'Време',

packages/translations/src/languages/bnBd.ts

@@ -421,7 +421,7 @@ export const bnBdTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} সফলভাবে তৈরি করা হয়েছে।',
     successfullyDuplicated: '{{label}} সফলভাবে ডুপ্লিকেট করা হয়েছে।',
     successfullyReindexed:
-      '{{collections}} থেকে {{total}} ডকুমেন্টের মধ্যে {{count}} টি সফলভাবে পুনরায় সূচিবদ্ধ করা হয়েছে',
+      '{{collections}} থেকে মোট {{total}}টি ডকুমেন্টের মধ্যে {{count}}টি সফলভাবে পুনরায় ইনডেক্স করা হয়েছে এবং {{skips}}টি খসড়া বাদ দেওয়া হয়েছে।',
     takeOver: 'দায়িত্ব নিন',
     thisLanguage: 'বাংলা (বাংলাদেশ)',
     time: 'সময়',

packages/translations/src/languages/bnIn.ts

@@ -420,7 +420,7 @@ export const bnInTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} সফলভাবে তৈরি করা হয়েছে।',
     successfullyDuplicated: '{{label}} সফলভাবে ডুপ্লিকেট করা হয়েছে।',
     successfullyReindexed:
-      '{{collections}} থেকে {{total}} ডকুমেন্টের মধ্যে {{count}} টি সফলভাবে পুনরায় সূচিবদ্ধ করা হয়েছে',
+      '{{collections}} থেকে মোট {{total}}টি নথির মধ্যে {{count}}টি সফলভাবে পুনরায় সূচীকৃত হয়েছে এবং {{skips}}টি খসড়া বাদ দেওয়া হয়েছে।',
     takeOver: 'দায়িত্ব নিন',
     thisLanguage: 'বাংলা (বাংলাদেশ)',
     time: 'সময়',

packages/translations/src/languages/ca.ts

@@ -418,7 +418,7 @@ export const caTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} creada correctament.',
     successfullyDuplicated: '{{label}} duplicada correctament.',
     successfullyReindexed:
-      'Reindexació correcta de {{count}} de {{total}} documents de {{collections}}',
+      "S'han reindexat correctament {{count}} de {{total}} documents de {{collections}} i s'han omès {{skips}} esborranys.",
     takeOver: 'Prendre el control',
     thisLanguage: 'Catala',
     time: 'Temps',

packages/translations/src/languages/cs.ts

@@ -414,7 +414,7 @@ export const csTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} úspěšně vytvořeno.',
     successfullyDuplicated: '{{label}} úspěšně duplikováno.',
     successfullyReindexed:
-      'Úspěšně přeindexováno {{count}} z {{total}} dokumentů z {{collections}} kolekcí.',
+      'Úspěšně reindexováno {{count}} z {{total}} dokumentů z {{collections}} a přeskočeno {{skips}} konceptů.',
     takeOver: 'Převzít',
     thisLanguage: 'Čeština',
     time: 'Čas',

packages/translations/src/languages/da.ts

@@ -415,7 +415,7 @@ export const daTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} oprettet.',
     successfullyDuplicated: '{{label}} duplikeret.',
     successfullyReindexed:
-      '{{count}} ud af {{total}} dokumenter fra {{collections}} samlinger blev genindekseret med succes.',
+      '{{count}} ud af {{total}} dokumenter fra {{collections}} blev succesfuldt genindekseret, og {{skips}} kladder blev sprunget over.',
     takeOver: 'Overtag',
     thisLanguage: 'Dansk',
     time: 'Tid',

packages/translations/src/languages/de.ts

@@ -426,7 +426,7 @@ export const deTranslations: DefaultTranslationsObject = {
     successfullyCreated: '{{label}} erfolgreich erstellt.',
     successfullyDuplicated: '{{label}} wurde erfolgreich dupliziert.',
     successfullyReindexed:
-      'Erfolgreich {{count}} von {{total}} Dokumenten aus {{collections}} Sammlungen neu indiziert.',
+      '{{count}} von insgesamt {{total}} Dokumenten aus {{collections}} wurden erfolgreich neu indexiert, {{skips}} Entwürfe wurden übersprungen.',
     takeOver: 'Übernehmen',
     thisLanguage: 'Deutsch',
     time: 'Zeit',