@danypops/pi-papyrus 0.36.1 → 0.36.3

package/extension/src/domain-tools.ts

@@ -675,14 +675,11 @@ export function registerPlaybooksTool(pi: ExtensionAPI): void {
 			try {
 				const params: Record<string, unknown> = { ...rawParams };
 				const action = params.action;
-				// invoke ends by calling tasks.focus server-side -- that focus write must land in the
-				// SAME session scope the tasks tool reads from (ctx.sessionManager.getSessionId()),
-				// the same resolution the tasks tool itself always applies, or the entry task's focus
-				// is invisible to tasks(action=focused/active) despite invoke reporting it as focused.
-				if (action === "invoke") {
-					const resolvedSessionId = params.session_id ?? ctx.sessionManager.getSessionId();
-					Object.assign(params, { session_id: resolvedSessionId, ...sessionSecretField(resolvedSessionId as string) });
-				}
+				// Name resolution must search regardless of project scope -- a Playbook itself is
+				// commonly unscoped (e.g. a cross-repo lab-deploy playbook), so resolutionRequest
+				// uses the caller's ORIGINAL project_root (undefined unless explicitly given), never
+				// the invoke-specific default applied below -- that default is only for where the
+				// resulting TASKS land, not for finding the playbook artifact itself.
 				const resolutionRequest = { project_root: params.project_root };
 				await resolveNameFields(params, [
 					{ nameKey: "name", idKey: "id", listOperation: "playbooks.list", baseRequest: resolutionRequest },
@@ -690,6 +687,24 @@ export function registerPlaybooksTool(pi: ExtensionAPI): void {
 					{ nameKey: "child_name", idKey: "child_id", listOperation: "playbooks.list", baseRequest: resolutionRequest },
 					{ nameKey: "dependency_name", idKey: "dependency_id", listOperation: "playbooks.list", baseRequest: resolutionRequest },
 				]);
+				// invoke ends by calling tasks.focus server-side -- that focus write must land in the
+				// SAME session scope the tasks tool reads from (ctx.sessionManager.getSessionId()),
+				// the same resolution the tasks tool itself always applies, or the entry task's focus
+				// is invisible to tasks(action=focused/active) despite invoke reporting it as focused.
+				// project_root defaults to ctx.cwd for the same reason: the tasks tool always scopes
+				// its OWN reads to ctx.cwd unless told otherwise, so an unscoped playbook-materialized
+				// task is invisible to tasks(action=focused) even with the right session -- confirmed
+				// live (the focus_set event existed with the correct sessionId, but Tasks.focused's own
+				// project-scope filter silently excluded the unscoped task from a cwd-scoped read).
+				// Applied AFTER name resolution: it must never affect finding the playbook itself.
+				if (action === "invoke") {
+					const resolvedSessionId = params.session_id ?? ctx.sessionManager.getSessionId();
+					Object.assign(params, {
+						project_root: params.project_root ?? ctx.cwd,
+						session_id: resolvedSessionId,
+						...sessionSecretField(resolvedSessionId as string),
+					});
+				}
 				if (action === "create" || action === "invoke" || action === "preview") normalizeJsonEncodedField(params, "arguments");
 				if (action === "create") {
 					const artifact = await callService<Record<string, unknown>, Artifact>("playbooks.create", params);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.36.1",
+  "version": "0.36.3",
   "description": "Pi host extension for Papyrus: native tools, TUI panels, and context injection over the daemon-backed graph store",
   "type": "module",
   "keywords": ["pi-package"],