sad-mcp 2.3.4 → 2.3.5

package/package.json

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

package/skills/uml-class-diagram/SKILL.md

@@ -207,10 +207,10 @@ Font:             'IBM Plex Mono', monospace (Google Fonts import)
 
 ## 7. Legend Bar
 
-Include at the bottom of every SVG:
+Include at the bottom of every SVG (no dependency entry — dependencies are not used, §3):
 ```
-── Association   ◇── Aggregation   ◆── Composition   ──▷ Generalization   --→ Dependency
-Multiplicity at both ends  |  italic = operations
+── Association   ◇── Aggregation   ◆── Composition   ──▷ Generalization
+Multiplicity at both ends (rendered outside both boxes)  |  italic = operations
 ```
 
 ---
@@ -385,13 +385,18 @@ function svgRel(rel) {
   // 'assoc': no markers — plain line
 
   let s = `<polyline points="${pts}" fill="none" stroke="#334155" stroke-width="1.8" ${dash} ${mS} ${mE}/>`;
-  if (rel.fromMult) {
-    const [fx, fy] = rel.points[0];
-    s += `<text x="${fx + 4}" y="${fy - 4}" fill="#64748b" font-size="9.5" ${FM}>${escXml(rel.fromMult)}</text>`;
-  }
+  // Multiplicity labels — placed just OUTSIDE each box, offset along the line
+  // (away from the box) and beside it, so BOTH ends are always legible.
+  const multLabel = (end, adj, txt) => {
+    const dx = adj[0] - end[0], dy = adj[1] - end[1];
+    const L = Math.hypot(dx, dy) || 1, ux = dx / L, uy = dy / L;
+    const x = end[0] + ux * 16 - uy * 9, y = end[1] + uy * 16 + ux * 9 + 3;
+    return `<text x="${x}" y="${y}" text-anchor="middle" fill="#64748b" font-size="9.5" ${FM}>${escXml(txt)}</text>`;
+  };
+  if (rel.fromMult) s += multLabel(rel.points[0], rel.points[1], rel.fromMult);
   if (rel.toMult) {
-    const [tx, ty] = rel.points[rel.points.length - 1];
-    s += `<text x="${tx + 4}" y="${ty - 4}" fill="#64748b" font-size="9.5" ${FM}>${escXml(rel.toMult)}</text>`;
+    const n = rel.points.length;
+    s += multLabel(rel.points[n - 1], rel.points[n - 2], rel.toMult);
   }
   if (rel.label) {
     const mid = rel.points[Math.floor(rel.points.length / 2)];
@@ -435,7 +440,7 @@ function render() {
   const rels  = RELS.map(svgRel).join('');
   const boxes = CLASSES.map(svgBox).join('');
   // Legend is placed at totalHeight - 60 so it always lands inside the viewBox.
-  const legend = `<text x="16" y="${totalHeight - 60}" fill="#64748b" font-size="9" ${FM}>── Association  ◇── Aggregation  ◆── Composition  ──▷ Generalization  --→ Dependency (enum only)</text>`;
+  const legend = `<text x="16" y="${totalHeight - 60}" fill="#64748b" font-size="9" ${FM}>── Association  ◇── Aggregation  ◆── Composition  ──▷ Generalization  |  קרדינליות בשני קצות הקשר</text>`;
 
   const svg = document.getElementById('diagram-svg');
   svg.setAttribute('width',   totalWidth);