From 803504360670fa5fbac234f3f07bdf9bacc60f58 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 25 Aug 2026 20:21:59 +0000
Subject: [PATCH 1/4] =?UTF-8?q?feat(notes):=20=D7=9E=D7=90=D7=A8=D7=A7?=
 =?UTF-8?q?=D7=93=D7=90=D7=95=D7=9F=20=D7=9E=D7=A8=D7=95=D7=A0=D7=93=D7=A8?=
 =?UTF-8?q?=20=D7=9B=D7=91=D7=A8=D7=99=D7=A8=D7=AA=20=D7=9E=D7=97=D7=93?=
 =?UTF-8?q?=D7=9C=20=D7=91=D7=9B=D7=9C=20=D7=A9=D7=9C=D7=95=D7=A9=D7=AA=20?=
 =?UTF-8?q?=D7=99=D7=A2=D7=93=D7=99=20=D7=94=D7=A4=D7=AA=D7=A7=D7=99=D7=9D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

ברירת המחדל הייתה ``!!this.boardId`` — דלוקה בלוח בלבד, כבויה בפתקים שעל
קובץ ב-CodeKeeper ועל קובץ בדפדפן הריפו. הנימוק שנרשם בזמנו היה שאין ליעדים
האלה מתג כיבוי, ולכן רינדור אוטומטי הוא שינוי שקט שאין למשתמש דרך לבטל.

בפועל הדגל לבדו אינו מרנדר כלום: ``_syncTaskView`` דורש גם אותו וגם
``_hasRenderableMarkdown``, ולכן פתק של טקסט רגיל נשאר תיבת עריכה ואינו
משתנה. מה שהשתנה הוא רק פתקים שכבר מכילים ``**`` או ``#`` — כלומר בדיוק
המקרה שבו הרינדור הוא מה שהמשתמש התכוון אליו.

מתג הכיבוי נשאר בלוח בלבד, ו-``markdown`` מפורש ב-opts ממשיך לגבור על
ברירת המחדל.

תיעוד — ``docs/user/sticky_notes.rst``:

- שתי ההצהרות על ברירת המחדל עודכנו לתיאור החדש.
- **פסקת העיגון בפתקי ריפו תוקנה, והיא הייתה שגויה מאז שההתנהגות
  השתנתה תחתיה.** היא טענה ש-``surface`` מעוגן למסגרת התצוגה ושהפתק אינו
  נגלל עם שורות הקוד. בפועל ``surface`` ליעד ריפו מודד מראשית התוכן
  (``_surfaceScrollShift`` מחסיר את ``scrollTop``), מאזין גלילה ייעודי
  מחשב מחדש את המיקום, ו-``_updatePinnedVisibility`` מסתיר פתק שיצא
  מהתחום הנראה — כלומר הפתק כן נגלל עם הקוד. הטענה על ``anchored`` נבדקה
  ונשארה: ``_resolveMode`` חוסם אותו לכל יעד ``surface``.

תיעוד — ``docs/dev/sticky_notes_extending.rst``: סעיף מודגש בסוף שקובע
שכל פיצ'ר או שינוי התנהגות מעדכן את עמוד המשתמש באותו PR, עם הפרדה
שמונעת כפילות בין השניים — התנהגות נראית בעמוד המשתמש, נימוק הנדסי כאן.

טסטים: שני הטסטים שאכפו את ברירת המחדל הישנה עודכנו, ונוספו שני שומרים —
פתק בלי מבנה מארקדאון נשאר תיבת עריכה, ו-``markdown`` מפורש גובר. שניהם
נופלים בלי השינוי.

אומת ב-Chromium מול שני היעדים: הפתק עם המבנה מרונדר בטעינה בלי נגיעה,
ופתק הטקסט נשאר textarea. על הקוד הקודם הדגל הוא ``false`` ואין רינדור.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019YULCppaQRPYN1RgeY6NBu
---
 AI-MAP.md                           |  2 +-
 docs/dev/sticky_notes_extending.rst | 16 +++++++++++-
 docs/user/sticky_notes.rst          |  8 +++---
 tests/sticky-notes-target.test.js   | 39 ++++++++++++++++++++++-------
 webapp/static/js/sticky-notes.js    | 28 +++++++++++++--------
 5 files changed, 68 insertions(+), 25 deletions(-)

diff --git a/AI-MAP.md b/AI-MAP.md
index 215303337..7b9adc3d7 100644
--- a/AI-MAP.md
+++ b/AI-MAP.md
@@ -23,7 +23,7 @@
 - `docs/whats-new.rst` — **What's New**: יומן השינויים של הבוט וה-WebApp לפי תאריך — מה נוסף, מה השתנה ומה תוקן בכל עדכון, עם קישורים ל-Issues הרלוונטיים.
 - `docs/architecture.rst` — **ארכיטקטורה**: המערכת מורכבת מבוט Telegram, שכבת שירותים (services), שכבת נתונים (MongoDB) ואפליקציית Web. הזרימה העיקרית: Handlers → Services → Database.
   - `docs/architecture/clean-architecture.rst` — **Clean Architecture ב-src**: ארכיטקטורה זו מפרידה בין לוגיקה עסקית, תזמור יישומי ותשתיות כך שניתן לבדוק יחידות קוד בנפרד, להחליף מקורות נתונים בלי לשבור את שאר המערכת ולרוץ גם בסביבות ללא MongoDB.
