🚀 פיצ'ר: מימוש הרצת קוד (Code Execution) בווב-אפ

[amirbiron]

🚀 פיצ'ר: מימוש הרצת קוד (Code Execution) בווב-אפ

תיאור
הוספת יכולת להריץ קוד Python ("סביבת Playground") בצורה מאובטחת ישירות מתוך ממשק ה-Code Tools בווב-אפ.
המימוש יתבסס על ארכיטקטורת Docker Sandbox כדי להבטיח בידוד מוחלט מהשרת.

שינוי מהמדריך המקורי: הפיצ'ר יהיה זמין למשתמשי Premium בנוסף לאדמינים.

---

📋 דרישות ותכולה (Scope)

1. Backend Service (services/code_execution_service.py)

• יצירת שירות CodeExecutionService המנהל הרצת קוד.
• Docker Sandbox: הרצה בתוך קונטיינר מבודד (python:3.11-slim).
• אבטחה והגבלות:
  • fail-closed: אם אין Docker בפרודקשן - אין הרצה.
  • חסימת גישה לרשת (--network none).
  • Read-only filesystem עם /tmp מוגבל.
  • הגבלת משאבים (Memory, CPU, PIDs).
  • Timeout קשיח (ברירת מחדל 5-30 שניות).

2. API & Permissions (webapp/code_tools_api.py)

• הוספת Endpoint חדש: POST /api/code/run.
• עדכון מודל הרשאות:
  • ה-Endpoint חייב להיות פתוח למשתמשים בעלי הרשאת is_premium בנוסף ל-is_admin.
  • הערה: יש לוודא שה-before_request הקיים ב-Blueprint לא חוסם משתמשי פרימיום, או להחריג את ה-Endpoint הספציפי הזה.
• הוספת Endpoint לקבלת מגבלות: GET /api/code/run/limits.
• בדיקת Feature Flag (FEATURE_CODE_EXECUTION).

3. Frontend (code_tools.html, code-tools-page.js)

• הוספת כפתור "▶️ הרץ" ב-Toolbar (קיצור מקלדת: Ctrl+Enter).
• הוספת אזור תצוגה חדש: "פלט" (Output Console).
• הצגת סטטוס ריצה (Loading / Success / Error).
• הצגת stdout, stderr ו-exit code בצורה קריאה.
• טיפול ב-Disabled State אם הפיצ'ר כבוי בשרת.

4. Infrastructure & Config

• הוספת משתני סביבה (.env):
  • FEATURE_CODE_EXECUTION=true
  • CODE_EXEC_USE_DOCKER=true
  • הגדרות Timeout ו-Memory.
• וידוא שה-Docker Image קיים בשרת (python:3.11-slim).
• מנגנון ניקוי קונטיינרים "יתומים" (Cleanup).

---

🔒 דגשי אבטחה קריטיים

1. בידוד: אסור לאפשר לקוד המורץ גישה לקבצי השרת, למסד הנתונים או לרשת החיצונית.
2. Logging: אין לכתוב ללוגים את הקוד שהמשתמש הריץ או את הפלט שלו (מחשש למידע רגיש).
3. Privileges: הקונטיינר ירוץ ללא הרשאות (--user=nobody, --cap-drop=ALL).

✅ קריטריונים לקבלה (DoD)

1. משתמש אדמין ומשתמש פרימיום יכולים להריץ קוד ולקבל פלט.
2. משתמש רגיל (Free) מקבל שגיאת הרשאה (403).
3. קוד עם לולאה אינסופית נעצר אוטומטית (Timeout).
4. קוד שמנסה לגשת לרשת או לקבצים נחסם/נכשל.
5. בדיקות (Unit & Integration) עוברות בהצלחה.

---

אם נוספו/שונו משתני סביבה – עדכן docs/environment-variables.rst וגם services/config_inspector_service.py

---

🔒 דרישת מימוש 1:1

חובה להצמד במדויק למדריך המצורף

כללים:

• מימוש מלא: אל תקצר חלקים שמוצעים במדריך
• אם חייב לסטות: תעצור ותתעד ב-"Delta מהמדריך" (מקסימום 5 דלתאות)

המדריך

[amirbiron]

@cursoragent

היי,
תוכל לממש?

🔒 דרישת מימוש 1:1

חובה להצמד במדויק למדריך המצורף

כללים:

• מימוש מלא: אל תקצר חלקים שמוצעים במדריך
• אם חייב לסטות: תעצור ותתעד ב-"Delta מהמדריך" (מקסימום 5 דלתאות)

[cursor]

Create PR

כן, אני על זה.

אצמד 1:1 למדריך המצורף, ואממש את כל הסעיפים end-to-end: CodeExecutionService עם Docker sandbox והגבלות (network none, read-only, caps drop, user nobody, limits + timeout), ה־API (POST /api/code/run, GET /api/code/run/limits) עם Feature Flag, ועדכוני ה־Frontend (כפתור ▶️ הרץ, Ctrl+Enter, Output Console, סטטוסים).

Delta מהמדריך (כרגע 1):

