Github message response

[amirbiron]

✨ תיאור קצר

הוספנו פאנל ניהול לערכות נושא ציבוריות (Shared Themes) עבור אדמינים, תיקנו בעיית "מסך לבן" (White Screen Flash) בעת מעבר בין ערכות נושא, ושיפרנו את מנגנון פרסום ערכות נושא בבונה הערכות כדי לשמר את כל משתני ה-CSS המקוריים. שינויים אלו משפרים את חווית המשתמש, שלמות הערכות ויכולות הניהול.

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

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

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

• הוספת פאנל ניהול לערכות נושא ציבוריות (Shared Themes) במסך ההגדרות (settings.html) המאפשר לאדמינים להציג ולמחוק ערכות.
• הטמעת פתרון למניעת "מסך לבן" בעת טעינת עמוד עם ערכת נושא חדשה, באמצעות מסכת מעבר ב-base.html ושימוש ב-sessionStorage.
• שיפור מנגנון הפרסום בבונה הערכות (theme_builder.html) כדי לשמר את כל משתני ה-CSS המקוריים של ערכה, גם אלו שאינם מוצגים בטופס, ובכך למנוע מחיקתם בטעות.
• הוספת משתני CSS חסרים (כמו --md-inline-code-bg, --md-table-bg) ל-whitelist ב-theme_parser_service.py כדי להבטיח שהם נשמרים בעת פרסום ערכות.

🧪 בדיקות

• בדיקות יחידה רלוונטיות לערכות נושא עברו בהצלחה. בוצעה בדיקה ידנית של פאנל הניהול החדש (הצגה ומחיקה של ערכות ציבוריות), בדיקה ידנית של מעבר בין ערכות נושא לוודא שהמסך הלבן נפתר, ובדיקה ידנית של פרסום ערכה מבונה הערכות לוודא שכל המשתנים נשמרים.
• 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: שינוי שובר תאימות

✅ צ'קליסט

• הקוד עוקב אחרי הסגנון (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)

• תוכנית חזרה לאחור במקרה תקלה: ביצוע Rollback ל-PR.

---

 

---

Note

Introduces admin management for shared themes, smoother theme switching, and richer syntax theming with backward-safe validations.

• Adds admin panel in settings.html to list and delete shared themes; fetches from /api/themes/list and calls DELETE /api/themes/shared/:id
• Prevents white-screen flash on theme change via a mask and sessionStorage (base.html), with transition state saved before reload
• Theme publishing now preserves all original CSS variables by merging form values with source variables, and includes syntax_css + syntax_colors (theme_builder.html)
• Extends whitelist in theme_parser_service.py with Markdown-enhanced tokens (e.g., --md-inline-code-bg, --md-table-bg, --md-mermaid-bg)
• Backend: SharedThemeService and themes_api.py now read/write syntax_colors alongside syntax_css; CRUD paths updated and validated

^{Written by Cursor Bugbot for commit 472b60e. 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:842: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/20725451354
• Artifact: perf-durations

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

[chatgpt-codex-connector]

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

[sentry]

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
services/shared_theme_service.py	20.00%	3 Missing and 1 partial ⚠️

📢 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: Mon, 05 Jan 2026 18:40:30 GMT

To view locally:

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