fix: skip pty helper strict failures on sdk backend

Author: MackDing

src/ops/healthcheck.ts

@@ -39,6 +39,7 @@ interface HealthcheckOptions {
   telegramLiveCheck?: boolean;
   codexLiveCheck?: boolean;
   codexLiveRunner?: (config: AppConfig) => Promise<CodexLiveCheckResult>;
+  ptyHelperCheck?: typeof repairNodePtySpawnHelperPermissions;
 }
 
 interface TelegramGetMeResponse {
@@ -258,11 +259,23 @@ export async function runHealthcheck(
         )
   );
 
-  const ptyHelper = repairNodePtySpawnHelperPermissions();
+  const ptyHelperCheck =
+    options.ptyHelperCheck || repairNodePtySpawnHelperPermissions;
+  const ptyHelper = ptyHelperCheck();
   if (ptyHelper.error) {
-    checks.push(
-      makeCheck("node-pty helper", strict ? "fail" : "warn", ptyHelper.error)
-    );
+    if (config.runner.backend === "sdk") {
+      checks.push(
+        makeCheck(
+          "node-pty helper",
+          "pass",
+          `Not required for sdk backend (${ptyHelper.error})`
+        )
+      );
+    } else {
+      checks.push(
+        makeCheck("node-pty helper", strict ? "fail" : "warn", ptyHelper.error)
+      );
+    }
   } else if (ptyHelper.changed) {
     checks.push(
       makeCheck(

tests/healthcheck.test.ts

@@ -126,6 +126,30 @@ test("runHealthcheck fails when the configured command is missing in strict mode
   );
 });
 
+test("runHealthcheck skips node-pty helper failures in strict mode when backend is sdk", async () => {
+  const root = fs.mkdtempSync(path.join(os.tmpdir(), "claws-health-"));
+  const config = createConfig(root);
+
+  const result = await runHealthcheck(config, {
+    env: process.env,
+    strict: true,
+    ptyHelperCheck: () => ({
+      path: "",
+      changed: false,
+      executable: false,
+      error: "simulated missing spawn-helper"
+    })
+  });
+
+  assert.equal(result.ok, true);
+  assert.equal(
+    result.checks.some(
+      (check) => check.name === "node-pty helper" && check.status === "pass"
+    ),
+    true
+  );
+});
+
 test("runHealthcheck reports a passing live Codex probe when the backend responds", async () => {
   const root = fs.mkdtempSync(path.join(os.tmpdir(), "claws-health-"));
   const config = createConfig(root);