feat: redirect Markdown files to preview view on upload - #3112
Conversation
אחרי העלאה או עריכה של קובץ Markdown, המערכת מפנה עכשיו ישירות לתצוגת Markdown (/md/<file_id>) במקום לתצוגת קוד (/file/<file_id>). https://claude.ai/code/session_01TyHrHY8vpBRNEMizo5gjWS
🧯 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 |
📖 Documentation PreviewThe documentation has been built successfully!
To view locally:
|
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! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit:8b85c0119b
ℹ️ 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".
Uh oh!
There was an error while loading. Please reload this page.
- הוספת 'md' לבדיקת שפה (בנוסף ל-'markdown') בהתאם לקוד קיים - הזזת הגדרת _skip_view_activity_once רק לנתיב view_file - ניקוי הדגל מה-session כשמפנים ל-md_preview כדי למנוע דילוג לא מכוון על לוגינג בביקור הבא ב-view_file https://claude.ai/code/session_01TyHrHY8vpBRNEMizo5gjWS
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
- helper חדש _is_markdown_file(language, file_name) מרכז את כל בדיקות זיהוי Markdown (שפה 'markdown'/'md' + סיומות .md/.markdown) - תיקון activity logging: נתיב Markdown מפעיל _log_webapp_user_activity כמו נתיב רגיל כדי שלא נאבד סטטיסטיקות - תיקון md_preview: מזהה גם קבצי .markdown (לא רק .md) - החלפת 7 בדיקות כפולות פזורות ב-helper המשותף https://claude.ai/code/session_01TyHrHY8vpBRNEMizo5gjWS
There was a problem hiding this comment.
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.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
✨ תיאור קצר
הוספת הפניה ישירה לתצוגת Markdown Preview כאשר משתמש מעלה או יוצר קובץ Markdown, במקום להציג אותו בתצוגת קוד רגילה.
📦 שינויים עיקריים
פירוט נקודות:
language == 'markdown') או בדיקת סיומת הקובץ (.mdאו.markdown)md_previewבמקום ל-view_file🧪 בדיקות
📝 סוג שינוי
✅ צ'קליסט
🧩 השפעות/סיכונים
md_previewבקוד🧯 סיכון / החזרה לאחור (Rollback)
https://claude.ai/code/session_01TyHrHY8vpBRNEMizo5gjWS