Skip to content

CLI improvements: multiple clinics, -S for list-filters, parseable output - #33

Open
andrewife wants to merge 2 commits into
rafsaf:mainfrom
andrewife:feat-cli-improvements
Open

andrewife wants to merge 2 commits into
rafsaf:mainfrom
andrewife:feat-cli-improvements

Conversation

@andrewife

Copy link
Copy Markdown

Three small quality-of-life improvements:

  1. find-appointment -c/--clinic accepts multiple IDs (same syntax as -s), passed as ClinicIds to the API. Useful when one location has separate clinic IDs — e.g. the general clinic and the dental clinic in the same building — and you want a single search/notification stream covering both.

  2. list-filters specialties and list-filters clinics accept -S/--slot-search-type and pass it to the filters endpoint (find_filters already had the parameter; it just wasn't exposed on the CLI).

  3. list-filters results are printed with print() instead of log.info(). RichHandler wraps long lines to the terminal width, which breaks scripts that parse the output (e.g. ... | grep '^[0-9]* - ' to build interactive pickers). Log records still go to the log file; the actual listing is now clean, one entry per line.

🤖 Generated with Claude Code

andrzejtomaszewski-tomtom and others added 2 commits July 5, 2026 21:33
…tput

- find-appointment -c/--clinic now accepts multiple IDs (like -s), passed
  as ClinicIds to the API; useful when one location has separate clinic
  IDs (e.g. general and dental in the same building)
- list-filters specialties/clinics accept -S/--slot-search-type and pass
  it to the filters endpoint (find_filters already supported it)
- list-filters results are printed with print() instead of log.info();
  RichHandler wraps long lines, which broke scripts parsing the output

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- find_appointments tests: clinic is now a list of IDs
- list-filters tests: find_filters receives slot_search_type, results go through print()

Co-Authored-By: Claude Fable 5 <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.

2 participants