@aglyn/plugins-ai 1.0.0-beta.218 → 1.0.0-beta.220

package/src/lib/runtime/ai-doctrine-validators.d.ts

@@ -14,7 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { type AiBuildPlan } from '../model/ai-build-plan';
+import { type AiBuildPlan, type AiBuildPlanEmbed } from '../model/ai-build-plan';
 import { type AiPlanCapabilities } from '../model/ai-plan-capabilities';
 import type { AiSiteInventory } from '../model/ai-site-inventory';
 import { type AiNodeTreeContext, type AiNodeTreeResult } from './ai-node-tree';
@@ -100,8 +100,12 @@ export interface AiDoctrineTreeContext extends AiNodeTreeContext {
     } | null;
     /** Placed library assets by media id. */
     assets?: Record<string, AiAssetFacts>;
-    /** The plan named a third-party embed the brief asked for, and its cost. */
-    allowEmbeds?: boolean;
+    /**
+     * The third-party players the confirmed plan lists for this output
+     * (AGL-3433): a Video embed is admitted only where one is listed, playing
+     * the link the brief gave or none. Absent or empty admits none.
+     */
+    plannedEmbeds?: readonly Pick<AiBuildPlanEmbed, 'host' | 'url'>[];
     framing?: AiCopyFraming;
     /**
      * Whether the workspace keeps reusable components and saved forms
@@ -495,7 +499,8 @@ export declare function detectCutLines(repairs: readonly string[], written: Read
  * wraps one other container and adds nothing, no empty containers, no inline
  * style repeated where a token or a component prop would carry it, no font
  * beyond the theme's, images lazy below the first, video by poster rather
- * than autoplay, and no third-party embed the plan did not name.
+ * than autoplay, no third-party player the confirmed plan does not list, and
+ * none playing a video the brief did not give.
  */
 export declare function detectHeavyDocument(tree: AiDoctrineTree, outputKind: AiOutputKind, context?: AiDoctrineTreeContext): AiDoctrineViolation[];
 /**
@@ -733,6 +738,22 @@ export declare function detectPlanOverFreeWall(plan: AiBuildPlan, inventory: AiS
  * instead of reporting itself done.
  */
 export declare function detectPlanUnreadableRegions(plan: AiBuildPlan): AiDoctrineViolation[];
+/**
+ * Rule 16 (plan): a third-party player is planned where it can play
+ * (AGL-3433). A screen of the plan or a component it creates carries at most
+ * its kind's budget. A template renders for every record it serves and a
+ * layout around every page, so neither carries one, and a form, an email or a
+ * dataset has nowhere to. A link the plan gives is one its host plays.
+ */
+export declare function detectPlanEmbeds(plan: AiBuildPlan): AiDoctrineViolation[];
+/**
+ * Rule 16 (plan, against the brief): a player is planned only where the brief
+ * asks for one (AGL-3433). The plan quotes the brief's words, and those words
+ * are found in the brief and ask for a video; a link it gives is one the brief
+ * gives. The plan step holds this, being the one door that reads the brief
+ * beside the plan.
+ */
+export declare function aiPlanEmbedBriefViolations(plan: AiBuildPlan, brief: string): AiDoctrineViolation[];
 /**
  * Every plan rule, against the site inventory the plan was made from and,
  * where the job read them, what it may create there (AGL-3030). `null`

package/src/lib/runtime/ai-doctrine-validators.js

@@ -17,11 +17,12 @@ import { _ as _extends } from "@swc/helpers/_/_extends";
  */ import { sanitizeAuthorHtml } from "@aglyn/aglyn/app-utils/author-html";
 import { formatBytes, measureNodeMap, nodeMapBytes } from "@aglyn/aglyn/app-utils/measure-node-map";
 import { MEDIA_CDN_VARIANT_WIDTHS, parseMediaRef } from "@aglyn/aglyn/app-utils/media-ref";
