jamdesk 1.1.82 → 1.1.83

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jamdesk",
-  "version": "1.1.82",
+  "version": "1.1.83",
   "description": "CLI for Jamdesk — build, preview, and deploy documentation sites from MDX. Dev server with hot reload, 50+ components, OpenAPI support, AI search, and Mintlify migration",
   "keywords": [
     "jamdesk",

package/vendored/components/mdx/Steps.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import { ReactNode, Children, cloneElement, isValidElement, memo } from 'react';
+import { ReactElement, ReactNode, Children, cloneElement, isValidElement, memo } from 'react';
 import { getIconClass } from '@/lib/icon-utils';
 import { useStepSlug } from './StepSlugContext';
 
@@ -22,19 +22,34 @@ interface StepProps {
   titleSize?: TitleSize;
 }
 
-// Helper to check if a child is a Step component
-// MDX can wrap components so we check displayName or function name
 function isStepComponent(child: unknown): boolean {
   if (!isValidElement(child)) return false;
   const type = child.type as { displayName?: string; name?: string };
-  const props = child.props as { title?: unknown; children?: unknown };
-  // Check for Step by displayName, name, or if it has children (Step-like)
   return (
     type === Step ||
     type?.displayName === 'Step' ||
-    type?.name === 'Step' ||
-    // Fallback: if wrapped by MDX and has children, treat as Step
-    props?.children !== undefined
+    type?.name === 'Step'
+  );
+}
+
+// Non-Step element children render as plain text inside <Steps> with no step
+// circle/number — surface them in dev so authors notice the silent fallback.
+function warnAboutNonStepChildren(childrenArray: readonly unknown[]): void {
+  if (process.env.NODE_ENV === 'production') return;
+  const offenders = childrenArray.filter(
+    (c): c is ReactElement => isValidElement(c) && !isStepComponent(c),
+  );
+  if (offenders.length === 0) return;
+  const labels = offenders.map((c) => {
+    const t = c.type;
+    if (typeof t === 'string') return `<${t}>`;
+    const named = t as { displayName?: string; name?: string };
+    return named?.displayName ?? named?.name ?? 'Component';
+  });
+  console.warn(
+    `<Steps> received non-<Step> children: ${labels.join(', ')}. ` +
+    'Wrap each step in `<Step title="…">…</Step>` so it renders as a numbered ' +
+    'step. Markdown headings inside <Steps> render as plain text.',
   );
 }
 
@@ -42,11 +57,12 @@ export const Steps = memo(function Steps({ children, titleSize = 'p' }: StepsPro
   const childrenArray = Children.toArray(children);
   const stepChildren = childrenArray.filter(isStepComponent);
   const totalSteps = stepChildren.length;
+  warnAboutNonStepChildren(childrenArray);
 
   let stepIndex = 0;
   return (
     <div className="relative my-8 space-y-0 not-prose">
-      {childrenArray.map((child, index) => {
+      {childrenArray.map((child) => {
         if (isStepComponent(child)) {
           const childProps = (child as any).props as StepProps;
           const currentStepIndex = stepIndex;