sad-mcp 1.1.5 → 1.1.7

package/dist/tools.js

@@ -10,6 +10,7 @@ import { trackToolCall } from "./tracking.js";
 import { renderBPMN, parseAndValidate, formatIssues, } from "./bpmn/index.js";
 const __tools_filename = fileURLToPath(import.meta.url);
 const __tools_dirname = dirname(__tools_filename);
+const PKG = JSON.parse(readFileSync(join(__tools_dirname, "..", "package.json"), "utf-8"));
 function loadSharedContent(skillsDir) {
     const sharedDir = join(skillsDir, "_shared");
     const files = ["layout-recipes.md", "model-shape.md", "export-buttons.md"];
@@ -354,6 +355,14 @@ export function registerToolHandlers(server) {
                     required: ["html_content", "filename"],
                 },
             },
+            {
+                name: "version",
+                description: "Return the version of the sad-mcp server currently running. Use this to verify which version is loaded when skills behave unexpectedly (npx can serve a stale cached version).",
+                inputSchema: {
+                    type: "object",
+                    properties: {},
+                },
+            },
         ],
     }));
     server.setRequestHandler(CallToolRequestSchema, async (request) => {
@@ -675,6 +684,11 @@ export function registerToolHandlers(server) {
             trackToolCall(name, toolArgs, { success: true, responseChars: summary.length }, Date.now() - startTime);
             return { content: [{ type: "text", text: summary }] };
         }
+        if (name === "version") {
+            const text = `${PKG.name}@${PKG.version}`;
+            trackToolCall(name, toolArgs, { success: true, responseChars: text.length }, Date.now() - startTime);
+            return { content: [{ type: "text", text }] };
+        }
         throw new Error(`Unknown tool: ${name}`);
     });
 }

package/package.json

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

package/skills/uml-use-case-diagram/SKILL.md

@@ -603,7 +603,7 @@ Fonts:              'Noto Sans Hebrew' + 'IBM Plex Mono' from Google Fonts
 </html>
 ```
 
-For a single-tab diagram, drop the tab bar, the split-rationale line, and the second `.diagram-part`; keep `diagramLayouts` as a one-element array so `showDiagram` / `fixLayout` wiring stays uniform.
+**Single-tab diagrams still define the full script.** Always emit `diagramLayouts` (one-element array), `activeTab`, `currentLayout()`, **and `showDiagram(index)`**. Only the HTML-level tab bar and the second `.diagram-part` are conditional on splitting. Omitting `showDiagram` is a silent bug: a future edit that adds tabs will have dead onclick handlers with no console error pointing at the cause.
 
 ---
 
@@ -629,6 +629,7 @@ For a single-tab diagram, drop the tab bar, the split-rationale line, and the se
 - [ ] Every UC label describes an action WITH the information system (data entry, query, state change) — no bare physical verbs (`הכנת X`, `ספירת X`, `מסירת X`, `תשלום במזומן`). Physical verbs are wrapped with a system verb (`רישום`, `הזנה`, `עדכון`, `סימון`, etc.).
 - [ ] Every system-time / cron actor association has a `timing` field (and the rendered diagram shows a visible label at the line midpoint) — no unlabeled cron connections.
 - [ ] No "send-only" UCs (no bare `שליחת X` / `הודעה ל-X`); message-sends are folded into the UC that decided to send.
+- [ ] `showDiagram(index)` is defined in every generated file, even single-tab diagrams (so tab switching is wired if splitting is ever added).
 - [ ] `fixLayout` uses adaptive column X (widens boundary if needed so left/center/right ellipses never overlap horizontally) AND expands viewBox width so right-side actors are not clipped.
 - [ ] If split: each tab has a split-rationale line citing the actual trigger that fired (count > 8, or 2+ disjoint actor groups); `showDiagram` and per-tab `fixLayout` wired per §11.
 - [ ] `diagramModel` defined per `model-shape.md`; VP export button present and wired to `exportXMI()`.