Slack: move outbound visibility test

This commit is contained in:
Gustavo Madeira Santana
2026-04-08 16:18:25 -04:00
parent 9bb7cc5cce
commit 14f202e1cc

View File

@@ -92,21 +92,6 @@ function requireSlackListPeers() {
}
describe("slackPlugin actions", () => {
it("treats ACP block text as visible delivered output", () => {
expect(
slackPlugin.outbound?.shouldTreatDeliveredTextAsVisible?.({
kind: "block",
text: "hello",
}),
).toBe(true);
expect(
slackPlugin.outbound?.shouldTreatDeliveredTextAsVisible?.({
kind: "tool",
text: "hello",
}),
).toBe(false);
});
it("prefers session lookup for announce target routing", () => {
expect(slackPlugin.meta.preferSessionLookupForAnnounceTarget).toBe(true);
});
@@ -387,6 +372,21 @@ describe("slackPlugin outbound", () => {
},
};
it("treats ACP block text as visible delivered output", () => {
expect(
slackPlugin.outbound?.shouldTreatDeliveredTextAsVisible?.({
kind: "block",
text: "hello",
}),
).toBe(true);
expect(
slackPlugin.outbound?.shouldTreatDeliveredTextAsVisible?.({
kind: "tool",
text: "hello",
}),
).toBe(false);
});
it("advertises the 8000-character Slack default chunk limit", () => {
expect(slackOutbound.textChunkLimit).toBe(8000);
expect(slackPlugin.outbound?.textChunkLimit).toBe(8000);