ezer-mishpati/legal-ai
1
0
Fork 0
Code Issues 29 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(ops): יישור MULTIMODAL_ENABLED בשרת ה-MCP + תיעוד דריפט האחסון שנותר #466

Merged
chaim merged 1 commits from worktree-mcp-env-align into main 2026-08-05 13:58:41 +00:00
Conversation 0 Commits 1 Files Changed 2 +36
chaim commented 2026-08-05 13:58:27 +00:00
Owner
Copy Link

הקונטיינר ושרת ה-MCP הם שתי דלתות לאותו קורפוס וחייבים לדרג אותו זהה. הם לא עשו זאת: הקונטיינר MULTIMODAL_ENABLED=true, השרת נפל ל-false (‏~/.env לא מגדיר). סוכן ו-UI יכלו לענות על אותה שאלה מקבוצות-תוצאות שונות (G2).

שונה: MULTIMODAL_ENABLED: "true" בקונפיג pm2 + תיעוד השירות ב-runbook, כולל האזהרה שקוד ב-mcp-server/ לא נכנס לתוקף עד restart — שלושה מיזוגים היום רצו על קוד ישן עד שהבחנתי.

לא שונה — STORAGE_BACKEND: דורש MINIO_ENDPOINT/ACCESS_KEY/SECRET_KEY שאינם ב-Infisical (רק ב-Coolify). בלעדיהם MINIO_ENDPOINT נפתר ל-http://minio:9000 — שם-רשת פנימי, לא נגיש מהמארח — וכל פעולת-בלוב נשברת. יצירת סוד חדש דורשת אישור חיים, אז הדריפט מתועד ב-runbook עם בדיוק מה שנדרש לסגירה. MinIO כן נגיש מהמארח (https://s3.nautilus.marcusgroup.org → 200).

invariants: G2 · §6

הקונטיינר ושרת ה-MCP הם שתי דלתות לאותו קורפוס וחייבים לדרג אותו זהה. הם לא עשו זאת: הקונטיינר `MULTIMODAL_ENABLED=true`, השרת נפל ל-`false` (‏`~/.env` לא מגדיר). סוכן ו-UI יכלו לענות על אותה שאלה מקבוצות-תוצאות שונות (G2). **שונה:** `MULTIMODAL_ENABLED: "true"` בקונפיג pm2 + תיעוד השירות ב-runbook, כולל האזהרה שקוד ב-`mcp-server/` **לא נכנס לתוקף עד restart** — שלושה מיזוגים היום רצו על קוד ישן עד שהבחנתי. **לא שונה — `STORAGE_BACKEND`:** דורש `MINIO_ENDPOINT`/`ACCESS_KEY`/`SECRET_KEY` שאינם ב-Infisical (רק ב-Coolify). בלעדיהם `MINIO_ENDPOINT` נפתר ל-`http://minio:9000` — שם-רשת פנימי, לא נגיש מהמארח — וכל פעולת-בלוב נשברת. יצירת סוד חדש דורשת אישור חיים, אז הדריפט **מתועד ב-runbook** עם בדיוק מה שנדרש לסגירה. MinIO כן נגיש מהמארח (`https://s3.nautilus.marcusgroup.org` → 200). **invariants:** G2 · §6
chaim added 1 commit 2026-08-05 13:58:27 +00:00
fix(ops): יישור MULTIMODAL_ENABLED בשרת ה-MCP + תיעוד דריפט האחסון שנותר
All checks were successful
INV-AG3 Agent Tool Grants / agent-tool-grants (pull_request) Successful in 6s
Details
G12 Leak-Guard / leak-guard (pull_request) Successful in 6s
Details
Lint — undefined names / undefined-names (pull_request) Successful in 14s
Details
95571e0f05 
הקונטיינר והשרת ה-HTTP הם שתי דלתות לאותו קורפוס, ולכן חייבים לדרג
אותו. הם לא עשו זאת: הקונטיינר רץ `MULTIMODAL_ENABLED=true`, בעוד שרת
ה-MCP נפל לברירת-המחדל `false` — `~/.env` אינו מגדיר את הדגל כלל. סוכן
ו-UI יכלו לענות על אותה שאלה מתוך קבוצות-תוצאות שונות, בלי שאיש ידע
(G2 — מסלול מקביל שמתפצל בשקט).

