APEX/BrowserUse_and_ComputerUse_skills
5
0
Fork 0
Code Issues 6 Pull requests 1 Projects 1 Releases Packages Wiki Activity Actions

fix: whatsapp reply_prefix config.yaml bridging was dead code (#1923)

Browse source 
The whatsapp reply_prefix bridging referenced config.platforms before
the config object was constructed, making it a silent NameError caught
by except Exception: pass.

Fix: fold reply_prefix into the per-platform bridging loop (introduced
in #1919) which correctly writes to gw_data dict pre-construction.
Removes the broken standalone whatsapp bridging block.

Co-authored-by: Test <test@test.com>
This commit is contained in:
Teknium 2026-03-18 04:18:33 -07:00 • committed by GitHub
parent 23e5e8dde9
commit 2f80bd9f87
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 12 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

25
gateway/config.py
View file

@ -451,7 +451,7 @@ def load_gateway_config() -> GatewayConfig:
"pair", "pair",
) )
# Bridge per-platform unauthorized_dm_behavior from config.yaml # Bridge per-platform settings from config.yaml into gw_data
platforms_data = gw_data.setdefault("platforms", {}) platforms_data = gw_data.setdefault("platforms", {})
if not isinstance(platforms_data, dict): if not isinstance(platforms_data, dict):
platforms_data = {} platforms_data = {}
@ -462,7 +462,16 @@ def load_gateway_config() -> GatewayConfig:
platform_cfg = yaml_cfg.get(plat.value) platform_cfg = yaml_cfg.get(plat.value)
if not isinstance(platform_cfg, dict): if not isinstance(platform_cfg, dict):
continue continue
if "unauthorized_dm_behavior" not in platform_cfg: # Collect bridgeable keys from this platform section
bridged = {}
if "unauthorized_dm_behavior" in platform_cfg:
bridged["unauthorized_dm_behavior"] = _normalize_unauthorized_dm_behavior(
platform_cfg.get("unauthorized_dm_behavior"),
gw_data.get("unauthorized_dm_behavior", "pair"),
)
if "reply_prefix" in platform_cfg:
bridged["reply_prefix"] = platform_cfg["reply_prefix"]
if not bridged:
continue continue
plat_data = platforms_data.setdefault(plat.value, {}) plat_data = platforms_data.setdefault(plat.value, {})
if not isinstance(plat_data, dict): if not isinstance(plat_data, dict):
@ -472,10 +481,7 @@ def load_gateway_config() -> GatewayConfig:
if not isinstance(extra, dict): if not isinstance(extra, dict):
extra = {} extra = {}
plat_data["extra"] = extra plat_data["extra"] = extra
extra["unauthorized_dm_behavior"] = _normalize_unauthorized_dm_behavior( extra.update(bridged)
platform_cfg.get("unauthorized_dm_behavior"),
gw_data.get("unauthorized_dm_behavior", "pair"),
)
# Discord settings → env vars (env vars take precedence) # Discord settings → env vars (env vars take precedence)
discord_cfg = yaml_cfg.get("discord", {}) discord_cfg = yaml_cfg.get("discord", {})
@ -489,13 +495,6 @@ def load_gateway_config() -> GatewayConfig:
os.environ["DISCORD_FREE_RESPONSE_CHANNELS"] = str(frc) os.environ["DISCORD_FREE_RESPONSE_CHANNELS"] = str(frc)
if "auto_thread" in discord_cfg and not os.getenv("DISCORD_AUTO_THREAD"): if "auto_thread" in discord_cfg and not os.getenv("DISCORD_AUTO_THREAD"):
os.environ["DISCORD_AUTO_THREAD"] = str(discord_cfg["auto_thread"]).lower() os.environ["DISCORD_AUTO_THREAD"] = str(discord_cfg["auto_thread"]).lower()
# Bridge whatsapp settings from config.yaml into platform config
whatsapp_cfg = yaml_cfg.get("whatsapp", {})
if isinstance(whatsapp_cfg, dict) and "reply_prefix" in whatsapp_cfg:
if Platform.WHATSAPP not in config.platforms:
config.platforms[Platform.WHATSAPP] = PlatformConfig()
config.platforms[Platform.WHATSAPP].extra["reply_prefix"] = whatsapp_cfg["reply_prefix"]
except Exception: except Exception:
pass pass