jamdesk 1.1.145 → 1.1.147

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jamdesk",
-  "version": "1.1.145",
+  "version": "1.1.147",
   "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/NativeSubscribeForm.tsx

@@ -12,11 +12,21 @@ export interface NativeSubscribeFormProps {
   className?: string;
 }
 
+// Shared control height for the email field + Subscribe button so they line up
+// exactly. The input must carry 16px text (iOS no-zoom rule) which otherwise
+// makes it taller than the smaller-font button; pinning both to one height with
+// border-box collapses the mismatch. 2rem (32px) stays compact yet clears the
+// WCAG 24px minimum target.
+const CONTROL_HEIGHT = '2rem';
+
 // Solid primary button (the collapsed trigger and the form's submit share it).
 // Kept compact — the iOS 16px-min rule is for inputs, not buttons. Radius tracks
 // the theme's button token (--radius-md) so it matches the docs' own buttons.
+// inline-flex centers the label inside the fixed CONTROL_HEIGHT box.
 const PRIMARY_BUTTON: CSSProperties = {
-  fontSize: '0.875rem', padding: '0.4rem 0.85rem', border: 0,
+  display: 'inline-flex', alignItems: 'center', justifyContent: 'center',
+  height: CONTROL_HEIGHT, boxSizing: 'border-box',
+  fontSize: '0.8125rem', padding: '0 0.85rem', border: 0,
   borderRadius: 'var(--radius-md, 0.375rem)', cursor: 'pointer',
   background: 'var(--color-primary, #2563eb)', color: '#fff',
 };
