Skip to content

Fix ask-codex shell quoting guidance#39

Merged
SihaoLiu merged 1 commit intoPolyArch:devfrom
dzwduan:fix/ask-codex-shell-quoting
Mar 8, 2026
Merged

Fix ask-codex shell quoting guidance#39
SihaoLiu merged 1 commit intoPolyArch:devfrom
dzwduan:fix/ask-codex-shell-quoting

Conversation

@dzwduan
Copy link
Copy Markdown
Contributor

@dzwduan dzwduan commented Mar 8, 2026

Summary

  • document safe ask-codex invocation for free-form prompts containing shell metacharacters
  • warn against bare $ARGUMENTS expansion in the ask-codex skill
  • add regression tests covering the skill guidance

Test Plan

  • bash tests/run-all-tests.sh

Copy link
Copy Markdown
Contributor

@SihaoLiu SihaoLiu left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@SihaoLiu SihaoLiu merged commit 813aa76 into PolyArch:dev Mar 8, 2026
4 checks passed
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