movehat 0.2.3 → 0.2.5

package/src/helpers/__tests__/semver-utils.test.ts

@@ -1,121 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { isNewerVersion } from '../semver-utils.js';
-
-describe('isNewerVersion', () => {
-  describe('standard semver comparisons', () => {
-    it('should detect newer major version', () => {
-      expect(isNewerVersion('1.0.0', '2.0.0')).toBe(true);
-      expect(isNewerVersion('0.1.0', '1.0.0')).toBe(true);
-    });
-
-    it('should detect newer minor version', () => {
-      expect(isNewerVersion('1.0.0', '1.1.0')).toBe(true);
-      expect(isNewerVersion('1.5.0', '1.6.0')).toBe(true);
-    });
-
-    it('should detect newer patch version', () => {
-      expect(isNewerVersion('1.0.0', '1.0.1')).toBe(true);
-      expect(isNewerVersion('1.0.9', '1.0.10')).toBe(true);
-    });
-
-    it('should return false for same version', () => {
-      expect(isNewerVersion('1.0.0', '1.0.0')).toBe(false);
-      expect(isNewerVersion('2.5.3', '2.5.3')).toBe(false);
-    });
-
-    it('should return false for older versions', () => {
-      expect(isNewerVersion('2.0.0', '1.0.0')).toBe(false);
-      expect(isNewerVersion('1.1.0', '1.0.0')).toBe(false);
-      expect(isNewerVersion('1.0.1', '1.0.0')).toBe(false);
-    });
-  });
-
-  describe('variable-length version handling', () => {
-    it('should handle two-part versions', () => {
-      expect(isNewerVersion('1.0', '1.1')).toBe(true);
-      expect(isNewerVersion('1.0', '2.0')).toBe(true);
-      expect(isNewerVersion('1.1', '1.0')).toBe(false);
-    });
-
-    it('should compare versions with different lengths', () => {
-      expect(isNewerVersion('1.0', '1.0.1')).toBe(true);
-      expect(isNewerVersion('1.0.0', '1.1')).toBe(true);
-      expect(isNewerVersion('1.0.1', '1.0')).toBe(false);
-    });
-
-    it('should handle four-part versions', () => {
-      expect(isNewerVersion('1.0.0.0', '1.0.0.1')).toBe(true);
-      expect(isNewerVersion('1.0.0.1', '1.0.0.0')).toBe(false);
-    });
-
-    it('should treat missing parts as zero', () => {
-      expect(isNewerVersion('1.0', '1.0.0')).toBe(false);
-      expect(isNewerVersion('1.0.0', '1.0')).toBe(false);
-    });
-  });
-
-  describe('pre-release version handling', () => {
-    it('should consider stable newer than pre-release with same base', () => {
-      expect(isNewerVersion('1.0.0-alpha', '1.0.0')).toBe(true);
-      expect(isNewerVersion('1.0.0-beta.1', '1.0.0')).toBe(true);
-      expect(isNewerVersion('1.0.0-rc.1', '1.0.0')).toBe(true);
-    });
-
-    it('should consider pre-release older than stable with same base', () => {
-      expect(isNewerVersion('1.0.0', '1.0.0-alpha')).toBe(false);
-      expect(isNewerVersion('1.0.0', '1.0.0-beta.1')).toBe(false);
-    });
-
-    it('should strip pre-release tags for base comparison', () => {
-      expect(isNewerVersion('1.0.0-alpha', '1.1.0-alpha')).toBe(true);
-      expect(isNewerVersion('1.0.0-alpha', '2.0.0-beta')).toBe(true);
-    });
-
-    it('should handle complex pre-release tags', () => {
-      expect(isNewerVersion('1.0.0-alpha.0', '1.0.0')).toBe(true);
-      expect(isNewerVersion('1.0.0-beta.2.3', '1.0.0')).toBe(true);
-    });
-  });
-
-  describe('edge cases', () => {
-    it('should handle large version numbers', () => {
-      expect(isNewerVersion('100.200.300', '100.200.301')).toBe(true);
-      expect(isNewerVersion('999.999.999', '1000.0.0')).toBe(true);
-    });
-
-    it('should handle version 0.x.x correctly', () => {
-      expect(isNewerVersion('0.0.1', '0.0.2')).toBe(true);
-      expect(isNewerVersion('0.1.0', '0.2.0')).toBe(true);
-      expect(isNewerVersion('0.9.9', '1.0.0')).toBe(true);
-    });
-
-    it('should throw for invalid version format', () => {
-      expect(() => isNewerVersion('invalid', '1.0.0')).toThrow('Invalid version format');
-      expect(() => isNewerVersion('1.0.0', 'invalid')).toThrow('Invalid version format');
-      expect(() => isNewerVersion('1.a.0', '1.0.0')).toThrow('Invalid version format');
-      expect(() => isNewerVersion('1..0', '1.0.0')).toThrow('Invalid version format');
-    });
-
-    it('should reject versions with unstripped "v" prefix', () => {
-      // isNewerVersion expects pre-stripped versions; callers must remove leading "v".
-      expect(() => isNewerVersion('v1.0.0', '1.0.0')).toThrow('Invalid version format');
-    });
-  });
-
-  describe('real-world scenarios', () => {
-    it('should handle npm-style version bumps', () => {
-      // Patch bump
-      expect(isNewerVersion('0.1.5', '0.1.6')).toBe(true);
-      // Minor bump
-      expect(isNewerVersion('0.1.6', '0.2.0')).toBe(true);
-      // Major bump
-      expect(isNewerVersion('0.9.9', '1.0.0')).toBe(true);
-    });
-
-    it('should handle movehat version progression', () => {
-      expect(isNewerVersion('0.0.1', '0.1.0')).toBe(true);
-      expect(isNewerVersion('0.1.0', '0.1.1')).toBe(true);
-      expect(isNewerVersion('0.1.9', '0.1.10')).toBe(true);
-    });
-  });
-});