-import { ESTIMATED_PAGE_TRANSFER_BYTES, FREE_AI_TASTE_CREDITS_PER_MONTH } from "@aglyn/aglyn/app-utils/plan-entitlements";
+import { ESTIMATED_PAGE_TRANSFER_BYTES } from "@aglyn/aglyn/app-utils/plan-entitlements";
+import { FREE_AI_TASTE_CREDITS_PER_MONTH } from "../plan-entitlements.js";
 import { REUSABLE_INSTANCE_COMPONENT_ID, REUSABLE_INSTANCE_PROP_VALUES_KEY } from "@aglyn/aglyn/app-utils/reusable-component-keys";
 import { parseBreakpointSpan } from "@aglyn/shared-data-enums/breakpoint-span";
 import { renderEmailHtml } from "@aglyn/shared-util-email/email-render";
-import { aiPlanCreateFor, aiPlanUndeclaredRefs, isAiPlanNewRef } from "../model/ai-build-plan.js";
+import { AI_BUILD_PLAN_CREATION_NOUNS, AI_BUILD_PLAN_EMBED_HOST_NAMES, AI_PLAN_ASKS_FOR_VIDEO, aiEmbedHostOf, aiEmbedVideoKey, aiPlanCreateFor, aiPlanSlugKey, aiPlanUndeclaredRefs, isAiPlanNewRef } from "../model/ai-build-plan.js";
 import { aiCreationNoun, aiPlanUncreatable, aiPlanUncreatableKind } from "../model/ai-plan-capabilities.js";
 import { AI_INSTANCE_REF_PROP, validateAiNodeTree } from "./ai-node-tree.js";
 import { AI_PAGE_SCROLL_TO_PROP, aiPageLinkTarget, aiStoredScrollTargets } from "./ai-page-links.js";
@@ -1421,7 +1422,7 @@ const GRID = 'muiGrid';
         {
             rule: 10,
             code: 'link-unrelated-screen',
-            message: `"${sent[0].label}" links the home page, which does not do what its words say. Link the screen that does, or leave the link out when the site has none.`,
+            message: `"${sent[0].label}" links the home page, which does not do what its words say. Link the page that does, or leave the link out when the site has none.`,
             nodeIds: unique(sent.map((entry)=>entry.id))
         }
     ];
@@ -1505,7 +1506,7 @@ const GRID = 'muiGrid';
         violations.push({
             rule: 10,
             code: 'link-fragment',
-            message: `${wordsOf(first)} links "${first.value}", an anchor, and no element on a page carries an id an anchor could name, so it goes nowhere. ${context.pageSections ? `${toSection} Otherwise give it the "screenId" of a screen the site has that does what its words say, or take it out.` : 'Give it the "screenId" of a screen the site has that does what its words say, or an "href" that is a path on this site or an https: address the brief gives, or take it out.'}`,
+            message: `${wordsOf(first)} links "${first.value}", an anchor, and no element on a page carries an id an anchor could name, so it goes nowhere. ${context.pageSections ? `${toSection} Otherwise give it the "screenId" of a page the site has that does what its words say, or take it out.` : 'Give it the "screenId" of a page the site has that does what its words say, or an "href" that is a path on this site or an https: address the brief gives, or take it out.'}`,
             nodeIds: unique(fragments.map((entry)=>entry.id))
         });
     }
@@ -1524,7 +1525,7 @@ const GRID = 'muiGrid';
         violations.push({
             rule: 10,
             code: 'link-without-destination',
-            message: `${wordsOf(first)} goes nowhere. Give it the "screenId" of a screen the site has that does what its words say, or an "href" that is a path on this site or an https: address the brief gives.${instead}`,
+            message: `${wordsOf(first)} goes nowhere. Give it the "screenId" of a page the site has that does what its words say, or an "href" that is a path on this site or an https: address the brief gives.${instead}`,
             nodeIds: unique(dead.map((entry)=>entry.id))
         });
     }
@@ -1884,12 +1885,25 @@ const EMBED_COMPONENTS = new Set([
 ]);
 const DUPLICATE_SX_MIN_KEYS = 2;
 const DUPLICATE_SX_MIN_NODES = 4;
