Cursor/worker push delivery url ba24 - #2713
Conversation
הפקודה מציגה עכשיו אבחנה ברורה האם המשתנה חסר ב-ENV או בקונפיג, עם הסבר שהגדרה ב-shell לא עוברת אוטומטית ל-service/Container וטיפ לבדיקה עם url=.
הוספתי שדות Remote Push Delivery ל-BotConfig ועדכנתי /status_worker להשתמש גם ב-config עבור enabled/token.
הפעלתי מחדש את סקשן Web Push בדף settings והוספתי עוגן id=push כדי ש-CTA לדף ההגדרות יגלול למקום הנכון.
הסרתי את ההסתרה לפי PUSH_NOTIFICATIONS_ENABLED כדי שתמיד תהיה אפשרות להפעיל/לכבות בדפדפן ולשלוח בדיקת פוש, עם אזהרה כשהשרת לא שולח אוטומטית.
אתחלתי את Web Push רק אחרי DOM-ready, הוספתי try/catch כדי שחריגה בהגדרות אחרות לא תקטע את הסקשן, והגדרתי type=button לכפתורים. נוספו גם הודעות סטטוס מיידיות כדי לראות שהקליק נקלט.
הוספתי binder קטן (ES5) לכפתורי Web Push בדף settings כדי שכפתורים יעבדו גם אם הסקריפט הגדול נופל בגלל תחביר מודרני (?. / ??) בדפדפנים/וובוויו ישנים.
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
🧯 Dangerous deletes guard reportPolicy: see .cursorrules — dangerous deletions are blocked unless wrapped safely. Summary:
Flagged findings (file:line:snippet): Excluded matches (by path pattern) |
⏱️ Performance report(No performance test durations collected. Mark tests with |
Uh oh!
There was an error while loading. Please reload this page.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
הסקריפט הראשי מסמן data-bound=1 על כפתורי Web Push כדי שה-binder הישן (ES5 fallback) ידלג בדפדפנים מודרניים ולא יווצרו קריאות API כפולות.
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Uh oh!
There was an error while loading. Please reload this page.
מונע מצב שבו data-bound נקבע לפני addEventListener ואז תקלה באמצע חוסמת גם את ה-binder הראשי וגם את ה-ES5 fallback.
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Uh oh!
There was an error while loading. Please reload this page.
הבנתי למה זה “לא עזר”: בדף ההגדרות יש הרבה תחביר JS מודרני (?. ו־??). בוובוויו/דפדפן ישן זה יכול להפיל את כל הסקריפט של הדף עם שגיאת syntax — ואז הכפתורים של Web Push לא יקבלו מאזיני click, ולכן גם לא תראה שום בקשות/לוגים.
כדי לעקוף את זה הוספתי סקריפט קטן ונפרד (ES5) שמחבר את שלושת הכפתורים ומעדכן את pushStatus, גם אם הסקריפט הגדול נפל. זה כבר commit+push לענף cursor/worker-push-delivery-url-ba24 (קומיט: 87030d8).
אחרי הדיפלוי, אמור להיות לפחות שינוי מיידי בטקסט “מצב לא ידוע” כשאתה לוחץ (למשל “בודק תמיכה בדפדפן…” / “שולח פוש בדיקה…”).
Note
Strengthens Web Push settings to work in older browsers/WebViews and avoids duplicate handlers.
pushStatusso buttons function even if the main settings script failsdata-boundand markingpushEnableBtn/pushDisableBtn/pushTestBtnas boundWritten by Cursor Bugbot for commit 00565d8. This will update automatically on new commits. Configure here.