@signalridge/pi-subagents 1.10.1 → 1.10.2

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 1.10.2
+### Patch Changes
+
+- 65e2481: Show supervisor questions in subagent input dialogs and identify the asking agent by its actual name.
+
 ## 1.10.1
 ### Patch Changes
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@signalridge/pi-subagents",
-  "version": "1.10.1",
+  "version": "1.10.2",
   "description": "Signalridge's managed subagent runtime with workflow-owned orchestration RPC.",
   "author": "tintinweb and signalridge contributors",
   "license": "MIT",

package/src/agent-runner.ts

@@ -1126,7 +1126,9 @@ export async function runAgent(
   const supervisorTools =
     ctx.hasUI && !isolated && options.supervisorQuestions !== false
       ? createSupervisorTool({
-          agentLabel: agentConfig?.displayName ?? type,
+          // Use the canonical agent type, not display_name: the question must
+          // identify the actual subagent that requested the decision.
+          agentLabel: agentConfig?.name ?? type,
           ask: {
             input: (title, placeholder) => ctx.ui.input(title, placeholder),
             select: (title, choices) => ctx.ui.select(title, choices),

package/src/supervisor.ts

@@ -36,7 +36,7 @@ export interface SupervisorAsk {
 export interface SupervisorToolContext {
   /** The parent's UI. Omitted when there is no human to ask. */
   ask?: SupervisorAsk;
-  /** Display name of the asking agent, for the prompt title. */
+  /** Actual type/name of the asking agent, for the prompt title. */
   agentLabel: string;
 }
 
@@ -82,7 +82,10 @@ export function createSupervisorTool(context: SupervisorToolContext) {
         const question = truncateCodePoints(sanitizeDisplayText(params.question), MAX_QUESTION, "…").trim();
         if (!question) return textResult("Ask a non-empty question.", true);
 
-        const title = `${context.agentLabel} asks`;
+        const title = `${sanitizeDisplayText(context.agentLabel)} asks`;
+        // Pi 0.85.1 ignores ui.input()'s placeholder argument. Keep the
+        // question in the title so the human sees it in both old and new hosts.
+        const promptTitle = `${title}: ${question}`;
         const options = (params.options ?? [])
           .map((option) => truncateCodePoints(sanitizeDisplayText(option), MAX_OPTION, "…").trim())
           .filter((option) => option.length > 0)
@@ -92,8 +95,8 @@ export function createSupervisorTool(context: SupervisorToolContext) {
         try {
           answer =
             options.length > 0
-              ? await ask.select(`${title}: ${question}`, options)
-              : await ask.input(title, question);
+              ? await ask.select(promptTitle, options)
+              : await ask.input(promptTitle);
         } catch (error: unknown) {
           // A dialog that cannot open must not fail the child's whole run; it
           // is told nobody answered and carries on under its own judgement.