package/src/helpers/__tests__/setupLocalTesting.fork-network.test.ts

@@ -1,212 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-/**
- * F1 — Harness.createFork(network) must honor the requested network.
- *
- * Mocks ForkManager so we can capture which RPC URL the fresh-fork
- * code path picks. Strategy mirrors the pattern in
- * src/fork/__tests__/manager.test.ts: replace the manager module with
- * a stub that records every call to `initialize`.
- *
- * The mock also implements `load()` + `getMetadata()` so the
- * "fork-exists, wrong network" case (audit-f1 follow-up) can exercise
- * the metadata-mismatch guard in the `else` branch of `setupWithFork`.
- */
-
-interface InitCall {
-  nodeUrl: string;
-  networkName?: string;
-  apiKey?: string;
-}
-const initializeCalls: InitCall[] = [];
-
-vi.mock("../../fork/manager.js", async () => {
-  const fs = await import("node:fs");
-  return {
-    ForkManager: class {
-      forkPath: string;
-      metadata: { network: string; nodeUrl: string } | null = null;
-      constructor(forkPath: string) {
-        this.forkPath = forkPath;
-      }
-      async initialize(nodeUrl: string, networkName?: string, apiKey?: string) {
-        const entry: InitCall = { nodeUrl };
-        if (networkName !== undefined) entry.networkName = networkName;
-        if (apiKey !== undefined) entry.apiKey = apiKey;
-        initializeCalls.push(entry);
-        this.metadata = { network: networkName ?? "custom", nodeUrl };
-      }
-      load() {
-        const raw = fs.readFileSync(`${this.forkPath}/metadata.json`, "utf-8");
-        this.metadata = JSON.parse(raw);
-      }
-      getMetadata() {
-        if (!this.metadata) {
-          throw new Error("Fork not initialized");
-        }
-        return this.metadata;
-      }
-      setApiKey() {}
-      async resetState() {}
-      async fundAccount() {}
-      async fundMultipleAccounts() {}
-    },
-  };
-});
-
-vi.mock("../../fork/server.js", () => {
-  return {
-    ForkServer: class {
-      constructor(_p: string, _port: number) {}
-      async start() {}
-      async stop() {}
-    },
-  };
-});
-
-vi.mock("../../runtime.js", () => ({
-  initRuntime: vi.fn(async () => ({})),
-}));
-
-vi.mock("../../core/AccountManager.js", () => {
-  let _seq = 0;
-  return {
-    AccountManager: {
-      createBatch(labels: readonly string[]) {
-        const out: Record<string, { accountAddress: { toString(): string } }> = {};
-        for (const l of labels) {
-          _seq++;
-          const addr = "0x" + _seq.toString(16).padStart(64, "0");
-          out[l] = { accountAddress: { toString: () => addr } };
-        }
-        return out;
-      },
-      exportPrivateKeys(_labels: readonly string[]) {
-        return { deployer: "0x" + "1".repeat(64) };
-      },
-    },
-  };
-});
-
-// Imported after mocks so vi.hoisted ordering applies.
-import { setupLocalTesting } from "../setupLocalTesting.js";
-import { logger } from "../../ui/index.js";
-
-describe("F1 — setupLocalTesting honors forkNetwork", () => {
-  let cwdBackup: string;
-  let tmpRoot: string;
-
-  beforeEach(() => {
-    initializeCalls.length = 0;
-    cwdBackup = process.cwd();
-    tmpRoot = mkdtempSync(join(tmpdir(), "movehat-f1-"));
-    process.chdir(tmpRoot);
-    vi.spyOn(logger, "step").mockImplementation(() => undefined);
-    vi.spyOn(logger, "success").mockImplementation(() => undefined);
-    vi.spyOn(logger, "plain").mockImplementation(() => undefined);
-    vi.spyOn(logger, "newline").mockImplementation(() => undefined);
-    vi.spyOn(logger, "warning").mockImplementation(() => undefined);
-    vi.spyOn(logger, "error").mockImplementation(() => undefined);
-  });
-
-  afterEach(() => {
-    process.chdir(cwdBackup);
-    rmSync(tmpRoot, { recursive: true, force: true });
-    vi.restoreAllMocks();
-  });
-
-  it("uses the mainnet RPC when forkNetwork = 'mainnet'", async () => {
-    await setupLocalTesting({
-      mode: "fork",
-      forkNetwork: "mainnet",
-      accountLabels: ["deployer"],
-      autoFund: false,
-    });
-
-    expect(initializeCalls).toHaveLength(1);
-    const call = initializeCalls[0]!;
-    expect(call.networkName).toBe("mainnet");
-    expect(call.nodeUrl).not.toMatch(/testnet/i);
-    expect(call.nodeUrl).toMatch(/mainnet/i);
-  });
-
-  it("uses the testnet RPC when forkNetwork = 'testnet'", async () => {
-    await setupLocalTesting({
-      mode: "fork",
-      forkNetwork: "testnet",
-      accountLabels: ["deployer"],
-      autoFund: false,
-    });
-
-    expect(initializeCalls).toHaveLength(1);
-    const call = initializeCalls[0]!;
-    expect(call.networkName).toBe("testnet");
-    expect(call.nodeUrl).toMatch(/testnet/i);
-  });
-
-  it("uses forkRpcUrl override when supplied for a custom network", async () => {
-    await setupLocalTesting({
-      mode: "fork",
-      forkNetwork: "custom",
-      forkRpcUrl: "https://my-custom-node.example/v1",
-      accountLabels: ["deployer"],
-      autoFund: false,
-    });
-
-    expect(initializeCalls).toHaveLength(1);
-    const call = initializeCalls[0]!;
-    expect(call.networkName).toBe("custom");
-    expect(call.nodeUrl).toBe("https://my-custom-node.example/v1");
-  });
-
-  it("rejects a non-built-in forkNetwork when no forkRpcUrl is provided", async () => {
-    await expect(
-      setupLocalTesting({
-        mode: "fork",
-        forkNetwork: "some-unknown-network",
-        accountLabels: ["deployer"],
-        autoFund: false,
-      })
-    ).rejects.toThrow(/forkRpcUrl/i);
-    expect(initializeCalls).toHaveLength(0);
-  });
-
-  it("rejects when an existing fork's saved network does not match the requested one (audit-f1 follow-up)", async () => {
-    // Pre-seed `.movehat/forks/test-local/metadata.json` with the
-    // wrong network so the `forkExists` branch fires and loads stale
-    // metadata. Without the metadata-mismatch guard, setupLocalTesting
-    // would silently serve a testnet snapshot while the caller thinks
-    // it's reading mainnet.
-    const forkDir = join(tmpRoot, ".movehat", "forks", "test-local");
-    mkdirSync(forkDir, { recursive: true });
-    writeFileSync(
-      join(forkDir, "metadata.json"),
-      JSON.stringify({
-        network: "testnet",
-        nodeUrl: "https://testnet.movementnetwork.xyz/v1",
-        chainId: 250,
-        ledgerVersion: "0",
-        timestamp: "0",
-        epoch: "0",
-        blockHeight: "0",
-        createdAt: new Date().toISOString(),
-      }),
-    );
-
-    await expect(
-      setupLocalTesting({
-        mode: "fork",
-        forkNetwork: "mainnet",
-        accountLabels: ["deployer"],
-        autoFund: false,
-      })
-    ).rejects.toThrow(/network mismatch|created for|requested/i);
-    // Must not have re-initialized — the existing dir is what poisons
-    // the load path, and silently reinitializing would clobber the
-    // user's saved snapshot.
-    expect(initializeCalls).toHaveLength(0);
-  });
-});