+/**
+ * Whether a third-party player is one the confirmed plan lists (AGL-3433). A
+ * Video embed is admitted where the plan lists a player, playing the video
+ * the brief gave or none yet, which the site owner pastes. Raw HTML never is.
+ */ function embedVerdict(node, planned) {
+    var _node_props;
+    if (node.componentId !== 'videoEmbed' || !planned.length) return 'unplanned';
+    const url = typeof ((_node_props = node.props) == null ? void 0 : _node_props['url']) === 'string' ? node.props['url'].trim() : '';
+    if (!url) return 'planned';
+    const key = aiEmbedVideoKey(url);
+    return key && planned.some((embed)=>embed.url && aiEmbedVideoKey(embed.url) === key) ? 'planned' : 'unplanned-link';
+}
 /**
  * Rule 16. The flattest tree that renders the design: no container that
  * wraps one other container and adds nothing, no empty containers, no inline
  * style repeated where a token or a component prop would carry it, no font
  * beyond the theme's, images lazy below the first, video by poster rather
- * than autoplay, and no third-party embed the plan did not name.
+ * than autoplay, no third-party player the confirmed plan does not list, and
+ * none playing a video the brief did not give.
  */ export function detectHeavyDocument(tree, outputKind, context = {}) {
     const visits = walkTree(tree);
     const wrappers = [];
@@ -1898,6 +1912,7 @@ const DUPLICATE_SX_MIN_NODES = 4;
     const eager = [];
     const video = [];
     const embeds = [];
+    const embedLinks = [];
     const sxSeen = new Map();
     let images = 0;
     for (const { id, node } of visits){
@@ -1920,7 +1935,12 @@ const DUPLICATE_SX_MIN_NODES = 4;
         if (node.componentId === 'video' && (((_node_props2 = node.props) == null ? void 0 : _node_props2['autoPlay']) === true || ((_node_props3 = node.props) == null ? void 0 : _node_props3['preload']) === 'auto' || typeof ((_node_props4 = node.props) == null ? void 0 : _node_props4['src']) === 'string' && node.props['src'] && !((_node_props5 = node.props) == null ? void 0 : _node_props5['poster']))) {
             video.push(id);
         }
-        if (EMBED_COMPONENTS.has(node.componentId) && !context.allowEmbeds) embeds.push(id);
+        if (EMBED_COMPONENTS.has(node.componentId)) {
+            var _context_plannedEmbeds;
+            const verdict = embedVerdict(node, (_context_plannedEmbeds = context.plannedEmbeds) != null ? _context_plannedEmbeds : []);
+            if (verdict === 'unplanned') embeds.push(id);
+            if (verdict === 'unplanned-link') embedLinks.push(id);
+        }
         const sx = (_node_sx1 = node.sx) != null ? _node_sx1 : {};
         if (Object.keys(sx).length >= DUPLICATE_SX_MIN_KEYS) {
             var _sxSeen_get;
@@ -1952,7 +1972,8 @@ const DUPLICATE_SX_MIN_NODES = 4;
     add('extra-font', fonts, "This names a font the theme does not load. Use the theme's typography.");
     add('eager-image', eager, 'An image below the first loads eagerly. Leave loading unset so images further down the page load when they are reached.');
     add('autoplay-video', video, 'A video loads its file before anyone plays it. Give it a poster, and let it play on click.');
-    add('third-party-embed', embeds, 'This embeds a third-party player or script, which loads its own code on every visit. Use a Video from the media library, or name the embed and its cost in the plan.');
+    add('third-party-embed', embeds, "This embeds a third-party player, which loads its host's code on every visit, and the confirmed plan lists none here. Remove it; a film from the media library plays in a Video (video).");
+    add('embed-link-unplanned', embedLinks, 'This player plays a video the brief did not give. Use the link the plan lists, or leave the link empty for the site owner to paste.');
     return violations;
 }
 /** The items a list or a row of cards repeats, as a re-ask names them. */ const ITEM_NOUNS = {
@@ -2513,7 +2534,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 1,
             code: 'plan-section-across-screens',
-            message: `The "${entries[0].section.name}" section is planned on ${screens.size} screens. Plan it as one reusable component and place it on each.`,
+            message: `The "${entries[0].section.name}" section is planned on ${screens.size} pages. Plan it as one reusable component and place it on each.`,
             paths: entries.map((entry)=>entry.path)
         });
     }
