deps: update axios to 1.13.5

package.json

@@ -20,15 +20,15 @@
     "@types/node": "25.2.0",
     "@types/node-cron": "3.0.11",
     "@types/semver": "7.7.1",
-    "@vitest/coverage-istanbul": "4.0.18",
-    "@vitest/coverage-v8": "4.0.18",
+    "@vitest/coverage-istanbul": "^4.0.18",
+    "@vitest/coverage-v8": "^4.0.18",
     "eslint": "9.39.2",
     "typescript": "5.9.3",
-    "vitest": "4.0.18"
+    "vitest": "^4.0.18"
   },
   "dependencies": {
     "@nhcarrigan/logger": "1.1.1",
-    "axios": "1.13.4",
+    "axios": "1.13.5",
     "node-cron": "4.2.1",
     "semver": "7.7.3"
   }

pnpm-lock.yaml

@@ -12,8 +12,8 @@ importers:
         specifier: 1.1.1
         version: 1.1.1
       axios:
-        specifier: 1.13.4
-        version: 1.13.4
+        specifier: 1.13.5
+        version: 1.13.5
       node-cron:
         specifier: 4.2.1
         version: 4.2.1
@@ -37,10 +37,10 @@ importers:
         specifier: 7.7.1
         version: 7.7.1
       '@vitest/coverage-istanbul':
-        specifier: 4.0.18
+        specifier: ^4.0.18
         version: 4.0.18(vitest@4.0.18(@types/node@25.2.0))
       '@vitest/coverage-v8':
-        specifier: 4.0.18
+        specifier: ^4.0.18
         version: 4.0.18(vitest@4.0.18(@types/node@25.2.0))
       eslint:
         specifier: 9.39.2
@@ -49,7 +49,7 @@ importers:
         specifier: 5.9.3
         version: 5.9.3
       vitest:
-        specifier: 4.0.18
+        specifier: ^4.0.18
         version: 4.0.18(@types/node@25.2.0)
 
 packages:
@@ -831,8 +831,8 @@ packages:
     resolution: {integrity: sha512-wvUjBtSGN7+7SjNpq/9M2Tg350UZD3q62IFZLbRAR1bSMlCo1ZaeW+BJ+D090e4hIIZLBcTDWe4Mh4jvUDajzQ==}
     engines: {node: '>= 0.4'}
 
-  axios@1.13.4:
-    resolution: {integrity: sha512-1wVkUaAO6WyaYtCkcYCOx12ZgpGf9Zif+qXa4n+oYzK558YryKqiL6UWwd5DqiH3VRW0GYhTZQ/vlgJrCoNQlg==}
+  axios@1.13.5:
+    resolution: {integrity: sha512-cz4ur7Vb0xS4/KUN0tPWe44eqxrIu31me+fbang3ijiNscE129POzipJJA6zniq2C/Z6sJCjMimjS8Lc/GAs8Q==}
 
   balanced-match@1.0.2:
     resolution: {integrity: sha512-3oSeUO0TMV67hN1AmbXsK4yaqU7tjiHlbxRDZOpH0KW9+CeX4bRAaX0Anxt0tx2MrpRpWwQaPwIlISEJhYU5Pw==}
@@ -2989,7 +2989,7 @@ snapshots:
     dependencies:
       possible-typed-array-names: 1.1.0
 
-  axios@1.13.4:
+  axios@1.13.5:
     dependencies:
       follow-redirects: 1.15.11
       form-data: 4.0.5

src/services/giteaService.ts

@@ -6,9 +6,7 @@
 
 import axios, { isAxiosError, type AxiosInstance } from "axios";
 import { config } from "../config.js";
-import { logger } from "../utils/logger.js";
 import type {
-  GiteaCombinedStatus,
   GiteaFile,
   GiteaPullRequest,
   GiteaRepository,
@@ -144,87 +142,6 @@ class GiteaService {
     );
     return data;
   }