package/src/helpers/assertions.ts

@@ -1,29 +0,0 @@
-import { type TransactionResult } from "../core/contract.js";
-
-/**
- * Assert that a transaction was successful
- */
-export function assertTransactionSuccess(result: TransactionResult): void {
-  if (!result.success) {
-    throw new Error(
-      `Transaction failed with status: ${result.vm_status}\nHash: ${result.hash}`
-    );
-  }
-}
-
-export function assertTransactionFailed(
-  result: TransactionResult,
-  expectedError?: string
-): void {
-  if (result.success) {
-    throw new Error(
-      `Transaction was expected to fail but succeeded.\nHash: ${result.hash}`
-    );
-  }
-
-  if(expectedError && !result.vm_status.includes(expectedError)) {
-    throw new Error(
-      `Transaction failed with unexpected error.\nExpected to include: ${expectedError}\nActual status: ${result.vm_status}\nHash: ${result.hash}`
-    );
-  }
-}

package/src/helpers/banner.ts

@@ -1,63 +0,0 @@
-import { applyGradient, gradients, shouldUseColor, colors } from '../ui/index.js';
-
-/**
- * MOVEHAT ASCII art banner lines
- */
-const bannerLines = [
-  " ███╗   ███╗ ██████╗ ██╗   ██╗███████╗██╗  ██╗ █████╗ ████████╗",
-  " ████╗ ████║██╔═══██╗██║   ██║██╔════╝██║  ██║██╔══██╗╚══██╔══╝",
-  " ██╔████╔██║██║   ██║██║   ██║█████╗  ███████║███████║   ██║   ",
-  " ██║╚██╔╝██║██║   ██║╚██╗ ██╔╝██╔══╝  ██╔══██║██╔══██║   ██║   ",
-  " ██║ ╚═╝ ██║╚██████╔╝ ╚████╔╝ ███████╗██║  ██║██║  ██║   ██║   ",
-  " ╚═╝     ╚═╝ ╚═════╝   ╚═══╝  ╚══════╝╚═╝  ╚═╝╚═╝  ╚═╝   ╚═╝   ",
-];
-
-/**
- * Divider line below the banner
- */
-const dividerLine = "━".repeat(68);
-
-/**
- * Tagline describing Movehat
- */
-const tagline = "  A powerful testing framework for Move smart contracts";
-
-/**
- * Render the Movehat banner with gradient colors
- * Uses the UI module's gradient system for consistency
- *
- * @returns Formatted banner string ready for console output
- *
- * @example
- * const banner = renderMovehatBanner();
- * console.log(banner);
- */
-export const renderMovehatBanner = (): string => {
-  if (!shouldUseColor()) {
-    return `${bannerLines.join("\n")}\n${dividerLine}\n${tagline}`;
-  }
-
-  // Apply Movehat brand gradient to each line with varying offsets
-  const coloredLines = bannerLines.map((line, idx) =>
-    applyGradient(line, gradients.movehat, idx * 2)
-  );
-
-  // Apply gradient to divider line
-  const coloredDivider = applyGradient(dividerLine, gradients.movehat, 0);
-
-  // Dim the tagline for subtle appearance
-  const coloredTagline = colors.dim(tagline);
-
-  return `${coloredLines.join("\n")}\n${coloredDivider}\n${coloredTagline}`;
-};
-
-/**
- * Print the Movehat banner to console
- * Convenience function that renders and logs the banner
- *
- * @example
- * printMovehatBanner();
- */
-export const printMovehatBanner = (): void => {
-  console.log(renderMovehatBanner());
-};

package/src/helpers/index.ts

@@ -1,37 +0,0 @@
-// Re-export all helpers
-export { setupTestEnvironment, createTestAccount } from "./setup.js";
-export type { TestEnvironment } from "./setup.js";
-export { MoveContract, getContract } from "../core/contract.js";
-export type { TransactionResult } from "../core/contract.js";
-export {
-  assertTransactionSuccess,
-  assertTransactionFailed,
-} from "./assertions.js";
-export {
-  saveDeployment,
-  loadDeployment,
-  getAllDeployments,
-  getDeployedAddress,
-} from "../core/deployments.js";
-export type { DeploymentInfo } from "../core/deployments.js";
-export {
-  snapshot,
-  getForkInfo,
-  viewForkResource,
-  compareForkState,
-  listSnapshots,
-} from "../fork/test.js";
-export type { SnapshotOptions, ForkInfo } from "../fork/test.js";
-export { AccountManager } from "../core/AccountManager.js";
-export type { StoredAccount } from "../core/AccountManager.js";
-export { LocalNodeManager } from "../node/LocalNodeManager.js";
-export type { LocalNodeOptions, LocalNodeInfo } from "../node/LocalNodeManager.js";
-export { setupLocalTesting } from "./setupLocalTesting.js";
-export type { LocalTestingContext } from "./setupLocalTesting.js";
-export {
-  setupTestFixture,
-  setupMinimalFixture,
-} from "./testFixtures.js";
-export type { TestFixture } from "./testFixtures.js";
-
-export type { MovehatConfig, LocalTestOptions } from "../types/config.js";

