Skip to content

feat(tool): execute_shell_command encoding on Windows#1103

Merged
xieyxclack merged 1 commit intoagentscope-ai:mainfrom
snai1557:fix-execute-shell-command-encoding
Mar 10, 2026
Merged

feat(tool): execute_shell_command encoding on Windows#1103
xieyxclack merged 1 commit intoagentscope-ai:mainfrom
snai1557:fix-execute-shell-command-encoding

Conversation

@snai1557
Copy link
Copy Markdown
Contributor

Description

#979

之前合并的代码漏了一处没有处理.

Type of Change

  • [ x ] Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring

Component(s) Affected

  • [ x ] 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

  • [ x ] I ran pre-commit run --all-files locally and it passes
  • If pre-commit auto-fixed files, I committed those changes and reran checks
  • I ran tests locally (pytest or as relevant) and they pass
  • Documentation updated (if needed)
  • [ x ] Ready for review

Testing

[How to test these changes]

Local Verification Evidence

pre-commit run --all-files
# paste summary result

pytest
# paste summary result

Additional Notes

[Optional: any other context]

@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
Member

@xieyxclack xieyxclack left a comment

Choose a reason for hiding this comment

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

LGTM

@xieyxclack xieyxclack merged commit 7256544 into agentscope-ai:main Mar 10, 2026
4 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.

2 participants