better-dsh 0.2.5 → 0.3.0-a

package/docs/80_upstream-docs-and-skills/hermes-dev-skill/src/docs/developer-guide/adding-platform-adapters.md

@@ -1,788 +0,0 @@
----
-sidebar_position: 9
----
-
-# Adding a Platform Adapter
-
-This guide covers adding a new messaging platform to the Hermes gateway. A platform adapter connects Hermes to an external messaging service (Telegram, Discord, WeCom, etc.) so users can interact with the agent through that service.
-
-:::tip
-There are two ways to add a platform:
-- **Plugin** (recommended for community/third-party): Drop a plugin directory into `~/.hermes/plugins/` — zero core code changes needed. See [Plugin Path](#plugin-path-recommended) below.
-- **Built-in**: Modify 20+ files across code, config, and docs. Use the [Built-in Checklist](#step-by-step-checklist-built-in-path) below.
-:::
-
-## Architecture Overview
-
-```
-User ↔ Messaging Platform ↔ Platform Adapter ↔ Gateway Runner ↔ AIAgent
-```
-
-Every adapter extends `BasePlatformAdapter` from `gateway/platforms/base.py` and implements:
-
-- **`connect()`** — Establish connection (WebSocket, long-poll, HTTP server, etc.) *(abstract)*
-- **`disconnect()`** — Clean shutdown *(abstract)*
-- **`send()`** — Send a text message to a chat *(abstract)*
-- **`send_typing()`** — Show typing indicator (optional override)
-- **`get_chat_info()`** — Return chat metadata (optional override)
-
-Inbound messages are received by the adapter and forwarded via `self.handle_message(event)`, which the base class routes to the gateway runner.
-
-## Plugin Path (Recommended)
-
-The plugin system lets you add a platform adapter without modifying any core Hermes code. Your plugin is a directory with two files:
-
-```
-~/.hermes/plugins/my-platform/
-  plugin.yaml      # Plugin metadata
-  adapter.py       # Adapter class + register() entry point
-```
-
-### plugin.yaml
-
-Plugin metadata. The `requires_env` and `optional_env` blocks auto-populate `hermes config` UI entries (see [Surfacing Env Vars](#surfacing-env-vars-in-hermes-config) below).
-
-```yaml
-name: my-platform
-label: My Platform
-kind: platform
-version: 1.0.0
-description: My custom messaging platform adapter
-author: Your Name
-requires_env:
-  - MY_PLATFORM_TOKEN          # bare string works
-  - name: MY_PLATFORM_CHANNEL  # or rich dict for better UX
-    description: "Channel to join"
-    prompt: "Channel"
-    password: false
-optional_env:
-  - name: MY_PLATFORM_HOME_CHANNEL
-    description: "Default channel for cron delivery"
-    password: false
-```
-
-#### Outbound client tools: `provides_tools`
-
-`kind: platform` plugins are **deferred**: the adapter module (and its SDK
-imports) only load when a gateway, cron, or `send_message` path first asks the
-platform registry for the platform. If your plugin also ships outbound *client
-tools* the agent should be able to call from any session (the bundled `a2a`
-plugin's `a2a_call` / `a2a_discover` etc.), put them in a dedicated `tools.py`
-with a `register_tools(ctx)` function and declare them in the manifest:
-
-```yaml
-provides_tools:
-  - my_platform_call
-  - my_platform_list
-```
-
-With `provides_tools` declared, Hermes imports only `tools.py` during plugin
-discovery and registers the client tools in every process — CLI and TUI
-included — while the adapter stays deferred. Keep the package `__init__.py`
-import-light and pull the adapter in from inside `register()` so the eager
-import stays cheap. Without the field, nothing changes: the whole plugin stays
-deferred.
-
-Users enable the toolset per platform like any other, e.g.
-`hermes tools enable my_platform --platform cli`, or by listing the toolset
-key under `platform_toolsets` in `config.yaml`. Plugin platform names are
-also valid `--platform` targets, so an inbound session on your platform can
-be granted its own outbound tools.
-
-### adapter.py
-
-```python
-from gateway.platforms._shared import extra_or_secret, get_scoped_secret, seed_extra_from_env
-from gateway.platforms.base import BasePlatformAdapter, SendResult
-from gateway.platforms.event import MessageEvent, MessageType
-from gateway.config import Platform, PlatformConfig
-
-
-class MyPlatformAdapter(BasePlatformAdapter):
-    def __init__(self, config: PlatformConfig):
-        super().__init__(config, Platform("my_platform"))
-        # Explicit env (profile-scoped) → this profile's config.extra (YAML) → default. Under multiplexing a
-        # scoped miss falls to the profile's own YAML, never to another profile's process env.
-        self.token = extra_or_secret(config.extra, "token", "MY_PLATFORM_TOKEN")
-
-    async def connect(self, *, is_reconnect: bool = False) -> bool:
-        # Connect to the platform API, start listeners
-        self._mark_connected()
-        return True
-
-    async def disconnect(self) -> None:
-        self._mark_disconnected()
-
-    async def send(self, chat_id, content, reply_to=None, metadata=None):
-        # Send message via platform API
-        return SendResult(success=True, message_id="...")
-
-    async def get_chat_info(self, chat_id):
-        return {"name": chat_id, "type": "dm"}
-
-
-def check_requirements() -> bool:
-    return bool(get_scoped_secret("MY_PLATFORM_TOKEN"))
-
-
-def validate_config(config) -> bool:
-    return bool(extra_or_secret(getattr(config, "extra", None), "token", "MY_PLATFORM_TOKEN"))
-
-
-def _env_enablement() -> dict | None:
-    if not get_scoped_secret("MY_PLATFORM_TOKEN", "").strip():
-        return None
-    return seed_extra_from_env(
-        (("MY_PLATFORM_TOKEN", "token", None), ("MY_PLATFORM_CHANNEL", "channel", None)),
-        home_env="MY_PLATFORM_HOME_CHANNEL",
-    )
-
-
-def register(ctx):
-    """Plugin entry point — called by the Hermes plugin system."""
-    ctx.register_platform(
-        name="my_platform",
-        label="My Platform",
-        adapter_factory=MyPlatformAdapter,
-        # PASSIVE probe — "are deps/config present right now?".  Called from
-        # status displays and config loading, so it must NEVER pip-install.
-        check_fn=check_requirements,
-        # ACTIVE installer (optional) — only for platforms with a
-        # lazy-installable SDK.  create_adapter() calls it when check_fn
-        # returns False, right before the gateway connects the platform.
-        # Typically wraps tools.lazy_deps.ensure_and_bind(...).  Omit it
-        # and a False check_fn is a hard block.
-        # ensure_deps_fn=ensure_requirements,
-        validate_config=validate_config,
-        required_env=["MY_PLATFORM_TOKEN"],
-        install_hint="pip install my-platform-sdk",
-        # Env-driven auto-configuration — seeds PlatformConfig.extra from
-        # env vars before adapter construction. See "Env-Driven Auto-
-        # Configuration" section below.
-        env_enablement_fn=_env_enablement,
-        # Cron home-channel delivery support. Lets deliver=my_platform cron
-        # jobs route without editing cron/scheduler.py. See "Cron Delivery"
-        # section below.
-        cron_deliver_env_var="MY_PLATFORM_HOME_CHANNEL",
-        # Per-platform user authorization env vars
-        allowed_users_env="MY_PLATFORM_ALLOWED_USERS",
-        allow_all_env="MY_PLATFORM_ALLOW_ALL_USERS",
-        # Message length limit for smart chunking (0 = no limit)
-        max_message_length=4000,
-        # LLM guidance injected into system prompt
-        platform_hint=(
-            "You are chatting via My Platform. "
-            "It supports markdown formatting."
-        ),
-        # Display
-        emoji="💬",
-    )
-
-    # Optional: register platform-specific tools
-    ctx.register_tool(
-        name="my_platform_search",
-        toolset="my_platform",
-        schema={...},
-        handler=my_search_handler,
-    )
-```
-
-### Configuration
-
-Users configure the platform in `config.yaml`:
-
-```yaml
-gateway:
-  platforms:
-    my_platform:
-      enabled: true
-      extra:
-        token: "..."
-        channel: "#general"
-```
-
-Or via environment variables (which the adapter reads in `__init__`).
-
-### What the Plugin System Handles Automatically
-
-When you call `ctx.register_platform()`, the following integration points are handled for you — no core code changes needed:
-
-| Integration point | How it works |
-|---|---|
-| Gateway adapter creation | Registry checked before the built-in `_BUILTIN_ADAPTERS` table |
-| Config parsing | `Platform._missing_()` accepts any platform name |
-| Connected platform validation | Registry `validate_config()` called |
-| User authorization | `allowed_users_env` / `allow_all_env` checked |
-| Env-only auto-enable | `env_enablement_fn` seeds `PlatformConfig.extra` + `home_channel` |
-| YAML config bridge | `apply_yaml_config_fn` translates `config.yaml` keys into env vars / extras |
-| Cron delivery | `cron_deliver_env_var` makes `deliver=<name>` work |
-| `hermes config` UI entries | `requires_env` / `optional_env` in `plugin.yaml` auto-populate |
-| send engine (`tools/send_message_tool.py`) | Routes through live gateway adapter |
-| Webhook cross-platform delivery | Registry checked for known platforms |
-| `/update` command access | `allow_update_command` flag |
-| Channel directory | Plugin platforms included in enumeration |
-| System prompt hints | `platform_hint` injected into LLM context |
-| Message chunking | `max_message_length` for smart splitting |
-| PII redaction | `pii_safe` flag |
-| `hermes status` | Shows plugin platforms with `(plugin)` tag |
-| `hermes gateway setup` | Plugin platforms appear in setup menu |
-| `hermes tools` / `hermes skills` | Plugin platforms in per-platform config |
-| Token lock (multi-profile) | Use `acquire_scoped_lock()` in your `connect()` |
-| Orphaned config warning | Descriptive log when plugin is missing |
-
-## Standalone send-path extensions
-
-A standalone platform can participate in host-driven outbound delivery through
-direct `hermes send --to ...` and cron `deliver=platform:...` by declaring send
-behavior on the same `PlatformEntry` created by `ctx.register_platform()`.
-`send_message` is intentionally not an agent-callable model tool; plugins must
-not register an equivalent model surface that lets the agent initiate outbound
-messages on its own.
-
-```python
-async def _send_request(args, chat_id, platform_name, pconfig):
-    # `args` contains the host-driven send request fields.
-    message_id = await client.send(
-        address=chat_id,
-        body=args["message"],
-        subject=args.get("subject"),
-    )
-    return {"success": True, "platform": platform_name,
-            "chat_id": chat_id, "message_id": message_id}
-
-
-def _parse_address(raw):
-    normalized = raw.strip().lower()
-    if normalized.startswith("@") and "@" in normalized[1:]:
-        return normalized, None  # (chat_id, optional thread_id)
-    return None                 # continue to channel-directory resolution
-
-
-def _validate_address(address):
-    # True accepts; False rejects; a string rejects with that diagnostic.
-    return True if address.endswith("@example.com") else "unsupported domain"
-
-
-def register(ctx):
-    ctx.register_platform(
-        name="fmsg",
-        label="Fixture Message",
-        adapter_factory=lambda cfg: FmsgAdapter(cfg),
-        check_fn=check_requirements,
-        parse_target_ref_fn=_parse_address,
-        validate_target_ref_fn=_validate_address,
-        # May be a regular function or async def. Hermes awaits any awaitable
-        # result, including callable objects and functools.partial wrappers.
-        send_message_handler=_send_request,
-        # Prefer this lower-level hook when cron must send from a process
-        # without the live gateway.
-        standalone_sender_fn=_standalone_send,
-    )
-```
-
-Target resolution is shared across all three outbound surfaces. Parser output
-is normalized first and channel-directory IDs are trusted. A plugin parser must
-explicitly accept native target syntax; unresolved strings are never passed
-through opaquely. Unknown platforms and validator failures return a diagnostic
-instead of silently attempting delivery. Plugin force-reload/profile
-transitions unregister owned entries, so parsers and handlers cannot leak into
-the next profile.
-
-## Env-Driven Auto-Configuration
-
-Most users set up a platform by dropping env vars into `~/.hermes/.env` rather than editing `config.yaml`. The `env_enablement_fn` hook lets your plugin pick those env vars up **before** the adapter is constructed, so `hermes gateway status`, `get_connected_platforms()`, and cron delivery see the correct state without instantiating the platform SDK.
-
-Read env through `gateway.platforms._shared.get_scoped_secret` — never `os.getenv`. Under `gateway.multiplex_profiles` the process env holds the DEFAULT profile's values; a secondary profile's `.env` exists only in its secret scope, and a raw read would enable your platform for the wrong profile with the wrong credentials. `seed_extra_from_env` builds the seed dict from a `(ENV_VAR, extra_key, conv)` table through that reader.
-
-```python
-from gateway.platforms._shared import get_scoped_secret, seed_extra_from_env
-
-
-def _env_enablement() -> dict | None:
-    """Seed PlatformConfig.extra from env vars.
-
-    Called by the platform registry during load_gateway_config().
-    Return None when the platform isn't minimally configured — the
-    caller then skips auto-enabling. Return a dict to seed extras.
-    """
-    token = get_scoped_secret("MY_PLATFORM_TOKEN", "").strip()
-    channel = get_scoped_secret("MY_PLATFORM_CHANNEL", "").strip()
-    if not (token and channel):
-        return None
-    # (ENV_VAR, extra_key, conv): blank values are skipped, a conv raising
-    # ValueError drops the key. home_env seeds the special 'home_channel'
-    # dict ({chat_id, name} from MY_PLATFORM_HOME_CHANNEL[_NAME], name
-    # defaults to "Home"), which the core hook lifts into a HomeChannel.
-    return {"token": token, "channel": channel, **seed_extra_from_env(
-        (("MY_PLATFORM_PORT", "port", int), ("MY_PLATFORM_NICK", "nick", None)),
-        home_env="MY_PLATFORM_HOME_CHANNEL",
-    )}
-
-
-def register(ctx):
-    ctx.register_platform(
-        name="my_platform",
-        label="My Platform",
-        adapter_factory=MyPlatformAdapter,
-        check_fn=check_requirements,
-        validate_config=validate_config,
-        env_enablement_fn=_env_enablement,
-        # ... other fields
-    )
-```
-
-
-## YAML→env Config Bridge
-
-Some users prefer setting `config.yaml` keys (`my_platform.require_mention`, `my_platform.allowed_channels`, etc.) over env vars. The `apply_yaml_config_fn` hook lets your plugin own this translation instead of forcing core `gateway/config.py` to know your platform's YAML schema.
-
-```python
-from gateway.platforms._shared import apply_yaml_bridge
-
-# (yaml key, ENV_VAR, kind). "lower" bridges whenever the key is present
-# (booleans become "true"/"false"); "str" skips null/blank; "csv" comma-joins
-# lists; "json" dumps the value.
-_YAML_BRIDGE = (
-    ("require_mention", "MY_PLATFORM_REQUIRE_MENTION", "lower"),
-    ("allowed_channels", "MY_PLATFORM_ALLOWED_CHANNELS", "csv"),
-)
-
-
-def _apply_yaml_config(yaml_cfg: dict, platform_cfg: dict) -> dict | None:
-    """Translate config.yaml `my_platform:` keys into env vars / extras.
-
-    yaml_cfg     — the full top-level parsed config.yaml dict
-    platform_cfg — the platform's own sub-dict (yaml_cfg.get("my_platform", {}))
-
-    apply_yaml_bridge writes each env var only when it is unset (env > YAML)
-    and NEVER while a multiplexed secondary profile's config is loading — a
-    write there would pin that profile's policy process-wide. It returns the
-    same values for PlatformConfig.extra, so read `extra` first in the adapter
-    and fall back to the env var (`_shared.extra_or_secret`).
-    """
-    return apply_yaml_bridge(platform_cfg, _YAML_BRIDGE)
-
-def register(ctx):
-    ctx.register_platform(
-        name="my_platform",
-        ...,
-        apply_yaml_config_fn=_apply_yaml_config,
-    )
-```
-
-The hook is invoked during `load_gateway_config()` after the generic shared-key loop (which handles common keys like `unauthorized_dm_behavior`, `notice_delivery`, `reply_prefix`, `require_mention`, etc.) and before `_apply_env_overrides()`, so your plugin only needs to bridge **platform-specific** keys.
-
-Exceptions raised by the hook are swallowed and logged at debug level — a misbehaving plugin never aborts gateway config load.
-
-
-## Cron Delivery
-
-To let `deliver=my_platform` cron jobs route to a configured home channel, set `cron_deliver_env_var` to the env var name that holds the default chat/room/channel ID:
-
-```python
-ctx.register_platform(
-    name="my_platform",
-    ...
-    cron_deliver_env_var="MY_PLATFORM_HOME_CHANNEL",
-)
-```
-
-The scheduler reads this env var when resolving the home target for `deliver=my_platform` jobs, and also treats the platform as a valid cron target in `_KNOWN_DELIVERY_PLATFORMS`-style checks. If your `env_enablement_fn` seeds a `home_channel` dict (see above), that takes precedence — `cron_deliver_env_var` is the fallback for cron jobs that run before env seeding.
-
-### Out-of-process cron delivery
-
-`cron_deliver_env_var` makes your platform a recognized `deliver=` target. To make the actual send succeed when the cron job runs in a separate process from the gateway (i.e., `hermes cron run` separate from `hermes gateway`), register a `standalone_sender_fn`:
-
-```python
-async def _standalone_send(
-    pconfig,
-    chat_id,
-    message,
-    *,
-    thread_id=None,
-    media_files=None,
-    force_document=False,
-):
-    """Open an ephemeral connection / acquire a fresh token, send, and close."""
-    # ... open connection, send message, return result ...
-    return {"success": True, "message_id": "..."}
-    # or {"error": "..."}
-
-ctx.register_platform(
-    name="my_platform",
-    ...
-    cron_deliver_env_var="MY_PLATFORM_HOME_CHANNEL",
-    standalone_sender_fn=_standalone_send,
-)
-```
-
-Why this hook is necessary: built-in platforms (Telegram, Discord, Slack, etc.) ship direct REST helpers in `tools/send_message_tool.py` so cron can deliver without holding the gateway in the same process. Plugin platforms historically depended on `_gateway_runner_ref()`, which returns `None` outside the gateway process, so without `standalone_sender_fn` the cron-side send fails with `No live adapter for platform '<name>'`.
-
-The function receives the same `pconfig` and `chat_id` that the live adapter would, plus optional `thread_id`, `media_files`, and `force_document` keyword arguments. Returning `{"success": True, "message_id": ...}` is treated as a successful delivery; returning `{"error": "..."}` surfaces the message in cron's `delivery_errors`. Exceptions raised inside the function are caught by the dispatcher and reported as `Plugin standalone send failed: <reason>`. Reference implementations live in `plugins/platforms/{irc,teams,google_chat}/adapter.py`.
-
-## Surfacing Env Vars in `hermes config`
-
-`hermes_cli/config.py` scans `plugins/platforms/*/plugin.yaml` at import time and auto-populates `OPTIONAL_ENV_VARS` from `requires_env` and (optional) `optional_env` blocks. Use the rich-dict form to contribute proper descriptions, prompts, password flags, and URLs — the CLI setup UI picks them up for free.
-
-```yaml
-# plugins/platforms/my_platform/plugin.yaml
-name: my_platform-platform
-label: My Platform
-kind: platform
-version: 1.0.0
-description: >
-  My Platform gateway adapter for Hermes Agent.
-author: Your Name
-requires_env:
-  - name: MY_PLATFORM_TOKEN
-    description: "Bot API token from the My Platform console"
-    prompt: "My Platform bot token"
-    url: "https://my-platform.example.com/bots"
-    password: true
-  - name: MY_PLATFORM_CHANNEL
-    description: "Channel to join (e.g. #hermes)"
-    prompt: "Channel"
-    password: false
-optional_env:
-  - name: MY_PLATFORM_HOME_CHANNEL
-    description: "Default channel for cron delivery (defaults to MY_PLATFORM_CHANNEL)"
-    prompt: "Home channel (or empty)"
-    password: false
-  - name: MY_PLATFORM_ALLOWED_USERS
-    description: "Comma-separated user IDs allowed to talk to the bot"
-    prompt: "Allowed users (comma-separated)"
-    password: false
-```
-
-**Supported dict keys:** `name` (required), `description`, `prompt`, `url`, `password` (bool; auto-detected from `*_TOKEN` / `*_SECRET` / `*_KEY` / `*_PASSWORD` / `*_JSON` suffix when omitted), `category` (defaults to `"messaging"`).
-
-Bare-string entries (`- MY_PLATFORM_TOKEN`) still work — they get a generic description auto-derived from the plugin's `label`. If a hardcoded entry for the same var already exists in `OPTIONAL_ENV_VARS`, it wins (back-compat); the plugin.yaml form acts as the fallback.
-
-## Platform-Specific Slow-LLM UX
-
-Some platforms have constraints that change how a slow LLM response should be presented:
-
-- **LINE** issues a single-use *reply token* that expires roughly 60 seconds after the inbound event. Replying with that token is free; falling back to the metered Push API is not. If the LLM hasn't finished by the deadline, the choice is "burn paid Push quota" or "do something cleverer with the reply token before it expires."
-- **WhatsApp** marks a session inactive after 24h, after which only template messages are accepted.
-- **SMS** has no concept of typing indicators or progressive updates — long responses just look like the bot is offline.
-
-These are real constraints the base `BasePlatformAdapter` can't anticipate. The plugin surface intentionally leaves the room for an adapter to layer platform-specific UX on top of the base typing loop without expanding the kwarg list.
-
-### Pattern: subclass `_keep_typing` to layer mid-flight UX
-
-`BasePlatformAdapter._keep_typing` is the typing-indicator heartbeat — it runs as a background task while the LLM is generating, and is cancelled when the response is delivered. To layer a platform-specific behavior at a threshold (e.g. send a "still thinking" bubble at 45s), override `_keep_typing` in your adapter, schedule your own task alongside `super()._keep_typing()`, and tear it down in `finally`:
-
-```python
-class LineAdapter(BasePlatformAdapter):
-    async def _keep_typing(self, chat_id: str, *args, **kwargs) -> None:
-        if self.slow_response_threshold <= 0:
-            await super()._keep_typing(chat_id, *args, **kwargs)
-            return
-
-        async def _fire_at_threshold() -> None:
-            try:
-                await asyncio.sleep(self.slow_response_threshold)
-            except asyncio.CancelledError:
-                raise
-            # Platform-specific work here — for LINE, send a Template
-            # Buttons "Get answer" bubble using the cached reply token
-            # so the user can fetch the cached response later via a
-            # fresh (free) reply token from the postback callback.
-            await self._send_slow_response_button(chat_id)
-
-        side_task = asyncio.create_task(_fire_at_threshold())
-        try:
-            await super()._keep_typing(chat_id, *args, **kwargs)
-        finally:
-            if not side_task.done():
-                side_task.cancel()
-                try:
-                    await side_task
-                except (asyncio.CancelledError, Exception):
-                    pass
-```
-
-Key points:
-
-- **Always `await super()._keep_typing(...)`.** The typing heartbeat is independently useful — don't replace it, layer on top of it.
-- **Tear down the side task in `finally`.** When the LLM finishes (or `/stop` cancels the run), the gateway cancels the typing task. Your side task must observe that cancellation too, otherwise it lingers and may fire after the response was already delivered.
-- **Pair with `interrupt_session_activity`** to resolve any orphan UX state when the user issues `/stop`. For LINE, this means transitioning the postback cache entry from `PENDING` to `ERROR` so the persistent "Get answer" button delivers a "Run was interrupted" message instead of looping.
-
-### Pattern: subclass `send` to route through a cache instead of sending immediately
-
-If your slow-response UX caches the response for later retrieval (LINE's postback flow), your `send` override needs to recognize three modes:
-
-1. **Pending postback active for this chat** → cache the response under the request_id, don't send anything visible.
-2. **System busy-ack** (`⚡ Interrupting`, `⏳ Queued`, `⏩ Steered`) → bypass the cache and send visibly so the user sees the gateway's response to their input.
-3. **Normal response** → send via reply-token-or-push as usual.
-
-```python
-async def send(self, chat_id: str, content: str, **kw) -> SendResult:
-    if _is_system_bypass(content):
-        return await self._send_text_chunks(chat_id, content, force_push=False)
-    pending_rid = self._pending_buttons.get(chat_id)
-    if pending_rid:
-        self._cache.set_ready(pending_rid, content)
-        return SendResult(success=True, message_id=pending_rid)
-    return await self._send_text_chunks(chat_id, content, force_push=False)
-```
-
-`_SYSTEM_BYPASS_PREFIXES` are the gateway's own busy-acknowledgment prefixes (`⚡`, `⏳`, `⏩`, `💾`). Always let those through visibly, regardless of cached UX state.
-
-### When this pattern is appropriate
-
-Use the typing-loop override approach when:
-
-- The platform's outbound API has a hard time-window constraint (single-use reply token, expiring sticky session, etc.) AND
-- A *visible mid-flight bubble* is acceptable UX on that platform.
-
-Use the simpler `slow_response_threshold = 0` always-Push path when:
-
-- The platform doesn't have a meaningful free vs. paid distinction, OR
-- The user community prefers "loading… loading… DONE" silence-then-response over an interactive intermediate bubble.
-
-LINE supports both: the threshold defaults to 45s for free postback fetch, and `LINE_SLOW_RESPONSE_THRESHOLD=0` reverts to "always Push fallback."
-
-### Reference Implementation
-
-See `plugins/platforms/line/adapter.py` for the full LINE postback implementation — a `RequestCache` state machine (`PENDING → READY → DELIVERED`, plus `ERROR` for `/stop`), a `_keep_typing` override that fires the Template Buttons bubble at threshold, a `send` override that routes through the cache, and an `interrupt_session_activity` override that resolves orphan PENDING entries.
-
-### Reference Implementations (Plugin Path)
-
-See `plugins/platforms/irc/` in the repo for a complete working example — a full async IRC adapter with zero external dependencies. `plugins/platforms/teams/` covers Bot Framework / Adaptive Cards, `plugins/platforms/google_chat/` covers OAuth-based REST APIs, and `plugins/platforms/line/` covers webhook-driven Messaging APIs with platform-specific slow-LLM UX.
-
----
-
-## Step-by-Step Checklist (Built-in Path)
-
-:::note
-This checklist is for adding a platform directly to the Hermes core codebase — typically done by core contributors for officially supported platforms. Community/third-party platforms should use the [Plugin Path](#plugin-path-recommended) above.
-:::
-
-### 1. Platform Enum
-
-Add your platform to the `Platform` enum in `gateway/config.py`:
-
-```python
-class Platform(Enum):
-    # ... existing platforms ...
-    NEWPLAT = "newplat"
-```
-
-### 2. Adapter File
-
-Create `plugins/platforms/newplat/adapter.py`:
-
-```python
-from gateway.config import Platform, PlatformConfig
-from gateway.platforms._shared import extra_or_secret
-from gateway.platforms.base import BasePlatformAdapter, SendResult
-from gateway.platforms.event import MessageEvent, MessageType
-
-def check_newplat_requirements() -> bool:
-    """Return True if dependencies are available."""
-    return SOME_SDK_AVAILABLE
-
-class NewPlatAdapter(BasePlatformAdapter):
-    def __init__(self, config: PlatformConfig):
-        super().__init__(config, Platform.NEWPLAT)
-        # Read config from config.extra dict
-        extra = config.extra or {}
-        self._api_key = extra_or_secret(extra, "api_key", "NEWPLAT_API_KEY")
-
-    async def connect(self, *, is_reconnect: bool = False) -> bool:
-        # Set up connection, start polling/webhook
-        self._mark_connected()
-        return True
-
-    async def disconnect(self) -> None:
-        self._running = False
-        self._mark_disconnected()
-
-    async def send(self, chat_id, content, reply_to=None, metadata=None):
-        # Send message via platform API
-        return SendResult(success=True, message_id="...")
-
-    async def get_chat_info(self, chat_id):
-        return {"name": chat_id, "type": "dm"}
-```
-
-For inbound messages, build a `MessageEvent` and call `self.handle_message(event)`:
-
-```python
-source = self.build_source(
-    chat_id=chat_id,
-    chat_name=name,
-    chat_type="dm",  # or "group"
-    user_id=user_id,
-    user_name=user_name,
-)
-event = MessageEvent(
-    text=content,
-    message_type=MessageType.TEXT,
-    source=source,
-    message_id=msg_id,
-)
-await self.handle_message(event)
-```
-
-### 3. Gateway Config (`gateway/config.py`)
-
-Three touchpoints:
-
-1. **`get_connected_platforms()`** — Add a check for your platform's required credentials
-2. **`load_gateway_config()`** — Add token env map entry: `Platform.NEWPLAT: "NEWPLAT_TOKEN"`
-3. **`_apply_env_overrides()`** — Map all `NEWPLAT_*` env vars to config
-
-### 4. Gateway Runner (`gateway/run.py` + `gateway/run_*.py` siblings)
-
-Six touchpoints:
-
-1. **`_BUILTIN_ADAPTERS` table** (`gateway/run.py`) — Add a `Platform.NEWPLAT: (module, class, check_fn, error_msg)` entry; `_instantiate_adapter()` (`gateway/run_adapters.py`) consults the plugin registry, then this table — there is no `elif` chain to extend. The `_create_adapter()` wrapper binds every successful adapter to its gateway runner.
-2. **`_is_user_authorized()` allowed_users map** — `Platform.NEWPLAT: "NEWPLAT_ALLOWED_USERS"`
-3. **`_is_user_authorized()` allow_all map** — `Platform.NEWPLAT: "NEWPLAT_ALLOW_ALL_USERS"`
-4. **Startup access-policy check** (`gateway/run_startup.py`) — Add `"NEWPLAT"` to `_ALLOWLIST_ENV_PLATFORMS` (derives both `NEWPLAT_ALLOWED_USERS` and `NEWPLAT_ALLOW_ALL_USERS`)
-5. **Startup `_BUILTIN_ALLOW_ALL_VARS`** (`gateway/run_startup.py`) — derived from the same `_ALLOWLIST_ENV_PLATFORMS` tuple; nothing extra to add
-6. **`_UPDATE_ALLOWED_PLATFORMS` frozenset** — Add `Platform.NEWPLAT`
-
-### 5. Cross-Platform Delivery
-
-1. **`gateway/platforms/webhook.py`** — Add `"newplat"` to the delivery type tuple
-2. **`cron/scheduler_delivery.py`** — Add to `_KNOWN_DELIVERY_PLATFORMS` frozenset and `_deliver_result()` platform map
-
-### 6. CLI Integration
-
-1. **`hermes_cli/config.py`** — Add all `NEWPLAT_*` vars to `_EXTRA_ENV_KEYS`
-2. **`hermes_cli/gateway.py`** — Add entry to `_PLATFORMS` list with key, label, emoji, token_var, setup_instructions, and vars
-3. **`hermes_cli/platforms.py`** — Add `PlatformInfo` entry with label and default_toolset (used by `skills_config` and `tools_config` TUIs)
-4. **`hermes_cli/setup.py`** — Add `_setup_newplat()` function (can delegate to `gateway.py`) and add tuple to the messaging platforms list
-5. **`hermes_cli/status.py`** — Add platform detection entry: `"NewPlat": ("NEWPLAT_TOKEN", "NEWPLAT_HOME_CHANNEL")`
-6. **`hermes_cli/dump.py`** — Add `"newplat": "NEWPLAT_TOKEN"` to platform detection dict
-
-### 7. Tools
-
-1. **`tools/send_message_tool.py`** — Add `"newplat": Platform.NEWPLAT` to platform map
-2. **`tools/cronjob_tools.py`** — Add `newplat` to the delivery target description string
-
-### 8. Toolsets
-
-1. **`toolsets.py`** — Add `"hermes-newplat"` toolset definition with `_HERMES_CORE_TOOLS`
-2. **`toolsets.py`** — Add `"hermes-newplat"` to the `"hermes-gateway"` includes list
-
-### 9. Optional: Platform Hints
-
-**`agent/prompt_builder.py`** — If your platform has specific rendering limitations (no markdown, message length limits, etc.), add an entry to the `PLATFORM_HINTS` dict. This injects platform-specific guidance into the system prompt:
-
-```python
-PLATFORM_HINTS = {
-    # ...
-    "newplat": (
-        "You are chatting via NewPlat. It supports markdown formatting "
-        "but has a 4000-character message limit."
-    ),
-}
-```
-
-Not all platforms need hints — only add one if the agent's behavior should differ.
-
-### 10. Tests
-
-Create `tests/gateway/test_newplat.py` covering:
-
-- Adapter construction from config
-- Message event building
-- Send method (mock the external API)
-- Platform-specific features (encryption, routing, etc.)
-
-### 11. Documentation
-
-| File | What to add |
-|------|-------------|
-| `website/docs/user-guide/messaging/newplat.md` | Full platform setup page |
-| `website/docs/user-guide/messaging/index.md` | Platform comparison table, architecture diagram, toolsets table, security section, next-steps link |
-| `website/docs/reference/environment-variables.md` | All NEWPLAT_* env vars |
-| `website/docs/reference/toolsets-reference.md` | hermes-newplat toolset |
-| `website/docs/integrations/index.md` | Platform link |
-| `website/sidebars.ts` | Sidebar entry for the docs page |
-| `website/docs/developer-guide/architecture.md` | Adapter count + listing |
-| `website/docs/developer-guide/gateway-internals.md` | Adapter file listing |
-
-## Parity Audit
-
-Before marking a new platform PR as complete, run a parity audit against an established platform:
-
-```bash
-# Find every .py file mentioning the reference platform
-search_files "bluebubbles" output_mode="files_only" file_glob="*.py"
-
-# Find every .py file mentioning the new platform
-search_files "newplat" output_mode="files_only" file_glob="*.py"
-
-# Any file in the first set but not the second is a potential gap
-```
-
-Repeat for `.md` and `.ts` files. Investigate each gap — is it a platform enumeration (needs updating) or a platform-specific reference (skip)?
-
-## Common Patterns
-
-### Long-Poll Adapters
-
-If your adapter uses long-polling (like Telegram or Weixin), use a polling loop task:
-
-```python
-async def connect(self):
-    self._poll_task = asyncio.create_task(self._poll_loop())
-    self._mark_connected()
-
-async def _poll_loop(self):
-    while self._running:
-        messages = await self._fetch_updates()
-        for msg in messages:
-            await self.handle_message(self._build_event(msg))
-```
-
-### Callback/Webhook Adapters
-
-If the platform pushes messages to your endpoint (like WeCom Callback), run an HTTP server:
-
-```python
-async def connect(self):
-    self._app = web.Application()
-    self._app.router.add_post("/callback", self._handle_callback)
-    # ... start aiohttp server
-    self._mark_connected()
-
-async def _handle_callback(self, request):
-    event = self._build_event(await request.text())
-    await self._message_queue.put(event)
-    return web.Response(text="success")  # Acknowledge immediately
-```
-
-For platforms with tight response deadlines (e.g., WeCom's 5-second limit), always acknowledge immediately and deliver the agent's reply proactively via API later. Agent sessions run 3–30 minutes — inline replies within a callback response window are not feasible.
-
-### Token Locks
-
-If the adapter holds a persistent connection with a unique credential, add a scoped lock to prevent two profiles from using the same credential:
-
-```python
-from gateway.status import acquire_scoped_lock, release_scoped_lock
-
-async def connect(self, *, is_reconnect: bool = False):
-    acquired, _existing = acquire_scoped_lock("newplat", self._token)
-    if not acquired:
-        logger.error("Token already in use by another profile")
-        return False
-    # ... connect
-
-async def disconnect(self):
-    release_scoped_lock("newplat", self._token)
-```
-
-## Reference Implementations
-
-| Adapter | Pattern | Complexity | Good reference for |
-|---------|---------|------------|-------------------|
-| `bluebubbles.py` | REST + webhook | Medium | Simple REST API integration |
-| `weixin.py` | Long-poll + CDN | High | Media handling, encryption |
-| `plugins/platforms/wecom/callback_adapter.py` | Callback/webhook | Medium | HTTP server, AES crypto, multi-app |
-| `plugins/platforms/irc/adapter.py` | Long-poll + IRC protocol | High | Full-featured plugin adapter with scoped token lock |