@mintlify/common 1.0.797 → 1.0.798

package/dist/mdx/plugins/remark/remarkMdxRemoveJs.js

@@ -127,7 +127,7 @@ const DEFAULT_PROP_EXPRESSIONS = {
     examples: { value: '{}', estree: objectEstree },
     summary: { value: '""', estree: createStringEstree('') },
     value: { value: '{}', estree: objectEstree },
-    contextual: { value: '{ "options": [] }', estree: falseEstree },
+    contextual: { value: '{}', estree: objectEstree },
     api: { value: '{}', estree: objectEstree },
     navigation: { value: '{}', estree: objectEstree },
     asyncapi: { value: '""', estree: createStringEstree('') },
@@ -142,9 +142,9 @@ export function isStringSafe(value) {
         // purely alphanumeric string
         case /^[_a-zA-Z0-9]+$/.test(value):
         // string literal with quotes
-        case /^'[_a-zA-Z0-9\s`".\-\\{}\?\(\)&*^%#@!~=+\[\]|:;,\/]+'$/.test(value):
-        case /^"[_a-zA-Z0-9\s`'.\-\\{}\?\(\)&*^%#@!~=+\[\]|:;,\/]+"$/.test(value):
-        case /^`[_a-zA-Z0-9\s'".\-\\{}\?\(\)&*^%#@!~=+\[\]|:;,\/]+`$/.test(value):
+        case /^'[_a-zA-Z0-9\s`".\-\\${}?\(\)&*^%#@!~=+\[\]|:;,\/]+'$/.test(value):
+        case /^"[_a-zA-Z0-9\s`'.\-\\${}?\(\)&*^%#@!~=+\[\]|:;,\/]+"$/.test(value):
+        case /^`[_a-zA-Z0-9\s'".\-\\{}?\(\)&*^%#@!~=+\[\]|:;,\/]+`$/.test(value):
         // purely alphanumeric string prefixed with `!!`
         case /^!![_a-zA-Z0-9]+$/.test(value):
         // purely alphanumeric string wrapped in parens
@@ -238,41 +238,72 @@ function collectIdentifiersFromPattern(pattern, out) {
         collectIdentifiersFromPattern(pattern.argument, out);
     }
 }
+// Returns the identifier name if the estree expression is a bare reference that will be kept
+// by isStringSafe but needs a stub to avoid a ReferenceError at render time:
+//   {foo}    → Identifier
+//   {!!foo}  → UnaryExpression(! UnaryExpression(! Identifier))
+// Parenthesised forms are transparent in ESTree so (foo) produces the same AST as foo.
+function extractIdentifierRef(estree) {
+    const stmt = estree === null || estree === void 0 ? void 0 : estree.body[0];
+    if ((stmt === null || stmt === void 0 ? void 0 : stmt.type) !== 'ExpressionStatement')
+        return null;
+    const expr = stmt.expression;
+    if (expr.type === 'Identifier')
+        return expr.name;
+    if (expr.type === 'UnaryExpression' &&
+        expr.operator === '!' &&
+        expr.argument.type === 'UnaryExpression' &&
+        expr.argument.operator === '!' &&
+        expr.argument.argument.type === 'Identifier')
+        return expr.argument.argument.name;
+    return null;
+}
 function collectExportedNames(estree) {
     var _a, _b;
     const names = [];
     for (const node of estree.body) {
-        if (node.type !== 'ExportNamedDeclaration')
-            continue;
-        const exportNode = node;
-        const decl = exportNode.declaration;
-        if (!decl) {
-            // Re-export specifiers: export { X } from 'module' or export { X }
-            for (const spec of exportNode.specifiers) {
-                // ES2022 allows Literal export names (e.g. export { foo as "bar" }),
-                // which have no .name — skip to avoid emitting `export const undefined = undefined`
-                // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-                if (spec.exported.type !== 'Identifier')
-                    continue;
-                names.push({ name: spec.exported.name, isFunction: false });
-            }
-            continue;
-        }
-        if (decl.type === 'FunctionDeclaration' && decl.id) {
-            names.push({ name: decl.id.name, isFunction: true });
-        }
-        else if (decl.type === 'VariableDeclaration') {
-            for (const declarator of decl.declarations) {
-                if (declarator.id.type === 'Identifier') {
-                    const isFn = ((_a = declarator.init) === null || _a === void 0 ? void 0 : _a.type) === 'ArrowFunctionExpression' ||
-                        ((_b = declarator.init) === null || _b === void 0 ? void 0 : _b.type) === 'FunctionExpression';
-                    names.push({ name: declarator.id.name, isFunction: isFn });
+        // Named exports: export const x, export function x, export { x }
+        if (node.type === 'ExportNamedDeclaration') {
+            const exportNode = node;
+            const decl = exportNode.declaration;
+            if (!decl) {
+                // Re-export specifiers: export { X } from 'module' or export { X }
+                for (const spec of exportNode.specifiers) {
+                    // ES2022 allows Literal export names (e.g. export { foo as "bar" }),
+                    // which have no .name — skip to avoid emitting `export const undefined = undefined`
+                    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+                    if (spec.exported.type !== 'Identifier')
+                        continue;
+                    // 'default' is a reserved word and cannot be used as a variable name in a stub
+                    if (spec.exported.name === 'default')
+                        continue;
+                    names.push({ name: spec.exported.name, isFunction: false });
                 }
-                else {
-                    collectIdentifiersFromPattern(declarator.id, names);
+                continue;
+            }
+            if (decl.type === 'FunctionDeclaration' && decl.id) {
+                names.push({ name: decl.id.name, isFunction: true });
+            }
+            else if (decl.type === 'VariableDeclaration') {
+                for (const declarator of decl.declarations) {
+                    if (declarator.id.type === 'Identifier') {
+                        const isFn = ((_a = declarator.init) === null || _a === void 0 ? void 0 : _a.type) === 'ArrowFunctionExpression' ||
+                            ((_b = declarator.init) === null || _b === void 0 ? void 0 : _b.type) === 'FunctionExpression';
+                        names.push({ name: declarator.id.name, isFunction: isFn });
+                    }
+                    else {
+                        collectIdentifiersFromPattern(declarator.id, names);
+                    }
                 }
             }
         }
+        // Default export: export default function Name() {} creates a named binding in scope
+        if (node.type === 'ExportDefaultDeclaration') {
+            const decl = node.declaration;
+            if ((decl.type === 'FunctionDeclaration' || decl.type === 'FunctionExpression') && decl.id) {
+                names.push({ name: decl.id.name, isFunction: true });
+            }
+        }
     }
     return names;
 }
@@ -350,34 +381,40 @@ function isArrayOfStringLiterals(estree) {
 export function remarkMdxRemoveJs() {
     return (tree) => {
         remove(tree, ['mdxTextExpression', 'mdxFlowExpression']);
-        // Collect exported names before stripping ESM so we can inject stubs
-        const exportedNames = [];
+        // Collect exported names before stripping ESM so we can inject stubs.
+        // Map keyed by name deduplicates naturally.
+        const stubbedNamesMap = new Map();
         visit(tree, (node) => {
             var _a;
             if (isMdxJsEsm(node) && ((_a = node.data) === null || _a === void 0 ? void 0 : _a.estree)) {
-                exportedNames.push(...collectExportedNames(node.data.estree));
+                for (const entry of collectExportedNames(node.data.estree)) {
+                    if (!stubbedNamesMap.has(entry.name))
+                        stubbedNamesMap.set(entry.name, entry);
+                }
             }
         });
         remove(tree, (node) => isMdxJsEsm(node));
-        // Inject stub declarations for stripped exports to prevent ReferenceErrors
-        if (exportedNames.length > 0) {
-            tree.children.unshift(buildStubEsmNode(exportedNames));
-        }
+        // Sanitize JSX attributes. Any bare identifier that survives is added to stubbedNamesMap
+        // so it gets a stub even if its import was already stripped by a prior plugin.
         visit(tree, (node) => {
             if (!('attributes' in node))
                 return CONTINUE;
             const newAttributes = node.attributes.map((attr) => {
-                var _a, _b, _c, _d, _e, _f;
+                var _a, _b, _c, _d, _e, _f, _g;
                 if (attr.type === 'mdxJsxAttribute' && !(attr.value instanceof Object))
                     return attr;
                 if (typeof attr.value === 'string' && isStringSafe(attr.value))
                     return attr;
                 if (attr.type === 'mdxJsxAttribute' && attr.value instanceof Object) {
-                    if (isStringSafe(attr.value.value))
+                    if (isStringSafe(attr.value.value)) {
+                        const ident = extractIdentifierRef((_a = attr.value.data) === null || _a === void 0 ? void 0 : _a.estree);
+                        if (ident && !stubbedNamesMap.has(ident))
+                            stubbedNamesMap.set(ident, { name: ident, isFunction: false });
                         return attr;
-                    if (((_a = attr.value.data) === null || _a === void 0 ? void 0 : _a.estree) && isArrayOfStringLiterals(attr.value.data.estree))
+                    }
+                    if (((_b = attr.value.data) === null || _b === void 0 ? void 0 : _b.estree) && isArrayOfStringLiterals(attr.value.data.estree))
                         return attr;
-                    if (attr.name === 'style' && ((_b = attr.value.data) === null || _b === void 0 ? void 0 : _b.estree)) {
+                    if (attr.name === 'style' && ((_c = attr.value.data) === null || _c === void 0 ? void 0 : _c.estree)) {
                         const filteredEstree = filterStyleProperties(attr.value.data.estree);
                         if (filteredEstree) {
                             const stmt = filteredEstree.body[0];
@@ -391,9 +428,9 @@ export function remarkMdxRemoveJs() {
                         return undefined;
                     }
                     if (Object.keys(DEFAULT_PROP_EXPRESSIONS).includes(attr.name)) {
-                        attr.value.value = (_d = (_c = DEFAULT_PROP_EXPRESSIONS[attr.name]) === null || _c === void 0 ? void 0 : _c.value) !== null && _d !== void 0 ? _d : '{}';
+                        attr.value.value = (_e = (_d = DEFAULT_PROP_EXPRESSIONS[attr.name]) === null || _d === void 0 ? void 0 : _d.value) !== null && _e !== void 0 ? _e : '{}';
                         attr.value.data = {
-                            estree: (_f = (_e = DEFAULT_PROP_EXPRESSIONS[attr.name]) === null || _e === void 0 ? void 0 : _e.estree) !== null && _f !== void 0 ? _f : objectEstree,
+                            estree: structuredClone((_g = (_f = DEFAULT_PROP_EXPRESSIONS[attr.name]) === null || _f === void 0 ? void 0 : _f.estree) !== null && _g !== void 0 ? _g : objectEstree),
                         };
                         return attr;
                     }
@@ -402,5 +439,9 @@ export function remarkMdxRemoveJs() {
             });
             node.attributes = newAttributes.filter((attr) => attr !== undefined);
         });
+        // Inject stubs after the attribute walk so identifiers collected from attributes are included
+        if (stubbedNamesMap.size > 0) {
+            tree.children.unshift(buildStubEsmNode([...stubbedNamesMap.values()]));
+        }
     };
 }