@bytesbrains/pi-telegram-bridge 1.0.0 → 1.0.2

package/src/__tests__/tools.test.ts

@@ -0,0 +1,251 @@
+/**
+ * pi-telegram-bridge — Tool Schema Tests
+ */
+import { describe, it, expect } from "vitest";
+import { Value } from "typebox/value";
+import {
+  listenSchema,
+  sendSchema,
+  askSchema,
+  statusSchema,
+  overrideSchema,
+} from "../tools/telegram";
+
+// ── listenSchema ──────────────────────────────────────────────────
+
+describe("listenSchema", () => {
+  it("accepts empty object", () => {
+    expect(Value.Check(listenSchema, {})).toBe(true);
+  });
+
+  it("still checks ok with extra properties (by default)", () => {
+    // TypeBox Object doesn't strip extra by default unless additionalProperties: false
+    expect(Value.Check(listenSchema, { extra: 1 })).toBe(true);
+  });
+});
+
+// ── sendSchema ────────────────────────────────────────────────────
+
+describe("sendSchema", () => {
+  it("accepts valid message", () => {
+    expect(Value.Check(sendSchema, { message: "Hello" })).toBe(true);
+  });
+
+  it("accepts empty message", () => {
+    expect(Value.Check(sendSchema, { message: "" })).toBe(true);
+  });
+
+  it("rejects missing message", () => {
+    expect(Value.Check(sendSchema, {})).toBe(false);
+    expect(Value.Check(sendSchema, { message: 123 })).toBe(false);
+    expect(Value.Check(sendSchema, { message: null })).toBe(false);
+  });
+
+  it("accepts long messages", () => {
+    expect(Value.Check(sendSchema, { message: "x".repeat(4096) })).toBe(true);
+  });
+});
+
+// ── askSchema ─────────────────────────────────────────────────────
+
+describe("askSchema", () => {
+  it("accepts minimal valid input (question only)", () => {
+    expect(Value.Check(askSchema, { question: "Deploy?" })).toBe(true);
+  });
+
+  it("accepts question with options", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "Which?",
+        options: ["A", "B", "C"],
+      }),
+    ).toBe(true);
+  });
+
+  it("accepts question with timeoutMinutes", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "Go?",
+        timeoutMinutes: 5,
+      }),
+    ).toBe(true);
+  });
+
+  it("accepts all fields", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "Full test",
+        options: ["Yes", "No"],
+        timeoutMinutes: 10,
+      }),
+    ).toBe(true);
+  });
+
+  it("rejects missing question", () => {
+    expect(Value.Check(askSchema, {})).toBe(false);
+    expect(Value.Check(askSchema, { question: 123 })).toBe(false);
+    expect(Value.Check(askSchema, { options: ["A"] })).toBe(false);
+  });
+
+  it("accepts empty options array", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "test",
+        options: [],
+      }),
+    ).toBe(true);
+  });
+
+  it("rejects non-array options", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "test",
+        options: "not-array",
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects non-number timeoutMinutes", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "test",
+        timeoutMinutes: "30",
+      }),
+    ).toBe(false);
+  });
+
+  it("accepts timeoutMinutes as 0", () => {
+    expect(
+      Value.Check(askSchema, {
+        question: "test",
+        timeoutMinutes: 0,
+      }),
+    ).toBe(true);
+  });
+});
+
+// ── statusSchema ──────────────────────────────────────────────────
+
+describe("statusSchema", () => {
+  it("accepts empty object", () => {
+    expect(Value.Check(statusSchema, {})).toBe(true);
+  });
+});
+
+// ── overrideSchema ────────────────────────────────────────────────
+
+describe("overrideSchema", () => {
+  it("accepts required fields only", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "git push --force",
+        reason: "Force push detected",
+      }),
+    ).toBe(true);
+  });
+
+  it("rejects missing command", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        reason: "test",
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects missing reason", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "rm -rf /",
+      }),
+    ).toBe(false);
+  });
+
+  it("accepts all optional fields", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "rm -rf node_modules",
+        reason: "Outside project boundary",
+        context: "User requested clean reinstall",
+        options: ["Yes", "No", "Maybe", "Explain"],
+        timeoutMinutes: 15,
+      }),
+    ).toBe(true);
+  });
+
+  it("accepts empty options array", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: "test",
+        options: [],
+      }),
+    ).toBe(true);
+  });
+
+  it("rejects non-string command", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: 123,
+        reason: "test",
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects non-string reason", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: null,
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects non-string context", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: "test",
+        context: 123,
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects non-array options", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: "test",
+        options: "yes,no",
+      }),
+    ).toBe(false);
+  });
+
+  it("rejects non-number timeoutMinutes", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: "test",
+        timeoutMinutes: "10",
+      }),
+    ).toBe(false);
+  });
+
+  it("accepts timeoutMinutes as 0", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "test",
+        reason: "test",
+        timeoutMinutes: 0,
+      }),
+    ).toBe(true);
+  });
+
+  it("accepts long command and reason strings", () => {
+    expect(
+      Value.Check(overrideSchema, {
+        command: "x".repeat(1000),
+        reason: "y".repeat(1000),
+      }),
+    ).toBe(true);
+  });
+});

