fix: תקרת הסימבולים נוסעת לתוך פרסור ה-RST במקום לסנן את הפלט

[amirbiron]

✨ תיאור קצר

התקרה על מספר הסימבולים יושבת בסורקים ולא במנתב בכוונה — סינון של פלט אינו חסימה של עבודה — אבל בסורק ה-RST היא נגעה רק ברשימה שנבנתה אחרי ש-parse_document סיים. לכן קובץ שנכנס בנוחות לתקרת ה-10MB של קריאת הטווח יכול לקנות עבודה גדולה: 10MB של כותרת בת תו אחד בכל שתי שורות הם 2.6 מיליון סקשנים, 18.5 שניות ו-1,132MB. אחרי השינוי: 0.30 שניות ו-83MB. הזיכרון הוא מה שמצדיק את ה-PR — בקשה איטית היא בקשה איטית, אבל ג'יגה-בייט בתהליך משותף מפיל שירות.

זה חיווט של התקרה שכבר קיימת, ולא כפתור חדש: אפס קבועים חדשים, אפס אוצר מילים חדש בתשובה, ואותו גבול בדיוק.

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

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

פירוט:

• services/rst_parser.py — parse_document מקבל max_sections (keyword-only) שברירת המחדל שלו ללא תקרה, ומרים TooManySections בזמן הפרסור. שלושת אתרי ההוספה — כותרת עם קו תחתון, כותרת עם קו מעליה, וקו עליון קצר שהודח — עוברים דרך הגדרה אחת (add_section) ולא שלוש בדיקות זהות. זה הלקח מכלל בליעת הפסקה שנכתב במודול הזה בשני אתרים, ואתר אחד נשאר מאחור עד שהסבב הבא תפס אותו.
• mcp_server/outline_scanners/rst.py — מעביר max_sections=_ceiling.MAX_SYMBOLS, והפרסור עבר לתוך ה-try. ה-except תופס את שתי החריגות בשמן ולא רחב מזה: TooManySections כשהכותרות לבדן ממלאות את התקרה, ו-TooManySymbols מהמכל כשהכותרות והתוויות יחד חוצות אותה.
• mcp_server/docs_handlers.py — לא נגעתי. זה הצרכן השני של הפארסר (docs_get_section), הוא אינו מעביר תקרה, ולכן הוא זהה בית-בית. יש לזה טסט שעובר דרכו.
• docs/mcp-server.rst — סעיף mcp-limits חדש עם כל תקרה שנאכפת בקוד, כולל תקרות העימוד שלא היו מתועדות כלל; הכלל שערך מעל התקרה נצמד בשקט ואינו נדחה (per_page=1000 באאוטליין מקבל 500, ואין שדה שמצהיר על זה); הרשימה הסגורה של ערכי status בתשובת כלי מול error כעיקרון — ותשובת HTTP של השרת אינה תשובת כלי, כי 401 נושא error בלי ok; המחלקה היחידה שסורק ה-RST אינו ממודל (בלוק literal מצוטט); שורה בפתרון תקלות על כך שתהליך ה-MCP אינו מגדיר לוגים — ולכן היעדר לוג אינו סימן שהכלי לא נקרא (תהליך ה-MCP אינו מתקין קונפיגורציית לוגים — כל רשומה בו אינה נראית #3375); ומה ש-file.encoding באמת אומר — utf-8/utf-8-sig הם זיהוי, latin-1 הוא נפילה-לאחור שאינה נכשלת לעולם (נמדד: קובץ עברי ב-cp1255 חוזר משובש), ו-BOM מוסר מ-content בזמן ש-size סופר אותו.

🧪 בדיקות

• Unit
• Integration
• Manual

מה	תוצאה
הטסטים החדשים על הקוד שלפני השינוי	חמישה נופלים שם; טסט הצרכן שומר על רגרסיה עתידית, ולכן נבדק בשלוש מוטציות
התקרה דרך ממשק ה-MCP	10MB פתולוגי ← no_outline · too_many_symbols · max, ב-0.31 שניות ו-85MB. בדיוק 50,000 כותרות ← ok עם 50,000 סימבולים. 50,001 ← סירוב
הגבול משני הכיוונים, כולל השער השני	49,999 כותרות + תווית אחת ← ok עם 50,000. 49,999 + שתיים ← סירוב (המכל הוא שעוצר)
עבודה ולא רק תשובה	מונה בנייה של Section: על 200 כותרות ותקרה של 20 נבנים 21 (20 שנכנסו, ואחד שנבנה ונדחה) — מימוש שמסנן פלט היה בונה 200. האסרשן הוא חסם ולא מספר מדויק, כדי שריפקטור לגיטימי לא ייפול עם הודעה שמאשימה בהיפוך
שלוש מוטציות על בדיקת הצרכן	ברירת מחדל שהופכת לתקרה · הכלי מעביר ליטרל · הכלי מעביר את קבוע הסורק — שלושתן נתפסות, והבדיקה O(1) ואינה תלויה בגודל התקרה
אפס דיף בפלט על כל קובצי ה-RST של התיעוד	208 קבצים, אפס דיף מול הפארסר שלפני השינוי
הסכמה מלאה עם docutils 0.23 על הקורפוס	אפס קבצים חולקים על רמות או על ספירה (1,340 = 1,340). ההבדל היחיד הוא טקסט כותרת גולמי מול מרונדר, וזו התנהגות מוצהרת בתיעוד
שתי טבלאות האמת מול docutils	3,840 צורות ← 100% · 6,600 צורות ← 100%
שאר השפות	1,734 קבצים (פייתון, HTML, CSS) — שני קבצים בלבד שינו פלט, ושניהם הקבצים שנערכו
הסוויטה	415 עוברים
flake8 --select=E9,F63,F7,F82 · py_compile · mypy · תווי BiDi	0 · נקי · נקי · 0
התפלגות ruff, מול main	17 UP006 · 4 UP045 · 3 E501 · 2 UP035 — זהה, אפס ממצאים חדשים
העמוד שנגעתי בו מול docutils	אפס אזהרות ואפס שגיאות; רק INFO על עוגנים שאינם מופנים, בדיוק כמו העוגנים שהיו בעמוד לפני

ומה שלא נחסם, במפורש — כתוב בקוד, בתיעוד וכאן: התקרה מגבילה סקשנים ולא שורות. 10MB של שורות ריקות הוא אפס סקשנים ו-6.3 שניות, ואותו זמן בדיוק עם התקרה ובלעדיה, כי אין שם מה לחסום. השארית נקבעת על ידי מספר השורות, והחסם היחיד עליה היום הוא תקרת הבתים — והיא מתועדת באישו #3379, עם המדידה, עם ההגדרים שקיימים היום (require_admin על מסלול האאוטליין, ו-500KB על docs_get_section הציבורי) ועם שני הכיוונים. נמדד גם ששתי מיקרו-אופטימיזציות שנשקלו אינן נוגעות בשארית הזאת כלל, ולכן הן לא נכנסו ל-PR.

ומה שהופך את השארית לנזק משותף ולא רק לבקשה איטית: פונקציית הכלי סינכרונית, ואומת מהמקור של ה-SDK המותקן (mcp 1.28.1, server/fastmcp/utilities/func_metadata.py) שהיא נקראת ישר, בלי to_thread. זה חל על כל הכלים ולא על RST בלבד, ולכן הוא PR נפרד — אחרי מיפוי של כלים שמסתמכים על ריצה בתהליך הראשי. גם הוא באישו #3379.

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

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

📝 סוג שינוי

• fix: תיקון באג

✅ צ'קליסט

• הקוד עוקב אחרי הסגנון (flake8 0, התפלגות ruff זהה ל-main, mypy נקי)
• בדיקות רצות ועוברות (415)
• תיעוד עודכן (docs/mcp-server.rst, docs/whats-new.rst)
• אין סודות/מפתחות בקוד
• אין מחיקות מסוכנות/פעולות על root — כל כתיבת הטסטים ל-tmp_path בלבד
• הודעת הקומיט תואמת Conventional Commits
• עיינתי במסמכי אתר התיעוד — AI-MAP.md כנקודת כניסה, ואז docs/doc-authoring.rst ו-docs/versioning-stable-anchors.rst לפני הכתיבה, ושני העמודים שנגעתי בהם
• עיינתי ב-amir-bug-patterns — CRITICAL-PATTERNS.md K11 במלואו, bugbot-rules/silent-fallback-to-worse-path.md, claude-md-snippets/testing.md, bugbot-rules/line-number-coupling.md, ו-bugbot-rules/external-input-isinstance.md

שתי הערות על הקריאה הזאת. K11 הוא מה שקבע שערוץ הכשל יהיה זריקה ולא ערך החזרה: "עצור" כערך דורש בדיקה בכל אתר קריאה, וכשל בדיקה בודד פירושו שהסריקה ממשיכה כאילו כלום. ו-line-number-coupling הוא הסיבה שטבלת הקבועים בתיעוד מפנה לשם המודול ולשם הקבוע ולא למספר שורה. external-input-isinstance נבדק והוכרע כלא-רלוונטי: max_sections הוא קבוע פנימי ולא ערך שהגיע מחוץ לתהליך.

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

services/rst_parser.py הוא מודול משותף עם צרכן בייצור, ולכן הסיכון מרוכז בשאלה אחת: האם הצרכן ההוא השתנה. הוא לא — max_sections הוא keyword-only שברירת המחדל שלו None, ו-docs_get_section אינו מעביר אותו. יש לזה טסט שבודק את התקרה האפקטיבית — שהכלי אינו מעביר תקרה, ושברירת המחדל היא None — ולא טסט שמציף את התקרה, כי מחירו של הטסט ההוא היה צמוד לקבוע שהוא מגן עליו.

החוזה כלפי חוץ לא זז: התשובה על קובץ שחוצה את התקרה נשארה no_outline עם too_many_symbols ו-max, בלי symbols ובלי total. מה שהשתנה הוא מתי מפסיקים לעבוד.

🔗 קישורים

• Issues קשורים: אאוטליין RST: התקרה חוסמת סקשנים ולא שורות, וקלט בלי סקשנים נועל את לולאת האירועים #3379 (השארית שהתקרה אינה חוסמת — סקשנים ולא שורות) · תהליך ה-MCP אינו מתקין קונפיגורציית לוגים — כל רשומה בו אינה נראית #3375 (חיבור הלוגים בתהליך ה-MCP — מוזכר בשורת פתרון התקלות שנוספה)
• Docs Preview:

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

git revert של הקומיטים. אין מיגרציה, אין מצב מתמשך, ואין תלות חדשה.

---

🤖 Generated with Claude Code

https://claude.ai/code/session_01ANHj2BCPu1bCSpsMQvZNQo

Summary by Sourcery

Move the existing symbol ceiling into RST parsing to prevent excessive work and memory consumption on large pathological documents while keeping externally visible behavior unchanged.

Bug Fixes:

• Enforce the existing symbol ceiling during RST parsing so pathological documents stop generating sections before the full parse completes, reducing excessive memory use while preserving the existing response contract.

Enhancements:

• Centralize RST section creation behind a single ceiling-aware path and preserve uncapped parsing for the document-section consumer.
• Document MCP limits, ceiling behavior, encoding semantics, tool response conventions, and related operational constraints.

Documentation:

• Add comprehensive documentation for enforced MCP limits and update the release notes.

Tests:

• Add coverage for all RST heading forms, exact ceiling boundaries, early parsing termination, consumer behavior without a ceiling, and protection against bypassing the centralized section limit.

[qodo-code-review]

ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing

[sourcery-ai]

Reviewer's Guide

The PR wires the existing outline symbol ceiling into RST parsing, stopping section construction at the limit while preserving the public error response and uncapped behavior for the other parser consumer. It consolidates the parser’s section insertion logic, adds boundary and performance-oriented regression tests, and updates MCP documentation to describe the limits and their remaining scope.

Sequence diagram for early RST section limit enforcement

sequenceDiagram
    participant Caller as RSTOutlineScanner
    participant Parser as parse_document
    participant Gate as add_section
    participant Ceiling as SymbolCeiling

    Caller->>Parser: parse_document(text, max_sections=MAX_SYMBOLS)
    Parser->>Gate: add_section(Section)
    alt section limit reached
        Gate-->>Parser: raise TooManySections
        Parser-->>Caller: TooManySections
        Caller->>Ceiling: too_many_symbols()
        Ceiling-->>Caller: no_outline / too_many_symbols / max
    else limit not reached
        Gate->>Gate: sections.append(section)
        Parser-->>Caller: Document
        Caller->>Ceiling: Capped.append(symbols)
        alt combined symbols exceed limit
            Ceiling-->>Caller: TooManySymbols
            Caller->>Ceiling: too_many_symbols()
        else within limit
            Ceiling-->>Caller: symbols
        end
    end

Loading

File-Level Changes

Change	Details	Files
Move the RST section ceiling into document parsing so oversized inputs stop generating sections instead of being filtered after parsing.	Add an optional keyword-only section limit and a dedicated TooManySections exception while preserving uncapped parser behavior by default. Centralize all three heading-to-section insertion paths through one guarded helper with the existing inclusive boundary semantics. Pass the outline symbol ceiling into the parser and translate parser/container limit exceptions into the existing no_outline response. Keep the docs section reader uncapped and add regression coverage for both parser behavior and work avoided.	services/rst_parser.py mcp_server/outline_scanners/rst.py tests/test_rst_parser.py tests/test_mcp_outline.py tests/test_mcp_docs_handlers.py
Document the enforced MCP limits, truncation semantics, RST coverage, and remaining synchronous-work limitations.	Add the new mcp-limits documentation covering symbol, pagination, status/error, unsupported literal-block behavior, and logging troubleshooting. Update RST scanner performance notes and the what's-new entry to describe early parser termination and its scope. Clarify that section limits do not bound line-processing work or address synchronous tool execution.	docs/mcp-server.rst docs/whats-new.rst

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[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): 129

Flagged findings (file:line:snippet):
(none)

Excluded matches (by path pattern)

./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]
./docs/DOCUMENTATION_GUIDE.md:453:rm -rf _build
./docs/Makefile:24:	rm -rf $(BUILDDIR)
./Dockerfile:42:    rm -rf /var/lib/apt/lists/*
./Dockerfile:121:    rm -rf /var/lib/apt/lists/*
./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
./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/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/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.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/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.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]
./README.md:842:find . -name "__pycache__" -exec rm -rf {} +

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: f61f14be-a547-4728-a35b-ca00b4cf3d95

📥 Commits

Reviewing files that changed from the base of the PR and between c72ea05 and 65fedde.

📒 Files selected for processing (3)

• docs/mcp-server.rst
• docs/whats-new.rst
• tests/test_rst_parser.py

🚧 Files skipped from review as they are similar to previous changes (2)

• docs/mcp-server.rst
• docs/whats-new.rst

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

Changes

נוספה תקרת סקשנים אופציונלית ל־parse_document. הפרסר עוצר בזמן יצירת סקשנים. סורק ה־RST ממיר את החריגה לתשובת too_many_symbols. נוספו בדיקות ותיעוד למגבלות MCP. תיעוד whats-new כולל גם תיקונים נוספים.

תקרת סקשנים ותיעוד MCP

שכבה / קבצים	סיכום
חוזה ואכיפת תקרת הפרסר services/rst_parser.py, tests/test_rst_parser.py	נוספו TooManySections ו־max_sections. כל מסלולי יצירת הסקשנים משתמשים בבדיקת התקרה. הבדיקות מאמתות את הגבול ואת היעדר מסלולי הוספה עוקפים.
חיבור הסורק ובדיקות הקצה mcp_server/outline_scanners/rst.py, tests/test_mcp_outline.py, tests/test_mcp_docs_handlers.py	סורק ה־RST מעביר את MAX_SYMBOLS בזמן הפרסור ומחזיר too_many_symbols. הבדיקות מאמתות עצירת עבודה ואת ברירת המחדל של parse_document.
תיעוד מגבלות MCP docs/mcp-server.rst, docs/whats-new.rst	נוסף mcp-limits עם תקרות, ערכי תגובה, כללי RST, קידוד קבצים, מידע על לוגים ותיעוד תיקונים נוספים.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 65fed

The section limit stops pathological RST inputs during parsing while preserving the existing MCP limit response. No actionable current-head risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	כותרת ה-PR מתארת באופן ברור את השינוי המרכזי: העברת אכיפת תקרת הסימבולים לתוך פרסור ה-RST במקום סינון הפלט לאחר הפרסור.
Description check	✅ Passed	תיאור ה-PR מקיף ורלוונטי. הוא כולל את מטרת השינוי, פירוט הקוד והתיעוד, תוצאות בדיקות, סיכונים, קישורים לבעיות ותוכנית rollback. רוב סעיפי התבנית מולאו, כולל סוג השינוי והצ'קליסט.
Docstring Coverage	✅ Passed	Docstring coverage is 88.24% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 17 functions across 5 files. (2 skipped: 2 …
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/rst-outline-section-ceiling-91q1m5

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{הסקשנים נעצרים בזמן ובדיוק,
הסורק מחזיר גבול ברור וחיוק.
התיעוד מאיר כל תקרה,
והבדיקות שומרות על המסגרת.
Claude Code קידד במסלול מאובטח,
CodeKeeper forever 💫}

_{Comment @coderabbitai help to get the list of available commands.}

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

[github-actions]

⏱️ Performance report

• Run: https://github.com/amirbiron/CodeBot/actions/runs/34831937661
• Artifact: perf-durations

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

[github-actions]

📖 Documentation Preview

The documentation has been built successfully!

• 📦 Download Documentation Artifacts
• 🔍 Check the workflow summary for coverage report
• 📊 Build completed at: Mon, 14 Sep 2026 10:14:12 GMT

To view locally:

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[deepsource-io]

`parse_document` has a cyclomatic complexity of 43 with "very-high" risk

A function with high cyclomatic complexity can be hard to understand and
maintain. Cyclomatic complexity is a software metric that measures the number of
independent paths through a function. A higher cyclomatic complexity indicates
that the function has more decision points and is more complex.