archgraph-argo 0.16.0 → 0.16.2

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

@@ -740,24 +740,66 @@ function syncGraphToQea(graph, qeaPath, opts) {
     stages.members = nowMs();
 
     // --- deletion reconcile (opt-in) ---------------------------------------
+    // Elements are anchored by t_object.Alias; relationships have NO Alias column — their
+    // canonical id lives in t_connectortag(schema_id). Using row.Alias for relationships made
+    // the candidate check always false, so deleted relationships were never projected out.
     const keepAliases = new Set();
     for (const e of graph.elements || []) { if (e && e.id !== undefined) { keepAliases.add(String(e.id)); } }
     for (const rel of graph.relationships || []) { if (rel && rel.id !== undefined) { keepAliases.add(String(rel.id)); } }
+    // connector id -> canonical relationship id (schema_id tag)
+    const relSchemaById = new Map();
+    try {
+      const relTagRows = db.prepare("SELECT ElementID, VALUE FROM t_connectortag WHERE Property='schema_id'").all();
+      for (const t of relTagRows) { relSchemaById.set(Number(t.ElementID), String(t.VALUE)); }
+    } catch { /* table may be absent on a hand-drawn model */ }
     const candidates = [];
     for (const row of existingElems) {
       if (row.Alias && !keepAliases.has(String(row.Alias))) { candidates.push({ type: 'element', id: Number(row.Object_ID), alias: row.Alias }); }
     }
     for (const row of existingRels) {
-      if (row.Alias && !keepAliases.has(String(row.Alias))) { candidates.push({ type: 'relationship', id: Number(row.Connector_ID), alias: row.Alias }); }
+      const relId = relSchemaById.get(Number(row.Connector_ID));
+      if (relId !== undefined && !keepAliases.has(relId)) { candidates.push({ type: 'relationship', id: Number(row.Connector_ID), alias: relId }); }
+    }
+    // views: a projected diagram is owned by the projector. Identify its canonical view id via
+    // the schema_view_id StyleEx token, or (when EA rewrote StyleEx and dropped the token) by
+    // matching the deterministic diagram ea_guid `diag:<viewId>` against the view-id catalog
+    // that kg_sync_meta retains. A view id no longer in canonical -> delete the diagram.
+    const canonicalViewIds = new Set();
+    for (const v of graph.views || []) { if (v && v.view_id !== undefined && v.view_id !== null) { canonicalViewIds.add(String(v.view_id)); } }
+    const guidToView = new Map();
+    try {
+      const viewMetaRows = db.prepare("SELECT key FROM kg_sync_meta WHERE kind='view'").all();
+      for (const r of viewMetaRows) { guidToView.set(deterministicGuid('diag:' + String(r.key)), String(r.key)); }
+    } catch { /* meta table may be absent */ }
+    for (const d of existingDiags) {
+      let vid = parseStyleToken(d.StyleEx, 'schema_view_id');
+      if (!vid && d.ea_guid) { vid = guidToView.get(String(d.ea_guid)) || ''; }
+      if (vid && !canonicalViewIds.has(String(vid))) { candidates.push({ type: 'diagram', id: Number(d.Diagram_ID), alias: String(vid) }); }
     }
     stats.deleteCandidates = candidates.length;
     if (candidates.length > 0 && o.allowDelete && !o.dryRun) {
       for (const c of candidates) {
         if (c.type === 'relationship') {
+          // SQLite has no FK cascade: clear the connector's tag + diagram-link rows too.
+          db.prepare('DELETE FROM t_connectortag WHERE ElementID=?').run(c.id);
+          db.prepare('DELETE FROM t_diagramlinks WHERE ConnectorID=?').run(c.id);
           db.prepare('DELETE FROM t_connector WHERE Connector_ID=?').run(c.id);
+        } else if (c.type === 'diagram') {
+          db.prepare('DELETE FROM t_diagramobjects WHERE Diagram_ID=?').run(c.id);
+          db.prepare('DELETE FROM t_diagramlinks WHERE DiagramID=?').run(c.id);
+          db.prepare('DELETE FROM t_diagram WHERE Diagram_ID=?').run(c.id);
         } else {
+          // remove connectors attached to this element (start or end) and their children
+          const attached = db.prepare('SELECT Connector_ID FROM t_connector WHERE Start_Object_ID=? OR End_Object_ID=?').all(c.id, c.id);
+          for (const a of attached) {
+            db.prepare('DELETE FROM t_connectortag WHERE ElementID=?').run(Number(a.Connector_ID));
+            db.prepare('DELETE FROM t_diagramlinks WHERE ConnectorID=?').run(Number(a.Connector_ID));
+            db.prepare('DELETE FROM t_connector WHERE Connector_ID=?').run(Number(a.Connector_ID));
+          }
           db.prepare('DELETE FROM t_diagramobjects WHERE Object_ID=?').run(c.id);
           db.prepare('DELETE FROM t_objectproperties WHERE Object_ID=?').run(c.id);
+          db.prepare('DELETE FROM t_attribute WHERE Object_ID=?').run(c.id);
+          db.prepare('DELETE FROM t_objecttests WHERE Object_ID=?').run(c.id);
           db.prepare('DELETE FROM t_object WHERE Object_ID=?').run(c.id);
         }
         stats.deleted++;

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

@@ -1948,7 +1948,11 @@ function runQeaProjection(target) {
       return;
     }
     const snapshotDir = path.join(target.workspaceRoot, '.argo', 'temp', 'qea-backups');
-    const args = [script, '--mode', 'sync', '--graph', target.graphPath, '--qea', target.qeaPath, '--snapshot-dir', snapshotDir];
+    // -y enables the projection-owned delete reconcile: objects that carry a schema anchor
+    // (t_object.Alias / t_connectortag schema_id) but are no longer in canonical are removed
+    // from the .qea, so a graph-side deletion actually disappears from EA on the next
+    // projection. Human-drawn (un-anchored) content is never a delete candidate.
+    const args = [script, '--mode', 'sync', '--graph', target.graphPath, '--qea', target.qeaPath, '--snapshot-dir', snapshotDir, '-y'];
     const started = Date.now();
     let stderr = '';
     let child;
@@ -2313,7 +2317,9 @@ async function callTool(name, args = {}, dependencies = undefined) {
         }
       }
 
-      const journey = await resolveSemanticOperatorJourney(dependencies);
+      const journey = await resolveSemanticOperatorJourney(dependencies, {
+        repositoryRoot: context.workspaceRoot,
+      });
       return applySemanticResponseProfile(await journey.query(query), query, contractOptions);
     }
 
@@ -2429,7 +2435,9 @@ async function memorySearchTool(args = {}, dependencies = undefined) {
   const context = await loadContext(args);
   let retrieved;
   try {
-    const journey = await resolveSemanticOperatorJourney(dependencies);
+    const journey = await resolveSemanticOperatorJourney(dependencies, {
+      repositoryRoot: context.workspaceRoot,
+    });
     retrieved = await journey.query({ purpose: 'general', intent: query });
   } catch (error) {
     return {
@@ -2581,10 +2589,21 @@ function queryNeo4jGraphSchemaResult(architecturePath, workspaceRoot) {
   });
 }
 
-async function resolveSemanticOperatorJourney(dependencies) {
-  return dependencies && dependencies.semanticOperatorJourney
-    ? dependencies.semanticOperatorJourney
-    : createDefaultProductionSemanticOperatorJourney();
+async function resolveSemanticOperatorJourney(dependencies, options = {}) {
+  if (dependencies && dependencies.semanticOperatorJourney) {
+    return dependencies.semanticOperatorJourney;
+  }
+  // Thread the caller's already-resolved workspace root into the journey: the
+  // no-arg resolveWorkspaceRoot() fallback is process.cwd(), i.e. whatever
+  // directory the host launched this server from (a global ~/.argo installation
+  // started by the host process resolves to that host's cwd, e.g.
+  // C:\Windows\System32), never the workspace the caller asked for.
+  const repositoryRoot = options && typeof options.repositoryRoot === 'string'
+    ? options.repositoryRoot
+    : '';
+  return createDefaultProductionSemanticOperatorJourney(
+    repositoryRoot ? { repositoryRoot } : {},
+  );
 }
 
 async function executeSemanticSystemArchitectureQuery(args, dependencies) {

package/package.json

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