@glossarist/concept-browser 0.7.29 → 0.7.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glossarist/concept-browser",
-  "version": "0.7.29",
+  "version": "0.7.31",
   "description": "Vue SPA for browsing Glossarist terminology datasets with cross-reference resolution, graph visualization, and multi-language support",
   "type": "module",
   "bin": {

package/scripts/generate-data.mjs

@@ -924,6 +924,9 @@ for (let i = 0; i < config.datasets.length; i++) {
       tags: ds.tags || reg?.tags || [],
       color: ds.color || DS_PALETTE[i % DS_PALETTE.length],
     },
+    datasetUri: ds.uri || reg?.urn || undefined,
+    uriBase: config.uriBase || undefined,
+    uriAliases: ds.uriAliases || reg?.urnAliases || undefined,
   });
 }
 writeJson(path.join(PUBLIC, 'datasets.json'), registry);

package/src/adapters/DatasetAdapter.ts

@@ -51,10 +51,10 @@ export class DatasetAdapter {
     this.baseUrl = baseUrl;
   }
 
-  setSummaryManifest(summary: DatasetSummary): void {
+  setSummaryManifest(summary: DatasetSummary, registry?: { datasetUri?: string; uriBase?: string; uriAliases?: string[] }): void {
     this.manifest = {
       id: this.registerId,
-      datasetUri: '',
+      datasetUri: registry?.datasetUri || '',
       title: summary.title,
       description: summary.description,
       owner: summary.owner,
@@ -64,7 +64,8 @@ export class DatasetAdapter {
       conceptUrlTemplate: '',
       indexUrl: '',
       contextUrl: '',
-      uriBase: '',
+      uriBase: registry?.uriBase || '',
+      uriAliases: registry?.uriAliases,
       status: '',
       schemaVersion: '',
       tags: summary.tags,

package/src/adapters/factory.ts

@@ -36,7 +36,7 @@ export class AdapterFactory {
       adapters.push(adapter);
 
       if (reg.summary) {
-        adapter.setSummaryManifest(reg.summary);
+        adapter.setSummaryManifest(reg.summary, reg);
       } else {
         needManifest.push(adapter);
       }

package/src/adapters/types.ts

@@ -113,6 +113,9 @@ export interface DatasetRegistry {
   id: string;
   manifestUrl: string;
   summary?: DatasetSummary;
+  datasetUri?: string;
+  uriBase?: string;
+  uriAliases?: string[];
 }
 
 // ── Graph types ────────────────────────────────────────────────────────────

package/src/components/ConceptDetail.vue

@@ -314,8 +314,18 @@ const conceptUriValue = computed(() =>
   conceptUri(props.concept, props.registerId, props.manifest.uriBase)
 );
 
-const outgoingEdges = computed(() => props.edges.filter(e => e.source === conceptUriValue.value && e.type !== 'domain' && e.type !== 'section'));
-const incomingEdges = computed(() => props.edges.filter(e => e.target === conceptUriValue.value && e.type !== 'domain' && e.type !== 'section'));
+const outgoingEdges = computed(() => dedupeEdges(props.edges.filter(e => e.source === conceptUriValue.value && e.type !== 'domain' && e.type !== 'section'), 'target'));
+const incomingEdges = computed(() => dedupeEdges(props.edges.filter(e => e.target === conceptUriValue.value && e.type !== 'domain' && e.type !== 'section'), 'source'));
+
+function dedupeEdges(edges: GraphEdge[], direction: 'source' | 'target'): GraphEdge[] {
+  const seen = new Set<string>();
+  return edges.filter(e => {
+    const key = `${e[direction]}\0${e.type}`;
+    if (seen.has(key)) return false;
+    seen.add(key);
+    return true;
+  });
+}
 
 function inverseEdgeType(type: string): string {
   return INVERSE_RELATIONSHIPS[type] || type;

package/src/graph/GraphEngine.ts

@@ -1,6 +1,11 @@
 import type { GraphNode, GraphEdge } from '../adapters/types';
 import { UriRouter } from '../adapters/UriRouter';
 
+function hasDesignations(node: GraphNode): boolean {
+  const d = node.designations;
+  return d != null && typeof d === 'object' && Object.keys(d).length > 0;
+}
+
 /**
  * Directed multigraph engine for concept relationships.
  * Supports cross-register edges with stub nodes for unresolved targets.
@@ -18,6 +23,8 @@ export class GraphEngine {
       this.nodes.set(node.uri, node);
     } else if (node.loaded && !existing.loaded) {
       this.nodes.set(node.uri, node);
+    } else if (!existing.loaded && hasDesignations(node) && !hasDesignations(existing)) {
+      this.nodes.set(node.uri, node);
     }
   }
 

package/src/stores/vocabulary.ts

@@ -6,6 +6,7 @@ import type { Manifest, SearchHit, GraphEdge } from '../adapters/types';
 import type { Concept } from 'glossarist';
 import { conceptUri } from '../adapters/model-bridge';
 import { GraphEngine } from '../graph';
+import { UriRouter } from '../adapters/UriRouter';
 import { deduplicateSearchHits } from '../utils/search';
 
 export const useVocabularyStore = defineStore('vocabulary', () => {
@@ -54,12 +55,14 @@ export const useVocabularyStore = defineStore('vocabulary', () => {
     error.value = null;
     try {
       const adapters = await factory.discoverDatasets(`${import.meta.env.BASE_URL}datasets.json`);
+      const newMap = new Map(datasets.value);
       for (const adapter of adapters) {
-        datasets.value.set(adapter.registerId, adapter);
+        newMap.set(adapter.registerId, adapter);
         if (adapter.manifest) {
           manifests.value.set(adapter.registerId, adapter.manifest);
         }
       }
+      datasets.value = newMap;
       initialized.value = true;
     } catch (e: unknown) {
       error.value = `Failed to discover datasets: ${e instanceof Error ? e.message : String(e)}`;
@@ -72,7 +75,9 @@ export const useVocabularyStore = defineStore('vocabulary', () => {
     error.value = null;
     try {
       const adapter = await factory.loadDataset(registerId);
-      datasets.value.set(registerId, adapter);
+      const newMap = new Map(datasets.value);
+      newMap.set(registerId, adapter);
+      datasets.value = newMap;
       if (adapter.manifest) {
         manifests.value.set(registerId, adapter.manifest);
       }
@@ -136,29 +141,76 @@ export const useVocabularyStore = defineStore('vocabulary', () => {
     touchGraph();
   }
 
-  async function loadEdges(adapter: DatasetAdapter) {
+  async function loadEdges(adapter: DatasetAdapter): Promise<GraphEdge[]> {
     try {
-      const [edges, domainNodes] = await Promise.all([
+      const [edges, domainNodes, graphNodes] = await Promise.all([
         adapter.loadEdgeIndex(),
         adapter.loadDomainNodes(),
+        adapter.loadGraphNodes(),
       ]);
       const engine = graph.value;
       for (const dn of domainNodes) {
         engine.addNode(dn);
       }
+      if (graphNodes.uriPrefix) {
+        for (const [id, designations, status] of graphNodes.nodes) {
+          engine.addNode({
+            uri: graphNodes.uriPrefix + id,
+            register: adapter.registerId,
+            conceptId: id,
+            designations: designations || {},
+            status: status || 'unknown',
+            loaded: false,
+          });
+        }
+      }
       for (const edge of edges) {
         engine.addEdge(edge);
       }
       edgeStatus.value[adapter.registerId] = { loaded: true, count: edges.length };
+      return edges;
     } catch {
       edgeStatus.value[adapter.registerId] = { loaded: false, count: 0 };
+      return [];
     }
   }
 
   async function ensureEdgesForDataset(registerId: string) {
     const adapter = datasets.value.get(registerId);
+    let loadedEdges: GraphEdge[] = [];
     if (adapter && !edgeStatus.value[registerId]?.loaded) {
-      await loadEdges(adapter);
+      loadedEdges = await loadEdges(adapter);
+    }
+
+    // Load graph nodes for any target datasets referenced by this dataset's edges
+    if (adapter && loadedEdges.length > 0) {
+      const targetRegisters = new Set<string>();
+      for (const edge of loadedEdges) {
+        const parsed = UriRouter.parseUri(edge.target);
+        if (parsed?.registerId && parsed.registerId !== registerId) {
+          targetRegisters.add(parsed.registerId);
+        }
+      }
+      await Promise.all([...targetRegisters].map(async (targetId) => {
+        const targetAdapter = datasets.value.get(targetId);
+        if (!targetAdapter) return;
+        try {
+          const gn = await targetAdapter.loadGraphNodes();
+          if (gn.uriPrefix) {
+            const engine = graph.value;
+            for (const [id, designations, status] of gn.nodes) {
+              engine.addNode({
+                uri: gn.uriPrefix + id,
+                register: targetId,
+                conceptId: id,
+                designations: designations || {},
+                status: status || 'unknown',
+                loaded: false,
+              });
+            }
+          }
+        } catch { /* non-critical */ }
+      }));
     }
 
     const index = await factory.loadCrossRefIndex();