@danypops/papyrus 0.33.1 → 0.33.3

package/extension/src/domain-tools.ts

@@ -164,7 +164,7 @@ export function normalizeJsonEncodedField(params: Record<string, unknown>, key:
 }
 
 /** Resolves every {nameKey -> idKey} pair present and not already satisfied by an explicit id, in place. */
-async function resolveNameFields(
+export async function resolveNameFields(
 	params: Record<string, unknown>,
 	fields: ReadonlyArray<{ nameKey: string; idKey: string; listOperation: OperationName; baseRequest: Record<string, unknown> }>,
 ): Promise<void> {
@@ -474,7 +474,7 @@ export function registerNotesTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "notes",
 		label: "Notes",
-		description: "Deferred human-intent inbox. ACTIONS: capture, list, show, consume, promote, archive. Capture stores a request without creating work. Consume marks it considered. To promote, first create the resulting Task, Doc, Rule, or Skill through its domain tool, then link it with target_id. Archive requires an explicit disposition. PREFER `name` (the note's exact title) over `id` for show/consume/promote/archive -- id is a backend implementation detail, resolved from name automatically.",
+		description: "Deferred human-intent inbox. ACTIONS: capture, list, show, consume, promote, archive. Capture stores a request without creating work. Consume marks it considered. To promote, first create the resulting Task, Doc, Rule, or Skill through its domain tool, then link it with target_id (or target_name). Archive requires an explicit disposition. PREFER `name` (the note's exact title) over `id` for show/consume/promote/archive, and `target_name` over `target_id` for promote -- all are backend implementation details, resolved from name automatically (target_name searches across every kind, since a promotion target can be a task, doc, rule, or skill).",
 		parameters: Type.Object({
 			action: Type.String(),
 			id: Type.Optional(Type.String()),
@@ -485,6 +485,7 @@ export function registerNotesTool(pi: ExtensionAPI): void {
 			text: Type.Optional(Type.String()),
 			limit: Type.Optional(Type.Number()),
 			target_id: Type.Optional(Type.String()),
+			target_name: Type.Optional(Type.String()),
 			disposition: Type.Optional(Type.Union(NOTE_DISPOSITIONS.map((value) => Type.Literal(value)))),
 			reason: Type.Optional(Type.String()),
 			session_id: Type.Optional(Type.String()),
@@ -497,7 +498,11 @@ export function registerNotesTool(pi: ExtensionAPI): void {
 				const params: Record<string, unknown> = { ...rawParams };
 				const action = params.action;
 				const baseRequest = { project_root: params.project_root ?? ctx.cwd, actor: "agent", source: "notes-tool" };
-				await resolveNameFields(params, [{ nameKey: "name", idKey: "id", listOperation: "notes.list", baseRequest }]);
+				await resolveNameFields(params, [
+					{ nameKey: "name", idKey: "id", listOperation: "notes.list", baseRequest },
+					// Kind-agnostic: a promotion target can be a task, doc, rule, or skill, so this searches every kind rather than only notes.
+					{ nameKey: "target_name", idKey: "target_id", listOperation: "artifact.query", baseRequest },
+				]);
 				const request = { ...params, ...baseRequest };
 				if (action === "capture") {
 					const artifact = await callService<Record<string, unknown>, Artifact>("notes.capture", request);

package/extension/src/index.ts

@@ -16,12 +16,13 @@ import {
 	TASK_DRIVER_MAX_UNCHANGED_TURNS,
 	PAPYRUS_CONTEXT_INJECTION_CHANNEL,
 	CONTEXT_ESTIMATE_CHARACTERS_PER_TOKEN,
+	TASK_WIDGET_POLL_INTERVAL_MS,
 } from "../../src/constants.ts";
 import type { Artifact } from "../../src/domain/artifact.ts";
 import type { GateResult } from "../../src/domain/gate.ts";
 import { formatMetadata } from "./artifact-format.ts";
 import { callService } from "./service-client.ts";
-import { registerDomainTools } from "./domain-tools.ts";
+import { registerDomainTools, resolveNameFields } from "./domain-tools.ts";
 import { ensureTypingCourtesyTracking, isLiveAskPending } from "./discuss-ask-view.ts";
 import { PLAYBOOK_BRIDGE_MAX_PLAYBOOKS, registerPlaybookBridge } from "./playbook-bridge.ts";
 import type { TaskGraph, TaskStatus } from "../../src/task-service.ts";
@@ -102,6 +103,7 @@ export class TaskOverlay {
 	private snapshot: TaskGraph = { nodes: [], rootIds: [] };
 	private projectRoot: string | undefined;
 	private sessionId: string | undefined;
+	private pollTimer: ReturnType<typeof setInterval> | undefined;
 
 	setUI(ctx: ExtensionUIContext): void {
 		if (ctx !== this.uiCtx) {
@@ -175,7 +177,24 @@ export class TaskOverlay {
 		return renderTaskWidgetLines(theme, buildTaskWidgetProjection(this.snapshot), width);
 	}
 
+	/**
+	 * Fallback for a Task mutation no event announces -- the CLI run directly from a shell, or
+	 * a second concurrent Pi session against the same daemon. Idempotent: a second call is a
+	 * no-op rather than starting a competing timer.
+	 */
+	startPolling(intervalMs: number = TASK_WIDGET_POLL_INTERVAL_MS): void {
+		if (this.pollTimer) return;
+		this.pollTimer = setInterval(() => { void this.refresh(); }, intervalMs);
+	}
+
+	stopPolling(): void {
+		if (!this.pollTimer) return;
+		clearInterval(this.pollTimer);
+		this.pollTimer = undefined;
+	}
+
 	dispose(): void {
+		this.stopPolling();
 		this.uiCtx?.setWidget(WIDGET_KEY, undefined);
 		this.registered = false;
 		this.tui = undefined;
@@ -312,12 +331,15 @@ export default async function (pi: ExtensionAPI) {
 			"ACTIONS: link (from+relation+to), unlink (from+relation+to — idempotent, no error if already absent; for Task depends_on/contains prefer the tasks tool's undepend/uncontain), " +
 			"tree (id → bounded BFS subgraph), " +
 			"history (who did what, when — requires id, actor, or session_id). " +
-			"status (id+status) exists at the protocol level but is refused for every kind with its own lifecycle (Doc/Rule/Skill/Playbook/Task/Note all reject it) -- use that kind's own domain tool for status changes (docs.activate, rules.enable, tasks.start, etc), never this.",
+			"status (id+status) exists at the protocol level but is refused for every kind with its own lifecycle (Doc/Rule/Skill/Playbook/Task/Note all reject it) -- use that kind's own domain tool for status changes (docs.activate, rules.enable, tasks.start, etc), never this. " +
+			"PREFER `from_name`/`to_name` over `from`/`to` for link/unlink -- both are backend implementation details, resolved from name automatically, searching across every kind since either end of an edge can be any artifact.",
 		parameters: Type.Object({
 			action: Type.String({ description: "link | unlink | tree | status | history" }),
 			from: Type.Optional(Type.String()),
+			from_name: Type.Optional(Type.String()),
 			relation: Type.Optional(Type.String()),
 			to: Type.Optional(Type.String()),
+			to_name: Type.Optional(Type.String()),
 			id: Type.Optional(Type.String()),
 			status: Type.Optional(Type.String()),
 			depth: Type.Optional(Type.Number({ description: "tree traversal depth; bounded by a hard ceiling" })),
@@ -329,18 +351,26 @@ export default async function (pi: ExtensionAPI) {
 		}),
 		renderCall(args, theme) { return renderPapyrusToolCall("Artifact graph", args, theme); },
 		renderResult(result, options, theme, context) { return renderPapyrusToolResult(result, options, theme, context); },
-		async execute(_id, params, _signal, _onUpdate, _ctx) {
+		async execute(_id, rawParams, _signal, _onUpdate, _ctx) {
 			try {
+				const params: Record<string, unknown> = { ...rawParams };
+				if (params.action === "link" || params.action === "unlink") {
+					// Kind-agnostic: either end of an edge can be a task, doc, rule, skill, or playbook.
+					await resolveNameFields(params, [
+						{ nameKey: "from_name", idKey: "from", listOperation: "artifact.query", baseRequest: {} },
+						{ nameKey: "to_name", idKey: "to", listOperation: "artifact.query", baseRequest: {} },
+					]);
+				}
 				if (params.action === "link") {
-					await callService("graph.link", { from: params.from!, relation: params.relation!, to: params.to! });
-					const names = await artifactNamesById([params.from!, params.to!]);
-					const output = `Linked "${names.get(params.from!) ?? "unknown artifact"}" --${params.relation}--> "${names.get(params.to!) ?? "unknown artifact"}"`;
+					await callService("graph.link", { from: params.from as string, relation: params.relation as string, to: params.to as string });
+					const names = await artifactNamesById([params.from as string, params.to as string]);
+					const output = `Linked "${names.get(params.from as string) ?? "unknown artifact"}" --${params.relation}--> "${names.get(params.to as string) ?? "unknown artifact"}"`;
 					return text(output, createPreviewDetails("graph.link", "Artifact relationship", output));
 				}
 				if (params.action === "unlink") {
-					const result = await callService<Record<string, unknown>, { removed: boolean }>("graph.unlink", { from: params.from!, relation: params.relation!, to: params.to! });
-					const names = await artifactNamesById([params.from!, params.to!]);
-					const relationship = `"${names.get(params.from!) ?? "unknown artifact"}" --${params.relation}--> "${names.get(params.to!) ?? "unknown artifact"}"`;
+					const result = await callService<Record<string, unknown>, { removed: boolean }>("graph.unlink", { from: params.from as string, relation: params.relation as string, to: params.to as string });
+					const names = await artifactNamesById([params.from as string, params.to as string]);
+					const relationship = `"${names.get(params.from as string) ?? "unknown artifact"}" --${params.relation}--> "${names.get(params.to as string) ?? "unknown artifact"}"`;
 					const output = result.removed ? `Unlinked ${relationship}` : `No such relationship: ${relationship}`;
 					return text(output, createPreviewDetails("graph.unlink", "Artifact relationship", output));
 				}
@@ -549,6 +579,7 @@ export default async function (pi: ExtensionAPI) {
 		overlay.setProjectRoot(ctx.cwd);
 		overlay.setSessionId(ctx.sessionManager.getSessionId());
 		await overlay.refresh();
+		overlay.startPolling(TASK_WIDGET_POLL_INTERVAL_MS);
 	});
 
 	pi.on("session_before_compact", () => { taskContinuation.onCompaction(); });

package/package.json

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

package/src/constants.ts

@@ -65,6 +65,12 @@ export const TASK_CONTEXT_CURRENT_LIMIT = 3;
 export const TASK_CONTEXT_REJECTED_LIMIT = 3;
 export const TASK_WIDGET_OPEN_LIMIT = 3;
 export const TASK_DETAIL_MIN_VISIBLE_LINES = 8;
+/**
+ * Event-triggered refresh (tool_execution_end, session_compact/tree) can't see a Task
+ * mutation from outside this Pi session -- another concurrent session, or the CLI run
+ * directly from a shell. This bounded poll is the fallback for exactly that gap.
+ */
+export const TASK_WIDGET_POLL_INTERVAL_MS = 20_000;
 export const TASK_DETAIL_MAX_VISIBLE_LINES = 24;
 export const TASK_DETAIL_RESERVED_ROWS = 8;
 export const TASK_DETAIL_HORIZONTAL_PAN_COLUMNS = 4;