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

fix(ops): שרת ה-MCP עובר ל-S3 — סוגר את דריפט האחסון מול הקונטיינר #467

Merged
chaim merged 1 commits from worktree-mcp-s3 into main 2026-08-05 14:03:51 +00:00
Conversation 0 Commits 1 Files Changed 2 +38 -19
chaim commented 2026-08-05 14:03:36 +00:00
Owner
Copy Link

מסמך שסוכן כתב דרך MCP נחת על הדיסק בעוד ה-web חיפש אותו ב-S3 — שני מסלולים לאותו נכס (G2 / INV-STG1).

שני חסמים שהתגלו בדרך:

  1. הסודות לא היו במקור-האמת — MINIO_ENDPOINT/ACCESS_KEY/SECRET_KEY חיו רק ב-Coolify. נוצרו ב-Infisical /apps/legal-ai (tag credentials, באישור חיים), נטענים ל-~/.legal-mcp-http.env כמו MCP_HTTP_SHARED_SECRET. הערכים עברו ב-stdin — לא ב-argv ולא בהיסטוריה.
  2. aioboto3 לא היה מותקן ב-venv — מוצהר ב-pyproject אך חסר. מסלול ה-S3 מייבא אותו עצלנית, כך שהשירות היה עולה יפה ונשבר רק בפעולת-הקובץ הראשונה. כשל שקט עד לשימוש — בדיוק למה הבדיקה רצה לפני ההפעלה.

אימות מהמארח: put/get/delete מלא מול https://s3.nautilus.marcusgroup.org + קריאת בלוב-ייצור קיים (573KB) מתוך 2,283 אובייקטים.

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

invariants: G2 · INV-STG1 · §6

מסמך שסוכן כתב דרך MCP נחת על הדיסק בעוד ה-web חיפש אותו ב-S3 — שני מסלולים לאותו נכס (G2 / INV-STG1). **שני חסמים שהתגלו בדרך:** 1. **הסודות לא היו במקור-האמת** — `MINIO_ENDPOINT`/`ACCESS_KEY`/`SECRET_KEY` חיו רק ב-Coolify. נוצרו ב-Infisical `/apps/legal-ai` (tag `credentials`, באישור חיים), נטענים ל-`~/.legal-mcp-http.env` כמו `MCP_HTTP_SHARED_SECRET`. הערכים עברו ב-stdin — לא ב-argv ולא בהיסטוריה. 2. **`aioboto3` לא היה מותקן ב-venv** — מוצהר ב-pyproject אך חסר. מסלול ה-S3 מייבא אותו **עצלנית**, כך שהשירות היה עולה יפה ונשבר רק בפעולת-הקובץ הראשונה. כשל שקט עד לשימוש — בדיוק למה הבדיקה רצה **לפני** ההפעלה. **אימות מהמארח:** put/get/delete מלא מול `https://s3.nautilus.marcusgroup.org` + קריאת בלוב-ייצור קיים (573KB) מתוך **2,283** אובייקטים. ה-runbook: בלוק-האזהרה הוחלף בתיעוד המצב הסגור, עם מלכודת הייבוא-העצל ופקודת-אימות להעתקה. **invariants:** G2 · INV-STG1 · §6
chaim added 1 commit 2026-08-05 14:03:37 +00:00
fix(ops): שרת ה-MCP עובר ל-S3 — סוגר את דריפט האחסון מול הקונטיינר
All checks were successful
INV-AG3 Agent Tool Grants / agent-tool-grants (pull_request) Successful in 5s
Details
G12 Leak-Guard / leak-guard (pull_request) Successful in 5s
Details
Lint — undefined names / undefined-names (pull_request) Successful in 12s
Details
adb9586055 
הקונטיינר רץ `STORAGE_BACKEND=s3` ושרת ה-MCP נפל ל-`filesystem`. כלומר
מסמך שסוכן כתב דרך MCP נחת על הדיסק בעוד ה-web חיפש אותו ב-S3 — שני
מסלולים לאותו נכס, שמתפצלים בשקט (G2 / INV-STG1).

מה נדרש כדי לסגור, ומה נמצא בדרך
- **הסודות לא היו קיימים במקור-האמת.** `MINIO_ENDPOINT`/`ACCESS_KEY`/
  `SECRET_KEY` חיו רק ב-env של Coolify. נוצרו ב-Infisical תחת
  `/apps/legal-ai` (tag `credentials`, באישור חיים) ונטענים
  ל-`~/.legal-mcp-http.env` — אותו דפוס כמו MCP_HTTP_SHARED_SECRET.
  הערכים עברו ב-stdin דרך infisical-set.sh; לא ב-argv ולא בהיסטוריה.
- **`aioboto3` לא היה מותקן ב-venv המקומי.** מוצהר ב-pyproject אך חסר.
  מסלול ה-S3 מייבא אותו **עצלנית**, ולכן השירות היה עולה בהצלחה ונשבר
  רק בפעולת-הקובץ הראשונה — כשל שקט עד לשימוש. זו בדיוק הסיבה שהבדיקה
  רצה לפני ההפעלה ולא אחריה.

אימות מהמארח לפני ההפעלה: put/get/delete מלא מול
https://s3.nautilus.marcusgroup.org, וקריאת בלוב-ייצור קיים
(573KB) מתוך 2,283 אובייקטים בדלי.

ה-runbook עודכן: בלוק-האזהרה הוחלף בתיעוד המצב הסגור, כולל מלכודת
הייבוא-העצל ופקודת-אימות שאפשר להעתיק.

invariants: G2 (מסלול-אחסון אחד לשתי הדלתות) · INV-STG1 · §6
chaim merged commit 50e3fa4023 into main 2026-08-05 14:03:51 +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#467
Delete Branch "worktree-mcp-s3"

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?