-- `docs/dev/sticky_notes_extending.rst` — **עקרונות להוספת פיצ'ר לסטיקי-נוטס**: המלכודות החוזרות של הפתקים הדביקים: יעד יחיד ב-build_note_target, מכסה fail-closed והפטור-לאדמין שהורג מכסות, flush לפני פעולה הרסנית, אינדקס ממוספר שנבנה לפני שמפילים, נורמליזציה בשני הקצוות, ויתומים בקריאה.
+- `docs/dev/sticky_notes_extending.rst` — **עקרונות להוספת פיצ'ר לסטיקי-נוטס**: המלכודות החוזרות של הפתקים הדביקים: יעד יחיד ב-build_note_target, מכסה fail-closed והפטור-לאדמין, flush לפני פעולה הרסנית, אינדקס ממוספר שנבנה לפני שמפילים, נורמליזציה בשני הקצוות, יתומים בקריאה, ועדכון עמוד המשתמש.
 - `docs/contributing.rst` — **מדריך תרומה**: לתת מסלול ברור לתרומות קוד, עם דגש על סוכני AI ו-CI.
 - `docs/branch-protection-and-pr-rules.rst` — **Branch Protection & PR Rules**: לרכז נהלים ברורים להגנה על ענפים (Branch Protection) ולחוקי PR בפרויקט.
 
diff --git a/docs/dev/sticky_notes_extending.rst b/docs/dev/sticky_notes_extending.rst
index c36317168..14e714052 100644
--- a/docs/dev/sticky_notes_extending.rst
+++ b/docs/dev/sticky_notes_extending.rst
@@ -1,6 +1,6 @@
 עקרונות להוספת פיצ'ר לסטיקי-נוטס
 ==================================
-:summary: המלכודות החוזרות של הפתקים הדביקים: יעד יחיד ב-build_note_target, מכסה fail-closed והפטור-לאדמין שהורג מכסות, flush לפני פעולה הרסנית, אינדקס ממוספר שנבנה לפני שמפילים, נורמליזציה בשני הקצוות, ויתומים בקריאה.
+:summary: המלכודות החוזרות של הפתקים הדביקים: יעד יחיד ב-build_note_target, מכסה fail-closed והפטור-לאדמין, flush לפני פעולה הרסנית, אינדקס ממוספר שנבנה לפני שמפילים, נורמליזציה בשני הקצוות, יתומים בקריאה, ועדכון עמוד המשתמש.
 
 הפיצ'ר של הפתקים הדביקים צובר משטחים לאורך זמן — קובץ ב-CodeKeeper ← לוח ← רינדור מארקדאון ← קובץ בריפו ממורר. כל תוספת כזו נוטה לגלות מחדש את אותן מלכודות, כי הן אינן בקוד של פיצ'ר בודד אלא בחוזה המשותף. העמוד הזה מרכז אותן, כדי שהפיצ'ר הבא (או סוכן AI שממש אותו) יקרא אותן פעם אחת במקום לגלות אותן שוב בפרודקשן.
 
@@ -85,3 +85,17 @@
 - **מודל טהור** (יעד, נורמליזציה, מכסה): טסטי יחידה, בלי מסד. כל טסט חדש נבדק במוטציה — ביטול התיקון חייב להפילו.
 - **אינדקס ייחודי ופילטר חלקי**: מול מונגו אמיתי בלבד. סטאב מחזיר ``None`` מ-``create_index`` וכל בדיקת כפילות עוברת בו, גם אם האינדקס מעולם לא נוצר.
 - **חיווט הלקוח** (הרכבה, שמירה, החלפת יעד): מול דפדפן אמיתי. השוואת מחרוזות אינה מוכיחה שהפתק באמת נשמר ונטען.
+
+התיעוד למשתמש הוא חלק מהפיצ'ר
+------------------------------
+
+.. important::
+   **כל פיצ'ר חדש בפתקים, וכל שינוי בהתנהגות קיימת, מעדכן את** :doc:`/user/sticky_notes` **באותו PR.**
+
+   העמוד הזה מתאר **איך** לבנות — מלכודות, חוזים בין שכבות, מה לבדוק ובמה. העמוד למשתמש מתאר **מה קורה** — מה רואים, מה ברירת המחדל, איפה יש מתג ואיפה אין. שינוי בהתנהגות שייך לשם, גם כשהוא נראה קטן.
+
+   מה נחשב שינוי בהתנהגות: ברירת מחדל שמתהפכת, מצב מיקום שמתחיל או מפסיק להיות זמין, יעד חדש לפתק, מגבלה או מכסה שמשתנה, ותוספת לתת-קבוצת המארקדאון. שינוי פנימי שאינו נראה מבחוץ — ריפקטור, אינדקס, סדר קריאות — אינו מצריך עדכון.
+
+**בלי כפילות בין שני העמודים.** עובדה שכתובה בשניהם מתיישנת באחד מהם, ואז שניהם אומרים דברים שונים על אותו קוד. הכלל הוא הפרדה לפי שאלה: התנהגות נראית נכתבת בעמוד המשתמש בלבד, והנימוק ההנדסי — למה זה בנוי ככה, מה נשבר בלי זה — נכתב כאן בלבד. כשצריך את שניהם, כותבים כל חצי במקומו ומקשרים ביניהם ב-``:doc:``, כפי שנעשה כבר בשתי ההפניות לעמוד הזה מתוך עמוד המשתמש.
+
+**התיעוד אינו סמכות על הקוד.** אם בזמן העדכון מתגלה שמה שכתוב בעמוד המשתמש כבר אינו נכון — זה ממצא לתקן, לא רקע להתעלם ממנו. פסקת העיגון בפתקי ריפו שרדה כך סבב שלם אחרי שההתנהגות השתנתה תחתיה.
diff --git a/docs/user/sticky_notes.rst b/docs/user/sticky_notes.rst
index f4ecc373a..a33d0ce26 100644
--- a/docs/user/sticky_notes.rst
+++ b/docs/user/sticky_notes.rst
@@ -115,14 +115,14 @@
 
 עיצוב מארקדאון
 ~~~~~~~~~~~~~~~
