Skip to content

Commit dc85b91

Browse files
committed
docs: refresh app-server parity audit
1 parent a70b91a commit dc85b91

3 files changed

Lines changed: 21 additions & 7 deletions

File tree

docs/app-server-events.md

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# App-Server Events Reference (Codex `5d4303510cc35571ae99a0a4a7d6ac40ccc03089`)
1+
# App-Server Events Reference (Codex `6baeec68bd1bdc11284885a6d00fa4db4e1327b6`)
22

33
This document helps agents quickly answer:
44
- Which app-server events CodexMonitor supports right now.
@@ -89,8 +89,11 @@ subscriptions.
8989
These arrive on the same frontend event stream but are not Codex v2
9090
`ServerNotification` methods:
9191

92-
- `item/commandExecution/requestApproval` and `item/fileChange/requestApproval`
93-
via suffix match in `isApprovalRequestMethod(method)`
92+
- approval requests ending in `requestApproval`, including
93+
`item/commandExecution/requestApproval`,
94+
`item/fileChange/requestApproval`, and
95+
`item/permissions/requestApproval`, via suffix match in
96+
`isApprovalRequestMethod(method)`
9497
- `item/tool/requestUserInput` (a Codex v2 server request, not a notification)
9598
- `codex/backgroundThread` (CodexMonitor synthetic bridge event)
9699
- `codex/connected` (CodexMonitor synthetic bridge event)
@@ -116,9 +119,12 @@ Compared against Codex app-server protocol v2 notifications, the following
116119
events are currently not routed:
117120

118121
- `configWarning`
122+
- `command/exec/outputDelta`
119123
- `deprecationNotice`
120124
- `fuzzyFileSearch/sessionCompleted`
121125
- `fuzzyFileSearch/sessionUpdated`
126+
- `hook/completed`
127+
- `hook/started`
122128
- `item/mcpToolCall/progress`
123129
- `mcpServer/oauthLogin/completed`
124130
- `model/rerouted`
@@ -166,6 +172,9 @@ Compared against Codex v2 request methods, CodexMonitor currently does not send:
166172

167173
- `account/logout`
168174
- `command/exec`
175+
- `command/exec/resize`
176+
- `command/exec/terminate`
177+
- `command/exec/write`
169178
- `config/batchWrite`
170179
- `config/mcpServer/reload`
171180
- `config/read`
@@ -181,10 +190,13 @@ Compared against Codex v2 request methods, CodexMonitor currently does not send:
181190
- `mock/experimentalMethod`
182191
- `plugin/install`
183192
- `plugin/list`
193+
- `plugin/uninstall`
184194
- `skills/config/write`
185195
- `skills/remote/export`
186196
- `skills/remote/list`
187197
- `thread/backgroundTerminals/clean`
198+
- `thread/decrement_elicitation`
199+
- `thread/increment_elicitation`
188200
- `thread/loaded/list`
189201
- `thread/metadata/update`
190202
- `thread/read`
@@ -203,6 +215,7 @@ Supported server requests:
203215

204216
- `item/commandExecution/requestApproval`
205217
- `item/fileChange/requestApproval`
218+
- `item/permissions/requestApproval`
206219
- `item/tool/requestUserInput`
207220

208221
Missing server requests:

src/features/app/hooks/useAppServerEvents.test.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -216,17 +216,17 @@ describe("useAppServerEvents", () => {
216216
listener?.({
217217
workspace_id: "ws-1",
218218
message: {
219-
method: "workspace/requestApproval",
219+
method: "item/permissions/requestApproval",
220220
id: 7,
221-
params: { mode: "full" },
221+
params: { mode: "full", threadId: "thread-2" },
222222
},
223223
});
224224
});
225225
expect(handlers.onApprovalRequest).toHaveBeenCalledWith({
226226
workspace_id: "ws-1",
227227
request_id: 7,
228-
method: "workspace/requestApproval",
229-
params: { mode: "full" },
228+
method: "item/permissions/requestApproval",
229+
params: { mode: "full", threadId: "thread-2" },
230230
});
231231

232232
act(() => {

src/utils/appServerEvents.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ describe("appServerEvents", () => {
3636
it("checks supported method and approval requests", () => {
3737
expect(isSupportedAppServerMethod("turn/started")).toBe(true);
3838
expect(isSupportedAppServerMethod("unknown/method")).toBe(false);
39+
expect(isApprovalRequestMethod("item/permissions/requestApproval")).toBe(true);
3940
expect(isApprovalRequestMethod("workspace/requestApproval")).toBe(true);
4041
expect(isApprovalRequestMethod("workspace/request")).toBe(false);
4142
});

0 commit comments

Comments
 (0)