fix(copilot): align dry-run prompt wording and tighten test assertion

- Align guide heading to "create -> dry-run -> fix" matching supplement
- Align error pattern names between guide and supplement to canonical form
- Drop loose "max " fallback in test assertion for precision
This commit is contained in:
Zamil Majdy
2026-03-27 01:00:48 +07:00
parent c564ac7277
commit e7ca07f4bf
2 changed files with 5 additions and 5 deletions

View File

@@ -66,7 +66,7 @@ class TestToolDescriptionsDryRunLoop:
dry_run_desc = params["properties"]["dry_run"]["description"]
assert "create_agent" in dry_run_desc or "edit_agent" in dry_run_desc
assert "wait_for_result" in dry_run_desc
assert "3 iterations" in dry_run_desc or "max " in dry_run_desc
assert "3 iterations" in dry_run_desc
def test_get_agent_building_guide_mentions_workflow(self):
tool = TOOL_REGISTRY["get_agent_building_guide"]

View File

@@ -236,7 +236,7 @@ call in a loop until the task is complete:
Regular blocks work exactly like sub-agents as tools — wire each input
field from `source_name: "tools"` on the Orchestrator side.
### REQUIRED: Dry-Run Verification Loop (create -> test -> fix)
### REQUIRED: Dry-Run Verification Loop (create -> dry-run -> fix)
After creating or editing an agent, you MUST dry-run it before telling the
user the agent is ready. NEVER skip this step.
@@ -252,15 +252,15 @@ user the agent is ready. NEVER skip this step.
`wait_for_result` returns only a summary, call
`view_agent_output(execution_id=..., show_execution_details=True)` to
see the full node-by-node execution trace. Look for:
- **Errors or failed nodes** — a node raised an exception or returned an
- **Errors / failed nodes** — a node raised an exception or returned an
error status. Common causes: wrong `source_name`/`sink_name` in links,
missing `input_default` values, or referencing a nonexistent block output.
- **Null / empty values** — data did not flow through a link. Verify that
- **Null / empty outputs** — data did not flow through a link. Verify that
`source_name` and `sink_name` match the block schemas exactly (case-
sensitive, including nested `_#_` notation).
- **Nodes that never executed** — the node was not reached. Likely a
missing or broken link from an upstream node.
- **Unexpected data format** — data arrived but in the wrong type or
- **Unexpected values** — data arrived but in the wrong type or
structure. Check type compatibility between linked ports.
4. **Fix**: If any issues are found, call `edit_agent` with the corrected
agent JSON, then go back to step 2.