fix(cli): keep /rc in the terminal UI when no relay key is set

Author: elkaix

apps/pythinker-code/src/tui/commands/web.ts

@@ -61,14 +61,23 @@ export async function handleRemoteControlCommand(host: SlashCommandHost): Promis
     return;
   }
 
+  // Before the takeover: a missing relay key is a configuration problem the
+  // user can still fix, so it must not cost them the terminal UI.
+  let relayKey: string;
+  try {
+    relayKey = resolveRelayKey();
+  } catch (error) {
+    host.showError(formatErrorMessage(error));
+    return;
+  }
+
   host.setExitForegroundTask(async () => {
     const options = parseServerOptions({});
     let remoteControl: Awaited<ReturnType<typeof startRemoteControl>> | undefined;
     try {
       // Inside the try: a malformed relay setting throws here, and the user
       // should see it through the same handler as any other startup failure.
       const relayOrigin = resolveRelayOrigin();
-      const relayKey = resolveRelayKey();
       await startServerForeground(options, {
         onReady: async (origin) => {
           const dataDir = getDataDir();

apps/pythinker-code/test/tui/commands/web.test.ts

@@ -147,6 +147,21 @@ describe('handleWebCommand', () => {
 });
 
 describe('handleRemoteControlCommand', () => {
+  beforeEach(() => {
+    vi.stubEnv('PYTHINKER_CODE_REMOTE_CONTROL_RELAY_KEY', 'relay-key-1');
+  });
+
+  it('stays in the TUI with a readable error when no relay key is configured', async () => {
+    vi.clearAllMocks();
+    vi.stubEnv('PYTHINKER_CODE_REMOTE_CONTROL_RELAY_KEY', '');
+    const host = makeHost();
+
+    await handleRemoteControlCommand(host);
+
+    expect(host.showError).toHaveBeenCalledWith(expect.stringContaining('relay key'));
+    expect(host.setExitForegroundTask).not.toHaveBeenCalled();
+  });
+
   it('stays in the TUI with a readable error when another instance holds Remote Control', async () => {
     vi.clearAllMocks();
     const { mkdtempSync, mkdirSync, rmSync, writeFileSync } = await import('node:fs');