Perf/api mongo projections

[Kuchizu]

findDailyEventsPortion: $limit before $lookups when no content filter (was the 8.8s prod aggregation). DataLoader batch helpers gain optional projection arg.

[neSpecc]

please explain in docs why it is useful

[neSpecc]

I see no usages. Where is it used?

[Copilot]

Pull request overview

Optimizes the findDailyEventsPortion MongoDB aggregation for the no-content-filters case by moving $limit earlier (before $lookup stages) and extends internal DataLoader batching helpers to optionally accept MongoDB projections to reduce fetched fields.

Changes:

• In findDailyEventsPortion, conditionally applies $limit before $lookup stages when no content filters are present, keeping $match + $limit after lookups when filters exist.
• Adds an optional projection parameter to DataLoader batch helpers (batchByIds / batchByField) and forwards it to MongoDB .find(...).
• Bumps package version to 1.5.1.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
src/models/eventsFactory.js	Moves $limit earlier in the daily-events aggregation when filters are absent to reduce lookup workload.
src/dataLoaders.ts	Adds optional MongoDB projection support to batch helper methods used by DataLoaders.
package.json	Version bump from 1.5.0 to 1.5.1.

Comments suppressed due to low confidence (1)

src/dataLoaders.ts:107

• valuesMap is typed as Map<string, FieldType>, but it stores the actual field values (T[FieldType]). Updating it to Map<string, T[FieldType]> would make the intent clearer and keep types accurate when building the $in array and mapping results.

    type Doc = WithId<T>;
    const valuesMap = new Map<string, FieldType>();

    for (const value of values) {
      valuesMap.set(value.toString(), value);
    }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.