@@ -189,6 +199,11 @@ export function NativeSubscribeForm({ provider, title, description, collapsed, c
     <Shell className={className}>
       {titleEl}
       {description && <p className="mb-3 text-sm text-theme-text-secondary">{description}</p>}
+      {/* The field keeps 16px text (iOS no-zoom), but the placeholder reads a
+          touch smaller — inline styles can't reach ::placeholder, so scope it by
+          the stable jd-emailsubscribe hook. Smaller ::placeholder font doesn't
+          re-trigger iOS zoom (that keys off the input's own font-size). */}
+      <style>{`.jd-emailsubscribe input[type="email"]::placeholder{font-size:0.875rem}`}</style>
       <form onSubmit={onSubmit} style={{ display: 'flex', gap: '0.5rem', flexWrap: 'wrap', alignItems: 'center', maxWidth: '28rem' }}>
         {/* Honeypot: a deliberately odd name (NOT website/url/email, which password
             managers autofill — a filled honeypot silently drops a real user). Bots
@@ -196,11 +211,12 @@ export function NativeSubscribeForm({ provider, title, description, collapsed, c
         <input type="text" name="jd_hp" tabIndex={-1} autoComplete="off" aria-hidden="true"
           defaultValue="" style={{ position: 'absolute', left: '-5000px' }} />
         {/* font-size:16px avoids iOS auto-zoom (monorepo UI rule). primary-bg field
-            pops against the card's secondary-bg surface; radius tracks the theme. */}
+            pops against the card's secondary-bg surface; radius tracks the theme.
+            height+border-box matches the Subscribe button exactly. */}
         <input type="email" name="email" required value={email}
           onChange={(e) => { setEmail(e.target.value); if (status === 'error') setStatus('idle'); }}
-          disabled={status === 'submitting'} placeholder="you@example.com" aria-label="Email address"
-          style={{ flex: 1, minWidth: 0, fontSize: '16px', padding: '0.4rem 0.75rem', border: 'var(--border-width, 1px) solid var(--color-border, #d4d4d8)', borderRadius: 'var(--radius-md, 0.375rem)', background: 'var(--color-bg-primary, #fff)', color: 'var(--color-text-primary, #18181b)' }} />
+          disabled={status === 'submitting'} placeholder="you@email.com" aria-label="Email address"
+          style={{ flex: 1, minWidth: 0, height: CONTROL_HEIGHT, boxSizing: 'border-box', fontSize: '16px', padding: '0 0.7rem', border: 'var(--border-width, 1px) solid var(--color-border, #d4d4d8)', borderRadius: 'var(--radius-md, 0.375rem)', background: 'var(--color-bg-primary, #fff)', color: 'var(--color-text-primary, #18181b)' }} />
         <button type="submit" disabled={status === 'submitting'} style={PRIMARY_BUTTON}>
           {status === 'submitting' ? 'Subscribing…' : 'Subscribe'}
         </button>

package/vendored/components/mdx/Update.tsx

@@ -1,4 +1,7 @@
+'use client';
+
 import { generateSlug } from '@/lib/heading-extractor';
+import { useUpdateSlug } from './UpdateSlugContext';
 
 interface UpdateProps {
   label?: string;
@@ -12,6 +15,11 @@ interface UpdateProps {
  * Generates a URL-friendly slug from a label string.
  * Used to create anchor IDs for Update components.
  * Uses shared generateSlug to stay in sync with TOC and link validation.
+ *
+ * Stateless: two Updates with the same label produce the same slug here. The
+ * Update component itself resolves duplicate-label collisions via useUpdateSlug
+ * (page-scoped dedup); this export stays for the no-provider fallback, the RSS
+ * feed's parallel anchor algorithm, and link validation.
  */
 export function generateUpdateId(label?: string): string | undefined {
   if (!label) return undefined;
@@ -21,9 +29,13 @@ export function generateUpdateId(label?: string): string | undefined {
 /**
  * Update component - for changelog/whatsnew entries.
  * Creates timeline-style entries with automatic anchor links and TOC integration.
+ *
+ * Anchor id comes from useUpdateSlug so duplicate labels on one page (e.g. two
+ * "June 2026" entries) get distinct, TOC-aligned ids (`june-2026`,
+ * `june-2026-1`) instead of colliding on a single `#june-2026`.
  */
 export function Update({ label, description, tags, date, children }: UpdateProps) {
-  const id = generateUpdateId(label);
+  const id = useUpdateSlug(label);
 
   return (
     <div

package/vendored/components/mdx/UpdateSlugContext.tsx

@@ -0,0 +1,60 @@
+'use client';
+
+import { createContext, useContext, useRef, ReactNode } from 'react';
+import { generateSlug } from '@/lib/heading-extractor';
+
+export interface UpdateSlugEntry {
+  label: string;
+  slug: string;
+}
+
+interface UpdateSlugContextValue {
+  // Per-label counter — advances each time useUpdateSlug is called for that label.
+  counts: Map<string, number>;
+  entries: UpdateSlugEntry[];
+}
+
+const UpdateSlugCtx = createContext<UpdateSlugContextValue | null>(null);
+
+export function UpdateSlugProvider({
+  entries,
+  children,
+}: {
+  entries: UpdateSlugEntry[];
+  children: ReactNode;
+}) {
+  // useRef so the same Map persists across renders. We rely on the same
+  // contract React's own useId rests on: source-order rendering during SSR
+  // and during client hydration produces matching ids.
+  const ref = useRef<UpdateSlugContextValue | null>(null);
+  if (ref.current === null || ref.current.entries !== entries) {
+    ref.current = { counts: new Map(), entries };
+  }
+  return <UpdateSlugCtx.Provider value={ref.current}>{children}</UpdateSlugCtx.Provider>;
+}
+
+/**
+ * Resolve the unique slug for an <Update> with this label. Each call advances
+ * the per-label counter, so two Updates with the same label (e.g. two
+ * "June 2026" changelog entries) get sequential slugs from the provider's
+ * `entries` list (`june-2026`, `june-2026-1`) — matching the page-scoped
+ * github-slugger ids that extractHeadings (and the TOC) already compute. Falls
+ * back to a stateless `generateSlug(label)` if no provider is mounted
+ * (preview/storybook/etc), which preserves the pre-dedup single-Update behavior.
+ */
+export function useUpdateSlug(label: string | undefined): string | undefined {
+  const ctx = useContext(UpdateSlugCtx);
+  if (!label) return undefined;
+  if (!ctx) return generateSlug(label) || undefined;
+
+  const idx = ctx.counts.get(label) ?? 0;
+  ctx.counts.set(label, idx + 1);
+
+  let seen = 0;
+  for (const entry of ctx.entries) {
+    if (entry.label !== label) continue;
+    if (seen === idx) return entry.slug;
+    seen += 1;
+  }
+  return generateSlug(label) || undefined;
+}

package/vendored/lib/heading-extractor.ts

@@ -15,6 +15,8 @@ export interface HeadingInfo {
   line: number; // 1-indexed
   /** When the entry is a <Step> inside a <Steps> block, this is its 1-based index within the block. */
   stepNumber?: number;
+  /** True when the entry came from an <Update label="..."> anchor (vs a markdown heading). */
+  isUpdate?: boolean;
 }
 
 /**
@@ -44,7 +46,13 @@ export function generateSlug(text: string): string {
 
 const HEADING_REGEX = /^(#{1,6})\s+(.+)$/;
 const FENCE_REGEX = /^(`{3,}|~{3,})/;
-const UPDATE_LABEL_REGEX = /<Update\s+label=["']([^"']+)["']/;
+// Global + matchAll (like STEP_TITLE_REGEX below) so multiple inline <Update>
+// tags on one line each get an anchor, and `[^>]*` so `label=` need not be the
+// first attribute (e.g. `<Update date=".." label="..">`). Alternation (not a
+// character class) keeps an apostrophe inside a double-quoted label from
+// terminating the capture. MUST only be used with matchAll — direct test/exec
+// would share lastIndex across calls and miscount.
+const UPDATE_LABEL_REGEX = /<Update\s+[^>]*label=(?:"([^"]+)"|'([^']+)')/g;
 const STEPS_OPEN_REGEX = /<Steps(\s|>)/;
 const STEPS_CLOSE_REGEX = /<\/Steps>/;
 // Global flag — we iterate with matchAll so authors who inline multiple
@@ -114,13 +122,11 @@ export function extractHeadings(content: string): HeadingInfo[] {
       }
     }
 
-    const updateMatch = line.match(UPDATE_LABEL_REGEX);
-    if (updateMatch) {
-      const text = updateMatch[1];
-      if (generateSlug(text)) {
-        const id = slugger.slug(text);
-        headings.push({ id, text, level: 2, line: i + 1 });
-      }
+    for (const match of line.matchAll(UPDATE_LABEL_REGEX)) {
+      const text = match[1] ?? match[2];
+      if (!generateSlug(text)) continue;
+      const id = slugger.slug(text);
+      headings.push({ id, text, level: 2, line: i + 1, isUpdate: true });
     }
 
     if (inStepsBlock) {

package/vendored/lib/render-doc-page.tsx

@@ -56,6 +56,7 @@ import { recmaGuardExpressions } from '@/lib/recma-guard-expressions';
 import { extractInlineComponents } from '@/lib/process-mdx-with-exports';
 import { extractHeadings } from '@/lib/heading-extractor';
 import { StepSlugProvider, type StepSlugEntry } from '@/components/mdx/StepSlugContext';
+import { UpdateSlugProvider, type UpdateSlugEntry } from '@/components/mdx/UpdateSlugContext';
 import { buildEndpointFromMdx } from '@/lib/build-endpoint-from-mdx';
 import { mdxSecurityOptions } from '@/lib/mdx-security-options';
 import { getContentDir } from '@/lib/docs';
@@ -365,9 +366,16 @@ export async function renderDocPage(input: RenderInput): Promise<ReactElement> {
     logger.warn(`[MDX] preprocess failed for ${pagePath}: ${preprocessError}`);
   }
 
-  const stepEntries: StepSlugEntry[] = extractHeadings(content)
+  // Both lists are derived from the SAME page-scoped slugger pass (extractHeadings),
+  // so Step and Update ids share one dedup namespace and never collide with each
+  // other or with markdown heading ids.
+  const pageHeadings = extractHeadings(content);
+  const stepEntries: StepSlugEntry[] = pageHeadings
     .filter(h => typeof h.stepNumber === 'number')
     .map(h => ({ title: h.text, slug: h.id }));
+  const updateEntries: UpdateSlugEntry[] = pageHeadings
+    .filter(h => h.isUpdate)
+    .map(h => ({ label: h.text, slug: h.id }));
 
   // [render-timing] proves the snippet/inline parallel win. With Promise.all,
   // wall-clock should be ~max(R2_snippets, CPU_inline). Compare against the
@@ -754,17 +762,19 @@ export async function renderDocPage(input: RenderInput): Promise<ReactElement> {
               ) : (
                 <MdxRenderBoundary fallback={<MdxErrorBlock label="page content" />}>
                   <StepSlugProvider entries={stepEntries}>
-                    <MDXRemote
-                      source={effectiveSource}
-                      components={ComponentsWithUnknownFallback}
-                      options={{
-                        ...mdxSecurityOptions,
-                        mdxOptions: {
-                          ...getCommonMdxOptions(config, highlighter),
-                          recmaPlugins: [recmaCompoundComponents, recmaGuardExpressions],
-                        },
-                      }}
-                    />
+                    <UpdateSlugProvider entries={updateEntries}>
+                      <MDXRemote
+                        source={effectiveSource}
+                        components={ComponentsWithUnknownFallback}
+                        options={{
+                          ...mdxSecurityOptions,
+                          mdxOptions: {
+                            ...getCommonMdxOptions(config, highlighter),
+                            recmaPlugins: [recmaCompoundComponents, recmaGuardExpressions],
+                          },
+                        }}
+                      />
+                    </UpdateSlugProvider>
                   </StepSlugProvider>
                 </MdxRenderBoundary>
               )}
@@ -784,17 +794,19 @@ export async function renderDocPage(input: RenderInput): Promise<ReactElement> {
   ) : (
     <MdxRenderBoundary fallback={<MdxErrorBlock label="page content" />}>
       <StepSlugProvider entries={stepEntries}>
-        <MDXRemote
-          source={content}
-          components={ComponentsWithUnknownFallback}
-          options={{
-            ...mdxSecurityOptions,
-            mdxOptions: {
-              ...getCommonMdxOptions(config, highlighter),
-              recmaPlugins: [recmaCompoundComponents, recmaGuardExpressions],
-            },
-          }}
-        />
+        <UpdateSlugProvider entries={updateEntries}>
+          <MDXRemote
+            source={content}
+            components={ComponentsWithUnknownFallback}
+            options={{
+              ...mdxSecurityOptions,
+              mdxOptions: {
+                ...getCommonMdxOptions(config, highlighter),
+                recmaPlugins: [recmaCompoundComponents, recmaGuardExpressions],
+              },
+            }}
+          />
+        </UpdateSlugProvider>
       </StepSlugProvider>
     </MdxRenderBoundary>
   );