@patch-adams/core 1.4.7 → 1.4.9

package/dist/index.cjs

@@ -1684,15 +1684,11 @@ function generateLrsBridgeCode(options) {
     }
 
     /**
-     * Fetch document metadata from Bravais API to get GUIDs
-     * Endpoint: /api/v3/documents/{documentId}
+     * Fetch document metadata from Bravais shared API to get GUIDs
+     * PRIMARY endpoint: /api/shared/{token}/documents (does NOT require auth)
+     * FALLBACK endpoint: /api/v3/documents/{documentId} (requires auth)
      */
-    function fetchDocumentMetadata(documentId, callback) {
-      if (!documentId) {
-        callback(null);
-        return;
-      }
-
+    function fetchDocumentMetadata(documentIdOrToken, callback) {
       if (documentApiFetched) {
         callback(documentApiData);
         return;
@@ -1706,8 +1702,24 @@ function generateLrsBridgeCode(options) {
         return;
       }
 
-      var apiUrl = coreUrl + '/api/v3/documents/' + documentId;
-      log('Fetching document metadata from:', apiUrl);
+      // Determine which endpoint to use
+      // If we have a shared link token, use /api/shared/{token}/documents (no auth required)
+      // This is the same endpoint Xyleme Cloud Player uses
+      var sharedToken = LRS.courseInfo ? LRS.courseInfo.sharedLinkToken : null;
+      var apiUrl;
+
+      if (sharedToken) {
+        apiUrl = coreUrl + '/api/shared/' + sharedToken + '/documents';
+        log('Fetching document via shared API (no auth required):', apiUrl);
+      } else if (documentIdOrToken) {
+        apiUrl = coreUrl + '/api/v3/documents/' + documentIdOrToken;
+        log('Fetching document via v3 API (requires auth):', apiUrl);
+      } else {
+        log('No shared token or document ID available for metadata fetch');
+        documentApiFetched = true;
+        callback(null);
+        return;
+      }
 
       var xhr = new XMLHttpRequest();
       xhr.open('GET', apiUrl, true);
@@ -2307,6 +2319,22 @@ function generateLrsBridgeCode(options) {
       // This allows filtering by type (video, assessment, interaction) in Analytics
       if (activityType) {
         courseObj.definition.type = activityType;
+
+        // Also update resourceType extension to human-readable form for Bravais Type column
+        courseObj.definition.extensions = courseObj.definition.extensions || {};
+        var resourceTypeMap = {
+          'https://w3id.org/xapi/video/activity-type/video': 'Video',
+          'https://w3id.org/xapi/audio/activity-type/audio': 'Audio',
+          'http://adlnet.gov/expapi/activities/media': 'Media',
+          'http://adlnet.gov/expapi/activities/assessment': 'Assessment',
+          'http://adlnet.gov/expapi/activities/question': 'Question',
+          'http://adlnet.gov/expapi/activities/interaction': 'Interaction',
+          'http://adlnet.gov/expapi/activities/lesson': 'Lesson',
+          'http://adlnet.gov/expapi/activities/module': 'Module',
+          'http://adlnet.gov/expapi/activities/course': 'Course',
+          'http://xyleme.com/bravais/activities/document': 'Course'
+        };
+        courseObj.definition.extensions['resourceType'] = resourceTypeMap[activityType] || 'Course';
       }
 
       // Add activity-specific details to extensions
@@ -3642,32 +3670,25 @@ function generateLrsBridgeCode(options) {
       }
 
       if (!LRS.courseInfo.guid) {
-        // First, try to get real document ID from shared link token
-        // The URL path contains a thin pack file ID, not the real document ID
+        // When we have a shared link token, use /api/shared/{token}/documents directly
+        // This endpoint does NOT require authentication and returns both document GUID and version GUID
         if (sharedLinkToken) {
-          log('Fetching shared link data to get real document ID...');
-          fetchSharedLinkData(sharedLinkToken, function(linkData) {
-            if (linkData && linkData.documentId) {
-              // Got the real document ID from shared link
-              log('Got real document ID from shared link:', linkData.documentId);
-              LRS.courseInfo.documentId = linkData.documentId;
-              // Update shared link name if available
-              if (linkData.documentName) {
-                LRS.courseInfo.sharedLinkName = linkData.documentName;
+          log('Have shared link token, fetching document data via shared API...');
+          // fetchDocumentMetadata will use /api/shared/{token}/documents when sharedLinkToken is present
+          fetchDocumentMetadata(null, function(docData) {
+            if (docData) {
+              updateCourseInfoFromApi(docData);
+              // Also update document name if available
+              if (docData.name) {
+                LRS.courseInfo.sharedLinkName = docData.name;
               }
-              fetchDocDataAndSend(linkData.documentId);
-            } else if (documentId) {
-              // Fallback to extracted document ID (may be thin pack ID)
-              warn('Could not get document ID from shared link, using extracted ID:', documentId);
-              fetchDocDataAndSend(documentId);
             } else {
-              // No document ID available
-              warn('No document ID available - statements may fail aggregation');
-              setTimeout(sendLaunchEvents, 100);
+              warn('Could not fetch document data from shared API - statements may fail aggregation');
             }
+            setTimeout(sendLaunchEvents, 100);
           });
         } else if (documentId) {
-          // No shared link token, try with extracted document ID
+          // No shared link token, try with extracted document ID (requires auth)
           log('No shared link token, fetching document data with ID:', documentId);
           fetchDocDataAndSend(documentId);
         } else {