npm - jamdesk - Versions diffs - 1.1.69 → 1.1.71 - Mend

jamdesk 1.1.69 → 1.1.71

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (75) hide show

package/dist/__tests__/integration/validate.integration.test.js +21 -0
package/dist/__tests__/integration/validate.integration.test.js.map +1 -1
package/dist/__tests__/unit/extract-hooks.test.js +102 -1
package/dist/__tests__/unit/extract-hooks.test.js.map +1 -1
package/dist/__tests__/unit/mdx-validator.test.js +22 -0
package/dist/__tests__/unit/mdx-validator.test.js.map +1 -1
package/dist/__tests__/unit/migrate-mdx.test.js +121 -1
package/dist/__tests__/unit/migrate-mdx.test.js.map +1 -1
package/dist/__tests__/unit/relative-mdx-imports.test.d.ts +2 -0
package/dist/__tests__/unit/relative-mdx-imports.test.d.ts.map +1 -0
package/dist/__tests__/unit/relative-mdx-imports.test.js +452 -0
package/dist/__tests__/unit/relative-mdx-imports.test.js.map +1 -0
package/dist/__tests__/unit/relocate-snippets.test.d.ts +2 -0
package/dist/__tests__/unit/relocate-snippets.test.d.ts.map +1 -0
package/dist/__tests__/unit/relocate-snippets.test.js +542 -0
package/dist/__tests__/unit/relocate-snippets.test.js.map +1 -0
package/dist/__tests__/unit/run-build-script.test.js +23 -1
package/dist/__tests__/unit/run-build-script.test.js.map +1 -1
package/dist/__tests__/unit/warn-relative-imports.test.d.ts +2 -0
package/dist/__tests__/unit/warn-relative-imports.test.d.ts.map +1 -0
package/dist/__tests__/unit/warn-relative-imports.test.js +44 -0
package/dist/__tests__/unit/warn-relative-imports.test.js.map +1 -0
package/dist/commands/dev.d.ts.map +1 -1
package/dist/commands/dev.js +22 -8
package/dist/commands/dev.js.map +1 -1
package/dist/commands/migrate/convert-mdx.d.ts +5 -1
package/dist/commands/migrate/convert-mdx.d.ts.map +1 -1
package/dist/commands/migrate/convert-mdx.js +19 -6
package/dist/commands/migrate/convert-mdx.js.map +1 -1
package/dist/commands/migrate/extract-hooks.d.ts +26 -0
package/dist/commands/migrate/extract-hooks.d.ts.map +1 -1
package/dist/commands/migrate/extract-hooks.js +71 -12
package/dist/commands/migrate/extract-hooks.js.map +1 -1
package/dist/commands/migrate/fix-mdx-syntax.d.ts +38 -0
package/dist/commands/migrate/fix-mdx-syntax.d.ts.map +1 -0
package/dist/commands/migrate/fix-mdx-syntax.js +80 -0
package/dist/commands/migrate/fix-mdx-syntax.js.map +1 -0
package/dist/commands/migrate/index.d.ts.map +1 -1
package/dist/commands/migrate/index.js +165 -3
package/dist/commands/migrate/index.js.map +1 -1
package/dist/commands/migrate/relocate-snippets.d.ts +30 -0
package/dist/commands/migrate/relocate-snippets.d.ts.map +1 -0
package/dist/commands/migrate/relocate-snippets.js +357 -0
package/dist/commands/migrate/relocate-snippets.js.map +1 -0
package/dist/commands/validate.d.ts.map +1 -1
package/dist/commands/validate.js +31 -1
package/dist/commands/validate.js.map +1 -1
package/dist/lib/mdx-syntax-fixes.d.ts +14 -0
package/dist/lib/mdx-syntax-fixes.d.ts.map +1 -0
package/dist/lib/mdx-syntax-fixes.js +38 -0
package/dist/lib/mdx-syntax-fixes.js.map +1 -0
package/dist/lib/mdx-validator.d.ts +11 -2
package/dist/lib/mdx-validator.d.ts.map +1 -1
package/dist/lib/mdx-validator.js +41 -4
package/dist/lib/mdx-validator.js.map +1 -1
package/dist/lib/relative-mdx-imports.d.ts +44 -0
package/dist/lib/relative-mdx-imports.d.ts.map +1 -0
package/dist/lib/relative-mdx-imports.js +145 -0
package/dist/lib/relative-mdx-imports.js.map +1 -0
package/dist/lib/run-build-script.d.ts.map +1 -1
package/dist/lib/run-build-script.js +7 -0
package/dist/lib/run-build-script.js.map +1 -1
package/dist/lib/warn-relative-imports.d.ts +2 -0
package/dist/lib/warn-relative-imports.d.ts.map +1 -0
package/dist/lib/warn-relative-imports.js +29 -0
package/dist/lib/warn-relative-imports.js.map +1 -0
package/package.json +1 -1
package/vendored/app/(unlock)/jd/unlock/page.tsx +32 -0
package/vendored/app/[[...slug]]/page.tsx +63 -15
package/vendored/components/navigation/SocialFooter.tsx +4 -18
package/vendored/lib/branding-url.ts +9 -0
package/vendored/lib/layout-helpers.tsx +7 -9
package/vendored/lib/preprocess-mdx.ts +42 -0
package/vendored/scripts/compile-snippets.cjs +50 -7
package/vendored/workspace-package-lock.json +3 -3

