lens-content-processor 0.10.0 → 0.12.0

package/dist/content-schema.js

@@ -65,6 +65,16 @@ export const SEGMENT_SCHEMAS = {
         "optional",
         "feedback",
     ], ["user-customizable", "optional", "feedback"]),
+    embed: segmentSchema([], [
+        "source",
+        "from",
+        "to",
+        "height",
+        "width",
+        "aspect-ratio",
+        "sandbox",
+        "optional",
+    ], ["optional"]),
 };
 /**
  * Valid fields per segment type, derived from SEGMENT_SCHEMAS.
@@ -83,6 +93,8 @@ const SECTION_LEVEL_FIELDS = [
     "learningOutcomeId",
     "sourceUrl",
     "hide",
+    "notes",
+    "add_to_ai_context", // submodule-level (on `# Submodule:` markers)
 ];
 const SECTION_LEVEL_BOOLEAN_FIELDS = ["hide"];
 /**

package/dist/content-schema.js.map

@@ -1 +1 @@
-{"version":3,"file":"content-schema.js","sourceRoot":"","sources":["../src/content-schema.ts"],"names":[],"mappings":"AAAA,kGAAkG;AAsBlG,SAAS,aAAa,CACpB,QAAkB,EAClB,QAAkB;IAElB,OAAO;QACL,cAAc,EAAE,QAAQ;QACxB,cAAc,EAAE,QAAQ;QACxB,SAAS,EAAE,CAAC,GAAG,QAAQ,EAAE,GAAG,QAAQ,CAAC;KACtC,CAAC;AACJ,CAAC;AAED,SAAS,aAAa,CACpB,QAAkB,EAClB,QAAkB,EAClB,aAAuB;IAEvB,OAAO;QACL,cAAc,EAAE,QAAQ;QACxB,cAAc,EAAE,QAAQ;QACxB,SAAS,EAAE,CAAC,GAAG,QAAQ,EAAE,GAAG,QAAQ,CAAC;QACrC,aAAa;KACd,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,eAAe,GAAsC;IAChE,MAAM,EAAE,aAAa,CACnB,CAAC,MAAM,EAAE,OAAO,CAAC,EACjB,CAAC,WAAW,EAAE,IAAI,EAAE,YAAY,EAAE,MAAM,CAAC,CAC1C;IACD,MAAM,EAAE,aAAa,CACnB,CAAC,MAAM,EAAE,OAAO,CAAC,EACjB;QACE,IAAI;QACJ,cAAc;QACd,MAAM;QACN,cAAc;QACd,cAAc;QACd,oBAAoB;QACpB,aAAa;KACd,CACF;IACD,IAAI,EAAE,aAAa,CACjB,CAAC,IAAI,CAAC,EACN,CAAC,OAAO,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,mBAAmB,CAAC,CACpE;IACD,kBAAkB,EAAE,aAAa,CAC/B,CAAC,IAAI,CAAC,EACN,CAAC,YAAY,EAAE,kBAAkB,EAAE,MAAM,CAAC,CAC3C;IACD,OAAO,EAAE,aAAa,CACpB,CAAC,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAE,WAAW,CAAC,EAC9C;QACE,SAAS;QACT,aAAa;QACb,MAAM;QACN,KAAK;QACL,oBAAoB;QACpB,qBAAqB;KACtB,CACF;IACD,kBAAkB,EAAE,aAAa,CAC/B,CAAC,OAAO,EAAE,SAAS,EAAE,KAAK,CAAC,EAC3B,CAAC,MAAM,EAAE,qBAAqB,CAAC,CAChC;CACF,CAAC;AAEF,MAAM,CAAC,MAAM,eAAe,GAAsC;IAChE,IAAI,EAAE,aAAa,CAAC,CAAC,SAAS,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC;IAC5D,yEAAyE;IACzE,wEAAwE;IACxE,oEAAoE;IACpE,IAAI,EAAE,aAAa,CACjB,CAAC,cAAc,CAAC,EAChB;QACE,mBAAmB;QACnB,6BAA6B;QAC7B,8BAA8B;KAC/B,EACD,CAAC,6BAA6B,EAAE,8BAA8B,CAAC,CAChE;IACD,OAAO,EAAE,aAAa,CACpB,EAAE,EACF,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,CAAC,EACpC,CAAC,UAAU,CAAC,CACb;IACD,KAAK,EAAE,aAAa,CAAC,EAAE,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC;IAC5E,QAAQ,EAAE,aAAa,CACrB,CAAC,SAAS,CAAC,EACX;QACE,yBAAyB;QACzB,UAAU;QACV,WAAW;QACX,mBAAmB;QACnB,eAAe;QACf,UAAU;QACV,UAAU;KACX,EACD,CAAC,eAAe,EAAE,UAAU,EAAE,UAAU,CAAC,CAC1C;IACD,QAAQ,EAAE,aAAa,CACrB,CAAC,IAAI,EAAE,SAAS,EAAE,iBAAiB,CAAC,EACpC;QACE,iBAAiB;QACjB,yBAAyB;QACzB,mBAAmB;QACnB,UAAU;QACV,UAAU;KACX,EACD,CAAC,mBAAmB,EAAE,UAAU,EAAE,UAAU,CAAC,CAC9C;CACF,CAAC;AAEF;;;GAGG;AACH,MAAM,CAAC,MAAM,4BAA4B,GAGrC,MAAM,CAAC,WAAW,CACpB,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,EAAE,EAAE,CAAC;IACtD,IAAI;IACJ,IAAI,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC;CAC1B,CAAC,CACH,CAAC;AAEF;;;GAGG;AACH,MAAM,oBAAoB,GAAG;IAC3B,QAAQ;IACR,mBAAmB;IACnB,WAAW;IACX,MAAM;CACP,CAAC;AACF,MAAM,4BAA4B,GAAG,CAAC,MAAM,CAAC,CAAC;AAE9C;;;GAGG;AACH,MAAM,CAAC,MAAM,gBAAgB,GAAa,CAAC,GAAG,EAAE;IAC9C,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,SAAS;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC1D,CAAC;IACD,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,SAAS;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC1D,CAAC;IACD,KAAK,MAAM,KAAK,IAAI,oBAAoB;QAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC5D,OAAO,CAAC,GAAG,MAAM,CAAC,CAAC;AACrB,CAAC,CAAC,EAAE,CAAC;AAEL;;GAEG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAa,CAAC,GAAG,EAAE;IAChD,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,aAAa;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC9D,CAAC;IACD,KAAK,MAAM,KAAK,IAAI,4BAA4B;QAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IACpE,OAAO,CAAC,GAAG,MAAM,CAAC,CAAC;AACrB,CAAC,CAAC,EAAE,CAAC"}
+{"version":3,"file":"content-schema.js","sourceRoot":"","sources":["../src/content-schema.ts"],"names":[],"mappings":"AAAA,kGAAkG;AAsBlG,SAAS,aAAa,CACpB,QAAkB,EAClB,QAAkB;IAElB,OAAO;QACL,cAAc,EAAE,QAAQ;QACxB,cAAc,EAAE,QAAQ;QACxB,SAAS,EAAE,CAAC,GAAG,QAAQ,EAAE,GAAG,QAAQ,CAAC;KACtC,CAAC;AACJ,CAAC;AAED,SAAS,aAAa,CACpB,QAAkB,EAClB,QAAkB,EAClB,aAAuB;IAEvB,OAAO;QACL,cAAc,EAAE,QAAQ;QACxB,cAAc,EAAE,QAAQ;QACxB,SAAS,EAAE,CAAC,GAAG,QAAQ,EAAE,GAAG,QAAQ,CAAC;QACrC,aAAa;KACd,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,eAAe,GAAsC;IAChE,MAAM,EAAE,aAAa,CACnB,CAAC,MAAM,EAAE,OAAO,CAAC,EACjB,CAAC,WAAW,EAAE,IAAI,EAAE,YAAY,EAAE,MAAM,CAAC,CAC1C;IACD,MAAM,EAAE,aAAa,CACnB,CAAC,MAAM,EAAE,OAAO,CAAC,EACjB;QACE,IAAI;QACJ,cAAc;QACd,MAAM;QACN,cAAc;QACd,cAAc;QACd,oBAAoB;QACpB,aAAa;KACd,CACF;IACD,IAAI,EAAE,aAAa,CACjB,CAAC,IAAI,CAAC,EACN,CAAC,OAAO,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,mBAAmB,CAAC,CACpE;IACD,kBAAkB,EAAE,aAAa,CAC/B,CAAC,IAAI,CAAC,EACN,CAAC,YAAY,EAAE,kBAAkB,EAAE,MAAM,CAAC,CAC3C;IACD,OAAO,EAAE,aAAa,CACpB,CAAC,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAE,WAAW,CAAC,EAC9C;QACE,SAAS;QACT,aAAa;QACb,MAAM;QACN,KAAK;QACL,oBAAoB;QACpB,qBAAqB;KACtB,CACF;IACD,kBAAkB,EAAE,aAAa,CAC/B,CAAC,OAAO,EAAE,SAAS,EAAE,KAAK,CAAC,EAC3B,CAAC,MAAM,EAAE,qBAAqB,CAAC,CAChC;CACF,CAAC;AAEF,MAAM,CAAC,MAAM,eAAe,GAAsC;IAChE,IAAI,EAAE,aAAa,CAAC,CAAC,SAAS,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC;IAC5D,yEAAyE;IACzE,wEAAwE;IACxE,oEAAoE;IACpE,IAAI,EAAE,aAAa,CACjB,CAAC,cAAc,CAAC,EAChB;QACE,mBAAmB;QACnB,6BAA6B;QAC7B,8BAA8B;KAC/B,EACD,CAAC,6BAA6B,EAAE,8BAA8B,CAAC,CAChE;IACD,OAAO,EAAE,aAAa,CACpB,EAAE,EACF,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,CAAC,EACpC,CAAC,UAAU,CAAC,CACb;IACD,KAAK,EAAE,aAAa,CAAC,EAAE,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC;IAC5E,QAAQ,EAAE,aAAa,CACrB,CAAC,SAAS,CAAC,EACX;QACE,yBAAyB;QACzB,UAAU;QACV,WAAW;QACX,mBAAmB;QACnB,eAAe;QACf,UAAU;QACV,UAAU;KACX,EACD,CAAC,eAAe,EAAE,UAAU,EAAE,UAAU,CAAC,CAC1C;IACD,QAAQ,EAAE,aAAa,CACrB,CAAC,IAAI,EAAE,SAAS,EAAE,iBAAiB,CAAC,EACpC;QACE,iBAAiB;QACjB,yBAAyB;QACzB,mBAAmB;QACnB,UAAU;QACV,UAAU;KACX,EACD,CAAC,mBAAmB,EAAE,UAAU,EAAE,UAAU,CAAC,CAC9C;IACD,KAAK,EAAE,aAAa,CAClB,EAAE,EACF;QACE,QAAQ;QACR,MAAM;QACN,IAAI;QACJ,QAAQ;QACR,OAAO;QACP,cAAc;QACd,SAAS;QACT,UAAU;KACX,EACD,CAAC,UAAU,CAAC,CACb;CACF,CAAC;AAEF;;;GAGG;AACH,MAAM,CAAC,MAAM,4BAA4B,GAGrC,MAAM,CAAC,WAAW,CACpB,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,EAAE,EAAE,CAAC;IACtD,IAAI;IACJ,IAAI,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC;CAC1B,CAAC,CACH,CAAC;AAEF;;;GAGG;AACH,MAAM,oBAAoB,GAAG;IAC3B,QAAQ;IACR,mBAAmB;IACnB,WAAW;IACX,MAAM;IACN,OAAO;IACP,mBAAmB,EAAE,8CAA8C;CACpE,CAAC;AACF,MAAM,4BAA4B,GAAG,CAAC,MAAM,CAAC,CAAC;AAE9C;;;GAGG;AACH,MAAM,CAAC,MAAM,gBAAgB,GAAa,CAAC,GAAG,EAAE;IAC9C,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,SAAS;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC1D,CAAC;IACD,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,SAAS;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC1D,CAAC;IACD,KAAK,MAAM,KAAK,IAAI,oBAAoB;QAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC5D,OAAO,CAAC,GAAG,MAAM,CAAC,CAAC;AACrB,CAAC,CAAC,EAAE,CAAC;AAEL;;GAEG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAa,CAAC,GAAG,EAAE;IAChD,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,KAAK,MAAM,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC;QACpD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,aAAa;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IAC9D,CAAC;IACD,KAAK,MAAM,KAAK,IAAI,4BAA4B;QAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IACpE,OAAO,CAAC,GAAG,MAAM,CAAC,CAAC;AACrB,CAAC,CAAC,EAAE,CAAC"}

package/dist/flattener/index.d.ts

@@ -10,6 +10,7 @@ export type BoundaryMarker = {
     __boundary: true;
     title: string;
     customSlug?: string;
+    addToAiContext?: string[];
 };
 export type FlatItem = Section | BoundaryMarker;
 export declare function isBoundary(item: FlatItem): item is BoundaryMarker;
@@ -21,6 +22,7 @@ interface VirtualModuleGroup {
     parentTitle: string;
     sections: Section[];
     contentId: string | null;
+    addToAiContext?: string[];
 }
 export declare function splitAtBoundaries(items: FlatItem[], parentSlug: string, parentTitle: string): VirtualModuleGroup[];
 /**

package/dist/flattener/index.js

@@ -86,6 +86,36 @@ function toKebabCase(str) {
         .replace(/[^a-z0-9]+/g, "-")
         .replace(/(^-|-$)/g, "");
 }
+/**
+ * Parse a `# Submodule:` marker's `add_to_ai_context::` field into raw wikilink
+ * strings (the same `[[...]]` form the backend resolves for every level).
+ * Returns undefined if the field is absent, [] if present with no wikilinks.
+ */
+function parseSubmoduleAiContext(fields) {
+    if (!("add_to_ai_context" in fields))
+        return undefined;
+    const raw = fields.add_to_ai_context ?? "";
+    return raw.match(/\[\[[^\]]+\]\]/g) ?? [];
+}
+/**
+ * Merge a module's add_to_ai_context with a submodule's own, deduped by exact
+ * wikilink string (first occurrence wins), module entries first. The backend
+ * treats this combined list as the section's module tier.
+ */
+function mergeAiContext(moduleCtx, submoduleCtx) {
+    const seen = new Set();
+    const out = [];
+    for (const entry of [
+        ...(Array.isArray(moduleCtx) ? moduleCtx : []),
+        ...submoduleCtx,
+    ]) {
+        if (seen.has(entry))
+            continue;
+        seen.add(entry);
+        out.push(entry);
+    }
+    return out;
+}
 export function validateBoundaries(items, file) {
     const errors = [];
     const hasBoundaries = items.some(isBoundary);
@@ -163,6 +193,7 @@ export function splitAtBoundaries(items, parentSlug, parentTitle) {
                 parentTitle,
                 sections: [],
                 contentId: null,
+                addToAiContext: item.addToAiContext,
             };
         }
         else if (currentGroup) {
@@ -261,6 +292,7 @@ export function flattenModule(modulePath, files, visitedPaths = new Set(), tierM
                 __boundary: true,
                 title: section.title,
                 customSlug: section.fields.slug,
+                addToAiContext: parseSubmoduleAiContext(section.fields),
             });
             // Process children
             if (section.children) {
@@ -271,14 +303,6 @@ export function flattenModule(modulePath, files, visitedPaths = new Set(), tierM
             }
         }
         else {
-            // Check if this is an LO with submodules
-            if (section.type === "learning-outcome") {
-                const loSections = processLOWithSubmodules(section, modulePath, files, visitedPaths, tierMap, errors);
-                if (loSections) {
-                    flatItems.push(...loSections);
-                    continue;
-                }
-            }
             const sections = processSection(section);
             flatItems.push(...sections);
         }
