@writechoice/mint-cli 0.0.15 → 0.0.17

package/bin/cli.js

@@ -135,6 +135,42 @@ fix
     await fixCodeblocks(mergedOptions);
   });
 
+// Fix inlineimages subcommand
+fix
+  .command("inlineimages")
+  .description("Convert inline images to <InlineImage> components in MDX files")
+  .option("-f, --file <path>", "Fix a single MDX file directly")
+  .option("-d, --dir <path>", "Fix MDX files in a specific directory")
+  .option("--dry-run", "Preview changes without writing files")
+  .option("--quiet", "Suppress terminal output")
+  .action(async (options) => {
+    const { loadConfig, mergeInlineImagesConfig } = await import("../src/utils/config.js");
+    const { fixInlineImages } = await import("../src/commands/fix/inlineimages.js");
+
+    const config = loadConfig();
+    const mergedOptions = mergeInlineImagesConfig(options, config);
+    mergedOptions.verbose = !mergedOptions.quiet;
+    await fixInlineImages(mergedOptions);
+  });
+
+// Fix h1 subcommand
+fix
+  .command("h1")
+  .description("Remove duplicate H1 headings that match the frontmatter title in MDX files")
+  .option("-f, --file <path>", "Fix a single MDX file directly")
+  .option("-d, --dir <path>", "Fix MDX files in a specific directory")
+  .option("--dry-run", "Preview changes without writing files")
+  .option("--quiet", "Suppress terminal output")
+  .action(async (options) => {
+    const { loadConfig, mergeH1Config } = await import("../src/utils/config.js");
+    const { fixH1 } = await import("../src/commands/fix/h1.js");
+
+    const config = loadConfig();
+    const mergedOptions = mergeH1Config(options, config);
+    mergedOptions.verbose = !mergedOptions.quiet;
+    await fixH1(mergedOptions);
+  });
+
 // Fix images subcommand
 fix
   .command("images")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@writechoice/mint-cli",
-  "version": "0.0.15",
+  "version": "0.0.17",
   "description": "CLI tool for Mintlify documentation validation and utilities",
   "main": "src/index.js",
   "type": "module",

package/src/commands/fix/h1.js

