Claude/add auto backup ds6iu

[amirbiron]

תבנית Pull Request

✨ תיאור קצר

• מה שיניתם ולמה? (2-3 משפטים)

📦 שינויים עיקריים

• קוד (Backend)
• בוט טלגרם
• מסד נתונים/מיגרציות
• תיעוד (docs/)
• DevOps/CI/CD

פירוט נקודות (רשימת תבליטים):

🧪 בדיקות

• איך בדקתם? מה עבר? מה נשאר?
• Unit
• Integration
• Manual

🧪 בדיקות נדרשות ב‑PR

• 🔍 Code Quality & Security
• Unit Tests (3.11)
• Unit Tests (3.12)

📝 סוג שינוי

• feat: פיצ'ר חדש
• fix: תיקון באג
• docs: שינוי תיעוד בלבד
• refactor: שינוי קוד ללא שינוי התנהגות
• perf: שיפור ביצועים
• chore/ci: תשתית/CI
• breaking change: שינוי שובר תאימות

דוגמאות Conventional Commits

סוג	דוגמה להודעה	מתי להשתמש
feat	feat: הוספת מסך הגדרות	פיצ'ר חדש למשתמש
fix	fix: תיקון קריסה בעת התחברות	תיקון באג מול משתמשים/פרודקשן
chore	chore: שדרוג Gradle ל-8.9	תחזוקה, כלי פיתוח, housekeeping
docs	docs: עדכון README עם הוראות התקנה	שינויי תיעוד בלבד
refactor	refactor: חילוץ Repository ל-UseCases	שינוי מבני ללא שינוי התנהגות
test	test: הוספת בדיקות ל-LoginViewModel	הוספת/עדכון בדיקות
build	build: הוספת flavor staging ל-CI	שינויים בבילד/תלויות/תצורה

✅ צ'קליסט