@@ -2638,7 +2659,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 2,
             code: 'plan-screen-without-layout',
-            message: noLayoutToName ? 'A screen names a layout the site does not have, and this job may not create one. Leave the layout empty.' : mayCreateLayout ? "A screen names no layout the site has or the plan creates. Put every screen in the site's layout, or plan one." : 'A screen names no layout the site has, and this job may not create one. Put every screen in a layout the site has.',
+            message: noLayoutToName ? 'A page names a layout the site does not have, and this job may not create one. Leave the layout empty.' : mayCreateLayout ? "A page names no layout the site has or the plan creates. Put every page in the site's layout, or plan one." : 'A page names no layout the site has, and this job may not create one. Put every page in a layout the site has.',
             paths: unlaid.map(({ index })=>`screens[${index}].layout`)
         });
     }
@@ -2647,7 +2668,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 2,
             code: 'plan-layout-region-section',
-            message: 'A screen plans its own header, navigation or footer. Those live in the layout; take the section off the screen.',
+            message: 'A page plans its own header, navigation or footer. Those live in the layout; take the section off the page.',
             paths: regions.map((entry)=>entry.path)
         });
     }
@@ -2656,7 +2677,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 2,
             code: 'plan-layout-in-section',
-            message: "A section places a layout. A layout frames a whole screen and is never placed inside one: name it as the screen's layout, and take it out of the section's uses.",
+            message: "A section places a layout. A layout frames a whole page and is never placed inside one: name it as the page's layout, and take it out of the section's uses.",
             paths: placed
         });
     }
@@ -2703,7 +2724,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 4,
             code: 'plan-similar-pages',
-            message: `${indexes.length} screens share one shape. Plan one template and apply it ${indexes.length} times with each page's copy, or bind it to a collection.`,
+            message: `${indexes.length} pages share one shape. Plan one template and apply it ${indexes.length} times with each page's copy, or bind it to a collection.`,
             paths: indexes.map((index)=>`screens[${index}].template`)
         });
     }
@@ -2712,7 +2733,7 @@ const FORM_SECTION_NAME = /\bform\b|\bsign[- ]?up\b|\bsubscribe\b|\bnewsletter\b
         violations.push({
             rule: 4,
             code: 'plan-template-in-section',
-            message: "A section places a template. A template is applied to a whole screen and is never placed inside one: take it out of the section's uses, and name it as the screen's template only when the whole screen is built from it.",
+            message: "A section places a template. A template is applied to a whole page and is never placed inside one: take it out of the section's uses, and name it as the page's template only when the whole page is built from it.",
             paths: placed
         });
     }
