chore(docs): remove need to ignore file, lint fix first

Author: jaikamat

eslint.config.mjs

@@ -41,12 +41,7 @@ export default tseslint.config(
    * Global ignores for common build and dependency directories
    */
   {
-    ignores: [
-      "**/node_modules/**",
-      "**/dist/**",
-      "**/storybook-static/**",
-      "**/docs-validation.generated.spec.tsx", // Auto-generated from .dev.mdx files
-    ],
+    ignores: ["**/node_modules/**", "**/dist/**", "**/storybook-static/**"],
   },
   /**
    * Base ESLint recommended rules

packages/nimbus/tsconfig.json

@@ -34,10 +34,5 @@
       // "@types-extensions",
       "node_modules/@types"
     ]
-  },
-  "exclude": [
-    "node_modules",
-    "dist",
-    "src/test/docs-validation.generated.spec.tsx"
-  ]
+  }
 }

scripts/validate-docs-tests.ts

@@ -103,6 +103,13 @@ function wrapRenderCalls(code: string): string {
     .join("\n");
 }
 
+/**
+ * Normalize imports to use double quotes (prettier standard)
+ */
+function normalizeImport(importStatement: string): string {
+  return importStatement.replace(/'/g, '"');
+}
+
 /**
  * Generate a complete test file from extracted blocks
  */
@@ -122,7 +129,7 @@ function generateTestFile(tests: ExtractedTest[]): string {
       // Collect imports
       if (inImports && (trimmed.startsWith("import ") || trimmed === "")) {
         if (trimmed) {
-          imports.add(trimmed);
+          imports.add(normalizeImport(trimmed));
         }
         continue;
       }
@@ -148,34 +155,28 @@ function generateTestFile(tests: ExtractedTest[]): string {
   }
 
   // Add required imports for provider wrapper
-  imports.add("import { NimbusProvider } from '@commercetools/nimbus';");
+  imports.add('import type { ReactNode } from "react";');
+  imports.add('import { NimbusProvider } from "@commercetools/nimbus";');
 
   // Combine into complete file with wrapper utility
   const importsArray = Array.from(imports).sort();
   const fileContent = [
     "// Auto-generated test file from .dev.mdx documentation",
     "// DO NOT EDIT - This file is generated by validate-docs-tests.ts",
     "",
+    "import { describe, it, expect, vi } from 'vitest';",
     ...importsArray,
     "",
     "// Wrapper utility to provide NimbusProvider context",
     "function renderWithProvider(ui: React.ReactElement) {",
-    "  const result = render(",
-    "    <NimbusProvider>",
-    "      {ui}",
-    "    </NimbusProvider>",
-    "  );",
-    "  ",
+    "  const result = render(<NimbusProvider>{ui}</NimbusProvider>);",
+    "",
     "  // Wrap rerender to also use provider",
     "  const originalRerender = result.rerender;",
-    "  result.rerender = (rerenderUi: React.ReactElement) => {",
-    "    return originalRerender(",
-    "      <NimbusProvider>",
-    "        {rerenderUi}",
-    "      </NimbusProvider>",
-    "    );",
+    "  result.rerender = (rerenderUi: ReactNode) => {",
+    "    return originalRerender(<NimbusProvider>{rerenderUi}</NimbusProvider>);",
     "  };",
-    "  ",
+    "",
     "  return result;",
     "}",
     "",
@@ -250,6 +251,17 @@ async function main() {
     `📝 Generated test file: ${relative(process.cwd(), tempTestFile)}\n`
   );
 
+  // Format with ESLint --fix (runs Prettier automatically)
+  console.log("🎨 Formatting generated file...\n");
+  try {
+    execSync(`pnpm eslint ${tempTestFile} --fix`, {
+      cwd: process.cwd(),
+      stdio: "pipe", // Suppress output
+    });
+  } catch {
+    console.warn("⚠️  Could not format file (continuing anyway)");
+  }
+
   // Run tests with Vitest
   console.log("🧪 Running extracted tests...\n");
   try {