• הרשאות: בנוסף לאדמין, אאפשר גם Premium (is_premium) כפי שמוגדר ב־PR. אם אצטרך עוד סטייה נקודתית, אתעד עד 5 דלתאות לכל היותר.

---

 

_{Learn more about Cursor Agents}

[amirbiron]

@cursoragent

תיקיית בעיות נפוצות בהרצת קוד

1. ❌ "המודול לא מורשה"

דוגמה

import time

print("👋 שלום מהשרת!")
print("מתחיל ספירה...")

for i in range(1, 6):
    print(f"ספירה: {i}")
    time.sleep(0.5)

print("✅ סיימתי לרוץ.")

שגיאה: המודול 'time' אינו מורשה. מודולים מותרים: array, bisect, collections, copy, dataclasses, datetime, decimal, enum, fractions, functools, heapq, itertools, json, math, operator, pprint, random, re, statistics, string, textwrap, typing

הסיבה

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

הפתרון

עדכן את קובץ services/code_execution_service.py והוסף את המודול הנחוץ ל‑ALLOWED_IMPORTS:

ALLOWED_IMPORTS: tuple[str, ...] = (
    "array",
    "bisect",
    "collections",
    "copy",
    "dataclasses",
    "datetime",
    "decimal",
    "enum",
    "fractions",
    "functools",
    "heapq",
    "itertools",
    "json",
    "math",
    "operator",
    "pprint",
    "random",
    "re",
    "statistics",
    "string",
    "textwrap",
    "time",          # ← הוסף את זה
    "typing",
)

---

2. ❌ FileNotFoundError בהרצה

דוגמה

import math
import random

radius = random.randint(5, 20)
area = math.pi * (radius ** 2)

print(f"הרדיוס שנבחר: {radius}")
print(f"שטח המעגל: {area:.2f}")
print(f"שורש של 144: {math.sqrt(144)}")

שגיאה: FileNotFoundError

הסיבה

הבעיה היא טכנית לחלוטין: כשהשרת מנסה להריץ קוד, הוא מחפש תוכנה בשם python בדיוק. בשרתים מודרניים (כמו Render), התוכנה נקראת לרוב python3, ולכן הוא "לא מוצא את הקובץ" (FileNotFoundError).

הפתרון

עדכן את הקובץ services/code_execution_service.py:

א. הוסף import בראש הקובץ:

import sys

ב. עדכן את פונקציית ההרצה _execute_subprocess:

במקום:

cmd = ["python", "-c", code]

שנה ל:

cmd = [sys.executable, "-c", code]

sys.executable הוא נתיב ההתקנה המדויק של Python בשרת הנוכחי, ולכן הוא תמיד יעבוד.

---

3. ❌ "הקוד מכיל ביטוי אסור: import os"

דוגמה

import sys
import os
import platform

print(f"🐍 Python Version: {sys.version.split()[0]}")
print(f"🖥️ OS: {platform.system()} {platform.release()}")
print(f"📂 Current Directory: {os.getcwd()}")
print(f"👤 User: {os.getlogin() if hasattr(os, 'getlogin') else 'unknown'}")

# רשימת קבצים בתיקייה הנוכחית (יראה את קבצי הפרויקט שלך)
print("\nקבצים בתיקייה:")
print(os.listdir('.')[:5]) # מציג רק את ה-5 הראשונים

שגיאה: הקוד מכיל ביטוי אסור: import os

הסיבה

מודולים כמו os ו‑sys נחסומים מטעמי אבטחה כי הם מאפשרים גישה ישירה למערכת הקבצים ולמשתני הסביבה, מה שיכול להוות סיכון בטיחות בפרודקשן.

הפתרון

אם אתה צריך מידע כזה, בחר חלופה מבטוחה:

למשל, במקום os.getcwd() או platform, בדוק אם ניתן לעבוד עם:

• datetime – למידע על זמן
• sys – רק אם הוא בRATION_IMPORTS (בדוק עם המנהל)
• פרמטרים מקובץ קונפיגורציה שנוצר בצד השרת

או שתפנה למנהל כדי להוסיף את os או sys ל‑ALLOWED_IMPORTS אם זה מבוקש לצורך ביצוע משימה ספציפית.

---

4. ❌ "הקוד מכיל ביטוי אסור: import sys"

דוגמה

print("זוהי הודעה רגילה (stdout)")

# הדפסה ישירה ל-stderr
import sys
print("זוהי הודעת שגיאה מודפסת (stderr)", file=sys.stderr)

# זריקת שגיאה אמיתית
raise ValueError("אוי לא! משהו השתבש בצורה יזומה 💥")

שגיאה: הקוד מכיל ביטוי אסור: import sys

הסיבה

