From d02001e14423c3646bbea61d2c094d8ba7369003 Mon Sep 17 00:00:00 2001
From: gavrielc <gabicohen22@yahoo.com>
Date: Tue, 21 Apr 2026 23:46:49 +0300
Subject: [PATCH] feat(setup): prompt for display name, hardcode agent persona
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Before the cli-agent step, ask the operator what the agent should
call them (defaults to \$USER). The agent's own persona name is
hardcoded to "Terminal Agent" — this is the scratch CLI agent, not
one of the operator's real personas. NANOCLAW_DISPLAY_NAME still
skips the prompt.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 setup/auto.ts | 39 ++++++++++++++++++++++++++++-----------
 1 file changed, 28 insertions(+), 11 deletions(-)

diff --git a/setup/auto.ts b/setup/auto.ts
index 3945d82..9a37a45 100644
--- a/setup/auto.ts
+++ b/setup/auto.ts
@@ -7,8 +7,9 @@
  * module check). This driver picks up from there.
  *
  * Config via env:
- *   NANOCLAW_DISPLAY_NAME  operator name for the CLI agent (default: $USER)
- *   NANOCLAW_AGENT_NAME    agent persona name (default: display name)
+ *   NANOCLAW_DISPLAY_NAME  operator name for the CLI agent — skips the
+ *                          interactive prompt before cli-agent. If unset,
+ *                          the driver asks, defaulting to $USER.
  *   NANOCLAW_SKIP          comma-separated step names to skip
  *                          (environment|container|onecli|auth|
  *                           mounts|service|cli-agent|verify)
@@ -24,6 +25,9 @@
  * and `/manage-channels` after this driver completes.
  */
 import { spawn, spawnSync } from 'child_process';
+import { createInterface } from 'readline/promises';
+
+const CLI_AGENT_NAME = 'Terminal Agent';
 
 type Fields = Record<string, string>;
 type StepResult = { ok: boolean; fields: Fields; exitCode: number };
@@ -114,6 +118,18 @@ function anthropicSecretExists(): boolean {
   }
 }
 
+async function askDisplayName(fallback: string): Promise<string> {
+  const rl = createInterface({ input: process.stdin, output: process.stdout });
+  try {
+    const answer = await rl.question(
+      `\nWhat should the agent call you? [${fallback}]: `,
+    );
+    return answer.trim() || fallback;
+  } finally {
+    rl.close();
+  }
+}
+
 function runBashScript(relPath: string): Promise<number> {
   return new Promise((resolve) => {
     const child = spawn('bash', [relPath], { stdio: 'inherit' });
@@ -223,19 +239,20 @@ async function main(): Promise<void> {
   }
 
   if (!skip.has('cli-agent')) {
-    const displayName =
-      process.env.NANOCLAW_DISPLAY_NAME?.trim() ||
-      process.env.USER?.trim() ||
-      'Operator';
-    const agentName = process.env.NANOCLAW_AGENT_NAME?.trim();
-    const args = ['--display-name', displayName];
-    if (agentName) args.push('--agent-name', agentName);
+    const fallback = process.env.USER?.trim() || 'Operator';
+    const preset = process.env.NANOCLAW_DISPLAY_NAME?.trim();
+    const displayName = preset || (await askDisplayName(fallback));
 
-    const res = await runStep('cli-agent', args);
+    const res = await runStep('cli-agent', [
+      '--display-name',
+      displayName,
+      '--agent-name',
+      CLI_AGENT_NAME,
+    ]);
     if (!res.ok) {
       fail(
         'CLI agent wiring failed',
-        'Re-run `pnpm exec tsx scripts/init-cli-agent.ts --display-name "<your name>"` to fix.',
+        `Re-run \`pnpm exec tsx scripts/init-cli-agent.ts --display-name "${displayName}" --agent-name "${CLI_AGENT_NAME}"\` to fix.`,
       );
     }
   }