package/vendored/lib/branding-url.ts ADDED Viewed

@@ -0,0 +1,9 @@
+// `NEXT_PUBLIC_PROJECT_SLUG` is read inline so Next.js inlines the value at build
+// time for client components (SocialFooter). Hoisting the read would defeat the
+// inline. Server components (the unlock page) just see it as runtime env.
+export function getBrandingUrl(projectSlug?: string | null): string {
+  const slug = encodeURIComponent(
+    projectSlug || process.env.NEXT_PUBLIC_PROJECT_SLUG || 'docs'
+  );
+  return `https://www.jamdesk.com?utm_campaign=poweredBy&utm_medium=referral&utm_source=${slug}`;
+}

package/vendored/lib/layout-helpers.tsx CHANGED Viewed

@@ -234,7 +234,9 @@ export async function DocsChrome({
   const linkPrefix = config.hostAtDocs ? '/docs' : '';
-  const analyticsScript = config.analytics?.enabled !== false
+  // Skip building the IIFE string in dev — both the script render below and
+  // the IIFE itself self-gate to non-localhost (analytics-script.ts:49).
+  const analyticsScript = process.env.NODE_ENV === 'production' && config.analytics?.enabled !== false
     ? getAnalyticsScript(resolvedProjectSlug)
     : null;
@@ -449,14 +451,10 @@ export async function DocsChrome({
           <HtmlLangSync defaultLanguage={projectDefaultLang} />
         </ThemeProvider>
         {/*
-          Inline scripts gated to production: avoids React 19's "scripts inside
-          React components are never executed when rendering on the client"
-          warning in `jamdesk dev`. All three are no-ops on localhost anyway —
-          Crisp won't load on dev domains, customJs is typically tracking code,
-          and analyticsScript self-gates to non-localhost (analytics-script.ts:49).
-          Same pattern as the project-subdomain hydration suppressor in <head>.
-          (Crisp websiteId pre-validated by regex; customJs/analyticsScript are
-          owner-controlled — same trust model as before this gate was added.)
+          Inline scripts gated to production — avoids React 19's "scripts
+          inside React components are never executed when rendering on the
+          client" warning in `jamdesk dev`. Same pattern as the
+          project-subdomain hydration suppressor in <head>.
         */}
         {process.env.NODE_ENV === 'production' && config.integrations?.crisp?.websiteId &&
           /^[a-f0-9-]{36}$/.test(config.integrations.crisp.websiteId) && (

package/vendored/lib/preprocess-mdx.ts CHANGED Viewed

@@ -518,6 +518,42 @@ export function normalizeJsxIndentation(content: string): string {
   return result.join('\n');
 }
+/**
+ * Un-indent a closing JSX tag absorbed into the previous markdown list item.
+ *
+ * Distinct from `unindentJsxAfterListItems` below: that one rewrites indented
+ * *opening* tags inside list context. Here the opening tag is at root level
+ * and only the closing tag is indented after the list — markdown reads it as
+ * listItem continuation, so MDX compiles see the parent component as unclosed
+ * ("Expected the closing tag `</Card>` either after the end of `listItem`...").
+ *
+ * CLI mirror: `builder/cli/src/lib/mdx-syntax-fixes.ts` — keep regex in sync.
+ */
+export function unindentClosingTagAfterList(content: string): string {
+  // Hot-path prefilter — preprocessMdx runs on every page render, and most
+  // MDX has no indented closing tags at all. Skip the split+walk in that case.
+  if (!/^\s+<\/[A-Z]/m.test(content)) return content;
+  const closingTagPattern = /^(\s+)(<\/[A-Z][\w-]*>)\s*$/;
+  const listItemPattern = /^\s*(?:[-*+]\s|\d+\.\s)/;
+  const lines = content.split('\n');
+  const out: string[] = [];
+  let lastNonBlank = '';
+  for (const line of lines) {
+    const closing = line.match(closingTagPattern);
+    if (closing && listItemPattern.test(lastNonBlank)) {
+      out.push(closing[2]);
+      lastNonBlank = closing[2];
+      continue;
+    }
+    out.push(line);
+    if (line.trim() !== '') lastNonBlank = line;
+  }
+  return out.join('\n');
+}
 /**
  * Un-indent JSX blocks that follow markdown list items.
  *
@@ -997,6 +1033,12 @@ export function preprocessMdx(content: string, options?: { assetVersion?: string
   // Convert noStyle to data-no-style for React compatibility
   processed = convertNoStyleToDataAttr(processed);
+  // Un-indent absorbed closing tags FIRST — handles the case where the
+  // opening tag is at root level but the closing tag got indented after a
+  // list. unindentJsxAfterListItems below only catches cases where the
+  // opening tag itself is indented inside list context.
+  processed = unindentClosingTagAfterList(processed);
   // Un-indent JSX blocks that follow list items to prevent
   // "Expected closing tag before end of listItem" errors
   // This must run BEFORE ensureBlankLinesInJsx since it changes block structure

package/vendored/scripts/compile-snippets.cjs CHANGED Viewed

@@ -321,6 +321,22 @@ export const SnippetComponents: Record<string, React.ComponentType<any>> = {};
   const snippetFiles = findSnippetFiles(snippetsDir);
+  // Deterministic order: within a (dir, baseName) collision, process .mdx
+  // first so the .tsx/.jsx code file overwrites the wrapped-MDX output on
+  // disk. Without this, fs.readdir order varies by platform and the
+  // surviving file content is non-deterministic. Outside of collisions,
+  // the order doesn't matter — sort makes it stable.
+  const EXT_PRIORITY = { '.mdx': 0, '.md': 1, '.ts': 2, '.js': 3, '.tsx': 4, '.jsx': 5 };
+  snippetFiles.sort((a, b) => {
+    const aDir = path.join(a.dir, '');
+    const bDir = path.join(b.dir, '');
+    if (aDir !== bDir) return aDir < bDir ? -1 : 1;
+    const aBase = a.file.replace(/\.[^.]+$/, '');
+    const bBase = b.file.replace(/\.[^.]+$/, '');
+    if (aBase !== bBase) return aBase < bBase ? -1 : 1;
+    return (EXT_PRIORITY[path.extname(a.file)] ?? 99) - (EXT_PRIORITY[path.extname(b.file)] ?? 99);
+  });
   if (snippetFiles.length === 0) {
     console.log('   No snippet files found');
@@ -376,7 +392,8 @@ export const SnippetComponents: Record<string, React.ComponentType<any>> = {};
           name: exportName,
           dir: dir,
           fileName,
-          needsClient
+          needsClient,
+          sourceFile: dir ? path.join(dir, file) : file,
         });
         const status = written ? '(updated)' : '(unchanged)';
@@ -399,7 +416,8 @@ export const SnippetComponents: Record<string, React.ComponentType<any>> = {};
         name: baseName,
         dir: dir,
         fileName,
-        needsClient
+        needsClient,
+        sourceFile: dir ? path.join(dir, file) : file,
       });
       const status = written ? '(updated)' : '(unchanged)';
@@ -414,8 +432,33 @@ export const SnippetComponents: Record<string, React.ComponentType<any>> = {};
   // For locale-specific snippets (e.g., es/Counter), we export under prefixed name (EsCounter).
   // The buildSnippetAliasMap function handles mapping import aliases to the correct component.
+  // Dedupe by (dir, name): two source files with the same base name (e.g.
+  // `vercel-json-generator.mdx` + `vercel-json-generator.tsx`) compile to the
+  // same generated fileName. The on-disk file is correctly overwritten by
+  // the last writer (sorted above so code files win), but componentInfo
+  // would otherwise carry both entries and produce duplicate import/export
+  // lines in ProjectSnippets.tsx. Keep the last (= the code file).
+  const seenComponentKeys = new Map();
+  for (const c of componentInfo) {
+    const key = `${c.dir}|${c.name}`;
+    if (seenComponentKeys.has(key)) {
+      const prior = seenComponentKeys.get(key);
+      // stderr (not stdout) so `runBuildScript` surfaces this in non-verbose
+      // mode — without it, the `jamdesk dev` user sees the collision is
+      // silently resolved and the next collision (a different name) crashes
+      // Turbopack with no breadcrumb.
+      console.warn(
+        `   ⚠ Snippet name collision: "${c.name}" defined in both ` +
+        `${prior.sourceFile} and ${c.sourceFile} — using ${c.sourceFile}. ` +
+        `Remove or rename one of these files to clear this warning.`,
+      );
+    }
+    seenComponentKeys.set(key, c);
+  }
+  const dedupedComponentInfo = [...seenComponentKeys.values()];
   // Build unique import names for each component (prefixed with dir if needed)
-  const importInfo = componentInfo.map(c => {
+  const importInfo = dedupedComponentInfo.map(c => {
     const prefix = c.dir ? toPascalCase(c.dir.replace(/\//g, '-')) : '';
     const uniqueName = prefix + c.name.charAt(0).toUpperCase() + c.name.slice(1);
     // Strip the .tsx extension from fileName to get the import path
@@ -468,17 +511,17 @@ ${importInfo.map(c => `  ${c.uniqueName},`).join('\n')}
     }
   }
-  const clientCount = componentInfo.filter(c => c.needsClient).length;
-  const serverCount = componentInfo.length - clientCount;
+  const clientCount = dedupedComponentInfo.filter(c => c.needsClient).length;
+  const serverCount = dedupedComponentInfo.length - clientCount;
   console.log(`\n✅ Generated snippets:`);
   console.log(`   Index: ${outputPath} ${indexWritten ? '(updated)' : '(unchanged)'}`);
-  console.log(`   Components: ${generatedDir}/ (${componentInfo.length} files)`);
+  console.log(`   Components: ${generatedDir}/ (${dedupedComponentInfo.length} files)`);
   if (removedCount > 0) {
     console.log(`   Removed: ${removedCount} stale file(s)`);
   }
   console.log(`   Client components: ${clientCount}, Server components: ${serverCount}`);
-  console.log(`   Exports: ${componentInfo.map(c => c.name).join(', ')}\n`);
+  console.log(`   Exports: ${dedupedComponentInfo.map(c => c.name).join(', ')}\n`);
 }
 // Run the compilation

package/vendored/workspace-package-lock.json CHANGED Viewed

@@ -2153,9 +2153,9 @@
       }
     },
     "node_modules/baseline-browser-mapping": {
-      "version": "2.10.25",
-      "resolved": "https://registry.npmjs.org/baseline-browser-mapping/-/baseline-browser-mapping-2.10.25.tgz",
-      "integrity": "sha512-QO/VHsXCQdnzADMfmkeOPvHdIAkoB7i0/rGjINPJEetLx75hNttVWGQ/jycHUDP9zZ9rupbm60WRxcwViB0MiA==",
+      "version": "2.10.27",
+      "resolved": "https://registry.npmjs.org/baseline-browser-mapping/-/baseline-browser-mapping-2.10.27.tgz",
+      "integrity": "sha512-zEs/ufmZoUd7WftKpKyXaT6RFxpQ5Qm9xytKRHvJfxFV9DFJkZph9RvJ1LcOUi0Z1ZVijMte65JbILeV+8QQEA==",
       "license": "Apache-2.0",
       "bin": {
         "baseline-browser-mapping": "dist/cli.cjs"