@qti-editor/core 1.4.0 → 1.4.1

package/dist/composer/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/composer/index.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAC;AAErE,MAAM,WAAW,mBAAmB;IAClC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,QAAQ,CAAC;IACpB,KAAK,CAAC,EAAE,KAAK,CAAC;QAAE,UAAU,CAAC,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACxD;AAED,MAAM,WAAW,mBAAmB;IAClC,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,QAAQ,GAAG,UAAU,GAAG,SAAS,CAAC;IAC/C,QAAQ,EAAE,YAAY,GAAG,cAAc,GAAG,OAAO,GAAG,QAAQ,CAAC;IAC7D,eAAe,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC;IACpC,aAAa,CAAC,EAAE;QACd,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,EAAE,KAAK,CAAC;YACb,MAAM,EAAE,MAAM,CAAC;YACf,WAAW,EAAE,MAAM,CAAC;YACpB,aAAa,EAAE,OAAO,CAAC;SACxB,CAAC,CAAC;KACJ,CAAC;IACF,sBAAsB,CAAC,EAAE,sBAAsB,CAAC;IAChD,WAAW,CAAC,EAAE;QACZ,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,EAAE,KAAK,CAAC;YACb,KAAK,EAAE,QAAQ,GAAG,MAAM,CAAC;YACzB,MAAM,EAAE,MAAM,CAAC;YACf,WAAW,EAAE,MAAM,CAAC;SACrB,CAAC,CAAC;KACJ,CAAC;IACF,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAiED,wBAAgB,2BAA2B,CAAC,YAAY,CAAC,EAAE,OAAO,GAAG,IAAI,GAAG,mBAAmB,EAAE,CAOhG;AAkFD,wBAAgB,sBAAsB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAyHhF;AAED;;;;;;;;GAQG;AACH,wBAAgB,+BAA+B,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAmCzF;AAED;;;GAGG;AACH,wBAAgB,4BAA4B,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAUtF;AAED;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAK5E;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,KAAK,CAAC;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CAAC,CAiChI;AAoKD,wBAAgB,SAAS,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CA4B7C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/composer/index.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAC;AAErE,MAAM,WAAW,mBAAmB;IAClC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,QAAQ,CAAC;IACpB,KAAK,CAAC,EAAE,KAAK,CAAC;QAAE,UAAU,CAAC,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACxD;AAED,MAAM,WAAW,mBAAmB;IAClC,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,QAAQ,GAAG,UAAU,GAAG,SAAS,CAAC;IAC/C,QAAQ,EAAE,YAAY,GAAG,cAAc,GAAG,OAAO,GAAG,QAAQ,CAAC;IAC7D,eAAe,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC;IACpC,aAAa,CAAC,EAAE;QACd,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,EAAE,KAAK,CAAC;YACb,MAAM,EAAE,MAAM,CAAC;YACf,WAAW,EAAE,MAAM,CAAC;YACpB,aAAa,EAAE,OAAO,CAAC;SACxB,CAAC,CAAC;KACJ,CAAC;IACF,sBAAsB,CAAC,EAAE,sBAAsB,CAAC;IAChD,WAAW,CAAC,EAAE;QACZ,YAAY,EAAE,MAAM,CAAC;QACrB,OAAO,EAAE,KAAK,CAAC;YACb,KAAK,EAAE,QAAQ,GAAG,MAAM,CAAC;YACzB,MAAM,EAAE,MAAM,CAAC;YACf,WAAW,EAAE,MAAM,CAAC;SACrB,CAAC,CAAC;KACJ,CAAC;IACF,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAkGD,wBAAgB,2BAA2B,CAAC,YAAY,CAAC,EAAE,OAAO,GAAG,IAAI,GAAG,mBAAmB,EAAE,CAOhG;AAkFD,wBAAgB,sBAAsB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAmIhF;AAED;;;;;;;;GAQG;AACH,wBAAgB,+BAA+B,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAqDzF;AAED;;;GAGG;AACH,wBAAgB,4BAA4B,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAUtF;AAED;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAK5E;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,KAAK,CAAC;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CAAC,CA0DhI;AAoKD,wBAAgB,SAAS,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CA4B7C"}

package/dist/composer/index.js

