@cohortapp/agent-sdk 2.17.0 → 2.18.4

package/scripts/poller/lib/cloud-relay-dedup.test.mjs

@@ -1,133 +0,0 @@
-/**
- * cloud-relay-dedup.test.mjs — node:test coverage for the pure SSE-parsing and
- * reconnect-replay dedup helpers used by the Slack cloud-relay client
- * (audit L30).
- *
- * These functions are pure (no network, no disk, no dotenv), so we import them
- * directly from the extracted module — importing the client itself would pull
- * in dotenv and open a live SSE connection.
- *
- * The reconnect-replay dedup (isReplayInSiblings) pins a production regression
- * fixed in commit ccfbaed, where the same event was re-delivered after each
- * relay reconnect; the `.processed` cases below guard that specifically.
- */
-
-import { test } from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  createSseParser,
-  isReplayInSiblings,
-} from "./cloud-relay-dedup.mjs";
-
-// ---------------------------------------------------------------------------
-// createSseParser — SSE framing
-// ---------------------------------------------------------------------------
-
-test("createSseParser emits one payload per blank-line-terminated event", () => {
-  const parser = createSseParser();
-  const out = parser.push('data: {"a":1}\n\n');
-  assert.deepEqual(out, ['{"a":1}']);
-});
-
-test("createSseParser buffers an event until its terminating blank line", () => {
-  const parser = createSseParser();
-  // No blank line yet — nothing should be emitted.
-  assert.deepEqual(parser.push('data: {"a":1}\n'), []);
-  // Blank line arrives in a later chunk — now it flushes.
-  assert.deepEqual(parser.push("\n"), ['{"a":1}']);
-});
-
-test("createSseParser concatenates multi-line data: with newlines", () => {
-  const parser = createSseParser();
-  const out = parser.push("data: line1\ndata: line2\n\n");
-  assert.deepEqual(out, ["line1\nline2"]);
-});
-
-test("createSseParser ignores ':' comment heartbeat lines", () => {
-  const parser = createSseParser();
-  // ": hb" is a heartbeat comment and must not produce or pollute a payload.
-  const out = parser.push(': hb\ndata: {"x":true}\n\n');
-  assert.deepEqual(out, ['{"x":true}']);
-});
-
-test("createSseParser strips exactly one leading space after 'data:'", () => {
-  const parser = createSseParser();
-  // Per the SSE spec, a single leading space after the colon is removed; a
-  // second space is significant payload content.
-  const out = parser.push("data:  two-spaces\n\n");
-  assert.deepEqual(out, [" two-spaces"]);
-});
-
-test("createSseParser tolerates CRLF line endings", () => {
-  const parser = createSseParser();
-  const out = parser.push('data: {"crlf":1}\r\n\r\n');
-  assert.deepEqual(out, ['{"crlf":1}']);
-});
-
-test("createSseParser handles two events split across chunk boundaries", () => {
-  const parser = createSseParser();
-  let out = parser.push('data: {"n":1}\n\nda');
-  assert.deepEqual(out, ['{"n":1}']);
-  out = parser.push('ta: {"n":2}\n\n');
-  assert.deepEqual(out, ['{"n":2}']);
-});
-
-test("createSseParser emits nothing for a comment-only / empty flush", () => {
-  const parser = createSseParser();
-  // A blank line with no preceding data: lines must not emit an empty payload.
-  assert.deepEqual(parser.push("\n"), []);
-  assert.deepEqual(parser.push(": keepalive\n\n"), []);
-});
-
-// ---------------------------------------------------------------------------
-// isReplayInSiblings — filesystem reconnect-replay dedup
-// ---------------------------------------------------------------------------
-
-const TS_KEY = "1700000000-001000"; // Slack ts "1700000000.001000" with dot→dash
-
-test("isReplayInSiblings is false when no sibling matches", () => {
-  const siblings = [
-    "2026-05-31T00-00-00-000Z-9999999999-000000-cloud-relay.json",
-    "unrelated.json",
-  ];
-  assert.equal(isReplayInSiblings(siblings, TS_KEY), false);
-});
-
-test("isReplayInSiblings detects a pending sibling for the same ts", () => {
-  const siblings = [
-    `2026-05-31T00-00-00-000Z-${TS_KEY}-cloud-relay.json`,
-  ];
-  assert.equal(isReplayInSiblings(siblings, TS_KEY), true);
-});
-
-test("isReplayInSiblings detects an already-processed sibling for the same ts", () => {
-  // The reconnect-replay regression (ccfbaed): a .processed file for the same
-  // ts must still count as a replay so we don't re-deliver after the daemon
-  // consumed it.
-  const siblings = [
-    `2026-05-31T00-00-00-000Z-${TS_KEY}-cloud-relay.json.processed`,
-  ];
-  assert.equal(isReplayInSiblings(siblings, TS_KEY), true);
-});
-
-test("isReplayInSiblings is false for a different ts key", () => {
-  const siblings = [
-    "2026-05-31T00-00-00-000Z-1700000000-999999-cloud-relay.json",
-    "2026-05-31T00-00-00-000Z-1700000000-999999-cloud-relay.json.processed",
-  ];
-  assert.equal(isReplayInSiblings(siblings, TS_KEY), false);
-});
-
-test("isReplayInSiblings is false on an empty inbox", () => {
-  assert.equal(isReplayInSiblings([], TS_KEY), false);
-});
-
-test("isReplayInSiblings does not false-match a ts that is a prefix of another", () => {
-  // Guard the suffix match: a longer ts key whose filename merely contains
-  // TS_KEY before extra digits must not be treated as a replay.
-  const siblings = [
-    `2026-05-31T00-00-00-000Z-${TS_KEY}9-cloud-relay.json`,
-  ];
-  assert.equal(isReplayInSiblings(siblings, TS_KEY), false);
-});