Skip to content

fix(cli): preserve positional prompt arguments#211

Open
caroline-li-studio wants to merge 1 commit into
lessweb:mainfrom
caroline-li-studio:codex/positional-prompt
Open

fix(cli): preserve positional prompt arguments#211
caroline-li-studio wants to merge 1 commit into
lessweb:mainfrom
caroline-li-studio:codex/positional-prompt

Conversation

@caroline-li-studio

Copy link
Copy Markdown

Summary

  • preserve positional [query..] arguments as the initial prompt
  • keep --prompt/-p behavior unchanged while covering positional conflicts
  • add CLI argument tests for positional prompts and resume + positional prompt usage

Tests

  • node --import tsx --test packages/cli/src/tests/cli-args.test.ts
  • npm run format:check -- packages/cli/src/cli-args.ts packages/cli/src/tests/cli-args.test.ts
  • npm run typecheck --workspace @vegamo/deepcode-cli
  • npm run test --workspace @vegamo/deepcode-cli -- --runInBand
  • npm run build
  • node packages/cli/dist/cli.js --version
  • node packages/cli/dist/cli.js --help

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.

1 participant