@openhands/extensions 0.5.0 → 0.7.0

package/tests/test_slack_channel_monitor.py

@@ -0,0 +1,142 @@
+import os
+from pathlib import Path
+
+
+SCRIPT_PATH = (
+    Path(__file__).parent.parent
+    / "skills"
+    / "slack-channel-monitor"
+    / "scripts"
+    / "main.py"
+)
+
+
+def load_slack_monitor_helpers():
+    os.environ["WORKSPACE_BASE"] = "/tmp/openhands/workspaces/slack-monitor-test/run"
+    source = SCRIPT_PATH.read_text(encoding="utf-8")
+    helper_source = source.split("\nPOLL_ITERATIONS = 10", 1)[0]
+    namespace: dict = {}
+    exec(compile(helper_source, str(SCRIPT_PATH), "exec"), namespace)
+    return namespace
+
+
+def test_post_message_sends_markdown_text(monkeypatch):
+    helpers = load_slack_monitor_helpers()
+    posted: dict = {}
+
+    def fake_slack_post(token: str, endpoint: str, body: dict) -> dict:
+        posted["token"] = token
+        posted["endpoint"] = endpoint
+        posted["body"] = body
+        return {"ts": "123.456"}
+
+    monkeypatch.setitem(helpers, "slack_post", fake_slack_post)
+    markdown_summary = "✅ Done!\n\n- **Bold:** [link](https://example.com)"
+
+    ts = helpers["post_message"](
+        "xoxb-test",
+        "C123",
+        markdown_summary,
+        thread_ts="111.222",
+    )
+
+    assert ts == "123.456"
+    assert posted["endpoint"] == "chat.postMessage"
+    assert posted["body"] == {
+        "channel": "C123",
+        "markdown_text": markdown_summary,
+        "unfurl_links": False,
+        "unfurl_media": False,
+        "thread_ts": "111.222",
+    }
+    assert "text" not in posted["body"]
+    assert "blocks" not in posted["body"]
+    assert "mrkdwn" not in posted["body"]
+
+
+def test_followup_quiet_poll_is_capped_at_watch_expiry(monkeypatch):
+    helpers = load_slack_monitor_helpers()
+    monkeypatch.setattr(helpers["time"], "time", lambda: 950.0)
+    calls: list[tuple[str, str, str]] = []
+
+    def fake_thread_replies(
+        token: str,
+        channel: str,
+        thread_ts: str,
+        oldest: str,
+    ) -> list[dict]:
+        calls.append((channel, thread_ts, oldest))
+        return []
+
+    monkeypatch.setitem(helpers, "thread_replies", fake_thread_replies)
+    rec = {
+        "channel_id": "C123",
+        "thread_ts": "900.000000",
+        "status": "watching",
+        "last_seen_reply_ts": "900.000000",
+        "reply_poll_backoff_seconds": 80,
+        "next_reply_poll_at": 950.0,
+        "watch_until": 1000.0,
+    }
+
+    replies = helpers["_poll_due_thread_replies"](
+        "xoxb-test",
+        {"C123:900.000000": rec},
+        "UBOT",
+        [],
+    )
+
+    assert replies == []
+    assert calls == [("C123", "900.000000", "900.000000")]
+    assert rec["status"] == "watching"
+    assert rec["next_reply_poll_at"] == 1000.0
+
+
+def test_expired_followup_watch_polls_once_before_closing(monkeypatch):
+    helpers = load_slack_monitor_helpers()
+    monkeypatch.setattr(helpers["time"], "time", lambda: 1001.0)
+    trigger = helpers["TRIGGER_PHRASE"]
+    reply = {
+        "ts": "999.500000",
+        "user": "U1",
+        "text": f"{trigger} please continue",
+    }
+    calls: list[tuple[str, str, str]] = []
+
+    def fake_thread_replies(
+        token: str,
+        channel: str,
+        thread_ts: str,
+        oldest: str,
+    ) -> list[dict]:
+        calls.append((channel, thread_ts, oldest))
+        return [reply]
+
+    monkeypatch.setitem(helpers, "thread_replies", fake_thread_replies)
+    rec = {
+        "channel_id": "C123",
+        "thread_ts": "900.000000",
+        "status": "watching",
+        "last_seen_reply_ts": "900.000000",
+        "reply_poll_backoff_seconds": 160,
+        "next_reply_poll_at": 1100.0,
+        "watch_until": 1000.0,
+    }
+
+    replies = helpers["_poll_due_thread_replies"](
+        "xoxb-test",
+        {"C123:900.000000": rec},
+        "UBOT",
+        [],
+    )
+
+    assert calls == [("C123", "900.000000", "900.000000")]
+    assert replies == [("C123", reply)]
+    assert rec["last_seen_reply_ts"] == "999.500000"
+    assert (
+        rec["reply_poll_backoff_seconds"]
+        == helpers["THREAD_REPLY_INITIAL_BACKOFF_SECONDS"]
+    )
+    assert rec["next_reply_poll_at"] == 1006.0
+    assert rec["watch_until"] == 1301.0
+

package/tests/test_version_alignment.py

