fix: version comparison logic (#50)

* fix: version comparison logic

* use existing method

Author: itszeeshan

src/releases.ts

@@ -1,4 +1,5 @@
 import { window } from "vscode";
+import semver from "semver";
 import { logger } from "./logger";
 import { getConfig } from "./config";
 import { state } from "./state";
@@ -80,24 +81,25 @@ export class Releases {
       return false;
     }
 
-    const prereleaseEnabled = this.prereleaseEnabled();
+    const filteredReleases = this.all();
 
-    const idx = this.releases
-      .filter((r) => prereleaseEnabled || !r.prerelease)
-      .findIndex((r) => r.tag_name === tag_name);
-
-    if (idx === -1) {
-      return true;
+    if (filteredReleases.length === 0) {
+      return false;
     }
 
-    if (idx >= 3) {
-      return true;
-    }
+    // Get the latest version (first in the sorted array)
+    const latestVersion = filteredReleases[0].tag_name;
+
+    // Use semantic version comparison to check if current version is outdated
+    // A version is considered outdated if it's less than the latest version
+    return semver.lt(tag_name, latestVersion);
   }
 
   latestVersion(): string | null {
-    if (this.releases.length > 0) {
-      return this.releases[0].tag_name;
+    const filteredReleases = this.all();
+    
+    if (filteredReleases.length > 0) {
+      return filteredReleases[0].tag_name;
     } else {
       return null;
     }
@@ -165,12 +167,17 @@ async function fromGithub(): Promise<Release[]> {
 
   const filteredAndSorted = releases
     .filter((r) => !r.draft)
-    .sort(
-      (a, b) =>
-        new Date(b.published_at).getTime() - new Date(a.published_at).getTime()
-    );
+    .sort((a, b) => {
+      // Sort by semantic version (highest first), then by publication date as fallback
+      const versionComparison = semver.rcompare(a.tag_name, b.tag_name);
+      if (versionComparison !== 0) {
+        return versionComparison;
+      }
+      // If versions are equal, sort by publication date (newest first)
+      return new Date(b.published_at).getTime() - new Date(a.published_at).getTime();
+    });
 
   await setCache(filteredAndSorted);
 
   return filteredAndSorted;
-}
+}

src/session.ts

@@ -87,7 +87,7 @@ export const createSession = async (
     Date.now() - new Date(lastNotifiedOfUpdate).getTime() > daysToMs(3)
   ) {
     window.showInformationMessage(
-      `PostgresTools ${version} is outdated, consider updating to ${state.releases.latestVersion()}.`
+      `A new version of PostgresTools is available! your version ${version} is outdated, consider updating to latest version ${state.releases.latestVersion()}.`
     );
     await state.context.globalState.update(
       "lastNotifiedOfUpdate",
@@ -100,7 +100,7 @@ export const createSession = async (
     !semver.gte(version, "0.8.0")
   ) {
     window.showInformationMessage(
-      `PostgresTools ${version} does not support multi root workspaces. Consider upgrading to >= 0.8.0.`
+      `Your current PostgresTools version ${version} does not support multi-root workspaces. Consider upgrading to version >= 0.8.0.`
     );
   }