Skip to content

feat: support context length shorthand#60

Closed
puneetdixit200 wants to merge 1 commit into
Andyyyy64:mainfrom
puneetdixit200:codex/context-length-shorthand
Closed

feat: support context length shorthand#60
puneetdixit200 wants to merge 1 commit into
Andyyyy64:mainfrom
puneetdixit200:codex/context-length-shorthand

Conversation

@puneetdixit200

Copy link
Copy Markdown

Summary

  • add parsing for --context-length values like 64k and 128k
  • apply the parser to ranking, plan, upgrade, and run commands
  • document shorthand usage in the CLI reference

Closes #59.

Verification

  • .venv/bin/python -m pytest tests/test_cli.py -q -> 32 passed
  • .venv/bin/python -m pytest -q -> 195 passed

AI-assisted work

This patch was prepared with AI assistance and manually reviewed. The targeted tests were written first, observed failing before implementation, then the full test suite was run locally.

@puneetdixit200
puneetdixit200 marked this pull request as ready for review May 20, 2026 05:56
@Andyyyy64

Copy link
Copy Markdown
Owner

Thanks for working on this.

I’m going to close this one in favor of #64, which covers the same issue and keeps the parsing logic in a reusable CLI parameter type.

I do appreciate the docs updates here. I’ll make sure the final version keeps the CLI docs covered as well.

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.

Support context length shorthand

2 participants