@danypops/papyrus 0.27.14 → 0.27.15

package/extension/src/discuss-ask-view.ts

@@ -56,6 +56,9 @@ export type AskDisplayMode = "overlay" | "inline";
 export interface AskQuestionParams {
 	question: string;
 	context?: string;
+	/** Plain orientation line ("which discussion is this"), shown dim above the question -- not a
+	 * labeled section like context. Typically the Discussion's own title. */
+	subtitle?: string;
 	options?: AskOption[];
 	allowMultiple?: boolean;
 	allowFreeform?: boolean;
@@ -621,6 +624,7 @@ class AskComponent extends Container {
 	constructor(
 		private question: string,
 		private context: string | undefined,
+		private subtitle: string | undefined,
 		private options: AskOption[],
 		private allowMultiple: boolean,
 		private allowFreeform: boolean,
@@ -781,6 +785,9 @@ class AskComponent extends Container {
 				"",
 			];
 		}
+		// Only meaningful when reached by escaping OUT of a real select list -- see showFreeformMode's
+		// identical guard for the non-overlay layout.
+		if (this.options.length === 0) return [];
 		return [...new Text(this.theme.fg("accent", this.theme.bold("Custom answer")), 1, 0).render(width), ""];
 	}
 
@@ -864,7 +871,11 @@ class AskComponent extends Container {
 
 	private updateStaticText(): void {
 		const theme = this.theme;
-		this.titleText.setText(theme.fg("accent", theme.bold(this.mode === "comment" ? "Optional comment" : "Question")));
+		// Reuses the same slot for two different purposes: a plain "which discussion is this" subtitle
+		// normally, or "Optional comment" while in comment mode. A generic "Question" header above the
+		// real question text added nothing beyond what the question itself already says, and read
+		// confusingly like the question text WAS the header.
+		this.titleText.setText(this.mode === "comment" ? theme.fg("accent", theme.bold("Optional comment")) : this.subtitle ? theme.fg("dim", this.subtitle) : "");
 		this.questionText.setText(theme.fg("text", theme.bold(this.question)));
 		if (this.contextComponent && this.context) {
 			if (this.contextComponent instanceof Markdown) (this.contextComponent as Markdown).setText(`**Context:**\n${this.context}`);
@@ -985,8 +996,13 @@ class AskComponent extends Container {
 		const editor = this.ensureEditor();
 		this.setEditorText(this.freeformDraft);
 		(editor as any).focused = this._focused;
-		this.modeContainer.addChild(new Text(this.theme.fg("accent", this.theme.bold("Custom answer")), 1, 0));
-		this.modeContainer.addChild(new Spacer(1));
+		// Only meaningful when reached by escaping OUT of a real select list ("instead of these
+		// options, here's a custom one") -- with no options at all there's nothing to contrast
+		// against, so the label is pure noise.
+		if (this.options.length > 0) {
+			this.modeContainer.addChild(new Text(this.theme.fg("accent", this.theme.bold("Custom answer")), 1, 0));
+			this.modeContainer.addChild(new Spacer(1));
+		}
 		this.modeContainer.addChild(editor);
 		this.updateHelpText();
 		this.invalidate();
@@ -1165,7 +1181,7 @@ async function askQuestionBlocking(
 		const factory = (tui: TUI, theme: Theme, keybindings: KeybindingsManager, done: (result: AskResponse | null) => void) => {
 			if (params.signal) params.signal.addEventListener("abort", () => done(null), { once: true });
 			if (params.timeout && params.timeout > 0) setTimeout(() => done(null), params.timeout);
-			return new AskComponent(params.question, normalizedContext, options, allowMultiple, allowFreeform, allowComment, displayMode, tui, theme, keybindings, shortcuts, done);
+			return new AskComponent(params.question, normalizedContext, params.subtitle, options, allowMultiple, allowFreeform, allowComment, displayMode, tui, theme, keybindings, shortcuts, done);
 		};
 
 		const overlayToggle = shortcuts.overlayToggle;

package/extension/src/discuss.ts

@@ -82,14 +82,15 @@ export async function showDiscussions(ctx: ExtensionCommandContext): Promise<voi
 			}
 			if (choice === "Reply") {
 				const pending = (() => { try { return readDiscussionExtra(discussion.extra); } catch { return undefined; } })();
-				const question = `Reply to "${discussion.title}":`;
-				// Same fix as the live discuss tool: the title alone isn't the actual question -- show
-				// the most recent round's real content as context, not a bare title prompt.
+				// Same fix as the live discuss tool: the most recent round's own content IS the real
+				// question -- the title becomes a plain orientation subtitle, not a labeled-backwards
+				// "Context:" section under a generic "Reply to <title>:" wrapper.
 				const transcript = await callService<Record<string, unknown>, DiscussionAndRounds>("discuss.show", { id: discussion.id });
-				const context = transcript.rounds.at(-1)?.content?.trim() || undefined;
+				const question = transcript.rounds.at(-1)?.content?.trim() || `Reply to "${discussion.title}":`;
+				const subtitle = discussion.title;
 				const answer = pending?.pendingOptions && pending.pendingOptions.length > 0 && pending.pendingOptionsMode
-					? await askQuestion(commandCtx, { question, context, options: pending.pendingOptions.map((title) => ({ title })), allowMultiple: pending.pendingOptionsMode === "multi" })
-					: await askQuestion(commandCtx, { question, context });
+					? await askQuestion(commandCtx, { question, subtitle, options: pending.pendingOptions.map((title) => ({ title })), allowMultiple: pending.pendingOptionsMode === "multi" })
+					: await askQuestion(commandCtx, { question, subtitle });
 				if (!answer) return; // canceled
 				await callService("discuss.reply", { id: discussion.id, actor: ACTOR, content: answer.content, ...(answer.selected ? { selected: answer.selected } : {}), source: SOURCE });
 				commandCtx.ui.notify(answer.selected ? `Selected: ${answer.selected.join(", ")}` : "Reply added.", "info");

package/extension/src/domain-tools.ts

@@ -42,22 +42,23 @@ function text(message: string, details: unknown = {}) {
 async function liveAnswer(ctx: ExtensionContext, discussion: Artifact, latestContent: string | undefined, onUpdate: AgentToolUpdateCallback | undefined, signal: AbortSignal | undefined): Promise<{ content: string; selected?: string[] } | undefined> {
 	if (!ctx.hasUI) return undefined;
 	const pending = (() => { try { return readDiscussionExtra(discussion.extra); } catch { return undefined; } })();
-	const question = `Reply to "${discussion.title}":`;
-	// The discussion's title alone is often not the actual question -- a human staring at a bare
-	// "Reply to '<title>':" prompt with no visible content has no way to tell what's being asked.
-	// The just-recorded round's own content is the real question text; show it as context.
-	const context = latestContent?.trim() || undefined;
+	// The just-recorded round's own content IS the real question -- a generic "Reply to <title>:"
+	// wrapper as the primary question, with the real content demoted to "Context:", left a human
+	// staring at a labeled-backwards prompt (live-observed). The wrapper is now only a fallback for
+	// the degenerate case of empty content; the title becomes a plain orientation subtitle instead.
+	const question = latestContent?.trim() || `Reply to "${discussion.title}":`;
+	const subtitle = discussion.title;
 	if (pending?.pendingOptions && pending.pendingOptions.length > 0 && pending.pendingOptionsMode) {
 		return askQuestion(ctx, {
 			question,
-			context,
+			subtitle,
 			options: pending.pendingOptions.map((title) => ({ title })),
 			allowMultiple: pending.pendingOptionsMode === "multi",
 			onUpdate,
 			signal,
 		});
 	}
-	return askQuestion(ctx, { question, context, onUpdate, signal });
+	return askQuestion(ctx, { question, subtitle, onUpdate, signal });
 }
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/papyrus",
-  "version": "0.27.14",
+  "version": "0.27.15",
   "description": "Daemon-backed graph artifacts, evidence-bearing tasks, rules, skills, and native TUI workflows for Pi",
   "type": "module",
   "keywords": ["pi-package"],