coursecode 0.1.40 → 0.1.42

package/framework/index.html

@@ -7,7 +7,7 @@
   <meta name="viewport" content="width=device-width, initial-scale=1" />
   <meta name="description" id="page-description" content="" />
   <link rel="stylesheet" href="css/framework.css" />
-  <link rel="stylesheet" href="../course/theme.css" />
+  <link rel="stylesheet" href="../template/course/theme.css" />
   <!-- Skip link is styled via framework.css for accessibility -->
 </head>
 
@@ -172,4 +172,4 @@
   <script type="module" src="./js/main.js"></script>
 </body>
 
-</html>
+</html>

package/framework/js/core/event-bus.js

@@ -120,7 +120,7 @@ class EventBus {
     // suppress to prevent infinite cascade
     if (isErrorEvent) {
       if (this._emittingError) {
-        console.warn(`[EventBus] Suppressed recursive error event: ${event}`);
+        logger.warn(`[EventBus] Suppressed recursive error event: ${event}`);
         return false;
       }
       this._emittingError = true;

package/framework/js/drivers/scorm-2004-driver.js

@@ -507,8 +507,7 @@ export class Scorm2004Driver extends ScormDriverBase {
         const value = this._scorm.get(key);
         const errorCode = this._scorm.debug.getCode();
 
-        // Error 403 = "Data Model Element Value Not Initialized" - expected for optional fields
-        if (errorCode === 403) {
+        if (this._isOptionalReadError(errorCode)) {
             return null;
         }
 
@@ -561,18 +560,12 @@ export class Scorm2004Driver extends ScormDriverBase {
      * Populates the CMI cache at init time. Single LMS read pass.
      */
     _populateCache() {
-        // Helper: read a CMI value via strict _getValue, but tolerate error 403
-        // ("Data Model Element Value Not Initialized") which strict LMSes like
-        // SCORM Cloud return for unset elements on a fresh session.
-        // Any other SCORM error still throws through _getValue's normal path.
+        // Startup cache hydration reads LMS state opportunistically. Some LMS
+        // adapters report supported-but-empty or unavailable optional fields as
+        // SCORM errors, so use the optional read path here and keep strict reads
+        // for required operations.
         const getOrDefault = (key, fallback) => {
-            try {
-                return this._getValue(key) || fallback;
-            } catch (e) {
-                const code = this._scorm.debug.getCode();
-                if (code === 403) return fallback;
-                throw e;
-            }
+            return this._getValueOptional(key) || fallback;
         };
 
         // Read-only scalars (may be uninitialized on first launch)
@@ -712,6 +705,11 @@ export class Scorm2004Driver extends ScormDriverBase {
         throw new Error(msg);
     }
 
+    _isOptionalReadError(code) {
+        const numericCode = Number(code);
+        return numericCode === 401 || numericCode === 403;
+    }
+
     // --- Recovery Mode Helpers ---
 
     _getValueRecovered(key) {
@@ -739,7 +737,7 @@ export class Scorm2004Driver extends ScormDriverBase {
             const value = api.GetValue(key);
             const errCode = api.GetLastError ? parseInt(api.GetLastError(), 10) : 0;
 
-            if (errCode === 403) {
+            if (this._isOptionalReadError(errCode)) {
                 return null;
             }
 

package/framework/js/vendor/pipwerks.js

@@ -31,13 +31,9 @@ further modified by Philip Hutchison
     if (typeof define === 'function' && define.amd) {
         // AMD. Register as an anonymous module.
         define([], factory);
-    } else if (typeof module === 'object' && module.exports) {
-        // Node. Does not work with strict CommonJS, but
-        // only CommonJS-like environments that support module.exports,
-        // like Node.
-        module.exports = factory();
     } else {
-        // Browser globals (root is window)
+        // Browser/global object. CourseCode imports this vendored copy as ESM,
+        // so avoid CommonJS export branches that modern bundlers warn about.
         root.pipwerks = factory();
     }
 }(typeof globalThis !== 'undefined' ? globalThis : typeof self !== 'undefined' ? self : typeof window !== 'undefined' ? window : this, function () {
@@ -927,5 +923,5 @@ further modified by Philip Hutchison
 // ESM Export for Vite/modern bundlers
 // Added to enable ES module imports while keeping original UMD intact
 // =============================================================================
-export default (typeof window !== 'undefined' && window.pipwerks) || (typeof global !== 'undefined' && global.pipwerks);
-export const SCORM = (typeof window !== 'undefined' && window.pipwerks?.SCORM) || (typeof global !== 'undefined' && global.pipwerks?.SCORM);
+export default globalThis.pipwerks;
+export const SCORM = globalThis.pipwerks?.SCORM;

package/framework/scripts/generate-narration.js

@@ -38,8 +38,7 @@ import {
     parseSlideNarration as parseSlideNarrationShared,
     hashContent,
     loadNarrationCache,
-    narrationCacheKey,
-    VOICE_SETTING_KEYS as _SHARED_VOICE_KEYS
+    narrationCacheKey
 } from './narration-parser.js';
 
 const __filename = fileURLToPath(import.meta.url);
@@ -54,9 +53,6 @@ const AUDIO_DIR = path.join(ASSETS_DIR, 'audio');
 const SLIDES_DIR = path.join(COURSE_DIR, 'slides');
 const CACHE_FILE = path.join(SCORM_TEMPLATE_DIR, '.narration-cache.json');
 
-// Reserved keys for voice settings (not narration content) — re-exported from shared module
-const VOICE_SETTING_KEYS = _SHARED_VOICE_KEYS;
-
 // Parse command line arguments
 const args = process.argv.slice(2);
 const FORCE_REGENERATE = args.includes('--force');
@@ -382,7 +378,7 @@ Examples:
             const contentHash = hashContent(text + JSON.stringify(settings));
             
             // Cache key includes the item key for multi-key narration
-            const cacheKey = key === 'slide' ? source.src : `${source.src}#${key}`;
+            const cacheKey = narrationCacheKey(source.src, key);
             const cachedHash = cache[cacheKey];
             const outputExists = fs.existsSync(outputPath);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coursecode",
-  "version": "0.1.40",
+  "version": "0.1.42",
   "description": "Multi-format course authoring framework with CLI tools (SCORM 2004, SCORM 1.2, cmi5, LTI 1.3)",
   "type": "module",
   "bin": {