test: order sandbox status lookups

This commit is contained in:
Shakker
2026-05-11 22:41:40 +01:00
parent c69b58157f
commit c60a4c9552

View File

@@ -1839,27 +1839,30 @@ describe("session_status tool", () => {
expect(loadSessionStoreMock).toHaveBeenCalledWith("/tmp/main/sessions.json");
expect(updateSessionStoreMock).not.toHaveBeenCalled();
expect(callGatewayMock).toHaveBeenCalledTimes(3);
expect(callGatewayMock.mock.calls).toContainEqual([
{
method: "sessions.resolve",
params: {
sessionId: "s-other",
spawnedBy: "agent:main:subagent:child",
includeGlobal: false,
includeUnknown: false,
},
expect(callGatewayMock).toHaveBeenNthCalledWith(1, {
method: "sessions.list",
params: {
includeGlobal: false,
includeUnknown: false,
spawnedBy: "agent:main:subagent:child",
},
]);
expect(callGatewayMock.mock.calls).toContainEqual([
{
method: "sessions.list",
params: {
includeGlobal: false,
includeUnknown: false,
spawnedBy: "agent:main:subagent:child",
},
});
expect(callGatewayMock).toHaveBeenNthCalledWith(2, {
method: "sessions.resolve",
params: {
key: "s-other",
spawnedBy: "agent:main:subagent:child",
},
]);
});
expect(callGatewayMock).toHaveBeenNthCalledWith(3, {
method: "sessions.resolve",
params: {
sessionId: "s-other",
spawnedBy: "agent:main:subagent:child",
includeGlobal: false,
includeUnknown: false,
},
});
});
it("blocks sandboxed child session_status parent sessionId access outside its tree", async () => {
@@ -1890,27 +1893,30 @@ describe("session_status tool", () => {
expect(loadSessionStoreMock).toHaveBeenCalledWith("/tmp/main/sessions.json");
expect(updateSessionStoreMock).not.toHaveBeenCalled();
expect(callGatewayMock).toHaveBeenCalledTimes(3);
expect(callGatewayMock.mock.calls).toContainEqual([
{
method: "sessions.resolve",
params: {
sessionId: "s-parent",
spawnedBy: "agent:main:subagent:child",
includeGlobal: false,
includeUnknown: false,
},
expect(callGatewayMock).toHaveBeenNthCalledWith(1, {
method: "sessions.list",
params: {
includeGlobal: false,
includeUnknown: false,
spawnedBy: "agent:main:subagent:child",
},
]);
expect(callGatewayMock.mock.calls).toContainEqual([
{
method: "sessions.list",
params: {
includeGlobal: false,
includeUnknown: false,
spawnedBy: "agent:main:subagent:child",
},
});
expect(callGatewayMock).toHaveBeenNthCalledWith(2, {
method: "sessions.resolve",
params: {
key: "s-parent",
spawnedBy: "agent:main:subagent:child",
},
]);
});
expect(callGatewayMock).toHaveBeenNthCalledWith(3, {
method: "sessions.resolve",
params: {
sessionId: "s-parent",
spawnedBy: "agent:main:subagent:child",
includeGlobal: false,
includeUnknown: false,
},
});
});
it("keeps legacy main requester keys for sandboxed session tree checks", async () => {