package/src/helpers/move-tests.ts

@@ -1,72 +0,0 @@
-import { existsSync } from "fs";
-import { resolve } from "path";
-import { loadUserConfig } from "../core/config.js";
-import { runCli } from "../utils/runCli.js";
-
-interface RunMoveTestsOptions {
-  filter?: string | undefined;
-  ignoreWarnings?: boolean | undefined;
-  skipIfMissing?: boolean | undefined; // If true, skip gracefully when Move dir missing (for orchestrated tests)
-}
-
-/**
- * Run Move unit tests using Movement CLI
- * @param options Test options including filter, warnings, and skip behavior
- * @returns Promise that resolves when tests complete successfully
- */
-export async function runMoveTests(options: RunMoveTestsOptions = {}): Promise<void> {
-  const userConfig = await loadUserConfig();
-  const moveDir = resolve(process.cwd(), userConfig.moveDir || "./move");
-
-  if (!existsSync(moveDir)) {
-    if (options.skipIfMissing) {
-      console.log("⊘ No Move directory found (./move not found)");
-      console.log("   Skipping Move tests...\n");
-      return;
-    } else {
-      throw new Error(
-        `Move directory not found: ${moveDir}\n` +
-        `   Update movehat.config.ts -> moveDir`
-      );
-    }
-  }
-
-  const args = ["move", "test", "--package-dir", moveDir];
-
-  // Add dev flag for auto-detected addresses
-  args.push("--dev");
-
-  if (options.filter) {
-    args.push("--filter", options.filter);
-  }
-
-  if (options.ignoreWarnings) {
-    args.push("--ignore-compile-warnings");
-  }
-
-  let result;
-  try {
-    result = await runCli(
-      {
-        command: "movement",
-        args,
-        cwd: process.cwd(),
-        inheritStdio: true,
-      },
-      { throwOnNonZeroExit: false }
-    );
-  } catch (error) {
-    // Spawn-time failure (ENOENT, etc.). The original code logged a
-    // Movement-CLI-install hint here; keep that.
-    console.error(`Failed to run Move tests: ${(error as Error).message}`);
-    console.error("   Make sure Movement CLI is installed");
-    throw error;
-  }
-
-  if (result.exitCode === 0) {
-    console.log("\n✓ Move tests passed");
-    return;
-  }
-
-  throw new Error("Move tests failed");
-}

package/src/helpers/npm-registry.ts