-
-  /**
-   * Gets the combined commit status for a specific commit by querying the Gitea API for all status checks.
-   * @param owner - The repository owner.
-   * @param repo - The repository name.
-   * @param sha - The commit SHA to check.
-   * @returns The combined status of all checks (pending, success, error, or failure).
-   */
-  public async getCommitStatus(
-    owner: string,
-    repo: string,
-    sha: string,
-  ): Promise<GiteaCombinedStatus> {
-    const { data } = await this.client.get<GiteaCombinedStatus>(
-      `/repos/${owner}/${repo}/commits/${sha}/status`,
-    );
-    return data;
-  }
-
-  /**
-   * Merges a pull request.
-   * @param owner - The repository owner.
-   * @param repo - The repository name.
-   * @param index - The pull request index number.
-   * @returns True if the merge was successful, false otherwise.
-   */
-  public async mergePullRequest(
-    owner: string,
-    repo: string,
-    index: number,
-  ): Promise<boolean> {
-    try {
-      await this.client.post(
-        `/repos/${owner}/${repo}/pulls/${String(index)}/merge`,
-        {
-          /* eslint-disable @typescript-eslint/naming-convention -- Gitea API uses snake_case */
-          Do:                        "merge",
-          MergeMessageField:         "",
-          MergeTitleField:           "",
-          delete_branch_after_merge: true,
-          force_merge:               true,
-          head_commit_id:            "",
-          merge_when_checks_succeed: false,
-          /* eslint-enable @typescript-eslint/naming-convention -- End Gitea API */
-        },
-      );
-      return true;
-    } catch (error) {
-      if (isAxiosError(error)) {
-        await logger.log(
-          "warn",
-          `Merge failed with status ${String(error.response?.status)}: ${JSON.stringify(error.response?.data)}`,
-        );
-        return false;
-      }
-      throw error;
-    }
-  }
-
-  /**
-   * Deletes a repository branch by name.
-   * @param owner - The repository owner.
-   * @param repo - The repository name.
-   * @param branch - The branch name to remove.
-   * @returns True if successful, false otherwise.
-   */
-  public async deleteBranch(
-    owner: string,
-    repo: string,
-    branch: string,
-  ): Promise<boolean> {
-    try {
-      await this.client.delete(`/repos/${owner}/${repo}/branches/${branch}`);
-      return true;
-    } catch (error) {
-      if (isAxiosError(error)) {
-        return false;
-      }
-      throw error;
-    }
-  }
 }
 
 export { GiteaService };

src/services/updateOrchestratorService.ts

@@ -6,10 +6,6 @@
 
 import { config } from "../config.js";
 import { logger } from "../utils/logger.js";
