@selesai/code 0.5.19 → 0.5.21

package/dist/extensions/handoff-new.test.ts

@@ -108,9 +108,9 @@ function createCtx(opts: {
 	customResult?: string | null;
 	mode?: string;
 } = {}) {
-	const calls: { newSession: any; editorText: string | null; notify: { msg: string; kind: string }[] } = {
+	const calls: { newSession: any; userMessage: string | null; notify: { msg: string; kind: string }[] } = {
 		newSession: null,
-		editorText: null,
+		userMessage: null,
 		notify: [],
 	};
 	const ctx: any = {
@@ -128,9 +128,9 @@ function createCtx(opts: {
 		ui: {
 			notify: (msg: string, kind: string) => calls.notify.push({ msg, kind }),
 			custom: async <T>(_factory: any): Promise<T> => (opts.customResult === undefined ? "HANDOFF PROMPT" : opts.customResult) as unknown as T,
-			setEditorText: (text: string) => {
-				calls.editorText = text;
-			},
+		},
+		async sendUserMessage(text: string) {
+			calls.userMessage = text;
 		},
 		async newSession(options: any) {
 			calls.newSession = options;
@@ -143,12 +143,12 @@ function createCtx(opts: {
 	return { ctx, calls };
 }
 
-test("happy path: newSession called with parentSession, editor text set", async () => {
+test("happy path: newSession called with parentSession, handoff submitted", async () => {
 	const { commands } = createPiHarness();
 	const { ctx, calls } = createCtx();
 	await commands.get("handoff-new")!.handler("continue the fix", ctx);
 	assert.equal(calls.newSession.parentSession, "/tmp/old.json");
-	assert.equal(calls.editorText, "HANDOFF PROMPT");
+	assert.equal(calls.userMessage, "HANDOFF PROMPT");
 	assert.equal(calls.newSession.cancelled, undefined);
 });
 
@@ -157,8 +157,7 @@ test("no goal arg uses DEFAULT_GOAL", async () => {
 	const { ctx, calls } = createCtx();
 	await commands.get("handoff-new")!.handler("   ", ctx);
 	// happy path still completes; default goal only affects the AI prompt text.
-	assert.equal(calls.editorText, "HANDOFF PROMPT");
-	assert.ok(calls.notify.some((n) => /Handoff ready/.test(n.msg)));
+	assert.equal(calls.userMessage, "HANDOFF PROMPT");
 });
 
 test("non-tui mode fails cleanly, no new session", async () => {

package/dist/extensions/handoff-new.ts

@@ -1,14 +1,14 @@
 /**
  * /handoff-new — generate a handoff prompt and open a clean new session with
- * that content as the first unsent prompt (editor text, not hidden system
- * prompt). Sibling of examples/extensions/handoff.ts, adapted to this repo's
+ * that content as the first prompt. Sibling of examples/extensions/handoff.ts,
+ * adapted to this repo's
  * import conventions and reduced to the one thing that ships.
  *
  * Usage:
  *   /handoff-new continue implementing the workflow fix
  *
- * If no goal argument is given, a default goal is used. The new session opens
- * with the generated text in its editor; the user can change it before submit.
+ * If no goal argument is given, a default goal is used. The generated handoff
+ * is submitted immediately in the new session.
  */
 
 import type { AgentMessage } from "@earendil-works/pi-agent-core";
@@ -86,7 +86,7 @@ export function buildAiContext(conversationText: string, goal: string): Context
 
 export default function (pi: ExtensionAPI) {
 	pi.registerCommand("handoff-new", {
-		description: "Generate a handoff prompt and open a clean new session with it as the first draft",
+		description: "Generate a handoff prompt and continue in a clean new session",
 		handler: handoffNew,
 	});
 }
@@ -152,12 +152,10 @@ async function handoffNew(args: string, ctx: ExtensionCommandContext) {
 		return;
 	}
 
-	// Editor text, NOT a hidden system prompt: the user can edit before submit.
 	const newSessionResult = await ctx.newSession({
 		parentSession: currentSessionFile,
 		withSession: async (replacementCtx) => {
-			replacementCtx.ui.setEditorText(result);
-			replacementCtx.ui.notify("Handoff ready. Submit when ready.", "info");
+			await replacementCtx.sendUserMessage(result);
 		},
 	});
 

package/dist/extensions/pi-intercom/broker/spawn.test.ts

@@ -0,0 +1,31 @@
+import { createRequire } from "module";
+import { describe, expect, it } from "vitest";
+import { getBrokerLaunchSpec, getWindowsBrokerCommandLine } from "./spawn.ts";
+
+const require = createRequire(import.meta.url);
+const brokerPath = "C:\\broker dir\\broker.ts";
+const extensionDir = "C:\\published-package\\dist\\extensions\\pi-intercom";
+const nodePath = "C:\\Program Files\\nodejs\\node.exe";
+
+describe("intercom broker launch", () => {
+  it("resolves default Windows tsx launcher from installed dependencies", () => {
+    const tsxCliPath = require.resolve("tsx/cli");
+
+    expect(getWindowsBrokerCommandLine(brokerPath, extensionDir, nodePath)).toBe(
+      `"${nodePath}" "${tsxCliPath}" "${brokerPath}"`,
+    );
+    expect(getWindowsBrokerCommandLine(brokerPath, extensionDir, nodePath)).not.toContain(extensionDir);
+  });
+
+  it("preserves custom broker commands on Windows and other platforms", () => {
+    expect(getWindowsBrokerCommandLine(brokerPath, extensionDir, nodePath, "broker-command", ["--port", "1234"])).toBe(
+      `"broker-command" "--port" "1234" "${brokerPath}"`,
+    );
+
+    expect(getBrokerLaunchSpec(brokerPath, "broker-command", ["--port", "1234"], extensionDir, "linux")).toEqual({
+      kind: "direct",
+      command: "broker-command",
+      args: ["--port", "1234", brokerPath],
+    });
+  });
+});

package/dist/extensions/pi-intercom/broker/spawn.ts

@@ -1,10 +1,12 @@
 import { spawn } from "child_process";
 import { existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "fs";
+import { createRequire } from "module";
 import { join, dirname } from "path";
 import { fileURLToPath } from "url";
 import net from "net";
 import { getBrokerSocketPath, getIntercomDir } from "./paths.js";
 
+const require = createRequire(import.meta.url);
 const INTERCOM_DIR = getIntercomDir();
 const EXTENSION_DIR = join(dirname(fileURLToPath(import.meta.url)), "..");
 const BROKER_SOCKET = getBrokerSocketPath();
@@ -29,8 +31,8 @@ function sleep(ms: number): Promise<void> {
   return new Promise(resolve => setTimeout(resolve, ms));
 }
 
-export function getTsxCliPath(extensionDir: string = EXTENSION_DIR): string {
-  return join(extensionDir, "node_modules", "tsx", "dist", "cli.mjs");
+export function getTsxCliPath(): string {
+  return require.resolve("tsx/cli");
 }
 
 function quoteWindowsArg(value: string): string {
@@ -56,7 +58,7 @@ export function getWindowsBrokerCommandLine(
   brokerArgs: string[] = ["--no-install", "tsx"],
 ): string {
   if (usesDefaultBrokerCommand(brokerCommand, brokerArgs)) {
-    return [quoteWindowsArg(nodePath), quoteWindowsArg(getTsxCliPath(extensionDir)), quoteWindowsArg(brokerPath)].join(" ");
+    return [quoteWindowsArg(nodePath), quoteWindowsArg(getTsxCliPath()), quoteWindowsArg(brokerPath)].join(" ");
   }
 
   return [quoteWindowsArg(brokerCommand), ...brokerArgs.map(quoteWindowsArg), quoteWindowsArg(brokerPath)].join(" ");