@patch-adams/core 1.4.32 → 1.4.34

package/dist/cli.js

@@ -4,6 +4,7 @@ import { existsSync, readFileSync, writeFileSync } from 'fs';
 import { resolve, basename, extname } from 'path';
 import chalk from 'chalk';
 import ora from 'ora';
+import crypto from 'crypto';
 import AdmZip from 'adm-zip';
 import archiver from 'archiver';
 import { PassThrough } from 'stream';
@@ -1625,8 +1626,8 @@ function generateLrsBridgeCode(options) {
         var cached = localStorage.getItem(cacheKey);
         if (cached) {
           var parsed = JSON.parse(cached);
-          // TTL: 7 days = 604800000 ms
-          if (parsed.timestamp && (Date.now() - parsed.timestamp) < 604800000) {
+          // TTL: 7 days = 604800000 ms; also invalidate if missing bravaisUserId (old cache)
+          if (parsed.timestamp && (Date.now() - parsed.timestamp) < 604800000 && parsed.bravaisUserId) {
             log('Employee data from localStorage cache');
             employeeLookupData = parsed;
             employeeLookupFetched = true;
@@ -5441,6 +5442,22 @@ $1</resource>`
       return [];
     }
   }
+  /**
+   * Update the manifest identifier attribute to a new value.
+   * Used to ensure every course has a unique identifier (e.g., project UUID).
+   */
+  updateIdentifier(zip, newIdentifier) {
+    const entry = zip.getEntry("imsmanifest.xml");
+    if (!entry) return false;
+    const xml = entry.getData().toString("utf-8");
+    const updated = xml.replace(
+      /(<manifest[^>]+identifier\s*=\s*["'])[^"']+(["'])/i,
+      `$1${newIdentifier}$2`
+    );
+    if (updated === xml) return false;
+    zip.updateFile("imsmanifest.xml", Buffer.from(updated, "utf-8"));
+    return true;
+  }
   /**
    * Get the file paths that will be added to the package
    */
@@ -5945,6 +5962,24 @@ var Patcher = class {
       if (metadata.description) console.log(`  - Description: ${metadata.description.substring(0, 50)}...`);
       if (metadata.language) console.log(`  - Language: ${metadata.language}`);
       if (metadata.masteryScore !== null) console.log(`  - Mastery Score: ${metadata.masteryScore}`);
+      const configDocGuid = this.config.lrsBridge?.documentGuid;
+      if (configDocGuid) {
+        console.log(`[Patcher] Overriding courseId with documentGuid: ${configDocGuid}`);
+        metadata.courseId = configDocGuid;
+        metadata.courseIdSource = "manifest";
+        result.courseId = configDocGuid;
+        if (this.manifestUpdater.updateIdentifier(zip, configDocGuid)) {
+          console.log(`[Patcher] Updated manifest identifier to: ${configDocGuid}`);
+        }
+      } else if (metadata.courseIdSource === "fallback") {
+        const generated = crypto.randomUUID();
+        console.log(`[Patcher] No course identifier found, generating UUID: ${generated}`);
+        metadata.courseId = generated;
+        result.courseId = generated;
+        if (this.manifestUpdater.updateIdentifier(zip, generated)) {
+          console.log(`[Patcher] Wrote generated UUID to manifest identifier`);
+        }
+      }
       const htmlInjector = this.getHtmlInjector(toolInfo.tool);
       htmlInjector.setMetadata(metadata);
       let fetchedFallbacks = {};