teknium1
2dfd831d3b
The salvaged flag was spelled --profile, which collides with the global -p/--profile that hermes_cli.main scans BEFORE argparse: `hermes webhook subscribe x --profile compta` would switch this CLI process to compta's HERMES_HOME and write the subscription into compta's webhook_subscriptions.json — a file the default gateway's webhook adapter never reads — while the route still lacked the profile key. #109020 special-cased the scanner for the webhook subcommand; naming the flag --route-profile removes the ambiguity without touching _scan_profile_flag: -p picks the gateway whose subscriptions file is written, --route-profile picks which /p/<profile>/ prefix may hit the route. Docs: cli-commands reference row, multi-profile-gateways webhook section, the route `profile` field. Builds on #109020 (fangliquanflq). Fixes #109016.
57 lines
2.9 KiB
Python
57 lines
2.9 KiB
Python
"""``hermes webhook`` subcommand parser."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import Callable
|
|
|
|
|
|
def build_webhook_parser(subparsers, *, cmd_webhook: Callable) -> None:
|
|
"""Attach the ``webhook`` subcommand to ``subparsers``."""
|
|
webhook_parser = subparsers.add_parser(
|
|
"webhook", help="Manage dynamic webhook subscriptions",
|
|
description="Create, list, and remove webhook subscriptions for event-driven agent activation",
|
|
)
|
|
webhook_subparsers = webhook_parser.add_subparsers(dest="webhook_action")
|
|
|
|
wh_sub = webhook_subparsers.add_parser(
|
|
"subscribe", aliases=["add"], help="Create a webhook subscription")
|
|
wh_sub.add_argument("name", help="Route name (used in URL: /webhooks/<name>)")
|
|
wh_sub.add_argument(
|
|
"--prompt", default="", help="Prompt template with {dot.notation} payload refs")
|
|
wh_sub.add_argument("--events", default="", help="Comma-separated event types to accept")
|
|
wh_sub.add_argument("--description", default="", help="What this subscription does")
|
|
wh_sub.add_argument("--skills", default="", help="Comma-separated skill names to load")
|
|
wh_sub.add_argument(
|
|
"--deliver", default="log", help="Delivery target: log, telegram, discord, slack, etc.")
|
|
wh_sub.add_argument(
|
|
"--deliver-chat-id", default="", help="Target chat ID for cross-platform delivery")
|
|
wh_sub.add_argument("--secret", default="", help="HMAC secret (auto-generated if omitted)")
|
|
wh_sub.add_argument(
|
|
"--route-profile", dest="route_profile", default=None, metavar="PROFILE",
|
|
help="Bind the route to a multiplexed profile: only POSTs to /p/PROFILE/webhooks/<name> "
|
|
"are accepted and the agent runs as that profile (default: default; kept on update). "
|
|
"Distinct from the global -p/--profile, which picks the gateway whose subscriptions "
|
|
"file is written.")
|
|
wh_sub.add_argument(
|
|
"--deliver-only", action="store_true",
|
|
help="Skip the agent — deliver the rendered prompt directly as the "
|
|
"message. Zero LLM cost. Requires --deliver to be a real target "
|
|
"(not 'log').")
|
|
wh_sub.add_argument(
|
|
"--script", default="",
|
|
help="Filter/transform script under ~/.hermes/scripts/. The route "
|
|
"payload is passed as JSON on stdin; empty stdout, [SILENT], or a "
|
|
"nonzero exit code ignores the webhook.")
|
|
|
|
webhook_subparsers.add_parser("list", aliases=["ls"], help="List all dynamic subscriptions")
|
|
|
|
wh_rm = webhook_subparsers.add_parser("remove", aliases=["rm"], help="Remove a subscription")
|
|
wh_rm.add_argument("name", help="Subscription name to remove")
|
|
|
|
wh_test = webhook_subparsers.add_parser("test", help="Send a test POST to a webhook route")
|
|
wh_test.add_argument("name", help="Subscription name to test")
|
|
wh_test.add_argument(
|
|
"--payload", default="", help="JSON payload to send (default: test payload)")
|
|
|
|
webhook_parser.set_defaults(func=cmd_webhook)
|