• הקוד עוקב אחרי הסגנון (Black/isort/flake8/mypy)
• בדיקות רצות ועוברות
• תיעוד עודכן (README/Docs)
• אם נוספו ג'ובים חדשים (Background Jobs) – וודא שהם רשומים ב-services/register_jobs.py (כולל Callback/Trigger להפעלה ידנית — למשל callback_name/trigger_func לפי המבנה) כדי שיופיעו בדשבורד
• אם נוספו/שונו משתני סביבה – עודכן docs/environment-variables.rst וגם services/config_inspector_service.py
• אם נוספו/השתנו טוקנים – עודכן גם docs/webapp/theming_and_css.rst + FEATURE_SUGGESTIONS/theme_matrix.md
• אין סודות/מפתחות בקוד
• אין מחיקות מסוכנות/פעולות על root (ראו .cursorrules)
• הודעת הקומיט תואמת Conventional Commits (ע"פ הטבלה)
• CHANGELOG עודכן אם נדרש
• כל ה‑Required Checks לעיל ירוקים
• צילום/וידאו UI מצורף אם רלוונטי

🧩 השפעות/סיכונים

• השפעה אפשרית על פרודקשן, ביצועים, או אבטחה:

🔗 קישורים

• Issues קשורים: #
• Docs Preview:
• מסמכים/מפרטים רלוונטיים:
• Branch Protection & PR Rules

🧯 סיכון / החזרה לאחור (Rollback)

• תוכנית חזרה לאחור במקרה תקלה:

[github-actions]

🧯 Dangerous deletes guard report

Policy: see .cursorrules — dangerous deletions are blocked unless wrapped safely.

Summary:

• Flagged findings (blocking): 0
  0
• Excluded matches (not blocking): 15
• Total matches (all files): 129

Flagged findings (file:line:snippet):
(none)

Excluded matches (by path pattern)

./Dockerfile:42:    rm -rf /var/lib/apt/lists/*
./Dockerfile:121:    rm -rf /var/lib/apt/lists/*
./node_modules/katex/src/fonts/Makefile:139:	rm -rf pfa ff otf ttf woff woff2
./node_modules/katex/package.json:153:    "build": "rimraf dist/ && mkdirp dist && cp README.md dist && rollup -c --failAfterWarnings && webpack && node update-sri.js package dist/README.md",
./node_modules/mermaid/dist/mermaid.min.js:1524:`,"getStyles"),c1e=RQe});var h1e={};dr(h1e,{diagram:()=>NQe});var NQe,f1e=N(()=>{"use strict";$ge();a1e();l1e();u1e();NQe={parser:Fge,db:n1e,renderer:o1e,styles:c1e}});var m1e,g1e=N(()=>{"use  … [truncated]
./node_modules/mermaid/dist/mermaid.min.js.map:4:  "sourcesContent": ["/**\n* Default values for dimensions\n*/\nconst defaultIconDimensions = Object.freeze({\n\tleft: 0,\n\ttop: 0,\n\twidth: 16,\n\theight: 16\n});\n/**\n* Default values fo … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm/chunk-2M32CCKP.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequence d … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm.min/chunk-4HFYJGYH.mjs:1:var r={name:"mermaid",version:"11.12.0",description:"Markdown-ish syntax for generating flowcharts, mindmaps, sequence diagrams, class diagrams, gantt charts, git graph … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm.min/chunk-4HFYJGYH.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequen … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.core/chunk-KS23V3DP.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequence  … [truncated]
./node_modules/mermaid/dist/mermaid.js.map:4:  "sourcesContent": ["/**\n* Default values for dimensions\n*/\nconst defaultIconDimensions = Object.freeze({\n\tleft: 0,\n\ttop: 0,\n\twidth: 16,\n\theight: 16\n});\n/**\n* Default values for tr … [truncated]
./README.md:842:find . -name "__pycache__" -exec rm -rf {} +
./docs/DOCUMENTATION_GUIDE.md:453:rm -rf _build
./docs/Makefile:24:	rm -rf $(BUILDDIR)
./webapp/static/js/md_preview.bundle.js.map:4:  "sourcesContent": ["// Markdown-it plugin to render GitHub-style task lists; see\n//\n// https://github.com/blog/1375-task-lists-in-gfm-issues-pulls-comments\n// https://github.com/blog/1825-t … [truncated]

[github-actions]

⏱️ Performance report

• Run: https://github.com/amirbiron/CodeBot/actions/runs/23962373805
• Artifact: perf-durations

(No performance test durations collected. Mark tests with @pytest.mark.performance.)

[chatgpt-codex-connector]

💡 Codex Review

CodeBot/webapp/backup_api.py

Lines 322 to 326 in e70116f

	existing = db.db.restore_jobs.find_one(
	{"user_id": int(user_id), "status": "running"},
	projection={"_id": 1},
	)
	if existing:

Serialize restore job claim per user

The async restore guard is a non-atomic check-then-insert: this find_one runs before _create_restore_job, so two concurrent /api/backup/restore-async requests from the same user can both pass the check and start parallel restores. That breaks the endpoint’s single-restore invariant and can lead to overlapping writes during restore. Use an atomic claim (e.g., unique partial index for status="running" per user or a single find_one_and_update/upsert-based lock).

---

CodeBot/webapp/backup_scheduler.py

Lines 91 to 94 in e70116f

	ts = _now_utc().strftime("%Y%m%d_%H%M%S")
	filename = f"webapp_backup_{user_id}_{ts}.zip"
	filepath = backup_dir / filename
	filepath.write_bytes(zip_bytes)

Generate unique disk backup filenames

Disk backup filenames are only second-granular (%Y%m%d_%H%M%S), so two backups for the same user in the same second (for example, manual trigger plus scheduler, or rapid retries) resolve to the same path and write_bytes overwrites the earlier archive. This can silently lose a backup copy; include sub-second precision or a random suffix in the filename.

---

CodeBot/webapp/drive_backup_api.py

Lines 117 to 119 in e70116f

	# מחכים עד 60 שניות
	result = future.result(timeout=60)
	return jsonify(result)

Return asynchronous status for long Drive backups

This endpoint waits only 60 seconds for future.result(), but backup uploads can legitimately exceed that; on timeout it returns an error while the worker thread continues running. Users then see a failed operation and may retrigger, causing duplicate/overlapping Drive backups. Expose a job status (or longer-lived async response) instead of converting slow-but-running work into an immediate failure.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[sentry]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[github-actions]

📖 Documentation Preview

The documentation has been built successfully!

• 📦 Download Documentation Artifacts
• 🔍 Check the workflow summary for coverage report
• 📊 Build completed at: Fri, 03 Apr 2026 21:13:31 GMT

To view locally:

1. Download the artifacts
2. Extract the zip file
3. Open index.html in your browser

[cursor]

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.}