coursecode 0.1.57 → 0.1.58

package/README.md

@@ -223,7 +223,13 @@ coursecode deploy --repair-binding
 coursecode build
 ```
 
-**Share for review**: Export a standalone preview for stakeholders. Deploy to GitHub Pages, Netlify, or any static host:
+**Share one file with no server**: Create a portable HTML course that recipients can double-click. Local progress restore is best effort and stays in that browser:
+
+```bash
+coursecode export html
+```
+
+**Share a hosted review preview**: Export the preview shell for GitHub Pages, Netlify, or another static host:
 
 ```bash
 coursecode preview --export
@@ -238,6 +244,7 @@ coursecode preview --export
 | `coursecode create <name>` | Create a new course project; `create .` derives a title-cased course title and hyphenated npm name from the current folder |
 | `coursecode init [name]` | Initialize the current directory, optionally with a course title |
 | `coursecode preview` | Preview your course locally |
+| `coursecode export html` | Create one shareable course file that opens without an LMS or server |
 | `coursecode convert` | Convert PDFs, Word, PowerPoint to markdown |
 | `coursecode mcp` | Start the MCP server for AI integration |
 | `coursecode lint` | Validate course structure and content |

package/bin/cli.js

@@ -8,6 +8,7 @@
  *   coursecode init [name]    - Initialize the current directory
  *   coursecode dev            - Start development server
  *   coursecode build          - Build course package
+ *   coursecode export html    - Create a portable single-file course
  *   coursecode upgrade        - Upgrade framework in current project
  *   coursecode clean          - Remove example files from project
  *   coursecode new <type>     - Create new slide, assessment, or config
@@ -123,6 +124,21 @@ program
     await build(options);
   });
 
+// Standalone export commands
+const exportCommand = program
+  .command('export')
+  .description('Export a course for use outside an LMS');
+
+exportCommand
+  .command('html')
+  .description('Create one shareable HTML file that opens without a server')
+  .option('-o, --output <file>', 'Output HTML file path (defaults to the course title)')
+  .option('--no-lint', 'Skip the course lint step')
+  .action(async (options) => {
+    const { exportPortableHtml } = await import('../lib/portable-html.js');
+    await exportPortableHtml(options);
+  });
+
 // Lint command
 program
   .command('lint')

package/framework/docs/FRAMEWORK_GUIDE.md

@@ -72,7 +72,7 @@ The framework supports multiple LMS formats:
 | `scorm1.2` | SCORM 1.2 (legacy LMS) | 4KB suspend_data (Strict Diet Mode found in `scorm-12-driver.js`) |
 | `lti` | LTI 1.3 (remote-hosted, JWT launch) | Unlimited (host-dependent) |
 
