tessera-learn 0.2.3 → 0.3.0

package/src/plugin/manifest.ts

@@ -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

@@ -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/validation.ts

@@ -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

@@ -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

@@ -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)) {