@@ -0,0 +1,182 @@
+/**
+ * Fix H1 Tool
+ *
+ * Removes duplicate H1 headings that match the frontmatter title field.
+ *
+ * If the first non-empty line after frontmatter is an H1 exactly equal
+ * to the frontmatter `title`, that line (and the immediately following
+ * blank line, if any) is removed.
+ *
+ * This mirrors the behaviour of remove_double_titles.py.
+ */
+
+import { existsSync, readdirSync, statSync, readFileSync, writeFileSync } from "fs";
+import { join, relative, resolve } from "path";
+import chalk from "chalk";
+
+const EXCLUDED_DIRS = ["node_modules", ".git"];
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Frontmatter helpers
+// ─────────────────────────────────────────────────────────────────────────────
+
+const FRONTMATTER_RE = /^---\r?\n[\s\S]*?\r?\n---\r?\n?/;
+const TITLE_LINE_RE = /^\s*title\s*:\s*["']?(.*?)["']?\s*$/im;
+const H1_RE = /^\s*#\s+(.*?)\s*$/;
+
+/**
+ * Returns the frontmatter title value, or null if not found.
+ */
+function extractFrontmatterTitle(content) {
+  const fmMatch = FRONTMATTER_RE.exec(content);
+  if (!fmMatch) return null;
+  const fmText = fmMatch[0];
+  const titleMatch = TITLE_LINE_RE.exec(fmText);
+  return titleMatch ? titleMatch[1].trim() : null;
+}
+
+/**
+ * Removes the duplicate H1 from `content` if present.
+ * Returns { newContent, changed }.
+ */
+function removeDuplicateH1(content, fmTitle) {
+  const fmMatch = FRONTMATTER_RE.exec(content);
+  if (!fmMatch) return { newContent: content, changed: false };
+
+  const fmEnd = fmMatch[0].length;
+  const afterFm = content.slice(fmEnd);
+
+  const lines = afterFm.split("\n");
+
+  // Find the first non-empty, non-import line after frontmatter
+  let targetIdx = null;
+  for (let i = 0; i < lines.length; i++) {
+    const trimmed = lines[i].trim();
+    if (trimmed === "") continue;
+    if (/^import\s/.test(trimmed)) continue; // skip import statements
+
+    const h1Match = H1_RE.exec(lines[i]);
+    if (h1Match && h1Match[1].trim() === fmTitle) {
+      targetIdx = i;
+    }
+    // Either it matched or it didn't — stop after first content line
+    break;
+  }
+
+  if (targetIdx === null) return { newContent: content, changed: false };
+
+  // Remove the H1 line
+  const newLines = [...lines.slice(0, targetIdx), ...lines.slice(targetIdx + 1)];
+
+  // Remove the immediately following blank line (now at targetIdx)
+  if (newLines[targetIdx] !== undefined && newLines[targetIdx].trim() === "") {
+    newLines.splice(targetIdx, 1);
+  }
+
+  const newContent = content.slice(0, fmEnd) + newLines.join("\n");
+  return { newContent, changed: true };
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// File discovery
+// ─────────────────────────────────────────────────────────────────────────────
+
+function findMdxFiles(repoRoot, directory = null, file = null) {
+  if (file) {
+    const fullPath = resolve(repoRoot, file);
+    return existsSync(fullPath) ? [fullPath] : [];
+  }
+
+  const searchDirs = directory ? [resolve(repoRoot, directory)] : [repoRoot];
+  const mdxFiles = [];
+
+  function walkDirectory(dir) {
+    const dirName = dir.split("/").pop();
+    if (EXCLUDED_DIRS.includes(dirName)) return;
+
+    try {
+      const entries = readdirSync(dir);
+      for (const entry of entries) {
+        const fullPath = join(dir, entry);
+        const stat = statSync(fullPath);
+        if (stat.isDirectory()) {
+          walkDirectory(fullPath);
+        } else if (stat.isFile() && entry.endsWith(".mdx")) {
+          mdxFiles.push(fullPath);
+        }
+      }
+    } catch (error) {
+      console.error(`Error reading directory ${dir}: ${error.message}`);
+    }
+  }
+
+  for (const dir of searchDirs) {
+    if (existsSync(dir)) walkDirectory(dir);
+  }
+
+  return mdxFiles.sort();
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Main export
+// ─────────────────────────────────────────────────────────────────────────────
+
+export async function fixH1(options) {
+  const repoRoot = process.cwd();
+
+  if (!options.quiet) {
+    console.log(chalk.bold("\n# H1 Duplicate Title Fixer\n"));
+  }
+
+  const files = findMdxFiles(repoRoot, options.dir, options.file);
+
+  if (files.length === 0) {
+    console.error(chalk.red("✗ No MDX files found."));
+    process.exit(1);
+  }
+
+  if (!options.quiet) {
+    console.log(`Found ${files.length} MDX file(s) to process\n`);
+    if (options.dryRun) {
+      console.log(chalk.yellow("Dry run — no files will be written\n"));
+    }
+  }
+
+  const changed = [];
+
+  for (const filePath of files) {
+    const content = readFileSync(filePath, "utf-8");
+    const fmTitle = extractFrontmatterTitle(content);
+    if (!fmTitle) continue;
+
+    const { newContent, changed: didChange } = removeDuplicateH1(content, fmTitle);
+
+    if (didChange) {
+      const relPath = relative(repoRoot, filePath);
+      changed.push(relPath);
+
+      if (options.verbose) {
+        console.log(`${chalk.cyan(relPath)}: removed duplicate H1`);
+      }
+
+      if (!options.dryRun) {
+        writeFileSync(filePath, newContent, "utf-8");
+      }
+    }
+  }
+
+  if (!options.quiet) {
+    if (changed.length > 0) {
+      const verb = options.dryRun ? "Would remove" : "Removed";
+      console.log(chalk.green(`\n✓ ${verb} duplicate H1 in ${changed.length} file(s)`));
+
+      if (!options.verbose) {
+        for (const relPath of changed) {
+          console.log(`  ${chalk.cyan(relPath)}`);
+        }
+      }
+    } else {
+      console.log(chalk.yellow("⚠️  No duplicate H1 headings found."));
+    }
+  }
+}

package/src/commands/fix/inlineimages.js

@@ -0,0 +1,305 @@
+/**
+ * Fix Inline Images Tool
+ *
+ * Converts images that appear inline within text lines to <InlineImage> components.
+ * Handles both Markdown images (![alt](url)) and HTML <img> tags.
+ *
+ * - Inline image (has other text on the line)  → <InlineImage src="url" />
+ * - Standalone image (alone on its line)        → left unchanged (use fix images)
+ *
+ * Also adds the required import at the top of the file (after frontmatter):
+ *   import { InlineImage } from "/snippets/InlineImage.jsx";
+ *
+ * Skips images inside:
+ * - Fenced code blocks
+ * - Inline code spans
+ * - Markdown tables
+ * - HTML tables
+ * - <Frame> blocks
+ */
+
+import { existsSync, readdirSync, statSync, readFileSync, writeFileSync } from "fs";
+import { join, relative, resolve } from "path";
+import chalk from "chalk";
+
+const EXCLUDED_DIRS = ["node_modules", ".git"];
+
+const IMPORT_LINE = 'import { InlineImage } from "/snippets/InlineImage.jsx";';
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Protection patterns (multi-line regions tokenized before line processing)
+// ─────────────────────────────────────────────────────────────────────────────
+
+const FRAME_RE = /<Frame(?:\s[^>]*)?>[\s\S]*?<\/Frame>/gi;
+const HTML_TABLE_RE = /<table(?:\s[^>]*)?>[\s\S]*?<\/table>/gi;
+const MD_TABLE_RE = /^(?:\|[^\n]*\n)+(?:\|[^\n]*)?/gm;
+const FENCE_RE = /^[ \t]*(`{3,}|~{3,})[ \t]*[^\n]*\n[\s\S]*?\n[ \t]*\1[ \t]*$/gm;
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Tokenizer helpers
+// ─────────────────────────────────────────────────────────────────────────────
+
+function tokenize(pattern, text, tag) {
+  const stash = [];
+  const result = text.replace(pattern, (match) => {
+    const idx = stash.length;
+    stash.push(match);
+    return `\x00${tag}${idx}\x00`;
+  });
+  return { text: result, stash };
+}
+
+function detokenize(text, tag, stash) {
+  const escapedTag = tag.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  return text.replace(new RegExp(`\x00${escapedTag}(\\d+)\x00`, "g"), (_, idx) => {
+    return stash[parseInt(idx, 10)];
+  });
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Import injection
+// ─────────────────────────────────────────────────────────────────────────────
+
+/**
+ * Returns the index immediately after the frontmatter closing ---, or -1 if none.
+ */
+function findFrontmatterEnd(content) {
+  if (!content.startsWith("---")) return -1;
+  const match = content.match(/^---\r?\n[\s\S]*?\r?\n---\r?\n?/);
+  if (!match) return -1;
+  return match[0].length;
+}
+
+/**
+ * Ensures the InlineImage import is present in the file.
+ * Inserts after frontmatter (if any) with an empty line below.
+ */
+function ensureImport(content) {
+  if (content.includes(IMPORT_LINE)) return content;
+
+  const fmEnd = findFrontmatterEnd(content);
+
+  if (fmEnd === -1) {
+    // No frontmatter — insert at top
+    return IMPORT_LINE + "\n\n" + content;
+  }
+
+  const before = content.slice(0, fmEnd);
+  const after = content.slice(fmEnd).replace(/^\n+/, ""); // normalise blank lines
+  return before + "\n" + IMPORT_LINE + "\n\n" + after;
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Per-line processing
+// ─────────────────────────────────────────────────────────────────────────────
+
+/**
+ * Processes a single line: replaces inline images (those sharing the line with text).
+ * Protects inline code spans so images inside backticks are not touched.
+ * Returns { line, count }.
+ */
+function processLine(line) {
+  // Quick check — skip lines with no image syntax at all
+  if (!line.includes("![") && !/<img\b/i.test(line)) {
+    return { line, count: 0 };
+  }
+
+  // Protect inline code spans within this line
+  const inlineCodeResult = tokenize(/`[^`\n]+`/g, line, "ICODE");
+  let text = inlineCodeResult.text;
+
+  const hasMdImage = /!\[[^\]\n]*\]\([^\)\n]+\)/.test(text);
+  const hasHtmlImg = /<img\b[^>\n]*\/?>/i.test(text);
+
+  if (!hasMdImage && !hasHtmlImg) {
+    return { line, count: 0 };
+  }
+
+  // Strip all images to check whether there's other text content on the line
+  const withoutImages = text
+    .replace(/!\[[^\]\n]*\]\([^\)\n]+\)/g, "")
+    .replace(/<img\b[^>\n]*\/?>/gi, "")
+    .trim();
+
+  if (withoutImages === "") {
+    // Line contains only images (standalone) — leave for fix images command
+    return { line, count: 0 };
+  }
+
+  // ── Replace inline markdown images ──────────────────────────────────────────
+  // Negative lookbehind (?<!\[) prevents matching linked images [![alt](url)](link)
+  let count = 0;
+  text = text.replace(/(?<!\[)!\[([^\]\n]*)\]\(([^\)\n]+)\)/g, (match, alt, src) => {
+    count++;
+    const altProp = alt.trim() ? ` alt="${alt.trim()}"` : "";
+    return `<InlineImage src="${src}"${altProp} />`;
+  });
+
+  // ── Replace inline HTML <img> tags ──────────────────────────────────────────
+  // Rename <img ...> / <img ... /> to <InlineImage ... />, preserving all attributes
+  text = text.replace(/<img\b([^>]*?)(\s*\/?)>/gi, (match, attrs) => {
+    if (!attrs.includes("src")) return match; // no src — leave as-is
+    count++;
+    const cleanAttrs = attrs.trimEnd().replace(/\/$/, "").trimEnd();
+    return `<InlineImage${cleanAttrs} />`;
+  });
+
+  // Restore inline code spans
+  text = detokenize(text, "ICODE", inlineCodeResult.stash);
+
+  return { line: text, count };
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Core processing
+// ─────────────────────────────────────────────────────────────────────────────
+
+/**
+ * Processes MDX content: replaces inline images and injects the import.
+ * Returns { newContent, count }.
+ */
+function processContent(content) {
+  let text = content;
+
+  // 1. Protect multi-line regions
+  const frameResult = tokenize(FRAME_RE, text, "FRAME");
+  text = frameResult.text;
+
+  const htmlTableResult = tokenize(HTML_TABLE_RE, text, "HTMLTABLE");
+  text = htmlTableResult.text;
+
+  const mdTableResult = tokenize(MD_TABLE_RE, text, "MDTABLE");
+  text = mdTableResult.text;
+
+  const fenceResult = tokenize(FENCE_RE, text, "FENCE");
+  text = fenceResult.text;
+
+  // 2. Process line by line
+  let totalCount = 0;
+  const lines = text.split("\n");
+  const processedLines = lines.map((line) => {
+    const { line: newLine, count } = processLine(line);
+    totalCount += count;
+    return newLine;
+  });
+  text = processedLines.join("\n");
+
+  // 3. Restore protected regions
+  text = detokenize(text, "FENCE", fenceResult.stash);
+  text = detokenize(text, "MDTABLE", mdTableResult.stash);
+  text = detokenize(text, "HTMLTABLE", htmlTableResult.stash);
+  text = detokenize(text, "FRAME", frameResult.stash);
+
+  // 4. Add import if any replacements were made
+  if (totalCount > 0) {
+    text = ensureImport(text);
+  }
+
+  return { newContent: text, count: totalCount };
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// File discovery
+// ─────────────────────────────────────────────────────────────────────────────
+
+function findMdxFiles(repoRoot, directory = null, file = null) {
+  if (file) {
+    const fullPath = resolve(repoRoot, file);
+    return existsSync(fullPath) ? [fullPath] : [];
+  }
+
+  const searchDirs = directory ? [resolve(repoRoot, directory)] : [repoRoot];
+  const mdxFiles = [];
+
+  function walkDirectory(dir) {
+    const dirName = dir.split("/").pop();
+    if (EXCLUDED_DIRS.includes(dirName)) return;
+
+    try {
+      const entries = readdirSync(dir);
+      for (const entry of entries) {
+        const fullPath = join(dir, entry);
+        const stat = statSync(fullPath);
+        if (stat.isDirectory()) {
+          walkDirectory(fullPath);
+        } else if (stat.isFile() && entry.endsWith(".mdx")) {
+          mdxFiles.push(fullPath);
+        }
+      }
+    } catch (error) {
+      console.error(`Error reading directory ${dir}: ${error.message}`);
+    }
+  }
+
+  for (const dir of searchDirs) {
+    if (existsSync(dir)) walkDirectory(dir);
+  }
+
+  return mdxFiles.sort();
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+// Main export
+// ─────────────────────────────────────────────────────────────────────────────
+
+export async function fixInlineImages(options) {
+  const repoRoot = process.cwd();
+
+  if (!options.quiet) {
+    console.log(chalk.bold("\n🖼️  Inline Image Fixer\n"));
+  }
+
+  const files = findMdxFiles(repoRoot, options.dir, options.file);
+
+  if (files.length === 0) {
+    console.error(chalk.red("✗ No MDX files found."));
+    process.exit(1);
+  }
+
+  if (!options.quiet) {
+    console.log(`Found ${files.length} MDX file(s) to process\n`);
+    if (options.dryRun) {
+      console.log(chalk.yellow("Dry run — no files will be written\n"));
+    }
+  }
+
+  const results = {};
+  let totalImages = 0;
+
+  for (const filePath of files) {
+    const content = readFileSync(filePath, "utf-8");
+    const { newContent, count } = processContent(content);
+
+    if (count > 0) {
+      const relPath = relative(repoRoot, filePath);
+      results[relPath] = count;
+      totalImages += count;
+
+      if (options.verbose) {
+        console.log(`${chalk.cyan(relPath)}: converted ${count} inline image(s)`);
+      }
+
+      if (!options.dryRun) {
+        writeFileSync(filePath, newContent, "utf-8");
+      }
+    }
+  }
+
+  // Summary
+  if (!options.quiet) {
+    const fileCount = Object.keys(results).length;
+
+    if (fileCount > 0) {
+      const verb = options.dryRun ? "Would convert" : "Converted";
+      console.log(chalk.green(`\n✓ ${verb} ${totalImages} inline image(s) in ${fileCount} file(s)`));
+
+      if (!options.verbose) {
+        for (const [filePath, count] of Object.entries(results)) {
+          console.log(`  ${chalk.cyan(filePath)}: ${count} inline image(s)`);
+        }
+      }
+    } else {
+      console.log(chalk.yellow("⚠️  No inline images found."));
+    }
+  }
+}

package/src/utils/config.js

@@ -141,6 +141,25 @@ export function mergeCodeblocksConfig(options, config) {
   };
 }
 
+/**
+ * Merges config file with CLI options for the inlineimages command
+ * CLI options take precedence over config file
+ *
+ * @param {Object} options - CLI options
+ * @param {Object|null} config - Loaded config object
+ * @returns {Object} Merged options
+ */
+export function mergeInlineImagesConfig(options, config) {
+  const imgConfig = config?.inlineimages || {};
+
+  return {
+    file: options.file || imgConfig.file || null,
+    dir: options.dir || imgConfig.dir || null,
+    dryRun: options.dryRun !== undefined ? options.dryRun : (imgConfig["dry-run"] ?? false),
+    quiet: options.quiet !== undefined ? options.quiet : (imgConfig.quiet ?? false),
+  };
+}
+
 /**
  * Merges config file with CLI options for the images command
  * CLI options take precedence over config file
@@ -160,6 +179,25 @@ export function mergeImagesConfig(options, config) {
   };
 }
 
+/**
+ * Merges config file with CLI options for the h1 command
+ * CLI options take precedence over config file
+ *
+ * @param {Object} options - CLI options
+ * @param {Object|null} config - Loaded config object
+ * @returns {Object} Merged options
+ */
+export function mergeH1Config(options, config) {
+  const h1Config = config?.h1 || {};
+
+  return {
+    file: options.file || h1Config.file || null,
+    dir: options.dir || h1Config.dir || null,
+    dryRun: options.dryRun !== undefined ? options.dryRun : (h1Config["dry-run"] ?? false),
+    quiet: options.quiet !== undefined ? options.quiet : (h1Config.quiet ?? false),
+  };
+}
+
 /**
  * Validates that required fields are present
  * @param {string|undefined} baseUrl - Base URL