Skip to content

[codex] fix(cli): allow dash-leading positional adapter args#1223

Draft
Yinchaochen wants to merge 1 commit intojackwener:mainfrom
Yinchaochen:fix/dash-leading-positional-args
Draft

[codex] fix(cli): allow dash-leading positional adapter args#1223
Yinchaochen wants to merge 1 commit intojackwener:mainfrom
Yinchaochen:fix/dash-leading-positional-args

Conversation

@Yinchaochen
Copy link
Copy Markdown

Summary

  • allow dynamic adapter commands with positional args to accept dash-leading values
  • add a regression test for opencli boss detail -123456abdc -f json

Root Cause

Commander treated values like -123456abdc as unknown options before OpenCLI's adapter argument preparation ran, so commands with positional IDs could fail before adapter-level validation had a chance to handle them.

Impact

Commands such as opencli boss detail -123456abdc -f json now parse the positional security-id correctly instead of failing with unknown option.

Validation

  • npm run typecheck
  • npm test -- --runInBand src/commanderAdapter.test.ts
  • npm run build
  • npm test (fails on this Windows environment due pre-existing unrelated symlink/path separator assertions in src/plugin.test.ts, src/engine.test.ts, and several adapter download/upload tests)

Fixes #1160

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.

[Bug]: BOSS直聘 detail 命令无法处理以"-"开头的 security-id 参数

1 participant