npm - tessera-learn - Versions diffs - 0.2.2 → 0.3.0 - Mend

tessera-learn 0.2.2 → 0.3.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (48) hide show

package/AGENTS.md +161 -535
package/README.md +2 -2
package/dist/{audit-B9VHgVjk.js → audit-DkXqQTqn.js} +92 -38
package/dist/audit-DkXqQTqn.js.map +1 -0
package/dist/{build-commands-D127jw0J.js → build-commands-CyzuCDXg.js} +2 -2
package/dist/{build-commands-D127jw0J.js.map → build-commands-CyzuCDXg.js.map} +1 -1
package/dist/{inline-config-eHjv9XuA.js → inline-config-BEXyRqsJ.js} +2 -2
package/dist/{inline-config-eHjv9XuA.js.map → inline-config-BEXyRqsJ.js.map} +1 -1
package/dist/plugin/cli.d.ts.map +1 -1
package/dist/plugin/cli.js +62 -54
package/dist/plugin/cli.js.map +1 -1
package/dist/plugin/index.d.ts +280 -3
package/dist/plugin/index.d.ts.map +1 -1
package/dist/plugin/index.js +3 -3
package/dist/{plugin--8H9xQIl.js → plugin-CFUFgwHB.js} +126 -83
package/dist/plugin-CFUFgwHB.js.map +1 -0
package/package.json +12 -9
package/src/components/DefaultLayout.svelte +2 -5
package/src/components/Quiz.svelte +18 -26
package/src/plugin/a11y/audit.ts +8 -13
package/src/plugin/a11y-cli.ts +1 -4
package/src/plugin/ast.ts +9 -2
package/src/plugin/cli.ts +46 -48
package/src/plugin/csp.ts +59 -0
package/src/plugin/duplicate-cli.ts +37 -1
package/src/plugin/export.ts +56 -27
package/src/plugin/index.ts +117 -61
package/src/plugin/manifest.ts +3 -23
package/src/plugin/new-cli.ts +2 -0
package/src/plugin/validate-cli.ts +10 -4
package/src/plugin/validation.ts +48 -12
package/src/runtime/App.svelte +10 -8
package/src/runtime/Sidebar.svelte +3 -1
package/src/runtime/adapters/cmi5.ts +59 -402
package/src/runtime/adapters/discovery.ts +11 -0
package/src/runtime/adapters/index.ts +27 -60
package/src/runtime/adapters/lms-error.ts +61 -0
package/src/runtime/adapters/scorm2004.ts +2 -1
package/src/runtime/adapters/web.ts +19 -4
package/src/runtime/adapters/xapi-launch-base.ts +346 -0
package/src/runtime/adapters/xapi.ts +26 -0
package/src/runtime/types.ts +19 -1
package/src/runtime/xapi/publisher.ts +5 -1
package/src/runtime/xapi/setup.ts +24 -15
package/src/virtual.d.ts +4 -1
package/templates/course/course.config.js +1 -0
package/dist/audit-B9VHgVjk.js.map +0 -1
package/dist/plugin--8H9xQIl.js.map +0 -1

package/src/plugin/index.ts CHANGED Viewed

@@ -10,7 +10,11 @@ import {
   cpSync,
   mkdirSync,
 } from 'node:fs';