@@ -297,9 +321,14 @@ export function flattenModule(modulePath, files, visitedPaths = new Set(), tierM
             ? { parentSlug: g.parentSlug, parentTitle: g.parentTitle }
             : {}),
         ...(moduleError ? { error: moduleError } : {}),
-        ...("addToAiContext" in parsedModule
-            ? { addToAiContext: parsedModule.addToAiContext }
-            : {}),
+        // Module-level add_to_ai_context, with any submodule-level entries merged in
+        // (module first, deduped). When the submodule has no field, this is exactly
+        // the module's own value.
+        ...(g.addToAiContext !== undefined
+            ? {
+                addToAiContext: mergeAiContext(parsedModule.addToAiContext, g.addToAiContext),
+            }
+            : { addToAiContext: parsedModule.addToAiContext }),
     }));
     // Enrich ::card links in text segments with computed metadata (duration, attribution, displayType)
     for (const mod of resultModules) {
@@ -317,7 +346,6 @@ export function flattenModule(modulePath, files, visitedPaths = new Set(), tierM
 }
 /**
  * Build a test Section from a ParsedTestRef with inline segments.
- * Shared by both processLOWithSubmodules() and flattenLearningOutcomeSection().
  */
 function buildTestSection(testRef, loId, loName, loPath, files, visitedPaths, tierMap, errors) {
     if (!testRef.segments.length)
@@ -346,123 +374,27 @@ function buildTestSection(testRef, loId, loName, loPath, files, visitedPaths, ti
     };
 }
 /**
- * Check if a Learning Outcome section resolves to an LO with submodules.
- * If so, emit boundary markers + sections. Returns null if no submodules.
- */
-function processLOWithSubmodules(section, modulePath, files, visitedPaths, tierMap, errors) {
-    // Resolve the LO file to check for submodules
-    const source = section.fields.source;
-    if (!source)
-        return null;
-    const wikilink = parseWikilink(source);
-    if (!wikilink || wikilink.error)
-        return null;
-    const loPathResolved = resolveWikilinkPath(wikilink.path, modulePath);
-    const loPath = findFileWithExtension(loPathResolved, files);
-    if (!loPath)
-        return null;
-    const loContent = files.get(loPath);
-    if (!loContent)
-        return null;
-    const loResult = parseLearningOutcome(loContent, loPath);
-    if (!loResult.learningOutcome?.submodules)
-        return null;
-    // This LO has submodules — emit boundaries and resolve lenses
-    const items = [];
-    for (const sub of loResult.learningOutcome.submodules) {
-        items.push({
-            __boundary: true,
-            title: sub.title,
-            customSlug: sub.customSlug,
-        });
-        // Resolve each lens in this submodule group
-        for (const lensRef of sub.lenses) {
-            const lensPath = findFileWithExtension(lensRef.resolvedPath, files);
-            if (!lensPath) {
-                const similarFiles = findSimilarFiles(lensRef.resolvedPath, files, "Lenses");
-                const suggestion = formatSuggestion(similarFiles, loPath) ??
-                    "Check the file path in the wiki-link";
-                errors.push({
-                    file: loPath,
-                    message: `Referenced lens file not found: ${lensRef.resolvedPath}`,
-                    suggestion,
-                    severity: "error",
-                });
-                continue;
-            }
-            // Check tier violation
-            if (tierMap) {
-                const parentTier = tierMap.get(loPath) ?? "production";
-                const childTier = tierMap.get(lensPath) ?? "production";
-                const violation = checkTierViolation(loPath, parentTier, lensPath, childTier, "lens");
-                if (violation) {
-                    errors.push(violation);
-                }
-                if (childTier === "ignored")
-                    continue;
-            }
-            if (visitedPaths.has(lensPath)) {
-                errors.push({
-                    file: loPath,
-                    message: `Circular reference detected: ${lensPath}`,
-                    severity: "error",
-                });
-                continue;
-            }
-            const sectionVisitedPaths = new Set(visitedPaths);
-            sectionVisitedPaths.add(lensPath);
-            const lensContent = files.get(lensPath);
-            const lensResult = parseLens(lensContent, lensPath);
-            errors.push(...lensResult.errors);
-            if (!lensResult.lens)
-                continue;
-            const lens = lensResult.lens;
-            const { segments, errors: flattenErrors } = flattenSingleLens(lens, lensPath, files, sectionVisitedPaths, tierMap);
-            errors.push(...flattenErrors);
-            const lo = loResult.learningOutcome;
-            const effectiveOptional = section.fields.optional?.toLowerCase() === "true" || lensRef.optional;
-            const effectiveHide = section.fields.hide?.toLowerCase() === "true" || lensRef.hide;
-            const hideConflict = checkHideOptionalConflict(effectiveOptional, effectiveHide, loPath, section.line);
-            if (hideConflict)
-                errors.push(hideConflict);
-            items.push({
-                type: "lens",
-                meta: { title: lens.title || section.title },
-                segments,
-                sourcePath: lensPath,
-                optional: effectiveOptional,
-                ...(effectiveHide && { hide: true }),
-                learningOutcomeId: lo.id ?? null,
-                learningOutcomeName: loPath.split("/").pop()?.replace(/\.md$/i, "") ?? null,
-                contentId: lens.id ?? null,
-                tldr: lens.tldr,
-                summaryForTutor: lens.summaryForTutor,
-                minChatMessages: lens.minChatMessages,
-                addToAiContext: lens.addToAiContext,
-                addToAiContextLo: lo.addToAiContext,
-                ...computeSectionStats(segments),
-                displayType: computeDisplayType(segments),
-            });
-        }
-        // Process test section if present in this submodule
-        if (sub.test) {
-            const lo = loResult.learningOutcome;
-            const testSection = buildTestSection(sub.test, lo.id ?? null, loPath.split("/").pop()?.replace(/\.md$/i, "") ?? null, loPath, files, visitedPaths, tierMap, errors);
-            if (testSection)
-                items.push(testSection);
-        }
-    }
-    // Also emit LO parse errors
-    errors.push(...loResult.errors);
-    return items;
-}
-/**
- * Flatten a Learning Outcome section by resolving its LO file and all referenced lenses.
- * Each lens in the LO becomes its own section.
+ * Flatten a Learning Outcome section from a module file.
+ * In the current format an LO's lenses are suggestions only (imported
+ * explicitly by the module), so importing an LO yields just its test section.
  */
 function flattenLearningOutcomeSection(section, modulePath, files, visitedPaths, tierMap) {
     const errors = [];
     const sections = [];
+    // The old format cascaded these onto the LO's auto-imported lenses; the
+    // import now contributes only the test (always required), so flag them
+    // instead of silently ignoring them.
+    for (const deadField of ["optional", "hide"]) {
+        if (section.fields[deadField] !== undefined) {
+            errors.push({
+                file: modulePath,
+                line: section.line,
+                message: `'${deadField}::' on a Learning Outcome import has no effect — importing an outcome contributes only its test, which is always required`,
+                suggestion: "Remove the field here; set it on the individual lens imports instead",
+                severity: "warning",
+            });
+        }
+    }
     // Get the source wikilink
     const source = section.fields.source;
     if (!source) {
@@ -539,81 +471,9 @@ function flattenLearningOutcomeSection(section, modulePath, files, visitedPaths,
         return { sections: [], errors };
     }
     const lo = loResult.learningOutcome;
-    // Each lens becomes its own section
-    for (const lensRef of lo.lenses) {
-        const lensPath = findFileWithExtension(lensRef.resolvedPath, files);
-        if (!lensPath) {
-            const similarFiles = findSimilarFiles(lensRef.resolvedPath, files, "Lenses");
-            const suggestion = formatSuggestion(similarFiles, loPath) ??
-                "Check the file path in the wiki-link";
-            errors.push({
-                file: loPath,
-                message: `Referenced lens file not found: ${lensRef.resolvedPath}`,
-                suggestion,
-                severity: "error",
-            });
-            continue;
-        }
-        // Check tier violation (LO -> Lens)
-        if (tierMap) {
-            const parentTier = tierMap.get(loPath) ?? "production";
-            const childTier = tierMap.get(lensPath) ?? "production";
-            const violation = checkTierViolation(loPath, parentTier, lensPath, childTier, "lens");
-            if (violation) {
-                errors.push(violation);
-            }
-            if (childTier === "ignored") {
-                continue;
-            }
-        }
-        // Check for circular reference
-        if (visitedPaths.has(lensPath)) {
-            errors.push({
-                file: loPath,
-                message: `Circular reference detected: ${lensPath}`,
-                severity: "error",
-            });
-            continue;
-        }
-        visitedPaths.add(lensPath);
-        const lensContent = files.get(lensPath);
-        // Parse the lens
-        const lensResult = parseLens(lensContent, lensPath);
-        errors.push(...lensResult.errors);
-        if (!lensResult.lens) {
-            continue;
-        }
-        const lens = lensResult.lens;
-        // Flatten lens segments using shared helper
-        const { segments, errors: flattenErrors } = flattenSingleLens(lens, lensPath, files, visitedPaths, tierMap);
-        errors.push(...flattenErrors);
-        // Create a section for this lens
-        const effectiveOptional = section.fields.optional?.toLowerCase() === "true" || lensRef.optional;
-        const effectiveHide = section.fields.hide?.toLowerCase() === "true" || lensRef.hide;
-        const hideConflict = checkHideOptionalConflict(effectiveOptional, effectiveHide, loPath, section.line);
-        if (hideConflict)
-            errors.push(hideConflict);
-        const resultSection = {
-            type: "lens",
-            meta: { title: lens.title || section.title },
-            segments,
-            sourcePath: lensPath,
-            optional: effectiveOptional,
-            ...(effectiveHide && { hide: true }),
-            learningOutcomeId: lo.id ?? null,
-            learningOutcomeName: loPath.split("/").pop()?.replace(/\.md$/i, "") ?? null,
-            contentId: lens.id ?? null,
-            tldr: lens.tldr,
-            summaryForTutor: lens.summaryForTutor,
-            minChatMessages: lens.minChatMessages,
-            addToAiContext: lens.addToAiContext,
-            addToAiContextLo: lo.addToAiContext,
-            ...computeSectionStats(segments),
-            displayType: computeDisplayType(segments),
-        };
-        sections.push(resultSection);
-    }
-    // After lens processing, add test section if present with inline segments
+    // Suggested lenses are intentionally NOT resolved here: they are
+    // suggestions for course creators, validated standalone in processContent.
+    // Emit the test section if the LO has one with inline segments
     if (lo.test) {
         const testSection = buildTestSection(lo.test, lo.id ?? null, loPath.split("/").pop()?.replace(/\.md$/i, "") ?? null, loPath, files, visitedPaths, tierMap, errors);
         if (testSection)
@@ -1092,6 +952,105 @@ function convertSegment(parsedSegment, lensPath, files, visitedPaths, tierMap) {
                 segment.feedback = true;
             return { segment, errors };
         }
+        case "embed": {
+            if (!parsedSegment.source) {
+                errors.push({
+                    file: lensPath,
+                    message: "Embed segment missing source (should have been set via inheritance)",
+                    severity: "error",
+                });
+                return { segment: null, errors };
+            }
+            const wikilink = parseWikilink(parsedSegment.source);
+            if (!wikilink || wikilink.error) {
+                const suggestion = wikilink?.correctedPath
+                    ? `Did you mean '[[${wikilink.correctedPath}]]'?`
+                    : undefined;
+                errors.push({
+                    file: lensPath,
+                    message: `Invalid wikilink in embed source: ${parsedSegment.source}`,
+                    suggestion,
+                    severity: "error",
+                });
+                return { segment: null, errors };
+            }
+            const articlePathResolved = resolveWikilinkPath(wikilink.path, lensPath);
+            const articlePath = findFileWithExtension(articlePathResolved, files);
+            if (!articlePath) {
+                const similarFiles = findSimilarFiles(articlePathResolved, files, "articles");
+                const suggestion = formatSuggestion(similarFiles, lensPath) ??
+                    "Check the file path in the wiki-link";
+                errors.push({
+                    file: lensPath,
+                    message: `Referenced article file not found: ${articlePathResolved}`,
+                    suggestion,
+                    severity: "error",
+                });
+                return { segment: null, errors };
+            }
+            if (tierMap) {
+                const parentTier = tierMap.get(lensPath) ?? "production";
+                const childTier = tierMap.get(articlePath) ?? "production";
+                const violation = checkTierViolation(lensPath, parentTier, articlePath, childTier, "article");
+                if (violation) {
+                    errors.push(violation);
+                }
+                if (childTier === "ignored") {
+                    return { segment: null, errors };
+                }
+            }
+            if (visitedPaths.has(articlePath)) {
+                errors.push({
+                    file: lensPath,
+                    message: `Circular reference detected: ${articlePath}`,
+                    severity: "error",
+                });
+                return { segment: null, errors };
+            }
+            const articleContent = files.get(articlePath);
+            const articleFrontmatter = parseFrontmatter(articleContent, articlePath);
+            const fm = articleFrontmatter.frontmatter;
+            if (!fm.source_url) {
+                errors.push({
+                    file: lensPath,
+                    message: `Embed source article missing source_url: ${articlePath}`,
+                    suggestion: "Add source_url to the article frontmatter; embeds use it as the iframe URL",
+                    severity: "error",
+                });
+                return { segment: null, errors };
+            }
+            const excerptResult = extractArticleExcerpt(articleContent, parsedSegment.fromAnchor, parsedSegment.toAnchor, articlePath);
+            if (excerptResult.error) {
+                errors.push({ ...excerptResult.error, file: lensPath });
+                return { segment: null, errors };
+            }
+            const segment = {
+                type: "embed",
+                url: String(fm.source_url),
+                content: excerptResult.content,
+                sourceUrl: String(fm.source_url),
+                sourcePath: articlePath,
+            };
+            if (fm.title)
+                segment.title = fm.title;
+            if (fm.author) {
+                const raw = fm.author;
+                segment.author = Array.isArray(raw) ? raw.join(", ") : String(raw);
+            }
+            if (fm.published)
+                segment.published = String(fm.published);
+            if (parsedSegment.height)
+                segment.height = parsedSegment.height;
+            if (parsedSegment.width)
+                segment.width = parsedSegment.width;
+            if (parsedSegment.aspectRatio)
+                segment.aspectRatio = parsedSegment.aspectRatio;
+            if (parsedSegment.sandbox)
+                segment.sandbox = parsedSegment.sandbox;
+            if (parsedSegment.optional)
+                segment.optional = true;
+            return { segment, errors };
+        }
         default:
             return { segment: null, errors };
     }