package/src/helpers.ts

@@ -67,7 +67,11 @@ export async function pollReply(
   // Drain pending updates to start fresh
   let lastId = 0;
   try {
-    const drain = await fetch(`${BASE_URL}${token}/getUpdates?timeout=0`);
+    const drain = await fetch(`${BASE_URL}${token}/getUpdates`, {
+      method: "POST",
+      headers: { "Content-Type": "application/json" },
+      body: JSON.stringify({ timeout: 0 }),
+    });
     if (drain.ok) {
       const d = (await drain.json()) as {
         ok: boolean;
@@ -83,7 +87,12 @@ export async function pollReply(
 
   while (Date.now() < deadline) {
     const res = await fetch(
-      `${BASE_URL}${token}/getUpdates?offset=${lastId + 1}&timeout=5`,
+      `${BASE_URL}${token}/getUpdates`,
+      {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({ offset: lastId + 1, timeout: 5 }),
+      },
     );
     if (!res.ok) {
       await new Promise((r) => setTimeout(r, 3000));
@@ -154,7 +163,10 @@ export async function seedLastUpdateId(
   if (lastUpdateId !== 0) return lastUpdateId;
   const token = getToken();
   try {
-    const res = await fetch(`${BASE_URL}${token}/getUpdates?timeout=0`, {
+    const res = await fetch(`${BASE_URL}${token}/getUpdates`, {
+      method: "POST",
+      headers: { "Content-Type": "application/json" },
+      body: JSON.stringify({ timeout: 0 }),
       signal,
     });
     if (res.ok) {
@@ -184,8 +196,13 @@ export async function pollUpdates(
   while (!signal.aborted) {
     try {
       const res = await fetch(
-        `${BASE_URL}${token}/getUpdates?offset=${lastUpdateId + 1}&timeout=10`,
-        { signal },
+        `${BASE_URL}${token}/getUpdates`,
+        {
+          method: "POST",
+          headers: { "Content-Type": "application/json" },
+          body: JSON.stringify({ offset: lastUpdateId + 1, timeout: 10 }),
+          signal,
+        },
       );
       if (!res.ok) {
         await new Promise((r) => setTimeout(r, 3000));

package/src/index.ts

@@ -102,7 +102,12 @@ export default function telegramBridge(pi: ExtensionAPI) {
         if (!botId) botId = await getBotId();
 
         const res = await fetch(
-          `https://api.telegram.org/bot${token}/getUpdates?offset=${lastUpdateId + 1}&timeout=5`,
+          `https://api.telegram.org/bot${token}/getUpdates`,
+          {
+            method: "POST",
+            headers: { "Content-Type": "application/json" },
+            body: JSON.stringify({ offset: lastUpdateId + 1, timeout: 5 }),
+          },
         );
         if (!res.ok)
           return {