docs(mcp): התקציב הוא הבטחה של השרת — לקוח אחר עשוי לחתוך לפני כן

Merged
AM
22d ago
+2-0

תבנית Pull Request

✨ תיאור קצר

  • פסקה אחת ב-docs/mcp-server.rst, מיד אחרי תיאור התקציב ("למה בתים, ומה נמדד") בסעיף mcp-answer-size.
  • היא אומרת שלושה דברים:
    • OUTPUT_BYTE_BUDGET הוא הבטחה של השרת. לכל לקוח יש תקרה משלו, והיא עשויה להיות נמוכה ממנו.
    • anthropic/maxResultSizeChars מתועד כמנגנון של Claude Code. לקוח אחר אינו מחויב לו, ולכן אצלו תשובה שנכנסת בתקציב עדיין עלולה להיחתך או להישמר לקובץ.
    • בקבצים גדולים, ובמיוחד בעברית, הדרך היציבה היא מפה ואז סעיף, או lines, ולא קריאה מלאה.
  • זה המשך של amirbiron/CodeBot#3492. הבדיקה אחרי הדיפלוי, מסשן Claude Code חדש, הראתה שההצהרה עובדת: הקובץ העברי מ-#3460 הגיע להקשר כטקסט. ההבהרה נכתבה ללקוחות שאינם Claude Code.

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

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

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

  • מקום אחד בלבד. תיאורי הכלים לא השתנו: הם קרובים לתקרת האורך, והמידע הזה לא משנה איך קוראים לכלי.
  • בלי מספר של לקוח אחר. תיעוד ה-connectors (claude.com/docs/connectors/building) נוקב בתקרה של claude.ai. זה ערך שנקבע מחוץ לריפו, ואיש לא יעדכן אותו כשהוא ישתנה (prose-restates-code-fact). הכלל עומד גם בלעדיו.
  • "לקוח אחר אינו מחויב לה" ולא "לקוחות אחרים לא קוראים אותה". לשאלה אם claude.ai מכבד את ההצהרה לא נמצא מקור. מה שכן מתועד הוא שזה מנגנון של Claude Code (code.claude.com/docs/en/mcp.md, "Raise the limit for a specific tool", נקרא ב-2026-09-29).
  • התקציב מוזכר בשם הקבוע, OUTPUT_BYTE_BUDGET, ולא במספר.

🧪 בדיקות

  • Unit
  • Integration
  • Manual: test_the_numbers_in_the_answer_size_section_are_the_code_numbers (הטסט שמצמיד את המספרים בסעיף הזה לקבועים) עובר. זו פרוזה בעמוד קיים, ולכן בלי בנייה מקומית של התיעוד: ה-check של Read the Docs על ה-PR יתפוס אזהרות.

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

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

📝 סוג שינוי

  • feat: פיצ'ר חדש
  • fix: תיקון באג
  • docs: שינוי תיעוד בלבד
  • refactor: שינוי קוד ללא שינוי התנהגות
  • perf: שיפור ביצועים
  • chore/ci: תשתית/CI
  • breaking change: שינוי שובר תאימות

✅ צ'קליסט

  • הקוד עוקב אחרי הסגנון — אין קוד
  • בדיקות רצות ועוברות
  • תיעוד עודכן (README/Docs)
  • אם נוספו ג'ובים חדשים — לא רלוונטי
  • אם נוספו/שונו משתני סביבה — לא רלוונטי
  • אם נוספו/השתנו טוקנים — לא רלוונטי
  • אין סודות/מפתחות בקוד
  • אין מחיקות מסוכנות/פעולות על root
  • הודעת הקומיט תואמת Conventional Commits
  • CHANGELOG עודכן — לא: הבהרה בלי שינוי התנהגות
  • כל ה‑Required Checks לעיל ירוקים — ממתין ל-CI
  • צילום/וידאו UI — לא רלוונטי
  • עיינתי במסמכי אתר התיעוד:
    • docs/doc-authoring.rst (ספירות בפרוזה, ומתי בונים מקומית)
    • docs/mcp-server.rst (הסעיף mcp-answer-size)
    • amir-bug-patterns: bugbot-rules/line-number-coupling.md, bugbot-rules/prose-restates-code-fact.md

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

  • אין. פסקה אחת בעמוד קיים, בלי עוגן חדש ובלי שינוי ב-toctree.

🔗 קישורים

  • Issues קשורים: #3460
  • PR קודם: amirbiron/CodeBot#3492
  • Docs Preview: ה-check של Read the Docs על ה-PR
  • מסמכים/מפרטים רלוונטיים: docs/mcp-server.rst → mcp-answer-size

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

  • revert של הקומיט.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FqzfvA6KiivEag5KC2mCub


Generated by Claude Code