feat: enhance service management by reconciling exited PTY sessions and improve terminal session handling

Author: DrSmoothl

resources/installer.nsh

@@ -1,18 +1,12 @@
+!ifndef BUILD_UNINSTALLER
 !macro customPageAfterChangeDir
-  !define MUI_PAGE_CUSTOMFUNCTION_SHOW maibotShowInstallDetails
+  ShowInstDetails hide
 !macroend
+!endif
 
 !macro customHeader
-  ShowInstDetails show
+  ShowInstDetails hide
   !ifdef BUILD_UNINSTALLER
-    ShowUninstDetails show
+    ShowUninstDetails hide
   !endif
 !macroend
-
-!macro customInstall
-  SetDetailsView show
-!macroend
-
-Function maibotShowInstallDetails
-  SetDetailsView show
-FunctionEnd

src/main/services/service-manager.ts

@@ -629,6 +629,7 @@ export class ServiceManager extends EventEmitter {
 
   async refresh(): Promise<ServiceDescriptor[]> {
     this.attachLivePtySessions();
+    this.reconcileExitedPtySessions();
 
     for (const definition of this.definitions) {
       const state = this.getState(definition.id);
@@ -961,6 +962,36 @@ export class ServiceManager extends EventEmitter {
     }
   }
 
+  private reconcileExitedPtySessions(): void {
+    const sessions = new Map(this.pty.list().map((session) => [session.id, session]));
+
+    for (const definition of this.definitions) {
+      const state = this.getState(definition.id);
+      if (!state.ptySessionId || state.status === "stopped" || state.status === "error") {
+        continue;
+      }
+
+      const session = sessions.get(state.ptySessionId);
+      if (!session || session.status === "starting" || session.status === "running" || session.status === "stopping") {
+        continue;
+      }
+
+      const stoppedByRequest = state.status === "stopping" || !state.desired;
+      this.setState(definition.id, {
+        ...state,
+        status: stoppedByRequest ? "stopped" : "error",
+        health: "unknown",
+        managed: false,
+        desired: stoppedByRequest ? false : state.desired,
+        pid: undefined,
+        ptySessionId: undefined,
+        error: stoppedByRequest ? undefined : (session.error ?? `进程异常退出: ${session.exitCode ?? "未知"}`),
+        detail: stoppedByRequest ? "已停止" : `进程异常退出: ${session.exitCode ?? "未知"}`,
+        stoppedAt: session.endedAt ?? Date.now(),
+      });
+    }
+  }
+
   private handlePtyData(event: PtyDataEvent): void {
     const serviceId = serviceIdFromSession(event.sessionId);
     if (!serviceId) {
@@ -1006,6 +1037,7 @@ export class ServiceManager extends EventEmitter {
       health: "unknown",
       managed: false,
       pid: undefined,
+      ptySessionId: undefined,
       detail: stoppedByRequest
         ? "已停止"
         : shouldRestart
@@ -1051,6 +1083,23 @@ export class ServiceManager extends EventEmitter {
       return;
     }
 
+    if (snapshot.status === "exited" || snapshot.status === "error") {
+      const stoppedByRequest = state.status === "stopping" || !state.desired;
+      this.setState(serviceId, {
+        ...state,
+        status: stoppedByRequest ? "stopped" : "error",
+        health: "unknown",
+        managed: false,
+        desired: stoppedByRequest ? false : state.desired,
+        pid: undefined,
+        ptySessionId: undefined,
+        error: stoppedByRequest ? undefined : (snapshot.error ?? `进程异常退出: ${snapshot.exitCode ?? "未知"}`),
+        detail: stoppedByRequest ? "已停止" : `进程异常退出: ${snapshot.exitCode ?? "未知"}`,
+        stoppedAt: snapshot.endedAt ?? Date.now(),
+      });
+      return;
+    }
+
     this.setState(serviceId, {
       ...state,
       pid: snapshot.pid,

src/renderer/src/components/app/TerminalPanel.tsx

@@ -54,6 +54,10 @@ function writeSystemLine(terminal: Terminal, message: string): void {
   terminal.writeln(`\x1b[38;2;155;213;108m[desktop]\x1b[0m ${message}`);
 }
 
+function canWriteToSession(session?: PtySessionSnapshot): boolean {
+  return session?.status === "running";
+}
+
 async function copyTerminalSelection(terminal: Terminal): Promise<void> {
   const selection = terminal.getSelection();
   if (!selection) {
@@ -156,6 +160,10 @@ export function TerminalPanel({
 
       const disposables: Disposable[] = [
         terminal.onData((data) => {
+          if (!canWriteToSession(sessionsRef.current.get(sessionId))) {
+            return;
+          }
+
           void bridgeRef.current?.pty.input({ sessionId, data }).catch((error: unknown) => {
             writeSystemLine(terminal, error instanceof Error ? error.message : String(error));
           });
@@ -173,6 +181,10 @@ export function TerminalPanel({
           return false;
         }
 
+        if (event.type === "keydown" && !canWriteToSession(sessionsRef.current.get(sessionId))) {
+          return false;
+        }
+
         return true;
       });