@@ -0,0 +1,64 @@
+"""Assert the JS (``package.json``) and Python (``pyproject.toml`` +
+``_version.py``) package versions never drift apart.
+
+``release-please`` is configured to bump both ``package.json`` and
+``pyproject.toml`` together (see ``release-please-config.json`` ->
+``extra-files``). This test is the guard that catches a manual edit that
+bumps only one.
+"""
+
+from __future__ import annotations
+
+import re
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parents[1]
+
+
+def _package_json_version() -> str:
+    text = (ROOT / "package.json").read_text()
+    match = re.search(r'"version"\s*:\s*"([^"]+)"', text)
+    assert match, "package.json has no version field"
+    return match.group(1)
+
+
+def _pyproject_version() -> str:
+    text = (ROOT / "pyproject.toml").read_text()
+    match = re.search(r'^version\s*=\s*"([^"]+)"', text, re.MULTILINE)
+    assert match, "pyproject.toml has no version field"
+    return match.group(1)
+
+
+def _version_module_version() -> str:
+    import openhands_extensions._version as v
+
+    text = (ROOT / "python" / "openhands_extensions" / "_version.py").read_text()
+    match = re.search(r'_FALLBACK_VERSION\s*=\s*"([^"]+)"', text)
+    assert match, "_version.py has no _FALLBACK_VERSION"
+    fallback = match.group(1)
+    # Runtime __version__ is metadata-derived when installed, fallback otherwise;
+    # both must agree with the declared versions.
+    assert v.__version__ in (fallback, _package_json_version()), (
+        f"runtime __version__ {v.__version__!r} disagrees with fallback/package.json"
+    )
+    return fallback
+
+
+def test_package_json_and_pyproject_versions_match() -> None:
+    assert _package_json_version() == _pyproject_version(), (
+        "package.json and pyproject.toml versions differ. release-please "
+        "should bump both together; check release-please-config.json."
+    )
+
+
+def test_version_module_matches_package_json() -> None:
+    assert _version_module_version() == _package_json_version(), (
+        "python/openhands_extensions/_version.py is out of sync with "
+        "package.json."
+    )
+
+
+def test_installed_package_version_matches_source() -> None:
+    import openhands_extensions
+
+    assert openhands_extensions.__version__ == _package_json_version()

package/integrations/logos.d.ts

@@ -1,5 +0,0 @@
-import type { ReactNode } from "react";
-
-export const INTEGRATION_FALLBACK_LOGO: ReactNode;
-export const INTEGRATION_LOGOS: Record<string, ReactNode>;
-export const INTEGRATION_LOGO_IDS: Set<string>;

package/integrations/logos.js

@@ -1,94 +0,0 @@
-import { createElement } from "react";
-import {
-  BookOpen,
-  Bot,
-  Brain,
-  Clock,
-  Database,
-  Flame,
-  Folder,
-  GitBranch,
-  Globe,
-  ListTree,
-  MousePointerClick,
-  Search,
-  Sparkles,
-  Telescope,
-  TestTube,
-} from "lucide-react";
-import {
-  SiAirtable,
-  SiAtlassian,
-  SiBrave,
-  SiClickhouse,
-  SiCloudflare,
-  SiElevenlabs,
-  SiFigma,
-  SiGithub,
-  SiHuggingface,
-  SiKagi,
-  SiLinear,
-  SiMongodb,
-  SiNotion,
-  SiObsidian,
-  SiPaypal,
-  SiRedis,
-  SiResend,
-  SiSentry,
-  SiSlack,
-  SiStripe,
-  SiSupabase,
-} from "react-icons/si";
-
-const LOGO = "h-5 w-5";
-
-const simpleIcon = (Icon) => createElement(Icon, { className: LOGO });
-const lucideIcon = (Icon) =>
-  createElement(Icon, { className: LOGO, strokeWidth: 2.25 });
-
-export const INTEGRATION_FALLBACK_LOGO = lucideIcon(Bot);
-
-export const INTEGRATION_LOGOS = {
-  github: simpleIcon(SiGithub),
-  slack: simpleIcon(SiSlack),
-  tavily: createElement(Search, { className: LOGO, strokeWidth: 2.5 }),
-  linear: simpleIcon(SiLinear),
-  notion: simpleIcon(SiNotion),
-  atlassian: simpleIcon(SiAtlassian),
-  sentry: simpleIcon(SiSentry),
-  stripe: simpleIcon(SiStripe),
-  paypal: simpleIcon(SiPaypal),
-  "cloudflare-docs": simpleIcon(SiCloudflare),
-  "cloudflare-bindings": simpleIcon(SiCloudflare),
-  "cloudflare-observability": simpleIcon(SiCloudflare),
-  huggingface: simpleIcon(SiHuggingface),
-  deepwiki: simpleIcon(BookOpen),
-  git: lucideIcon(GitBranch),
-  "brave-search": simpleIcon(SiBrave),
-  exa: lucideIcon(Telescope),
-  firecrawl: lucideIcon(Flame),
-  apify: lucideIcon(Bot),
-  fetch: lucideIcon(Globe),
-  "browser-mcp": lucideIcon(MousePointerClick),
-  playwright: lucideIcon(TestTube),
-  supabase: simpleIcon(SiSupabase),
-  neon: lucideIcon(Database),
-  mongodb: simpleIcon(SiMongodb),
-  redis: simpleIcon(SiRedis),
-  filesystem: lucideIcon(Folder),
-  memory: lucideIcon(Brain),
-  "sequential-thinking": lucideIcon(ListTree),
-  time: lucideIcon(Clock),
-  everything: lucideIcon(Sparkles),
-  figma: simpleIcon(SiFigma),
-  airtable: simpleIcon(SiAirtable),
-  obsidian: simpleIcon(SiObsidian),
-  elevenlabs: simpleIcon(SiElevenlabs),
-  resend: simpleIcon(SiResend),
-  "cloudflare-builds": simpleIcon(SiCloudflare),
-  "cloudflare-browser-rendering": simpleIcon(SiCloudflare),
-  kagi: simpleIcon(SiKagi),
-  clickhouse: simpleIcon(SiClickhouse),
-};
-
-export const INTEGRATION_LOGO_IDS = new Set(Object.keys(INTEGRATION_LOGOS));