@glossarist/concept-browser 0.7.121 → 0.7.122

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glossarist/concept-browser",
-  "version": "0.7.121",
+  "version": "0.7.122",
   "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.js

@@ -1211,6 +1211,7 @@ function extractInlineRefs(localizedData, refMaps2, conceptSources = []) {
   }
   for (const m of fullText.matchAll(/\{\{([^{}]+?)\}\}/g)) {
     const body = m[1];
+    if (/^(bib|link|image):/i.test(body)) continue;
     const parsed = parseMention(body);
     let ref = null;
     if (parsed.kind === "cite-ref") {

package/scripts/generate-data.ts

@@ -375,9 +375,22 @@ function extractInlineRefs(localizedData, refMaps, conceptSources = []) {
     if (pattern) { refs.push(pattern); }
   }
 
-  // Double-brace mentions: dispatched by parseMention kind
+  // Double-brace mentions: dispatched by parseMention kind.
+  //
+  // DATA/DEPLOYMENT BOUNDARY: only cite-ref, numeric, urn-ref, and
+  // designation mentions produce concept cross-references (gl:references).
+  // Other mention kinds (bib:, link:, image:) are deployment-specific
+  // renderings — bibliography entries, external URLs, inline images —
+  // NOT concept pages. They must NOT appear in gl:references or be
+  // turned into concept URIs. The deployment's renderer handles them
+  // at runtime (bibResolver, linkResolver, imageResolver).
   for (const m of fullText.matchAll(/\{\{([^{}]+?)\}\}/g)) {
     const body = m[1];
+
+    // Pre-parse deployment-specific kinds that parseMention doesn't know.
+    // These are NEVER concept cross-references — skip them entirely.
+    if (/^(bib|link|image):/i.test(body)) continue;
+
     const parsed = parseMention(body);
 
     let ref = null;
@@ -399,6 +412,8 @@ function extractInlineRefs(localizedData, refMaps, conceptSources = []) {
       // {{designation,render term}} — same-dataset designation reference
       ref = handleDesignation(parsed, refMaps);
     } else {
+      // Unresolved mentions: only include if they look like concept refs
+      // (numeric IDs or known patterns). Never include bib:/link:/image:.
       ref = handleUnresolved(body, refMaps);
     }
     if (ref) refs.push(ref);

package/src/__tests__/scripts/bib-boundary.test.ts

@@ -0,0 +1,52 @@
+import { describe, it, expect } from 'vitest';
+
+/**
+ * Regression test: extractInlineRefs must NOT turn bib:/link:/image: mentions
+ * into concept cross-references. These are deployment-specific renderings
+ * (bibliography entries, external URLs, inline images), not concept pages.
+ *
+ * The data/deployment boundary requires that dataset authors can write
+ * {{bib:...}} without knowing whether IEV or any other dataset is
+ * co-deployed. The deployment's renderer handles bib/link/image at
+ * runtime. concept-browser MUST NOT create concept URIs for them.
+ */
+
+// Simulate the extractInlineRefs double-brace loop logic from generate-data.ts
+function shouldSkipMention(body: string): boolean {
+  return /^(bib|link|image):/i.test(body);
+}
+
+describe('data/deployment boundary — bib:/link:/image: mentions never become concept URIs', () => {
+  it('skips bib: mentions', () => {
+    expect(shouldSkipMention('bib:702-02-07, IEV 702-02-07')).toBe(true);
+    expect(shouldSkipMention('bib:ref_1')).toBe(true);
+  });
+
+  it('skips link: mentions', () => {
+    expect(shouldSkipMention('link:https://example.com')).toBe(true);
+    expect(shouldSkipMention('link:https://example.com, click here')).toBe(true);
+  });
+
+  it('skips image: mentions', () => {
+    expect(shouldSkipMention('image:diagram.png')).toBe(true);
+    expect(shouldSkipMention('image:diagram.png, alt text')).toBe(true);
+  });
+
+  it('does NOT skip cite: mentions (concept cross-references)', () => {
+    expect(shouldSkipMention('cite:iso704')).toBe(false);
+    expect(shouldSkipMention('cite:iso704, ISO 704')).toBe(false);
+  });
+
+  it('does NOT skip numeric mentions (e.g. 17-21-004)', () => {
+    expect(shouldSkipMention('17-21-004')).toBe(false);
+    expect(shouldSkipMention('112-01-10')).toBe(false);
+  });
+
+  it('does NOT skip urn: mentions', () => {
+    expect(shouldSkipMention('urn:iso:std:iso:704')).toBe(false);
+  });
+
+  it('does NOT skip designation mentions', () => {
+    expect(shouldSkipMention('measurement unit')).toBe(false);
+  });
+});