Skip to content

Commit e5417bd

Browse files
committed
docs: align managed selector name with Domain
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c7f00b84-b0a7-4cdf-aca9-ffd49737f26e
1 parent 03184d2 commit e5417bd

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

nodejs/src/types.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1100,7 +1100,8 @@ export type SystemMessageConfig =
11001100
*
11011101
* `managedApprovalRequired` indicates that managed policy requires an explicit
11021102
* user decision. Hosts should bypass automatic approval and present their
1103-
* normal confirmation UI.
1103+
* normal confirmation UI. The runtime currently emits it for managed Shell,
1104+
* Read, Edit, and Domain selector asks.
11041105
*/
11051106
import type { PermissionRequest as GeneratedPermissionRequest } from "./generated/session-events.js";
11061107
export type PermissionRequest = GeneratedPermissionRequest & {

nodejs/test/session-event-types.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -104,11 +104,11 @@ describe("Session event type exports (#1156)", () => {
104104
expect(data.turnId).toBe("turn-1");
105105
});
106106

107-
it("exposes whether managed policy requires explicit user approval", () => {
107+
it("exposes explicit user approval metadata for managed Domain requests", () => {
108108
const request: PermissionRequest = {
109-
kind: "read",
110-
path: "/workspace/file.txt",
111-
intention: "Read a file",
109+
kind: "url",
110+
url: "https://api.example.com/data",
111+
intention: "Fetch domain data",
112112
managedApprovalRequired: true,
113113
};
114114

0 commit comments

Comments
 (0)