macroclaw 0.13.0 → 0.15.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "macroclaw",
-  "version": "0.13.0",
+  "version": "0.15.0",
   "description": "Telegram-to-Claude-Code bridge",
   "license": "MIT",
   "type": "module",

package/src/cli.test.ts

@@ -1,9 +1,9 @@
 import { describe, expect, it, mock } from "bun:test";
 import { runCommand } from "citty";
 import { Cli, createReadlineIo, handleError } from "./cli";
-import type { SystemService } from "./service";
 import { SettingsManager } from "./settings";
 import type { SetupWizard } from "./setup";
+import type { SystemServiceManager } from "./system-service";
 
 // Only mock ./index — safe since no other test imports it
 const mockStart = mock(async () => {});
@@ -109,17 +109,18 @@ describe("Cli.setup", () => {
 	});
 });
 
-function mockService(overrides?: Partial<SystemService>): SystemService {
+function mockService(overrides?: Record<string, unknown>): SystemServiceManager {
 	return {
 		install: mock(() => ""),
 		uninstall: mock(() => {}),
 		start: mock(() => ""),
 		stop: mock(() => {}),
-		update: mock(() => ""),
+		update: mock(() => ({ previousVersion: "0.6.0", currentVersion: "0.7.0" })),
+		isRunning: false,
 		status: mock(() => ({ installed: false, running: false, platform: "systemd" as const })),
 		logs: mock(() => "journalctl -u macroclaw -n 50 --no-pager"),
 		...overrides,
-	};
+	} as unknown as SystemServiceManager;
 }
 
 describe("Cli.service", () => {
@@ -151,11 +152,26 @@ describe("Cli.service", () => {
 		expect(stop).toHaveBeenCalled();
 	});
 
-	it("runs update action", () => {
-		const update = mock(() => "tail -f /logs");
-		const cli = new Cli({ systemService: mockService({ update }) });
+	it("runs update action — stops and starts when running", () => {
+		const stop = mock(() => {});
+		const start = mock(() => "tail -f /logs");
+		const update = mock(() => ({ previousVersion: "0.6.0", currentVersion: "0.7.0" }));
+		const cli = new Cli({ systemService: mockService({ stop, start, update, isRunning: true }) });
 		cli.service("update");
+		expect(stop).toHaveBeenCalled();
 		expect(update).toHaveBeenCalled();
+		expect(start).toHaveBeenCalled();
+	});
+
+	it("runs update action — skips stop but still starts when not running", () => {
+		const stop = mock(() => {});
+		const start = mock(() => "tail -f /logs");
+		const update = mock(() => ({ previousVersion: "0.6.0", currentVersion: "0.7.0" }));
+		const cli = new Cli({ systemService: mockService({ stop, start, update, isRunning: false }) });
+		cli.service("update");
+		expect(stop).not.toHaveBeenCalled();
+		expect(update).toHaveBeenCalled();
+		expect(start).toHaveBeenCalled();
 	});
 
 	it("runs status action", () => {

package/src/cli.ts

@@ -2,20 +2,20 @@ import {execSync} from "node:child_process";
 import {createInterface} from "node:readline";
 import {defineCommand} from "citty";
 import pkg from "../package.json" with {type: "json"};
-import {ServiceManager, type SystemService} from "./service";
 import {loadSessions} from "./sessions";
 import {SettingsManager} from "./settings";
 import {type SetupIo, SetupWizard} from "./setup";
+import {SystemServiceManager} from "./system-service";
 
 export class Cli {
 	readonly #settingsManager: SettingsManager;
 	readonly #setupWizard: SetupWizard;
-	readonly #serviceManager: SystemService;
+	readonly #serviceManager: SystemServiceManager;
 
-	constructor(opts?: { wizard?: SetupWizard; settings?: SettingsManager; systemService?: SystemService }) {
+	constructor(opts?: { wizard?: SetupWizard; settings?: SettingsManager; systemService?: SystemServiceManager }) {
 		this.#settingsManager = opts?.settings ?? new SettingsManager();
 		this.#setupWizard = opts?.wizard ?? new SetupWizard(createReadlineIo());
-		this.#serviceManager = opts?.systemService ?? new ServiceManager();
+		this.#serviceManager = opts?.systemService ?? new SystemServiceManager();
 	}
 
 	async setup(): Promise<void> {
@@ -55,8 +55,19 @@ export class Cli {
 				console.log("Service stopped.");
 				break;
 			case "update": {
-				const logCmd = this.#serviceManager.update();
-				console.log(`Service updated. Check logs:\n  ${logCmd}`);
+				if (this.#serviceManager.isRunning) {
+					this.#serviceManager.stop();
+					console.log("Service stopped.");
+				}
+				const result = this.#serviceManager.update();
+				if (result.previousVersion === result.currentVersion) {
+					console.log(`macroclaw v${result.currentVersion} (already up to date)`);
+				} else {
+					console.log(`Updated macroclaw v${result.previousVersion} → v${result.currentVersion}`);
+				}
+
+				const logCmd = this.#serviceManager.start();
+				console.log(`Service started. Check logs:\n  ${logCmd}`);
 				break;
 			}
 			case "status": {

package/src/setup.ts

@@ -128,7 +128,7 @@ export class SetupWizard {
       }
 
       try {
-        const svc = this.#serviceInstaller ?? new (await import("./service")).ServiceManager();
+        const svc = this.#serviceInstaller ?? new (await import("./system-service")).SystemServiceManager();
         const logCmd = svc.install(oauthToken);
         this.#io.write(`Service installed and started. Check logs:\n  ${logCmd}\n`);
       } catch (err) {