fix(review): add platform context to renders_native_tables() and standalone rich_messages check

- Add platform parameter to ChatAdapter::renders_native_tables() trait method
  so unified adapter can make per-platform decisions (fixes regression where
  all platforms would skip table wrapping when telegram_rich_messages=true)
- UnifiedGatewayAdapter now checks platform == "telegram" && rich_messages
- GatewayAdapter (standalone) now checks both platform_name and a new
  telegram_rich_messages config field (added to GatewayConfig/GatewayParams)
- Update all call sites (AdapterRouter, Slack, tests) to pass platform arg

Fixes: unified mode multi-platform regression (F1)
Fixes: standalone mode Rich Messages assumption (F2)

Author: chaodu-agent

crates/openab-core/src/adapter.rs

@@ -407,7 +407,10 @@ pub trait ChatAdapter: Send + Sync + 'static {
     /// lets the platform render the raw Markdown table itself.
     /// Default: `false` (keep converting). Overridden by Slack (Block Kit
     /// `markdown` blocks / `markdown_text` stream chunks render tables natively).
-    fn renders_native_tables(&self) -> bool {
+    /// The `platform` parameter allows shared adapters (e.g. UnifiedGatewayAdapter)
+    /// to make per-platform decisions.
+    fn renders_native_tables(&self, platform: &str) -> bool {
+        let _ = platform;
         false
     }
 
@@ -697,7 +700,7 @@ impl AdapterRouter {
         // Platforms that render Markdown tables natively (e.g. Slack Block Kit
         // `markdown` blocks / `markdown_text` stream chunks) skip the
         // table→code/bullets pre-pass so the raw table renders natively.
-        let table_mode = if adapter.renders_native_tables() {
+        let table_mode = if adapter.renders_native_tables(&thread_channel.platform) {
             TableMode::Off
         } else {
             self.table_mode
@@ -1794,7 +1797,7 @@ mod tests {
         assert!(!adapter.use_streaming(false));
         // renders_native_tables defaults to false: platforms that don't override
         // it keep the table→code/bullets conversion (e.g. Discord, Gateway).
-        assert!(!adapter.renders_native_tables());
+        assert!(!adapter.renders_native_tables("discord"));
     }
 
     #[test]

crates/openab-core/src/config.rs

@@ -567,6 +567,11 @@ pub struct GatewayConfig {
     /// Show "…" placeholder at streaming start. Default: true. Set false for platforms using drafts.
     #[serde(default = "default_true")]
     pub streaming_placeholder: bool,
+    /// Whether the connected gateway renders tables natively (e.g. Telegram Rich Messages).
+    /// Default: true (matches Telegram default). Set false if Rich Messages is disabled
+    /// on the gateway daemon to preserve table code-block wrapping.
+    #[serde(default = "default_true")]
+    pub telegram_rich_messages: bool,
     /// Message dispatch mode. Default: per-message.
     #[serde(default)]
     pub message_processing_mode: MessageProcessingMode,

crates/openab-core/src/gateway.rs

@@ -233,6 +233,7 @@ pub struct GatewayAdapter {
     platform_name: &'static str,
     streaming: bool,
     streaming_placeholder: bool,
+    telegram_rich_messages: bool,
 }
 
 impl GatewayAdapter {
@@ -242,13 +243,15 @@ impl GatewayAdapter {
         platform_name: &'static str,
         streaming: bool,
         streaming_placeholder: bool,
+        telegram_rich_messages: bool,
     ) -> Self {
         Self {
             ws_tx,
             pending,
             platform_name,
             streaming,
             streaming_placeholder,
+            telegram_rich_messages,
         }
     }
 
@@ -731,10 +734,11 @@ impl ChatAdapter for GatewayAdapter {
         self.streaming_placeholder
     }
 
-    fn renders_native_tables(&self) -> bool {
+    fn renders_native_tables(&self, _platform: &str) -> bool {
         // Telegram renders markdown tables natively via Rich Messages;
-        // skip the table→code-block pre-pass for that platform.
-        self.platform_name == "telegram"
+        // skip the table→code-block pre-pass for that platform only when
+        // Rich Messages is confirmed enabled.
+        self.platform_name == "telegram" && self.telegram_rich_messages
     }
 }
 
@@ -754,6 +758,7 @@ pub struct GatewayParams {
     pub trusted_bot_ids: Vec<String>,
     pub streaming: bool,
     pub streaming_placeholder: bool,
+    pub telegram_rich_messages: bool,
     pub stt: crate::config::SttConfig,
 }
 
@@ -788,6 +793,7 @@ pub async fn run_gateway_adapter(
         params.streaming
     };
     let streaming_placeholder = params.streaming_placeholder;
+    let telegram_rich_messages = params.telegram_rich_messages;
     let stt_config = params.stt;
 
     let connect_url = match &params.token {
@@ -838,6 +844,7 @@ pub async fn run_gateway_adapter(
             platform,
             streaming,
             streaming_placeholder,
+            telegram_rich_messages,
         ));
         let slash_ws_tx = ws_tx.clone(); // for fire-and-forget slash command responses
         let mut tasks: tokio::task::JoinSet<()> = tokio::task::JoinSet::new();

crates/openab-core/src/slack.rs

@@ -544,7 +544,7 @@ impl ChatAdapter for SlackAdapter {
         self.streaming && !other_bot_present
     }
 
-    fn renders_native_tables(&self) -> bool {
+    fn renders_native_tables(&self, _platform: &str) -> bool {
         true
     }
 
@@ -2307,7 +2307,7 @@ mod tests {
     fn slack_renders_native_tables() {
         let ttl = std::time::Duration::from_secs(300);
         let adapter = SlackAdapter::new("xoxb-test".into(), ttl, AllowBots::Mentions, true, crate::multibot_cache::MultibotCache::load("/dev/null".into()), true);
-        assert!(adapter.renders_native_tables());
+        assert!(adapter.renders_native_tables("slack"));
     }
 }
 

src/main.rs

@@ -587,6 +587,7 @@ async fn main() -> anyhow::Result<()> {
             trusted_bot_ids: gw_cfg.trusted_bot_ids,
             streaming: gw_cfg.streaming,
             streaming_placeholder: gw_cfg.streaming_placeholder,
+            telegram_rich_messages: gw_cfg.telegram_rich_messages,
             stt: cfg.stt.clone(),
         };
         let gw_router = router.clone();

src/unified_adapter.rs

@@ -218,9 +218,10 @@ impl ChatAdapter for UnifiedGatewayAdapter {
         false
     }
 
-    fn renders_native_tables(&self) -> bool {
+    fn renders_native_tables(&self, platform: &str) -> bool {
         // Telegram Rich Messages render markdown tables natively — skip the
         // table→code-block pre-pass so tables display with proper formatting.
-        self.gw_state.telegram_rich_messages
+        // Only applies to Telegram; other platforms in unified mode keep wrapping.
+        platform == "telegram" && self.gw_state.telegram_rich_messages
     }
 }