Skip to content

fallback: queue pure media message#389

Merged
xieyxclack merged 1 commit intoagentscope-ai:mainfrom
rayrayraykk:weirui/dev/fix_dingtalk_redundant
Mar 2, 2026
Merged

fallback: queue pure media message#389
xieyxclack merged 1 commit intoagentscope-ai:mainfrom
rayrayraykk:weirui/dev/fix_dingtalk_redundant

Conversation

@rayrayraykk
Copy link
Copy Markdown
Member

Description

[Describe what this PR does and why]

Related Issue: Fixes #(issue_number) or Relates to #(issue_number)

Security Considerations: [If applicable, e.g. channel auth, env/config handling]

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring

Component(s) Affected

  • Core / Backend (app, agents, config, providers, utils, local_models)
  • Console (frontend web UI)
  • Channels (DingTalk, Feishu, QQ, Discord, iMessage, etc.)
  • Skills
  • CLI
  • Documentation (website)
  • Tests
  • CI/CD
  • Scripts / Deploy

Checklist

  • Pre-commit hooks pass (pre-commit run --all-files or CI)
  • Tests pass locally (pytest or as relevant)
  • Documentation updated (if needed)
  • Ready for review

Testing

[How to test these changes]

Additional Notes

[Optional: any other context]

Copilot AI review requested due to automatic review settings March 2, 2026 14:25
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts the channel “no-text debounce” behavior so that only non-empty TEXT and REFUSAL content counts as “has text,” changing how media-only messages are handled during request consumption.

Changes:

  • Update _content_has_text docstring to reflect a stricter definition of “has text.”
  • Remove treating media/file-only content as “actionable text” for the no-text debounce path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/copaw/app/channels/base.py
Comment thread src/copaw/app/channels/base.py
@xieyxclack xieyxclack merged commit ae50ec6 into agentscope-ai:main Mar 2, 2026
7 checks passed
tudan110 pushed a commit to tudan110/QwenPaw that referenced this pull request Apr 4, 2026
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.

3 participants