@@ -1,72 +0,0 @@
-export interface NpmRegistryResponse {
-  "dist-tags": {
-    latest: string;
-    [tag: string]: string;
-  };
-  versions: Record<string, unknown>;
-}
-
-export interface FetchOptions {
-  timeout?: number; // Timeout in milliseconds
-  throwOnError?: boolean; // If true, throw errors; if false, return null on error
-}
-
-/**
- * Fetch latest version from npm registry
- * @param packageName - The npm package name
- * @param options - Fetch options (timeout, error handling)
- * @returns Latest version string, or null if failed and throwOnError is false
- */
-export async function fetchLatestVersion(
-  packageName: string,
-  options: FetchOptions = {}
-): Promise<string | null> {
-  const { timeout = 0, throwOnError = false } = options;
-
-  try {
-    const controller = timeout > 0 ? new AbortController() : undefined;
-    const timeoutId = timeout > 0 && controller
-      ? setTimeout(() => controller.abort(), timeout)
-      : undefined;
-
-    const response = await fetch(
-      `https://registry.npmjs.org/${packageName}`,
-      controller ? { signal: controller.signal } : {}
-    );
-
-    if (timeoutId) clearTimeout(timeoutId);
-
-    if (!response.ok) {
-      const error = new Error(`Failed to fetch package info: ${response.statusText}`);
-      if (throwOnError) throw error;
-      return null;
-    }
-
-    const data = (await response.json()) as NpmRegistryResponse;
-
-    // Defensive validation of registry response structure
-    if (
-      !data ||
-      typeof data !== "object" ||
-      !data["dist-tags"] ||
-      typeof data["dist-tags"] !== "object" ||
-      typeof data["dist-tags"].latest !== "string"
-    ) {
-      const error = new Error(
-        `Invalid npm registry response: missing or malformed "dist-tags.latest" field`
-      );
-      if (throwOnError) throw error;
-      return null;
-    }
-
-    return data["dist-tags"].latest;
-  } catch (error) {
-    if (throwOnError) {
-      throw new Error(
-        `Failed to check for updates: ${error instanceof Error ? error.message : String(error)}`
-      );
-    }
-    // Silently fail - don't interrupt user's workflow
-    return null;
-  }
-}

package/src/helpers/semver-utils.ts

@@ -1,54 +0,0 @@
-/**
- * Compare two semver versions
- * Returns true if newVersion > currentVersion
- * Handles variable-length versions (1.2, 1.2.3, 1.2.3.4, etc.)
- */
-export function isNewerVersion(currentVersion: string, newVersion: string): boolean {
-  // Remove any pre-release tags (e.g., -alpha.0, -beta.1).
-  // String.split always returns ≥1 element; `?? ""` is a no-op narrowing.
-  const cleanCurrent = currentVersion.split("-")[0] ?? "";
-  const cleanNew = newVersion.split("-")[0] ?? "";
-
-  // Split and validate numeric parts
-  const currentParts = cleanCurrent.split(".").map((part) => {
-    const num = Number(part);
-    if (isNaN(num) || !part.trim()) {
-      throw new Error(`Invalid version format: ${currentVersion}`);
-    }
-    return num;
-  });
-
-  const newerParts = cleanNew.split(".").map((part) => {
-    const num = Number(part);
-    if (isNaN(num) || !part.trim()) {
-      throw new Error(`Invalid version format: ${newVersion}`);
-    }
-    return num;
-  });
-
-  // Compare up to the maximum length, treating missing parts as 0
-  const maxLength = Math.max(currentParts.length, newerParts.length);
-
-  for (let i = 0; i < maxLength; i++) {
-    const currentPart = currentParts[i] || 0;
-    const newerPart = newerParts[i] || 0;
-
-    if (newerPart > currentPart) return true;
-    if (newerPart < currentPart) return false;
-  }
-
-  // If base versions are equal, check pre-release tags
-  // A version with no pre-release tag is considered newer than one with a tag
-  const currentHasPrerelease = currentVersion.includes("-");
-  const newHasPrerelease = newVersion.includes("-");
-
-  if (!currentHasPrerelease && newHasPrerelease) {
-    return false; // Current stable is newer than new pre-release
-  }
-
-  if (currentHasPrerelease && !newHasPrerelease) {
-    return true; // New stable is newer than current pre-release
-  }
-
-  return false;
-}