-מלבד צ'קבוקסים, הפתק מרנדר תת-קבוצה קלילה של מארקדאון: **מודגש** (``**``), *נטוי* (``*``), ``קוד`` בשורה, קו חוצה (``~~``), כותרות (``#`` עד ``###``), רשימות (``-``/``*``/``1.``), ציטוט (``>``), קו מפריד (``---``), קישורים (``[טקסט](כתובת)``) ובלוקי קוד (שורת גדר של שלושה תווי backtick, מעל התוכן ומתחתיו). כשהעיצוב פעיל, התצוגה נפתחת בדיוק כמו עם צ'קבוקס — כשיש בפתק מבנה מארקדאון כלשהו — ופתק של טקסט רגיל נשאר תיבת עריכה ולא משתנה. (בפתקי לוח העיצוב דלוק כברירת מחדל; בפתקי קבצים כבוי — ראו למטה.)
+מלבד צ'קבוקסים, הפתק מרנדר תת-קבוצה קלילה של מארקדאון: **מודגש** (``**``), *נטוי* (``*``), ``קוד`` בשורה, קו חוצה (``~~``), כותרות (``#`` עד ``###``), רשימות (``-``/``*``/``1.``), ציטוט (``>``), קו מפריד (``---``), קישורים (``[טקסט](כתובת)``) ובלוקי קוד (שורת גדר של שלושה תווי backtick, מעל התוכן ומתחתיו). כשהעיצוב פעיל, התצוגה נפתחת בדיוק כמו עם צ'קבוקס — כשיש בפתק מבנה מארקדאון כלשהו — ופתק של טקסט רגיל נשאר תיבת עריכה ולא משתנה. (העיצוב דלוק כברירת מחדל בכל סוגי הפתקים — ראו למטה.)
 
 .. important::
    **טקסט של משתמש — או של סוכן דרך** ``codekeeper_create_note`` **— אינו יכול להפוך לתגית.** זריקת ``<script>`` מוצגת כטקסט, וקישורים נפתחים רק בסכימות ``http``/``https``. איך זה נאכף בקוד, ולמה אין כאן ספריית מארקדאון שפולטת HTML — :doc:`/dev/sticky_notes_extending`.
 
 מה **לא** נכנס, ולמה: טבלאות (רחבות מדי לפתק), תמונות (בקשת רשת ומידות בלתי צפויות), ו-HTML גולמי (שובר את חוזה ה-``textContent``). הקו התחתון ``_`` **אינו** נטוי — ``note_id`` ו-``user_id`` נפוצים מדי בכלי שכולו קוד. קישורים נפתחים בלשונית חדשה עם ``rel="noopener noreferrer"``, ורק סכימות ``http``/``https`` — ``javascript:`` ו-``data:`` מרונדרים כטקסט, לא כקישור.
 
-העיצוב **דלוק כברירת מחדל בלוח**, ואפשר לכבות אותו במודאל ההגדרות (גלגל השיניים); ההעדפה נשמרת מקומית לכל לוח, ורק כיבוי מפורש נשמר. כיבוי מציג את התוכן הגולמי כפי שהוקלד. **בפתקים שעל קבצים** ברירת המחדל שמרנית — כבוי — כי אין להם מתג כיבוי, וצ'קבוקסים ממשיכים לעבוד שם כרגיל.
+העיצוב **דלוק כברירת מחדל בכל סוגי הפתקים** — בלוח, על קובץ ב-CodeKeeper, ועל קובץ בדפדפן הריפו. **מתג הכיבוי קיים בלוח בלבד**, במודאל ההגדרות (גלגל השיניים); ההעדפה נשמרת מקומית לכל לוח, ורק כיבוי מפורש נשמר, וכיבוי מציג את התוכן הגולמי כפי שהוקלד. בפתקים שעל קבצים אין מתג, אבל גם אין מה לכבות בפתק טקסט רגיל: התצוגה נפתחת רק כשיש בפתק מבנה מארקדאון, ופתק בלי מבנה נשאר תיבת עריכה. צ'קבוקסים עובדים בכל המקרים, בלי תלות בהגדרה.
 
 לוחות דרך MCP
 ~~~~~~~~~~~~~~~
@@ -146,7 +146,9 @@
 
 הפתק מזוהה בזוג ``(repo_name, repo_path)``, ולא במזהה יחיד. אין ענף במפתח בכוונה: פתק שנרשם כשהיית על ``main`` מופיע גם כשאתה מסתכל על ענף PR — המלכודת שייכת לקובץ, לא לענף.
 
