fix(service): make Windows status locale independent

Author: luvs01

src/service.ts

@@ -17,6 +17,7 @@ import { isWslRuntime } from "./codex/home";
 import { durableBunPath, durableBunRuntime } from "./lib/bun-runtime";
 import { isProcessAlive, stopProxy } from "./lib/process-control";
 import { serviceApiTokenFilePath } from "./lib/service-secrets";
+import { findLiveProxy } from "./server/proxy-liveness";
 import { randomUUID } from "node:crypto";
 import {
   ELEVATION_REQUEST_TIMEOUT_MS,
@@ -394,6 +395,70 @@ export type WindowsSchedulerTaskProbe =
   | { status: "absent" }
   | { status: "unknown"; detail: string };
 
+export type WindowsSchedulerProxyProbe =
+  | { status: "running"; port: number }
+  | { status: "not-running" }
+  | { status: "unknown" };
+
+/**
+ * Render Task Scheduler status without exposing localized `schtasks` table output.
+ * The task probe answers installation state; the identity-checked health probe answers
+ * runtime state. Keep probe details out of this user-facing line because they can contain
+ * incorrectly decoded, locale-specific command output.
+ */
+export function formatWindowsSchedulerServiceStatus(
+  task: WindowsSchedulerTaskProbe,
+  proxy: WindowsSchedulerProxyProbe,
+): string {
+  if (task.status === "present") {
+    if (proxy.status === "running") {
+      return `✅ service installed (Task Scheduler); OpenCodex proxy running on port ${proxy.port}.`;
+    }
+    if (proxy.status === "not-running") {
+      return "⚠️  service installed (Task Scheduler); OpenCodex proxy not running.";
+    }
+    return "⚠️  service installed (Task Scheduler); OpenCodex proxy status unknown.";
+  }
+  if (task.status === "absent") {
+    if (proxy.status === "running") {
+      return `❌ service not installed (Task Scheduler); OpenCodex proxy is running independently on port ${proxy.port}.`;
+    }
+    if (proxy.status === "unknown") {
+      return "❌ service not installed (Task Scheduler); OpenCodex proxy status unknown.";
+    }
+    return "❌ service not installed (Task Scheduler).";
+  }
+  if (proxy.status === "running") {
+    return `⚠️  Task Scheduler registration unknown; OpenCodex proxy running on port ${proxy.port}.`;
+  }
+  if (proxy.status === "not-running") {
+    return "⚠️  service status unknown (Task Scheduler query failed); OpenCodex proxy not running.";
+  }
+  return "⚠️  service status unknown (Task Scheduler and proxy checks failed).";
+}
+
+export async function inspectWindowsSchedulerServiceStatus(io: {
+  probeTask?: () => WindowsSchedulerTaskProbe;
+  findProxy?: () => Promise<{ port: number } | null>;
+} = {}): Promise<string> {
+  let task: WindowsSchedulerTaskProbe;
+  try {
+    task = (io.probeTask ?? probeWindowsSchedulerTask)();
+  } catch (error) {
+    task = { status: "unknown", detail: schtasksErrorDetail(error) };
+  }
+
+  let proxy: WindowsSchedulerProxyProbe;
+  try {
+    const live = await (io.findProxy ?? findLiveProxy)();
+    proxy = live ? { status: "running", port: live.port } : { status: "not-running" };
+  } catch {
+    proxy = { status: "unknown" };
+  }
+
+  return formatWindowsSchedulerServiceStatus(task, proxy);
+}
+
 function schtasksErrorDetail(error: unknown): string {
   return error instanceof Error ? error.message : String(error);
 }
@@ -1756,8 +1821,12 @@ export async function serviceCommand(...args: (string | undefined)[]): Promise<v
       }
       break;
     case "status": {
-      const s = ops.status();
-      console.log(s ? `✅ running:\n${s}` : "❌ service not installed/running.");
+      if (process.platform === "win32" && backend === "scheduler") {
+        console.log(await inspectWindowsSchedulerServiceStatus());
+      } else {
+        const s = ops.status();
+        console.log(s ? `✅ running:\n${s}` : "❌ service not installed/running.");
+      }
       console.log(`Diagnostics: ${serviceDiagnosticsSummary()}`);
       break;
     }

