Skip to content

fix(tool): enforce prompt Fix #200#201

Open
xiaobaicai66695 wants to merge 1 commit intocloudwego:mainfrom
xiaobaicai66695:fix/issue-200
Open

fix(tool): enforce prompt Fix #200#201
xiaobaicai66695 wants to merge 1 commit intocloudwego:mainfrom
xiaobaicai66695:fix/issue-200

Conversation

@xiaobaicai66695
Copy link
Copy Markdown

What type of PR is this?

fix

Check the PR title.

  • This PR title match the format: (optional scope):
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
Fix issue #200: In the human-in-the-loop/1_approval example, the Qwen model sometimes passed phone_number as a number instead of a string. This PR strengthens the agent's Instruction prompt to explicitly require string type for all fields, provide an example for phone number, and forbid number type. After the change, the Qwen model behaves correctly.

zh(optional): 修复 #200:在 human-in-the-loop/1_approval 示例中,Qwen 模型有时将 phone_number 作为数字而非字符串传递。此 PR 强化了 Agent 的 Instruction 提示词,明确要求所有字段为字符串类型,为电话号码提供示例,并禁止使用数字类型。修改后 Qwen 模型行为正确。

(Optional) Which issue(s) this PR fixes:

Fixes #200

(optional) The PR that updates user documentation:

N/A

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 9, 2026

CLA assistant check
All committers have signed the CLA.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

在adk/human-in-the-loop/1_approval样例中,调用tool时,输入参数phone_number可能为非string类型

2 participants