מה שונה
- `MULTIMODAL_ENABLED: "true"` בקונפיג ה-pm2 של legal-mcp-http.
- תיעוד השירות ב-operations-runbook (פורט, שער-Bearer, ו**האזהרה
  שהחשובה מכולן**: קוד ב-`mcp-server/` לא נכנס לתוקף עד restart — זהו
  תהליך ארוך-חיים, בניגוד ל-stdio שנטען בכל סשן. שלושה מיזוגים היום
  רצו על קוד ישן עד שהבחנתי).

מה **לא** שונה, ולמה
`STORAGE_BACKEND` נשאר לא-מוגדר. הקונטיינר רץ `s3`, השרת נופל
ל-`filesystem`, וזה דריפט אמיתי — סוכן שכותב דרך MCP כותב לדיסק בעוד
ה-web קורא מ-S3. אבל הפיכת הדגל דורשת `MINIO_ENDPOINT`/`ACCESS_KEY`/
`SECRET_KEY`, ואלה **אינם ב-Infisical** — רק ב-env של Coolify. בלעדיהם
`MINIO_ENDPOINT` נפתר לברירת-המחדל `http://minio:9000` (שם-רשת פנימי של
דוקר, לא נגיש מהמארח) וכל פעולת-בלוב נשברת.

יצירת סוד חדש מחייבת אישור חיים לשם/תיקייה/תגית, ולכן הדריפט מתועד
בבירור ב-runbook עם בדיוק מה שצריך כדי לסגור אותו — במקום להישאר
ידיעה שבעל-פה. MinIO **כן** נגיש מהמארח: `https://s3.nautilus.marcusgroup.org`
מחזיר 200 ב-`/minio/health/live` (אומת), כך שהחסם היחיד הוא הסודות.

invariants: G2 (שתי דלתות, דירוג אחד) · §6 (הדריפט הנותר מתועד, לא נבלע)
chaim merged commit 09c737bddf into main 2026-08-05 13:58:41 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
area:agents
סוכני Paperclip, תזמור, שער-הפלטפורמה (X15)
 area:backend
FastAPI web/, endpoints
 area:ci
Gitea Actions, שערי-CI
 area:docs
ספ-המערכת, תיעוד, תכנון
 area:extraction
קליטה, OCR, חילוץ הלכות/מטא/טענות/תכניות
 area:frontend
web-ui/ (Next.js), RTL
 area:infra
Deploy, Coolify, MinIO, אחסון, גיבויים
 area:mcp
שרת ה-MCP וחוזה-הכלים (X9)
 area:retrieval
RAG, חיפוש, קורפוס, גרף-ציטוטים, יומונים
 area:security
אבטחה, סודות, PII, audit
 area:writing
12 הבלוקים, סגנון דפנה, QA, רכישת-סגנון
 priority:p0-blocker
Blocks everything
 priority:p1-high
High priority
 priority:p2-normal
Normal priority
 priority:p3-low
Low priority / nice to have
 size:l
a week+ — consider splitting
 size:m
2-3 days
 size:s
~1 day
 size:xs
< half a day
 status:blocked
Waiting on another issue/decision
 status:deferred
Decided to defer — parked backlog; not auto-picked
 status:good-first-issue
Small, self-contained starting point
 status:in-progress
Being worked on (a branch/worktree exists)
 status:in-review
A PR is open for this issue
 status:needs-decision
Requires a call before proceeding
 status:ready
Scoped + pickable (has acceptance criteria + size)
 type:bug
Something works incorrectly
 type:chore
Infra, deps, refactor, CI, docs
 type:feature
New capability
 type:spike
Time-boxed investigation -> a decision/doc
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
chaim
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#466
Delete Branch "worktree-mcp-env-align"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?