sad-mcp 0.1.21 → 0.1.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sad-mcp",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "description": "MCP server for Software Analysis and Design course materials at BGU",
   "type": "module",
   "bin": {

package/skills/bpmn-diagram/SKILL.md

@@ -73,6 +73,11 @@ STRUCTURAL ANALYSIS:
 - Putting a customer/patient/citizen as a lane when they act independently — they need a separate pool with message flows, not sequence flows
 - Using sequence flows between independent participants instead of message flows between separate pools
 - Forgetting that external companies (e.g., חברת אשראי, ספק, קבלן) also need their own pool — they are not part of the organization
+- Omitting a participant entirely — if the process description mentions a customer/patient/citizen, they MUST appear in the diagram as a pool with their own flow
+
+### Element Placement — CRITICAL
+
+**Every BPMN element (task, gateway, event) must be fully contained within a single lane or pool.** No element may overlap a lane divider line or sit on the boundary between two lanes. If an element belongs to a lane, it must be clearly centered vertically within that lane's area, with visible spacing from the lane borders above and below.
 
 ### Color Scheme by Participant Type
 
@@ -109,13 +114,28 @@ Example: If a Resident interacts with both a Delivery Company and a Development
 
 ### Gateways — CRITICAL RULE
 
-**Every gateway — both split AND merge/join — MUST display the X marker (for XOR) or + marker (for AND/parallel).** Merge gateways are never left as blank diamonds. This is a non-negotiable requirement.
+**Gateway diamonds must contain ONLY the marker symbol — NEVER text.** No labels, descriptions, or questions inside the diamond shape. Gateway labels go ABOVE or BESIDE the diamond, never inside it.
+
+- XOR (exclusive): **X** marker inside the diamond
+- AND (parallel): **+** marker inside the diamond
+- OR (inclusive): **O** marker inside the diamond
+
+**Every gateway — both split AND merge/join — MUST display its marker symbol.** Merge gateways are never left as blank diamonds. This is a non-negotiable requirement.
+
+**Anti-pattern — NEVER do this:**
+```svg
+<!-- WRONG: text inside gateway diamond -->
+<polygon points="..." fill="..." stroke="..."/>
+<text x="CX" y="CY">כמה ימים?</text>  <!-- FORBIDDEN -->
+```
 
+**Correct pattern:**
 ```svg
-<!-- XOR Gateway template (both split and merge) -->
+<!-- RIGHT: X marker inside, label outside -->
 <rect x="X" y="Y" width="34" height="34" class="gateway" transform="rotate(45, CX, CY)"/>
 <line x1="CX-7" y1="CY-10" x2="CX+7" y2="CY+10" class="gateway-x"/>
 <line x1="CX+7" y1="CY-10" x2="CX-7" y2="CY+10" class="gateway-x"/>
+<text x="CX" y="CY-25" text-anchor="middle">כמה ימים?</text>  <!-- label ABOVE -->
 ```
 
 ### Intermediate Events (Waiting/Catching)
@@ -225,9 +245,12 @@ Output a single self-contained HTML file with:
 
 **STOP before delivering the diagram. Verify EVERY item below. If any item fails, you MUST fix it before delivering. Do NOT skip this step.**
 
-- [ ] Every gateway (split AND merge) has an X or + symbol
+- [ ] Every gateway (split AND merge) has ONLY an X/+/O marker inside — no text inside the diamond
+- [ ] Gateway labels (questions, conditions) are positioned ABOVE or BESIDE the diamond, never inside
+- [ ] Every element (task, gateway, event) is fully inside a single lane — nothing sits on a lane boundary
 - [ ] No arrow paths overlap or cross ambiguously
 - [ ] Data objects (📄) and data stores (🗄️) use different icons
+- [ ] Every participant mentioned in the process description appears in the diagram — none are omitted
 - [ ] External participants are in separate pools with message flows
 - [ ] Passive waiting uses intermediate events, not regular tasks
 - [ ] Flow labels (כן/לא) are clearly positioned near their gateway