-**העיגון הוא ברמת קובץ, לא ברמת שורה.** תצוגת הקובץ בדפדפן הריפו היא עורך CodeMirror, שאינו מרנדר שורות שנמצאות מחוץ למסך — אין אלמנט DOM יציב להיצמד אליו. לכן שני מצבי המיקום — ``surface`` (מעוגן למסגרת התצוגה) ו-``screen`` (צמוד לחלון) — מצמידים את הפתק למסגרת הקובץ, והוא **אינו נגלל יחד עם שורות הקוד**. ``anchored`` נדחה כאן מאותה סיבה שהוא נדחה בלוח.
+**הפתק נצמד למקום בתוכן, לא לשורה.** שני מצבי המיקום מתנהגים כאן שונה זה מזה: ``surface`` מודד את מיקומו מראשית **התוכן** של הקובץ, ולכן הפתק נגלל יחד עם הקוד ונשאר ליד מה שהוא מדבר עליו — וכשהוא נגלל אל מחוץ לתחום הנראה הוא נעלם מהמסך עד שגוללים אליו בחזרה. ``screen`` הוא ההפך: הפתק צמוד לחלון ונשאר גלוי בכל גלילה. המעבר בין השניים הוא כפתור 📌 שעל הפתק.
+
+מה שאין כאן הוא הצמדה ל**שורה** מסוימת. תצוגת הקובץ היא עורך CodeMirror, שאינו מרנדר שורות שנמצאות מחוץ למסך — אין אלמנט DOM יציב שאפשר להיקשר אליו. לכן מצב ``anchored``, שקיים בתצוגת מארקדאון, אינו זמין בדפדפן הריפו, מאותה סיבה שהוא אינו זמין בלוח.
 
 .. note::
    דפדפן הריפו פתוח לאדמינים בלבד, ולכן פתקי ריפו הם פיצ'ר אדמין. בפועל: **20 פתקים לקובץ** נאכפים על כולם, אדמין כולל; התקרה הכללית של 1000 פתקים למשתמש אינה נאכפת על אדמין. למה שתי התקרות מתנהגות שונה — :doc:`/dev/sticky_notes_extending`.
diff --git a/tests/sticky-notes-target.test.js b/tests/sticky-notes-target.test.js
index 4f434977c..1a32ef5fa 100644
--- a/tests/sticky-notes-target.test.js
+++ b/tests/sticky-notes-target.test.js
@@ -917,15 +917,26 @@ check('פתיחה: קישור פסול לפני מארקדאון תקין — ע
   eq(!!view.querySelector('.sticky-md-link'), false, 'והקישור הפסול נשאר טקסט');
 });
 
