You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up items from PR #1171 (unified webhook runtime)
Tracked items from group review — non-blocking improvements for the unified webhook path.
Tasks
Add GATEWAY_ALLOW_BOT_MESSAGES / GATEWAY_TRUSTED_BOT_IDS env — process_gateway_event() currently drops all bot messages; multi-agent scenarios need trusted bot passthrough.
Wire Google Chat in unified mode — Currently google_chat: None with TODO; needs SA key parsing + JWT verifier initialization.
Deduplicate event processing logic — process_gateway_event() and run_gateway_adapter() share filtering/dispatch logic that should be extracted into a shared helper.
Document production-must-set env vars — GATEWAY_ALLOW_ALL_CHANNELS, GATEWAY_ALLOW_ALL_USERS default fail-open; flag in Helm/deploy docs.
Add user-facing fallback when create_thread fails — Server-side tracing::warn added; user-facing message removed due to ordering issues on Telegram/LINE (warning appeared before reply). Needs a deferred-message approach to show after reply.
Log when audio attachment path/data both empty — Currently silent skip with no trace.
Follow-up items from PR #1171 (unified webhook runtime)
Tracked items from group review — non-blocking improvements for the unified webhook path.
Tasks
GATEWAY_ALLOW_BOT_MESSAGES/GATEWAY_TRUSTED_BOT_IDSenv —process_gateway_event()currently drops all bot messages; multi-agent scenarios need trusted bot passthrough.google_chat: Nonewith TODO; needs SA key parsing + JWT verifier initialization.process_gateway_event()andrun_gateway_adapter()share filtering/dispatch logic that should be extracted into a shared helper.AppState::from_env()factory — Avoid unified/standalone config divergence by sharing initialization.GATEWAY_ALLOW_ALL_CHANNELS,GATEWAY_ALLOW_ALL_USERSdefault fail-open; flag in Helm/deploy docs.create_threadfails — Server-sidetracing::warnadded; user-facing message removed due to ordering issues on Telegram/LINE (warning appeared before reply). Needs a deferred-message approach to show after reply.Context
cc @pahudnet