fimo 0.9.0-experimental.10 → 0.9.0-experimental.11

package/dist/scripts/routes/page-structure.d.ts

@@ -68,5 +68,69 @@ export interface PageSingleDraft {
     pageId: string;
     schema: Record<string, unknown>;
     document: Record<string, unknown>;
+    /** Section components left OUT of the single because they read as
+     *  illustrations/product demos, not editable content — the agent confirms
+     *  each is decorative (exempts it) or pulls it back into the model. */
+    illustrations?: string[];
 }
-export declare function buildPageSingleDraft(draft: PageStructureDraft, labelValues: Record<string, string>): PageSingleDraft | null;
+/**
+ * A section whose visible text is a DENSE PILE of copy strings reads as an
+ * ILLUSTRATION / product demo (a fake dashboard, a device mockup, an animated
+ * terminal), not editable content — e.g. a UI mock with 75 strings (column
+ * headers, buttons, status chips, plus a little fake body prose). Modeling
+ * that as 75 CMS fields is noise that makes a page uneditable.
+ *
+ * The primary signal is SIZE (framework-neutral: no real editable section has
+ * 30+ flat sibling copy fields — real content is small or structured into
+ * collections). A section that is SUBSTANTIALLY prose is exempt (an article /
+ * legal page is content, however long). For a medium pile (8-29 strings) the
+ * component NAME must corroborate. Conservative by design — a false positive
+ * silently drops editable content, so the caller SURFACES the flag for the
+ * agent to confirm rather than deleting it silently.
+ */
+export declare function isIllustrationSection(section: PageSectionDraft, labelValues: Record<string, string>): boolean;
+export declare function buildPageSingleDraft(draft: PageStructureDraft, labelValues: Record<string, string>, contentConfirmed?: ReadonlySet<string>): PageSingleDraft | null;
+/**
+ * A section the detector reads as an illustration/product-demo, with enough
+ * context for the human to confirm in a glance: WHERE it lives (`file`,
+ * `component`), WHAT strings it holds (`labelKeys` + a few `samples`). The
+ * interview turns each into an `IllustrationDecisionRecord`; keyed by
+ * `(file, component)` so the decision survives the agent's rename pass.
+ */
+export interface IllustrationCandidate {
+    file: string;
+    component: string;
+    labelKeys: string[];
+    samples: string[];
+}
+/**
+ * Every section (top-level or nested) the illustration detector flags across
+ * the page-structure drafts, deduped by `(file, component)` — the same file
+ * rendered on several pages is one candidate, one question. Rich payload, not
+ * just a name: the caller relays it to the human and, on `drop`, exempts the
+ * `labelKeys` from the coverage/parity gates.
+ */
+export declare function collectIllustrationCandidates(drafts: PageStructureDraft[], labelValues: Record<string, string>): IllustrationCandidate[];
+/**
+ * A section the structural detector left as a page-local block (a default
+ * Component) but whose content model is genuinely in question: its component
+ * renders on 2+ pages, so the owner may know it is actually the SAME content
+ * everywhere (a Single Type) or a repeated managed entry (a Collection Type) —
+ * a call `markSharedSections` cannot make when the copies live in different
+ * files. Single-page sections are unambiguously Components and never surface.
+ */
+export interface ContentModelCandidate {
+    file: string;
+    component: string;
+    /** How many page drafts render this component — the ambiguity signal. */
+    pages: number;
+    samples: string[];
+}
+/**
+ * The content-model classification candidates: non-shared, non-illustration
+ * sections whose component appears across 2+ page drafts. Deduped by
+ * `(file, component)`; the interview asks which are Single Types / Collection
+ * Types and leaves the rest as Components. Shared chrome (auto Single Type) and
+ * detected collections (auto Collection Type) never reach here.
+ */
+export declare function collectContentModelCandidates(drafts: PageStructureDraft[], labelValues: Record<string, string>, contentConfirmed?: ReadonlySet<string>): ContentModelCandidate[];

package/dist/scripts/routes/page-structure.js

@@ -2,7 +2,12 @@ import { readFileSync, statSync } from 'node:fs';
 import { join, relative } from 'node:path';
 import { frameworkSectionStyle } from '../frameworks/discover.js';
 import { componentImports, resolveModuleFile, resolveRouteFiles } from './label-prefixes.js';
-const MAX_SECTION_DEPTH = 3;
+// How deep the section recursion nests components into blocks. The platform
+// caps structured-content nesting at 7 and the page Single wraps each top
+// section as a block (one level), so 6 keeps the deepest child inside the
+// limit while mirroring real page-builders that nest 5-6 deep (a features
+// grid → groups → cards → …). Was 3, which flattened deep structure.
+const MAX_SECTION_DEPTH = 6;
 const MAX_DRAFT_ROWS = 50;
 const pascal = (value) => value
     .replace(/([a-z0-9])([A-Z])/g, '$1-$2')