sys נחסום כברירת מחדל מטעמי אבטחה, אך יתכן שתצטרך אותו לפעמיות מסוימות (כמו דוגמת הפתרון לבעיה #2).

הפתרון

אפשרות א: אם אתה צריך הדפסה ל‑stderr ללא sys, השתמש בפונקציה print:

print("זוהי הודעה רגילה (stdout)")
print("זוהי הודעת שגיאה (stderr)")

# או זריקת שגיאה ישירות
raise ValueError("אוי לא! משהו השתבש בצורה יזומה 💥")

אפשרות ב: אם sys דרוש לך (למשל למציאת sys.executable), פנה למנהל לקבל הרשאה להוסיף אותו ל‑ALLOWED_IMPORTS.

---

סיכום השינויים הנדרשים בקובץ services/code_execution_service.py

# ========== בראש הקובץ ==========
import sys  # ← הוסף את זה

# ========== בתוך הפונקציה _execute_subprocess ==========
def _execute_subprocess(self, code, timeout):
    # במקום "python" → השתמש בנתיב המלא
    cmd = [sys.executable, "-c", code]  # ← שנה את זה
    # ... שאר הקוד ...

# ========== ברשימת ALLOWED_IMPORTS ==========
ALLOWED_IMPORTS: tuple[str, ...] = (
    "array",
    "bisect",
    "collections",
    "copy",
    "dataclasses",
    "datetime",
    "decimal",
    "enum",
    "fractions",
    "functools",
    "heapq",
    "itertools",
    "json",
    "math",
    "operator",
    "pprint",
    "random",
    "re",
    "statistics",
    "string",
    "textwrap",
    "time",  # ← הוסף מודולים שאתה צריך
    "typing",
)

[cursor]

Create PR

מעולה — זה בדיוק סוג ה־“פינות” שכדאי לסגור כדי שה־Playground יעבוד חלק, בלי לפגוע באבטחה.

מה אני עושה בעקבות זה

• time לא מורשה: אוסיף את time ל־ALLOWED_IMPORTS ב־services/code_execution_service.py כדי שהדוגמה שלך תרוץ.
• FileNotFoundError: אתקן את מסלול ההרצה הלא־Docker (אם קיים אצלנו לפיתוח/בדיקות) כך שישתמש ב־sys.executable במקום "python", כדי שלא נהיה תלויים בשם בינארי ספציפי (python/python3).
• sys/os בקוד משתמש: אשאיר אותם חסומים כברירת מחדל (מטעמי אבטחה). השימוש ב־sys הוא פנימי לשירות בלבד (לבניית הפקודה), ולא “פותח” import sys למשתמשים.

הערה קטנה על #4 (stderr)

אם המטרה היא רק להציג שגיאות למשתמש, אפשר לקבל stderr גם בלי לאפשר import sys בקוד המשתמש (למשל דרך חריגה/traceback). אם בכל זאת תרצה לאפשר sys למשתמשים — אני אעדיף לעשות את זה כחריג מפורש ומדוד, כי זה מגדיל משטח תקיפה.

---

 

_{Learn more about Cursor Agents}

[amirbiron]

@cursoragent

• להוסיף כפתור קישור לעמוד /tools/code בעמוד כל הקבצים, בשורה של כפתורים אחרים, בין הכפתור הוסף קטע קוד חדש לכפתור השוואת קבצים - כפתור שהוא רק אימוגי בלי טקסט, כמו שאר הכפתורים בשורה, אימוגי בצבע לבן, עדיף כזה : 🎮, אפשרות שנייה:: 🧪
• להוסיף לכותרת העמוד את הפיצר החדש שיהיה רשום ככה: Playground לעיצוב, בדיקת איכות, תיקון אוטומטי והרצת קוד (Python) במקום מה שמופיע עכשיו: Playground לעיצוב, בדיקת איכות ותיקון אוטומטי (Python)
• לפתוח את עמוד /tools/code לכל המשתמשים - מלבד הפיצ`ר של "הרץ קוד" שיישאר רק ללאדמינים ומשתמי פרימיום
• לפתוח את הפיצ`ר של עיצוב לינטר בעמוד העלאת/עריכת קובץ לכל המשתמשים

[cursor]

View PR

עדכנתי לפי הבקשה

• כפתור קישור בעמוד “כל הקבצים”: הוספתי כפתור אימוג’י 🎮 (בלי טקסט) שמקשר ל־/tools/code, ממוקם בין “הוסף קובץ חדש” ל־“השוואת קבצים” (webapp/templates/files.html).
• כותרת Playground: עדכנתי את הטקסט ל־Playground לעיצוב, בדיקת איכות, תיקון אוטומטי והרצת קוד (Python) (webapp/templates/code_tools.html).
• הרשאות /tools/code: פתחתי את העמוד לכל משתמש מחובר (Login), אבל השארתי את “הרץ קוד” (API של /api/code/run ו־/api/code/run/limits) רק ל־Admin/Premium (webapp/app.py, webapp/code_tools_api.py).
• עיצוב/Lint בעמוד העלאה/עריכה לכל המשתמשים: הסרתי את החסימה לפי אדמין, כך שהטולבר וה־JS נטענים לכל משתמש (webapp/templates/components/editor_components.html, webapp/templates/upload.html, webapp/templates/edit_file.html).

בנוסף עדכנתי גם את התיעוד כדי שישקף שהעמוד פתוח לכולם אבל ההרצה עצמה מוגבלת ל־Premium/Admin (docs/webapp/user-interfaces.rst, docs/webapp/code-execution.rst, docs/security.rst).

---

 

_{Learn more about Cursor Agents}