tests/windows-scheduler-install-verification.test.ts

@@ -2,6 +2,8 @@ import { afterEach, describe, expect, test } from "bun:test";
 import {
   buildWindowsTaskXml,
   evaluateWindowsSchedulerInstallVerification,
+  formatWindowsSchedulerServiceStatus,
+  inspectWindowsSchedulerServiceStatus,
   probeWindowsSchedulerTask,
   setQuerySchtasksForTests,
   windowsSchedulerCsvIncludesTask,
@@ -44,6 +46,26 @@ describe("probeWindowsSchedulerTask", () => {
     expect(windowsSchedulerTaskInstalled("opencodex-proxy")).toBe(true);
   });
 
+  test("recognizes the task without exposing mojibake from localized table output", () => {
+    Object.defineProperty(process, "platform", { configurable: true, value: "win32" });
+    const localizedTable = [
+      "����: \\",
+      "�۾� �̸�                                ���� ���� �ð�         ����",
+      "======================================== ====================== ===============",
+      "opencodex-proxy                          N/A                    �غ�",
+    ].join("\n");
+    setQuerySchtasksForTests(() => localizedTable);
+
+    const result = formatWindowsSchedulerServiceStatus(
+      probeWindowsSchedulerTask("opencodex-proxy"),
+      { status: "running", port: 10100 },
+    );
+
+    expect(result).toBe("✅ service installed (Task Scheduler); OpenCodex proxy running on port 10100.");
+    expect(result).not.toContain("����");
+    expect(result).not.toContain("�۾�");
+  });
+
   test("falls back to CSV listing when the specific query fails", () => {
     Object.defineProperty(process, "platform", { configurable: true, value: "win32" });
     setQuerySchtasksForTests((args) => {
@@ -83,6 +105,34 @@ describe("probeWindowsSchedulerTask", () => {
   });
 });
 
+describe("formatWindowsSchedulerServiceStatus", () => {
+  test("reports task and identity-checked proxy state independently", () => {
+    expect(formatWindowsSchedulerServiceStatus({ status: "present" }, { status: "not-running" }))
+      .toBe("⚠️  service installed (Task Scheduler); OpenCodex proxy not running.");
+    expect(formatWindowsSchedulerServiceStatus({ status: "present" }, { status: "unknown" }))
+      .toBe("⚠️  service installed (Task Scheduler); OpenCodex proxy status unknown.");
+    expect(formatWindowsSchedulerServiceStatus({ status: "absent" }, { status: "running", port: 3593 }))
+      .toBe("❌ service not installed (Task Scheduler); OpenCodex proxy is running independently on port 3593.");
+    expect(formatWindowsSchedulerServiceStatus({ status: "absent" }, { status: "not-running" }))
+      .toBe("❌ service not installed (Task Scheduler).");
+    expect(formatWindowsSchedulerServiceStatus({ status: "unknown", detail: "����" }, { status: "running", port: 10100 }))
+      .toBe("⚠️  Task Scheduler registration unknown; OpenCodex proxy running on port 10100.");
+    expect(formatWindowsSchedulerServiceStatus({ status: "unknown", detail: "����" }, { status: "not-running" }))
+      .toBe("⚠️  service status unknown (Task Scheduler query failed); OpenCodex proxy not running.");
+  });
+
+  test("keeps scheduler and runtime probe failures locale-independent", async () => {
+    const status = await inspectWindowsSchedulerServiceStatus({
+      probeTask: () => { throw new Error("���� ����"); },
+      findProxy: async () => { throw new Error("connection failure"); },
+    });
+
+    expect(status).toBe("⚠️  service status unknown (Task Scheduler and proxy checks failed).");
+    expect(status).not.toContain("����");
+    expect(status).not.toContain("connection failure");
+  });
+});
+
 describe("evaluateWindowsSchedulerInstallVerification", () => {
   const wscript = "C:\\Windows\\System32\\wscript.exe";
   const launcher = "C:\\Users\\Test\\.opencodex\\opencodex-service-launcher.vbs";