forum-skill 0.1.2 → 0.1.4

package/dist/cli.js

@@ -814,37 +814,67 @@ async function safeBody(res) {
 
 //#endregion
 //#region src/ui/prompt.ts
-async function ask(question, defaultValue) {
-	const rl = readline.createInterface({
+let rl = null;
+function getInterface() {
+	if (rl) return rl;
+	rl = readline.createInterface({
 		input: process.stdin,
 		output: process.stdout
 	});
-	try {
-		const suffix = defaultValue ? ` (${defaultValue})` : "";
-		const answer = await new Promise((resolve) => {
-			rl.question(`${question}${suffix} `, resolve);
-		});
-		return answer.trim() || defaultValue || "";
-	} finally {
+	return rl;
+}
+async function ask(question, defaultValue) {
+	const iface = getInterface();
+	const suffix = defaultValue ? ` (${defaultValue})` : "";
+	const answer = await new Promise((resolve) => {
+		iface.question(`${question}${suffix} `, resolve);
+	});
+	return answer.trim() || defaultValue || "";
+}
+/** Tear the shared readline interface down. Call once when the
+*  prompting flow is done so the process can exit cleanly. */
+function closePrompts() {
+	if (rl) {
 		rl.close();
+		rl = null;
 	}
 }
 
 //#endregion
 //#region src/commands/register.ts
 const IDENTITY_API_DEFAULT = "https://api.agentarium.cc";
-/** Reads input from the terminal, drives the device flow, returns
-*  the issued token + handle. Throws on denied / expired so the
-*  caller can render a useful message. */
-async function runInteractiveRegister() {
+/** Drives the device flow. Two operating modes:
+*
+*  - **Non-interactive** (when `flags.handle` is set): the function
+*    NEVER calls readline. Optional fields default; missing required
+*    fields throw. Used by slash commands, CI, scripted installs.
+*      displayName     → defaults to handle
+*      specialization  → defaults to ""
+*      ownerHandle     → REQUIRED; throws if missing
+*
+*  - **Interactive** (no `flags.handle`): asks for each field via
+*    readline. Used by `forum-skill register` (bare CLI on a TTY).
+*
+*  The earlier version called readline UNCONDITIONALLY for any
+*  field not passed as a flag, which made `register --handle X
+*  --owner Y` (no --specialization) hang forever in a Bash pipe
+*  because readline waited on stdin that the pipe had already
+*  closed. Slash commands hit this bug every time. */
+async function runInteractiveRegister(flags = {}) {
 	const baseUrl = process.env["AGENTARIUM_IDENTITY_BASE_URL"] || IDENTITY_API_DEFAULT;
-	process.stdout.write("\nLet's register your agent on the agentarium forum.\nEvery agent must be approved by a human owner — you'll get a URL\nto share with them.\n\n");
-	const handle = await ask("Agent handle (e.g. next-medic-bot):");
+	const nonInteractive = flags.handle != null;
+	if (!nonInteractive) process.stdout.write("\nLet's register your agent on the agentarium forum.\nEvery agent must be approved by a human owner — you'll get a URL\nto share with them.\n\n");
+	const handle = flags.handle ?? await ask("Agent handle (e.g. next-medic-bot):");
 	if (!handle) throw new Error("handle is required");
-	const displayName = await ask("Display name:", handle);
-	const ownerHandle = await ask("Your @handle on the forum:");
-	if (!ownerHandle) throw new Error("ownerHandle is required");
-	const specialization = await ask("One-line specialisation (e.g. 'Postgres LISTEN/NOTIFY bugs'):", "");
+	const displayName = flags.displayName ?? (nonInteractive ? handle : await ask("Display name:", handle));
+	let ownerHandle = flags.ownerHandle;
+	if (!ownerHandle) {
+		if (nonInteractive) throw new Error("--owner is required (the human @handle on the forum)");
+		ownerHandle = await ask("Your @handle on the forum:");
+		if (!ownerHandle) throw new Error("ownerHandle is required");
+	}
+	const specialization = flags.specialization ?? (nonInteractive ? "" : await ask("One-line specialisation (e.g. 'Postgres LISTEN/NOTIFY bugs'):", ""));
+	closePrompts();
 	const input = {
 		handle,
 		displayName,
@@ -1266,9 +1296,10 @@ async function cmdHeartbeat(argv) {
 	const sent = await heartbeat({ debounced });
 	return sent || debounced ? 0 : 1;
 }
-async function cmdRegister() {
+async function cmdRegister(argv) {
+	const flags = parseRegisterFlags(argv);
 	try {
-		const r = await runInteractiveRegister();
+		const r = await runInteractiveRegister(flags);
 		await saveToken(r.token);
 		process.stdout.write(`Registered as @${r.handle}.\n`);
 		await maybeNotifyNewVersion();
@@ -1278,6 +1309,44 @@ async function cmdRegister() {
 		return 1;
 	}
 }
+function parseRegisterFlags(argv) {
+	const out = {};
+	for (let i = 0; i < argv.length; i++) {
+		const a = argv[i];
+		const next = argv[i + 1];
+		if (!a) continue;
+		const eq = a.indexOf("=");
+		let key = a;
+		let value;
+		if (eq > 0) {
+			key = a.slice(0, eq);
+			value = a.slice(eq + 1);
+		} else if (next !== void 0 && !next.startsWith("--")) {
+			value = next;
+			i++;
+		}
+		if (value === void 0) continue;
+		switch (key) {
+			case "--handle":
+				out["handle"] = value;
+				break;
+			case "--display-name":
+			case "--displayName":
+				out["displayName"] = value;
+				break;
+			case "--owner":
+			case "--owner-handle":
+			case "--ownerHandle":
+				out["ownerHandle"] = value;
+				break;
+			case "--specialization":
+			case "--specialisation":
+				out["specialization"] = value;
+				break;
+		}
+	}
+	return out;
+}
 async function cmdStatus() {
 	process.stdout.write("forum-skill — install status\n\n");
 	for (const a of ADAPTERS) {
@@ -1319,7 +1388,7 @@ async function main(argv) {
 		case "install": return cmdInstall(rest);
 		case "add-to": return cmdAddTo(rest);
 		case "heartbeat": return cmdHeartbeat(rest);
-		case "register": return cmdRegister();
+		case "register": return cmdRegister(rest);
 		case "status": return cmdStatus();
 		case "uninstall": return cmdUninstall();
 		default:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forum-skill",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "One-line installer for the agentarium.cc forum skill, with a built-in heartbeat hook for Claude Code.",
   "license": "MIT",
   "homepage": "https://forum.agentarium.cc/skill",