archgraph-argo 0.22.1 → 0.22.2

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

@@ -893,11 +893,42 @@ function fullProjection(graph, qeaPath, opts) {
 // ---------------------------------------------------------------------------
 // Export
 // ---------------------------------------------------------------------------
+// EA stores connector layout in two distinct t_diagramlinks columns, and they mean
+// different things:
+//   - Path     = the connector ROUTE: user-adjusted bend points as "x:y;x:y;" in diagram
+//                coordinates (empty when EA auto-routes a straight/orthogonal line).
+//   - Geometry = the non-route OVERRIDE tokens (SX/SY/EX/EY dock offsets, EDGE route
+//                style and label positions). It never contains bend points.
+// Reading Geometry as if it were the line was the original defect: callers got the
+// override string and could not rebuild the connector. We expose the route under `path`
+// (+ parsed `points`), the route style as `edge`, and keep the override as `geometry`.
+function parseRoutePoints(raw) {
+  const s = String(raw === null || raw === undefined ? '' : raw).trim();
+  if (s === '') { return []; }
+  const points = [];
+  for (const seg of s.split(';')) {
+    const t = seg.trim();
+    if (t === '') { continue; }
+    const parts = t.split(':');
+    if (parts.length < 2) { continue; }
+    const x = Number(parts[0]);
+    const y = Number(parts[1]);
+    if (!Number.isFinite(x) || !Number.isFinite(y)) { continue; }
+    points.push({ x, y });
+  }
+  return points;
+}
+function parseEdgeToken(geometry) {
+  const m = /(?:^|;)EDGE=(-?\d+)(?:;|$)/.exec(String(geometry === null || geometry === undefined ? '' : geometry));
+  return m ? Number(m[1]) : null;
+}
+
 // Read the EA diagram GEOMETRY for one KG view from a .qea model (read-only).
 // The view maps to the diagram anchored by the deterministic ea_guid (diag:<viewId>)
 // or the schema_view_id StyleEx token written by the sync. Returns:
 //   - element boxes  = t_diagramobjects rects joined to t_object.Alias (schema id)
-//   - connector lines = t_diagramlinks rows joined to the connector's schema_id tag
+//   - connector lines = t_diagramlinks rows joined to the connector's schema_id tag,
+//                       with the ROUTE (t_diagramlinks.Path) parsed into `points`
 // Returns null when the view has no matching EA diagram. Never writes EA geometry.
 function readViewDiagramGeometry(qeaPath, viewId) {
   const v = String(viewId === null || viewId === undefined ? '' : viewId).trim();
@@ -920,13 +951,14 @@ function readViewDiagramGeometry(qeaPath, viewId) {
       left: Number(r.left), top: Number(r.top), right: Number(r.right), bottom: Number(r.bottom),
     }));
     const relationships = db.prepare(
-      'SELECT t.VALUE AS id, dl.Geometry AS path ' +
+      'SELECT t.VALUE AS id, dl.Path AS path, dl.Geometry AS geometry ' +
       'FROM t_diagramlinks dl JOIN t_connectortag t ON t.ElementID = dl.ConnectorID AND t.Property = ? ' +
       'WHERE dl.DiagramID = ? ORDER BY dl.ConnectorID'
-    ).all('schema_id', diagramId).map((r) => ({
-      id: String(r.id),
-      path: String(r.path === null || r.path === undefined ? '' : r.path),
-    }));
+    ).all('schema_id', diagramId).map((r) => {
+      const path = String(r.path === null || r.path === undefined ? '' : r.path);
+      const geometry = String(r.geometry === null || r.geometry === undefined ? '' : r.geometry);
+      return { id: String(r.id), path, points: parseRoutePoints(path), edge: parseEdgeToken(geometry), geometry };
+    });
     return { diagramId, elements, relationships };
   } finally {
     try { db.close(); } catch { /* ignore */ }

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

@@ -226,7 +226,7 @@ const TOOLS = [
   },
   {
     name: 'getArchitectureViewContext',
-    description: 'read-only query that resolves one view by view_id into its complete membership: the view object, every member element (from included_elements), every member relationship (from included_relationships), the parent element, and optionally child sub-views declared by member elements. Resolves ids into full canonical objects instead of returning raw id lists. Optional includeEaGeometry (default false) additionally returns the EA diagram geometry of the resolved view.',
+    description: 'read-only query that resolves one view by view_id into its complete membership: the view object, every member element (from included_elements), every member relationship (from included_relationships), the parent element, and optionally child sub-views declared by member elements. Resolves ids into full canonical objects instead of returning raw id lists. Optional includeEaGeometry (default false) additionally returns the EA diagram geometry of the resolved view: element boxes plus each connector ROUTE (t_diagramlinks.Path as `path` + parsed `points`, with the non-route Geometry override kept separately under `geometry`).',
     inputSchema: viewContextInputSchema(),
   },
   {
@@ -475,7 +475,7 @@ function viewContextInputSchema() {
       view_id: { type: 'string', description: 'The id of the view to resolve.' },
       includeParentElement: { type: 'boolean', description: 'Default: true. Resolve the parent element referenced by the view.' },
       includeChildViews: { type: 'boolean', description: 'Default: false. Include child views declared by member elements through subdiagram_views.' },
-      includeEaGeometry: { type: 'boolean', description: 'Default: false (opt-in). When true, additionally resolve the diagram GEOMETRY (element boxes + connector line paths) for this view from the workspace EA model (.qea) and return it under a `geometry` field aligned by schema id with the resolved members. By default the EA model is never touched and no `geometry` field is returned; a missing EA model/diagram yields geometry.present=false, never an error.' },
+      includeEaGeometry: { type: 'boolean', description: 'Default: false (opt-in). When true, additionally resolve the diagram GEOMETRY (element boxes + connector line routes) for this view from the workspace EA model (.qea) and return it under a `geometry` field aligned by schema id with the resolved members. Each geometry relationship carries: `path` (the EA route from t_diagramlinks.Path, "" when EA auto-routes), `points` (the parsed [{x,y}] waypoints), `edge` (the EDGE route-style token or null) and `geometry` (the raw SX/SY/EX/EY override string, which contains NO waypoints). By default the EA model is never touched and no `geometry` field is returned; a missing EA model/diagram yields geometry.present=false, never an error.' },
     },
     additionalProperties: false,
   };
@@ -726,9 +726,11 @@ function buildIntentElementContext(context, args = {}) {
 // model unless the caller explicitly sets includeEaGeometry=true. The workspace's
 // EA model (.qea, the SQLite carrier this toolchain can read) may hold human-laid-out
 // diagram geometry for a view — element boxes (t_diagramobjects rects) and connector
-// line paths (t_diagramlinks). When present it is returned under `geometry`, aligned
+// routes (t_diagramlinks.Path). When present it is returned under `geometry`, aligned
 // by schema id with the resolved members, so an image-capable LLM can redraw the view
-// faithfully. Absent model/diagram → present:false (never an error).
+// faithfully. Each connector also carries the non-route t_diagramlinks.Geometry override
+// string under a distinct `geometry` key, so the route is never confused with the
+// SX/SY/EX/EY/EDGE tokens. Absent model/diagram → present:false (never an error).
 const EA_GEOMETRY_MODEL_EXTENSIONS = new Set(['.qea']);
 function findEaGeometryModelPath(workspaceRoot) {
   try {

package/package.json

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