archgraph-argo 0.19.0 → 0.19.1

package/argo/scripts/ea-qea-sync-lib.js

@@ -698,6 +698,7 @@ function syncGraphToQea(graph, qeaPath, opts) {
           RectRight: 40 + col * 260 + 180, RectBottom: 40 + row * 160 + 90,
           Sequence: seq,
         });
+        placedObjs.add(Number(oid));
         seq++;
       }
       if (newObjs.length > 0 && !o.dryRun) {
@@ -731,6 +732,7 @@ function syncGraphToQea(graph, qeaPath, opts) {
         if (cid === undefined || cid < 0) { continue; }
         if (placedLinks.has(Number(cid))) { continue; }
         newLinks.push({ DiagramID: diagramId, ConnectorID: Number(cid), Style: '', Geometry: '' });
+        placedLinks.add(Number(cid));
       }
       if (newLinks.length > 0 && !o.dryRun) {
         insertMany(db, 't_diagramlinks', ['DiagramID', 'ConnectorID', 'Style', 'Geometry'], newLinks);

package/argo/scripts/graph-semantics.js

@@ -116,12 +116,19 @@ function validateGraphSemantics(document, errors) {
       }
     }
     const includedElementIds = new Set(view.included_elements || []);
+    if (includedElementIds.size !== (view.included_elements || []).length) {
+      errors.push(`views '${view.view_id}' must not contain duplicate included_elements`);
+    }
     for (const elementId of view.included_elements || []) {
       elementIdsIncludedInViews.add(elementId);
       if (!elementById.has(elementId)) {
         errors.push(`views '${view.view_id}' references missing included element '${elementId}'`);
       }
     }
+    const includedRelationshipIds = new Set(view.included_relationships || []);
+    if (includedRelationshipIds.size !== (view.included_relationships || []).length) {
+      errors.push(`views '${view.view_id}' must not contain duplicate included_relationships`);
+    }
     for (const relationshipId of view.included_relationships || []) {
       relationshipIdsIncludedInViews.add(relationshipId);
       const rel = relationshipById.get(relationshipId);

package/argo/scripts/systemarchitecture-mcp-server.js

@@ -1519,11 +1519,18 @@ function applyMutations(document, mutations) {
           requestedId: mutation.view.view_id,
         });
       } else {
-        nextDocument.views.push(clone(mutation.view));
-        upsertSubdiagramViewIntoElement(nextDocument, mutation.view.parent_element_id, mutation.view);
-        touchedViewIds.add(mutation.view.view_id);
-        viewLimitCheckIds.add(mutation.view.view_id);
-        mutationSummaries.push({ type: mutation.type, id: mutation.view.view_id });
+        const newView = clone(mutation.view);
+        if (Array.isArray(newView.included_elements)) {
+          newView.included_elements = addUnique([], newView.included_elements);
+        }
+        if (Array.isArray(newView.included_relationships)) {
+          newView.included_relationships = addUnique([], newView.included_relationships);
+        }
+        nextDocument.views.push(newView);
+        upsertSubdiagramViewIntoElement(nextDocument, newView.parent_element_id, newView);
+        touchedViewIds.add(newView.view_id);
+        viewLimitCheckIds.add(newView.view_id);
+        mutationSummaries.push({ type: mutation.type, id: newView.view_id });
       }
       continue;
     }
@@ -1538,7 +1545,14 @@ function applyMutations(document, mutations) {
       }
       const oldParentId = view.parent_element_id;
       const oldViewId = view.view_id;
-      Object.assign(view, clone(mutation.patch));
+      const patch = clone(mutation.patch);
+      if (Array.isArray(patch.included_elements)) {
+        patch.included_elements = addUnique([], patch.included_elements);
+      }
+      if (Array.isArray(patch.included_relationships)) {
+        patch.included_relationships = addUnique([], patch.included_relationships);
+      }
+      Object.assign(view, patch);
       if (oldParentId !== view.parent_element_id) {
         removeSubdiagramViewFromElement(nextDocument, oldParentId, oldViewId);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "archgraph-argo",
-  "version": "0.19.0",
+  "version": "0.19.1",
   "description": "Deploy the ArchGraph ARGO toolchain, skills, and rules (schema, scripts, argo-init skill, global rule) with one command.",
   "license": "MIT",
   "bin": {