@glossarist/concept-browser 0.7.87 → 0.7.88

package/package.json

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

package/src/__tests__/uri-router.test.ts

@@ -103,4 +103,33 @@ describe('UriRouter', () => {
       expect(UriRouter.parseUri('https://example.com/other')).toBeNull();
     });
   });
+
+  describe('SSOT: UriRouter.buildConceptUri is the single concept-URI constructor', () => {
+    // Per AUDIT-2026-07-29 P1-1: ConceptIdentity, the conceptUri()
+    // bridge, and UriRouter.buildConceptUri must all funnel through
+    // UriRouter.buildConceptUri. No inline template literals for
+    // `${uriBase}/${registerId}/concept/${id}` outside UriRouter.
+
+    it('UriRouter.buildConceptUri constructs the canonical URI', () => {
+      expect(UriRouter.buildConceptUri('https://x.org', 'vim', '1.3'))
+        .toBe('https://x.org/vim/concept/1.3');
+    });
+
+    it('UriRouter.buildDomainUri constructs the canonical domain URI', () => {
+      expect(UriRouter.buildDomainUri('https://x.org', 'vim', 'physics'))
+        .toBe('https://x.org/vim/domain/physics');
+    });
+
+    it('ConceptIdentity.uri delegates to UriRouter.buildConceptUri', async () => {
+      const { ConceptIdentity } = await import('../adapters/concept-identity');
+      const id = new ConceptIdentity('1.3', 'vim', 'https://x.org');
+      expect(id.uri).toBe(UriRouter.buildConceptUri('https://x.org', 'vim', '1.3'));
+    });
+
+    it('ConceptIdentity.domainUri delegates to UriRouter.buildDomainUri', async () => {
+      const { ConceptIdentity } = await import('../adapters/concept-identity');
+      const id = new ConceptIdentity('1.3', 'vim', 'https://x.org');
+      expect(id.domainUri('physics')).toBe(UriRouter.buildDomainUri('https://x.org', 'vim', 'physics'));
+    });
+  });
 });

package/src/adapters/GraphDataSource.ts

@@ -137,7 +137,7 @@ export class GraphDataSource {
       if (lc?.domain) {
         edges.push({
           source: sourceUri,
-          target: `${this.uriBase}/${this.registerId}/domain/${slugify(lc.domain)}`,
+          target: UriRouter.buildDomainUri(this.uriBase, this.registerId, slugify(lc.domain)),
           type: 'domain',
           label: lc.domain,
           register: this.registerId,

package/src/adapters/UriRouter.ts

@@ -1,5 +1,4 @@
 import type { Manifest } from './types';
-import { ConceptIdentity } from './concept-identity';
 
 // ── URI pattern matching ────────────────────────────────────────────────────
 
@@ -106,11 +105,16 @@ export class UriRouter {
     return `${uriBase}/${registerId}/concept/${conceptId}`;
   }
 
-  /** Construct a canonical URI for a concept. */
+  /** Construct a canonical domain URI from components. */
+  static buildDomainUri(uriBase: string, registerId: string, domainSlug: string): string {
+    return `${uriBase}/${registerId}/domain/${domainSlug}`;
+  }
+
+  /** Construct a canonical URI for a concept from its register-id + concept-id. */
   buildUri(registerId: string, conceptId: string): string {
     const info = this.registerMap.get(registerId);
     const uriBase = info?.uriBase ?? '';
-    return new ConceptIdentity(conceptId, registerId, uriBase).uri;
+    return UriRouter.buildConceptUri(uriBase, registerId, conceptId);
   }
 
   getRegisteredIds(): string[] {

package/src/adapters/concept-identity.ts

@@ -1,4 +1,5 @@
 import { InvalidConceptIdentityError, InvalidConceptUriError } from '../errors';
+import { UriRouter } from './UriRouter';
 
 export interface ConceptIdentityParts {
   readonly localId: string;
@@ -24,7 +25,10 @@ export class ConceptIdentity implements ConceptIdentityParts {
         { localId, registerId, uriBase },
       );
     }
-    this._uri = `${uriBase}/${registerId}/concept/${localId}`;
+    // Single SSOT for concept URI construction: UriRouter.buildConceptUri.
+    // ConceptIdentity, the conceptUri() bridge, and UriRouter.buildConceptUri
+    // all funnel here — no inline template literals scattered across files.
+    this._uri = UriRouter.buildConceptUri(uriBase, registerId, localId);
     this._slug = localId;
     this._path = `${registerId}/concepts/${localId}`;
   }
@@ -52,7 +56,7 @@ export class ConceptIdentity implements ConceptIdentityParts {
   }
 
   domainUri(domainSlug: string): string {
-    return `${this.uriBase}/${this.registerId}/domain/${domainSlug}`;
+    return UriRouter.buildDomainUri(this.uriBase, this.registerId, domainSlug);
   }
 
   static fromUri(uri: string): ConceptIdentity {

package/src/adapters/non-verbal/glossarist-augment.d.ts

@@ -224,6 +224,12 @@ declare module 'glossarist' {
 // Re-declare the same exports under the 'glossarist/models' subpath so
 // deep-import callers can use them. Upstream's d.ts is stale on this
 // subpath too.
+//
+// The duplication between the 'glossarist' block above and this block
+// is forced by upstream's broken d.ts (PR glossarist/glossarist-js#31).
+// We cannot `export import` from a local namespace because TypeScript
+// requires `declare module` blocks to be self-contained. When upstream
+// ships proper d.ts, DELETE this entire file.
 declare module 'glossarist/models' {
   export type Completeness = 'complete' | 'partial';
   export const COMPLETENESS: { readonly COMPLETE: 'complete'; readonly PARTIAL: 'partial' };
@@ -252,23 +258,8 @@ declare module 'glossarist/models' {
   export function isValidMultiplicity(value: unknown): value is Multiplicity;
   export function multiplicityFromPair(presence: PartitivePresence, count: PartitiveCount): Multiplicity;
   export function pairFromMultiplicity(name: Multiplicity): { presence: PartitivePresence; count: PartitiveCount };
-
-  export class TypeSharedPlurality extends GlossaristModel {
-    constructor(data?: {
-      isShared?: boolean;
-      is_shared?: boolean;
-      isUncertain?: boolean;
-      is_uncertain?: boolean;
-      sharedType?: ConceptRef;
-      shared_type?: ConceptRef;
-    });
-    readonly isShared: boolean;
-    readonly isUncertain: boolean;
-    readonly sharedType: ConceptRef | null;
-    hasSharedType(): boolean;
-    toJSON(): { is_shared: boolean; is_uncertain?: boolean; shared_type?: ReturnType<ConceptRef['toJSON']> };
-    static fromJSON(data: Record<string, unknown>): TypeSharedPlurality;
-  }
+  // (TypeSharedPlurality removed — superseded by presence/count on
+  // PartitiveMember per MECE refactor. Was dead code.)
 
   // ── PartitiveRelation (glossarist 0.4.26 — MECE presence × count) ─