Skip to content

feat: add shell completion support#40

Open
mvanhorn wants to merge 1 commit intoPolymarket:mainfrom
mvanhorn:osc/34-add-shell-completion
Open

feat: add shell completion support#40
mvanhorn wants to merge 1 commit intoPolymarket:mainfrom
mvanhorn:osc/34-add-shell-completion

Conversation

@mvanhorn
Copy link

@mvanhorn mvanhorn commented Mar 10, 2026

Summary

Adds a polymarket completions <shell> subcommand that generates shell completion scripts using clap_complete.

Fixes #34

Usage

# Bash
polymarket completions bash >> ~/.bashrc

# Zsh
polymarket completions zsh >> ~/.zshrc

# Fish
polymarket completions fish > ~/.config/fish/completions/polymarket.fish

Supported shells: bash, zsh, fish, elvish, powershell.

Changes

File Change
Cargo.toml Add clap_complete = "4" dependency
src/main.rs Add Completions subcommand with Shell argument and clap_complete::generate handler

Test plan

  • cargo fmt --check passes
  • cargo clippy -- -D warnings passes
  • cargo test — all 131 tests pass (82 unit + 49 integration)
  • polymarket completions bash outputs valid completion script
  • polymarket completions zsh outputs valid completion script

This contribution was developed with AI assistance (Claude Code).


Note

Low Risk
Adds an optional CLI subcommand and a dependency; no changes to trading/auth flows or data handling.

Overview
Adds a new polymarket completions <shell> subcommand that prints shell completion scripts to stdout via clap_complete::generate.

Wires in the new clap_complete dependency (and lockfile updates) and updates CLI parsing to expose Shell as an argument for completion generation.

Written by Cursor Bugbot for commit cc5cbfb. This will update automatically on new commits. Configure here.

Add `polymarket completions <shell>` subcommand that generates
shell completion scripts for bash, zsh, fish, elvish, and
PowerShell using clap_complete.

Usage:
  polymarket completions bash >> ~/.bashrc
  polymarket completions zsh >> ~/.zshrc
  polymarket completions fish > ~/.config/fish/completions/polymarket.fish

Fixes Polymarket#34

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

Add auto complete for sub commands

1 participant