-> **Note on SCORM 1.2 Strict Diet Mode:** To stay within the 4KB limit, this mode only persists interaction responses for the *current* slide. If a user navigates away and returns, their previous answers on other slides may not be restored visually, although their completion status/score for assessments is always preserved.
+> **Note on SCORM 1.2 Strict Diet Mode:** The framework preserves the complete CourseCode state and compresses it into an ASCII-safe representation. If the compressed value still exceeds SCORM 1.2's 4,096-character limit, the write fails loudly instead of silently discarding learner state. Courses that exceed the limit must reduce persisted data or use SCORM 2004, cmi5, or LTI.
 
 **Set format in `course-config.js`** (for local CLI builds):
 ```javascript
@@ -138,11 +138,20 @@ The browser only downloads the one chunk matching the meta tag. Unused driver ch
 |---------|--------|---------|
 | `coursecode build` | `dist/` | Universal build + format manifest + meta tag stamped |
 | `coursecode build` (with `PACKAGE=true`) | `dist/` + ZIP | Same + format-specific ZIP for LMS upload |
+| `coursecode export html` | `<course-title>.html` | One portable learner file with an embedded standalone driver and course assets; no LMS manifest or server |
 | `coursecode preview --export` | `course-preview/` | Copy of `dist/` wrapped in stub player (for Netlify/GitHub Pages) |
 | `coursecode deploy` | Uploads `dist/` | Cloud hosts universal build, assembles format ZIPs on demand. Flags: `--promote` (force live), `--stage` (force staged), `--preview` (preview-only: production untouched, preview always moved), `--repair-binding` (clear stale local cloud binding first if the remote course was deleted). `--promote`/`--stage` are mutually exclusive; `--preview` stacks with either. **GitHub-linked courses**: production deploy blocked; only `--preview` allowed (see GitHub Source Guard below). |
 
 The ZIP never includes preview/stub player assets. Preview is a separate concern (see below).
 
+#### Portable HTML Architecture
+
+`coursecode export html` is intentionally smaller than preview export. It builds the normal learner runtime with `LMS_FORMAT=standalone`, uses `vite-plugin-singlefile` to inline generated JavaScript and CSS, embeds copied files from `course/assets/` plus the document-gallery manifest as data URLs, and omits LMS schemas, manifests, packaging, and the preview/stub-player shell.
+
+The standalone driver keeps the same semantic state contract as the LMS drivers. It saves bookmark, completion, success, score, and CourseCode suspend state to browser `localStorage`, keyed by course title and version. Restore is best effort because browsers define storage behavior for `file://` documents; when storage is unavailable, the course remains usable and keeps state only for the open session.
+
+Portable HTML is for direct file sharing, offline reference/training, USB delivery, and unmanaged kiosks. It does not provide learner identity, centralized reporting, cross-device sync, tamper-resistant records, or an administrator view. Remote embeds such as YouTube still require network access, and media-heavy courses can produce large HTML files.
+
 #### Re-Stamping for Different Formats
 
 `lib/build-packaging.js` exports `stampFormat(html, format)` for re-stamping the meta tag:
@@ -166,7 +175,9 @@ Both are pure Node utilities.
 |------|---------|
 | `framework/js/state/lms-connection.js` | `getLMSFormat()` — runtime priority chain (meta → env → default) |
 | `framework/js/drivers/driver-factory.js` | Dynamic `import()` switch — loads one driver at runtime |
+| `framework/js/drivers/standalone-driver.js` | Best-effort local persistence for portable HTML |
 | `lib/build-packaging.js` | `stampFormat()` — pure string meta tag re-stamping; `stampFormatInHtml()` — file-based wrapper |
+| `lib/portable-html.js` | Single-file Vite plugin wrapper, copied-asset embedding, and export validation |
 | `lib/manifest/manifest-factory.js` | `generateManifest()` — generates format-specific manifests |
 | Both `vite.config.js` files | Post-build `closeBundle` hook stamps meta tag into `dist/index.html` |
 
@@ -385,7 +396,7 @@ After images are acquired, the module:
 | `state-validation.js` | State hydration, migration, validation |
 | `transaction-log.js` | Ring buffer for debugging |
 
-> **Compression:** `cmi.suspend_data` is automatically compressed via `lz-string` (UTF16) in the driver layer. This is transparent to all consumers.
+> **Persistence encoding:** SCORM 2004 uses `lz-string` UTF-16 compression. SCORM 1.2 uses an ASCII-safe `CC12:` encoded representation because legacy LMS implementations may reject UTF-16 suspend data. cmi5 and LTI persist structured JSON through their HTTP state endpoints. These format differences are transparent to framework consumers.
 
 #### Domain API
 
@@ -474,7 +485,7 @@ Migrations run sequentially (v1→v2→v3), so each only handles one version jum
 
 ## Logging & Error Handling
 
-**Two complementary systems - use BOTH:**
+**Logger and event communication are integrated:** use `logger.*` for observability. `logger.warn()`, `logger.error()`, and `logger.fatal()` automatically emit structured `log:warn` or `log:error` events. Emit an additional domain event only when another framework component needs to react to that specific failure; do not duplicate a `log:error` event manually.
 
 ### Logger (Observability)
 
