refactor: Extract version calculation to a separate utility

Author: taj54

src/updaters/dockerUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class DockerUpdater implements Updater {
   platform = 'docker';
@@ -19,7 +20,7 @@ export class DockerUpdater implements Updater {
     const current = this.getCurrentVersion();
     if (!current) throw new Error('Docker version not found');
 
-    const newVersion = semver.inc(current, releaseType) || current;
+    const newVersion = calculateNextVersion(current, releaseType);
     let content = fs.readFileSync('Dockerfile', 'utf8');
     content = content.replace(/LABEL version="[^"]+"/, `LABEL version="${newVersion}"`);
     fs.writeFileSync('Dockerfile', content);

src/updaters/goUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class GoUpdater implements Updater {
   platform = 'go';
@@ -20,7 +21,7 @@ export class GoUpdater implements Updater {
     const current = this.getCurrentVersion();
     if (!current) throw new Error('Go version not found');
 
-    const newVersion = semver.inc(current, releaseType) || current;
+    const newVersion = calculateNextVersion(current, releaseType);
     let content = fs.readFileSync('go.mod', 'utf8');
     content = content.replace(/v\d+\.\d+\.\d+/, `v${newVersion}`);
     fs.writeFileSync('go.mod', content);

src/updaters/nodeUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class NodeUpdater implements Updater {
   platform = 'node';
@@ -17,7 +18,7 @@ export class NodeUpdater implements Updater {
 
   bumpVersion(releaseType: ReleaseType): string {
     const pkg = JSON.parse(fs.readFileSync('package.json', 'utf8'));
-    const newVersion = semver.inc(pkg.version, releaseType) || pkg.version;
+    const newVersion = calculateNextVersion(pkg.version, releaseType);
     pkg.version = newVersion;
     fs.writeFileSync('package.json', JSON.stringify(pkg, null, 2));
     return newVersion;

src/updaters/phpUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class PHPUpdater implements Updater {
   platform = 'php';
@@ -45,7 +46,7 @@ export class PHPUpdater implements Updater {
 
   bumpVersion(releaseType: ReleaseType): string {
     const version = this.getCurrentVersion() || '0.1.0';
-    const newVersion = semver.inc(version, releaseType) || version;
+    const newVersion = calculateNextVersion(version, releaseType);
 
     // composer.json
     if (fs.existsSync('composer.json')) {
@@ -64,7 +65,7 @@ export class PHPUpdater implements Updater {
     // version.php
     if (fs.existsSync('version.php')) {
       const content = fs.readFileSync('version.php', 'utf8');
-      const updated = content.replace(/(['"])[\d.]+(['"])/, `'${newVersion}'`);
+      const updated = content.replace(/(['"])[\d.]+(['"])/, `$1${newVersion}$2`);
       fs.writeFileSync('version.php', updated);
       return newVersion;
     }

src/updaters/pythonUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class PythonUpdater implements Updater {
   platform = 'python';
@@ -27,7 +28,7 @@ export class PythonUpdater implements Updater {
     const current = this.getCurrentVersion();
     if (!current) throw new Error('Python version not found');
 
-    const newVersion = semver.inc(current, releaseType) || current;
+    const newVersion = calculateNextVersion(current, releaseType);
 
     if (fs.existsSync('pyproject.toml')) {
       let content = fs.readFileSync('pyproject.toml', 'utf8');

src/updaters/rustUpdater.ts

@@ -1,6 +1,7 @@
 import fs from 'fs';
-import semver, { ReleaseType } from 'semver';
+import { ReleaseType } from 'semver';
 import { Updater } from '../interface';
+import { calculateNextVersion } from '../utils';
 
 export class RustUpdater implements Updater {
   platform = 'rust';
@@ -20,7 +21,7 @@ export class RustUpdater implements Updater {
     const current = this.getCurrentVersion();
     if (!current) throw new Error('Rust version not found');
 
-    const newVersion = semver.inc(current, releaseType) || current;
+    const newVersion = calculateNextVersion(current, releaseType);
     let content = fs.readFileSync('Cargo.toml', 'utf8');
     content = content.replace(/version\s*=\s*"[^"]+"/, `version = "${newVersion}"`);
     fs.writeFileSync('Cargo.toml', content);

src/utils/index.ts

@@ -1 +1,2 @@
 export * from './git';
+export * from './versionUtil';

src/utils/versionUtil.ts

@@ -0,0 +1,9 @@
+import semver, { ReleaseType } from 'semver';
+
+export function calculateNextVersion(currentVersion: string, releaseType: ReleaseType): string {
+  const newVersion = semver.inc(currentVersion, releaseType);
+  if (!newVersion) {
+    return currentVersion;
+  }
+  return newVersion;
+}