@qti-editor/core 1.3.2 → 1.4.1

package/dist/composer/index.d.ts

@@ -10,6 +10,10 @@ export interface ComposerItemContext {
     lang?: string;
     title?: string;
     itemBody?: Document;
+    items?: Array<{
+        identifier?: string;
+        title?: string;
+    }>;
 }
 export interface ResponseDeclaration {
     identifier: string;

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;CACrB;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,CAqHhF;AAED;;;;;;;;GAQG;AACH,wBAAgB,+BAA+B,CAAC,WAAW,CAAC,EAAE,mBAAmB,GAAG,MAAM,CAkCzF;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,CAyChI;AAiMD,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');
@@ -153,7 +179,7 @@ export function buildAssessmentItemXml(itemContext) {
     const composedItemBody = sourceBodyRoot != null
         ? xmlDoc.importNode(sourceBodyRoot, true)
         : xmlDoc.createElementNS(QTI_NS, 'qti-item-body');
-    const { declarations, responseTemplate, maxScore } = composeAndNormalizeItemBody(composedItemBody, xmlDoc);
+    const { declarations, responseTemplate, maxScore, hasAutomatedProcessing } = composeAndNormalizeItemBody(composedItemBody, xmlDoc);
     declarations.forEach(declaration => {
         const responseDeclaration = xmlDoc.createElementNS(QTI_NS, 'qti-response-declaration');
         responseDeclaration.setAttribute('identifier', declaration.identifier);
@@ -218,17 +244,20 @@ export function buildAssessmentItemXml(itemContext) {
         root.appendChild(maxScoreOutcomeDeclaration);
     }
     root.appendChild(composedItemBody);
-    if (maxScore > 0) {
-        if (declarations.length === 1) {
+    if (maxScore > 0 && hasAutomatedProcessing) {
+        const single = declarations.length === 1 ? declarations[0] : null;
+        const canUseTemplate = single !== null &&
+            !(single.responseProcessingKind === 'match_correct' && (single.score ?? 1) !== 1);
+        if (canUseTemplate) {
             const responseProcessing = xmlDoc.createElementNS(QTI_NS, 'qti-response-processing');
             responseProcessing.setAttribute('template', responseTemplate);
             root.appendChild(responseProcessing);
         }
-        else if (declarations.length > 1) {
+        else {
             root.appendChild(buildMultiInteractionResponseProcessing(xmlDoc, declarations));
         }
     }
-    return new XMLSerializer().serializeToString(xmlDoc);
+    return new XMLSerializer().serializeToString(xmlDoc).replace(/\s+xmlns=""/g, '');
 }
 /**
  * Build multiple QTI assessment items from a single editor document.
@@ -248,17 +277,36 @@ 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: `${baseIdentifier}-${itemNumber}`,
-            title: `${baseTitle} ${itemNumber}`,
+            identifier: uniqueIdentifier,
+            title: perItem?.title?.trim() || `${baseTitle} ${itemNumber}`,
             lang,
             itemBody: fragmentDoc,
         };
@@ -303,33 +351,49 @@ export function getItemFragmentXmls(itemContext) {
     const fragments = splitItemBodyAtDividers(itemContext.itemBody);
     // If only one fragment (no dividers), return single item
     if (fragments.length <= 1) {
-        const xml = buildAssessmentItemXml(itemContext);
-        return [{
-                identifier: itemContext.identifier?.trim() || 'item-1',
-                title: itemContext.title?.trim() || 'Untitled Item',
-                xml,
-            }];
+        const perItem = itemContext.items?.[0];
+        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 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);
-        const identifier = `${baseIdentifier}-${itemNumber}`;
-        const title = `${baseTitle} ${itemNumber}`;
-        const fragmentContext = {
-            identifier,
-            title,
-            lang,
-            itemBody: fragmentDoc,
-        };
+        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(fragmentContext),
+            xml: buildAssessmentItemXml({ identifier: uniqueIdentifier, title, lang, itemBody: fragmentDoc }),
         };
     });
 }
@@ -368,14 +432,17 @@ function composeAndNormalizeItemBody(itemBody, xmlDoc) {
             if (composeResult.responseDeclaration) {
                 maxScore += composeResult.responseDeclaration.score ?? 1;
             }
-            if (composeResult.responseDeclaration && !seenIdentifiers.has(composeResult.responseDeclaration.identifier)) {
+            if (composeResult.responseDeclaration) {
+                const declaration = composeResult.responseDeclaration;
+                if (seenIdentifiers.has(declaration.identifier)) {
+                    const freshId = `RESPONSE_${crypto.randomUUID()}`;
+                    composeResult.normalizedElement.setAttribute('response-identifier', freshId);
+                    declaration.identifier = freshId;
+                }
                 const responseProcessingKind = composeResult
                     .responseProcessingKind;
-                declarations.push({
-                    ...composeResult.responseDeclaration,
-                    responseProcessingKind,
-                });
-                seenIdentifiers.add(composeResult.responseDeclaration.identifier);
+                declarations.push({ ...declaration, responseProcessingKind });
+                seenIdentifiers.add(declaration.identifier);
             }
             if (composeResult.responseProcessingTemplate && composeResult.responseDeclaration) {
                 templateCandidates.add(composeResult.responseProcessingTemplate);
@@ -392,16 +459,17 @@ function composeAndNormalizeItemBody(itemBody, xmlDoc) {
     itemBody.querySelectorAll('[score]').forEach(element => {
         element.removeAttribute('score');
     });
-    normalizeResponseIdentifiers(itemBody, declarations);
     if (declarations.length === 1 && templateCandidates.size === 1) {
-        return { declarations, responseTemplate: Array.from(templateCandidates)[0], maxScore };
+        return { declarations, responseTemplate: Array.from(templateCandidates)[0], maxScore, hasAutomatedProcessing: true };
     }
-    return { declarations, responseTemplate: MATCH_CORRECT_TEMPLATE, maxScore };
+    return { declarations, responseTemplate: MATCH_CORRECT_TEMPLATE, maxScore, hasAutomatedProcessing: templateCandidates.size > 0 };
 }
 function buildMultiInteractionResponseProcessing(xmlDoc, declarations) {
     const responseProcessing = xmlDoc.createElementNS(QTI_NS, 'qti-response-processing');
     declarations.forEach(declaration => {
-        const kind = declaration.responseProcessingKind ?? 'match_correct';
+        if (declaration.responseProcessingKind === undefined)
+            return;
+        const kind = declaration.responseProcessingKind;
         if (kind === 'match_correct') {
             responseProcessing.appendChild(createMatchCorrectContribution(xmlDoc, declaration.identifier, declaration.score ?? 1));
             return;
@@ -434,16 +502,12 @@ function createMatchCorrectContribution(xmlDoc, responseIdentifier, score = 1) {
 }
 function createMapResponseContribution(xmlDoc, responseIdentifier) {
     const mapResponse = xmlDoc.createElementNS(QTI_NS, 'qti-map-response');
-    const variable = xmlDoc.createElementNS(QTI_NS, 'qti-variable');
-    variable.setAttribute('identifier', responseIdentifier);
-    mapResponse.appendChild(variable);
+    mapResponse.setAttribute('identifier', responseIdentifier);
     return createScoreIncrement(xmlDoc, mapResponse);
 }
 function createMapResponsePointContribution(xmlDoc, responseIdentifier) {
     const mapResponsePoint = xmlDoc.createElementNS(QTI_NS, 'qti-map-response-point');
-    const variable = xmlDoc.createElementNS(QTI_NS, 'qti-variable');
-    variable.setAttribute('identifier', responseIdentifier);
-    mapResponsePoint.appendChild(variable);
+    mapResponsePoint.setAttribute('identifier', responseIdentifier);
     return createScoreIncrement(xmlDoc, mapResponsePoint);
 }
 function createScoreIncrement(xmlDoc, contribution) {
@@ -457,34 +521,6 @@ function createScoreIncrement(xmlDoc, contribution) {
     setOutcomeValue.appendChild(sum);
     return setOutcomeValue;
 }
-function normalizeResponseIdentifiers(itemBody, declarations) {
-    if (declarations.length === 0)
-        return;
-    const identifierMap = new Map();
-    if (declarations.length === 1) {
-        identifierMap.set(declarations[0].identifier, 'RESPONSE');
-    }
-    else {
-        declarations.forEach((declaration, index) => {
-            identifierMap.set(declaration.identifier, `RESPONSE${index + 1}`);
-        });
-    }
-    itemBody.querySelectorAll('[response-identifier]').forEach(interaction => {
-        const currentIdentifier = interaction.getAttribute('response-identifier')?.trim();
-        if (!currentIdentifier)
-            return;
-        const mappedIdentifier = identifierMap.get(currentIdentifier);
-        if (mappedIdentifier) {
-            interaction.setAttribute('response-identifier', mappedIdentifier);
-        }
-    });
-    declarations.forEach(declaration => {
-        const mappedIdentifier = identifierMap.get(declaration.identifier);
-        if (mappedIdentifier) {
-            declaration.identifier = mappedIdentifier;
-        }
-    });
-}
 export function formatXml(xml) {
     const PADDING = '  ';
     const reg = /(>)(<)(\/*)/g;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qti-editor/core",
-  "version": "1.3.2",
+  "version": "1.4.1",
   "description": "QTI semantics, composer registry, and XML export orchestration for QTI Editor",
   "repository": {
     "type": "git",
@@ -24,19 +24,19 @@
     }
   },
   "dependencies": {
-    "@qti-editor/interfaces": "1.2.0",
-    "@qti-editor/interaction-associate": "1.2.1",
-    "@qti-editor/interaction-choice": "1.2.1",
-    "@qti-editor/interaction-extended-text": "1.2.1",
-    "@qti-editor/interaction-gap-match": "1.1.0",
-    "@qti-editor/interaction-hottext": "1.2.2",
-    "@qti-editor/interaction-inline-choice": "1.2.1",
-    "@qti-editor/interaction-match": "1.2.0",
-    "@qti-editor/interaction-order": "0.6.0",
-    "@qti-editor/interaction-select-point": "1.2.0",
-    "@qti-editor/interaction-shared": "1.3.0",
-    "@qti-editor/interaction-text-entry": "1.2.0",
-    "@qti-editor/qti-item-divider": "1.1.0"
+    "@qti-editor/interfaces": "1.2.1",
+    "@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",