fimo 0.9.0-experimental.6 → 0.9.0-experimental.7

package/dist/scripts/frameworks/discover.js

@@ -66,6 +66,11 @@ export function frameworkJsxWrapperModule(detection) {
             return 'fimo/next';
         case 'remix':
             return 'fimo/remix';
+        case 'react-router':
+            // Library-mode React Router SPAs (react-router dep on Vite) wrap with
+            // the client-side fimo/react primitives — same shape as vite+react.
+            // Proven on the Fimo marketing site (1211 items auto-wrapped).
+            return 'fimo/react';
         case 'vite':
             return detection.react ? 'fimo/react' : null;
         default:
@@ -101,6 +106,10 @@ export function frameworkRootWiring(detection) {
     switch (detection.framework) {
         case 'next':
             return (cwd) => ensureNextLayoutWiring(cwd, detection);
+        case 'react-router':
+            // Same createRoot(...).render entry shape as vite+react; framework-mode
+            // RR apps have no src/main.* and bail with a reason (agent §0).
+            return ensureSpaEntryWiring;
         case 'vite':
             return detection.react ? ensureSpaEntryWiring : null;
         default:
@@ -160,6 +169,14 @@ export function frameworkWiringMarker(detection, cwd) {
                 marker: /FimoSpaProviders/,
                 hint: 'wrap the root render with FimoSpaProviders from fimo/react (agent prompt section 0)',
             };
+        case 'react-router':
+            // Accept either provider: FimoSpaProviders (the vite+react codemod) or
+            // FimoProviders (the fimo/react-router adapter, hand-wired on RR sites).
+            return {
+                files: ['src/main.tsx', 'src/main.jsx', 'src/index.tsx', 'src/index.jsx', 'app/root.tsx', 'app/root.jsx'],
+                marker: /Fimo(Spa)?Providers/,
+                hint: 'wrap the root render with FimoSpaProviders (fimo/react) or FimoProviders (fimo/react-router) — agent prompt section 0',
+            };
         case 'remix':
             return {
                 files: ['app/root.tsx', 'app/root.jsx'],

package/dist/scripts/scan-wrappable.d.ts

@@ -4,7 +4,7 @@
  * user-visible attribute strings. Drives the wrap migration's coverage gate:
  * every item must end wrapped, ignored-with-a-reason, or unresolved.
  */
-export type WrappableKind = 'text' | 'image' | 'video' | 'file-link' | 'attribute';
+export type WrappableKind = 'text' | 'formatted-text' | 'image' | 'video' | 'file-link' | 'attribute';
 export interface WrappableItem {
     kind: WrappableKind;
     file: string;

package/dist/scripts/scan-wrappable.js

@@ -118,6 +118,36 @@ const WRAPPABLE_ATTRIBUTES = new Set(['placeholder', 'title', 'aria-label']);
 const FILE_LINK_PATTERN = /\.(pdf|zip|docx?|xlsx?|pptx?|csv|mp4|mov|webm)$/i;
 const SKIP_DIRS = new Set(['node_modules', '.next', '.git', 'dist', 'build', '.fimo']);
 const MIN_TEXT_LENGTH = 3;
+/**
+ * Prose containers whose children mix raw text with inline elements or
+ * expressions. Wrapping such text node-by-node produces FRAGMENT labels
+ * (", and agencies building…") that nobody can find or edit as a sentence —
+ * the whole block merges into ONE `formatted-text` item instead (agent work:
+ * the fimo`` tagged template or RichText, never the auto-wrap codemod).
+ */
+const PROSE_TAGS = new Set([
+    'p',
+    'h1',
+    'h2',
+    'h3',
+    'h4',
+    'h5',
+    'h6',
+    'li',
+    'blockquote',
+    'figcaption',
+    'dt',
+    'dd',
+    'label',
+    'legend',
+    'td',
+    'th',
+    'caption',
+    'summary',
+]);
+function meaningfulLength(text) {
+    return text.replace(/[^\p{L}\p{N}]/gu, '').length;
+}
 const PREVIEW_LENGTH = 60;
 function toPreview(value) {
     const collapsed = value.replace(/\s+/g, ' ').trim();
@@ -465,6 +495,50 @@ export function scanWrappableFile(filePath, code, relPath, opts = {}) {
     function detectCollection(declarator) {
         detectCollectionFromDeclarator(declarator, code, 0, relPath, collections);
     }
+    // Text nodes absorbed into a merged formatted-text item (keyed by offset) —
+    // the JSXText visitor skips them so fragments never become labels.
+    const mergedText = new Set();
+    function containsWrappedContent(node) {
+        if (!node || typeof node !== 'object') {
+            return false;
+        }
+        if (node.type === 'CallExpression' && node.callee?.type === 'Identifier' && node.callee.name === 't') {
+            return true;
+        }
+        if (node.type === 'TaggedTemplateExpression' && node.tag?.type === 'Identifier' && node.tag.name === 'fimo') {
+            return true;
+        }
+        if (node.type === 'JSXElement') {
+            const name = jsxElementName(node);
+            if (name && WRAPPING_COMPONENTS.has(name)) {
+                return true;
+            }
+        }
+        for (const key of Object.keys(node)) {
+            const child = node[key];
+            if (Array.isArray(child)) {
+                if (child.some((item) => containsWrappedContent(item))) {
+                    return true;
+                }
+            }
+            else if (typeof child === 'object' && child !== null && containsWrappedContent(child)) {
+                return true;
+            }
+        }
+        return false;
+    }
+    /** Every JSXText inside a prose element, recursing through inline children
+     *  (inside a `<p>`, nested elements are inline by HTML rules). */
+    function collectProseTextNodes(node, out) {
+        for (const child of node.children ?? []) {
+            if (child.type === 'JSXText') {
+                out.push(child);
+            }
+            else if (child.type === 'JSXElement') {
+                collectProseTextNodes(child, out);
+            }
+        }
+    }
     function visit(node, insideWrapped, parentTag, scopes) {
         if (!node || typeof node !== 'object') {
             return;
@@ -548,7 +622,43 @@ export function scanWrappableFile(filePath, code, relPath, opts = {}) {
                 }
             }
         }
-        if (node.type === 'JSXText' && !insideWrapped) {
+        // Mixed prose detection runs on the ELEMENT (before its children are
+        // visited): a prose tag whose direct children interleave raw text with
+        // inline elements/expressions becomes ONE formatted-text item, and its
+        // text nodes are recorded so the JSXText visitor below skips them.
+        if (node.type === 'JSXElement' && !insideWrapped) {
+            const proseName = jsxElementName(node);
+            if (proseName && PROSE_TAGS.has(proseName)) {
+                const children = node.children ?? [];
+                const directText = children.filter((child) => child.type === 'JSXText' && meaningfulLength(String(child.value ?? '')) > 0);
+                const inline = children.filter((child) => child.type === 'JSXElement' ||
+                    (child.type === 'JSXExpressionContainer' && child.expression?.type !== 'JSXEmptyExpression'));
+                if (directText.length > 0 &&
+                    inline.length > 0 &&
+                    !directText.some((child) => mergedText.has(child.start)) &&
+                    !children.some((child) => containsWrappedContent(child))) {
+                    const textNodes = [];
+                    collectProseTextNodes(node, textNodes);
+                    const combined = textNodes.map((textNode) => String(textNode.value ?? '')).join(' ');
+                    if (meaningfulLength(combined) >= MIN_TEXT_LENGTH) {
+                        for (const textNode of textNodes) {
+                            if (typeof textNode.start === 'number') {
+                                mergedText.add(textNode.start);
+                            }
+                        }
+                        const inner = code.slice(node.openingElement?.end ?? node.start ?? 0, node.closingElement?.start ?? node.end ?? 0);
+                        items.push({
+                            kind: 'formatted-text',
+                            file: relPath,
+                            line: lineOf(code, node.start ?? 0),
+                            preview: toPreview(inner),
+                            span: { start: node.start, end: node.end, parentTag: proseName },
+                        });
+                    }
+                }
+            }
+        }
+        if (node.type === 'JSXText' && !insideWrapped && !mergedText.has(node.start)) {
             const text = String(node.value ?? '');
             const meaningful = text.replace(/[^\p{L}\p{N}]/gu, '');
             if (meaningful.length >= MIN_TEXT_LENGTH) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fimo",
-  "version": "0.9.0-experimental.6",
+  "version": "0.9.0-experimental.7",
   "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.6",
+    "fimo": "0.9.0-experimental.7",
     "input-otp": "^1.4.2",
     "isbot": "^5",
     "lucide-react": "^0.577.0",