-check('פתק קובץ: מארקדאון כבוי כברירת מחדל', () => {
-  // ``fileMgr`` הוא פתק קובץ — אין לו מתג כיבוי, ולכן ברירת המחדל
-  // שמרנית: טקסט גולמי, בדיוק כמו לפני הפיצ'ר.
-  eq(fileMgr.markdown, false, 'פתק קובץ — כבוי');
+check('פתק קובץ: מארקדאון דלוק כברירת מחדל, כמו בלוח', () => {
+  // ברירת המחדל אחידה לכל היעדים. היא הייתה פעם כבויה בפתקי קובץ מתוך
+  // חשש שרינדור אוטומטי הוא שינוי שקט — אבל הדגל לבדו אינו מרנדר כלום,
+  // ולכן מה שהשתנה הוא רק פתקים שכבר מכילים מבנה מארקדאון.
+  eq(fileMgr.markdown, true, 'פתק קובץ — דלוק');
   eq(mdMgr.markdown, true, 'פתק לוח — דלוק');
   const parts = makeNote('# כותרת\n**מ**');
   fileMgr._syncTaskView(parts.el);
-  eq(!!parts.view.querySelector('.sticky-md-h1'), false, 'אין רינדור בפתק קובץ');
+  eq(!!parts.view.querySelector('.sticky-md-h1'), true, 'מרונדר בפתק קובץ');
+  eq(parts.ta.hidden, true, 'וה-textarea מוסתר');
+});
+
+check('פתק קובץ בלי מבנה מארקדאון נשאר תיבת עריכה', () => {
+  // **זה מה שמחזיק את רדיוס השינוי קטן.** ``_syncTaskView`` דורש גם את
+  // הדגל וגם מבנה בפועל, ולכן הדלקת ברירת המחדל אינה נוגעת בפתקי טקסט.
+  // בלי הבדיקה הזו, הדלקה גורפת הייתה נראית זהה בטסטים.
+  const parts = makeNote('סתם טקסט בלי שום מבנה');
+  fileMgr._syncTaskView(parts.el);
   eq(parts.ta.hidden, false, 'ה-textarea גלוי');
+  eq(parts.view.hidden, true, 'והתצוגה מוסתרת');
 });
 
 check('רינדור: charOffset נשמר לכל שורה', () => {
@@ -1079,11 +1090,21 @@ check('פתק ריפו לעולם אינו anchored', () => {
   eq(m._resolveMode({ mode: 'anchored' }), 'surface');
 });
 
-check('ברירת המחדל של מארקדאון בפתקי ריפו שמרנית', () => {
-  // כמו בפתקי קובץ: אין מתג כיבוי, ולכן רינדור אוטומטי היה שינוי שקט
-  // ובלתי-הפיך-למשתמש.
+check('מארקדאון דלוק כברירת מחדל גם בפתקי ריפו', () => {
+  // שלושת היעדים מתנהגים אותו דבר. מתג הכיבוי קיים בלוח בלבד, וזה
+  // מתועד ב-``docs/user/sticky_notes.rst``.
   const m = new StickyNotesManager({ repo: 'CodeBot', path: 'a.py' });
-  eq(m.markdown, false);
+  eq(m.markdown, true);
+});
+
+check('markdown מפורש ב-opts גובר על ברירת המחדל', () => {
+  // הדלת שדרכה קורא יכול לבטל — ``('markdown' in opts)`` נבדק לפני
+  // ברירת המחדל. בלי הבדיקה הזו, הדלקה גורפת הייתה יכולה להתעלם ממנה
+  // בשקט.
+  const off = new StickyNotesManager({ repo: 'CodeBot', path: 'a.py', markdown: false });
+  eq(off.markdown, false, 'false מפורש מכבה');
+  const on = new StickyNotesManager({ repo: 'CodeBot', path: 'a.py', markdown: true });
+  eq(on.markdown, true, 'true מפורש מדליק');
 });
 
 // -- destroy: flush לפני פירוק --
diff --git a/webapp/static/js/sticky-notes.js b/webapp/static/js/sticky-notes.js
index ce4c98e9d..da3270350 100644
--- a/webapp/static/js/sticky-notes.js
+++ b/webapp/static/js/sticky-notes.js
@@ -295,15 +295,21 @@
       // ולעולם אינו מקצר את הלוח מתחת לתוכן שכבר עליו — כלומר אף פתק לא
       // "מתכווץ פנימה" ואף מיקום לא הולך לאיבוד.
       this.infiniteScroll = ('infiniteScroll' in opts) ? !!opts.infiniteScroll : false;
-      // מארקדאון מרונדר כברירת מחדל (אופציה ג'), עם מתג לכיבוי במודאל
-      // הגדרות הלוח. צ'קבוקסים אינם תלויים בדגל הזה — הם מרונדרים תמיד,
-      // כי הם כתיבה למסד ולא רק תצוגה.
-      // דלוק כברירת מחדל **בלוח** (אופציה ג', עם מתג במודאל). בפתקי
-      // קובץ ברירת המחדל שמרנית — ``false`` — כי אין להם מתג כיבוי,
-      // ורינדור אוטומטי היה שינוי שקט ובלתי-הפיך-למשתמש לכל פתקי הקבצים
-      // הקיימים. md_preview.html יכול להעביר ``markdown: true`` במפורש
-      // אם ירצה בעתיד.
-      this.markdown = ('markdown' in opts) ? !!opts.markdown : !!this.boardId;
+      // **מארקדאון מרונדר כברירת מחדל בכל שלושת היעדים** — לוח, קובץ
+      // ב-CodeKeeper, וקובץ בדפדפן הריפו. מתג הכיבוי קיים בלוח בלבד
+      // (מודאל ההגדרות), ובשני האחרים אין מתג.
+      //
+      // ברירת המחדל כאן הייתה פעם ``!!this.boardId`` — כלומר כבוי בפתקי
+      // קובץ — מתוך חשש שרינדור אוטומטי הוא שינוי שקט שאין למשתמש דרך
+      // לבטל. בפועל הדגל הזה אינו מספיק כדי לרנדר: ``_syncTaskView``
+      // דורש גם ``_hasRenderableMarkdown``, ולכן פתק של טקסט רגיל נשאר
+      // תיבת עריכה ואינו משתנה כלל. מה שהשתנה הוא רק פתקים שכבר מכילים
+      // מבנה מארקדאון — ושם הרינדור הוא מה שהמשתמש התכוון אליו כשכתב
+      // ``**`` או ``#``.
+      //
+      // צ'קבוקסים אינם תלויים בדגל הזה — הם מרונדרים תמיד, כי הם כתיבה
+      // למסד ולא רק תצוגה.
+      this.markdown = ('markdown' in opts) ? !!opts.markdown : true;
       this._renderedFromCache = false;
       this._pendingSeq = new Map(); // noteId -> monotonic version of pending edits
       // מנהל שנוצר ומפורק שוב ושוב (החלפת קובץ בדפדפן הריפו) חייב להסיר
@@ -2979,8 +2985,8 @@
         try {
           const entry = this.notes.get(id);
           if (entry && entry.data) {
-            // **כאן הכוונה מומשה, ורק כאן היא נצרכת.** ההשוואה מגינה מפני
-            // הסרת הכוונה לאחר התאמה מונעת גלילה חוזרת במנהל הבא.
+            // **כאן הכוונה מומשה, ורק כאן היא נצרכת.** הסרתה לאחר התאמה
+            // מונעת גלילה חוזרת במנהל הבא.
             if (pendingNoteIdFromUrl === id) pendingNoteIdFromUrl = '';
             this.scrollToNote(id);
             return;

From 16bf15f3f64a354673d7714d3185118e3d597440 Mon Sep 17 00:00:00 2001
From: "coderabbitai[bot]"
 <136622811+coderabbitai[bot]@users.noreply.github.com>
Date: Tue, 25 Aug 2026 20:56:52 +0000
Subject: [PATCH 2/4] docs(notes): clarify repo note anchoring limitation

---
 docs/dev/sticky_notes_extending.rst | 4 ++++
 docs/user/sticky_notes.rst          | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/docs/dev/sticky_notes_extending.rst b/docs/dev/sticky_notes_extending.rst
index 14e714052..129cf6a4f 100644
--- a/docs/dev/sticky_notes_extending.rst
+++ b/docs/dev/sticky_notes_extending.rst
@@ -72,6 +72,10 @@
 -------------------------------------------------------
 **טקסט של משתמש או של סוכן לעולם אינו נכתב דרך HTML גולמי.** כל צומת שנושא תוכן כזה נבנה עם ``createElement`` ו-``textContent``, ותכונות של קישור נכתבות עם ``setAttribute`` אחרי אימות סכימה. לכן זריקת ``<script>`` מוצגת כטקסט, ואל תכניסו ספריית מארקדאון שפולטת מחרוזת HTML. תבנית סטטית לגמרי, בלי שום קלט משתמש (למשל שלד מודאל התזכורת), מותרת דרך ``innerHTML`` — הגבול הוא הקלט, לא ה-API. הכלל המוחלט הוא על **טקסט לא-מהימן**, לא על השיטה.
 
+עיגון שורה בפתקי ריפו: למה ``anchored`` אינו זמין
+--------------------------------------------------
+``anchored`` מחייב עוגן שורה יציב. דפדפן הריפו משתמש ב-CodeMirror, שמרנדר רק את השורות שבתחום הנראה; לכן אין אלמנט DOM יציב שאפשר להיקשר אליו עבור שורה מחוץ למסך. פתקי ריפו תומכים רק ב-``surface`` וב-``screen``. תיאור ההתנהגות למשתמש נמצא ב-:doc:`/user/sticky_notes`.
+
 סדר: ``flush`` לפני כל פעולה הרסנית
 -----------------------------------
 תור השמירה עובד עם ``debounce``, ולכן עריכה שנעשתה זה עתה עדיין ממתינה בו. כל פעולה שעלולה למחוק את התור — פירוק המנהל בהחלפת יעד, סימון צ'קבוקס שמפעיל כתיבה משלו — חייבת **לרוקן את התור תחילה**. אחרת העריכה האחרונה נעלמת בלי שום סימן. זו תקלת סדר שכבר נתפסה כאן פעמיים (``_flushFor`` לפני ה-toggle; ``destroy`` שמרוקן לפני שהוא מפרק).
diff --git a/docs/user/sticky_notes.rst b/docs/user/sticky_notes.rst
index a33d0ce26..79662b027 100644
--- a/docs/user/sticky_notes.rst
+++ b/docs/user/sticky_notes.rst
@@ -148,7 +148,7 @@
 
 **הפתק נצמד למקום בתוכן, לא לשורה.** שני מצבי המיקום מתנהגים כאן שונה זה מזה: ``surface`` מודד את מיקומו מראשית **התוכן** של הקובץ, ולכן הפתק נגלל יחד עם הקוד ונשאר ליד מה שהוא מדבר עליו — וכשהוא נגלל אל מחוץ לתחום הנראה הוא נעלם מהמסך עד שגוללים אליו בחזרה. ``screen`` הוא ההפך: הפתק צמוד לחלון ונשאר גלוי בכל גלילה. המעבר בין השניים הוא כפתור 📌 שעל הפתק.
 
-מה שאין כאן הוא הצמדה ל**שורה** מסוימת. תצוגת הקובץ היא עורך CodeMirror, שאינו מרנדר שורות שנמצאות מחוץ למסך — אין אלמנט DOM יציב שאפשר להיקשר אליו. לכן מצב ``anchored``, שקיים בתצוגת מארקדאון, אינו זמין בדפדפן הריפו, מאותה סיבה שהוא אינו זמין בלוח.
+מצב ``anchored`` אינו זמין בפתקי ריפו.
 
 .. note::
    דפדפן הריפו פתוח לאדמינים בלבד, ולכן פתקי ריפו הם פיצ'ר אדמין. בפועל: **20 פתקים לקובץ** נאכפים על כולם, אדמין כולל; התקרה הכללית של 1000 פתקים למשתמש אינה נאכפת על אדמין. למה שתי התקרות מתנהגות שונה — :doc:`/dev/sticky_notes_extending`.

From 9d060db4394d2ea7f509cadd9f0bb923d2cf46eb Mon Sep 17 00:00:00 2001
From: amir haim <amirbiron@gmail.com>
Date: Wed, 26 Aug 2026 00:11:25 +0300
Subject: [PATCH 3/4] Update docs/user/sticky_notes.rst

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
---
 docs/user/sticky_notes.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/user/sticky_notes.rst b/docs/user/sticky_notes.rst
index 79662b027..7b7be7e55 100644
--- a/docs/user/sticky_notes.rst
+++ b/docs/user/sticky_notes.rst
@@ -148,7 +148,7 @@
 
 **הפתק נצמד למקום בתוכן, לא לשורה.** שני מצבי המיקום מתנהגים כאן שונה זה מזה: ``surface`` מודד את מיקומו מראשית **התוכן** של הקובץ, ולכן הפתק נגלל יחד עם הקוד ונשאר ליד מה שהוא מדבר עליו — וכשהוא נגלל אל מחוץ לתחום הנראה הוא נעלם מהמסך עד שגוללים אליו בחזרה. ``screen`` הוא ההפך: הפתק צמוד לחלון ונשאר גלוי בכל גלילה. המעבר בין השניים הוא כפתור 📌 שעל הפתק.
 
-מצב ``anchored`` אינו זמין בפתקי ריפו.
+מצב ``anchored`` אינו זמין בפתקי ריפו: עורך CodeMirror אינו מרנדר שורות שמחוץ למסך, ולכן אין אלמנט DOM יציב שאפשר להיקשר אליו.
 
 .. note::
    דפדפן הריפו פתוח לאדמינים בלבד, ולכן פתקי ריפו הם פיצ'ר אדמין. בפועל: **20 פתקים לקובץ** נאכפים על כולם, אדמין כולל; התקרה הכללית של 1000 פתקים למשתמש אינה נאכפת על אדמין. למה שתי התקרות מתנהגות שונה — :doc:`/dev/sticky_notes_extending`.

From 5d427f16b0d64ccd199055dfa18e934531c037fb Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 25 Aug 2026 21:14:54 +0000
Subject: [PATCH 4/4] =?UTF-8?q?docs:=20=D7=A1=D7=A4=D7=99=D7=A8=D7=AA=20?=
 =?UTF-8?q?=D7=9E=D7=95=D7=A4=D7=A2=D7=99=D7=9D=20=D7=90=D7=99=D7=A0=D7=94?=
 =?UTF-8?q?=20=D7=A0=D7=9B=D7=AA=D7=91=D7=AA=20=D7=91=D7=AA=D7=99=D7=A2?=
 =?UTF-8?q?=D7=95=D7=93,=20=D7=95=D7=94=D7=94=D7=91=D7=97=D7=A0=D7=94=20?=
 =?UTF-8?q?=D7=A0=D7=9B=D7=A0=D7=A1=D7=AA=20=D7=9C-doc-authoring?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

‏cubic מצא שהמשפט "כפי שנעשה כבר בשתי ההפניות לעמוד הזה" אינו נכון: יש שש
הפניות ``:doc:`` ל-``/dev/sticky_notes_extending`` בתוך עמוד המשתמש, בשורות
84, 121, 137, 154, 158 ו-164. הממצא ישב בתוך סעיף שכל נושאו דיוק בתיעוד.

**המניין הוסר ולא עודכן ל"שש".** עדכון המספר מאפס מונה שייסחף שוב בהוספה
הבאה, כלומר מתקן תסמין. הניסוח החדש נכון בלי תלות בכמות ולכן אינו יכול
להתיישן.

‏``docs/doc-authoring.rst`` קיבל סעיף חדש שמכליל את מה שכבר היה שם. הכלל
הקיים אסר ספירה **בתקציר** בלבד, ו-``tests/test_doc_summary_style.py``
אוכף בדיוק את זה — ולכן ספירה בפרוזה רגילה לא נתפסה על ידי דבר. הסעיף
מבחין בין שני סוגי מספרים:

- **ספירת מופעים** ("שתי ההפניות", "28 אייקונים") — אף אחד לא שומר עליה,
  כל הוספה שוברת אותה, ואין להשתמש בה.
- **ערך שנאכף בקוד** ("20 פתקים לקובץ") — חי בקוד, נאכף בו, ולכן אינו
  נסחף; הקורא חייב אותו והוא כן מתועד.

המבחן שנוסח: אם המספר יכול להשתנות בלי שאף שורת קוד תשתנה — הוא ספירת
מופעים.

התקציר של ``doc-authoring`` עודכן בהתאם (154 תווים, מתחת לתקרת ה-220 של
שורת המפה) ו-``AI-MAP.md`` חודשה. אין צורך בשורת טריגר חדשה ב-CLAUDE.md:
הקיימת כבר מפנה ל-``doc-authoring`` בכל עריכת עמוד תחת ``docs/``.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019YULCppaQRPYN1RgeY6NBu
---
 AI-MAP.md                           |  2 +-
 docs/dev/sticky_notes_extending.rst |  2 +-
 docs/doc-authoring.rst              | 13 ++++++++++++-
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/AI-MAP.md b/AI-MAP.md
index 7b9adc3d7..6a56d56fe 100644
--- a/AI-MAP.md
+++ b/AI-MAP.md
@@ -17,7 +17,7 @@
 - `docs/quickstart-contrib.rst` — **Quickstart לתרומה**: דף קצר שמאפשר להתחיל לתרום במהירות ובבטחה.
 - `docs/ai-guidelines.rst` — **הנחיות מלאות לסוכני AI**: ההנחיות המלאות לסוכני AI שעובדים בריפו: המגבלות הקריטיות, איך מריצים פקודות, אילו כלי קבצים מאושרים, עקרונות עריכת קוד, ומדיניות הקומיטים וה-Pull Requests.
 - `docs/agents/rate-limiting.md` — **🚦 מערכת Rate Limiting לסוכני AI ולווב**: מטרה: להסביר איך מפעילים ומנטרים Rate Limiting בבוט ובווב, עם דגש על Shadow Mode, ניטור וקונפיג.
-- `docs/doc-authoring.rst` — **Doc Authoring Guide (Sphinx/RTD)**: כללי כתיבת תיעוד בפרויקט — הצהרת תקציר בראש כל עמוד, הטמעת קוד לפי שם או סימון ולא לפי מספרי שורות, ובנייה ללא אזהרות.
+- `docs/doc-authoring.rst` — **Doc Authoring Guide (Sphinx/RTD)**: כללי כתיבת תיעוד בפרויקט — הצהרת תקציר בראש כל עמוד, הטמעת קוד לפי שם או סימון ולא לפי מספרי שורות, ההבחנה בין ספירת מופעים לערך שנאכף, ובנייה ללא אזהרות.
 - `docs/style-glossary.rst` — **Style & Naming Glossary**: מילון המונחים והשמות בפרויקט: מיפוי בין מונחים מקבילים, כללי הניסוח, ועוגני התיעוד שמפנים אליהם.
 - `docs/versioning-stable-anchors.rst` — **Versioning & Stable Anchors**: מדיניות הגרסאות והעוגנים היציבים בתיעוד: אילו עוגנים מובטחים לא להישבר, איך מתעדים שינוי ב-What's New, ודוגמאות.
 - `docs/whats-new.rst` — **What's New**: יומן השינויים של הבוט וה-WebApp לפי תאריך — מה נוסף, מה השתנה ומה תוקן בכל עדכון, עם קישורים ל-Issues הרלוונטיים.
diff --git a/docs/dev/sticky_notes_extending.rst b/docs/dev/sticky_notes_extending.rst
index 129cf6a4f..3a1575f71 100644
--- a/docs/dev/sticky_notes_extending.rst
+++ b/docs/dev/sticky_notes_extending.rst
@@ -100,6 +100,6 @@
 
    מה נחשב שינוי בהתנהגות: ברירת מחדל שמתהפכת, מצב מיקום שמתחיל או מפסיק להיות זמין, יעד חדש לפתק, מגבלה או מכסה שמשתנה, ותוספת לתת-קבוצת המארקדאון. שינוי פנימי שאינו נראה מבחוץ — ריפקטור, אינדקס, סדר קריאות — אינו מצריך עדכון.
 
-**בלי כפילות בין שני העמודים.** עובדה שכתובה בשניהם מתיישנת באחד מהם, ואז שניהם אומרים דברים שונים על אותו קוד. הכלל הוא הפרדה לפי שאלה: התנהגות נראית נכתבת בעמוד המשתמש בלבד, והנימוק ההנדסי — למה זה בנוי ככה, מה נשבר בלי זה — נכתב כאן בלבד. כשצריך את שניהם, כותבים כל חצי במקומו ומקשרים ביניהם ב-``:doc:``, כפי שנעשה כבר בשתי ההפניות לעמוד הזה מתוך עמוד המשתמש.
+**בלי כפילות בין שני העמודים.** עובדה שכתובה בשניהם מתיישנת באחד מהם, ואז שניהם אומרים דברים שונים על אותו קוד. הכלל הוא הפרדה לפי שאלה: התנהגות נראית נכתבת בעמוד המשתמש בלבד, והנימוק ההנדסי — למה זה בנוי ככה, מה נשבר בלי זה — נכתב כאן בלבד. כשצריך את שניהם, כותבים כל חצי במקומו ומקשרים ביניהם ב-``:doc:``, כפי שעמוד המשתמש כבר עושה בכל מקום שבו הוא נוגע בנימוק הנדסי.
 
 **התיעוד אינו סמכות על הקוד.** אם בזמן העדכון מתגלה שמה שכתוב בעמוד המשתמש כבר אינו נכון — זה ממצא לתקן, לא רקע להתעלם ממנו. פסקת העיגון בפתקי ריפו שרדה כך סבב שלם אחרי שההתנהגות השתנתה תחתיה.
diff --git a/docs/doc-authoring.rst b/docs/doc-authoring.rst
index 729d1f7aa..6da481b30 100644
--- a/docs/doc-authoring.rst
+++ b/docs/doc-authoring.rst
@@ -1,6 +1,6 @@
 Doc Authoring Guide (Sphinx/RTD)
 ================================
-:summary: כללי כתיבת תיעוד בפרויקט — הצהרת תקציר בראש כל עמוד, הטמעת קוד לפי שם או סימון ולא לפי מספרי שורות, ובנייה ללא אזהרות.
+:summary: כללי כתיבת תיעוד בפרויקט — הצהרת תקציר בראש כל עמוד, הטמעת קוד לפי שם או סימון ולא לפי מספרי שורות, ההבחנה בין ספירת מופעים לערך שנאכף, ובנייה ללא אזהרות.
 
 מטרות
 ------
@@ -25,6 +25,17 @@ Doc Authoring Guide (Sphinx/RTD)
 - **אל תשאירו כותרת יתומה.** אם ההצהרה מחליפה פסקת פתיחה והסעיף שהכיל אותה נשאר ריק — מחקו גם את הכותרת. סעיף ריק מרונדר בעמוד כשורה ללא גוף.
 - **למה הצהרה ולא חילוץ אוטומטי:** הגרסה הקודמת חילצה את פסקת הפרוזה הראשונה, וכדי לעשות זאת מימשה חלקים מ-GFM ומ-RST ביד. ארבעה סבבי ריוויו רצופים מצאו שם מקרי קצה — כולל תקציר שיצא טבלה שלמה ותקציר שיצא ריק. מי שכתב את העמוד יודע מה התקציר.
 
+ספירות בפרוזה: ספירת מופעים לעומת ערך שנאכף
+--------------------------------------------
+הכלל שלמעלה אוסר ספירה **בתקציר**, אבל הסיבה אינה מוגבלת לשם: כל מספר שנכתב בפרוזה מתיישן באותה שקט. ההבחנה שמכריעה אינה איפה המספר יושב אלא **מי אחראי עליו**.
+
+- **ספירת מופעים — אל תכתבו.** "בשתי ההפניות לעמוד הזה", "28 אייקונים", "שבעה מסלולי API". אף אחד לא שומר על המספר הזה: כל הוספה שוברת אותו, ושום בדיקה לא מתריעה. נסחו בלי כמות — "בכל מקום שבו העמוד נוגע בנימוק הנדסי" נכון גם אחרי התוספת הבאה.
+- **ערך שנאכף — כן תעדו.** "20 פתקים לקובץ", "תקרה של 1000 פתקים למשתמש", "עד 500KB לקובץ". המספר הזה חי בקוד ונאכף בו, ולכן הוא אינו נסחף מעצמו — ומי שקורא את התיעוד חייב אותו. שינוי שלו הוא שינוי מוצר שממילא מחייב עדכון תיעוד.
+
+המבחן: **אם המספר יכול להשתנות בלי שאף שורת קוד תשתנה — הוא ספירת מופעים, ואין לכתוב אותו.**
+
+``tests/test_doc_summary_style.py`` אוכף את הכלל על תקצירים בלבד. בפרוזה אין אכיפה אוטומטית, וזה הפער שכבר נתפס כאן: הכלל הזה נוסח אחרי שריוויוור מצא ספירה שגויה בפרוזה של עמוד — בתוך סעיף שכל נושאו דיוק בתיעוד.
+
 הטמעת קוד מהמקור (``literalinclude``)
 --------------------------------------
 - **אסור** למען לפי מספרי שורות (``:lines:``) — הקוד זז והתיעוד ממשיך להציג את הטווח הישן בלי שום אזהרה. זה דפוס ``line-number-coupling`` (ראו amir-bug-patterns), והוא כבר קרה כאן: בלוק שהצביע על ``main.py:739`` הציג פנימיות של פונקציה אחרת, במרחק 2,900 שורות מהיעד.