@@ -2851,15 +2872,15 @@ const REUSE_BEFORE_CREATE_KINDS = new Set([
         const section = first.field === 'uses' ? screen.sections[first.sectionIndex] : null;
         const kind = section ? undeclaredPlacementKind(first.name, section, plan, kinds) : 'template';
         const refused = capabilities ? aiPlanUncreatableKind(plan, kind, capabilities) : null;
-        const where = section ? `${section.name ? `The "${section.name}" section` : 'A section'} places a creation named "${first.name}", but the plan never creates it` : `${screen.title ? `The screen "${screen.title}"` : 'A screen'} applies a template named "${first.name}", but the plan never creates it`;
+        const where = section ? `${section.name ? `The "${section.name}" section` : 'A section'} places a creation named "${first.name}", but the plan never creates it` : `${screen.title ? `The page "${screen.title}"` : 'A page'} applies a template named "${first.name}", but the plan never creates it`;
         let message;
         if (!refused) {
-            const otherwise = section ? `place ${aiCreationNoun(kind)} the site already has by its id` : "leave the screen's template empty";
+            const otherwise = section ? `place ${aiCreationNoun(kind)} the site already has by its id` : "leave the page's template empty";
             message = `${where}. Declare it in create as ${aiCreationNoun(kind)}, with why nothing the site has will do, or ${otherwise}.`;
         } else if (section) {
             message = `${where}, and ${refused.reason}. ${refused.instead} Take it out of the section's uses.`;
         } else {
-            message = `${where}, and ${refused.reason}. Leave the screen's template empty, or apply a template the site already has.`;
+            message = `${where}, and ${refused.reason}. Leave the page's template empty, or apply a template the site already has.`;
         }
         return {
             rule: 7,
@@ -2951,7 +2972,7 @@ function slugKey(slug) {
         violations.push({
             rule: 10,
             code: 'plan-slug',
-            message: 'A screen has no usable address. Give each one a slug of lowercase words joined by hyphens.',
+            message: 'A page has no usable address. Give each one a slug of lowercase words joined by hyphens.',
             paths: badSlugs
         });
     }
@@ -2959,7 +2980,7 @@ function slugKey(slug) {
         violations.push({
             rule: 10,
             code: 'plan-slug-taken',
-            message: 'A screen reuses an address the site or the plan already uses. Give each screen its own slug.',
+            message: 'A page reuses an address the site or the plan already uses. Give each page its own slug.',
             paths: collisions
         });
     }
@@ -2967,7 +2988,7 @@ function slugKey(slug) {
         violations.push({
             rule: 10,
             code: 'plan-seo',
-            message: `A screen is missing its search title or description, or runs past ${AI_SEO_TITLE_MAX} and ${AI_SEO_DESCRIPTION_MAX} characters. Write both for every screen.`,
+            message: `A page is missing its search title or description, or runs past ${AI_SEO_TITLE_MAX} and ${AI_SEO_DESCRIPTION_MAX} characters. Write both for every page.`,
             paths: seo
         });
     }
@@ -3014,7 +3035,7 @@ function slugKey(slug) {
             violations.push({
                 rule: 15,
                 code: 'plan-missed-duplicate',
-                message: `The site already has "${nearest.name}". Start the new screen from a duplicate of it, which keeps its bindings and SEO, and edit that.`,
+                message: `The site already has "${nearest.name}". Start the new page from a duplicate of it, which keeps its bindings and SEO, and edit that.`,
                 paths: [
                     `screens[${index}].duplicateOf`
                 ]
@@ -3155,6 +3176,141 @@ function slugKey(slug) {
         }
     ];
 }
+/**
+ * Rule 16 (plan): a third-party player is planned where it can play
+ * (AGL-3433). A screen of the plan or a component it creates carries at most
+ * its kind's budget. A template renders for every record it serves and a
+ * layout around every page, so neither carries one, and a form, an email or a
+ * dataset has nowhere to. A link the plan gives is one its host plays.
+ */ export function detectPlanEmbeds(plan) {
+    var _plan_embeds;
+    const nowhere = [];
+    const elsewhere = [];
+    const wrongHost = [];
+    const placed = new Map();
+    ((_plan_embeds = plan.embeds) != null ? _plan_embeds : []).forEach((embed, index)=>{
+        var _placed_get;
+        const path = `embeds[${index}]`;
+        if (embed.url !== null && aiEmbedHostOf(embed.url) !== embed.host) wrongHost.push(`${path}.url`);
+        let target = null;
+        if (isAiPlanNewRef(embed.where)) {
+            const creation = aiPlanCreateFor(plan, embed.where);
+            if (creation && creation.kind !== 'component') {
+                elsewhere.push({
+                    kind: creation.kind,
+                    path: `${path}.where`
+                });
+                return;
+            }
+            if (creation) target = {
+                key: embed.where.trim().toLowerCase(),
+                label: creation.name,
+                kind: 'component'
+            };
+        } else {
+            const screen = plan.screens.find((row)=>aiPlanSlugKey(row.slug) === aiPlanSlugKey(embed.where));
+            if (screen) target = {
+                key: aiPlanSlugKey(screen.slug),
+                label: screen.slug,
+                kind: 'page'
+            };
+        }
+        if (!target) {
+            nowhere.push(`${path}.where`);
+            return;
+        }
+        const entry = (_placed_get = placed.get(target.key)) != null ? _placed_get : {
+            label: target.label,
+            kind: target.kind,
+            paths: []
+        };
+        entry.paths.push(path);
+        placed.set(target.key, entry);
+    });
+    const violations = [];
+    if (nowhere.length) {
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-nowhere',
+            message: 'A player is planned where the plan builds nothing. Name a screen of the plan by its slug, or new:<name> of a component it creates.',
+            paths: nowhere
+        });
+    }
+    if (elsewhere.length) {
+        const nouns = [
+            ...new Set(elsewhere.map((entry)=>AI_BUILD_PLAN_CREATION_NOUNS[entry.kind].noun))
+        ];
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-refused',
+            message: `Only a page or a component carries a third-party player, and this plan puts one on a ${nouns.join(' and a ')}. A template or a layout would load the player's code on every page it serves. Place it on a page or a component, or leave it off.`,
+            paths: elsewhere.map((entry)=>entry.path)
+        });
+    }
+    for (const entry of placed.values()){
+        const limit = AI_OUTPUT_BUDGETS[entry.kind].embeds;
+        if (entry.paths.length <= limit) continue;
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-over-budget',
+            message: `${entry.label} is planned ${entry.paths.length} players, and a ${entry.kind} carries at most ${limit}. Keep the one the brief asks for most.`,
+            paths: entry.paths
+        });
+    }
+    if (wrongHost.length) {
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-link-host',
+            message: `A planned player's link is not a video link of its host, ${Object.values(AI_BUILD_PLAN_EMBED_HOST_NAMES).join(' or ')}. Give the brief's own link for that host, or null.`,
+            paths: wrongHost
+        });
+    }
+    return violations;
+}
+/** Text as two copies of one phrase compare: lowercase, no quotes, one space. */ function phraseKey(text) {
+    return text.toLowerCase().replace(/[“”"‘’'`]/g, '').replace(/\s+/g, ' ').trim();
+}
+/**
+ * Rule 16 (plan, against the brief): a player is planned only where the brief
+ * asks for one (AGL-3433). The plan quotes the brief's words, and those words
+ * are found in the brief and ask for a video; a link it gives is one the brief
+ * gives. The plan step holds this, being the one door that reads the brief
+ * beside the plan.
+ */ export function aiPlanEmbedBriefViolations(plan, brief) {
+    var _brief_match, _plan_embeds;
+    const said = phraseKey(brief);
+    const given = new Set(((_brief_match = brief.match(/https:\/\/\S+/g)) != null ? _brief_match : []).map((link)=>aiEmbedVideoKey(link.replace(/[).,;!?]+$/, ''))).filter((key)=>key !== null));
+    const unasked = [];
+    const ungiven = [];
+    ((_plan_embeds = plan.embeds) != null ? _plan_embeds : []).forEach((embed, index)=>{
+        const asked = phraseKey(embed.asked);
+        if (!asked || !said.includes(asked) || !AI_PLAN_ASKS_FOR_VIDEO.test(asked)) {
+            unasked.push(`embeds[${index}].asked`);
+        }
+        if (embed.url !== null) {
+            const key = aiEmbedVideoKey(embed.url);
+            if (!key || !given.has(key)) ungiven.push(`embeds[${index}].url`);
+        }
+    });
+    const violations = [];
+    if (unasked.length) {
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-not-asked',
+            message: 'A player is planned that the brief does not ask for: its words are not words of the brief asking for a video. Take it off the plan; a film from the media library plays in a Video.',
+            paths: unasked
+        });
+    }
+    if (ungiven.length) {
+        violations.push({
+            rule: 16,
+            code: 'plan-embed-link-not-given',
+            message: "A planned player's link is not one the brief gives. Give the brief's own link, or null for the site owner to paste.",
+            paths: ungiven
+        });
+    }
+    return violations;
+}
 /**
  * Every plan rule, against the site inventory the plan was made from and,
  * where the job read them, what it may create there (AGL-3030). `null`
@@ -3169,6 +3325,7 @@ function slugKey(slug) {
         ...detectPlanInlineForms(plan, inventory, capabilities),
         ...detectUntemplatedSimilarPages(plan, inventory),
         ...detectPlanLiteralColors(plan),
+        ...detectPlanEmbeds(plan),
         ...detectCreateBeforeReuse(plan, inventory),
         ...detectPlanUncreatable(plan, capabilities),
         ...detectPlanUndeclaredCreations(plan, inventory, capabilities),