Respond to github configuration issues

[amirbiron]

✨ תיאור קצר

תיקנו את ולידציית קבצי הקונפיגורציה (כמו alerts.yml) כדי לטפל נכון במספרים שלמים ולספק הודעות שגיאה ברורות יותר. בנוסף, שיפרנו את ממשק המשתמש כך שיציג את כל רשימת בעיות הקונפיגורציה במקום הודעה מקוצרת.

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

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

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

• הוספת pyyaml==6.0.2 ל־requirements/base.txt ו־requirements/minimal.txt לטיפול בפענוח YAML.
• שיפור לוגיקת טעינת קבצי קונפיגורציה (alerts.yml, error_signatures.yml, image_settings.yaml) ב־webapp/config_radar.py לזיהוי כשלים בפענוח YAML/JSON והצגת הודעות שגיאה מפורטות.
• עדכון webapp/templates/settings.html להצגת רשימה מלאה וממוספרת של כל בעיות הוולידציה, במקום הודעת "ועוד בעיות נוספות...".
• הוספת בדיקת יחידה ב־tests/test_config_radar_api.py לוודא שהמערכת מדווחת נכון על חוסר במודול PyYAML.

🧪 בדיקות

• איך בדקתם? מה עבר? מה נשאר?
  • נוסף טסט יחידה ב־test_config_radar_api.py המדמה חוסר ב־PyYAML ומוודא שהודעת השגיאה המתאימה מוצגת.
  • בדיקה ידנית של ה־UI (מסך Config Radar) לוודא שכל רשימת השגיאות מוצגת כראוי עם מיספור וכמות כוללת.
  • לא הורצו בדיקות אוטומטיות מלאות בסביבת הפיתוח של הבוט, אך מומלץ להריץ python3 -m pytest tests/test_config_radar_api.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)
• אין סודות/מפתחות בקוד
• אין מחיקות מסוכנות/פעולות על root (ראו .cursorrules)
• הודעת הקומיט תואמת Conventional Commits (ע"פ הטבלה)
• CHANGELOG עודכן אם נדרש
• כל ה‑Required Checks לעיל ירוקים
• צילום/וידאו UI מצורף אם רלוונטי

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

• השפעה אפשרית על פרודקשן, ביצועים, או אבטחה: הוספת תלות חדשה (PyYAML). שיפור בוולידציה עשוי לחשוף בעיות קונפיגורציה קיימות שלא זוהו קודם לכן.

🔗 קישורים

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

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

• תוכנית חזרה לאחור במקרה תקלה: החזרה לגרסה קודמת של הקוד.

---

---

Note

Strengthens YAML/JSON parsing and error reporting for Config Radar, updates UI to show full numbered issues, adds PyYAML dependency, and includes a test for missing PyYAML.

• Backend (Config Radar):
  • Refactor _parse_yaml_or_json to return (data, issue) with explicit YAML/JSON error messages; handle missing PyYAML gracefully.
  • Propagate parse issues in _load_mapping to validation output; improve integer coercion/validation flow untouched elsewhere.
  • Affects loaders for alerts.yml, error_signatures.yml, image_settings.yaml in webapp/config_radar.py.
• Frontend/UI:
  • In webapp/templates/settings.html, display full, numbered validation issues with total count and accessibility hint (aria-live), plus minor styling tweaks for .radar-issues.
• Tests:
  • Add tests/test_config_radar_api.py::test_config_radar_reports_missing_yaml to verify error messaging when PyYAML is unavailable.
• Dependencies:
  • Add pyyaml==6.0.2 to requirements/base.txt and requirements/minimal.txt.

^{Written by Cursor Bugbot for commit ac53999. 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:37: rm -rf /var/lib/apt/lists/*
./Dockerfile:109: 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:800: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/19847795358
• 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: Tue, 02 Dec 2025 05:02:46 GMT

To view locally:

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