-
-
Notifications
You must be signed in to change notification settings - Fork 202
Enhance KB: Document maxUsers cap and performance trade-offs in DevIndex #9226
Copy link
Copy link
Closed
Labels
Description
Following the implementation of the maxUsers cap (50k limit) for the DevIndex, we need to apply the Knowledge Base Enhancement Strategy to document this architectural decision.
Rationale: If we allowed the index to grow unbounded (e.g., 100k users), the users.jsonl file would reach ~40MB. While we stream and gzip the data, client-side memory constraints and parsing overhead would eventually degrade the application's responsiveness and UX. The cap ensures the app remains fast and "fun to use" while maintaining high quality via the dynamic contribution threshold.
Tasks:
- Update
Storage.mjs: Enhance the class-level JSDoc to explain the pruning mechanism and the performance trade-off rationale behind theusers.jsonlfile size. - Update
config.mjs: Add detailed context to themaxUsersproperty explaining why the 50,000 limit exists (40MB file size, client-side performance). - Update
Updater.mjs: Briefly mention the dynamic threshold interaction with themaxUserscap in the workflow description.
Reactions are currently unavailable