@@ -24,6 +24,28 @@ const TEXT_ENTRY_DATA_ATTRIBUTE_MAPPINGS = [
 const SELECT_POINT_DATA_ATTRIBUTE_MAPPINGS = [
     { source: 'area-mappings', target: 'data-area-mappings' },
 ];
+function sanitizeIdentifier(value, fallback) {
+    const sanitized = value?.trim().replace(/[^A-Za-z0-9_.-]+/g, '-').replace(/^-+|-+$/g, '');
+    return sanitized || fallback;
+}
+function hashIdentifierSeed(value) {
+    let hash = 0x811c9dc5;
+    for (let i = 0; i < value.length; i += 1) {
+        hash ^= value.charCodeAt(i);
+        hash = Math.imul(hash, 0x01000193);
+    }
+    return (hash >>> 0).toString(36);
+}
+function createAutoIdentifier(options) {
+    const titleBase = sanitizeIdentifier(options.title, '');
+    const base = titleBase || sanitizeIdentifier(options.baseIdentifier, 'item');
+    const serializedBody = options.body == null
+        ? ''
+        : new XMLSerializer().serializeToString(options.body instanceof Document ? options.body.documentElement : options.body);
+    const seed = [base, options.itemNumber ?? 1, serializedBody].join('|');
+    const suffix = hashIdentifierSeed(seed).slice(0, 8);
+    return `${base}-${suffix}`;
+}
 function parseCorrectResponseValues(declaration) {
     const value = declaration.correctResponse;
     if (value == null)
@@ -140,7 +162,11 @@ export function buildAssessmentItemXml(itemContext) {
     const root = xmlDoc.documentElement;
     root.setAttribute('xmlns', QTI_NS);
     root.setAttributeNS(XSI_NS, 'xsi:schemaLocation', SCHEMA_LOCATION);
-    root.setAttribute('identifier', itemContext.identifier?.trim() || 'item-1');
+    root.setAttribute('identifier', sanitizeIdentifier(itemContext.identifier, createAutoIdentifier({
+        title: itemContext.title,
+        baseIdentifier: itemContext.identifier,
+        body: itemContext.itemBody,
+    })));
     root.setAttribute('title', itemContext.title?.trim() || 'Untitled Item');
     root.setAttribute('adaptive', 'false');
     root.setAttribute('time-dependent', 'false');
@@ -251,17 +277,35 @@ export function buildMultipleAssessmentItemsXml(itemContext) {
         return buildAssessmentItemXml(itemContext);
     }
     // Build multiple items
-    const baseIdentifier = itemContext.identifier?.trim() || 'item';
+    const baseIdentifier = sanitizeIdentifier(itemContext.identifier, 'item');
     const baseTitle = itemContext.title?.trim() || 'Untitled Item';
     const lang = itemContext.lang?.trim() || 'en';
+    const usedIdentifiers = new Set();
     const itemXmls = fragments.map((fragmentBody, index) => {
         const itemNumber = index + 1;
         const perItem = itemContext.items?.[index];
         const fragmentDoc = document.implementation.createDocument(QTI_NS, 'qti-item-body', null);
         const importedFragment = fragmentDoc.importNode(fragmentBody, true);
         fragmentDoc.replaceChild(importedFragment, fragmentDoc.documentElement);
+        let identifier = perItem?.identifier?.trim();
+        if (!identifier) {
+            identifier = createAutoIdentifier({
+                title: perItem?.title || `${baseTitle} ${itemNumber}`,
+                baseIdentifier: `${baseIdentifier}-${itemNumber}`,
+                itemNumber,
+                body: fragmentDoc,
+            });
+        }
+        identifier = sanitizeIdentifier(identifier, `${baseIdentifier}-${itemNumber}`);
+        let uniqueIdentifier = identifier;
+        let duplicateIndex = 2;
+        while (usedIdentifiers.has(uniqueIdentifier)) {
+            uniqueIdentifier = `${identifier}-${duplicateIndex}`;
+            duplicateIndex += 1;
+        }
+        usedIdentifiers.add(uniqueIdentifier);
         const fragmentContext = {
-            identifier: perItem?.identifier?.trim() || `${baseIdentifier}-${itemNumber}`,
+            identifier: uniqueIdentifier,
             title: perItem?.title?.trim() || `${baseTitle} ${itemNumber}`,
             lang,
             itemBody: fragmentDoc,
@@ -308,26 +352,48 @@ export function getItemFragmentXmls(itemContext) {
     // If only one fragment (no dividers), return single item
     if (fragments.length <= 1) {
         const perItem = itemContext.items?.[0];
-        const identifier = perItem?.identifier?.trim() || itemContext.identifier?.trim() || 'item-1';
+        const identifier = sanitizeIdentifier(perItem?.identifier || itemContext.identifier, createAutoIdentifier({
+            title: perItem?.title || itemContext.title,
+            baseIdentifier: itemContext.identifier,
+            itemNumber: 1,
+            body: itemContext.itemBody,
+        }));
         const title = perItem?.title?.trim() || itemContext.title?.trim() || 'Untitled Item';
         const xml = buildAssessmentItemXml({ ...itemContext, identifier, title });
         return [{ identifier, title, xml }];
     }
-    const baseIdentifier = itemContext.identifier?.trim() || 'item';
+    const baseIdentifier = sanitizeIdentifier(itemContext.identifier, 'item');
     const baseTitle = itemContext.title?.trim() || 'Untitled Item';
     const lang = itemContext.lang?.trim() || 'en';
+    const usedIdentifiers = new Set();
     return fragments.map((fragmentBody, index) => {
         const itemNumber = index + 1;
         const perItem = itemContext.items?.[index];
-        const identifier = perItem?.identifier?.trim() || `${baseIdentifier}-${itemNumber}`;
         const title = perItem?.title?.trim() || `${baseTitle} ${itemNumber}`;
         const fragmentDoc = document.implementation.createDocument(QTI_NS, 'qti-item-body', null);
         const importedFragment = fragmentDoc.importNode(fragmentBody, true);
         fragmentDoc.replaceChild(importedFragment, fragmentDoc.documentElement);
+        let identifier = perItem?.identifier?.trim();
+        if (!identifier) {
+            identifier = createAutoIdentifier({
+                title,
+                baseIdentifier: `${baseIdentifier}-${itemNumber}`,
+                itemNumber,
+                body: fragmentDoc,
+            });
+        }
+        identifier = sanitizeIdentifier(identifier, `${baseIdentifier}-${itemNumber}`);
+        let uniqueIdentifier = identifier;
+        let duplicateIndex = 2;
+        while (usedIdentifiers.has(uniqueIdentifier)) {
+            uniqueIdentifier = `${identifier}-${duplicateIndex}`;
+            duplicateIndex += 1;
+        }
+        usedIdentifiers.add(uniqueIdentifier);
         return {
-            identifier,
+            identifier: uniqueIdentifier,
             title,
-            xml: buildAssessmentItemXml({ identifier, title, lang, itemBody: fragmentDoc }),
+            xml: buildAssessmentItemXml({ identifier: uniqueIdentifier, title, lang, itemBody: fragmentDoc }),
         };
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qti-editor/core",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "QTI semantics, composer registry, and XML export orchestration for QTI Editor",
   "repository": {
     "type": "git",
@@ -25,18 +25,18 @@
   },
   "dependencies": {
     "@qti-editor/interfaces": "1.2.1",
-    "@qti-editor/interaction-associate": "1.2.3",
-    "@qti-editor/interaction-choice": "1.2.2",
-    "@qti-editor/interaction-extended-text": "1.3.0",
-    "@qti-editor/interaction-gap-match": "1.1.1",
-    "@qti-editor/interaction-hottext": "1.2.3",
-    "@qti-editor/interaction-inline-choice": "1.3.0",
-    "@qti-editor/interaction-match": "1.2.1",
-    "@qti-editor/interaction-order": "0.6.1",
-    "@qti-editor/interaction-select-point": "1.2.1",
-    "@qti-editor/interaction-shared": "1.3.1",
-    "@qti-editor/interaction-text-entry": "1.2.1",
-    "@qti-editor/qti-item-divider": "1.1.2"
+    "@qti-editor/interaction-associate": "1.2.4",
+    "@qti-editor/interaction-choice": "1.2.3",
+    "@qti-editor/interaction-extended-text": "1.3.1",
+    "@qti-editor/interaction-gap-match": "1.1.2",
+    "@qti-editor/interaction-hottext": "1.2.4",
+    "@qti-editor/interaction-inline-choice": "1.3.1",
+    "@qti-editor/interaction-match": "1.2.2",
+    "@qti-editor/interaction-order": "0.6.2",
+    "@qti-editor/interaction-select-point": "1.2.2",
+    "@qti-editor/interaction-shared": "1.3.2",
+    "@qti-editor/interaction-text-entry": "1.2.2",
+    "@qti-editor/qti-item-divider": "1.1.3"
   },
   "devDependencies": {
     "@types/node": "^20.0.0",