-import {
-  isMajorVersionBump,
-  stripVersionPrefix,
-} from "../utils/versionComparison.js";
 import { DependencyAnalyzerService } from "./dependencyAnalyzerService.js";
 import { GiteaService } from "./giteaService.js";
 import {
@@ -21,6 +17,15 @@ import { NpmService } from "./npmService.js";
 import type { GiteaRepository } from "../types/gitea.types.js";
 import type { DependencyUpdate, PackageJson } from "../types/package.types.js";
 
+/**
+ * Strips version prefix characters from a version string.
+ * @param version - The version string with potential prefixes.
+ * @returns The version without prefix characters.
+ */
+const stripVersionPrefix = (version: string): string => {
+  return version.replace(/^[<=>^~]*/, "");
+};
+
 /**
  * Generates the body content for a PR.
  * @param update - The dependency update information.
@@ -137,103 +142,6 @@ class UpdateOrchestratorService {
     await logger.log("info", "Dependency update check complete!");
   }
 
-  /**
-   * Attempts to merge an existing PR after checking CI status.
-   * @param repo - The repository information.
-   * @param update - The dependency update details.
-   * @param matchingPR - The existing PR with head SHA and number.
-   * @param matchingPR.head - The PR head information.
-   * @param matchingPR.head.sha - The commit SHA to check.
-   * @param matchingPR.number - The PR number for merging.
-   * @returns True if merge was attempted, false otherwise.
-   */
-  private async attemptPRMerge(
-    repo: GiteaRepository,
-    update: DependencyUpdate,
-    matchingPR: { head: { sha: string }; number: number },
-  ): Promise<boolean> {
-    const commitStatus = await this.giteaService.getCommitStatus(
-      config.giteaOrg,
-      repo.name,
-      matchingPR.head.sha,
-    );
-
-    if (commitStatus.state !== "success") {
-      await logger.log(
-        "info",
-        `    PR exists for ${update.packageName} but CI status is ${commitStatus.state}, skipping auto-merge...`,
-      );
-      return true;
-    }
-
-    await logger.log(
-      "info",
-      `    Auto-merging PR for ${update.packageName} (CI passed, non-major bump)...`,
-    );
-
-    const merged = await this.giteaService.mergePullRequest(
-      config.giteaOrg,
-      repo.name,
-      matchingPR.number,
-    );
-
-    if (merged) {
-      await logger.log(
-        "info",
-        `    Successfully merged PR for ${update.packageName}`,
-      );
-      return true;
-    }
-
-    await logger.log(
-      "warn",
-      `    Failed to merge PR for ${update.packageName}`,
-    );
-    return true;
-  }
-
-  /**
-   * Checks if an existing PR can be auto-merged based on CI status and version bump type.
-   * @param repo - The repository information.
-   * @param update - The dependency update details.
-   * @param branchName - The branch name for the PR.
-   * @returns True if the PR exists and was handled, false otherwise.
-   */
-  private async checkAndMergeExistingPR(
-    repo: GiteaRepository,
-    update: DependencyUpdate,
-    branchName: string,
-  ): Promise<boolean> {
-    const existingPRs = await this.giteaService.listPullRequests(
-      config.giteaOrg,
-      repo.name,
-      "open",
-    );
-
-    const matchingPR = existingPRs.find((pr) => {
-      return pr.head.ref === branchName;
-    });
-
-    if (matchingPR === undefined) {
-      return false;
-    }
-
-    const isMajorBump = isMajorVersionBump(
-      update.currentVersion,
-      update.latestVersion,
-    );
-
-    if (isMajorBump) {
-      await logger.log(
-        "info",
-        `    PR exists for ${update.packageName} but is a major version bump, skipping auto-merge...`,
-      );
-      return true;
-    }
-
-    return await this.attemptPRMerge(repo, update, matchingPR);
-  }
-
   /**
    * Creates or updates a PR for a dependency update.
    * @param repo - The repository information.
@@ -249,16 +157,6 @@ class UpdateOrchestratorService {
       = `${config.prBranchPrefix}${update.packageName.replaceAll(/[/@]/g, "-")}`;
 
     try {
-      const existingPRMerged = await this.checkAndMergeExistingPR(
-        repo,
-        update,
-        branchName,
-      );
-
-      if (existingPRMerged) {
-        return;
-      }
-
       const result = await createOrUpdateBranch({
         branchName:    branchName,
         clonedRepo:    clonedRepo,

src/types/gitea.types.ts

@@ -39,33 +39,6 @@ interface GiteaPullRequest {
   state:  "closed" | "open";
   title:  string;
 }
-
-interface GiteaCommitStatus {
-  context:     string;
-  created_at:  string;
-  description: string;
-  id:          number;
-  state:       "error" | "failure" | "pending" | "success" | "warning";
-  target_url:  string;
-  updated_at:  string;
-  url:         string;
-}
-
-interface GiteaCombinedStatus {
-  commit_url:  string;
-  repository:  GiteaRepository;
-  sha:         string;
-  state:       "error" | "failure" | "pending" | "success" | "warning";
-  statuses:    Array<GiteaCommitStatus>;
-  total_count: number;
-  url:         string;
-}
 /* eslint-enable @typescript-eslint/naming-convention -- End Gitea API types */
 
-export type {
-  GiteaCombinedStatus,
-  GiteaCommitStatus,
-  GiteaFile,
-  GiteaPullRequest,
-  GiteaRepository,
-};
+export type { GiteaFile, GiteaPullRequest, GiteaRepository };

src/utils/logger.ts

@@ -4,14 +4,12 @@
  * @author Naomi Carrigan
  */
 
-import type { Logger } from "@nhcarrigan/logger";
+import { Logger } from "@nhcarrigan/logger";
 
-// Import { Logger } from "@nhcarrigan/logger";
+// import { Logger } from "@nhcarrigan/logger";
 
-// Const logger = new Logger("Minori", process.env.LOG_TOKEN ?? "");
+// const logger = new Logger("Minori", process.env.LOG_TOKEN ?? "");
 
-/* eslint-disable no-console -- Temporary mock logger for development */
-/* eslint-disable @typescript-eslint/consistent-type-assertions -- Mock logger requires type assertion */
 const logger = {
   error: (message: string, error: Error) => {
     console.error(message, error);
@@ -20,6 +18,5 @@ const logger = {
     console.log(level, message);
   },
 } as unknown as Logger;
-/* eslint-enable no-console, @typescript-eslint/consistent-type-assertions -- Re-enable rules after mock logger */
 
 export { logger };

src/utils/versionComparison.ts

@@ -1,64 +0,0 @@
-/**
- * @copyright NHCarrigan
- * @license Naomi's Public License
- * @author Naomi Carrigan
- */
-
-/**
- * Strips version prefix characters from a version string.
- * @param version - The version string with potential prefixes.
- * @returns The version without prefix characters.
- */
-const stripVersionPrefix = (version: string): string => {
-  return version.replace(/^[<=>^~]*/, "");
-};
-
-/**
- * Parses a semantic version string into its components.
- * @param version - The version string to parse (e.g., "1.2.3").
- * @returns An object with major, minor, and patch numbers, or null if invalid.
- */
-const parseVersion = (
-  version: string,
-): { major: number; minor: number; patch: number } | null => {
-  const cleaned = stripVersionPrefix(version);
-  const parts = cleaned.split(".");
-
-  if (parts.length < 3) {
-    return null;
-  }
-
-  const major = Number.parseInt(parts[0] ?? "0", 10);
-  const minor = Number.parseInt(parts[1] ?? "0", 10);
-  const patchPart = parts[2] ?? "0";
-  const patch = Number.parseInt(patchPart.split("-")[0] ?? "0", 10);
-
-  if (Number.isNaN(major) || Number.isNaN(minor) || Number.isNaN(patch)) {
-    return null;
-  }
-
-  return { major, minor, patch };
-};
-
-/**
- * Determines if a version update is a major version bump.
- * A major bump occurs when the major version number increases.
- * @param fromVersion - The current version.
- * @param toVersion - The target version.
- * @returns True if this is a major version bump, false otherwise.
- */
-const isMajorVersionBump = (
-  fromVersion: string,
-  toVersion: string,
-): boolean => {
-  const from = parseVersion(fromVersion);
-  const to = parseVersion(toVersion);
-
-  if (from === null || to === null) {
-    return false;
-  }
-
-  return to.major > from.major;
-};
-
-export { isMajorVersionBump, stripVersionPrefix };

test/services/giteaService.spec.ts

@@ -6,16 +6,11 @@
 
 /* eslint-disable vitest/valid-expect -- Test expectations don't need messages */
 /* eslint-disable max-lines-per-function -- Test suites require many test cases */
-/* eslint-disable max-lines -- Test suites naturally have many cases */
-/* eslint-disable max-statements -- Test suites naturally have many statements */
 /* eslint-disable @typescript-eslint/consistent-type-assertions -- Required for mocking */
 /* eslint-disable @typescript-eslint/consistent-type-imports -- Dynamic imports */
 /* eslint-disable @typescript-eslint/naming-convention -- Environment variables and Gitea API format */
 /* eslint-disable max-nested-callbacks -- Vitest structure requires nested callbacks */
 /* eslint-disable vitest/require-to-throw-message -- Generic throw assertion */
-/* eslint-disable vitest/prefer-to-be-truthy -- toBe(true) is clearer for boolean functions */
-/* eslint-disable vitest/prefer-to-be-falsy -- toBe(false) is clearer for boolean functions */
-/* eslint-disable stylistic/max-len -- Test files have long object literals */
 
 import axios, { AxiosError, type AxiosResponse } from "axios";
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
@@ -29,7 +24,6 @@ vi.mock("axios", async() => {
     default:    {
       create: vi.fn(() => {
         return {
-          delete: vi.fn(),
           get:  vi.fn(),
           post: vi.fn(),
         };
@@ -75,8 +69,6 @@ describe("giteaService", () => {
   let mockGet: ReturnType<typeof vi.fn>;
   // eslint-disable-next-line @typescript-eslint/init-declarations -- Reassigned in beforeEach
   let mockPost: ReturnType<typeof vi.fn>;
-  // eslint-disable-next-line @typescript-eslint/init-declarations -- Reassigned in beforeEach
-  let mockDelete: ReturnType<typeof vi.fn>;
   const originalEnvironment = process.env;
 
   beforeEach(() => {
@@ -85,9 +77,7 @@ describe("giteaService", () => {
 
     mockGet = vi.fn();
     mockPost = vi.fn();
-    mockDelete = vi.fn();
     vi.mocked(axios.create).mockReturnValue({
-      delete: mockDelete,
       get:  mockGet,
       post: mockPost,
     } as unknown as ReturnType<typeof axios.create>);
@@ -316,69 +306,4 @@ describe("giteaService", () => {
       { params: { state: "closed" } },
     );
   });
-
-  it("should get commit status", async() => {
-    expect.assertions(2);
-    const mockStatus = {
-      commit_url:  "https://git.nhcarrigan.com/api/v1/repos/owner/repo/commits/abc123",
-      repository:  createMockRepository({ id: 1, name: "test-repo" }),
-      sha:         "abc123",
-      state:       "success",
-      statuses:    [],
-      total_count: 0,
-      url:         "https://git.nhcarrigan.com/api/v1/repos/owner/repo/commits/abc123/status",
-    };
-    mockGet.mockResolvedValueOnce({ data: mockStatus });
-    const result = await giteaService.getCommitStatus("owner", "repo", "abc123");
-    expect(result).toStrictEqual(mockStatus);
-    expect(mockGet).toHaveBeenCalledWith("/repos/owner/repo/commits/abc123/status");
-  });
-
-  it("should merge a pull request successfully", async() => {
-    expect.assertions(2);
-    mockPost.mockResolvedValueOnce({ data: {} });
-    const result = await giteaService.mergePullRequest("owner", "repo", 1);
-    expect(result).toBe(true);
-    expect(mockPost).toHaveBeenCalledWith("/repos/owner/repo/pulls/1/merge", {
-      Do:                        "merge",
-      MergeMessageField:         "",
-      MergeTitleField:           "",
-      delete_branch_after_merge: true,
-      force_merge:               false,
-      head_commit_id:            "",
-      merge_when_checks_succeed: false,
-    });
-  });
-
-  it("should return false when merge fails", async() => {
-    expect.assertions(1);
-    const axiosError = new AxiosError("Merge conflict");
-    mockPost.mockRejectedValueOnce(axiosError);
-    const result = await giteaService.mergePullRequest("owner", "repo", 1);
-    expect(result).toBe(false);
-  });
-
-  it("should delete a branch successfully", async() => {
-    expect.assertions(2);
-    mockDelete.mockResolvedValueOnce({ data: {} });
-    const result = await giteaService.deleteBranch(
-      "owner",
-      "repo",
-      "feature-branch",
-    );
-    expect(result).toBe(true);
-    expect(mockDelete).toHaveBeenCalledWith("/repos/owner/repo/branches/feature-branch");
-  });
-
-  it("should return false when branch deletion fails", async() => {
-    expect.assertions(1);
-    const axiosError = new AxiosError("Branch not found");
-    mockDelete.mockRejectedValueOnce(axiosError);
-    const result = await giteaService.deleteBranch(
-      "owner",
-      "repo",
-      "nonexistent-branch",
-    );
-    expect(result).toBe(false);
-  });
 });

test/services/updateOrchestratorService.spec.ts

@@ -18,8 +18,6 @@ const mockGiteaGetFileContent = vi.fn();
 const mockGiteaListOrgRepositories = vi.fn();
 const mockGiteaCreatePullRequest = vi.fn();
 const mockGiteaListPullRequests = vi.fn();
-const mockGiteaGetCommitStatus = vi.fn();
-const mockGiteaMergePullRequest = vi.fn();
 const mockNpmGetPackageChangelog = vi.fn();
 const mockNpmGetPackageInfo = vi.fn();
 const mockAnalyzePackageJson = vi.fn();
@@ -41,11 +39,9 @@ vi.mock("../../src/services/giteaService.js", () => {
 
     GiteaService: class MockGiteaService {
       public createPullRequest = mockGiteaCreatePullRequest;
-      public getCommitStatus = mockGiteaGetCommitStatus;
       public getFileContent = mockGiteaGetFileContent;
       public listOrgRepositories = mockGiteaListOrgRepositories;
       public listPullRequests = mockGiteaListPullRequests;
-      public mergePullRequest = mockGiteaMergePullRequest;
     },
   };
 });
@@ -225,7 +221,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockNpmGetPackageChangelog.mockResolvedValue("## Changelog");
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
@@ -272,7 +267,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
     mockCreateOrUpdateBranch.mockResolvedValue({
@@ -295,7 +289,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
     mockCreateOrUpdateBranch.mockResolvedValue({
@@ -319,7 +312,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
     mockCreateOrUpdateBranch.mockResolvedValue({
@@ -362,7 +354,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
     mockCreateOrUpdateBranch.mockResolvedValue({
@@ -386,7 +377,6 @@ describe("updateOrchestratorService", () => {
     const mockUpdates = [ createMockUpdate() ];
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockNpmGetPackageChangelog.mockResolvedValue("## Changelog");
     mockCloneRepository.mockResolvedValue(createMockClonedRepo());
@@ -436,7 +426,6 @@ describe("updateOrchestratorService", () => {
     const mockCleanup = vi.fn();
     mockGiteaListOrgRepositories.mockResolvedValue(mockRepos);
     mockGiteaGetFileContent.mockResolvedValue(mockFileContent);
-    mockGiteaListPullRequests.mockResolvedValue([]);
     mockAnalyzePackageJson.mockResolvedValue(mockUpdates);
     mockCloneRepository.mockResolvedValue(createMockClonedRepo(mockCleanup));
     mockCreateOrUpdateBranch.mockResolvedValue({

test/utils/versionComparison.spec.ts

@@ -1,108 +0,0 @@
-/**
- * @copyright NHCarrigan
- * @license Naomi's Public License
- * @author Naomi Carrigan
- */
-
-/* eslint-disable vitest/valid-expect -- Test expectations don't need messages */
-/* eslint-disable max-lines-per-function -- Test suites naturally have many cases */
-/* eslint-disable max-nested-callbacks -- Vitest structure requires nesting */
-/* eslint-disable vitest/prefer-to-be-truthy -- toBe(true) is clearer for boolean functions */
-/* eslint-disable vitest/prefer-to-be-falsy -- toBe(false) is clearer for boolean functions */
-
-import { describe, expect, it } from "vitest";
-import {
-  isMajorVersionBump,
-  stripVersionPrefix,
-} from "../../src/utils/versionComparison.js";
-
-describe("versionComparison", () => {
-  describe("stripVersionPrefix", () => {
-    it("should strip caret prefix", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix("^1.2.3")).toBe("1.2.3");
-    });
-
-    it("should strip tilde prefix", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix("~1.2.3")).toBe("1.2.3");
-    });
-
-    it("should strip greater than prefix", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix(">1.2.3")).toBe("1.2.3");
-    });
-
-    it("should strip less than prefix", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix("<1.2.3")).toBe("1.2.3");
-    });
-
-    it("should strip equals prefix", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix("=1.2.3")).toBe("1.2.3");
-    });
-
-    it("should strip multiple prefix characters", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix(">=1.2.3")).toBe("1.2.3");
-    });
-
-    it("should return version without prefix unchanged", () => {
-      expect.assertions(1);
-      expect(stripVersionPrefix("1.2.3")).toBe("1.2.3");
-    });
-  });
-
-  describe("isMajorVersionBump", () => {
-    it("should detect major version bump", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "2.0.0")).toBe(true);
-    });
-
-    it("should detect major version bump with prefixes", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("^1.2.3", "^2.0.0")).toBe(true);
-    });
-
-    it("should not detect minor version bump as major", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "1.3.0")).toBe(false);
-    });
-
-    it("should not detect patch version bump as major", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "1.2.4")).toBe(false);
-    });
-
-    it("should handle version with pre-release tags", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "2.0.0-beta.1")).toBe(true);
-    });
-
-    it("should return false for invalid from version", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("invalid", "2.0.0")).toBe(false);
-    });
-
-    it("should return false for invalid to version", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "invalid")).toBe(false);
-    });
-
-    it("should return false for both invalid versions", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("invalid", "also-invalid")).toBe(false);
-    });
-
-    it("should handle 0.x.x to 1.x.x as major bump", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("0.9.5", "1.0.0")).toBe(true);
-    });
-
-    it("should not detect same version as major bump", () => {
-      expect.assertions(1);
-      expect(isMajorVersionBump("1.2.3", "1.2.3")).toBe(false);
-    });
-  });
-});