-import { generateManifest, readCourseConfig } from './manifest.js';
+import {
+  generateManifest,
+  readCourseConfig,
+  type CourseConfigRead,
+} from './manifest.js';
 import type { Manifest } from './manifest.js';
 import type { CourseConfig } from '../runtime/types.js';
 import {
@@ -25,6 +29,7 @@ import {
   isIgnored,
   type A11ySettings,
 } from './validation.js';
+import { buildCsp } from './csp.js';
 import { runExport } from './export.js';
 import { tesseraLayoutPlugin } from './layout.js';
 import { tesseraQuizPlugin } from './quiz.js';
@@ -188,9 +193,10 @@ function tesseraEntryPlugin(): Plugin {
     // For build mode: write index.html so Rollup can find it
     buildStart() {
       if (isBuild) {
+        const read = readCourseConfig(projectRoot);
         writeFileSync(
           resolve(projectRoot, 'index.html'),
-          generateIndexHtml(readLanguage(projectRoot)),
+          generateIndexHtml(readLanguage(read), cspMeta(read)),
           'utf-8',
         );
       }
@@ -221,7 +227,9 @@ function tesseraEntryPlugin(): Plugin {
       return () => {
         server.middlewares.use(async (req, res, next) => {
           if (req.url === '/' || req.url === '/index.html') {
-            const html = generateIndexHtml(readLanguage(projectRoot));
+            const html = generateIndexHtml(
+              readLanguage(readCourseConfig(projectRoot)),
+            );
             const transformed = await server.transformIndexHtml(req.url, html);
             res.setHeader('Content-Type', 'text/html');
             res.statusCode = 200;
@@ -252,18 +260,35 @@ function tesseraEntryPlugin(): Plugin {
 // 'en' fallback applied here: the config default-merge runs later than buildStart.
 // Only a validated BCP-47 tag is interpolated into <html lang>, so a malformed
 // value (caught separately as a warning) can't ship a broken attribute.
-function readLanguage(projectRoot: string): string {
-  const read = readCourseConfig(projectRoot);
+function readLanguage(read: CourseConfigRead): string {
   const lang = read.ok ? read.config.language : undefined;
   return isPlausibleLanguageTag(lang) ? lang : 'en';
 }
-function generateIndexHtml(lang: string): string {
+// Fail closed on an unreadable config: cspMeta only emits for exactly 'web', so
+// an unknown standard withholds the CSP rather than guess the one mode it breaks.
+function readExportStandard(read: CourseConfigRead): string {
+  if (!read.ok) return 'unknown';
+  return read.config.export?.standard || 'web';
+}
+// Web export only — never on LMS packages (whose iframe JS bridges a meta CSP
+// could break) and never on the dev server (a meta connect-src would block
+// Vite's HMR websocket). `export.csp` extends the baseline per-directive, or
+// `false` drops the meta for deployments that set a CSP header themselves.
+function cspMeta(read: CourseConfigRead): string {
+  if (readExportStandard(read) !== 'web') return '';
+  const csp = read.ok ? read.config.export?.csp : undefined;
+  if (csp === false) return '';
+  return `\n  <meta http-equiv="Content-Security-Policy" content="${buildCsp(csp)}" />`;
+}
+function generateIndexHtml(lang: string, csp = ''): string {
   return `<!DOCTYPE html>
 <html lang="${lang}">
 <head>
   <meta charset="UTF-8" />
-  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+  <meta name="viewport" content="width=device-width, initial-scale=1.0" />${csp}
   <title>Tessera Course</title>
 </head>
 <body>
@@ -326,6 +351,12 @@ function completionDefaults(mode: string | undefined): {
   if (mode === 'manual') {
     return { completion: { mode: 'manual' }, passingScore: 0 };
   }
+  if (mode === 'quiz') {
+    return {
+      completion: { mode: 'quiz' },
+      passingScore: DEFAULT_PASSING_SCORE,
+    };
+  }
   return {
     completion: {
       mode: 'percentage',
@@ -353,6 +384,21 @@ function tesseraConfigDefaultsPlugin(): Plugin {
   };
 }
+/** Fill runtime defaults into a parsed course.config.js. Exported for tests. */
+export function mergeCourseConfig(userConfig: Partial<CourseConfig>) {
+  const { completion, passingScore } = completionDefaults(
+    userConfig.completion?.mode,
+  );
+  return {
+    ...userConfig,
+    title: userConfig.title || 'Untitled Course',
+    navigation: { mode: 'free', ...userConfig.navigation },
+    completion: { ...completion, ...userConfig.completion },
+    scoring: { passingScore, ...userConfig.scoring },
+    export: { standard: 'web', ...userConfig.export },
+  };
+}
 function tesseraConfigPlugin(): Plugin {
   return virtualModule(
     'tessera:config',
@@ -362,18 +408,7 @@ function tesseraConfigPlugin(): Plugin {
       if (existsSync(configPath)) this.addWatchFile(configPath);
       const read = readCourseConfig(projectRoot);
       const userConfig: Partial<CourseConfig> = read.ok ? read.config : {};
-      const { completion, passingScore } = completionDefaults(
-        userConfig.completion?.mode,
-      );
-      const merged = {
-        title: userConfig.title || 'Untitled Course',
-        ...userConfig,
-        navigation: { mode: 'free', ...userConfig.navigation },
-        completion: { ...completion, ...userConfig.completion },
-        scoring: { passingScore, ...userConfig.scoring },
-        export: { standard: 'web', ...userConfig.export },
-      };
-      return `export default ${JSON.stringify(merged)};`;
+      return `export default ${JSON.stringify(mergeCourseConfig(userConfig))};`;
     },
   );
 }
@@ -611,7 +646,8 @@ function tesseraManifestPlugin(manifestRef: {
           value === Infinity ? 1e9 : value,
         );
         const b64 = Buffer.from(json).toString('base64');
-        return `export default JSON.parse(atob("${b64}"));`;
+        // atob yields Latin1 bytes; decode through UTF-8 or non-ASCII titles ship as mojibake.
+        return `export default JSON.parse(new TextDecoder().decode(Uint8Array.from(atob("${b64}"),(c)=>c.charCodeAt(0))));`;
       }
       return null;
     },
@@ -620,6 +656,55 @@ function tesseraManifestPlugin(manifestRef: {
 const VIRTUAL_ADAPTER_ID = 'virtual:tessera-adapter';
+// `takesApi`: SCORM detectors return the API object the constructor needs;
+// cmi5/xAPI ones return a boolean.
+const LMS_ADAPTER_GEN: Record<
+  'scorm12' | 'scorm2004' | 'cmi5' | 'xapi',
+  { adapter: string; module: string; detect: string; takesApi: boolean }
+> = {
+  scorm12: {
+    adapter: 'SCORM12Adapter',
+    module: 'scorm12',
+    detect: 'findSCORM12API',
+    takesApi: true,
+  },
+  scorm2004: {
+    adapter: 'SCORM2004Adapter',
+    module: 'scorm2004',
+    detect: 'findSCORM2004API',
+    takesApi: true,
+  },
+  cmi5: {
+    adapter: 'CMI5Adapter',
+    module: 'cmi5',
+    detect: 'hasCMI5LaunchParams',
+    takesApi: false,
+  },
+  xapi: {
+    adapter: 'XAPIAdapter',
+    module: 'xapi',
+    detect: 'hasXAPILaunchParams',
+    takesApi: false,
+  },
+};
+function generateLmsAdapterModule(
+  standard: keyof typeof LMS_ADAPTER_GEN,
+): string {
+  const { adapter, module, detect, takesApi } = LMS_ADAPTER_GEN[standard];
+  const guard = takesApi
+    ? `const api = ${detect}();\n  if (!api) throw missingApiError('${standard}');\n  return new ${adapter}(api);`
+    : `if (!${detect}()) throw missingApiError('${standard}');\n  return new ${adapter}();`;
+  return `
+import { ${adapter} } from 'tessera-learn/runtime/adapters/${module}.js';
+import { ${detect} } from 'tessera-learn/runtime/adapters/discovery.js';
+import { missingApiError } from 'tessera-learn/runtime/adapters/lms-error.js';
+export function createAdapter() {
+  ${guard}
+}
+`;
+}
 function tesseraAdapterPlugin(): Plugin {
   return virtualModule(
     'tessera:adapter',
@@ -641,47 +726,17 @@ function tesseraAdapterPlugin(): Plugin {
       // cmi5 adapters throw when their API is absent, so render with WebAdapter.
       if (isAuditBuild()) standard = 'web';
-      switch (standard) {
-        case 'scorm12':
-          return `
-import { SCORM12Adapter } from 'tessera-learn/runtime/adapters/scorm12.js';
-import { findSCORM12API } from 'tessera-learn/runtime/adapters/discovery.js';
-import { LMSAdapterError } from 'tessera-learn/runtime/adapters/index.js';
-export function createAdapter() {
-  const api = findSCORM12API();
-  if (!api) throw new LMSAdapterError('scorm12', 'Tessera: SCORM 1.2 API not found in window.parent/opener chain. Course must be launched from a SCORM 1.2 LMS.');
-  return new SCORM12Adapter(api);
-}
-`;
-        case 'scorm2004':
-          return `
-import { SCORM2004Adapter } from 'tessera-learn/runtime/adapters/scorm2004.js';
-import { findSCORM2004API } from 'tessera-learn/runtime/adapters/discovery.js';
-import { LMSAdapterError } from 'tessera-learn/runtime/adapters/index.js';
-export function createAdapter() {
-  const api = findSCORM2004API();
-  if (!api) throw new LMSAdapterError('scorm2004', 'Tessera: SCORM 2004 API not found in window.parent/opener chain. Course must be launched from a SCORM 2004 LMS.');
-  return new SCORM2004Adapter(api);
-}
-`;
-        case 'cmi5':
-          return `
-import { CMI5Adapter } from 'tessera-learn/runtime/adapters/cmi5.js';
-import { hasCMI5LaunchParams } from 'tessera-learn/runtime/adapters/discovery.js';
-import { LMSAdapterError } from 'tessera-learn/runtime/adapters/index.js';
-export function createAdapter() {
-  if (!hasCMI5LaunchParams()) throw new LMSAdapterError('cmi5', 'Tessera: cmi5 launch parameters not present on URL. Course must be launched from a cmi5-compliant LMS.');
-  return new CMI5Adapter();
-}
-`;
-        default:
-          return `
+      if (standard in LMS_ADAPTER_GEN) {
+        return generateLmsAdapterModule(
+          standard as keyof typeof LMS_ADAPTER_GEN,
+        );
+      }
+      return `
 import { WebAdapter } from 'tessera-learn/runtime/adapters/web.js';
-export function createAdapter(config) {
-  return new WebAdapter(config);
+export function createAdapter(config, options) {
+  return new WebAdapter(config, options && options.manifest);
 }
 `;
-      }
     },
   );
 }
@@ -711,9 +766,10 @@ function tesseraXAPISetupPlugin(): Plugin {
         hasXapi = read.config.xapi != null;
       }
-      // cmi5 needs the publisher regardless of explicit xapi config (cmi5
-      // adapter shares the publisher queue for its own LMS-required statements).
-      if (hasXapi || standard === 'cmi5') {
+      // The launch standards (cmi5, plain xAPI) own a publisher the runtime
+      // can share for `endpoint: 'lms'`, so wire the client regardless of
+      // explicit xapi config.
+      if (hasXapi || standard === 'cmi5' || standard === 'xapi') {
         return `export { buildXAPIClient } from 'tessera-learn/runtime/xapi/setup.js';`;
       }

package/src/plugin/manifest.ts CHANGED Viewed

@@ -90,23 +90,7 @@ export function deriveSlug(name: string, isFile = false): string {
   return stripPrefix(name);
 }
-export type DefaultExportLiteralResult =
-  | { kind: 'literal'; text: string }
-  | { kind: 'none' }
-  | { kind: 'invalid' }
-  | { kind: 'parse-error' };
-/**
- * Locate `export default { ... }` and return its object-literal text. Returns
- * a discriminated result so callers can tell parse failure from a missing or
- * non-literal default export. Used by both manifest extraction and project
- * validation.
- */
-export function extractDefaultExportObjectLiteral(
-  source: string,
-): DefaultExportLiteralResult {
-  return defaultExportObjectLiteral(source);
-}
+export { defaultExportObjectLiteral as extractDefaultExportObjectLiteral } from './ast.js';
 export type CourseConfigRead =
   | { ok: true; config: Partial<CourseConfig> }
@@ -126,9 +110,7 @@ export type CourseConfigRead =
 export function readCourseConfig(projectRoot: string): CourseConfigRead {
   const configPath = resolve(projectRoot, 'course.config.js');
   if (!existsSync(configPath)) return { ok: false, reason: 'missing' };
-  const result = extractDefaultExportObjectLiteral(
-    readSourceFileCached(configPath),
-  );
+  const result = defaultExportObjectLiteral(readSourceFileCached(configPath));
   if (result.kind === 'parse-error')
     return { ok: false, reason: 'parse-error' };
   if (result.kind !== 'literal') return { ok: false, reason: 'no-export' };
@@ -150,9 +132,7 @@ export function readMetaFile(metaPath: string): {
 } {
   if (!existsSync(metaPath)) return {};
-  const result = extractDefaultExportObjectLiteral(
-    readSourceFileCached(metaPath),
-  );
+  const result = defaultExportObjectLiteral(readSourceFileCached(metaPath));
   if (result.kind !== 'literal') return {};
   try {

package/src/plugin/new-cli.ts CHANGED Viewed

@@ -1,4 +1,5 @@
 import { existsSync } from 'node:fs';
+import { randomUUID } from 'node:crypto';
 import { join, relative, resolve } from 'node:path';
 import { findWorkspaceRoot } from './course-root.js';
 import { validateProjectName, toTitleCase } from './project-name.js';
@@ -43,6 +44,7 @@ export function runNew(name: string | undefined, cwd: string): number {
   const templateDir = join(resolvePackageRoot(), 'templates', 'course');
   copyTemplate(templateDir, courseDir, {
     PROJECT_TITLE: toTitleCase(name),
+    COURSE_ID: `urn:uuid:${randomUUID()}`,
   });
   const rel = relative(workspaceRoot, courseDir);

package/src/plugin/validate-cli.ts CHANGED Viewed

@@ -1,6 +1,10 @@
+import { basename } from 'node:path';
 import { validateProject, reportValidationIssues } from './validation.js';
-export function runValidate(projectRoot: string): number {
+export function runValidate(
+  projectRoot: string,
+  { showA11yTip = true }: { showA11yTip?: boolean } = {},
+): number {
   const { errors, warnings } = validateProject(projectRoot);
   reportValidationIssues({ errors, warnings });
@@ -23,8 +27,10 @@ export function runValidate(projectRoot: string): number {
       '\x1b[32m[tessera]\x1b[0m Validation passed — no issues found.',
     );
   }
-  console.log(
-    '\x1b[2m[tessera] Static checks only. For a full runtime accessibility audit, run: pnpm exec tessera a11y\x1b[0m',
-  );
+  if (showA11yTip) {
+    console.log(
+      `\x1b[2m[tessera] Static checks only. For a full runtime accessibility audit, run: pnpm a11y ${basename(projectRoot)}\x1b[0m`,
+    );
+  }
   return 0;
 }

package/src/plugin/validation.ts CHANGED Viewed

@@ -24,8 +24,13 @@ import {
 } from '../runtime/xapi/agent-rules.js';
 import { httpOrigin } from '../runtime/xapi/derive-actor.js';
 import { shortIdentifier } from '../runtime/interaction-format.js';
-import { FEEDBACK_MODES, RETRY_MODES } from '../runtime/types.js';
+import {
+  FEEDBACK_MODES,
+  RETRY_MODES,
+  courseIdentity,
+} from '../runtime/types.js';
 import { contrastRatio } from './a11y/contrast.js';
+import { isCspOverrides } from './csp.js';
 import { isVideoEmbed } from '../components/video-embed.js';
 // ---------- Types ----------
@@ -143,6 +148,7 @@ export function reportValidationIssues({
 // Known top-level config fields
 const KNOWN_CONFIG_FIELDS = new Set([
   'title',
+  'id',
   'description',
   'author',
   'version',
@@ -168,7 +174,7 @@ export function isPlausibleLanguageTag(value: unknown): value is string {
 const VALID_NAV_MODES = ['free', 'sequential'];
 const VALID_COMPLETION_MODES = ['quiz', 'percentage', 'manual'];
-const VALID_EXPORT_STANDARDS = ['web', 'scorm12', 'scorm2004', 'cmi5'];
+const VALID_EXPORT_STANDARDS = ['web', 'scorm12', 'scorm2004', 'cmi5', 'xapi'];
 const VALID_MANUAL_TRIGGERS = ['page'];
 const VALID_REQUIRE_SUCCESS_STATUS = ['passed', 'failed'];
 // Derived from the runtime types (single source of truth) — widened to
@@ -235,6 +241,7 @@ export function validateProject(projectRoot: string): ValidationResult {
 interface ParsedConfig {
   title?: string;
+  id?: string;
   navigation?: { mode?: string };
   completion?: {
     mode?: string;
@@ -243,7 +250,7 @@ interface ParsedConfig {
     requireSuccessStatus?: string;
   };
   scoring?: { passingScore?: number };
-  export?: { standard?: string };
+  export?: { standard?: string; csp?: unknown };
   [key: string]: unknown;
 }
@@ -316,6 +323,20 @@ function parseConfig(
     );
   }
+  // Identity matters for web (storage key) and cmi5/xAPI (LRS activity id);
+  // SCORM identity is owned by the LMS, so only nudge for the others.
+  const standard = config.export?.standard;
+  const identityStandard =
+    standard === undefined ||
+    standard === 'web' ||
+    standard === 'cmi5' ||
+    standard === 'xapi';
+  if (identityStandard && !courseIdentity(config)) {
+    warnings.push(
+      `course.config.js: no "id" set — the web storage key and cmi5/xAPI activity id then share a fixed fallback that collides across courses. Add a unique id (e.g. "urn:uuid:…"); scaffolded courses include one.`,
+    );
+  }
   // Validate a11y config block
   if (config.a11y !== undefined) {
     validateA11yConfig(config.a11y, errors);
@@ -371,7 +392,21 @@ function parseConfig(
   if (config.export?.standard !== undefined) {
     if (!VALID_EXPORT_STANDARDS.includes(config.export.standard)) {
       errors.push(
-        `course.config.js: "export.standard" must be "web", "scorm12", "scorm2004", or "cmi5", got "${config.export.standard}"`,
+        `course.config.js: "export.standard" must be "web", "scorm12", "scorm2004", "cmi5", or "xapi", got "${config.export.standard}"`,
+      );
+    }
+  }
+  // Validate export.csp (web-only CSP extension)
+  if (config.export?.csp !== undefined) {
+    const csp = config.export.csp;
+    if (csp !== false && !isCspOverrides(csp)) {
+      warnings.push(
+        'course.config.js: "export.csp" must be false or an object of directive → string[]; ignoring it and using the baseline CSP',
+      );
+    } else if ((config.export.standard ?? 'web') !== 'web') {
+      warnings.push(
+        `course.config.js: "export.csp" is ignored when "export.standard" is "${config.export.standard}" (the CSP meta is web-export only)`,
       );
     }
   }
@@ -567,7 +602,7 @@ function validateXAPIConfig(
     ).length;
     if (lmsCount > 1) {
       errors.push(
-        "course.config.js: xapi has multiple entries with endpoint: 'lms' — only one cmi5 launch-inherited destination is allowed",
+        "course.config.js: xapi has multiple entries with endpoint: 'lms' — only one launch-inherited destination is allowed",
       );
     }
     // Warn on duplicate explicit endpoints.
@@ -630,14 +665,15 @@ function validateSingleXAPIEntry(
   }
   if (endpoint === 'lms') {
-    // Forbid under non-cmi5 export.
-    if (standard !== 'cmi5') {
+    // 'lms' inherits the LRS from the launch — only the launch-based
+    // standards (cmi5, plain xAPI) carry one.
+    if (standard !== 'cmi5' && standard !== 'xapi') {
       errors.push(
-        `course.config.js: ${label}.endpoint: 'lms' requires export.standard: 'cmi5' (you have "${standard}"). ` +
+        `course.config.js: ${label}.endpoint: 'lms' requires export.standard: 'cmi5' or 'xapi' (you have "${standard}"). ` +
           'Either change the export standard or specify an explicit LRS endpoint.',
       );
     }
-    // Forbid extra fields — everything is inherited from the cmi5 launch.
+    // Forbid extra fields — everything is inherited from the launch.
     const forbidden = [
       'auth',
       'actor',
@@ -648,7 +684,7 @@ function validateSingleXAPIEntry(
     for (const f of forbidden) {
       if (entry[f] !== undefined) {
         errors.push(
-          `course.config.js: ${label}.${f} must be omitted when ${label}.endpoint is 'lms' — it is inherited from the cmi5 launch.`,
+          `course.config.js: ${label}.${f} must be omitted when ${label}.endpoint is 'lms' — it is inherited from the launch.`,
         );
       }
     }
@@ -764,7 +800,7 @@ function validateSingleXAPIEntry(
         `course.config.js: ${label}.actorAccountHomePage is ignored when ${label}.actor is supplied explicitly.`,
       );
     }
-    if (standard === 'cmi5' || standard === 'web') {
+    if (standard === 'cmi5' || standard === 'xapi' || standard === 'web') {
       warnings.push(
         `course.config.js: ${label}.actorAccountHomePage is only used under scorm12/scorm2004 actor synthesis; ignored under "${standard}".`,
       );
@@ -794,7 +830,7 @@ function validateSingleXAPIEntry(
         `course.config.js: ${label}.registration must be a UUID v4, got "${String(registration)}"`,
       );
     }
-    if (standard !== 'cmi5') {
+    if (standard !== 'cmi5' && standard !== 'xapi') {
       warnings.push(
         `course.config.js: ${label}.registration is a cmi5 concept; the LRS will accept it under "${standard}" but most analytics tools won't know what to do with it.`,
       );

package/src/runtime/App.svelte CHANGED Viewed

@@ -24,7 +24,7 @@
   } from './contexts.js';
   // ---- Persistence ----
-  const adapter = createAdapter(config);
+  const adapter = createAdapter(config, { manifest });
   let persistenceReady = $state(false);
   // Holds the resolved xAPI client for unload-time markUnloading. Set
   // after adapter.init() resolves and registered globally so useXAPI()
@@ -209,10 +209,12 @@
       v: [...progress.visitedPages],
       q,
       d: duration.totalSeconds,
-      c,
-      s,
-      gs: [...progress.gradedStandalonePages],
-      u: { ...userState },
+      ...(progress.chunkProgress.size > 0 ? { c } : {}),
+      ...(progress.standaloneQuestionScores.size > 0 ? { s } : {}),
+      ...(progress.gradedStandalonePages.size > 0
+        ? { gs: [...progress.gradedStandalonePages] }
+        : {}),
+      ...(Object.keys(userState).length > 0 ? { u: { ...userState } } : {}),
       ...(progress.manuallyCompleted ? { m: 1 } : {}),
     };
   }
@@ -297,7 +299,7 @@
   // ---- Persistence: report score/completion/success to adapter ----
   // These are no-ops for WebAdapter but used by LMS adapters (Step 10)
-  let prevReportedScore = $state(null);
+  let prevReportedScore = null;
   $effect(() => {
     void progress.version;
     if (!persistenceReady) return;
@@ -322,7 +324,7 @@
     });
   });
-  let prevCompletionStatus = $state('incomplete');
+  let prevCompletionStatus = 'incomplete';
   $effect(() => {
     const status = progress.completionStatus;
     if (!persistenceReady) return;
@@ -335,7 +337,7 @@
     });
   });
-  let prevSuccessStatus = $state('unknown');
+  let prevSuccessStatus = 'unknown';
   $effect(() => {
     const status = progress.successStatus;
     if (!persistenceReady) return;

package/src/runtime/Sidebar.svelte CHANGED Viewed

@@ -1,9 +1,11 @@
 <script>
+  import { SvelteSet } from 'svelte/reactivity';
   let { manifest, config, currentPageIndex, nav, onnavigate, onclose } =
     $props();
   // Track which sections are collapsed. All expanded by default.
-  let collapsedSections = $state(new Set());
+  const collapsedSections = new SvelteSet();
   function toggleSection(slug) {
     if (collapsedSections.has(slug)) {