Respond to github message

[amirbiron]

✨ תיאור קצר

• טיפלתי במספר באגים בעורך: שמירת מיקום הסמן בעת מעבר בין עורכים, ותיקון כפתור ההדבקה שלא מחק את כל המסמך.
• הוספתי יכולת להעלות תמונות לקובצי Markdown, כולל ממשק משתמש, תצוגה מקדימה וטיפול בצד השרת.

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

• קוד (Backend)
• בוט טלגרם
• מסד נתונים/מיגרציות
• תיעוד (docs/) - הערה: לא עודכן בפועל, אך רלוונטי לפיצ'ר
• DevOps/CI/CD

פירוט נקודות:

• webapp/static/js/editor-manager.js:
  • הטמעת פונקציות captureSelection, restoreSelection ו-insertTextAtCursor לשמירת מיקום הסמן בעת מעבר בין עורכים (פשוט ו-CodeMirror).
  • כפתור ההדבקה (Paste) ופעולת הדבקה כללית מכניסים כעת טקסט במיקום הסמן/הבחירה הנוכחית במקום להחליף את כל תוכן העורך.
• webapp/templates/edit_file.html:
  • הוספת רכיבי UI חדשים לעמוד עריכת קובץ: כפתור "הוסף תמונה" (🖼️), אזור תצוגה מקדימה לתמונות Markdown, הודעות שגיאה ומצב, ותצוגת מסך מלא לתמונות.
  • כפתור העלאת התמונות מופיע רק כאשר הקובץ הוא מסוג Markdown (לפי סיומת .md או בחירת שפה).
  • טיפול בצד הלקוח לבחירת תמונות, כולל ולידציה על סוג קובץ (PNG, JPG, WEBP, GIF), גודל (עד 2MB) וכמות (עד 6 תמונות).
• webapp/app.py:
  • לוגיקת ה-edit_file_page עודכנה לקבל ולטפל בתמונות המצורפות לקובצי Markdown.
  • ולידציה בצד השרת עבור סוג, גודל וכמות התמונות, בדומה לולידציה בצד הלקוח.
  • קריאה לפונקציה _save_markdown_images לשמירת התמונות המשויכות לגרסה החדשה של קובץ ה-Markdown.

🧪 בדיקות

• Manual:
  • מעבר בין העורכים (פשוט ל-CodeMirror ובחזרה) ולוודא שמיקום הסמן נשמר ושפעולת הדבקה אינה מוחקת את כל המסמך.
  • עריכת קובץ Markdown, צירוף מספר תמונות (כולל מחיקה, תצוגה מלאה), שמירת הקובץ ואימות שהתמונות נשמרו ומוצגות בגרסה החדשה.
  • פתיחת קובץ שאינו Markdown ולוודא שכפתור העלאת התמונות אינו גלוי ושהשרת אינו מקבל קובצי md_images.

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

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

📝 סוג שינוי

• feat: פיצ'ר חדש
• fix: תיקון באג

✅ צ'קליסט

• הקוד עוקב אחרי הסגנון (Black/isort/flake8/mypy)
• בדיקות רצות ועוברות (בדיקות אוטומטיות לא הורצו על ידי ה-AI)
• תיעוד עודכן (README/Docs)
• אין סודות/מפתחות בקוד
• אין מחיקות מסוכנות/פעולות על root (ראו .cursorrules)
• הודעת הקומיט תואמת Conventional Commits (ע"פ הטבלה)
• CHANGELOG עודכן אם נדרש
• כל ה‑Required Checks לעיל ירוקים
• צילום/וידאו UI מצורף אם רלוונטי

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

• הוספת תמונות לקובצי Markdown עלולה להגדיל את נפח הנתונים הנשמרים ב-DB ואת גודל הבקשות לשרת.
• סיכון קל לבעיות תאימות בדפדפנים ישנים יותר עם ה-JS החדש של העלאת התמונות.

🔗 קישורים

• Issues קשורים: #

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

• במקרה של תקלה, ניתן לבצע Rollback לגרסה הקודמת של הקוד. יש לוודא שאין תמונות שבורות שנוצרו ב-DB.

---

---

Note

Adds Markdown image attachments with UI and server save, preserves cursor on editor/preview toggles, and syncs theme-wizard preference to user settings.

• Backend:
  • Markdown images: Handle md_images on edit/upload (/edit/<id>), validate type/size/count, persist via _save_markdown_images, and serve via GET /file/<id>/images/<image_id>.
  • File view: Include associated Markdown images in view_file payload.
  • UI prefs: Extend /api/ui_prefs to store ui_prefs.theme_wizard_seen; expose theme_wizard_seen in template globals.
• Frontend:
  • Editor (static/js/editor-manager.js): Add captureSelection/restoreSelection and insertTextAtCursor; keep cursor/selection when switching simple↔CodeMirror and improve paste.
  • Live Preview (static/js/live-preview.js): Refine mode eligibility (lang vs. extension), preserve caret state on enable/disable, and minor UX tweaks.
  • Edit File UI (templates/edit_file.html): Add Markdown image picker/preview/remove/fullscreen, status/errors, and form integration (multi-file); show trigger only for .md/Markdown.
  • Theme Wizard & Welcome (templates/base.html): Persist wizard seen-state locally and via /api/ui_prefs; make welcome ack robust.
  • Split view CSS: Improve panel sizing/overflow behavior for preview/editor.

^{Written by Cursor Bugbot for commit 706021b. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[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): 125

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/*
./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]
./README.md:836:find . -name "__pycache__" -exec rm -rf {} +
./docs/DOCUMENTATION_GUIDE.md:453:rm -rf _build
./docs/Makefile:24:	rm -rf $(BUILDDIR)
./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/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.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/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/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.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]
./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/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/katex/src/fonts/Makefile:139:	rm -rf pfa ff otf ttf woff woff2

[github-actions]

⏱️ Performance report

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

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

[sentry]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[github-actions]

📖 Documentation Preview

The documentation has been built successfully!

• 📦 Download Documentation Artifacts
• 🔍 Check the workflow summary for coverage report
• 📊 Build completed at: Thu, 11 Dec 2025 07:50:30 GMT

To view locally:

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