further refactors

Author: barrymichaeldoyle

src/core/dependency-checker.ts

@@ -4,7 +4,8 @@ import { PatchPulseConfig, shouldSkipPackage } from '../services/config';
 import { getLatestVersion } from '../services/npm';
 import { type DependencyInfo } from '../types';
 import { ProgressSpinner } from '../ui/progress';
-import { getUpdateType, isVersionOutdated } from '../ui/version';
+import { getUpdateType } from '../utils/getUpdateType';
+import { isVersionOutdated } from '../utils/isVersionOutdated';
 
 export async function checkDependencyVersions(
   dependencies: Record<string, string> | undefined,

src/services/tests/config.test.ts renamed to src/services/__tests__/config.test.ts

@@ -1,6 +1,6 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { type DependencyInfo } from '../../types';
-import { displaySummary } from '../display/summary';
+import { type DependencyInfo } from '../../../types';
+import { displaySummary } from '../summary';
 
 describe('displaySummary', () => {
   let consoleSpy: ReturnType<typeof vi.spyOn>;

src/services/tests/package.test.ts renamed to src/services/__tests__/package.test.ts

@@ -0,0 +1,33 @@
+import chalk from 'chalk';
+import { parseVersion } from './parseVersion';
+
+/**
+ * Determines the type of update required based on the current and latest versions
+ * @param current - The current version string
+ * @param latest - The latest version string
+ * @returns The type of update required ('patch', 'minor', or 'major')
+ */
+export function getUpdateType(
+  current: string,
+  latest: string
+): 'patch' | 'minor' | 'major' {
+  try {
+    const currentVersion = parseVersion(current);
+    const latestVersion = parseVersion(latest);
+
+    if (latestVersion.major > currentVersion.major) {
+      return 'major';
+    }
+    if (latestVersion.minor > currentVersion.minor) {
+      return 'minor';
+    }
+    if (latestVersion.patch > currentVersion.patch) {
+      return 'patch';
+    }
+    return 'patch';
+  } catch (error) {
+    // Handle invalid version formats gracefully
+    console.warn(chalk.yellow(`⚠️  Invalid version format: ${error}`));
+    return 'patch'; // Default fallback
+  }
+}

src/ui/tests/progress.test.ts renamed to src/ui/__tests__/progress.test.ts

@@ -0,0 +1,38 @@
+import chalk from 'chalk';
+import { parseVersion } from './parseVersion';
+
+/**
+ * Checks if the current version is outdated compared to the latest version
+ * @param current - The current version string
+ * @param latest - The latest version string
+ * @returns True if the current version is outdated, false otherwise
+ */
+export function isVersionOutdated(current: string, latest: string): boolean {
+  try {
+    const currentVersion = parseVersion(current);
+    const latestVersion = parseVersion(latest);
+
+    // Compare major versions first
+    if (latestVersion.major > currentVersion.major) {
+      return true;
+    }
+    if (latestVersion.major < currentVersion.major) {
+      return false;
+    }
+
+    // If major versions are equal, compare minor versions
+    if (latestVersion.minor > currentVersion.minor) {
+      return true;
+    }
+    if (latestVersion.minor < currentVersion.minor) {
+      return false;
+    }
+
+    // If minor versions are equal, compare patch versions
+    return latestVersion.patch > currentVersion.patch;
+  } catch (error) {
+    // Handle invalid version formats gracefully
+    console.warn(chalk.yellow(`⚠️  Invalid version format: ${error}`));
+    return false; // Default to not outdated if we can't parse
+  }
+}