Skip to content

fix: collapse intermediate output to summary#310

Closed
ModerRAS wants to merge 1 commit intomasterfrom
fix/collapse-summary-display
Closed

fix: collapse intermediate output to summary#310
ModerRAS wants to merge 1 commit intomasterfrom
fix/collapse-summary-display

Conversation

@ModerRAS
Copy link
Copy Markdown
Owner

@ModerRAS ModerRAS commented Apr 23, 2026

Summary

  • replace the previous hidden/spoiler-style collapse with a compact one-line summary of intermediate iterations
  • keep unsupported Telegram blockquotes out of the generated HTML
  • update regression tests to cover compact collapsed output and normal blockquotes

Validation

  • dotnet build TelegramSearchBot.sln -c Release
  • dotnet test TelegramSearchBot.sln -c Release --no-build

Summary by CodeRabbit

  • Bug Fixes
    • Improved display of collapsed LLM intermediate iterations: now shows a compact one-line summary indicating the number of tool calls and recent activity, replacing the previous expandable format.
    • Enhanced HTML conversion for blockquote elements with consistent rendering.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 49613b60-78dc-4c94-a12f-b4a3a3825e4c

📥 Commits

Reviewing files that changed from the base of the PR and between cfc0919 and ac62057.

📒 Files selected for processing (2)
  • TelegramSearchBot.Test/Helper/MessageFormatHelperTests.cs
  • TelegramSearchBot/Helper/MessageFormatHelper.cs

📝 Walkthrough

Walkthrough

The changes refactor how collapsed LLM intermediate iterations are represented in Telegram messages. Instead of emitting collapsible spoiler blocks, the system now generates a single-line compact summary indicating the number of collapsed tool calls, replacing the verbose container with concise text. Related tests were updated accordingly.

Changes

Cohort / File(s) Summary
Test Updates
TelegramSearchBot.Test/Helper/MessageFormatHelperTests.cs
Renamed test method from EmitsSpoiler to EmitsCompactSummary and updated assertions to verify compact summary format (已折叠前面的中间过程和 1 次工具调用) and absence of <tg-spoiler> tags instead of checking for spoiler blocks.
HTML Conversion Logic
TelegramSearchBot/Helper/MessageFormatHelper.cs
Removed spoiler block output paths, unified blockquote and tg-expandable-blockquote handling as quoted blocks, and replaced collapsible container emission with single-line summary generation containing tool call count and recent tool text.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Spoilers shrink to summaries so neat,
Collapsed iterations, one-line tweet,
No more expandable mystery blocks,
Just compact text that gently talks,
The chat flows cleaner down the street!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/collapse-summary-display

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ModerRAS ModerRAS closed this Apr 23, 2026
@ModerRAS ModerRAS deleted the fix/collapse-summary-display branch April 23, 2026 13:21
@ModerRAS
Copy link
Copy Markdown
Owner Author

中间内容是隐藏不是清理

@github-actions
Copy link
Copy Markdown
Contributor

🔍 PR检查报告

📋 检查概览

🧪 测试结果

平台 状态 详情
Ubuntu 🟢 成功 测试通过,产物已上传
Windows 🟢 成功 测试通过,产物已上传

📊 代码质量

  • ✅ 代码格式化检查
  • ✅ 安全漏洞扫描
  • ✅ 依赖包分析
  • ✅ 代码覆盖率收集

📁 测试产物

  • 测试结果 artifacts 数量: 2
  • 代码覆盖率已上传到Codecov

🔗 相关链接


此报告由GitHub Actions自动生成

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant