blume 0.6.0 → 0.6.2

package/src/openapi/render-mdx.ts

@@ -25,7 +25,13 @@ const ENTITIES: Record<string, string> = {
 // SDK…" is common spec prose). Entity-escape the keyword's first letter so the
 // construct can't match; it still renders as the literal word.
 const MDX_ESM_KEYWORD = /^(?<keyword>import|export)\b/gmu;
-const mdxSafe = (text: string): string =>
+// Backtick code — inline spans and fences alike — is already literal in MDX,
+// and entities are NOT decoded inside it, so escaping there would render the
+// entity text verbatim (`/pets/&#123;petId&#125;`). Matching any balanced
+// backtick run covers `code`, ``code``, and ```fences``` in one shot.
+const BACKTICK_CODE = /(?<bt>`+)[\s\S]*?\k<bt>/gu;
+
+const escapeProse = (text: string): string =>
   text
     .replace(MDX_UNSAFE, (char) => ENTITIES[char] ?? char)
     .replace(
@@ -33,6 +39,19 @@ const mdxSafe = (text: string): string =>
       (keyword) => `&#${keyword.codePointAt(0)};${keyword.slice(1)}`
     );
 
+/** Escape MDX-special syntax in prose while leaving backtick code verbatim. */
+const mdxSafe = (text: string): string => {
+  let out = "";
+  let cursor = 0;
+  for (const match of text.matchAll(BACKTICK_CODE)) {
+    const start = match.index ?? 0;
+    out += escapeProse(text.slice(cursor, start));
+    out += match[0];
+    cursor = start + match[0].length;
+  }
+  return out + escapeProse(text.slice(cursor));
+};
+
 /** Frontmatter + body for one operation or overview page. */
 export interface RenderedPage {
   data: Record<string, unknown>;
@@ -80,8 +99,16 @@ export const overviewMdx = (spec: ApiSpecData): RenderedPage => {
   // markdown pipeline gives them ids, permalink anchors, and table-of-contents
   // entries; only the operation-link list defers to a component.
   const operations = Object.values(spec.operations);
-  const sections = [...spec.tags];
-  const known = new Set(spec.tags.map((tag) => tag.slug));
+  // Dedupe by slug: two declared tags that slugify identically (`Store` and
+  // `store`) must render one section, not the same operation list twice.
+  const sections: typeof spec.tags = [];
+  const known = new Set<string>();
+  for (const tag of spec.tags) {
+    if (!known.has(tag.slug)) {
+      known.add(tag.slug);
+      sections.push(tag);
+    }
+  }
   for (const operation of operations) {
     if (!known.has(operation.tagSlug)) {
       known.add(operation.tagSlug);

package/src/openapi/source.ts

@@ -59,8 +59,10 @@ const specEntries = (
   );
   // Overview last so an operation sets the section's routePath before the index
   // page is inserted (the group's routePath is derived from its first child).
+  // A root-mounted reference refs `index.mdx`, not `/index.mdx`.
+  const base = routeToRef(spec.route);
   entries.push(
-    toEntry(overviewMdx(spec), `${routeToRef(spec.route)}/index.mdx`)
+    toEntry(overviewMdx(spec), base ? `${base}/index.mdx` : "index.mdx")
   );
   return entries;
 };

package/src/registry/eject.ts

@@ -12,6 +12,7 @@ import {
   buildRuntimeData,
   collectStaged,
   detectNeedsReact,
+  detectUsesMath,
 } from "../astro/generate.ts";
 import { discoverIslands } from "../astro/islands.ts";
 import { customOgRoutes, discoverPages, routeIsTaken } from "../astro/pages.ts";
@@ -102,19 +103,25 @@ export const eject = async (root: string): Promise<string[]> => {
   const exportPdf = config.export.pdf;
   const exportEpub = config.export.epub;
 
-  const [pages, needsReactRaw, userTheme, rawMarkdown, islands, examples] =
-    await Promise.all([
-      context.pagesRoot
-        ? discoverPages(context.pagesRoot)
-        : Promise.resolve([]),
-      detectNeedsReact(root),
-      context.themeFile
-        ? readFile(context.themeFile, "utf-8")
-        : Promise.resolve(""),
-      buildRawMarkdown(project),
-      discoverIslands(root),
-      discoverExamples(root, config.examples),
-    ]);
+  const [
+    pages,
+    needsReactRaw,
+    usesMath,
+    userTheme,
+    rawMarkdown,
+    islands,
+    examples,
+  ] = await Promise.all([
+    context.pagesRoot ? discoverPages(context.pagesRoot) : Promise.resolve([]),
+    detectNeedsReact(root),
+    detectUsesMath(root),
+    context.themeFile
+      ? readFile(context.themeFile, "utf-8")
+      : Promise.resolve(""),
+    buildRawMarkdown(project),
+    discoverIslands(root),
+    discoverExamples(root, config.examples),
+  ]);
   // Island/example frameworks drive which Astro renderers the ejected config
   // wires in; React also switches on for project `.tsx`/`.jsx` and Ask AI.
   const frameworks = new Set<string>([
@@ -192,7 +199,7 @@ export const eject = async (root: string): Promise<string[]> => {
         askEnabled,
         exportEpub,
         exportPdf,
-        mathEnabled: config.markdown.math,
+        mathEnabled: usesMath,
         needsReact,
       }),
       path: join(srcDir, "pages", "[...slug].astro"),

package/src/search/documents.ts

@@ -39,7 +39,10 @@ export interface SearchRecord {
 
 const CODE_FENCE = /```[\s\S]*?```/gu;
 const INLINE_CODE = /`(?<code>[^`]+)`/gu;
-const HTML_OR_JSX = /<[^>]+>/gu;
+// Tag-shaped only: a name (or closing slash/fragment) right after `<`, and no
+// newline inside. A bare `<` in prose ("costs < 5 credits") must not swallow
+// everything up to some later `>` — potentially whole paragraphs.
+const HTML_OR_JSX = /<\/?[a-zA-Z][^\n<>]*>|<\/?>/gu;
 const IMAGE = /!\[[^\]]*\]\([^)]*\)/gu;
 const LINK = /\[(?<text>[^\]]*)\]\([^)]*\)/gu;
 const HEADING_MARK = /^#{1,6}\s+/gmu;

package/src/theme/entry.ts

@@ -435,6 +435,13 @@ blume-diff {
   font-size: 0.8125rem;
 }
 
+/* GFM renders cells as <td><code> directly, which the descendant form alone
+   never matches (a cell is not its own descendant). */
+.prose :where(td, th) > code,
+.prose :where(td, th) :not(pre) > code {
+  white-space: nowrap;
+}
+
 blume-tabs pre,
 .not-prose > div > pre {
   background: var(--blume-code-background);
@@ -610,9 +617,9 @@ pre:has(.line.focused):hover .line:not(.focused) {
   content: none;
 }
 
-/* Inline code highlighting (markdown.code.inline): Shiki colors the tokens of a
-   \`code\`{:lang} snippet via the same dual-theme CSS variables as fenced blocks,
-   keeping the inline pill background. */
+/* Inline code highlighting: Shiki colors the tokens of a \`code\`{:lang} snippet
+   via the same dual-theme CSS variables as fenced blocks, keeping the inline
+   pill background. Always on — it only fires on the trailing {:lang} marker. */
 .prose code.blume-inline-code span {
   color: var(--shiki-light);
 }

package/src/theme/icons.ts

@@ -79,14 +79,10 @@ export const resolveIcon = (name: string): ResolvedIcon | null => {
   return fromSet(DEFAULT_SET, normalized);
 };
 
-/** Whether a name resolves to a known Lucide icon. */
-export const hasIcon = (name: string): boolean => {
-  if (resolveIcon(name)) {
-    return true;
-  }
-  const normalized = normalize(name);
-  const bare = normalized.includes(":")
-    ? normalized.slice(normalized.indexOf(":") + 1)
-    : normalized;
-  return Object.values(SETS).some((set) => getIconData(set, bare) !== null);
-};
+/**
+ * Whether a name resolves to a renderable icon. Exactly mirrors
+ * {@link resolveIcon}: a laxer check (e.g. matching the bare name under an
+ * unknown prefix) would let callers suppress their fallback and diagnostics
+ * for a name `<Icon>` then renders as nothing.
+ */
+export const hasIcon = (name: string): boolean => resolveIcon(name) !== null;