@@ -495,7 +506,7 @@ Global as `logger` or `window.logger` (no import needed). Auto-filtered by envir
 |------|------|----------|---------|
 | **Tier 1** | Contract violations, programming errors | Always `throw` | Wrong parameter types, API misuse, double-init |
 | **Tier 2** | Runtime/init failures (missing DOM, bad config) | `logger.fatal()` → throws in DEV, logs+degrades in PROD | Missing container element, invalid data attributes |
-| **Tier 3** | Existing dual-mode functions | Logger + eventBus only | Driver warnings, state persistence failures |
+| **Tier 3** | Recoverable background or unload failures | Structured logger event; retain/retry or emit a domain event when actionable | Debounced commit retry, best-effort unload save |
 
 **`logger.fatal(message, context)`** — Tier 2 handler in `framework/js/utilities/logger.js`:
 
@@ -515,7 +526,14 @@ All error events follow the standardized shape:
 { domain, operation, message, stack?, context? }
 ```
 
-**When to catch:** Only to show user notification or cleanup, then MUST re-throw:
+**When to catch depends on who can still act on the failure:**
+
+- **Critical awaited work** (assessment submission, completion, explicit flush/terminate): log or clean up, then re-throw so the caller cannot treat the operation as successful.
+- **Detached background work** (debounced commits): do not create an unhandled rejection. Log with structured context, preserve dirty state, and retry or emit an actionable domain event.
+- **Unload/pagehide work**: the browser is leaving and cannot await a recovery path. Use the driver's best-effort unload transport and log any synchronous or asynchronous failure with structured context; do not re-throw into the abandoned event handler.
+- **Expected capability probes**: a documented unsupported optional LMS field may degrade to a conservative default. Do not suppress failures for required resume, completion, score, or assessment fields.
+
+Critical-path example:
 
 ```javascript
 try {
@@ -633,11 +651,11 @@ Courses linked to GitHub for production deploys are protected from accidental CL
 | Layer | Mechanism | Key |
 |-------|-----------|-----|
 | **CLI (local)** | `deploy()` reads `sourceType` from `.coursecoderc.json` | Fast fail before build — no wasted time |
-| **Server (safety net)** | Deploy endpoint rejects non-preview uploads for `source_type = 'github'` | Can't bypass via old CLI or `curl` |
+| **Server (safety net)** | Finalize endpoint rejects non-preview CLI uploads when `github_repo` is set | Can't bypass via old CLI or `curl` |
 
 - **`--preview` always allowed** — useful for testing without touching production pointer
 - **`.coursecoderc.json`** carries `sourceType` and `githubRepo` (committed to repo by cloud's GitHub integration), so anyone cloning gets the guard automatically
-- **Self-healing on unlink:** If the cloud course is unlinked from GitHub, both `status()` and `deploy()` reconcile — they detect the server no longer reports `source_type: 'github'` and clear the local `sourceType`/`githubRepo` from `.coursecoderc.json`. No manual cleanup or repo commit needed.
+- **Self-healing link state:** Both `status()` and `deploy()` treat the server's `github_repo` as authoritative. They add or refresh local `sourceType`/`githubRepo` metadata when linked and clear it when unlinked. No manual cleanup or repo commit is needed.
 - CLI error code: `github_source_blocked` (structured JSON for Desktop/CI consumers)
 
 ---

package/framework/docs/USER_GUIDE.md

@@ -51,6 +51,7 @@ A complete guide to creating interactive e-learning courses with AI assistance.
 9. [Sharing and Deploying](#sharing-and-deploying)
    - [Sharing Previews](#sharing-previews)
    - [Preview Export Options](#preview-export-options)
+   - [Sharing One File Without a Server](#sharing-one-file-without-a-server)
    - [Understanding LMS Formats](#understanding-lms-formats)
    - [Standard Deployment](#standard-deployment)
    - [CDN Deployment (Advanced)](#cdn-deployment-advanced)
@@ -637,6 +638,19 @@ coursecode preview --export --no-content
 - `--nojekyll`: add `.nojekyll` (important for GitHub Pages)
 - `--no-content`: hide the content viewer panel in exported preview
 
+### Sharing One File Without a Server
+
+For training that will be opened directly from email, a shared drive, USB media, or a local computer, export one portable HTML file:
+
+```bash
+coursecode export html
+coursecode export html --output ./Safety_Training.html
+```
+
+The recipient double-clicks the `.html` file and uses it in a modern browser. CourseCode embeds the learner runtime and local course assets, so no LMS, web server, install, or ZIP extraction is required. Progress and resume state are saved locally in the browser when `file://` storage is available. If the browser blocks local storage, the course still works but restore is limited to the current session.
+
+This export is a good fit for informal training, reference material, stakeholder review, offline delivery, and unmanaged kiosks. It does not provide centralized learner records, identity, administrator reporting, cross-device sync, or certified completion evidence. Internet-hosted content such as YouTube still needs a connection, and large audio/video files make the exported HTML correspondingly large.
+
 ### Understanding LMS Formats
 
 An LMS (Learning Management System) is the platform your organization uses to deliver training — think Cornerstone, Moodle, Canvas, Docebo, etc. CourseCode packages your course in a format your LMS understands.

package/framework/js/app/AppActions.js

@@ -107,40 +107,39 @@ export async function checkCompletion() {
         return false;
     }
 
-    // If already marked as completed, no need to re-evaluate.
-    if (currentCompletionStatus === 'completed') {
-        eventBus.emit('course:statusChanged', {
-            completionStatus: 'completed',
-            successStatus: stateManager.getSuccess() || 'unknown',
-            isOnLastSlide: true
-        });
-        return true;
-    }
-
-    // On a completion slide, so evaluate final status.
+    // Always re-evaluate on a completion slide. Completion and success are
+    // independent: a learner can complete a first attempt with a failed
+    // success status, then pass a permitted retake later in the same or a
+    // resumed session.
     const assessmentSlides = await CourseHelpers.getSlidesByType('assessment');
     const assessmentConfigs = await CourseHelpers.getAssessmentConfigs();
 
     // Completion is based on submission requirements.
     const assessmentsRequiringSubmission = assessmentSlides.filter(s =>
-        assessmentConfigs.get(s.assessmentId)?.completionRequirements?.requireSubmission
+        assessmentConfigs.get(AssessmentManager.resolveAssessmentId(s))?.completionRequirements?.requireSubmission
     );
 
     const allSubmitted = assessmentsRequiringSubmission.every(s =>
-        AssessmentManager.meetsCompletionRequirements(s.assessmentId, { requireSubmission: true })
+        AssessmentManager.meetsCompletionRequirements(
+            AssessmentManager.resolveAssessmentId(s),
+            { requireSubmission: true }
+        )
     );
 
     const completionStatus = allSubmitted ? 'completed' : 'incomplete';
 
     // Success is based on passing requirements.
     const assessmentsRequiringPass = assessmentSlides.filter(s =>
-        assessmentConfigs.get(s.assessmentId)?.completionRequirements?.requirePass
+        assessmentConfigs.get(AssessmentManager.resolveAssessmentId(s))?.completionRequirements?.requirePass
     );
 
     let successStatus = 'unknown';
     if (assessmentsRequiringPass.length > 0) {
         const allPassed = assessmentsRequiringPass.every(s =>
-            AssessmentManager.meetsCompletionRequirements(s.assessmentId, { requirePass: true })
+            AssessmentManager.meetsCompletionRequirements(
+                AssessmentManager.resolveAssessmentId(s),
+                { requirePass: true }
+            )
         );
         successStatus = allPassed ? 'passed' : 'failed';
         // Note: Course score reporting is handled by ScoreManager (configure in course-config.js)
@@ -188,14 +187,9 @@ function _finalizeCurrentSlideProgress() {
     if (currentSlideId) {
         NavigationState.addVisitedSlide(currentSlideId);
 
-        // Update progress measure one final time before exit
-        const slides = NavigationActions.getAllSlides();
-        if (slides) {
-            const totalSequentialSlides = slides.filter(s =>
-                s.navigation?.sequence?.included !== false
-            ).length;
-            stateManager.updateProgressMeasure(totalSequentialSlides);
-        }
+        // Update progress one final time using the same dynamic sequence rules
+        // as navigation (conditional/remedial slides must not inflate it).
+        NavigationActions.syncProgressMeasure();
     }
 }
 

package/framework/js/app/AppUI.js

@@ -24,6 +24,7 @@ const completionFeedbackTemplate = `
 
 // Store the active rating interaction instance
 let activeRatingInteraction = null;
+const isStandaloneRuntime = document.querySelector('meta[name="lms-format"]')?.content === 'standalone';
 
 const MODAL_DEFINITIONS = {
     exit: {
@@ -45,8 +46,9 @@ const MODAL_DEFINITIONS = {
         title: 'Course Complete!',
         body: `
             <p><strong>Congratulations!</strong> You have successfully completed this course.</p>
-            <p>Your completion status and final score have been recorded in the Learning Management System.</p>
-            <p>When you click "Complete & Exit" below, this window will close and you will be returned to the LMS.</p>
+            ${isStandaloneRuntime
+                ? '<p>Your completion status and final score have been saved in this browser.</p><p>When you click "Complete & Exit" below, the session will finish. You can close the file or reopen it later.</p>'
+                : '<p>Your completion status and final score have been recorded in the Learning Management System.</p><p>When you click "Complete & Exit" below, this window will close and you will be returned to the LMS.</p>'}
             ${completionFeedbackTemplate}
         `,
         footer: `
@@ -61,8 +63,9 @@ const MODAL_DEFINITIONS = {
     postExit: {
         title: 'Session Closed',
         body: `
-            <p>Your progress has been saved in the LMS. It is now safe to close this window.</p>
-            <p>If this window does not close automatically, please close it manually and return to the LMS.</p>
+            ${isStandaloneRuntime
+                ? '<p>Your progress has been saved in this browser. It is now safe to close this file.</p><p>Open the HTML file again to resume later.</p>'
+                : '<p>Your progress has been saved in the LMS. It is now safe to close this window.</p><p>If this window does not close automatically, please close it manually and return to the LMS.</p>'}
         `,
         footer: '',
         config: {

package/framework/js/assessment/AssessmentActions.js

@@ -70,7 +70,7 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
         await stateManager.setAttemptNumber(attemptNumber);
 
         _manageFooterVisibility('question');
-        uiManager.showView('question');
+        await uiManager.showView('question');
     }
 
     async function handlePrev() {
@@ -79,7 +79,7 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
         const currentIndex = stateManager.getCurrentQuestionIndex();
         if (currentIndex > 0) {
             await stateManager.setCurrentQuestionIndex(currentIndex - 1);
-            uiManager.showView('question');
+            await uiManager.showView('question');
         }
     }
 
@@ -94,7 +94,7 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
                 await stateManager.updateSession({ reviewReached: true });
                 await stateManager.setCurrentView('review');
                 _manageFooterVisibility('review');
-                uiManager.showView('review');
+                await uiManager.showView('review');
             } else {
                 // FIX: Route through handleSubmit to ensure unanswered checks are performed
                 // previously called submitAssessment() directly, bypassing validation
@@ -102,7 +102,7 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
             }
         } else {
             await stateManager.setCurrentQuestionIndex(currentIndex + 1);
-            uiManager.showView('question');
+            await uiManager.showView('question');
         }
     }
 
@@ -119,21 +119,21 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
         await stateManager.setCurrentView('question');
         await stateManager.setCurrentQuestionIndex(index);
         _manageFooterVisibility('question');
-        uiManager.showView('question');
+        await uiManager.showView('question');
     }
 
     async function handleBackToQuestions() {
         await _forceSaveCurrentResponse();
         await stateManager.setCurrentView('question');
         _manageFooterVisibility('question');
-        uiManager.showView('question');
+        await uiManager.showView('question');
     }
 
     async function handleJumpToReview() {
         await _forceSaveCurrentResponse();
         await stateManager.setCurrentView('review');
         _manageFooterVisibility('review');
-        uiManager.showView('review');
+        await uiManager.showView('review');
     }
 
     async function handleSubmit() {
@@ -242,14 +242,14 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
             // This will cause slide to create NEW assessment instance
             const currentSlideId = getCurrentSlideId();
             if (currentSlideId) {
-                goToSlide(currentSlideId, { refreshAssessment: true });
+                await goToSlide(currentSlideId, { refreshAssessment: true });
                 return; // Exit - new instance will be created
             }
         }
 
         // Simple reset - same questions
         _manageFooterVisibility('intro');
-        uiManager.showView('intro');
+        await uiManager.showView('intro');
     }
 
     function handleRestartCourse() {
@@ -353,7 +353,7 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
             throw new Error(errorMessage);
         }
 
-        goToSlide(firstRemedialSlide, {
+        await goToSlide(firstRemedialSlide, {
             fromAssessment: config.id,
             remedialReview: true
         });
@@ -413,11 +413,6 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
         const displayData = _prepareResultsDisplayData(finalResults);
 
         await stateManager.setCurrentView('results');
-        _manageFooterVisibility('results');
-        uiManager.showView('results', displayData);
-
-        // Keep full results in memory for this session only (not persisted)
-        // This allows detailed review immediately after submission
 
         // Automatically set linked objective if configured
         if (config.assessmentObjective) {
@@ -446,6 +441,12 @@ export function createAssessmentActions(stateManager, uiManager, questionInstanc
         // Assessment submission is a critical action - don't rely on debounce
         await globalStateManager.flush();
 
+        // Only transition the UI after the assessment outcome, objective, and
+        // course score are durably committed. A rendering failure must never
+        // prevent the LMS from receiving the learner's submitted result.
+        _manageFooterVisibility('results');
+        await uiManager.showView('results', displayData);
+
         if (typeof config.onComplete === 'function') {
             config.onComplete(finalResults);
         }

package/framework/js/assessment/AssessmentFactory.js

@@ -231,15 +231,37 @@ export function createAssessmentInstance(config) {
                     });
                     throw error;
                 } else {
-                    // Production mode: Log warning and filter out missing questions
-                    // Also emit event so course can potentially track this
-                    logger.warn(`[AssessmentFactory] ${errorMessage} Continuing with available questions.`);
+                    // Index-keyed responses cannot be safely paired with a
+                    // filtered question list. Restart only the in-progress
+                    // session and preserve the learner's submitted summaries.
+                    logger.warn(`[AssessmentFactory] ${errorMessage} Restarting the in-progress attempt safely.`);
                     eventBus.emit('state:recovered', {
                         domain: 'assessment',
                         message: errorMessage,
                         context: { assessmentId, missingIds },
-                        action: 'filtered_missing_questions'
+                        action: 'restarted_changed_assessment'
                     });
+
+                    const replacementQuestions = _selectNewQuestions();
+                    const existingSession = assessmentState.getSession() || {};
+                    assessmentState.updateSession({
+                        ...existingSession,
+                        currentView: 'intro',
+                        currentQuestionIndex: 0,
+                        startTime: null,
+                        submitted: false,
+                        responses: {},
+                        selectedQuestions: replacementQuestions.map(question => question.id),
+                        reviewReached: false
+                    }, 'assessment-state:course-update').catch(error => {
+                        logger.error(`Failed to persist safe assessment restart: ${error.message}`, {
+                            domain: 'assessment',
+                            operation: 'recover-changed-questions',
+                            assessmentId,
+                            stack: error.stack
+                        });
+                    });
+                    return replacementQuestions;
                 }
             }
 
@@ -248,22 +270,19 @@ export function createAssessmentInstance(config) {
                 .filter(q => q !== undefined); // Filter out any missing questions
         }
 
+        return _selectNewQuestions();
+    }
+
+    function _selectNewQuestions() {
         if (config.questionBanks && Array.isArray(config.questionBanks) && config.questionBanks.length > 0) {
-            // New session with banks: select and optionally randomize
             let selected = _selectQuestionsFromBanks(config.questionBanks);
-
-            if (config.settings?.randomizeQuestions) {
-                selected = shuffleArray(selected);
-            }
-
+            if (config.settings?.randomizeQuestions) selected = shuffleArray(selected);
             return selected;
         }
 
-        // Direct mode: use questions array as-is
         if (config.settings?.randomizeQuestions && Array.isArray(config.questions)) {
             return shuffleArray(config.questions);
         }
-
         return config.questions || [];
     }
 
@@ -320,10 +339,20 @@ export function createAssessmentInstance(config) {
         // Resolve active questions for this session
         activeQuestions = _resolveActiveQuestions();
 
-        // Persist selection if this is a new bank-based session (fire-and-forget)
-        if (config.questionBanks && !session?.selectedQuestions) {
+        // Persist every randomized order, including direct question arrays.
+        // Responses are keyed by index, so resume must reconstruct the exact
+        // same question order or answers can be scored against another item.
+        const requiresPersistedOrder = Boolean(
+            config.questionBanks || config.settings?.randomizeQuestions
+        );
+        if (requiresPersistedOrder && !session?.selectedQuestions) {
             assessmentState.setSelectedQuestions(activeQuestions).catch(error => {
-                logger.error(`Failed to persist selected questions: ${error.message}`, { domain: 'assessment', operation: 'persist-selected-questions', assessmentId });
+                logger.error(`Failed to persist selected questions: ${error.message}`, {
+                    domain: 'assessment',
+                    operation: 'persist-selected-questions',
+                    assessmentId,
+                    stack: error.stack
+                });
             });
         }
 

package/framework/js/automation/api-engagement.js

@@ -61,7 +61,7 @@ export function createEngagementMethods(logTrace) {
             if (!slideId) {
                 throw new Error('CourseCodeAutomation: No active slide');
             }
-            if (typeof percentage !== 'number' || percentage < 0 || percentage > 100) {
+            if (!Number.isFinite(percentage) || percentage < 0 || percentage > 100) {
                 throw new Error('CourseCodeAutomation: Scroll depth must be a number between 0 and 100');
             }
             engagementManager.trackScrollDepth(slideId, percentage);
@@ -143,7 +143,7 @@ export function createEngagementMethods(logTrace) {
             const currentSlideId = NavigationState.getCurrentSlideId();
 
             if (state.duration > 0) {
-                const targetPosition = state.duration * (state.completionThreshold || 0.95);
+                const targetPosition = state.duration * state.completionThreshold;
                 audioManager.seek(targetPosition);
             }
 
@@ -163,7 +163,10 @@ export function createEngagementMethods(logTrace) {
                 case 'tab':
                 case 'standalone':
                     if (currentSlideId) {
-                        engagementManager.trackStandaloneAudioComplete(currentSlideId, state.contextId);
+                        const audioId = state.contextType === 'standalone'
+                            ? state.contextId.replace(/^standalone-/, '')
+                            : state.contextId;
+                        engagementManager.trackStandaloneAudioComplete(currentSlideId, audioId);
                     }
                     break;
             }

package/framework/js/components/ui-components/audio-player.js

@@ -60,6 +60,7 @@ import engagementManager from '../../engagement/engagement-manager.js';
 import * as NavigationState from '../../navigation/NavigationState.js';
 import { logger } from '../../utilities/logger.js';
 import { iconManager } from '../../utilities/icons.js';
+import { normalizeCompletionThreshold } from '../../utilities/media-utils.js';
 
 
 /** @type {HTMLElement|null} */
@@ -351,6 +352,7 @@ function _handleProgressMousedown(event) {
 
     const progressBar = elements.progressBar;
     const rect = progressBar.getBoundingClientRect();
+    if (rect.width <= 0) return;
 
     const seek = (clientX) => {
         const percentage = Math.max(0, Math.min(100, ((clientX - rect.left) / rect.width) * 100));
@@ -823,7 +825,7 @@ class StandaloneAudioPlayer {
         this.audioSrc = container.dataset.audioSrc;
         this.required = container.dataset.audioRequired === 'true';
         this.compact = container.dataset.audioCompact === 'true';
-        this.threshold = parseFloat(container.dataset.audioThreshold) || 0.95;
+        this.threshold = normalizeCompletionThreshold(container.dataset.audioThreshold);
 
         if (!this.audioId) {
             throw new Error('[AudioPlayer] Standalone player requires data-audio-id');
@@ -836,6 +838,12 @@ class StandaloneAudioPlayer {
         this.isActive = false;
         this.eventHandlers = {};
         this.elements = {};
+        this.domEventHandlers = {
+            click: this._handleClick.bind(this),
+            progressKeydown: this._handleProgressKeydown.bind(this),
+            progressMousedown: this._handleProgressMousedown.bind(this)
+        };
+        this.dragCleanup = null;
 
         this._render();
         this._cacheElements();
@@ -867,11 +875,11 @@ class StandaloneAudioPlayer {
     }
 
     _setupEventListeners() {
-        this.container.addEventListener('click', this._handleClick.bind(this));
+        this.container.addEventListener('click', this.domEventHandlers.click);
 
         if (this.elements.progressBar) {
-            this.elements.progressBar.addEventListener('keydown', this._handleProgressKeydown.bind(this));
-            this.elements.progressBar.addEventListener('mousedown', this._handleProgressMousedown.bind(this));
+            this.elements.progressBar.addEventListener('keydown', this.domEventHandlers.progressKeydown);
+            this.elements.progressBar.addEventListener('mousedown', this.domEventHandlers.progressMousedown);
         }
     }
 
@@ -915,7 +923,9 @@ class StandaloneAudioPlayer {
                 completionThreshold: this.threshold
             }, this.contextId, 'standalone');
         } catch (error) {
-            logger.error(`[AudioPlayer] Failed to load ${this.audioId}:`, error.message);
+            if (error.name !== 'AbortError') {
+                logger.error(`[AudioPlayer] Failed to load ${this.audioId}:`, error.message);
+            }
         }
     }
 
@@ -939,8 +949,11 @@ class StandaloneAudioPlayer {
     _handleProgressMousedown(event) {
         if (!this.isActive || !audioManager.hasAudio()) return;
 
+        this.dragCleanup?.();
+
         const progressBar = this.elements.progressBar;
         const rect = progressBar.getBoundingClientRect();
+        if (rect.width <= 0) return;
 
         const seek = (clientX) => {
             const pct = Math.max(0, Math.min(100, ((clientX - rect.left) / rect.width) * 100));
@@ -950,15 +963,19 @@ class StandaloneAudioPlayer {
         seek(event.clientX);
 
         const onMouseMove = (e) => seek(e.clientX);
-        const onMouseUp = () => {
+        const cleanupDrag = () => {
             document.removeEventListener('mousemove', onMouseMove);
-            document.removeEventListener('mouseup', onMouseUp);
+            document.removeEventListener('mouseup', cleanupDrag);
             progressBar.classList.remove('dragging');
+            if (this.dragCleanup === cleanupDrag) {
+                this.dragCleanup = null;
+            }
         };
 
         progressBar.classList.add('dragging');
         document.addEventListener('mousemove', onMouseMove);
-        document.addEventListener('mouseup', onMouseUp);
+        document.addEventListener('mouseup', cleanupDrag);
+        this.dragCleanup = cleanupDrag;
     }
 
     _subscribeToAudioEvents() {
@@ -1111,6 +1128,11 @@ class StandaloneAudioPlayer {
     }
 
     destroy() {
+        this.dragCleanup?.();
+        this.container.removeEventListener('click', this.domEventHandlers.click);
+        this.elements.progressBar?.removeEventListener('keydown', this.domEventHandlers.progressKeydown);
+        this.elements.progressBar?.removeEventListener('mousedown', this.domEventHandlers.progressMousedown);
+
         eventBus.off('audio:loadStart', this.eventHandlers.loadStart);
         eventBus.off('audio:loaded', this.eventHandlers.loaded);
         eventBus.off('audio:unloaded', this.eventHandlers.unloaded);