@larkiny/astro-github-loader 0.11.2 → 0.12.0

package/src/github.dryrun.ts

@@ -1,9 +1,11 @@
 import { promises as fs } from "node:fs";
 import { existsSync } from "node:fs";
 import { join } from "node:path";
+import { Octokit } from "octokit";
+import type { Logger } from "./github.logger.js";
 import type { ImportOptions, LoaderContext } from "./github.types.js";
 
-const STATE_FILENAME = '.github-import-state.json';
+const STATE_FILENAME = ".github-import-state.json";
 
 /**
  * Represents the state of a single import configuration
@@ -55,30 +57,48 @@ export interface RepositoryChangeInfo {
  * Creates a unique identifier for an import configuration
  */
 export function createConfigId(config: ImportOptions): string {
-  return `${config.owner}/${config.repo}@${config.ref || 'main'}`;
+  return `${config.owner}/${config.repo}@${config.ref || "main"}`;
 }
 
 /**
  * Loads the import state from the state file
  */
-export async function loadImportState(workingDir: string): Promise<StateFile> {
+export async function loadImportState(
+  workingDir: string,
+  logger?: Logger,
+): Promise<StateFile> {
   const statePath = join(workingDir, STATE_FILENAME);
-  
+
   if (!existsSync(statePath)) {
     return {
       imports: {},
-      lastChecked: new Date().toISOString()
+      lastChecked: new Date().toISOString(),
     };
   }
 
   try {
-    const content = await fs.readFile(statePath, 'utf-8');
-    return JSON.parse(content);
+    const content = await fs.readFile(statePath, "utf-8");
+    const parsed = JSON.parse(content);
+    // Validate the parsed state has the expected shape
+    if (
+      !parsed ||
+      typeof parsed !== "object" ||
+      !("imports" in parsed) ||
+      typeof parsed.imports !== "object"
+    ) {
+      const msg = `Malformed state file at ${statePath}, starting fresh`;
+      // eslint-disable-next-line no-console -- fallback when no logger provided
+      logger ? logger.warn(msg) : console.warn(msg);
+      return { imports: {}, lastChecked: new Date().toISOString() };
+    }
+    return parsed;
   } catch (error) {
-    console.warn(`Failed to load import state from ${statePath}, starting fresh:`, error);
+    const msg = `Failed to load import state from ${statePath}, starting fresh: ${error}`;
+    // eslint-disable-next-line no-console -- fallback when no logger provided
+    logger ? logger.warn(msg) : console.warn(msg);
     return {
       imports: {},
-      lastChecked: new Date().toISOString()
+      lastChecked: new Date().toISOString(),
     };
   }
 }
@@ -86,13 +106,19 @@ export async function loadImportState(workingDir: string): Promise<StateFile> {
 /**
  * Saves the import state to the state file
  */
-async function saveImportState(workingDir: string, state: StateFile): Promise<void> {
+async function saveImportState(
+  workingDir: string,
+  state: StateFile,
+  logger?: Logger,
+): Promise<void> {
   const statePath = join(workingDir, STATE_FILENAME);
-  
+
   try {
-    await fs.writeFile(statePath, JSON.stringify(state, null, 2), 'utf-8');
+    await fs.writeFile(statePath, JSON.stringify(state, null, 2), "utf-8");
   } catch (error) {
-    console.warn(`Failed to save import state to ${statePath}:`, error);
+    const msg = `Failed to save import state to ${statePath}: ${error}`;
+    // eslint-disable-next-line no-console -- fallback when no logger provided
+    logger ? logger.warn(msg) : console.warn(msg);
   }
 }
 
@@ -100,9 +126,9 @@ async function saveImportState(workingDir: string, state: StateFile): Promise<vo
  * Gets the latest commit information for a repository path
  */
 export async function getLatestCommitInfo(
-  octokit: any,
+  octokit: Octokit,
   config: ImportOptions,
-  signal?: AbortSignal
+  signal?: AbortSignal,
 ): Promise<{ sha: string; message: string; date: string } | null> {
   const { owner, repo, ref = "main" } = config;
 
@@ -113,7 +139,7 @@ export async function getLatestCommitInfo(
       repo,
       sha: ref,
       per_page: 1,
-      request: { signal }
+      request: { signal },
     });
 
     if (data.length === 0) {
@@ -123,11 +149,19 @@ export async function getLatestCommitInfo(
     const latestCommit = data[0];
     return {
       sha: latestCommit.sha,
-      message: latestCommit.commit.message.split('\n')[0], // First line only
-      date: latestCommit.commit.committer?.date || latestCommit.commit.author?.date || new Date().toISOString()
+      message: latestCommit.commit.message.split("\n")[0], // First line only
+      date:
+        latestCommit.commit.committer?.date ||
+        latestCommit.commit.author?.date ||
+        new Date().toISOString(),
     };
-  } catch (error: any) {
-    if (error.status === 404) {
+  } catch (error: unknown) {
+    if (
+      typeof error === "object" &&
+      error !== null &&
+      "status" in error &&
+      (error as { status: number }).status === 404
+    ) {
       throw new Error(`Repository not found: ${owner}/${repo}`);
     }
     throw error;
@@ -138,26 +172,28 @@ export async function getLatestCommitInfo(
  * Checks a single repository for changes
  */
 async function checkRepositoryForChanges(
-  octokit: any,
+  octokit: Octokit,
   config: ImportOptions,
   currentState: ImportState,
-  signal?: AbortSignal
+  signal?: AbortSignal,
 ): Promise<RepositoryChangeInfo> {
   const configName = config.name || `${config.owner}/${config.repo}`;
 
   try {
     const latestCommit = await getLatestCommitInfo(octokit, config, signal);
-    
+
     if (!latestCommit) {
       return {
         config,
         state: currentState,
         needsReimport: false,
-        error: "No commits found in repository"
+        error: "No commits found in repository",
       };
     }
 
-    const needsReimport = !currentState.lastCommitSha || currentState.lastCommitSha !== latestCommit.sha;
+    const needsReimport =
+      !currentState.lastCommitSha ||
+      currentState.lastCommitSha !== latestCommit.sha;
 
     return {
       config,
@@ -165,15 +201,14 @@ async function checkRepositoryForChanges(
       needsReimport,
       latestCommitSha: latestCommit.sha,
       latestCommitMessage: latestCommit.message,
-      latestCommitDate: latestCommit.date
+      latestCommitDate: latestCommit.date,
     };
-
-  } catch (error: any) {
+  } catch (error: unknown) {
     return {
       config,
       state: currentState,
       needsReimport: false,
-      error: error.message
+      error: error instanceof Error ? error.message : String(error),
     };
   }
 }
@@ -184,9 +219,10 @@ async function checkRepositoryForChanges(
 export async function updateImportState(
   workingDir: string,
   config: ImportOptions,
-  commitSha?: string
+  commitSha?: string,
+  logger?: Logger,
 ): Promise<void> {
-  const state = await loadImportState(workingDir);
+  const state = await loadImportState(workingDir, logger);
   const configId = createConfigId(config);
   const configName = config.name || `${config.owner}/${config.repo}`;
 
@@ -195,10 +231,10 @@ export async function updateImportState(
     repoId: configId,
     lastCommitSha: commitSha,
     lastImported: new Date().toISOString(),
-    ref: config.ref || 'main'
+    ref: config.ref || "main",
   };
 
-  await saveImportState(workingDir, state);
+  await saveImportState(workingDir, state, logger);
 }
 
 /**
@@ -207,14 +243,14 @@ export async function updateImportState(
 export async function performDryRun(
   configs: ImportOptions[],
   context: LoaderContext,
-  octokit: any,
+  octokit: Octokit,
   workingDir: string = process.cwd(),
-  signal?: AbortSignal
+  signal?: AbortSignal,
 ): Promise<RepositoryChangeInfo[]> {
   const { logger } = context;
-  
+
   logger.info("🔍 Performing dry run - checking for repository changes...");
-  
+
   // Load current state
   const state = await loadImportState(workingDir);
   const results: RepositoryChangeInfo[] = [];
@@ -222,33 +258,40 @@ export async function performDryRun(
   // Check each configuration
   for (const config of configs) {
     if (config.enabled === false) {
-      logger.debug(`Skipping disabled config: ${config.name || `${config.owner}/${config.repo}`}`);
+      logger.debug(
+        `Skipping disabled config: ${config.name || `${config.owner}/${config.repo}`}`,
+      );
       continue;
     }
 
     const configId = createConfigId(config);
     const configName = config.name || `${config.owner}/${config.repo}`;
-    
+
     // Get current state for this config
     const currentState: ImportState = state.imports[configId] || {
       name: configName,
       repoId: configId,
-      ref: config.ref || 'main'
+      ref: config.ref || "main",
     };
 
     logger.debug(`Checking ${configName}...`);
-    
+
     try {
-      const changeInfo = await checkRepositoryForChanges(octokit, config, currentState, signal);
+      const changeInfo = await checkRepositoryForChanges(
+        octokit,
+        config,
+        currentState,
+        signal,
+      );
       results.push(changeInfo);
-    } catch (error: any) {
+    } catch (error: unknown) {
       if (signal?.aborted) throw error;
-      
+
       results.push({
         config,
         state: currentState,
         needsReimport: false,
-        error: `Failed to check repository: ${error.message}`
+        error: `Failed to check repository: ${error instanceof Error ? error.message : String(error)}`,
       });
     }
   }
@@ -263,16 +306,20 @@ export async function performDryRun(
 /**
  * Formats and displays the dry run results
  */
-export function displayDryRunResults(results: RepositoryChangeInfo[], logger: any): void {
+export function displayDryRunResults(
+  results: RepositoryChangeInfo[],
+  logger: { info: (msg: string) => void },
+): void {
   logger.info("\n📊 Repository Import Status:");
-  logger.info("=" .repeat(50));
+  logger.info("=".repeat(50));
 
   let needsReimportCount = 0;
   let errorCount = 0;
 
   for (const result of results) {
-    const configName = result.config.name || `${result.config.owner}/${result.config.repo}`;
-    
+    const configName =
+      result.config.name || `${result.config.owner}/${result.config.repo}`;
+
     if (result.error) {
       logger.info(`❌ ${configName}: ${result.error}`);
       errorCount++;
@@ -304,12 +351,16 @@ export function displayDryRunResults(results: RepositoryChangeInfo[], logger: an
     }
   }
 
-  logger.info("=" .repeat(50));
-  logger.info(`📈 Summary: ${needsReimportCount} of ${results.length} repositories need re-import, ${errorCount} errors`);
-  
+  logger.info("=".repeat(50));
+  logger.info(
+    `📈 Summary: ${needsReimportCount} of ${results.length} repositories need re-import, ${errorCount} errors`,
+  );
+
   if (needsReimportCount > 0) {
     logger.info("\n💡 To import updated repositories:");
-    logger.info("1. Delete the target import folders for repositories that need re-import");
+    logger.info(
+      "1. Delete the target import folders for repositories that need re-import",
+    );
     logger.info("2. Run the import process normally (dryRun: false)");
     logger.info("3. Fresh content will be imported automatically");
   } else {
@@ -336,4 +387,4 @@ function getTimeAgo(date: Date): string {
   } else {
     return date.toLocaleDateString();
   }
-}
+}