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

feat(ops): שירות pm2 legal-mcp-http (#231.3) #428

Merged
chaim merged 1 commits from worktree-mcp-http-service into main 2026-08-05 08:27:08 +00:00
Conversation 0 Commits 1 Files Changed 2 +97
chaim commented 2026-08-05 08:21:04 +00:00
Owner
Copy Link

למה

סוכנים המונעים דרך Agent Client Protocol מקבלים את שרתי-ה-MCP שלהם מהלקוח בפתיחת הסשן, והערוץ הזה נושא שרתי HTTP בלבד. ל-stdio אין מסלול לשם — ומכאן ש-108 הכלים נעדרו מהסוכנים. זהו הקצה ה-HTTP שאפשר להפנות אותם אליו.

שלב 3 מתוך 5 ב-#231.

אינו מחליף את stdio

כל סשן אינטראקטיבי ממשיך להגיע לאותו שרת דרך הרשומה legal-ai ב-~/.claude.json. אותו קוד, אותו מרשם-כלים, שתי תחבורות (G2) — דלת שנייה, לא שרת שני. אומת שה-stdio ממשיך לעבוד.

אבטחה — שתי הגנות, שתיהן נדרשות

  1. bind ל-127.0.0.1 — צר יותר מ-10.0.1.1 של legal-chat-service בכוונה: שום קונטיינר לא צריך לקרוא ל-MCP.
  2. Bearer מ-MCP_HTTP_SHARED_SECRET. השרת מסרב לעלות בלי טוקן, ולכן תקלת-הגדרה לא יכולה לייצר בשקט מאזין לא-מאומת — pm2 יציג crash במקום להגיש תעבורה פתוחה.

max_restarts נמוך במתכוון: טוקן חסר מפיל את התהליך מיד, ועדיף ש-pm2 יפסיק לנסות ויותיר רשומת errored גלויה מאשר ילולאה על טעות-הגדרה.

הסוד

מקור-אמת ב-Infisical: All Infrastructure / main / /apps/legal-ai/MCP_HTTP_SHARED_SECRET, תג credentials. ‏~/.legal-mcp-http.env הוא עותק-ריצה chmod 600 — אותו סידור כמו legal-chat-service ו-legal-court-fetch-service.

אומת על השירות החי

בדיקה תוצאה
POST בלי Authorization 401
POST עם טוקן שגוי 401
POST עם הטוקן הנכון 200
אחרי pm2 restart 200
pm2 save ✓

Invariants

  • G2 — מקיים: תחבורה שנייה לאותו שרת, לא מימוש מקביל.
  • G12 / INV-AG3 — לא נגועים.

נרשם ב-scripts/SCRIPTS.md כנדרש.

## למה סוכנים המונעים דרך Agent Client Protocol מקבלים את שרתי-ה-MCP שלהם **מהלקוח** בפתיחת הסשן, והערוץ הזה נושא **שרתי HTTP בלבד**. ל-stdio אין מסלול לשם — ומכאן ש-108 הכלים נעדרו מהסוכנים. זהו הקצה ה-HTTP שאפשר להפנות אותם אליו. שלב 3 מתוך 5 ב-#231. ## אינו מחליף את stdio כל סשן אינטראקטיבי ממשיך להגיע לאותו שרת דרך הרשומה `legal-ai` ב-`~/.claude.json`. **אותו קוד, אותו מרשם-כלים, שתי תחבורות** (G2) — דלת שנייה, לא שרת שני. אומת שה-stdio ממשיך לעבוד. ## אבטחה — שתי הגנות, שתיהן נדרשות 1. **bind ל-`127.0.0.1`** — צר יותר מ-`10.0.1.1` של `legal-chat-service` **בכוונה**: שום קונטיינר לא צריך לקרוא ל-MCP. 2. **Bearer** מ-`MCP_HTTP_SHARED_SECRET`. השרת **מסרב לעלות בלי טוקן**, ולכן תקלת-הגדרה לא יכולה לייצר בשקט מאזין לא-מאומת — pm2 יציג crash במקום להגיש תעבורה פתוחה. `max_restarts` נמוך במתכוון: טוקן חסר מפיל את התהליך מיד, ועדיף ש-pm2 יפסיק לנסות ויותיר רשומת `errored` גלויה מאשר ילולאה על טעות-הגדרה. ## הסוד מקור-אמת ב-Infisical: **All Infrastructure / `main` / `/apps/legal-ai/MCP_HTTP_SHARED_SECRET`**, תג `credentials`. ‏`~/.legal-mcp-http.env` הוא עותק-ריצה chmod 600 — אותו סידור כמו `legal-chat-service` ו-`legal-court-fetch-service`. ## אומת על השירות החי | בדיקה | תוצאה | |---|---| | POST בלי `Authorization` | **401** | | POST עם טוקן שגוי | **401** | | POST עם הטוקן הנכון | **200** | | אחרי `pm2 restart` | **200** | | `pm2 save` | ✓ | ## Invariants - **G2** — מקיים: תחבורה שנייה לאותו שרת, לא מימוש מקביל. - **G12** / **INV-AG3** — לא נגועים. נרשם ב-`scripts/SCRIPTS.md` כנדרש.
chaim added 1 commit 2026-08-05 08:21:05 +00:00
feat(ops): שירות pm2 legal-mcp-http — שרת ה-MCP חשוף ב-HTTP מאחורי שער Bearer (#231.3)
All checks were successful
Lint — undefined names / undefined-names (pull_request) Successful in 11s
Details
INV-AG3 Agent Tool Grants / agent-tool-grants (pull_request) Successful in 4s
Details
G12 Leak-Guard / leak-guard (pull_request) Successful in 5s
Details
3f48dc0e11 
סוכנים המונעים דרך Agent Client Protocol מקבלים את שרתי-ה-MCP שלהם מהלקוח
בפתיחת הסשן, והערוץ הזה נושא שרתי HTTP בלבד. ל-stdio אין מסלול לשם, ומכאן
שהסוכנים נותרו בלי 108 הכלים. זהו הקצה ה-HTTP שאפשר להפנות אותם אליו.

**אינו מחליף את stdio.** כל סשן אינטראקטיבי ממשיך להגיע לאותו שרת דרך
הרשומה legal-ai ב-~/.claude.json. אותו קוד, אותו מרשם-כלים, שתי תחבורות
(G2) — דלת שנייה, לא שרת שני.

אבטחה — שתי הגנות, שתיהן נדרשות:
1. bind ל-127.0.0.1. צר יותר מ-10.0.1.1 של legal-chat-service בכוונה: שום
   קונטיינר לא צריך לקרוא ל-MCP.
2. Bearer מ-MCP_HTTP_SHARED_SECRET. השרת מסרב לעלות בלי טוקן
   (services/http_auth.py), ולכן תקלת-הגדרה לא יכולה לייצר בשקט מאזין
   לא-מאומת — pm2 יציג crash במקום להגיש תעבורה פתוחה.

הסוד: מקור-אמת ב-Infisical (All Infrastructure / main / /apps/legal-ai,
תג credentials). ~/.legal-mcp-http.env הוא עותק-ריצה chmod 600, אותו סידור
כמו legal-chat-service.

max_restarts נמוך במתכוון: טוקן חסר מפיל את התהליך מיד, ועדיף ש-pm2 יפסיק
לנסות ויותיר רשומת errored גלויה מאשר ילולאה על טעות-הגדרה.

אומת על השירות החי:
  POST בלי Authorization → 401
  POST עם טוקן שגוי      → 401
  POST עם הטוקן הנכון    → 200
  אחרי pm2 restart       → 200
  pm2 save               → ✓

invariants: G2 — מקיים (תחבורה שנייה לאותו שרת, לא מימוש מקביל).
G12 — לא נגוע. INV-AG3 — לא נגוע.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
chaim merged commit b394177207 into main 2026-08-05 08:27:08 +00:00
chaim deleted branch worktree-mcp-http-service 2026-08-05 08:27:08 +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#428
Delete Branch "worktree-mcp-http-service"

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?