tessera-learn 0.0.10 → 0.0.13

package/src/runtime/adapters/retry.ts

@@ -24,29 +24,37 @@ function lmsCallSucceeded(result: unknown): boolean {
 
 function readLastErrorCode(reporter: LMSErrorReporter | undefined): string {
   if (!reporter) return '';
-  try { return reporter.code(); } catch { return ''; }
+  try {
+    return reporter.code();
+  } catch {
+    return '';
+  }
 }
 
 function logRetryGiveUp(
   errorReporter: LMSErrorReporter | undefined,
   lastErrCode: string,
-  context: string | undefined
+  context: string | undefined,
 ): void {
   const ctx = context ? ` [${context}]` : '';
   console.warn(
-    `Tessera: LMS call failed after retries${ctx}${formatLMSErrorDetail(errorReporter, lastErrCode)}, continuing without persistence`
+    `Tessera: LMS call failed after retries${ctx}${formatLMSErrorDetail(errorReporter, lastErrCode)}, continuing without persistence`,
   );
 }
 
 export function formatLMSErrorDetail(
   errorReporter: LMSErrorReporter | undefined,
-  code: string
+  code: string,
 ): string {
   if (!errorReporter || !code || code === '0') return '';
   let msg = '';
   let diag = '';
-  try { msg = errorReporter.message(code); } catch {}
-  try { diag = errorReporter.diagnostic?.(code) ?? ''; } catch {}
+  try {
+    msg = errorReporter.message(code);
+  } catch {}
+  try {
+    diag = errorReporter.diagnostic?.(code) ?? '';
+  } catch {}
   let detail = ` (LMS error ${code}`;
   if (msg) detail += `: ${msg}`;
   if (diag && diag !== msg) detail += ` — ${diag}`;
@@ -56,24 +64,21 @@ export function formatLMSErrorDetail(
 
 /** Sync call that warns with the LMS error code on failure (terminate-path). */
 export function callSyncOrWarn(
-  fn: () => any,
+  fn: () => unknown,
   context: string,
-  errorReporter?: LMSErrorReporter
+  errorReporter?: LMSErrorReporter,
 ): boolean {
-  let ok = false;
+  let ok: boolean;
   try {
     ok = lmsCallSucceeded(fn());
   } catch (err) {
-    console.warn(
-      `Tessera: LMS call threw [${context}] during terminate`,
-      err
-    );
+    console.warn(`Tessera: LMS call threw [${context}] during terminate`, err);
     return false;
   }
   if (!ok) {
     const code = readLastErrorCode(errorReporter);
     console.warn(
-      `Tessera: LMS call failed [${context}] during terminate${formatLMSErrorDetail(errorReporter, code)}`
+      `Tessera: LMS call failed [${context}] during terminate${formatLMSErrorDetail(errorReporter, code)}`,
     );
   }
   return ok;
@@ -95,10 +100,10 @@ export function callSyncOrWarn(
  * for SCORM adapters where the underlying API calls are synchronous.
  */
 export async function withRetry(
-  fn: () => any,
+  fn: () => unknown,
   maxRetries = RETRY_ATTEMPTS,
   errorReporter?: LMSErrorReporter,
-  context?: string
+  context?: string,
 ): Promise<boolean> {
   let lastErrCode = '';
   let threw = false;
@@ -128,7 +133,7 @@ export async function withRetry(
  * Synchronous single-attempt LMS call. Used during page unload
  * where async retries cannot run.
  */
-export function callSync(fn: () => any): boolean {
+export function callSync(fn: () => unknown): boolean {
   try {
     return lmsCallSucceeded(fn());
   } catch {
@@ -143,7 +148,7 @@ export function callSync(fn: () => any): boolean {
  * the failed operation on the next flush trigger.
  */
 interface QueueEntry {
-  fn: () => any;
+  fn: () => unknown;
   context?: string;
 }
 
@@ -164,10 +169,10 @@ export class WriteQueue {
   /**
    * Enqueue an operation and trigger a flush.
    */
-  enqueue(fn: () => any, context?: string): void {
+  enqueue(fn: () => unknown, context?: string): void {
     this.#queue.push({ fn, context });
     if (!this.#flushing) {
-      this.#flush();
+      void this.#flush();
     }
   }
 
@@ -296,67 +301,3 @@ export function findLMSAPI(propName: string): unknown {
   // Check window.parent chain (iframe launch pattern)
   return scan(window);
 }
-
-/**
- * Format integer seconds as SCORM 1.2 `CMITimespan` (HHHH:MM:SS.SS).
- *
- * `DurationTracker.sessionSeconds` always feeds integer seconds via
- * `Math.floor`, so the centisecond field is always `.00`. The format
- * still includes it because `CMITimespan` is defined that way and some
- * older LMS importers reject the bare HHHH:MM:SS form.
- */
-export function formatHHMMSS(totalSeconds: number): string {
-  const whole = Math.floor(totalSeconds);
-  const hours = Math.floor(whole / 3600);
-  const minutes = Math.floor((whole % 3600) / 60);
-  const seconds = whole % 60;
-  const hh = String(hours).padStart(4, '0');
-  const mm = String(minutes).padStart(2, '0');
-  const ss = String(seconds).padStart(2, '0');
-  return `${hh}:${mm}:${ss}.00`;
-}
-
-/**
- * SCORM 2004 4E §4.2/§4.3 define CMIDecimal-like elements as real(10,7) —
- * `String(1/3)` exceeds that and trips SCORM Cloud with error 406. Rounds,
- * then trims trailing zeros (no padded "0.8500000" forms).
- */
-export function formatReal107(value: number): string {
-  if (!Number.isFinite(value)) return '0';
-  const rounded = Math.round(value * 1e7) / 1e7;
-  return rounded
-    .toFixed(7)
-    .replace(/(\.\d*?)0+$/, '$1')
-    .replace(/\.$/, '');
-}
-
-/**
- * SCORM 2004 4E §3.3.10.1 references ISO 8601 §5.3.3 — local date+time, no
- * zone designator. Strict validators reject `Z`, `±hh:mm`, and fractional
- * seconds with error 406. UTC components are used so writes don't drift
- * across local-TZ flips even though the format is zone-free.
- */
-export function formatISO8601Timestamp(date: Date): string {
-  const yyyy = date.getUTCFullYear();
-  const mm = String(date.getUTCMonth() + 1).padStart(2, '0');
-  const dd = String(date.getUTCDate()).padStart(2, '0');
-  const hh = String(date.getUTCHours()).padStart(2, '0');
-  const mi = String(date.getUTCMinutes()).padStart(2, '0');
-  const ss = String(date.getUTCSeconds()).padStart(2, '0');
-  return `${yyyy}-${mm}-${dd}T${hh}:${mi}:${ss}`;
-}
-
-/**
- * Format seconds as ISO 8601 duration: PT1H30M45S
- */
-export function formatISO8601Duration(totalSeconds: number): string {
-  const hours = Math.floor(totalSeconds / 3600);
-  const minutes = Math.floor((totalSeconds % 3600) / 60);
-  const seconds = totalSeconds % 60;
-
-  let result = 'PT';
-  if (hours > 0) result += `${hours}H`;
-  if (minutes > 0) result += `${minutes}M`;
-  if (seconds > 0 || result === 'PT') result += `${seconds}S`;
-  return result;
-}

package/src/runtime/adapters/scorm-base.ts

@@ -71,11 +71,11 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
       () => this.dialect.initialize(this.api),
       undefined,
       this.errorReporter,
-      'Initialize'
+      'Initialize',
     );
     if (!initialized) {
       console.warn(
-        'Tessera: LMS Initialize failed — all subsequent persistence calls will fail with error 301 (Not Initialized). Reload the launch from the LMS.'
+        'Tessera: LMS Initialize failed — all subsequent persistence calls will fail with error 301 (Not Initialized). Reload the launch from the LMS.',
       );
       return;
     }
@@ -86,7 +86,7 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
     } catch (err) {
       console.warn(
         'Tessera: LMS threw on GetValue(cmi.suspend_data); resume disabled for this launch',
-        err
+        err,
       );
     }
     if (raw && raw.trim()) {
@@ -95,7 +95,7 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
       } catch (err) {
         console.warn(
           'Tessera: cmi.suspend_data is not valid JSON; resume disabled for this launch (the LMS may have truncated a prior write)',
-          err
+          err,
         );
         this.#state = null;
       }
@@ -103,13 +103,13 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
 
     // n indexing must continue from _count — restarting at 0 would overwrite
     // the prior session's records (the LMS uses n as the array key).
-    let countRaw = '';
+    let countRaw: string;
     try {
       countRaw = this.dialect.getValue(this.api, 'cmi.interactions._count');
     } catch (err) {
       console.warn(
         'Tessera: LMS threw on GetValue(cmi.interactions._count); new interactions will be written from index 0 and may overwrite prior session records',
-        err
+        err,
       );
       return;
     }
@@ -119,7 +119,7 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
       this.interactionCount = n;
     } else {
       console.warn(
-        `Tessera: LMS returned non-numeric cmi.interactions._count="${countRaw}"; new interactions will be written from index 0 and may overwrite prior session records`
+        `Tessera: LMS returned non-numeric cmi.interactions._count="${countRaw}"; new interactions will be written from index 0 and may overwrite prior session records`,
       );
     }
   }
@@ -141,12 +141,12 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
           `${this.dialect.suspendDataLimitLabel} limit. The LMS will likely ` +
           `truncate it and the next resume will lose state. Reduce ` +
           `usePersistence() payloads or switch export.standard to a ` +
-          `larger-limit standard (scorm2004/cmi5).`
+          `larger-limit standard (scorm2004/cmi5).`,
       );
     }
     this.queue.enqueue(
       () => this.dialect.setValue(this.api, 'cmi.suspend_data', json),
-      'cmi.suspend_data'
+      'cmi.suspend_data',
     );
   }
 
@@ -155,14 +155,14 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
     this.queue.enqueue(
       () =>
         this.dialect.setValue(this.api, this.dialect.sessionTimeKey, formatted),
-      this.dialect.sessionTimeKey
+      this.dialect.sessionTimeKey,
     );
   }
 
   reportInteraction(
     questionId: string,
     interaction: Interaction,
-    correct: boolean | null
+    correct: boolean | null,
   ): void {
     const n = this.interactionCount++;
     const fields = buildScormInteractionFields(
@@ -177,12 +177,12 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
         typeValue: this.dialect.interactionFields.typeValue(interaction.type),
         resultLabels: this.dialect.interactionFields.resultLabels,
         format: this.dialect.interactionFields.format,
-      }
+      },
     );
     for (const [key, value] of fields) {
       this.queue.enqueue(
         () => this.dialect.setValue(this.api, key, value),
-        key
+        key,
       );
     }
   }
@@ -196,11 +196,15 @@ export abstract class BaseScormAdapter<TApi> implements PersistenceAdapter {
     this.#terminated = true;
     // Async retries can't run during page unload — drain + commit + finish synchronously.
     this.queue.drainSync();
-    callSyncOrWarn(() => this.dialect.commit(this.api), 'Commit', this.errorReporter);
+    callSyncOrWarn(
+      () => this.dialect.commit(this.api),
+      'Commit',
+      this.errorReporter,
+    );
     callSyncOrWarn(
       () => this.dialect.terminate(this.api),
       'Terminate',
-      this.errorReporter
+      this.errorReporter,
     );
   }
 

package/src/runtime/adapters/scorm12.ts

@@ -4,7 +4,7 @@ import {
 } from '../interaction-format.js';
 import type { SavedState } from '../persistence.js';
 import { BaseScormAdapter, type ScormDialect } from './scorm-base.js';
-import { formatHHMMSS, formatReal107 } from './retry.js';
+import { formatHHMMSS, formatReal107 } from './format.js';
 
 /**
  * SCORM 1.2 API interface.
@@ -67,24 +67,23 @@ export class SCORM12Adapter extends BaseScormAdapter<SCORM12API> {
     super.saveState(state);
     // §3.4.5.3 — bookmark for LMS "Resume from page N" affordances.
     this.queue.enqueue(
-      () =>
-        this.api.LMSSetValue('cmi.core.lesson_location', String(state.b)),
-      'cmi.core.lesson_location'
+      () => this.api.LMSSetValue('cmi.core.lesson_location', String(state.b)),
+      'cmi.core.lesson_location',
     );
   }
 
   setScore(score: number): void {
     this.queue.enqueue(
       () => this.api.LMSSetValue('cmi.core.score.raw', formatReal107(score)),
-      'cmi.core.score.raw'
+      'cmi.core.score.raw',
     );
     this.queue.enqueue(
       () => this.api.LMSSetValue('cmi.core.score.min', '0'),
-      'cmi.core.score.min'
+      'cmi.core.score.min',
     );
     this.queue.enqueue(
       () => this.api.LMSSetValue('cmi.core.score.max', '100'),
-      'cmi.core.score.max'
+      'cmi.core.score.max',
     );
   }
 
@@ -104,7 +103,7 @@ export class SCORM12Adapter extends BaseScormAdapter<SCORM12API> {
     const value = this.#successStatus ?? this.#completionStatus;
     this.queue.enqueue(
       () => this.api.LMSSetValue('cmi.core.lesson_status', value),
-      'cmi.core.lesson_status'
+      'cmi.core.lesson_status',
     );
   }
 
@@ -113,7 +112,7 @@ export class SCORM12Adapter extends BaseScormAdapter<SCORM12API> {
     const value = mode === 'suspend' ? 'suspend' : '';
     this.queue.enqueue(
       () => this.api.LMSSetValue('cmi.core.exit', value),
-      'cmi.core.exit'
+      'cmi.core.exit',
     );
   }
 }

package/src/runtime/adapters/scorm2004.ts

@@ -5,7 +5,7 @@ import {
   formatISO8601Duration,
   formatISO8601Timestamp,
   formatReal107,
-} from './retry.js';
+} from './format.js';
 
 export interface SCORM2004API {
   Initialize(param: string): string;
@@ -48,13 +48,11 @@ export type SCORM2004Mode = 'browse' | 'normal' | 'review';
 /**
  * Per §4.2.1.5, the SCO MUST NOT alter the learner record in `browse` or
  * `review` mode — every write below is gated on `#mode === 'normal'`.
- * `#masteryScore` (§4.2.4.3) and `#completionThreshold` (§4.2.4.4) are
- * LMS-supplied thresholds in [0,1].
+ * `#masteryScore` (§4.2.4.3) is the LMS-supplied pass threshold in [0,1].
  */
 export class SCORM2004Adapter extends BaseScormAdapter<SCORM2004API> {
   #mode: SCORM2004Mode = 'normal';
   #masteryScore: number | null = null;
-  #completionThreshold: number | null = null;
 
   constructor(api: SCORM2004API) {
     super(api, SCORM2004_DIALECT);
@@ -64,9 +62,6 @@ export class SCORM2004Adapter extends BaseScormAdapter<SCORM2004API> {
     await super.init();
     this.#mode = this.#readMode();
     this.#masteryScore = this.#readScaledThreshold('cmi.scaled_passing_score');
-    this.#completionThreshold = this.#readScaledThreshold(
-      'cmi.completion_threshold'
-    );
   }
 
   getLaunchMode(): SCORM2004Mode {
@@ -78,8 +73,8 @@ export class SCORM2004Adapter extends BaseScormAdapter<SCORM2004API> {
     return this.#masteryScore;
   }
 
-  getCompletionThreshold(): number | null {
-    return this.#completionThreshold;
+  get #canWrite(): boolean {
+    return this.#mode === 'normal';
   }
 
   #readMode(): SCORM2004Mode {
@@ -91,7 +86,7 @@ export class SCORM2004Adapter extends BaseScormAdapter<SCORM2004API> {
   }
 
   #readScaledThreshold(key: string): number | null {
-    let raw = '';
+    let raw: string;
     try {
       raw = this.api.GetValue(key);
     } catch {
@@ -104,83 +99,80 @@ export class SCORM2004Adapter extends BaseScormAdapter<SCORM2004API> {
   }
 
   saveState(state: SavedState): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     super.saveState(state);
     // §4.2.1.4 — bookmark for LMS "Resume from page N" affordances.
     this.queue.enqueue(
       () => this.api.SetValue('cmi.location', String(state.b)),
-      'cmi.location'
+      'cmi.location',
     );
   }
 
   setDuration(seconds: number): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     super.setDuration(seconds);
   }
 
   reportInteraction(
     questionId: string,
     interaction: import('../interaction.js').Interaction,
-    correct: boolean | null
+    correct: boolean | null,
   ): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     super.reportInteraction(questionId, interaction, correct);
   }
 
   setScore(score: number): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     const raw = formatReal107(score);
     // §4.2.4.3.5 — score.scaled is bounded to [-1, 1].
     const scaled = formatReal107(Math.max(0, Math.min(1, score / 100)));
     this.queue.enqueue(
       () => this.api.SetValue('cmi.score.raw', raw),
-      'cmi.score.raw'
+      'cmi.score.raw',
     );
     this.queue.enqueue(
       () => this.api.SetValue('cmi.score.min', '0'),
-      'cmi.score.min'
+      'cmi.score.min',
     );
     this.queue.enqueue(
       () => this.api.SetValue('cmi.score.max', '100'),
-      'cmi.score.max'
+      'cmi.score.max',
     );
     this.queue.enqueue(
       () => this.api.SetValue('cmi.score.scaled', scaled),
-      'cmi.score.scaled'
+      'cmi.score.scaled',
     );
   }
 
   setCompletionStatus(status: 'incomplete' | 'complete'): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     const value = status === 'complete' ? 'completed' : 'incomplete';
     this.queue.enqueue(
       () => this.api.SetValue('cmi.completion_status', value),
-      'cmi.completion_status'
+      'cmi.completion_status',
     );
     // §4.2.4.2 — writing 1.0 surfaces a "100%" reading on LMS dashboards.
     if (status === 'complete') {
       this.queue.enqueue(
         () => this.api.SetValue('cmi.progress_measure', '1'),
-        'cmi.progress_measure'
+        'cmi.progress_measure',
       );
     }
   }
 
   setSuccessStatus(status: 'passed' | 'failed' | 'unknown'): void {
-    if (this.#mode !== 'normal') return;
+    if (!this.#canWrite) return;
     // Setting "unknown" explicitly prevents SCORM Cloud from rolling up
     // a null status to "passed".
     this.queue.enqueue(
       () => this.api.SetValue('cmi.success_status', status),
-      'cmi.success_status'
+      'cmi.success_status',
     );
   }
 
   setExit(mode: 'suspend' | 'normal'): void {
-    if (this.#mode !== 'normal') return;
-    this.queue.enqueue(
-      () => this.api.SetValue('cmi.exit', mode),
-      'cmi.exit'
-    );
+    if (!this.#canWrite) return;
+    this.queue.enqueue(() => this.api.SetValue('cmi.exit', mode), 'cmi.exit');
   }
 }

package/src/runtime/adapters/web.ts

@@ -51,7 +51,7 @@ export class WebAdapter implements PersistenceAdapter {
   reportInteraction(
     _questionId: string,
     _interaction: Interaction,
-    _correct: boolean | null
+    _correct: boolean | null,
   ): void {
     // Web adapter has no external LMS; learner interaction data lives only
     // in memory. Authors who want to persist per-question state can use