גודל חבילות js - #3048
Conversation
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
Cursor Agent can help with this pull request. Just |
🧯 Dangerous deletes guard reportPolicy: see .cursorrules — dangerous deletions are blocked unless wrapped safely. Summary:
Flagged findings (file:line:snippet): Excluded matches (by path pattern) |
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
⏱️ Performance report(No performance test durations collected. Mark tests with |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
📖 Documentation PreviewThe documentation has been built successfully!
To view locally:
|
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: amir haim <amirbiron@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
✨ תיאור קצר
שיפרנו באופן משמעותי את ביצועי טעינת הקבצים הנעוצים (Pinned Files) בדשבורד. במקום להשתמש ב-Aggregation Pipeline כבד, עברנו לשאילתות ישירות על אינדקס חדש, מה שמפחית את זמן הטעינה ממאות מילישניות בודדות למילישניות בודדות.
📦 שינויים עיקריים
פירוט נקודות (רשימת תבליטים):
get_pinned_files()ו-get_pinned_count()ב-database/manager.pyשונו משימוש ב-aggregate()לשימוש ב-find()ו-count_documents()ישירים.code_snippetsבשםuser_active_pinned_pin_order_idx(וב-app.pyבשםuser_active_pinned_pin_order) על השדות(user_id, is_active, is_pinned, pin_order, pinned_at)כדי לתמוך בשאילתות החדשות.tests/test_pin_to_dashboard.pyכדי להתאים לשינויים בשיטות הגישה ל-DB.🧪 בדיקות
aggregate()ל-find()/count_documents().🧪 בדיקות נדרשות ב‑PR
📝 סוג שינוי
✅ צ'קליסט
services/register_jobs.py(כולל Callback/Trigger להפעלה ידנית — למשלcallback_name/trigger_funcלפי המבנה) כדי שיופיעו בדשבורדdocs/environment-variables.rstוגםservices/config_inspector_service.pydocs/webapp/theming_and_css.rst+FEATURE_SUGGESTIONS/theme_matrix.md🧩 השפעות/סיכונים
is_pinned: trueקיים רק על הגרסה האחרונה של קובץ. נוסף מנגנון הגנה קטן ב-get_pinned_filesלטיפול במקרה נדיר של דאטה "מלוכלך".🔗 קישורים
🧯 סיכון / החזרה לאחור (Rollback)