-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Problem
In scripts/run_tasks_in_agents_if_exists.py at line 58, the extra argument is declared with nargs="*". This means values starting with - (e.g. --fix, -k, --) will be parsed as options by argparse and cause an "unrecognized arguments" error instead of being forwarded to Poe as intended.
File and Location
- File:
scripts/run_tasks_in_agents_if_exists.py - Line: 58
Suggested Fix
If this script is intended to pass through arbitrary task args, use argparse.REMAINDER (or parse_known_args) for the forwarded arguments so flags are accepted:
parser.add_argument("extra", nargs=argparse.REMAINDER, ...)or
args, extra = parser.parse_known_args()Original Review Comment
extrais declared withnargs="*", so values starting with-(e.g.--fix,-k,--) will be parsed as options and cause an "unrecognized arguments" error instead of being forwarded to Poe. If this script is intended to pass through arbitrary task args, useargparse.REMAINDER(orparse_known_args) for the forwarded arguments so flags are accepted.
References
- Related PR: feat: add agentic workflows and security posture #51
- Review comment: feat: add agentic workflows and security posture #51 (comment)
Generated by PR Review Comment — Create Issue for issue #51