@@ -149,7 +154,67 @@ export function countDraftSections(draft) {
 }
 const labelFieldName = (key) => camel(key.split('.').pop() ?? key);
 const scalarType = (value) => value.length > 80 || value.includes('\n') ? 'text' : 'string';
-function sectionBlock(section, labelValues) {
+/** Names that corroborate (never trigger alone) an illustration/demo. */
+const ILLUSTRATION_NAME = /mock|demo|preview|illustration|visual|showcase|diagram|orbit|screenshot|animation|graphic|dashboard|widget|figure/i;
+/** A section with this many flat sibling copy strings is a dump, not content:
+ *  real editable sections stay small (a hero has ~3 fields, a card ~3), and
+ *  genuinely repeated content is a collection/block, not N sibling labels. A
+ *  UI mock reproduced in code (a fake dashboard, a device frame, a terminal
+ *  demo) is the usual culprit — one carried 75 strings in the wild. */
+const ILLUSTRATION_DENSE = 30;
+const ILLUSTRATION_MIN_LABELS = 8;
+const wordCount = (value) => value.trim().split(/\s+/).filter(Boolean).length;
+/** Editable prose: a heading + paragraph section has some; a UI-chrome dump
+ *  (a product mock's nav/table/button strings) is mostly tiny fragments. */
+const looksLikeProse = (value) => {
+    const words = wordCount(value);
+    return words >= 8 || (words >= 5 && /[.!?]/.test(value.trim()));
+};
+/**
+ * A section whose visible text is a DENSE PILE of copy strings reads as an
+ * ILLUSTRATION / product demo (a fake dashboard, a device mockup, an animated
+ * terminal), not editable content — e.g. a UI mock with 75 strings (column
+ * headers, buttons, status chips, plus a little fake body prose). Modeling
+ * that as 75 CMS fields is noise that makes a page uneditable.
+ *
+ * The primary signal is SIZE (framework-neutral: no real editable section has
+ * 30+ flat sibling copy fields — real content is small or structured into
+ * collections). A section that is SUBSTANTIALLY prose is exempt (an article /
+ * legal page is content, however long). For a medium pile (8-29 strings) the
+ * component NAME must corroborate. Conservative by design — a false positive
+ * silently drops editable content, so the caller SURFACES the flag for the
+ * agent to confirm rather than deleting it silently.
+ */
+export function isIllustrationSection(section, labelValues) {
+    const values = section.labelKeys.map((key) => labelValues[key] ?? '').filter((value) => value.trim().length > 0);
+    if (values.length < ILLUSTRATION_MIN_LABELS) {
+        return false;
+    }
+    // Substantially prose → a text/article/legal section, not a UI dump.
+    const proseRatio = values.filter(looksLikeProse).length / values.length;
+    if (proseRatio > 0.4) {
+        return false;
+    }
+    // A dense pile is a dump on size alone — the confident trigger.
+    if (values.length >= ILLUSTRATION_DENSE) {
+        return true;
+    }
+    // Medium pile: only when the component name says mock/demo AND it is mostly
+    // tiny fragments — keeps the false-positive rate near zero.
+    const shortRatio = values.filter((value) => wordCount(value) <= 3).length / values.length;
+    return ILLUSTRATION_NAME.test(section.component) && shortRatio >= 0.7;
+}
+const sectionIdentity = (section) => `${section.file}::${section.component}`;
+/**
+ * A section reads as an illustration to leave OUT of the model — UNLESS the
+ * owner confirmed in the interview that it IS editable content
+ * (`wrap.illustrations` disposition `content`, passed as `contentConfirmed`).
+ * That override is the whole point of asking, so it must beat the detector.
+ */
+function isDroppableIllustration(section, labelValues, contentConfirmed) {
+    return isIllustrationSection(section, labelValues) && !contentConfirmed.has(sectionIdentity(section));
+}
+function sectionBlock(section, labelValues, illustrations, contentConfirmed) {
     const fields = {};
     const value = {};
     let fieldCount = 0;
@@ -192,7 +257,11 @@ function sectionBlock(section, labelValues) {
         if (child.shared) {
             continue;
         }
-        const block = sectionBlock(child, labelValues);
+        if (isDroppableIllustration(child, labelValues, contentConfirmed)) {
+            illustrations.push(child.component);
+            continue;
+        }
+        const block = sectionBlock(child, labelValues, illustrations, contentConfirmed);
         if (!block || fields[child.key]) {
             continue;
         }
@@ -205,15 +274,24 @@ function sectionBlock(section, labelValues) {
     }
     return { fields, value, fieldCount };
 }
-export function buildPageSingleDraft(draft, labelValues) {
+export function buildPageSingleDraft(draft, labelValues, contentConfirmed = new Set()) {
     const fields = {};
     const document = {};
+    const illustrations = [];
     let totalFields = 0;
     for (const section of draft.sections) {
         if (section.shared || fields[section.key]) {
             continue;
         }
-        const block = sectionBlock(section, labelValues);
+        // Illustration/demo sections (a product mock's UI-chrome strings) are noise
+        // in a CMS — leave them out of the single. Surfaced, never silently lost.
+        // The interview's `content` override folds one back in (from the next run,
+        // once the decision is recorded — singles are built before the interview).
+        if (isDroppableIllustration(section, labelValues, contentConfirmed)) {
+            illustrations.push(section.component);
+            continue;
+        }
+        const block = sectionBlock(section, labelValues, illustrations, contentConfirmed);
         if (!block) {
             continue;
         }
@@ -230,5 +308,86 @@ export function buildPageSingleDraft(draft, labelValues) {
         pageId: draft.pageId,
         schema: { uid: draft.single, name: draft.single, isSingleton: true, fields },
         document,
+        ...(illustrations.length > 0 ? { illustrations: [...new Set(illustrations)] } : {}),
+    };
+}
+/** How many sample strings a candidate carries so a human can judge in 3s. */
+const SECTION_SAMPLE_COUNT = 6;
+const sectionSamples = (section, labelValues) => section.labelKeys
+    .map((key) => labelValues[key] ?? '')
+    .filter((value) => value.trim().length > 0)
+    .slice(0, SECTION_SAMPLE_COUNT);
+/**
+ * Every section (top-level or nested) the illustration detector flags across
+ * the page-structure drafts, deduped by `(file, component)` — the same file
+ * rendered on several pages is one candidate, one question. Rich payload, not
+ * just a name: the caller relays it to the human and, on `drop`, exempts the
+ * `labelKeys` from the coverage/parity gates.
+ */
+export function collectIllustrationCandidates(drafts, labelValues) {
+    const byIdentity = new Map();
+    const walk = (sections) => {
+        for (const section of sections) {
+            if (isIllustrationSection(section, labelValues)) {
+                const identity = `${section.file}::${section.component}`;
+                if (!byIdentity.has(identity)) {
+                    byIdentity.set(identity, {
+                        file: section.file,
+                        component: section.component,
+                        labelKeys: [...section.labelKeys],
+                        samples: sectionSamples(section, labelValues),
+                    });
+                }
+            }
+            walk(section.children ?? []);
+        }
     };
+    for (const draft of drafts) {
+        walk(draft.sections);
+    }
+    return [...byIdentity.values()];
+}
+/**
+ * The content-model classification candidates: non-shared, non-illustration
+ * sections whose component appears across 2+ page drafts. Deduped by
+ * `(file, component)`; the interview asks which are Single Types / Collection
+ * Types and leaves the rest as Components. Shared chrome (auto Single Type) and
+ * detected collections (auto Collection Type) never reach here.
+ */
+export function collectContentModelCandidates(drafts, labelValues, contentConfirmed = new Set()) {
+    const eligible = (section) => !section.shared && !isDroppableIllustration(section, labelValues, contentConfirmed);
+    const pagesByComponent = new Map();
+    const count = (pageId, sections) => {
+        for (const section of sections) {
+            if (eligible(section)) {
+                pagesByComponent.set(section.component, (pagesByComponent.get(section.component) ?? new Set()).add(pageId));
+            }
+            count(pageId, section.children ?? []);
+        }
+    };
+    for (const draft of drafts) {
+        count(draft.pageId, draft.sections);
+    }
+    const byIdentity = new Map();
+    const collect = (sections) => {
+        for (const section of sections) {
+            const pages = pagesByComponent.get(section.component)?.size ?? 0;
+            if (eligible(section) && pages >= 2) {
+                const identity = `${section.file}::${section.component}`;
+                if (!byIdentity.has(identity)) {
+                    byIdentity.set(identity, {
+                        file: section.file,
+                        component: section.component,
+                        pages,
+                        samples: sectionSamples(section, labelValues),
+                    });
+                }
+            }
+            collect(section.children ?? []);
+        }
+    };
+    for (const draft of drafts) {
+        collect(draft.sections);
+    }
+    return [...byIdentity.values()];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fimo",
-  "version": "0.9.0-experimental.10",
+  "version": "0.9.0-experimental.11",
   "description": "Fimo CLI - create, deploy, and manage Fimo projects",
   "bin": {
     "fimo": "dist/cli/index.js"

package/templates/react-router/package.json

@@ -21,7 +21,7 @@
     "cmdk": "^1.1.1",
     "date-fns": "^4.1.0",
     "embla-carousel-react": "^8.6.0",
-    "fimo": "0.9.0-experimental.10",
+    "fimo": "0.9.0-experimental.11",
     "input-otp": "^1.4.2",
     "isbot": "^5",
     "lucide-react": "^0.577.0",