@larkiny/astro-github-loader 0.11.3 → 0.13.0

package/src/github.link-transform.ts

@@ -1,7 +1,14 @@
-import { slug } from 'github-slugger';
-import path from 'node:path';
-import type { LinkMapping, LinkTransformContext, MatchedPattern, IncludePattern, PathMappingValue, EnhancedPathMapping } from './github.types.js';
-import type { Logger } from './github.logger.js';
+import { slug } from "github-slugger";
+import path from "node:path";
+import type {
+  LinkMapping,
+  LinkTransformContext,
+  MatchedPattern,
+  IncludePattern,
+  PathMappingValue,
+  EnhancedPathMapping,
+} from "./github.types.js";
+import type { Logger } from "./github.logger.js";
 
 /**
  * Represents an imported file with its content and metadata
@@ -66,8 +73,8 @@ interface LinkContext {
  */
 function extractAnchor(link: string): { path: string; anchor: string } {
   const anchorMatch = link.match(/#.*$/);
-  const anchor = anchorMatch ? anchorMatch[0] : '';
-  const path = link.replace(/#.*$/, '');
+  const anchor = anchorMatch ? anchorMatch[0] : "";
+  const path = link.replace(/#.*$/, "");
   return { path, anchor };
 }
 
@@ -83,16 +90,12 @@ function isExternalLink(link: string): boolean {
     /^tel:/.test(link) ||
     /^ftp:/.test(link) ||
     /^ftps:\/\//.test(link) ||
-
     // Any protocol with ://
-    link.includes('://') ||
-
+    link.includes("://") ||
     // Anchor-only links (same page)
-    link.startsWith('#') ||
-
+    link.startsWith("#") ||
     // Data URLs
     /^data:/.test(link) ||
-
     // File protocol
     /^file:\/\//.test(link)
   );
@@ -101,17 +104,30 @@ function isExternalLink(link: string): boolean {
 /**
  * Normalize path separators and resolve relative paths
  */
-function normalizePath(linkPath: string, currentFilePath: string, logger?: Logger): string {
-  logger?.debug(`[normalizePath] BEFORE: linkPath="${linkPath}", currentFilePath="${currentFilePath}"`);
+function normalizePath(
+  linkPath: string,
+  currentFilePath: string,
+  logger?: Logger,
+): string {
+  logger?.debug(
+    `[normalizePath] BEFORE: linkPath="${linkPath}", currentFilePath="${currentFilePath}"`,
+  );
 
   // Handle relative paths (including simple relative paths without ./ prefix)
   // A link is relative if it doesn't start with / or contain a protocol
-  const isAbsoluteOrExternal = linkPath.startsWith('/') || linkPath.includes('://') || linkPath.startsWith('#');
+  const isAbsoluteOrExternal =
+    linkPath.startsWith("/") ||
+    linkPath.includes("://") ||
+    linkPath.startsWith("#");
 
   if (!isAbsoluteOrExternal) {
     const currentDir = path.dirname(currentFilePath);
-    const resolved = path.posix.normalize(path.posix.join(currentDir, linkPath));
-    logger?.debug(`[normalizePath] RELATIVE PATH RESOLVED: "${linkPath}" -> "${resolved}" (currentDir: "${currentDir}")`);
+    const resolved = path.posix.normalize(
+      path.posix.join(currentDir, linkPath),
+    );
+    logger?.debug(
+      `[normalizePath] RELATIVE PATH RESOLVED: "${linkPath}" -> "${resolved}" (currentDir: "${currentDir}")`,
+    );
     return resolved;
   }
 
@@ -125,7 +141,7 @@ function normalizePath(linkPath: string, currentFilePath: string, logger?: Logge
 function applyLinkMappings(
   linkUrl: string,
   linkMappings: LinkMapping[],
-  context: LinkContext
+  context: LinkContext,
 ): string {
   const { path: linkPath, anchor } = extractAnchor(linkUrl);
   let transformedPath = linkPath;
@@ -141,24 +157,27 @@ function applyLinkMappings(
     // Handle relative links automatically if enabled
     if (mapping.relativeLinks && context.currentFile.linkContext) {
       // Check if this is a relative link (doesn't start with /, http, etc.)
-      if (!linkPath.startsWith('/') && !isExternalLink(linkPath)) {
+      if (!linkPath.startsWith("/") && !isExternalLink(linkPath)) {
         // Check if the link points to a known directory structure
-        const knownPaths = ['modules/', 'classes/', 'interfaces/', 'enums/'];
-        const isKnownPath = knownPaths.some(p => linkPath.startsWith(p));
+        const knownPaths = ["modules/", "classes/", "interfaces/", "enums/"];
+        const isKnownPath = knownPaths.some((p) => linkPath.startsWith(p));
 
         if (isKnownPath) {
           // Strip .md extension from the link path
-          const cleanLinkPath = linkPath.replace(/\.md$/, '');
+          const cleanLinkPath = linkPath.replace(/\.md$/, "");
 
           // Convert relative path to absolute path using the target base
-          const targetBase = generateSiteUrl(context.currentFile.linkContext.basePath, context.global.stripPrefixes);
+          const targetBase = generateSiteUrl(
+            context.currentFile.linkContext.basePath,
+            context.global.stripPrefixes,
+          );
 
           // Construct final URL with proper Starlight formatting
-          let finalUrl = targetBase.replace(/\/$/, '') + '/' + cleanLinkPath;
+          let finalUrl = targetBase.replace(/\/$/, "") + "/" + cleanLinkPath;
 
           // Add trailing slash if it doesn't end with one and isn't empty
-          if (finalUrl && !finalUrl.endsWith('/')) {
-            finalUrl += '/';
+          if (finalUrl && !finalUrl.endsWith("/")) {
+            finalUrl += "/";
           }
 
           transformedPath = finalUrl;
@@ -168,16 +187,30 @@ function applyLinkMappings(
     }
 
     let matched = false;
-    let replacement = '';
+    let replacement = "";
+
+    const getLinkTransformContext = (): LinkTransformContext =>
+      context.currentFile.linkContext ?? {
+        sourcePath: context.currentFile.sourcePath,
+        targetPath: context.currentFile.targetPath,
+        basePath: "",
+      };
 
-    if (typeof mapping.pattern === 'string') {
+    if (typeof mapping.pattern === "string") {
       // String pattern - exact match or contains
       if (transformedPath.includes(mapping.pattern)) {
         matched = true;
-        if (typeof mapping.replacement === 'string') {
-          replacement = transformedPath.replace(mapping.pattern, mapping.replacement);
+        if (typeof mapping.replacement === "string") {
+          replacement = transformedPath.replace(
+            mapping.pattern,
+            mapping.replacement,
+          );
         } else {
-          replacement = mapping.replacement(transformedPath, anchor, context);
+          replacement = mapping.replacement(
+            transformedPath,
+            anchor,
+            getLinkTransformContext(),
+          );
         }
       }
     } else {
@@ -185,10 +218,17 @@ function applyLinkMappings(
       const match = transformedPath.match(mapping.pattern);
       if (match) {
         matched = true;
-        if (typeof mapping.replacement === 'string') {
-          replacement = transformedPath.replace(mapping.pattern, mapping.replacement);
+        if (typeof mapping.replacement === "string") {
+          replacement = transformedPath.replace(
+            mapping.pattern,
+            mapping.replacement,
+          );
         } else {
-          replacement = mapping.replacement(transformedPath, anchor, context);
+          replacement = mapping.replacement(
+            transformedPath,
+            anchor,
+            getLinkTransformContext(),
+          );
         }
       }
     }
@@ -218,35 +258,37 @@ function generateSiteUrl(targetPath: string, stripPrefixes: string[]): string {
   }
 
   // Remove leading slash if present
-  url = url.replace(/^\//, '');
+  url = url.replace(/^\//, "");
 
   // Remove file extension
-  url = url.replace(/\.(md|mdx)$/i, '');
+  url = url.replace(/\.(md|mdx)$/i, "");
 
   // Handle index files - they should resolve to parent directory
-  if (url.endsWith('/index')) {
-    url = url.replace('/index', '');
-  } else if (url === 'index') {
-    url = '';
+  if (url.endsWith("/index")) {
+    url = url.replace("/index", "");
+  } else if (url === "index") {
+    url = "";
   }
 
   // Split path into segments and slugify each
-  const segments = url.split('/').map(segment => segment ? slug(segment) : '');
+  const segments = url
+    .split("/")
+    .map((segment) => (segment ? slug(segment) : ""));
 
   // Reconstruct URL
-  url = segments.filter(s => s).join('/');
+  url = segments.filter((s) => s).join("/");
 
   // Ensure leading slash
-  if (url && !url.startsWith('/')) {
-    url = '/' + url;
+  if (url && !url.startsWith("/")) {
+    url = "/" + url;
   }
 
   // Add trailing slash for non-empty paths
-  if (url && !url.endsWith('/')) {
-    url = url + '/';
+  if (url && !url.endsWith("/")) {
+    url = url + "/";
   }
 
-  return url || '/';
+  return url || "/";
 }
 
 /**
@@ -260,7 +302,11 @@ function generateSiteUrl(targetPath: string, stripPrefixes: string[]): string {
  * 5. Apply non-global path mappings if unresolved
  * 6. Check custom handlers
  */
-function transformLink(linkText: string, linkUrl: string, context: LinkContext): string {
+function transformLink(
+  linkText: string,
+  linkUrl: string,
+  context: LinkContext,
+): string {
   // Skip external links FIRST - no transformations should ever be applied to them
   if (isExternalLink(linkUrl)) {
     return `[${linkText}](${linkUrl})`;
@@ -269,14 +315,22 @@ function transformLink(linkText: string, linkUrl: string, context: LinkContext):
   const { path: linkPath, anchor } = extractAnchor(linkUrl);
 
   // Normalize the link path relative to current file FIRST
-  const normalizedPath = normalizePath(linkPath, context.currentFile.sourcePath, context.global.logger);
+  const normalizedPath = normalizePath(
+    linkPath,
+    context.currentFile.sourcePath,
+    context.global.logger,
+  );
 
   // Apply global path mappings to the normalized path
   let processedNormalizedPath = normalizedPath;
   if (context.global.linkMappings) {
-    const globalMappings = context.global.linkMappings.filter(m => m.global);
+    const globalMappings = context.global.linkMappings.filter((m) => m.global);
     if (globalMappings.length > 0) {
-      processedNormalizedPath = applyLinkMappings(normalizedPath + anchor, globalMappings, context);
+      processedNormalizedPath = applyLinkMappings(
+        normalizedPath + anchor,
+        globalMappings,
+        context,
+      );
       // Extract path again after global mappings
       const { path: newPath } = extractAnchor(processedNormalizedPath);
       processedNormalizedPath = newPath;
@@ -287,8 +341,10 @@ function transformLink(linkText: string, linkUrl: string, context: LinkContext):
   let targetPath = context.global.sourceToTargetMap.get(normalizedPath);
 
   // If not found and path ends with /, try looking for index.md
-  if (!targetPath && normalizedPath.endsWith('/')) {
-    targetPath = context.global.sourceToTargetMap.get(normalizedPath + 'index.md');
+  if (!targetPath && normalizedPath.endsWith("/")) {
+    targetPath = context.global.sourceToTargetMap.get(
+      normalizedPath + "index.md",
+    );
   }
 
   if (targetPath) {
@@ -299,10 +355,16 @@ function transformLink(linkText: string, linkUrl: string, context: LinkContext):
 
   // Apply non-global path mappings to unresolved links
   if (context.global.linkMappings) {
-    const nonGlobalMappings = context.global.linkMappings.filter(m => !m.global);
+    const nonGlobalMappings = context.global.linkMappings.filter(
+      (m) => !m.global,
+    );
     if (nonGlobalMappings.length > 0) {
-      const mappedUrl = applyLinkMappings(processedNormalizedPath + anchor, nonGlobalMappings, context);
-      if (mappedUrl !== (processedNormalizedPath + anchor)) {
+      const mappedUrl = applyLinkMappings(
+        processedNormalizedPath + anchor,
+        nonGlobalMappings,
+        context,
+      );
+      if (mappedUrl !== processedNormalizedPath + anchor) {
         return `[${linkText}](${mappedUrl})`;
       }
     }
@@ -321,7 +383,7 @@ function transformLink(linkText: string, linkUrl: string, context: LinkContext):
 
   // No transformation matched - strip .md extension from unresolved internal links
   // This handles links to files that weren't imported but should still use Starlight routing
-  const cleanPath = processedNormalizedPath.replace(/\.md$/i, '');
+  const cleanPath = processedNormalizedPath.replace(/\.md$/i, "");
   return `[${linkText}](${cleanPath + anchor})`;
 }
 
@@ -336,7 +398,7 @@ export function globalLinkTransform(
     customHandlers?: LinkHandler[];
     linkMappings?: LinkMapping[];
     logger?: Logger;
-  }
+  },
 ): ImportedFile[] {
   // Build global context
   const sourceToTargetMap = new Map<string, string>();
@@ -359,31 +421,31 @@ export function globalLinkTransform(
   // Transform links in all files
   const markdownLinkRegex = /\[([^\]]*)\]\(([^)]+)\)/g;
 
-  return importedFiles.map(file => ({
+  return importedFiles.map((file) => ({
     ...file,
-    content: file.content.replace(markdownLinkRegex, (match, linkText, linkUrl) => {
-      const linkContext: LinkContext = {
-        currentFile: file,
-        originalLink: linkUrl,
-        anchor: extractAnchor(linkUrl).anchor,
-        global: globalContext,
-      };
-
-      return transformLink(linkText, linkUrl, linkContext);
-    }),
+    content: file.content.replace(
+      markdownLinkRegex,
+      (match, linkText, linkUrl) => {
+        const linkContext: LinkContext = {
+          currentFile: file,
+          originalLink: linkUrl,
+          anchor: extractAnchor(linkUrl).anchor,
+          global: globalContext,
+        };
+
+        return transformLink(linkText, linkUrl, linkContext);
+      },
+    ),
   }));
 }
 
-
 /**
  * Infer cross-section path from basePath
  * @param basePath - The base path from include pattern (e.g., 'src/content/docs/reference/api')
  * @returns Inferred cross-section path (e.g., '/reference/api')
  */
 function inferCrossSectionPath(basePath: string): string {
-  return basePath
-    .replace(/^src\/content\/docs/, '')
-    .replace(/\/$/, '') || '/';
+  return basePath.replace(/^src\/content\/docs/, "").replace(/\/$/, "") || "/";
 }
 
 /**
@@ -394,7 +456,7 @@ function inferCrossSectionPath(basePath: string): string {
  */
 export function generateAutoLinkMappings(
   includes: IncludePattern[],
-  stripPrefixes: string[] = []
+  stripPrefixes: string[] = [],
 ): LinkMapping[] {
   const linkMappings: LinkMapping[] = [];
 
@@ -403,28 +465,43 @@ export function generateAutoLinkMappings(
 
     const inferredCrossSection = inferCrossSectionPath(includePattern.basePath);
 
-    for (const [sourcePath, mappingValue] of Object.entries(includePattern.pathMappings)) {
+    for (const [sourcePath, mappingValue] of Object.entries(
+      includePattern.pathMappings,
+    )) {
       // Handle both string and enhanced object formats
-      const targetPath = typeof mappingValue === 'string' ? mappingValue : mappingValue.target;
-      const crossSectionPath = typeof mappingValue === 'object' && mappingValue.crossSectionPath
-        ? mappingValue.crossSectionPath
-        : inferredCrossSection;
-
-      if (sourcePath.endsWith('/')) {
+      const targetPath =
+        typeof mappingValue === "string" ? mappingValue : mappingValue.target;
+      const crossSectionPath =
+        typeof mappingValue === "object" && mappingValue.crossSectionPath
+          ? mappingValue.crossSectionPath
+          : inferredCrossSection;
+
+      if (sourcePath.endsWith("/")) {
         // Folder mapping - use regex with capture group
-        const sourcePattern = sourcePath.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+        const sourcePattern = sourcePath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 
         linkMappings.push({
           pattern: new RegExp(`^${sourcePattern}(.+)$`),
-          replacement: (transformedPath: string, anchor: string, context: any) => {
-            const relativePath = transformedPath.replace(new RegExp(`^${sourcePattern}`), '');
+          replacement: (
+            transformedPath: string,
+            _anchor: string,
+            _context: LinkTransformContext,
+          ) => {
+            const relativePath = transformedPath.replace(
+              new RegExp(`^${sourcePattern}`),
+              "",
+            );
             let finalPath: string;
-            if (crossSectionPath && crossSectionPath !== '/') {
-              finalPath = targetPath === ''
-                ? `${crossSectionPath}/${relativePath}`
-                : `${crossSectionPath}/${targetPath}${relativePath}`;
+            if (crossSectionPath && crossSectionPath !== "/") {
+              finalPath =
+                targetPath === ""
+                  ? `${crossSectionPath}/${relativePath}`
+                  : `${crossSectionPath}/${targetPath}${relativePath}`;
             } else {
-              finalPath = targetPath === '' ? relativePath : `${targetPath}${relativePath}`;
+              finalPath =
+                targetPath === ""
+                  ? relativePath
+                  : `${targetPath}${relativePath}`;
             }
             return generateSiteUrl(finalPath, stripPrefixes);
           },
@@ -432,14 +509,19 @@ export function generateAutoLinkMappings(
         });
       } else {
         // File mapping - exact string match
-        const sourcePattern = sourcePath.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+        const sourcePattern = sourcePath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 
         linkMappings.push({
           pattern: new RegExp(`^${sourcePattern}$`),
-          replacement: (transformedPath: string, anchor: string, context: any) => {
-            const finalPath = crossSectionPath && crossSectionPath !== '/'
-              ? `${crossSectionPath}/${targetPath}`
-              : targetPath;
+          replacement: (
+            _transformedPath: string,
+            _anchor: string,
+            _context: LinkTransformContext,
+          ) => {
+            const finalPath =
+              crossSectionPath && crossSectionPath !== "/"
+                ? `${crossSectionPath}/${targetPath}`
+                : targetPath;
             return generateSiteUrl(finalPath, stripPrefixes);
           },
           global: true,
@@ -454,4 +536,4 @@ export function generateAutoLinkMappings(
 /**
  * Export types for use in configuration
  */
-export type { LinkContext, GlobalLinkContext };
+export type { LinkContext, GlobalLinkContext };

package/src/github.loader.spec.ts

@@ -1,27 +1,47 @@
-import { beforeEach, describe, it, expect } from "vitest";
+import { describe, it, expect } from "vitest";
 import { githubLoader } from "./github.loader.js";
-import { globalLinkTransform, type ImportedFile } from "./github.link-transform.js";
+import {
+  globalLinkTransform,
+  type ImportedFile,
+} from "./github.link-transform.js";
 import { createLogger, type ImportSummary } from "./github.logger.js";
+import type { ImportOptions, VersionConfig } from "./github.types.js";
 import { Octokit } from "octokit";
 
-const FIXTURES = [
-  {
-    owner: "awesome-algorand",
-    repo: "algokit-cli",
-    ref: "docs/starlight-preview",
-    path: ".devportal/starlight",
-  },
-];
 describe("githubLoader", () => {
-  let octokit: Octokit;
-  beforeEach(() => {
-    octokit = new Octokit({ auth: process.env.GITHUB_TOKEN });
+  it("should return a loader object", () => {
+    const octokit = new Octokit({ auth: "mock-token" });
+    const result = githubLoader({ octokit, configs: [] });
+    expect(result).toHaveProperty("name", "github-loader");
+    expect(result).toHaveProperty("load");
+    expect(typeof result.load).toBe("function");
   });
 
-  it("should work", async () => {
-    const result = githubLoader({ octokit, configs: FIXTURES });
+  it("should accept configs with language and versions fields", () => {
+    const octokit = new Octokit({ auth: "mock-token" });
+    const configs: ImportOptions[] = [
+      {
+        name: "AlgoKit Utils TS",
+        owner: "algorandfoundation",
+        repo: "algokit-utils-ts",
+        ref: "docs-dist",
+        language: "TypeScript",
+        versions: [
+          { slug: "latest", label: "Latest" },
+          { slug: "v8.0.0", label: "v8.0.0" },
+        ],
+        includes: [
+          {
+            pattern: "docs/**/*.md",
+            basePath: "src/content/docs/docs/algokit-utils/typescript",
+          },
+        ],
+      },
+    ];
 
-    console.log(result);
+    // Should not throw when constructing the loader with new fields
+    const result = githubLoader({ octokit, configs });
+    expect(result).toHaveProperty("name", "github-loader");
   });
 
   describe("context-aware link transformations", () => {
@@ -30,76 +50,81 @@ describe("githubLoader", () => {
         {
           id: "api-readme",
           sourcePath: "docs/code/README.md",
-          targetPath: "src/content/docs/reference/algokit-utils-ts/api/README.md",
-          content: 'Check out the [modules](modules/) for more info.',
+          targetPath:
+            "src/content/docs/reference/algokit-utils-ts/api/README.md",
+          content: "Check out the [modules](modules/) for more info.",
           linkContext: {
             sourcePath: "docs/code/README.md",
-            targetPath: "src/content/docs/reference/algokit-utils-ts/api/README.md",
+            targetPath:
+              "src/content/docs/reference/algokit-utils-ts/api/README.md",
             basePath: "src/content/docs/reference/algokit-utils-ts/api",
-            pathMappings: { "docs/code/": "" }
-          }
+            pathMappings: { "docs/code/": "" },
+          },
         },
         {
           id: "modules-index",
           sourcePath: "docs/code/modules/index.md",
-          targetPath: "src/content/docs/reference/algokit-utils-ts/api/modules/index.md",
-          content: 'This is the modules index.',
+          targetPath:
+            "src/content/docs/reference/algokit-utils-ts/api/modules/index.md",
+          content: "This is the modules index.",
           linkContext: {
             sourcePath: "docs/code/modules/index.md",
-            targetPath: "src/content/docs/reference/algokit-utils-ts/api/modules/index.md",
+            targetPath:
+              "src/content/docs/reference/algokit-utils-ts/api/modules/index.md",
             basePath: "src/content/docs/reference/algokit-utils-ts/api",
-            pathMappings: { "docs/code/": "" }
-          }
-        }
+            pathMappings: { "docs/code/": "" },
+          },
+        },
       ];
 
       const result = globalLinkTransform(testFiles, {
-        stripPrefixes: ['src/content/docs'],
+        stripPrefixes: ["src/content/docs"],
         linkMappings: [
           {
-            contextFilter: (context) => context.sourcePath.startsWith('docs/code/'),
+            contextFilter: (context) =>
+              context.sourcePath.startsWith("docs/code/"),
             relativeLinks: true,
             pattern: /.*/,
-            replacement: '',
-            global: false
-          }
-        ]
+            replacement: "",
+            global: false,
+          },
+        ],
       });
 
-      // The relative link `modules/` should be transformed to `/reference/algokit-utils-ts/api/modules/`
-      expect(result[0].content).toContain('[modules](/reference/algokit-utils-ts/api/modules/)');
+      expect(result[0].content).toContain(
+        "[modules](/reference/algokit-utils-ts/api/modules/)",
+      );
     });
   });
 
   describe("logging system", () => {
     it("should create logger with different levels", () => {
-      const silentLogger = createLogger('silent');
-      const defaultLogger = createLogger('default');
-      const verboseLogger = createLogger('verbose');
-      const debugLogger = createLogger('debug');
+      const silentLogger = createLogger("silent");
+      const defaultLogger = createLogger("default");
+      const verboseLogger = createLogger("verbose");
+      const debugLogger = createLogger("debug");
 
-      expect(silentLogger.getLevel()).toBe('silent');
-      expect(defaultLogger.getLevel()).toBe('default');
-      expect(verboseLogger.getLevel()).toBe('verbose');
-      expect(debugLogger.getLevel()).toBe('debug');
+      expect(silentLogger.getLevel()).toBe("silent");
+      expect(defaultLogger.getLevel()).toBe("default");
+      expect(verboseLogger.getLevel()).toBe("verbose");
+      expect(debugLogger.getLevel()).toBe("debug");
     });
 
-    it("should format import summary correctly", () => {
-      const logger = createLogger('default');
+    it("should format import summary without throwing", () => {
+      const logger = createLogger("default");
       const summary: ImportSummary = {
-        configName: 'Test Config',
-        repository: 'test/repo',
-        ref: 'main',
+        configName: "Test Config",
+        repository: "test/repo",
+        ref: "main",
         filesProcessed: 10,
         filesUpdated: 5,
         filesUnchanged: 5,
         assetsDownloaded: 3,
         assetsCached: 2,
         duration: 1500,
-        status: 'success'
+        status: "success",
       };
 
-      // This test mainly verifies the types work correctly
       expect(() => logger.logImportSummary(summary)).not.toThrow();
     });
   });